svn commit: r692768 - /incubator/jspwiki/branches/JSPWIKI_2_9_STRIPES_BRANCH/doc/README - Stripes Migration

[aj...@apache.org]

Author: ajaquith
Date: Sat Sep  6 19:28:52 2008
New Revision: 692768

URL: http://svn.apache.org/viewvc?rev=692768&view=rev
Log:
StripesJspMigrator changes and tests. Now properly parses JSPs, including tag & directive attributes.

Modified:
    incubator/jspwiki/branches/JSPWIKI_2_9_STRIPES_BRANCH/doc/README - Stripes Migration

Modified: incubator/jspwiki/branches/JSPWIKI_2_9_STRIPES_BRANCH/doc/README - Stripes Migration
URL: http://svn.apache.org/viewvc/incubator/jspwiki/branches/JSPWIKI_2_9_STRIPES_BRANCH/doc/README%20-%20Stripes%20Migration?rev=692768&r1=692767&r2=692768&view=diff
==============================================================================
--- incubator/jspwiki/branches/JSPWIKI_2_9_STRIPES_BRANCH/doc/README - Stripes Migration (original)
+++ incubator/jspwiki/branches/JSPWIKI_2_9_STRIPES_BRANCH/doc/README - Stripes Migration Sat Sep  6 19:28:52 2008
@@ -258,6 +258,21 @@
 
 Note the @EventPermission annotation. It defines the Permission class and its target and actions. The "permissionClass" attribute tells use that the Permission class this method needs is "PagePermission". Note also the JSTL-style syntax in the target and actions attributes -- these allow JSTL-access to bean properties for the instantiated ViewActionBean. In this case, "page" is the bean attribute that returns the value of this ViewActionBean's getPage() method. The nested syntax "page.qualifiedName" is equivalent to getPage().getQualifiedName(). Neat, huh?
 
+Notes from Janne Visit:
+----------------------
+RequestState: interface to most of the forwarding methods below. WikiActionBeanContext implements this.
+WikiActionBeanContext would contain (private/protected) get/sets for JCR context. Not in RequestState or WikiContext interfaces.
+
+Mappings from JCR paths to permissions and URLs:
+
+JCR node:          /wikis/{space}/{page}/{sub}/{subsub}
+Permission target: {space}:{page}/{sub}/{subsub}
+URL:               /wiki/{space}:{page}/{sub}/{subsub} -- where space == "default" if omitted
+
+Agreed: wipe out ISO text support and use UTF-8 completely.
+Deprecations: WikiSession messages should/will go away with Stripes.
+Don't need Stripes templates yet.
+JMX: use annotations to expose particular Manager methods as MBean.
 
 WIKICONTEXT
 -----------
@@ -384,7 +399,7 @@
 JSP Migration
 -------------
 <form> becomes <stripes:form>, </form> becomes </stripes:form>
-Remove <form> onsubmit="return Wiki.submitOnce(this);" because Stripes takes care of double-submits. accept-charset becomes "acceptcharset" and should use EL.
+Remove <form> onsubmit="return Wiki.submitOnce(this);" because Stripes takes care of double-submits. accept-charset becomes "acceptcharset" and should use the following EL expression.
 
     acceptcharset="${wikiEngine.contentEncoding}"
     
@@ -425,7 +440,9 @@
         <stripes:errors beanclass="com.ecyrd.jspwiki.action.LoginActionBean"/>
       </div>
 
-LocalePicker will need to default to the user's Preferences setting... we will need to create a separate class for this.
+However, MessagesTag has been retrofitted to print Stripes errors for the current ActionBean in addition to the standard kind. So even though <stripes:errors> are preferred, <wiki:Messages> works for now (but should be considered deprecated). <wiki:Messages> tag re-factored so the set of Valididation for the current ActionBean are always appended to the messages list. This allows <wiki:message> to be used transparently in place of <stripes:errors>.
+
+LocalePicker will need to default to the user's Preferences setting... we will need to create a separate class for this that extends DefaultLocalePicker.
 
 Validation guidelines:
 1. Default the @Validate messages to the same resource bundle used by the templates. This would be default*.properties. Practically speaking, this means that the current contents of StripesResources.properties (which have the default messages for @Validate validations) would be appended to default*.properties. Stripes messages and keys are VERY stable.