[Aplaws-commit] r159 - ccm-forum/trunk/src/com/arsdigita/forum/ui

[<mb...@re...>]

Author: mbooth
Date: 2004-12-21 20:20:24 +0100 (Tue, 21 Dec 2004)
New Revision: 159

Modified:
   ccm-forum/trunk/src/com/arsdigita/forum/ui/EditPostForm.java
   ccm-forum/trunk/src/com/arsdigita/forum/ui/NewPostForm.java
   ccm-forum/trunk/src/com/arsdigita/forum/ui/PostForm.java
   ccm-forum/trunk/src/com/arsdigita/forum/ui/ThreadComponent.java
Log:
Don't redirect after posting because it results in the wizard not being properly reset. This still needs to be fixed properly.


Modified: ccm-forum/trunk/src/com/arsdigita/forum/ui/EditPostForm.java
===================================================================
--- ccm-forum/trunk/src/com/arsdigita/forum/ui/EditPostForm.java	2004-12-17 17:26:33 UTC (rev 158)
+++ ccm-forum/trunk/src/com/arsdigita/forum/ui/EditPostForm.java	2004-12-21 19:20:24 UTC (rev 159)
@@ -36,7 +36,6 @@
 import com.arsdigita.kernel.permissions.PrivilegeDescriptor;
 import com.arsdigita.kernel.ui.ACSObjectSelectionModel;
 import com.arsdigita.util.UncheckedWrapperException;
-import com.arsdigita.web.RedirectSignal;
 import java.io.IOException;
 import java.math.BigDecimal;
 import org.apache.log4j.Logger;

Modified: ccm-forum/trunk/src/com/arsdigita/forum/ui/NewPostForm.java
===================================================================
--- ccm-forum/trunk/src/com/arsdigita/forum/ui/NewPostForm.java	2004-12-17 17:26:33 UTC (rev 158)
+++ ccm-forum/trunk/src/com/arsdigita/forum/ui/NewPostForm.java	2004-12-21 19:20:24 UTC (rev 159)
@@ -32,7 +32,6 @@
 import com.arsdigita.forum.Forum;
 import com.arsdigita.kernel.Kernel;
 import com.arsdigita.util.UncheckedWrapperException;
-import com.arsdigita.web.RedirectSignal;
 
 import java.io.IOException;
 import java.math.BigDecimal;

Modified: ccm-forum/trunk/src/com/arsdigita/forum/ui/PostForm.java
===================================================================
--- ccm-forum/trunk/src/com/arsdigita/forum/ui/PostForm.java	2004-12-17 17:26:33 UTC (rev 158)
+++ ccm-forum/trunk/src/com/arsdigita/forum/ui/PostForm.java	2004-12-21 19:20:24 UTC (rev 159)
@@ -171,7 +171,7 @@
         public void process(FormSectionEvent e)
             throws FormProcessException {
 
-            PageState state = e.getPageState();
+            final PageState state = e.getPageState();
             
             Post post = getPost(state, true);
             processWidgets(state, post);
@@ -182,6 +182,8 @@
             post.save();
 
             fireCompletionEvent(state);
+            /* XXX: This is the right thing to do, however it results in 
+               the wizard not being reset. I haven't tracked down why.
 
             state.clearControlEvent();
             try {
@@ -189,6 +191,7 @@
             } catch( IOException ex ) {
                 throw new UncheckedWrapperException( ex );
             }
+            */
         }
     }
 

Modified: ccm-forum/trunk/src/com/arsdigita/forum/ui/ThreadComponent.java
===================================================================
--- ccm-forum/trunk/src/com/arsdigita/forum/ui/ThreadComponent.java	2004-12-17 17:26:33 UTC (rev 158)
+++ ccm-forum/trunk/src/com/arsdigita/forum/ui/ThreadComponent.java	2004-12-21 19:20:24 UTC (rev 159)
@@ -49,7 +49,6 @@
 import com.arsdigita.messaging.MessageThread;
 import com.arsdigita.messaging.ThreadedMessage;
 import com.arsdigita.util.Assert;
-import com.arsdigita.web.RedirectSignal;
 import com.arsdigita.web.URL;
 import com.arsdigita.xml.Element;