Here is a patch file for the SkinBrowser.  It contains just some minor clean
up.

Jeremy Ford
Index: SkinUpdateAction.java
===================================================================
RCS file: 
/home/cvspublic/jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/portlets/SkinUpdateAction.java,v
retrieving revision 1.1
diff -u -r1.1 SkinUpdateAction.java
--- SkinUpdateAction.java       13 Feb 2003 06:15:58 -0000      1.1
+++ SkinUpdateAction.java       23 Feb 2003 22:16:44 -0000
@@ -246,8 +246,6 @@
                 rundata.setRedirectURI(duri.toString());
 
                 resetForm(rundata);
-
-                clearUserData(rundata);
             }
             else
             {
@@ -257,6 +255,8 @@
                 skinEntry.getParameterMap().clear();
 
                 addParameters(rundata, skinEntry);
+                
+                clearUserData(rundata);
             }
         }
         catch (Exception e)
@@ -305,6 +305,7 @@
                 JetspeedLinkFactory.putInstance(link);
                 rundata.setRedirectURI(duri.toString());
 
+                               resetForm(rundata);
             }
             else
             {
@@ -330,6 +331,12 @@
         }
     }
 
+    /**
+     * Cleanup method
+     * @param rundata The turbine rundata context for this request.
+     * @param context The velocity context for this request.
+     * @throws Exception
+     */
     public void doCancel(RunData rundata, Context context) throws Exception
     {
         clearUserData(rundata);
@@ -395,6 +402,10 @@
         }
     }
 
+    /**
+     * Clears the temporary storage of any data that was used
+     * @param rundata
+     */
     private void clearUserData(RunData rundata)
     {
         try

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

Reply via email to