Bug 243744 - [Dialogs] Adding content to "Compare with Other" dialog from Open Resource dialog
Summary: [Dialogs] Adding content to "Compare with Other" dialog from Open Resource di...
Status: ASSIGNED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Compare (show other bugs)
Version: 3.4   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-Compare-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 248300 (view as bug list)
Depends on: 73923 239959 244623
Blocks: 224562
  Show dependency tree
 
Reported: 2008-08-11 09:19 EDT by Aleksandra Wozniak CLA
Modified: 2019-09-06 15:29 EDT (History)
2 users (show)

See Also:


Attachments
patch #1 (4.64 KB, patch)
2008-08-11 09:23 EDT, Aleksandra Wozniak CLA
no flags Details | Diff
patch #2 (5.53 KB, patch)
2008-08-20 07:06 EDT, Aleksandra Wozniak CLA
no flags Details | Diff
patch #3 (27.54 KB, patch)
2008-08-28 05:34 EDT, Aleksandra Wozniak CLA
no flags Details | Diff
patch #4 (2.22 KB, patch)
2008-11-15 20:21 EST, Aleksandra Wozniak CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Aleksandra Wozniak CLA 2008-08-11 09:19:06 EDT
Currently one can add a workspace file to "Compare with Other" dialog by two ways:
* by dragging it from a view and dropping it into the dialog
* by manually entering its relative path (workspace path).
It would be nice if is was possible to choose a file from "Open Resource" dialog and pass it to "Compare with Other" dialog.
Comment 1 Aleksandra Wozniak CLA 2008-08-11 09:23:45 EDT
Created attachment 109662 [details]
patch #1

The simplest solution.
Comment 2 Tomasz Zarna CLA 2008-08-12 08:35:54 EDT
I know you're planning to add other buttons to the dialog, but at this moment wouldn't simple "Browse..." be enough?
Comment 3 Tomasz Zarna CLA 2008-08-19 10:50:17 EDT
Ola, could you update the patch? Or do you want me to do it?
Comment 4 Aleksandra Wozniak CLA 2008-08-19 11:31:47 EDT
Today it will be done.
Comment 5 Aleksandra Wozniak CLA 2008-08-19 18:11:25 EDT
Since a small but important bug was found (see bug 244623) update will be done when 244623 is resolved. 
Comment 6 Aleksandra Wozniak CLA 2008-08-20 07:06:22 EDT
Created attachment 110434 [details]
patch #2
Comment 7 Aleksandra Wozniak CLA 2008-08-28 05:34:02 EDT
Created attachment 111179 [details]
patch #3

Based on latest changes from bug 73923. Browsing workspace via OpenResourceDialog is now possible.
Comment 8 Tomasz Zarna CLA 2008-09-01 06:32:24 EDT
Click Browse... for the Workspace option and then click Cancel on the dialog. You'll get the following error:

java.lang.NullPointerException
	at org.eclipse.compare.internal.CompareWithOtherResourceDialog$WorkspaceContent.setResource(CompareWithOtherResourceDialog.java:308)
	at org.eclipse.compare.internal.CompareWithOtherResourceDialog$1.widgetSelected(CompareWithOtherResourceDialog.java:275)
	at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:228)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1003)
	at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:3841)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3439)
	at org.eclipse.jface.window.Window.runEventLoop(Window.java:825)
	at org.eclipse.jface.window.Window.open(Window.java:801)
	at org.eclipse.compare.internal.ResourceCompareInput.showCompareWithOtherResourceDialog(ResourceCompareInput.java:238)
	at org.eclipse.compare.internal.ResourceCompareInput.setSelection(ResourceCompareInput.java:208)
	at org.eclipse.compare.internal.CompareAction.run(CompareAction.java:37)
	at org.eclipse.compare.internal.CompareWithOtherResourceAction.run(CompareWithOtherResourceAction.java:28)
	at org.eclipse.compare.internal.BaseCompareAction.run(BaseCompareAction.java:26)
	at org.eclipse.ui.internal.handlers.ActionDelegateHandlerProxy.execute(ActionDelegateHandlerProxy.java:289)
	at org.eclipse.core.commands.Command.executeWithChecks(Command.java:476)
	at org.eclipse.core.commands.ParameterizedCommand.executeWithChecks(ParameterizedCommand.java:508)
	at org.eclipse.ui.internal.handlers.HandlerService.executeCommand(HandlerService.java:169)
	at org.eclipse.ui.internal.keys.WorkbenchKeyboard.executeCommand(WorkbenchKeyboard.java:472)
	at org.eclipse.ui.internal.keys.WorkbenchKeyboard.press(WorkbenchKeyboard.java:824)
	at org.eclipse.ui.internal.keys.WorkbenchKeyboard.processKeyEvent(WorkbenchKeyboard.java:882)
	at org.eclipse.ui.internal.keys.WorkbenchKeyboard.filterKeySequenceBindings(WorkbenchKeyboard.java:571)
	at org.eclipse.ui.internal.keys.WorkbenchKeyboard.access$3(WorkbenchKeyboard.java:512)
	at org.eclipse.ui.internal.keys.WorkbenchKeyboard$KeyDownFilter.handleEvent(WorkbenchKeyboard.java:127)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
	at org.eclipse.swt.widgets.Display.filterEvent(Display.java:1187)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1002)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1027)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1012)
	at org.eclipse.swt.widgets.Control.traverse(Control.java:3499)
	at org.eclipse.swt.widgets.Control.translateMnemonic(Control.java:3343)
	at org.eclipse.swt.widgets.Composite.translateMnemonic(Composite.java:1064)
	at org.eclipse.swt.widgets.Control.translateMnemonic(Control.java:3361)
	at org.eclipse.swt.widgets.Display.translateMnemonic(Display.java:4362)
	at org.eclipse.swt.widgets.Display.filterMessage(Display.java:1201)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3435)
	at org.eclipse.ui.internal.Workbench.runEventLoop(Workbench.java:2382)
	at org.eclipse.ui.internal.Workbench.runUI(Workbench.java:2346)
	at org.eclipse.ui.internal.Workbench.access$4(Workbench.java:2198)
	at org.eclipse.ui.internal.Workbench$5.run(Workbench.java:493)
	at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:288)
	at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:488)
	at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:149)
	at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:113)
	at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:193)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:110)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:79)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:370)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:179)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:549)
	at org.eclipse.equinox.launcher.Main.basicRun(Main.java:504)
	at org.eclipse.equinox.launcher.Main.run(Main.java:1236)
	at org.eclipse.equinox.launcher.Main.main(Main.java:1212)
Comment 9 Tomasz Zarna CLA 2008-10-01 04:57:44 EDT
*** Bug 248300 has been marked as a duplicate of this bug. ***
Comment 10 Aleksandra Wozniak CLA 2008-11-15 20:21:24 EST
Created attachment 117987 [details]
patch #4

Synchronized with latest changes in repository.
Comment 11 Eclipse Webmaster CLA 2019-09-06 15:29:42 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.