[ http://jira.dspace.org/jira/browse/DS-688?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
César Sabater updated DS-688: ----------------------------- Comment: was deleted (was: The patch is working fine for my DSpace 1.6.2 XMLUI interface. Thanks!) > AuthorizeManager.isAdmin(Context, DSpaceObject) throws exception when EPerson > is null > ------------------------------------------------------------------------------------- > > Key: DS-688 > URL: http://jira.dspace.org/jira/browse/DS-688 > Project: DSpace 1.x > Issue Type: Bug > Components: DSpace API > Affects Versions: 1.5.0, 1.5.1, 1.5.2, 1.6.0, 1.6.1, 1.6.2 > Reporter: Kim Shepherd > Assignee: Kim Shepherd > Priority: Minor > Fix For: 1.7.0 > > Attachments: AuthorizeManager.java, > DS-688_AuthorizeManager_isAdmin.patch > > > AuthorizeManager.isAdmin(Context, DSpaceObject) doesn't test whether EPerson > is null before trying getID(), which causes an exception to be thrown when > the user is anonymous. > Other isAdmin() methods set or initialize userid to 0 (anonymous) to prevent > this. > The attached patch brings isAdmin(c, dso) in line with other isAdmin methods, > optimises the tests slightly, and fixes some indenting (I think) > The attached patch will apply correctly to 1.6.2 and trunk (5378), and has > been tested against both builds. > A patched 1.6.2 AuthorizeManager.java is also attached in case some users > find it easier to simply copy source files in place. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.dspace.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira ------------------------------------------------------------------------------ Beautiful is writing same markup. Internet Explorer 9 supports standards for HTML5, CSS3, SVG 1.1, ECMAScript5, and DOM L2 & L3. Spend less time writing and rewriting code and more time creating great experiences on the web. Be a part of the beta today. http://p.sf.net/sfu/beautyoftheweb _______________________________________________ Dspace-devel mailing list Dspace-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-devel