Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.4.2
    • 1.5-M1
    • wicket
    • None

    Description

      "It would be nice if any component can listen to form's event that signals form components to update their models"
      -There are some components that implemented they input logic using the request parameter (default FormComponet behavior). When Wicket bring those components outside the from to his processing, we get some problems: they will not receive the user input on page by request parameter map.
      Currently on the user mailing list we can see some case:
      http://markmail.org/search/list:org.apache.wicket.users#query:list%3Aorg.apache.wicket.users%20date%3A200909%20from%3A%22Flavius%22+page:1+mid:ykgrynnsyqaum22o+state:results
      If we execute the project sent ed by Flavius, we can see that the request has no parameter to the DropDownChoice component "project", that is on the page and outside the form. The component get his convertInput and updateModel called by the form process. The convertInput call getInputAsArray, that calls:
      String[] values = getRequest().getParameters(getInputName());
      and get null.
      To resolve that problem on wicket trunk, I'm sending a path that remove from the form processing, calls to components outside him(Flavius, maybe you want to apply on your workspace Wicket). But this is only an option. I think Wicket need review the way to process the form:
      1- add inputs from outside form components to request
      2 - remove outside form components from his processing
      3 - any other option

      Attachments

        1. BaseBorder.java
          8 kB
          Juegen Donnerstag
        2. BorderVisit.zip
          66 kB
          Flavius
        3. patch.txt
          17 kB
          Pedro Santos
        4. patch.txt
          0.9 kB
          Pedro Santos
        5. patch3.txt
          1 kB
          Pedro Santos

        Activity

          People

            jdonnerstag Juegen Donnerstag
            pedrosans Pedro Santos
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: