Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 110871 - Library Example: setName in WriterImpl does not send notifications
Summary: Library Example: setName in WriterImpl does not send notifications
Status: VERIFIED FIXED
Alias: None
Product: EMF Services
Classification: Modeling
Component: Validation (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal
Target Milestone: ---   Edit
Assignee: Christian Vogt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-09-27 16:33 EDT by Christian Vogt CLA
Modified: 2017-02-24 15:09 EST (History)
1 user (show)

See Also:


Attachments
Proposed Patch (1.45 KB, patch)
2005-09-27 16:36 EDT, Christian Vogt CLA
no flags Details | Diff
patch for fix (1.87 KB, patch)
2005-10-04 16:00 EDT, Christian Vogt CLA
ahunter.eclipse: iplog+
Details | Diff
patch for fix (773 bytes, patch)
2005-10-04 16:02 EDT, Christian Vogt CLA
ahunter.eclipse: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Vogt CLA 2005-09-27 16:33:20 EDT
org.eclipse.emf.examples.extlibrary.impl.WriterImpl
Method setName in WriterImpl does not send notifcations.

Found while testing Live Validation example 
(org.eclipse.emf.validation.examples).
There is a java constraint which requires Writers to not have blank names. 
Because no notifaction is being sent when the name of a writer is changed, this 
constraint never comes into play.
Comment 1 Christian Vogt CLA 2005-09-27 16:36:08 EDT
Created attachment 27593 [details]
Proposed Patch
Comment 2 Christian Damus CLA 2005-09-28 17:37:07 EDT
In addition to what the attached patch does, the notification would also have to
be changed by the setFirstName() and setLastName() methods, for the attributes
on which the derived "name" attribute depends.
Comment 3 Christian Vogt CLA 2005-10-04 13:16:07 EDT
After a discussion with Christian Damus, it was decided that the example 
constraint which deals with the derived attribute Writer.name will be removed 
rather than add a notification. Adding a notification to the derived attribute 
introduces complications which go against the point of providing a simple 
constraint example.

There are already two other constraint examples which demonstrate the same 
functionality as the Writer.name constraint would have. Therefore removing the 
Writer.name constraint will not lessen the value of the example.
Comment 4 Christian Vogt CLA 2005-10-04 16:00:25 EDT
Created attachment 27826 [details]
patch for fix

Removed Writer constraints.
Comment 5 Christian Vogt CLA 2005-10-04 16:02:00 EDT
Created attachment 27827 [details]
patch for fix

Previously when the user deleted the Writer's name, the name was being set to "
".
Comment 6 Anthony Hunter CLA 2005-10-05 14:10:53 EDT
Patches Committed
Comment 7 Christian Vogt CLA 2005-10-05 17:01:36 EDT
resolving
Comment 8 Nick Boldt CLA 2008-01-28 16:34:49 EST
Move to verified as per bug 206558.