[JS1-356] [PATCH] - Unable to add Pane when entering customizer from top level

• Type: Bug
• Status: Resolved
• Resolution: Fixed
• Affects Version/s: 1.4b5-dev / CVS
• Component/s: Customizer
• Environment: Operating System: All; Platform: All
• Bugzilla Id: 21324
• Assignee: Unassigned
• Reporter: John Christopher
• Created: 04/Jul/03 01:30
• Updated: 16/Feb/11 21:04
• Resolved: 24/Mar/04 14:39

Description

When trying to add a pane when entering the customizer from the top level (The
customize html link at the top of the page in a default jetspeed deploy) the add
Pane button will never enable. If you enter the customizer from the pane level,
tab level, or portlet level, the button to add a pane is enabled. Below is a
patch to CustomizeSetAction that fixes this problem. Apparently, when entering
the customizer from the pane level or below, the pane name was being returned as
null. The name was then set to "*" and this was being used to enable the add
pane button in the template. If you enter at the top level and then drill down,
a pane name would be returned and would not match "*".

Index: src/java/org/apache/jetspeed/modules/actions/portlets/CustomizeSetAction.java
===================================================================
RCS file:
/home/cvspublic/jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/portlets/CustomizeSetAction.java,v
retrieving revision 1.42
diff -u -r1.42 CustomizeSetAction.java
—
src/java/org/apache/jetspeed/modules/actions/portlets/CustomizeSetAction.java
27 May 2003 23:58:41 -0000 1.42
+++
src/java/org/apache/jetspeed/modules/actions/portlets/CustomizeSetAction.java
3 Jul 2003 18:14:10 -0000
@@ -258,7 +258,7 @@
// do not allow panes for wml profiles
if ((!mediaType.equalsIgnoreCase("wml")) && (set.getController()
instanceof PortletSetController))
{

• if (customizedPaneName.equalsIgnoreCase("*"))
  + if (customizedPaneName != null) { context.put("allowpane", "true"); }