Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337761 - attachment flags not displayed for products with space in the name
Summary: attachment flags not displayed for products with space in the name
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P2 normal (vote)
Target Milestone: 3.5   Edit
Assignee: Frank Becker CLA
QA Contact: Frank Becker CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-21 14:55 EST by Steffen Pingel CLA
Modified: 2011-02-23 16:04 EST (History)
2 users (show)

See Also:

Attachments
sample attachment (1 bytes, patch)
2011-02-21 14:56 EST, Steffen Pingel CLA 		no flags Details | Diff
mylyn/context/zip (71.85 KB, application/octet-stream)
2011-02-23 12:27 EST, Frank Becker CLA 		no flags Details
View All 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 Steffen Pingel CLA 2011-02-21 14:55:21 EST 
Steps:
1. Create a bug in Mylyn Tasks/Bugzilla
2. Attach file
3. Open attachment details

The attachment flags are not visible e.g. for flagging a file as iplog+. If I open the attachment in the web UI flags are displayed. Also if I set a flag through the web UI it shows in the attachment details dialog after a refresh.
Comment 1 Steffen Pingel CLA 2011-02-21 14:56:12 EST 
Created attachment 189438 [details]
sample attachment
Comment 2 Steffen Pingel CLA 2011-02-21 14:56:43 EST 
 Frank, do you have any thought on this?
Comment 3 Frank Becker CLA 2011-02-23 12:27:22 EST 
Now you can add flags on attachments.

Steffen should we do an backport for this?

I only add this one line:
currentProduct = currentProduct.replaceAll("%20", " "); //$NON-NLS-1$ //$NON-NLS-2$

after line 526 in file SaxConfigurationContentHandler
Comment 4 Frank Becker CLA 2011-02-23 12:27:37 EST 
Created attachment 189620 [details]
mylyn/context/zip
Comment 5 Steffen Pingel CLA 2011-02-23 16:04:08 EST 
Thanks for fixing this. The maintenance release is already done and the Helios SR2 RC4 deadline is today so I don't think it's worth the trouble backporting but it's good to have it fixed for 3.5. In case we do a 3.4.4 release we can reconsider but I don't expect that will happen.