NanoContainer NanoWar

request parameter with multiple values

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    2

Description

NanoWebServlet#setPropertiesWithOgnl use HttpServletRequest#getParameter(String).
So, in case of request paramete has multiple values (ex, checkbox) only gets first value.

  1. Text File
    getParameterValues.patch
    31/Mar/04 1:22 PM
    6 kB
    Kouhei Mori
  2. File
    mparaz-stringsetter.diff
    02/Apr/04 7:44 AM
    31 kB
    Miguel Paraz

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Kouhei Mori added a comment -

probably works, but requires xwork.jar

Show
Kouhei Mori added a comment - probably works, but requires xwork.jar
Hide
Permalink
Miguel Paraz added a comment -

I wrote a class that bypasses OGNL and uses direct reflection to call setters. It addresses this issue. However, I can't test it properly due to the jMock version problem.

Show
Miguel Paraz added a comment - I wrote a class that bypasses OGNL and uses direct reflection to call setters. It addresses this issue. However, I can't test it properly due to the jMock version problem.
Hide
Permalink
Miguel Paraz added a comment -

StringSetterInvoker allows direct setter calls for String and String[] properties, and its test case. I used it for a NanoWebStringSetterServlet. I think OGNL is too heavyweight to be used just for invoking setters. The TestCase setup here uses the latest jMock jar and API calls as specified by Jacob Kjome in PICO-164.

Show
Miguel Paraz added a comment - StringSetterInvoker allows direct setter calls for String and String[] properties, and its test case. I used it for a NanoWebStringSetterServlet. I think OGNL is too heavyweight to be used just for invoking setters. The TestCase setup here uses the latest jMock jar and API calls as specified by Jacob Kjome in PICO-164 .
Hide
Permalink
Aslak Hellesøy added a comment -

Groovy also has some utility class(es) that allow to set properties via reflection. Might be worthwhile to check that option too.

Show
Aslak Hellesøy added a comment - Groovy also has some utility class(es) that allow to set properties via reflection. Might be worthwhile to check that option too.

People

  • Assignee:
    Unassigned
    Reporter:
    Kouhei Mori
Vote (0)
Watch (1)

Dates

  • Created:
    Updated: