[COLLECTIONS-350] MapUtils.getNumber sends output to System.out

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.2
• Fix Version/s: 3.2.2, 4.0-alpha1, 4.0
• Assignee: Unassigned
• Reporter: Michael Akerman
• Created: 15/Jan/10 16:35
• Updated: 07/Nov/15 20:59
• Resolved: 28/Feb/13 20:07

Description

MapUtils.getNumber calls "logInfo" on a ParseException and it produces output on the System.out, instead of just returning "null" as the documentation states.

Since the expected behavior is to return null, not print to System.out. I recommend we remove the call to logInfo and/or remove the System.out.println inside it.

At minimum, if that's not agreed upon, the "side-effect" of writing to System.out should be documented.

I'm happy to offer code for either of these fixes.