Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-394

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

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: 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. wicket-ajax.js.394.patch.txt
        0.8 kB
        Vincent Demay
      3. wicket-ajax.js.394.patch.txt
        0.9 kB
        Vincent Demay
      4. palette.component.394.txt
        2 kB
        Vincent Demay
      5. AbstractOptions.java.394.patch.txt
        1 kB
        Vincent Demay
      6. AbstractOptions.java.394.patch.txt
        1 kB
        Vincent Demay

        Issue Links

          Activity

          Hide
          jbq 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
          jbq 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>
          Hide
          knopp Matej Knopp added a comment -

          I've applied sligthly modified version of the patch.

          Show
          knopp Matej Knopp added a comment - I've applied sligthly modified version of the patch.
          Show
          vincent.demay@anyware-tech.com 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
          Hide
          vincent.demay@anyware-tech.com 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@anyware-tech.com 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?

            People

            • Assignee:
              jbq Jean-Baptiste Quenot
              Reporter:
              vincent.demay@anyware-tech.com Vincent Demay
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development