Bug 189472 - [Site][Editors] Clipboard operations do not work in tree viewer on "Site Map" page
Summary: [Site][Editors] Clipboard operations do not work in tree viewer on "Site Map"...
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 3.6 M2   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday
: 189470 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-05-28 12:29 EDT by Grant Gayed CLA
Modified: 2009-09-16 14:57 EDT (History)
3 users (show)

See Also:

Attachments
Patch for CategorySection.java (2.37 KB, patch)
2009-07-15 14:32 EDT, Simon Muschel CLA 		curtis.windatt.public: iplog+
Details | Diff
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 Grant Gayed CLA 2007-05-28 12:29:45 EDT 
3.3RC2, observed on gtk-64, likely happens on all platforms

- create an Update Site project, which auto-opens an editor
- create a category
- select the category, add a feature to it
- select the feature, right-click -> Cut
  -> it disappears
- select the category, right-click -> Paste
  -> it does not re-appear
Comment 1 Mike Pawlowski CLA 2007-05-28 12:49:34 EDT 
Agree.  Problem for all platforms.
Copy and paste is a problem as well as cut.
Comment 2 Mike Pawlowski CLA 2007-05-28 12:56:05 EDT 
*** Bug 189470 has been marked as a duplicate of this bug. ***
Comment 3 Simon Muschel CLA 2009-07-15 14:32:30 EDT 
Created attachment 141688 [details]
Patch for CategorySection.java

I found this bug through the bug day wiki and thought, I'd give it a try...

I guess the problem is, that the ISiteFeature returned from clipboard has no connection to the current ISiteModel. So I used that Feature as a template to create a new ISiteFeature in the ISiteModel.

With these changes, cut/copy/paste work fine for me.
Comment 4 Benjamin Cabé CLA 2009-07-15 14:35:22 EDT 
Thanks Simon, we will have a look! :)
Comment 5 Curtis Windatt CLA 2009-08-21 17:25:22 EDT 
Tested the patch and it appears to work well.  I'll test some more and go over the code before committing.
Comment 6 Curtis Windatt CLA 2009-08-25 14:37:01 EDT 
Applied patch to HEAD with minor changes.  Thanks Simon.
Comment 7 Curtis Windatt CLA 2009-09-16 14:57:15 EDT 
Verified in I20090915-0100