Closed Bug 6544 Opened 25 years ago Closed 25 years ago

Cannot search on XML page

Categories

(Core :: XML, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: hjtoi-bugzilla, Assigned: kinmoz)

References

Details

Attachments

(1 file)

You cannot search on XML page because in nsTextServicesDocument::GetBodyNode we are looking for an element called "body", which is generally not present in an XML document. I believe this could be fixed by changing the above method to retrieve the body node for HTML documents ONLY, and in other cases get the root node. You should be able to QI the mDOMDocument to see if it is nsIHTMLDocument. If it was not, just return the document's root node. Note that it would probably be a good idea to change the method's name as well, because we would be getting other tags as well for XML docs.
Assignee: mjudge → kin
this seems like kins back yard so i will let him clean up the mess ;) Boing! its in your court now kin
Status: NEW → ASSIGNED
Target Milestone: M7
QA Contact: 4130 → 4110
QA Contact: 4110 → 4130
Changing qa contact to claudius@netscape.com because the issue is 'Selection and Search' - just on a different type of document.
Target Milestone: M7 → M8
I'll revisit this in milestone M8.
Per a request from Selection and Search component eng (mjudge) and qa (elig), moving all "Selection and Search" bugs to new "Selection" component. Original "Selection and Search" component will be retired.
Component: Selection → XML
[Changing component to XML; Claudius, please reassign if you believe this assignation to be erroneous. Thanks!]
Changing milestone to M9 since I don't have time to look at spellchecker issues yet.
Blocks: 9184
Moving all spellchecker/text services related bugs to M11 since I will still be working on AutoComplete features through M10.
Moving some SpellChecker/TextServices feature requests and bugs, that can wait till after beta 1, to milestone M14.
Keywords: patch
*** Bug 20622 has been marked as a duplicate of this bug. ***
*** Bug 18566 has been marked as a duplicate of this bug. ***
I checked in a fix for this: mozilla/editor/txtsvc/src/nsTextServicesDocument.cpp revision 1.23 mozilla/editor/txtsvc/src/nsTextServicesDocument.h revision 1.8 Fix should appear in the 01/27/2000 QA builds.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
I also reviewed the code, and tried searching on an XML page. Works, so marking verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: