Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 228717 - [KeyBindings] "Alt + Enter" does not open breakpoint properties
Summary: [KeyBindings] "Alt + Enter" does not open breakpoint properties
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 RC1   Edit
Assignee: Paul Webster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 228405 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-04-24 13:06 EDT by Darin Wright CLA
Modified: 2008-08-11 08:23 EDT (History)
6 users (show)

See Also:
emoffatt: review+


Attachments
Action Delegate v01 (1.12 KB, patch)
2008-05-07 10:52 EDT, Paul Webster CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Wright CLA 2008-04-24 13:06:04 EDT
The "Alt + Enter" key combo is not opening the breakpoint properties for Java breakpoints. The action is invoked, but it has no "active" breakpoint. It appears the action is not getting selection change notifications.
Comment 1 Curtis Windatt CLA 2008-04-25 10:10:31 EDT
Moving to Platform UI for comment.

When just changing the selection in the view, the action never receives selection change notifications.

If I right click, two calls to selectionChanged() are made, one with an empty selection and then one with a proper selection.  This allows the action to work correctly if the action is invoked from the context menu.

Even if I use right clicking to update the action, invoking it using the Alt-Enter keybinding always fails, because before run() is called, there is an additional call to selectionChanged() with a null selection.
Comment 2 Eric Moffatt CLA 2008-04-25 14:33:12 EDT
Darin, which build are you using ? I'll pass it to Paul for a look...
Comment 3 Curtis Windatt CLA 2008-04-25 14:36:44 EDT
I20080422
Comment 4 Paul Webster CLA 2008-05-06 13:58:15 EDT
What's the FQ class name for your action?  I'll take a look at this soon.

PW
Comment 5 Darin Wright CLA 2008-05-06 14:05:08 EDT
org.eclipse.jdt.internal.debug.ui.actions.JavaBreakpointPropertiesAction
Comment 6 Paul Webster CLA 2008-05-07 10:52:41 EDT
Created attachment 99089 [details]
Action Delegate v01

the context menu variables sources now return IEvaluationContext.UNDEFINED_VARIABLE when they are not set and it was overriding a casual selection.

That's why it worked from the context menu.

PW
Comment 7 Paul Webster CLA 2008-05-07 10:53:21 EDT
*** Bug 228405 has been marked as a duplicate of this bug. ***
Comment 8 Paul Webster CLA 2008-05-07 11:03:46 EDT
Eric, could you please take a look?

PW
Comment 9 Eric Moffatt CLA 2008-05-07 11:14:10 EDT
Looks good...
Comment 10 Paul Webster CLA 2008-05-07 11:16:49 EDT
Released to HEAD for I20080507-2000
PW
Comment 11 Tomasz Zarna CLA 2008-05-08 04:33:06 EDT
Paul's patch seems to fix all the issues related to Team/CVS actions gathered in bug 228405. Thanks!
Comment 12 Patrick Schonbach CLA 2008-06-16 19:11:40 EDT
Regarding the commit action, there is still a problem, at least in RC3: The keyboard shortcut works perfectly fine, if I use it while the project is selected. However, it does not work, if the topmost package is selected. Selecting other packages seem to work, at least in a quick test.
Comment 13 Paul Webster CLA 2008-06-16 21:10:08 EDT
Please open a new bug with a reproducible test case on RC4 ... commit is not ALT+ENTER.

PW
Comment 14 Joe Thomas-Kerr CLA 2008-08-03 20:26:31 EDT
I am still having issues with this in the 3.4 release. I am finding that the Team/commit keyboard shortcut works intermittently. In my most recent test using the C/C++ projects view:

- selecting one file with changes and pressing the commit shortcut works.
- selecting its parent folder and pressing the shortcut gives a "the selected resources do not contain any changes"
- selecting a second file with pending changes in a different folder gives the same error message (even though the file does contain changes and is annotated as such).
- pressing the commit shortcut of the parent folder of this file also gives the same error message.
- Using the commit shortcut on these same 4 resources in the Package Explorer view opens the commit dialog as expected. However, I have previously experienced the same problems using Project Explorer.

Joe.
Comment 15 Paul Webster CLA 2008-08-11 08:23:45 EDT
(In reply to comment #14)

I've opened Bug 243740 to track this issue.

PW