|
||||||||||||||
|
This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira |
||||||||||||||
------------------------------------------------------------------------------ October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60133471&iu=/4140/ostg.clktrk
_______________________________________________ Dspace-devel mailing list Dspace-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-devel
[20:29] <mhwood> dspace.user.effective is used only in dspace-xmlui. It's done with a manifest constant so it could just have its value changed and that one literal replaced.
[20:32] <helix84> mhwood: so what should we do with PR#291?
[20:34] <mhwood> 291 looks to me like a correct fix. But I'd like to see it use the manifest constant like AuthenticationUtil does. At some point someone could change the value of AuthenticationUtil.EFFECTIVE_USER_ID to match what jspui uses, removing a possible point of confusion, but that's for later.
[20:35] <helix84> mhwood: would you please comment to that effect in the PR or issue?
[20:35] <mhwood> WIll do.