Greg, me no like, instead of just using the "cancel" string in umpteen JSPs, why not define a constant CANCEL in the parent UIAction class like we do for the others like INPUT, LIST, SUCCESS, etc. and use that?

+    public String cancel() {
+        return "cancel";
+    }


Cheers,
Glen

On 03/25/2014 10:41 AM, ghu...@apache.org wrote:
Author: ghuber
Date: Tue Mar 25 14:41:11 2014
New Revision: 1581353

URL: http://svn.apache.org/r1581353
Log:
Tidy interceptors in struts.xml.
Don't chain on "cancel" requests, otherwise parameter issues.
Testing changes!!

Modified:
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/BookmarkAdd.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/BookmarkEdit.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/CategoryAdd.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/CategoryEdit.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/CategoryRemove.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/EntryRemove.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/FolderAdd.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/FolderEdit.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/MembersInvite.java
     
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/editor/TemplateRemove.java
     roller/trunk/app/src/main/resources/struts.xml
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/BookmarkAdd.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/BookmarkEdit.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/Bookmarks.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/CategoryAdd.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/CategoryEdit.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/CategoryRemove.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/EntryRemove.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/EntryRemoveViaList.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/FolderAdd.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/FolderEdit.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/MembersInvite.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/TemplateRemove.jsp
     roller/trunk/app/src/main/webapp/WEB-INF/jsps/editor/WeblogConfig.jsp


Reply via email to