[
http://issues.apache.org/jira/browse/MYFACES-623?page=comments#action_12330644
]
Bill Dudney commented on MYFACES-623:
-------------------------------------
Sean,
looking at the patch it seems to have something to do with the converter.
this test;
public void testSetValueSimpleBeanInRequestMapWithInitialValue() {
Map map = new HashMap();
String initialValue = "hello world";
map.put("baz", initialValue);
TestBean bean = new TestBean(map);
facesContext.getExternalContext().getRequestMap().put("bean",
bean);
ValueBinding binding =
application.createValueBinding("#{bean.map['baz']}");
assertEquals(initialValue, binding.getValue(facesContext));
Integer value = new Integer(14);
binding.setValue(facesContext, value);
assertEquals(14, value.intValue());
}
passes.
This test is getting the value through the ValueBinding before setting it to an
integer.
What else am I missing in the test besides a converter?
I'll do that next.
> setValue() method of ValueBindingImpl does not behave properly
> --------------------------------------------------------------
>
> Key: MYFACES-623
> URL: http://issues.apache.org/jira/browse/MYFACES-623
> Project: MyFaces
> Type: Bug
> Components: JSR-127
> Versions: 1.1.0
> Reporter: sean schofield
> Attachments: MYFACES-623.patch
>
> According to section 5.1.4 of the specification (p. 5-4) ...
> If you have #{expr-a[value-b]} where value-a is a Map then ...
> call value-a.put(value-b, newValue).
> The MyFaces implementation is coercing newValue into String which is
> incorrect behavior.
> NOTE: I discovered the problem while using a bean that was programatically
> added to the session map. So there is is no class defined in
> faces-config.xml. I don't think this should matter but I thought I would
> mention it.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
