You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by jk...@apache.org on 2006/03/28 17:09:45 UTC

svn commit: r389517 - in /jakarta/tapestry/branches/4.0: framework/src/java/org/apache/tapestry/AbstractPage.java framework/src/java/org/apache/tapestry/form/DatePicker.js status.xml

Author: jkuhnert
Date: Tue Mar 28 07:09:43 2006
New Revision: 389517

URL: http://svn.apache.org/viewcvs?rev=389517&view=rev
Log:
Reverted TAPESTRY-875 as it was broken, merged in mind bridge changes to event firing ordering

Modified:
    jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/AbstractPage.java
    jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/form/DatePicker.js
    jakarta/tapestry/branches/4.0/status.xml

Modified: jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/AbstractPage.java
URL: http://svn.apache.org/viewcvs/jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/AbstractPage.java?rev=389517&r1=389516&r2=389517&view=diff
==============================================================================
--- jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/AbstractPage.java (original)
+++ jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/AbstractPage.java Tue Mar 28 07:09:43 2006
@@ -23,11 +23,11 @@
 import org.apache.commons.logging.LogFactory;
 import org.apache.hivemind.ApplicationRuntimeException;
 import org.apache.tapestry.event.ChangeObserver;
+import org.apache.tapestry.event.PageAttachListener;
 import org.apache.tapestry.event.PageBeginRenderListener;
 import org.apache.tapestry.event.PageDetachListener;
 import org.apache.tapestry.event.PageEndRenderListener;
 import org.apache.tapestry.event.PageEvent;
-import org.apache.tapestry.event.PageAttachListener;
 import org.apache.tapestry.event.PageRenderListener;
 import org.apache.tapestry.event.PageValidateListener;
 import org.apache.tapestry.util.StringSplitter;
@@ -414,7 +414,7 @@
         PageEvent event = null;
         Object[] listeners = _listenerList.getListenerList();
 
-        for (int i = 0; i < listeners.length; i += 2)
+        for(int i = listeners.length-2; i >= 0; i -= 2) 
         {
             if (listeners[i] == PageAttachListener.class)
             {
@@ -466,11 +466,11 @@
         PageEvent event = null;
         Object[] listeners = _listenerList.getListenerList();
 
-        for (int i = 0; i < listeners.length; i += 2)
+        for(int i = listeners.length-2; i >= 0; i -= 2) 
         {
-            if (listeners[i] == PageBeginRenderListener.class)
+            if (listeners[i] == PageBeginRenderListener.class) 
             {
-                PageBeginRenderListener l = (PageBeginRenderListener) listeners[i + 1];
+                PageBeginRenderListener l = (PageBeginRenderListener)listeners[i + 1];
 
                 if (event == null)
                     event = new PageEvent(this, _requestCycle);

Modified: jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/form/DatePicker.js
URL: http://svn.apache.org/viewcvs/jakarta/tapestry/branches/4.0/framework/src/java/org/apache/tapestry/form/DatePicker.js?rev=389517&r1=389516&r2=389517&view=diff
==============================================================================
Binary files - no diff available.

Modified: jakarta/tapestry/branches/4.0/status.xml
URL: http://svn.apache.org/viewcvs/jakarta/tapestry/branches/4.0/status.xml?rev=389517&r1=389516&r2=389517&view=diff
==============================================================================
--- jakarta/tapestry/branches/4.0/status.xml (original)
+++ jakarta/tapestry/branches/4.0/status.xml Tue Mar 28 07:09:43 2006
@@ -88,14 +88,6 @@
       <action type="fix" dev="JK" due-to="Zedros Schwart" >
       	Fixed links.ent reference to ExternalAsset, was missing asset package.
       </action>
-      <action type="fix" dev="JK" fixes-bug="TAPESTRY-875" due-to="Matthias Zitzmann/Brian K. Wallace" >
-      	Using the DatePicker for a date format containing no "word delimiter" 
-      	doesn't work because of an inefficient limitational replacement-method 
-      	in DatePicker.js. I added a patch for this, because I've to work with a 
-      	date format of ddMMyy (without delimiter). With that patch it won't break 
-      	any more neither for regular date formats nor for this delimiter-free format 
-      	used in financial or touristics for example. 
-      </action>
       <action type="fix" dev="JK" fixes-bug="TAPESTRY-170" due-to="Vjeran Marcinko/Brian K. Wallace">
       	Modified contrib table column headers to use inlined style and css classes to
       	do table and image style definitions. Much more compatible with css now.



---------------------------------------------------------------------
To unsubscribe, e-mail: tapestry-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tapestry-dev-help@jakarta.apache.org