DO NOT REPLY [Bug 21634] New: - [FIX] Customizing portlet logic broken

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21634>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21634

[FIX] Customizing portlet logic broken

           Summary: [FIX] Customizing portlet logic broken
           Product: Jetspeed
           Version: 1.4b5-dev / CVS
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Customizer
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


Based off a report by kylin.

boolean changeRequested = ( (params == null) || (newSkinName == null) ||
(newSecurityParent == null) || (newTitle == null));

if ((p==null) || (changeRequested = false ))

should be replace by 

boolean changeRequested = ( (params != null) || (newSkinName != null) ||
(newSecurityParent != null) || (newTitle != null));

if ((p==null) || (changeRequested == false ))

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]