Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 360076

Summary:	IApplicationStore should align with ISessionStore
Product:	[RT] RAP	Reporter:	Rüdiger Herrmann <ruediger.herrmann>
Component:	RWT	Assignee:	Project Inbox <rap-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	enhancement
Priority:	P3
Version:	1.5
Target Milestone:	1.5 M3
Hardware:	All
OS:	All
Whiteboard:

Description Rüdiger Herrmann

2011-10-06 05:48:49 EDT

ISessionStore does not allow null-values in its setAttribute() and removeAttribute() methods. ApplicationStoreImpl currently does not check for null-values.
I suggest to check for null-values in ApplicationStoreImpl to align its behavior with the session store

Comment 1 Rüdiger Herrmann

2011-10-07 06:05:24 EDT

If this was implemented as suggested, the null-checks in ApplicationConfigurationImpl#setAttribute() would becoome obsolete.

Comment 2 Ralf Sternberg

2011-10-08 09:51:20 EDT

+1 for disallowing null values

Comment 3 Rüdiger Herrmann

2011-10-17 07:00:46 EDT

Implemented as suggested. Changes are in CVS HEAD.