Wicket
  1. Wicket
  2. WICKET-394

[Patch] do not serialize choices and select in palette.

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.3.0-beta1
    • Fix Version/s: 1.3.0-beta2, 1.3.0-beta3
    • Component/s: wicket
    • Labels:
      None

      Description

      It takes a long useless time

      1. wicket-ajax.js.394.txt
        0.7 kB
        Vincent Demay
      2. palette.component.394.txt
        2 kB
        Vincent Demay
      3. AbstractOptions.java.394.patch.txt
        1 kB
        Vincent Demay
      4. wicket-ajax.js.394.patch.txt
        0.8 kB
        Vincent Demay
      5. AbstractOptions.java.394.patch.txt
        1 kB
        Vincent Demay
      6. wicket-ajax.js.394.patch.txt
        0.9 kB
        Vincent Demay

        Issue Links

          Activity

          Vincent Demay created issue -
          Vincent Demay made changes -
          Field Original Value New Value
          Attachment wicket-ajax.js.394.txt [ 12353405 ]
          Vincent Demay made changes -
          Attachment palette.component.394.txt [ 12353406 ]
          Vincent Demay made changes -
          Link This issue depends on WICKET-393 [ WICKET-393 ]
          Hide
          Vincent Demay added a comment -

          Well, Problem is wicket:serialize-ajax do not work in non-debug mode because wicket namespace is removed. On the other hand, putting a non html tag such as serialize-ajax="false" make Xhtml failed.

          So the best to do, I think is generated a js table of elements that do not need to be serialized in ajax-serialization so the patch to do that is the 2 last files I added to this bug.

          WDYT?

          Show
          Vincent Demay added a comment - Well, Problem is wicket:serialize-ajax do not work in non-debug mode because wicket namespace is removed. On the other hand, putting a non html tag such as serialize-ajax="false" make Xhtml failed. So the best to do, I think is generated a js table of elements that do not need to be serialized in ajax-serialization so the patch to do that is the 2 last files I added to this bug. WDYT?
          Vincent Demay made changes -
          Attachment AbstractOptions.java.394.patch.txt [ 12353968 ]
          Vincent Demay made changes -
          Attachment wicket-ajax.js.394.patch.txt [ 12353969 ]
          Vincent Demay made changes -
          Attachment AbstractOptions.java.394.patch.txt [ 12354057 ]
          Vincent Demay made changes -
          Attachment wicket-ajax.js.394.patch.txt [ 12354058 ]
          Show
          Vincent Demay added a comment - The simpliest is better : https://issues.apache.org/jira/secure/attachment/12354058/wicket-ajax.js.394.patch.txt https://issues.apache.org/jira/secure/attachment/12354057/AbstractOptions.java.394.patch.txt
          Matej Knopp made changes -
          Assignee Matej Knopp [ knopp ]
          Hide
          Matej Knopp added a comment -

          I've applied sligthly modified version of the patch.

          Show
          Matej Knopp added a comment - I've applied sligthly modified version of the patch.
          Matej Knopp made changes -
          Resolution Fixed [ 1 ]
          Fix Version/s 1.3.0-beta3 [ 12312533 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Alastair Maw made changes -
          Fix Version/s 1.3.0-beta3 [ 12312533 ]
          Fix Version/s 1.3.0-beta2 [ 12312502 ]
          Hide
          Jean-Baptiste Quenot added a comment -

          The line with Wicket.Form.excludeFromAjaxSerialization is output for every option, that's overkill. Need to add it only once for the whole <select>

          Show
          Jean-Baptiste Quenot added a comment - The line with Wicket.Form.excludeFromAjaxSerialization is output for every option, that's overkill. Need to add it only once for the whole <select>
          Jean-Baptiste Quenot made changes -
          Status Resolved [ 5 ] Reopened [ 4 ]
          Resolution Fixed [ 1 ]
          Assignee Matej Knopp [ knopp ] Jean-Baptiste Quenot [ jbq ]
          Jean-Baptiste Quenot made changes -
          Resolution Fixed [ 1 ]
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 1.3.0-beta3 [ 12312533 ]
          Gavin made changes -
          Link This issue depends on WICKET-393 [ WICKET-393 ]
          Gavin made changes -
          Link This issue depends upon WICKET-393 [ WICKET-393 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          87d 18h 29m 1 Matej Knopp 11/Jun/07 13:04
          Resolved Resolved Reopened Reopened
          49d 20h 36m 1 Jean-Baptiste Quenot 31/Jul/07 09:41
          Reopened Reopened Resolved Resolved
          32s 1 Jean-Baptiste Quenot 31/Jul/07 09:41

            People

            • Assignee:
              Jean-Baptiste Quenot
              Reporter:
              Vincent Demay
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development