[tw.forms] #29: tw.forms update breaks divs

0 views
Skip to first unread message

Trac from toscawidgets.org

unread,
Jun 10, 2009, 11:25:17 AM6/10/09
to toscawidge...@groups.google.com
#29: tw.forms update breaks divs
--------------------+-------------------------------------------------------
Reporter: shon | Owner:
Type: defect | Status: new
Priority: blocker | Version: 0.9
Keywords: div |
--------------------+-------------------------------------------------------
I have updated tw.forms from 0.9.3 to 0.9.6 and suprise all <div> tags
generated by tw.forms are thrown back escaped like &lt;div&gt; . I did
not actually realized that this is due to update for a couple of hours.
Reverting back to old version solved problem for me.
I use tw.forms on TG2 platform.
Below is sample code to reproduce
{{{
class TicketForm(ListForm, twd.HidingTableForm):

submit_text = "Create Ticket"
show_errors = False

class fields(WidgetsList):
problemtext = Label(text='What describes your request best')
}}}

--
Ticket URL: <http://toscawidgets.org/trac/tw.forms/ticket/29>
tw.forms </trac/tw.forms>
tw.forms - Form building widgets

Trac from toscawidgets.org

unread,
Jun 12, 2009, 2:24:24 PM6/12/09
to toscawidge...@groups.google.com
#29: tw.forms update breaks divs
--------------------+-------------------------------------------------------
Reporter: shon | Owner:
Type: defect | Status: new
Priority: blocker | Version: 0.9
Keywords: div |
--------------------+-------------------------------------------------------

Comment(by shon):

BTW updating ToscaWidgets and tw.form packages to 0.9.6 is not a problem.

--
Ticket URL: <http://toscawidgets.org/trac/tw.forms/ticket/29#comment:1>

Trac from toscawidgets.org

unread,
Jun 12, 2009, 3:41:45 PM6/12/09
to toscawidge...@groups.google.com
#29: tw.forms update breaks divs
--------------------+-------------------------------------------------------
Reporter: shon | Owner:
Type: defect | Status: new
Priority: blocker | Version: 0.9
Keywords: div |
--------------------+-------------------------------------------------------

Comment(by chrisz):

I think this is the problem that I mentioned
[http://groups.google.com/group/toscawidgets-
discuss/browse_thread/thread/b0c62df293282441 here].

--
Ticket URL: <http://toscawidgets.org/trac/tw.forms/ticket/29#comment:2>

Trac from toscawidgets.org

unread,
Jun 16, 2009, 7:14:21 PM6/16/09
to toscawidge...@groups.google.com
#29: tw.forms update breaks divs
--------------------+-------------------------------------------------------
Reporter: shon | Owner:
Type: defect | Status: new
Priority: blocker | Version: 0.9
Keywords: div |
--------------------+-------------------------------------------------------

Comment(by anthonyt):

It's worth noting that #32 is caused by this same issue. It was introduced
in c9e7839c6b0b. I've patched it to the best of my understanding. I don't
know why these lines were commented out in the first place.

I've tested the change using tw.forms with Genshi in my TurboGears 2.0
app, and it looks like it works just fine.

--
Ticket URL: <http://toscawidgets.org/trac/tw.forms/ticket/29#comment:3>

Trac from toscawidgets.org

unread,
Jun 16, 2009, 7:14:32 PM6/16/09
to toscawidge...@groups.google.com
#29: [PATCH] tw.forms update breaks divs

--------------------+-------------------------------------------------------
Reporter: shon | Owner:
Type: defect | Status: new
Priority: blocker | Version: 0.9
Keywords: div |
--------------------+-------------------------------------------------------

--
Ticket URL: <http://toscawidgets.org/trac/tw.forms/ticket/29#comment:4>

Trac from toscawidgets.org

unread,
Jun 19, 2009, 7:31:59 AM6/19/09
to toscawidge...@groups.google.com
#29: [PATCH] tw.forms update breaks divs
----------------------+-----------------------------------------------------
Reporter: shon | Owner:
Type: defect | Status: closed
Priority: blocker | Version: 0.9
Resolution: fixed | Keywords: div
----------------------+-----------------------------------------------------
Changes (by alberto):

* status: new => closed
* resolution: => fixed


Comment:

Just released tw.forms and TW 0.9.7 solving the issue. Many thanks for the
patch although I finally decided to try to solve the issue more cleanly
upstream in TW hence didn't apply it.

Alberto

--
Ticket URL: <http://toscawidgets.org/trac/tw.forms/ticket/29#comment:5>

Reply all
Reply to author
Forward
0 new messages