Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 317751

Summary:

Details and Mark as menu on attachment shows on all connectors

Product:

z_Archived

Reporter:

Shawn Minto <shawn.minto>

Component:

Mylyn

Assignee:

Frank Becker <eclipse>

Status:

RESOLVED FIXED

QA Contact:

Severity:

major

Priority:

CC:

robert.elves, steffen.pingel

Version:

unspecified

Target Milestone:

3.4.1

Hardware:

OS:

Windows 7

Whiteboard:

Attachments:

Description	Flags
commited patch	none
mylyn/context/zip	none
commited patch	eclipse: review?
mylyn/context/zip	none
backported patch	none
mylyn/context/zip	none
additional changes for head	none

Description Shawn Minto

2010-06-23 17:26:16 EDT

Bugzilla contributes the "Details..." and "Mark As" menu to task attachments, but it uses an object contribution for ITaskAttachment which means that it will show up for all attachments on any connector (i.e. JIRA), but not do anything.  I think that this should be converted to a command and then visibility rules can be used to ensure that it only shows for Bugzilla tasks.

Comment 1 Steffen Pingel

2010-06-23 17:33:58 EDT

Rob, Frank, this is a usability problem that needs to be addressed for 3.4.1.

Comment 2 Steffen Pingel

2010-07-08 19:51:15 EDT

Frank, I'll tentatively assign this to you. Please feel free to assign it to Rob if you don't get to it.

Comment 3 Frank Becker

2010-07-10 10:44:28 EDT

Created attachment 173934 [details]
commited patch

Steffen,

can you do the backport?

Comment 4 Frank Becker

2010-07-10 10:44:30 EDT

Created attachment 173935 [details]
mylyn/context/zip

Comment 5 Steffen Pingel

2010-07-11 00:17:54 EDT

I would prefer if we could use a menu contribution that was only enabled for Bugzilla attachments instead of extending an internal part. This should work through using a property tester. I'll try to post an example next week.

Comment 6 Frank Becker

2010-07-11 03:26:10 EDT

(In reply to comment #5)
> I would prefer if we could use a menu contribution that was only enabled for
> Bugzilla attachments instead of extending an internal part. This should work
> through using a property tester. I'll try to post an example next week.

Steffen,

I try to use AbstractSourceProvider and hope that I can do this today.

Comment 7 Frank Becker

2010-07-11 10:39:48 EDT

Created attachment 173975 [details]
commited patch

We now use an IActionFilter for TaskAttachment with the following defined names.

* ConnectorKind
* ContentType
* isDeprecated
* isPatch

Do you think that we still need others?

Comment 8 Frank Becker

2010-07-11 10:39:51 EDT

Created attachment 173976 [details]
mylyn/context/zip

Comment 9 Steffen Pingel

2010-07-12 18:38:13 EDT

Thanks! That patch looks good. I'll look into back-porting the changes. I don't think we need anymore than these properties.

Comment 10 Steffen Pingel

2010-07-13 14:51:51 EDT

Created attachment 174207 [details]
backported patch

Comment 11 Steffen Pingel

2010-07-13 14:51:53 EDT

Created attachment 174208 [details]
mylyn/context/zip

Comment 12 Steffen Pingel

2010-07-13 14:55:31 EDT

Created attachment 174209 [details]
additional changes for head

Comment 13 Steffen Pingel

2010-07-13 14:55:54 EDT

Thanks Frank! Should be all backported now.

Comment 14 Frank Becker

2010-07-13 16:37:40 EDT

fixed a typo
               <objectState name="ConnectorKind" value="bugzilla"></objectState>
changed to
               <objectState name="connectorKind" value="bugzilla"></objectState>

Comment 15 Steffen Pingel

2010-07-13 16:49:50 EDT

Thanks for catching that!