Bug 2273 - [EditorMgmt] Need more control on launching external editors (1GEHWEH)
Summary: [EditorMgmt] Need more control on launching external editors (1GEHWEH)
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P4 enhancement with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: Kim Horne CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted, investigate
Depends on:
Blocks:
 
Reported: 2001-10-10 22:33 EDT by Kevin Haaland CLA
Modified: 2006-06-22 12:51 EDT (History)
1 user (show)

See Also:

Attachments
Modified java files to fix the bug (4.29 KB, patch)
2002-12-03 20:39 EST, Jeff Woldan CLA 		no flags Details | Diff
java files for version 2.1 (4.46 KB, patch)
2002-12-17 17:24 EST, Jeff Woldan CLA 		no flags Details | Diff
Show Obsolete (1) 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 Kevin Haaland CLA 2001-10-10 22:33:01 EDT 
Registering an external editor in a plug-in manifest file needs to allow the developer to
specify where the selected file resource goes in the command using something like %f. 
It can't always go at the end.

NOTES:
Comment 1 DJ Houghton CLA 2001-10-29 18:04:22 EST 
PRODUCT VERSION: 0.111
Comment 2 Kevin Haaland CLA 2002-01-21 21:34:29 EST 
If you would like to work on this defect please let us know.
Comment 3 Randy Giffen CLA 2002-08-08 10:29:02 EDT 
Reopen for investigation
Comment 4 Jeff Woldan CLA 2002-12-03 20:39:02 EST 
Created attachment 2637 [details]
Modified java files to fix the bug

The two files in the zip file are modified class files that I believe resolve
this bug.  I have not submitted a bug fix before, so if this is in the wrong
format please let me know.  To apply the fix, Replace the files of the same
name with these, as found in org.eclipse.ui package.
Comment 5 Nick Edgar CLA 2002-12-04 11:37:08 EST 
Thanks for the patch Jeff.  Eduardo, could you please review it?
Comment 6 Eduardo Pereira CLA 2002-12-05 16:49:43 EST 
Jeff is your fix based on 2.1 stream. The class ShellCommand does not exist 
anymore so I am getting some compilation errors.
Comment 7 Jeff Woldan CLA 2002-12-16 21:38:36 EST 
I'll try to submit fix of the version problem asap, sorry for the delay. i have 
been busy.
Comment 8 Jeff Woldan CLA 2002-12-17 17:24:28 EST 
Created attachment 2829 [details]
java files for version 2.1

I had modified code from an older version (2.0.1) of Eclipse for my last
attempt at a fix.  I modified the fix for 2.1 in this new patch.
Comment 9 Eduardo Pereira CLA 2003-01-06 11:49:38 EST 
Jeff, what should happen in other platforms? Do they use %f as well?
Comment 10 Nick Edgar CLA 2003-02-19 11:12:24 EST 
There are no plans for the UI team to work on this defect until higher 
priority items are addressed.
Comment 11 Michael Van Meekeren CLA 2006-04-21 13:58:33 EDT 
Moving Dougs bugs
Comment 12 Kevin Haaland CLA 2006-06-22 12:51:49 EDT 
Marking as wontfix based on comment #10