Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 318423 - getModelObject(IPath) does not force model load.
Summary: getModelObject(IPath) does not force model load.
Status: NEW
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: Future   Edit
Assignee: Dimitar Giormov CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-30 04:26 EDT by Dimitar Giormov CLA
Modified: 2012-08-31 10:57 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitar Giormov CLA 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 CLA 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 CLA 2010-06-30 04:30:27 EDT
Bug create from: 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=315423
Comment 3 Dimitar Giormov CLA 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 CLA 2011-05-20 02:53:48 EDT
Re-targeting for SR1
Comment 5 Dimitar Giormov CLA 2011-08-26 10:11:21 EDT
re-targeting for 3.4