Bug 243744

Summary: [Dialogs] Adding content to "Compare with Other" dialog from Open Resource dialog
Product: [Eclipse Project] Platform Reporter: Aleksandra Wozniak <aleksandra.k.wozniak>
Component: CompareAssignee: Platform-Compare-Inbox <platform-compare-inbox>
Status: ASSIGNED --- QA Contact:
Severity: enhancement    
Priority: P3 CC: aleksandra.k.wozniak, benno.baumgartner
Version: 3.4   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 73923, 239959, 244623    
Bug Blocks: 224562    
Attachments:
Description Flags
patch #1
none
patch #2
none
patch #3
none
patch #4 none

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.