[CLK-600] Page.getMessage( message, params) does not accept null for params

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.1.0
• Fix Version/s: 2.1.0
• Component/s: core
• Assignee: Malcolm Edgar
• Reporter: WarnerJan Veldhuis
• Created: 30/Dec/09 12:30
• Updated: 13/Jan/10 09:24
• Resolved: 13/Jan/10 09:24

Description

Page.getMessage() method does not accept null as parameter for Object[] args while it should.

Instead of this:
content = getMessage("exception_" + ex.getErrorType() + "_message", ex.getParameters());

I have to write code like this:
if (ex.getParameters() != null)

{ content = getMessage("exception_" + ex.getErrorType() + "_message", ex.getParameters()); }

else

{ content = getMessage("exception_" + ex.getErrorType() + "_message"); }

The underlying mechanism, MessageFormat.format(String, Object[]) has no trouble handling null as args, so I don't see the reason to bomb out with an IllegalArgumentException("Null args parameter");