219825 - Remove search-db.rdf and use the index and contents to search instead.

Status: VERIFIED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[R.J. Keller]

search-db.rdf, according to oeschger, was used only to modify the search results
of the Help menu search. Mainly used for marketing. This file is not needed,
since any entries in this file should also be in the help index.

So, we should remove search-db.rdf, and add searching of the help index. Any
entries that should be searched for should be added to the index.

Comment 1

[R.J. Keller]

Attachment: Patch

Note that this patch includes removing search-db.rdf. Not sure how to say this
in a diff ;).

Comment 2

[R.J. Keller]

Comment on attachment 132267 [details] [diff] [review]
Patch

Neil, I'm not sure if you can review this, but it contains JS code related to
searching RDF. I assume you could review it, but not 100% sure. No actual RDF
in the patch.

Comment 3

[[e-mail]]

Comment on attachment 132267 [details] [diff] [review]
Patch

I think we shouldn't just remove the code for additional datasources, you
should just remove the reference to search-db.rdf from mozillahelp.rdf (?)

Comment 4

[R.J. Keller]

That's true. I'll make the change back.

Comment 5

[R.J. Keller]

Attachment: Patch with Neils comments

Comment 6

[[e-mail]]

Comment on attachment 132279 [details] [diff] [review]
Patch with Neils comments

>-                    nc:datasources="search-db.rdf"
>+                    nc:datasources=""
I've got too many of your patches installed so I can't try this yet, but I
think it should work if you just remove this line without replacing it.

Comment 7

[R.J. Keller]

> I've got too many of your patches installed so I can't try this yet, but I
> think it should work if you just remove this line without replacing it.

Yes, I was thinking of removing this line, but the Search worked fine (with no
JS errors) with this link being blank, and I was hoping that people could use
Mozilla Help as a reference for the help viewer.

There is no reference (at least to my knowledge) about how the help viewer
works, so I want Mozilla Help to demonstrate all of the options that a content
pack can use. This is there just so 3rd parties can know that it's there. If you
think that it's a waste, then I'll remove it.

Comment 8

[[e-mail]]

Hmm... what about changing that line to use the index, rather than coding it?

Comment 9

[R.J. Keller]

> Hmm... what about changing that line to use the index, rather than coding it?

That would work, but I would think that the index should be up by default and
not give a choice to anyone who wishes to use the Help viewer. I consider
searching the index equivalent to searching the TOC or Glossary, which is coded
into help.

Comment 10

[[e-mail]]

Comment on attachment 132279 [details] [diff] [review]
Patch with Neils comments

>-                    nc:datasources="search-db.rdf"
>+                    nc:datasources=""
OK, but I think this should be changed to rdf:null instead.

Comment 11

[Alec Flett]

Comment on attachment 132279 [details] [diff] [review]
Patch with Neils comments

sr=alecf

Comment 12

[R.J. Keller]

Fix checked in.

Comment 13

[Daniel Wang]

v

several regressions:
"what" returns "what is"
"related" returns "finding related"
neither link makes any sense. We need to clean our Index and Contents db.