_getContextName in Products.Five.browser.absoluteurl doesn't always return value

[sagblmi]

Zope2-2.12.3

In Products.Five.browser.absoluteurl the _getContextName(self, context) is not completed

Current implementation:

 def _getContextName(self, context):
        if getattr(context, 'getId', None) is not None:
            return context.getId()
        getattr(context, '__name__', None)

expected implementation;
 def _getContextName(self, context):
        if getattr(context, 'getId', None) is not None:
            return context.getId()
        return getattr(context, '__name__', None)

[Tres Seaver (tseaver)]

MJ has checked in a fix for this bug, but still owes us a test.

[Tres Seaver (tseaver)]

See:

 http://svn.zope.org/Zope/branches/2.12/src/Products/Five/browser/?rev=109473&view=rev

 http://svn.zope.org/Zope/trunk/src/Products/Five/browser/?rev=109474&view=rev

[Hanno Schlichting (hannosch)]

I'll let this slip through without a test.