Find & replace is not working properly with Tables in the content.

[Satya Minnekanti]

To reproduce the defect:

1. Open Ajax sample.

2. Type the word test 2 times and then insert a Table with 3 rows and 2 columns.

3. Type the word test in all Tables Table cells

4. Open find and replace dialog, go to replace tab Enter the word(tt) in Find what field and word(xx) in Replace with field

5. click on Replace button.

Expected result:

An error dialog should come up saying "The specified text was not found".

Actual Result:

It is highlighting last Char 't' in the last word before the table and first character 't' in the first word in first Table cell.

6. Click on Replace button again.

It will create a new word xx in the new paragraph above the Table

7. Click on Replace button again.

It is highlighting last Char 't' in the word in first Table cell and first character 't' in the first word in second Table cell.

8. Click on Replace button again.

It is adding a new cell in between the first two cells with a new word xx

9. Click on Replace button again

It is removing all Table cells and leaving just one Cell with word es

And when we Replace after that we are not getting any error dialog or any error.

if we repeat the above procedure by clicking on Replace All button we are getting a dialog saying 6 ocurrences replaced and it is adding anew cell between each Table Row and it is adding a new word xx above the table.

[Satya Minnekanti]

This is also happening when we have words in 2 different paragraphs.

Type the same text (test test test ) in 2 different paragraphs click on find and replace and enter 'tt' in the Find what field and enter 'xxx' in th Replace with field.

Try to press Replace button and see that it will highlight the lats 't' in first paragraph and first 't' in second paragraph and when we click replace button for second time it will create a new word 'xxx' in a new paragraph between 2 paragraphs.

[Sa'ar Zac Elias]

The problem is that we're not checking the boundary unless we use "match full word". It also happens with images. Fixing it is easy, but it might not be the expected behaviour so I think it'd be better to hear some more opinions. I'm attaching a patch for this approach meanwhile.

[Damian]

As I see it, the walker should not match across line breaks, cell boundaries etc. This seems just wrong. The majority of text editors/word processors that I've seen would not match words like this.

[Garry Yao]

It's definitely a bug as stated by damo and it's a regression of [5329].

[Garry Yao]

@Sarre, as you caught, the checking the boundary logic should only be performed as guard when checking a full match.

[Garry Yao]

Fixed with [5955].