Fix correctly setting Find-Scopes when opening Find/Replace-Dialog eclipse-platform#1819

This PR correctly disables SearchOptions.GLOBAL in the FindReplaceLogic
object of the Dialog if a multiline-selection was selected while opening
the Find/Replace-Dialog.

This PR also provides a unit-test for this behavior.

Fixes eclipse-platform#1819

Author: Maximilian Wittmer

bundles/org.eclipse.ui.workbench.texteditor/src/org/eclipse/ui/texteditor/FindReplaceDialog.java

@@ -946,6 +946,7 @@ private void initFindStringFromSelection() {
 		} else {
 			fGlobalRadioButton.setSelection(false);
 			fSelectedRangeRadioButton.setSelection(true);
+			findReplaceLogic.deactivate(SearchOptions.GLOBAL);
 		}
 	}
 

tests/org.eclipse.ui.workbench.texteditor.tests/src/org/eclipse/ui/workbench/texteditor/tests/FindReplaceDialogTest.java

@@ -13,6 +13,8 @@
  *******************************************************************************/
 package org.eclipse.ui.workbench.texteditor.tests;
 
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.Matchers.is;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
@@ -98,6 +100,8 @@ private class DialogAccess {
 
 		private Runnable closeOperation;
 
+		public Button replaceAllButton;
+
 
 		DialogAccess(Accessor findReplaceDialogAccessor, boolean checkInitialConfiguration) {
 			findReplaceLogic= (FindReplaceLogic) findReplaceDialogAccessor.get("findReplaceLogic");
@@ -110,6 +114,7 @@ private class DialogAccess {
 			incrementalCheckBox= (Button) findReplaceDialogAccessor.get("fIncrementalCheckBox");
 			regExCheckBox= (Button) findReplaceDialogAccessor.get("fIsRegExCheckBox");
 			replaceFindButton= (Button) findReplaceDialogAccessor.get("fReplaceFindButton");
+			replaceAllButton= (Button) findReplaceDialogAccessor.get("fReplaceAllButton");
 			shellRetriever= () -> ((Shell) findReplaceDialogAccessor.get("fActiveShell"));
 			closeOperation= () -> findReplaceDialogAccessor.invoke("close", null);
 			if (checkInitialConfiguration) {
@@ -426,6 +431,19 @@ public void testActivateWholeWordsAndSearchForTwoWords() {
 		assertFalse(dialog.wholeWordCheckBox.getEnabled());
 	}
 
+	@Test
+	public void testActivateDialogWithSelectionActive() {
+		openTextViewer("text" + System.lineSeparator() + "text" + System.lineSeparator() + "text");
+		fTextViewer.setSelectedRange(4 + System.lineSeparator().length(), 8 + System.lineSeparator().length());
+		openFindReplaceDialogForTextViewer(false);
+
+		assertFalse(dialog.globalRadioButton.getSelection());
+		dialog.findCombo.setText("text");
+		select(dialog.replaceAllButton);
+
+		assertThat(fTextViewer.getDocument().get(), is("text" + System.lineSeparator() + System.lineSeparator()));
+	}
+
 	@Test
 	public void testIncrementalSearchOnlyEnabledWhenAllowed() {
 		openTextViewer("text text text");