From 06e8b6f369791eeb7cd735d692877e3ed5e0e8e6 Mon Sep 17 00:00:00 2001
From: "akkana%netscape.com" <akkana%netscape.com>
Date: Thu, 21 Mar 2002 02:37:58 +0000
Subject: [PATCH] 116405: Fix replace all, backwards case. r=jfrancis
 sr=sfraser a=asa

---
 editor/ui/dialogs/content/EdReplace.js | 68 +++++++++++++++++---------
 1 file changed, 44 insertions(+), 24 deletions(-)

diff --git a/editor/ui/dialogs/content/EdReplace.js b/editor/ui/dialogs/content/EdReplace.js
index dd04ab6032dd..d52fe6f7562a 100644
--- a/editor/ui/dialogs/content/EdReplace.js
+++ b/editor/ui/dialogs/content/EdReplace.js
@@ -227,7 +227,7 @@ function onReplaceAll()
 
   var finder = Components.classes["@mozilla.org/embedcomp/rangefind;1"].createInstance().QueryInterface(Components.interfaces.nsIFind);
 
-  finder.caseSensitive    = gReplaceDialog.caseSensitive.checked;
+  finder.caseSensitive = gReplaceDialog.caseSensitive.checked;
   finder.findBackwards = gReplaceDialog.searchBackwards.checked;
 
   // Make a range containing the current selection, 
@@ -244,13 +244,18 @@ function onReplaceAll()
   wholeDocRange.selectNodeContents(rootNode);
 
   // And start and end points:
-  var startPt = gEditor.document.createRange();
-  startPt.setStart(wholeDocRange.startContainer, wholeDocRange.startOffset);
-  startPt.setEnd(wholeDocRange.startContainer, wholeDocRange.startOffset);
-
   var endPt = gEditor.document.createRange();
-  endPt.setStart(wholeDocRange.endContainer, wholeDocRange.endOffset);
-  endPt.setEnd(wholeDocRange.endContainer, wholeDocRange.endOffset);
+
+  if (gReplaceDialog.searchBackwards.checked)
+  {
+    endPt.setStart(wholeDocRange.startContainer, wholeDocRange.startOffset);
+    endPt.setEnd(wholeDocRange.startContainer, wholeDocRange.startOffset);
+  }
+  else
+  {
+    endPt.setStart(wholeDocRange.endContainer, wholeDocRange.endOffset);
+    endPt.setEnd(wholeDocRange.endContainer, wholeDocRange.endOffset);
+  }
 
   // Find and replace from here to end of document:
   var foundRange;
@@ -267,25 +272,40 @@ function onReplaceAll()
     selecRange = selection.getRangeAt(0);
   }
 
-  // If wrapping, find from start of document back to start point.
-  selecRange.setStart(wholeDocRange.startContainer, wholeDocRange.startOffset);
-  selecRange.setEnd(wholeDocRange.startContainer, wholeDocRange.startOffset);
-  // Collapse origRange to start:
-  origRange.setEnd(origRange.startContainer, origRange.startOffset);
-  if (gReplaceDialog.wrap.checked)
+  // If no wrapping, then we're done
+  if (!gReplaceDialog.wrap.checked)
+    return;
+
+  // If wrapping, find from start/end of document back to start point.
+  if (gReplaceDialog.searchBackwards.checked)
   {
-    while ((foundRange = finder.Find(findStr, wholeDocRange,
-                                     selecRange, origRange)) != null)
-    {
-      gEditor.selection.removeAllRanges();
-      gEditor.selection.addRange(foundRange);
-      gEditor.insertText(repStr);
-      selection = gEditor.selection;
-      if (selection.rangeCount <= 0) {
-        return;
-      }
-      selecRange = selection.getRangeAt(0);
+    // Collapse origRange to end
+    origRange.setStart(origRange.endContainer, origRange.endOffset);
+    // Set current position to document end
+    selecRange.setEnd(wholeDocRange.endContainer, wholeDocRange.endOffset);
+    selecRange.setStart(wholeDocRange.endContainer, wholeDocRange.endOffset);
+  }
+  else
+  {
+    // Collapse origRange to start
+    origRange.setEnd(origRange.startContainer, origRange.startOffset);
+    // Set current position to document start
+    selecRange.setStart(wholeDocRange.startContainer,
+                        wholeDocRange.startOffset);
+    selecRange.setEnd(wholeDocRange.startContainer, wholeDocRange.startOffset);
+  }
+
+  while ((foundRange = finder.Find(findStr, wholeDocRange,
+                                   selecRange, origRange)) != null)
+  {
+    gEditor.selection.removeAllRanges();
+    gEditor.selection.addRange(foundRange);
+    gEditor.insertText(repStr);
+    selection = gEditor.selection;
+    if (selection.rangeCount <= 0) {
+      return;
     }
+    selecRange = selection.getRangeAt(0);
   }
 }