Grails
  1. Grails
  2. GRAILS-6313

Regression: params with underscores automap differently from prior grails versions

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.3.1
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Testcase included:
      yes

      Description

      If you use underscores in your form params for non-checkboxes - which is/was perfectly legal - doing so with 1.3.1 yields unexpected results as the automaps have been adapted to support nested checkboxes (GRAILS-6224) which now breaks non-checkbox usage.

      I'm not sure there is a solution, but it needs flagging that this can break old apps.

      See attached app.

        Issue Links

          Activity

          Hide
          Marc Palmer added a comment -

          This issue has been rolled over so many times it has become sausage-shaped.

          Show
          Marc Palmer added a comment - This issue has been rolled over so many times it has become sausage-shaped.
          Hide
          Graeme Rocher added a comment -

          Patches welcome

          Show
          Graeme Rocher added a comment - Patches welcome
          Hide
          Jeff Scott Brown added a comment -

          A pull request with a failing functional test would be helpful (even if the pull request doesn't include a fix). https://github.com/grails/grails-functional-tests/tree/master/apps/kitchen_sink_app/test/functional/parambinding might be a good place for the test.

          Show
          Jeff Scott Brown added a comment - A pull request with a failing functional test would be helpful (even if the pull request doesn't include a fix). https://github.com/grails/grails-functional-tests/tree/master/apps/kitchen_sink_app/test/functional/parambinding might be a good place for the test.

            People

            • Assignee:
              Unassigned
              Reporter:
              Marc Palmer
            • Votes:
              2 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Last Reviewed:

                Development