Wicket
  1. Wicket
  2. WICKET-5158

Ignore missing Page in StatelessForm#process()

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 6.7.0
    • Fix Version/s: 6.8.0, 1.5.11
    • Component/s: wicket
    • Labels:
      None

      Description

      I use StatelessForm because I want the page to be stateless with it in most cases but in other cases the page should become stateful.

      A StateLessForm with multiple buttons replaces its parent panel at the end of the button's onSubmit().

      After onSubmit(), StateLessForm#process() calls getPage() with the result of a RuntimeException "No Page found for component".

      Martin Grigorov suggested to use #findPage() instead of #getPage()

      Looking at StatelessForm#process(), I guess that a null check of the result of #findPage() may be required.

      I don't understand the mechanics of "Remove the page parameters for all form component otherwise they get appended to action URL" so I can't assess the consequences of NOT doing this when a panel that contains the form replaces itself.

      I guess it is ok to NOT execute that code in case where the page is gone.

        Activity

        bernard created issue -
        Martin Grigorov made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Martin Grigorov [ mgrigorov ]
        Fix Version/s 6.8.0 [ 12324068 ]
        Fix Version/s 1.5.11 [ 12324069 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        14h 46m 1 Martin Grigorov 26/Apr/13 13:29

          People

          • Assignee:
            Martin Grigorov
            Reporter:
            bernard
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development