Hi Glenn,
[I've cc'd the DEV list]
I forgot that I made that change...
I think I was probably just being consistent with the other stuff
that seemed to use the INFO link to customize a portlet.
I would agree that it looks like a bug - especially as your looking
to fix the place I was being consistent with!
Regards,
Chris
--- Glenn Golden <[EMAIL PROTECTED]> wrote:
> Chris -
>
> According to cvs annotate, you made this change in BasePortletSet:
>
> 1.3 (raphael 29-May-01): public boolean
> allowCustomize(
> RunData rundata )
> 1.3 (raphael 29-May-01): {
> 1.13 (kimptoc 29-Nov-01): if ( (rundata.getUser()
> == null)
> || !rundata.getUser().hasLoggedIn() )
> 1.13 (kimptoc 29-Nov-01): {
> 1.13 (kimptoc 29-Nov-01): return false; //No
> permission
> if user not logged in
> 1.13 (kimptoc 29-Nov-01): }
> 1.13 (kimptoc 29-Nov-01): return
> (JetspeedSecurity.checkPermission(rundata,
> 1.13 (kimptoc 29-Nov-01):
> JetspeedSecurity.PERMISSION_INFO,
> 1.13 (kimptoc 29-Nov-01):
> this));
> 1.3 (raphael 29-May-01): }
>
> I would think this would use PERMISSION_CUSTOMIZE, not _INFO.
>
> Feature, or bug?
>
> Thanks.
>
> - Glenn
>
> --------------------------------------------
> Glenn R. Golden, Systems Research Programmer
> University of Michigan School of Information
> [EMAIL PROTECTED] 734-615-1419
> --------------------------------------------
=====
------------------------------------------
http://www.soccer2002.org.uk - The Game is On!
__________________________________________________
Do You Yahoo!?
LAUNCH - Your Yahoo! Music Experience
http://launch.yahoo.com
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
