#[map-payload:key1,key2?] does not evaluate correctly

The following test case added to the org.mule.util.TemplateParserTestCase tests for the bug:
/**

• Tests replacing properties with question mark in the name.
• This can occur, for instance, when retrieving optional
• keys from a map using the map-payload evaluator.
  */
  @Test
  public void testStringParserQuestionMarkInPropertyName()
  {
  TemplateParser tp = TemplateParser.createAntStyleParser();
  Map props = new HashMap();
  props.put("prop1?", "value1");
  props.put("prop1-2", "value2");
  String string = "Some String with ${prop1?} and ${prop1-2} in it";

String result = tp.parse(props, string);
assertEquals("Some String with value1 and value2 in it", result);
}

Modifying (added case '?': in the switch) the following method in the org.mule.util.TemplateParser makes the above test case pass. I have not tested whether other components/test are affected.

private String escape(String string)
{
int length = string.length();
if (length == 0)

else
{
StringBuffer buffer = new StringBuffer(length * 2);
for (int i = 0; i < length; i++)
{
char currentCharacter = string.charAt;
switch (currentCharacter)
{
case '[':
case ']':
case '{': case '}':
case '(':
case ')':
case '$':
case '#':
case '*':
case '?':
buffer.append("
");
// $FALL-THROUGH$ fall through to append original character
default:
buffer.append(currentCharacter);
}
}
return buffer.toString();
}
}

I cannot reproduce the issue described. If you apply MapPayloadExpressionEvaluatorTestCase and run the test, it ignores the optional value as it should.

Can you please post a use case/config that exposes this bug?

If I create a message with a map payload as this:
Map<String, String> thePayload = new HashMap<String, String>();
thePayload.put("Key1", "Value1");
MuleMessage theMuleMessage = new DefaultMuleMessage(thePayload, theContext1);
The in my configuration file do this:
<logger level="ERROR" message="***** Keys in the map: #[map-payload:Key1,Key2?]"/>
I get this console output:

• • • • • Keys in the map: #[map-payload:Key1,Key2?]
          With my patch in place, I get this output:
        • Keys in the map: {Key1=Value1}

Hope this suffices. Please let me know if you want a complete example.

3.1.x: http://fisheye.codehaus.org/changelog/mule/?cs=23326
3.2.x: http://fisheye.codehaus.org/changelog/mule/?cs=23328
3.x: http://fisheye.codehaus.org/changelog/mule/?cs=23329

Reopening to assign correct fix versions