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.

General request to the TG developers: When modifying code, please don't
forget to add/adapt docstrings, comments and unit tests! Also, more
detailed commit messages would be helpful.

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

-- Christoph

--

You received this message because you are subscribed to the Google Groups 
"TurboGears Trunk" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/turbogears-trunk?hl=en.


Reply via email to