[PLUTO-609] PortletPreferencesImpl doesn't handle null preferences correctly

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.2
• Fix Version/s: 2.0.3, 2.1.0-M3, 2.1.0
• Assignee: Ate Douma
• Reporter: Eric Dalquist
• Created: 07/Jul/11 17:41
• Updated: 24/Aug/12 20:53
• Resolved: 17/Sep/11 19:07

Description

PLT.17.1 states "Preference attributes are String array objects. Preferences attributes can be set to null." In Pluto if you call PortletPreference.setValue("name", null), PortletPreference.setValues("name", String[]

{null}

), or PortletPreference.setValues("name", null) the correct data is passed to the underlying preference storage SPI.

The problem is when calling getValue("name", "DEFAULT") or getValues("name", new String[]

{ "DEFAULT" }

) for any of the three previous cases "DEFAULT" is returned. From my reading of the spec this is not correct as in each case the preference has been set but with a single null value or a null values array.