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

Bug 318423

Summary:	getModelObject(IPath) does not force model load.
Product:	[WebTools] WTP Java EE Tools	Reporter:	Dimitar Giormov <dimitar.giormov>
Component:	jst.j2ee	Assignee:	Dimitar Giormov <dimitar.giormov>
Status:	NEW ---	QA Contact:	Chuck Bridgham <cbridgha>
Severity:	normal
Priority:	P3	CC:	kaloyan, larryisaacs
Version:	3.2
Target Milestone:	Future
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Dimitar Giormov

2010-06-30 04:26:07 EDT

With respect to the calling getModelObject() before getModelObject(IPath), I'm
refering to the use case where the model is being inspected and not necessarily
modified, i.e. validateEdit() hasn't be called.  For web.xml,
getModelObject(IPath) will return null if ddProvider is null and will not force
ddProvider to be set if it isn't already.  Calling getModelObject() will force
ddProvider to be set.  I've seen this issue in our JUnit tests, so it might be
a dispose() that's causing ddProvider to be null and calling getModelObject()
helps ensure it is restored since getModelObject(IPath) won't do it.

Comment 1 Dimitar Giormov

2010-06-30 04:29:01 EDT

Larry if you can provide junit tests for this?
This will help a lot identifying the problem as well as having a test that will not allow regressions?

Comment 2 Dimitar Giormov

2010-06-30 04:30:27 EDT

Bug create from: 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=315423

Comment 3 Dimitar Giormov

2011-01-18 03:44:22 EST

re-targeting for 3.3
Fixing this functionality may cause problems and it is dangerous for 3.2.3

Comment 4 Dimitar Giormov

2011-05-20 02:53:48 EDT

Re-targeting for SR1

Comment 5 Dimitar Giormov

2011-08-26 10:11:21 EDT

re-targeting for 3.4