Note that the require refactoring would make impossible to have empty user in a document anymore since getAuthor() would return XWikiGuest in that case.
On Thu, Mar 21, 2013 at 6:13 PM, Thomas Mortagne <[email protected]> wrote: > Hi devs, > > Following http://markmail.org/message/snnlesexie4ox3i5 I would like to > modify user related methods in XWikiDocument to behave like the what > has been voted for XWikiContext and follow the same logic. > > As a reminder here it is: not logged in user (guest) reference is > null. Plain and simple. > > The idea would be to do exactly the same thing than XWikiContext > meaning that old String APIs continue to deal with "XWikiGuest" and do > convertions for retro compatibility especially with existing > databases. > > It will change the behavior of XWikiDocument#getUserReference() from > external point of view so it's a breakage but we might actually fix > more bugs than we create when thinking about codes like > context.setUserRefererence(document.getAuthorReference()) or > $context.userReference == $doc.authorReference which I'm sure we can > find here and there. > > 5.0 and new right service is a good occasion for this improvement. > > WDYT ? > > Here is my +1 > -- > Thomas Mortagne -- Thomas Mortagne _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
