Default IAbsoluteURL adapter adapts recursively using name rather than interface

Bug #338101 reported by Gary Poster
4
Affects Status Importance Assigned to Milestone
Zope 3
Fix Released
Undecided
Gary Poster

Bug Description

in zope.traversing.browser.absoluteurl.AbsoluteURL.__str__, the recursive call uses the view name spelling rather than an IAbsoluteURL spelling. That is,

        url = str(zope.component.getMultiAdapter((container, request),
                                                 name='absolute_url'))

should be

        url = str(zope.component.getMultiAdapter((container, request),
                                                 IAbsoluteURL))

Otherwise you have to register your IAbsoluteURL adapters for both spellings, which might be OK if you are following zope.app/ZMI conventions and you want the view available as 'absolute_url', but is unnecessary and annoying otherwise.

Gary Poster (gary)
Changed in zope3:
assignee: nobody → garyposter
Revision history for this message
Dan Korostelev (nadako) wrote :

Fixed in rev 97522

Changed in zope3:
status: New → Fix Committed
Revision history for this message
Dan Korostelev (nadako) wrote :

The fix was released in zope.traversing 3.5.3

Changed in zope3:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.