Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 513462 - [DS] bind/updated/unbind parameter show error on field injection
Summary: [DS] bind/updated/unbind parameter show error on field injection
Status: CLOSED WORKSFORME
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: PDE-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 490058
  Show dependency tree
 
Reported: 2017-03-10 09:54 EST by Dirk Fauth CLA
Modified: 2017-03-23 17:29 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Fauth CLA 2017-03-10 09:54:39 EST 
The DS Annotation Support for DS 1.3 annotations added with Oxygen M6 shows errors if the bind/updated/unbind parameters are used on field injection. They are valid at these places and also supported at runtime.
Comment 1 Peter Nehrer CLA 2017-03-20 15:45:30 EDT 
(In reply to Dirk Fauth from comment #0)
> The DS Annotation Support for DS 1.3 annotations added with Oxygen M6 shows
> errors if the bind/updated/unbind parameters are used on field injection.
> They are valid at these places and also supported at runtime.

Dirk, I did get that change into the original Bug 510775 with another changeset (not the original one). Can you please check if that fixed it for you?
Comment 2 Dirk Fauth CLA 2017-03-21 02:49:30 EDT 
(In reply to Peter Nehrer from comment #1)
> Dirk, I did get that change into the original Bug 510775 with another
> changeset (not the original one). Can you please check if that fixed it for
> you?

Can you point me to changeset? I'm not sure where to check. Is it already included in M6 or did you merge it afterwards?
Comment 3 Peter Nehrer CLA 2017-03-21 09:17:58 EDT 
(In reply to Dirk Fauth from comment #2)
> Can you point me to changeset? I'm not sure where to check. Is it already
> included in M6 or did you merge it afterwards?

It's this one: https://git.eclipse.org/r/#/c/92331/

It's already part of M6 (merged before the deadline).
Comment 4 Dirk Fauth CLA 2017-03-23 17:29:57 EDT 
Yes I can verify that this error does not exist in M6 anymore. Sorry for the noise here, I just remembered the issue when I reviewed the previous patch before M6.