[T5] Palette keeps resetting order of selected values on page reload in 5.0.18

[Thomas Meier <de...@googlemail.com>]

Hello,

I'm experiencing a problem with tapestry's palette component. I'm
using it with the reorder parameter set to true. Reordering the
selected values works fine and the values are sent to the server in
the expected order. However, if the page reloads and I submit the
containing form again without changing the palette's values, the
original order is restored and sent to the server resetting the
previously arranged order.
AFAICT, this relates to the hidden form field used by the palette and
the updateHidden Javascript method. This method is called whenever the
order of the selected options is changed. It is not called however
when the Palette object is instantiated, so the hidden field keeps the
sorting that comes from the server.
I believe, this could be fixed by calling the updateHidden method at
the end of the Tapestry.Palette.initialize method. The following patch
does this:

Index: palette.js
===================================================================
--- palette.js	(Revision 755168)
+++ palette.js	(Arbeitskopie)
@@ -45,6 +45,7 @@
         }.bind(this));

         this.bindEvents();
+        this.updateHidden();
     },

     bindEvents : function()

Regards,
Thomas

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