#2437: tg.url() docstring does not match behavior, pagination problem
------------------------+---------------------------------------------------
 Reporter:  chrisz      |       Owner:                   
     Type:  defect      |      Status:  new              
 Priority:  high        |   Milestone:  2.1              
Component:  TurboGears  |     Version:  2.1a3            
 Severity:  normal      |    Keywords:  tg.url pagination
------------------------+---------------------------------------------------
 Just noticed that the docstring of the `tg.url()` function does not match
 with the actual behavior any more in TG 2.1.

 The docstring says "For backwards compatibility you can also pass in a
 params dictionary which is turned into url params, ..."

 However, it's the other way round now, **kwargs are supported for
 backward compatibility only (with a deprecation warning), and only
 passing a dict in params seems to be supported in the future. (Is this
 really what we want?)

 Second, the docstring says "In general `tg.url  is just a proxy for
 `pylons.url` which is in turn a proxy for routes `url_for` function."

 However, `tg.url` does not seem to resort to `pylons.url` any more.

 I'd like to fix that docstring, but I'm not sure whether these changes
 are intentional or not.

 Remark made by Jonathan on tg-trunk:
 This also seems to have broken the pagination system (when you show the
 pagination links, it throws an error 500).

 See also #2272 and #2371 for other issues with `tg.url()`.

-- 
Ticket URL: <http://trac.turbogears.org/ticket/2437>
TurboGears <http://www.turbogears.org/>
TurboGears front-to-back web development
-- 
You received this message because you are subscribed to the Google
Groups "TurboGears Tickets" group.
This group is read-only. No posting by normal members allowed.
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/turbogears-tickets?hl=en?hl=en

Reply via email to