Opened 14 years ago
Closed 14 years ago
#5794 closed Bug (fixed)
Second search with no result leads to a JS error
| Reported by: | Sa'ar Zac Elias | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3.1 |
| Component: | General | Version: | 3.2.2 |
| Keywords: | Confirmed Review+ | Cc: |
Description
Steps to reproduce
- Load a sample or the demo.
- Search for something that does not exist, twice.
Notice that there is an error: startNode is null; _source/core/dom/range.js L1386.
Regression of 3.2.2.
Attachments (2)
Change History (6)
comment:1 Changed 14 years ago by
| Keywords: | HasPatch added |
|---|
Changed 14 years ago by
| Attachment: | 5794.patch added |
|---|
comment:2 Changed 14 years ago by
| Keywords: | Confirmed Review? added; HasPatch removed |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
The regression is correct, while let's place the fix to the actual bug as range::setStart doesn't suppose to protect against invalid nodes.
Changed 14 years ago by
| Attachment: | 5794_2.patch added |
|---|
comment:3 Changed 14 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Note: See
TracTickets for help on using
tickets.
Regression of [5463].