Question about how remoteURL are handled in navigation

Today I found a strange bahavior that can easily understood looking at
this change:
https://github.com/plone/Products.CMFPlone/commit/be6b828870387259a731bf13c45150258efc48cb

I'm wondering why this change has been added.
For what I remember Plone navigation system historically used the
getRemoteUrl brain data to send users to remote URL, but later we
changed the way to do it: we sent the user to the Link object and rely
onto the link_redirect view, that check for user permission.

However it seems that in late 2011 we re-introduced the getRemoteUrl
usage with some weird logic: if the current user is not the creator:
display the use the remote URL (even worst: there's no type
checking... I found that because a custom content types we use is
implementing the getRemoveUrl method, but it's not a Link).

>From my point of view this was a regression.

Any comment?

-- 
-- luca

twitter: http://twitter.com/keul
linkedin: http://linkedin.com/in/lucafbb
blog: http://blog.keul.it/

------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft