Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 315465 - [binding-editor] Creating callout-to-field via binding editor
Summary: [binding-editor] Creating callout-to-field via binding editor
Status: VERIFIED FIXED
Alias: None
Product: Objectteams
Classification: Tools
Component: OTDT (show other bugs)
Version: 1.4   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: 0.7 M4   Edit
Assignee: Stephan Herrmann CLA
QA Contact:
URL:
Whiteboard: trac
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-02 15:30 EDT by Stephan Herrmann CLA
Modified: 2010-06-29 10:07 EDT (History)
0 users

See Also:


Attachments
Implementation (10.61 KB, patch)
2010-06-02 17:42 EDT, Stephan Herrmann CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Stephan Herrmann CLA 2010-06-02 15:30:15 EDT
(originally from http://trac.objectteams.org/ot/ticket/83)

(original report by Robin Sedlaczek:)
When using the binding editor on a team class with bound roles, the lower right listbox shows fields of the bound base class. After selecting 'New method() and -> and a base field, the apply button is enabled. When pressed an error dialog appears saying "Creation of Callin/Calloutbinding failed".

(comment by stephan:)
In class BindingConfiguration method #createCalloutMapping is prepared to create callout-to-field, but this implementation is unfinished.
 
Also note, that r15431 has introduced the capability to display callout-to-field, which previously caused an exception.

Open issue: In order to support generating callout to field, the binding editor must handle the set/get distinction by either: 
 - more buttons for selecting the desired kind, or 
 - generating get *and* set callouts
Comment 1 Stephan Herrmann CLA 2010-06-02 17:42:21 EDT
Created attachment 170887 [details]
Implementation

This patch implements the second option: 
creating get *and* set callout-to-field simultaneously.
(If only one is needed the other one can be removed already
within the binding editor).
Comment 2 Stephan Herrmann CLA 2010-06-02 17:43:46 EDT
Patch has been committed as r412.
Comment 3 Stephan Herrmann CLA 2010-06-11 18:01:45 EDT
Verified for M4 using build 201006111053.