Changeset 1541 for FCKeditor/trunk/editor/dialog/fck_replace.html

Timestamp:

2008-02-18 04:14:39 (2 years ago)

Author:

martinkou

Message:

Fixed #1828 : The Find/Replace dialog will no longer display wrong starting positions for the match when there are multiple and identical characters preceding the character at the real starting point of the match.

Files:

• FCKeditor/trunk/editor/dialog/fck_replace.html (modified) (5 diffs)

FCKeditor/trunk/editor/dialog/fck_replace.html

@@ -98 +98 @@
 function btnStat()
 {
-        document.getElementById('btnReplace').disabled =
-                document.getElementById('btnReplaceAll').disabled =
-                        document.getElementById('btnFind').disabled =
-                                ( document.getElementById(idMap["FindText"]).value.length == 0 ) ;
+        GetE('btnReplace').disabled =
+                GetE('btnReplaceAll').disabled =
+                        GetE('btnFind').disabled =
+                                ( GetE(idMap["FindText"]).value.length == 0 ) ;
 }
 
@@ -111 +111 @@
 function GetSearchString()
 {
-        return document.getElementById(idMap['FindText']).value ;
+        return GetE(idMap['FindText']).value ;
 }
 
 function GetReplaceString()
 {
-        return document.getElementById("txtReplace").value ;
+        return GetE("txtReplace").value ;
 }
 
 function GetCheckCase()
 {
-        return !! ( document.getElementById(idMap['CheckCase']).checked ) ;
+        return !! ( GetE(idMap['CheckCase']).checked ) ;
 }
 
 function GetMatchWord()
 {
-        return !! ( document.getElementById(idMap['CheckWord']).checked ) ;
+        return !! ( GetE(idMap['CheckWord']).checked ) ;
 }
 
@@ -227 +227 @@
 // Knuth-Morris-Pratt Algorithm for stream input
 KMP_NOMATCH = 0 ;
-KMP_ADVANCED = 1 ;
-KMP_MATCHED = 2 ;
+KMP_STARTED = 1 ;
+KMP_ADVANCED = 2 ;
+KMP_MATCHED = 3 ;
 function KmpMatch( pattern, ignoreCase )
 {
@@ -263 +264 @@
                                         return KMP_MATCHED;
                                 }
-                                return KMP_ADVANCED;
+                                return this._State > 1 ? KMP_ADVANCED : KMP_STARTED ;
                         }
                         else if ( this._State == 0 )
@@ -307 +308 @@
                                 matchState = matcher.FeedCharacter(data) ;
 
+                                // No possible match of any useful substring in the pattern for the currently scanned character.
+                                // So delete any positional information.
                                 if ( matchState == KMP_NOMATCH )
                                         matchBookmark = null ;
-                                else if ( matchState == KMP_ADVANCED && matchBookmark == null )
+                                // The currently scanned character is a possible start, so mark down the starting position.
+                                else if ( matchState == KMP_STARTED )
                                         matchBookmark = { Start : cursor.concat( [] ) } ;
+                                // Found a complete match! Mark down the ending position as well.
                                 else if ( matchState == KMP_MATCHED )
                                 {
+                                        // It is possible to get a KMP_MATCHED without KMP_STARTED when the match pattern is only 1 character.
+                                        // So need to check and mark down the starting position as well.
                                         if ( matchBookmark == null )
                                                 matchBookmark = { Start : cursor.concat( [] ) } ;
+
                                         matchBookmark.End = cursor.concat( [] ) ;
                                         matchBookmark.End[ matchBookmark.End.length - 1 ]++;
 
                                         // Wait, do we have to match a whole word?
+                                        // If yes, carry out additional checks on what we've got.
                                         if ( GetMatchWord() )
                                         {