Robert,
Updating Entry is a very tricky thing and that particular bug may be more of a feature. Did you run a problem in ICE with something that uses Entry to make sure it still works OK?
If you change Entry then you have to test job launchers and models to make sure that they still store and update values properly.
In the long term, we actually need to fix the way the Entry works with content providers. The current method is unused and duplicates values in the XML.
Jay
On 09/11/2015 11:59 AM, GitHub wrote:
Branch: refs/heads/next
Home: https://github.com/eclipse/ice
Commit: b9aa177f3bde720f51a3add27dee89cba360b025
https://github.com/eclipse/ice/commit/b9aa177f3bde720f51a3add27dee89cba360b025
Author: Robert Smith <SmithRW@xxxxxxxx>
Date: 2015-09-11 (Fri, 11 Sep 2015)
Changed paths:
M org.eclipse.ice.datastructures.test/src/org/eclipse/ice/datastructures/test/EntryTester.java
M org.eclipse.ice.datastructures/src/org/eclipse/ice/datastructures/form/Entry.java
M org.eclipse.ice.viz.service.test/src/org/eclipse/ice/viz/service/datastructures/test/VizEntryTester.java
M org.eclipse.ice.viz.service/src/org/eclipse/ice/viz/service/datastructures/VizEntry.java
Log Message:
-----------
Fixed Entry and VizEntry constructors
The Entry and VizEntry constructors had a bug in which overloaded
setup() methods may not have their changes applied if the constructor
taking a content provider as an arguement was used. All constructors now
properly invoke setup() once.
Signed-off-by: Robert Smith <SmithRW@xxxxxxxx>
_______________________________________________
ice-build mailing list
ice-build@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/ice-build
--
Jay Jay Billings
Oak Ridge National Laboratory
(865) 241-6308
Twitter Handle: @jayjaybillings
I only check email at 8:30, 12:30 and 16:30 every day. Please call me or send
me a tweet if your message is urgent and I will take a look.