Issue Details (XML | Word | Printable)

Key: MBS-4033
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Oliver Charles
Reporter: Lasse Mattila
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
MusicBrainz Server

Offered and installed search plugin name mismatch

Created: 20/Dec/11 07:57 AM   Updated: 16/Oct/12 07:55 PM   Resolved: 16/Oct/12 07:55 PM
Component/s: None
Affects Version/s: None
Fix Version/s: Schema change, 2012-10-15

Environment: For example: Windows, Firefox


 Description  « Hide

MusicBrainz.org offers browser search plugins with a name pattern like "MusicBrainz: Artist". In Mozilla Firefox, these offerings are displayed in the browser's search field drop down menu. If one clicks for example the aforementioned plugin, it gets installed but with the string " (indexed)" appended at the end of its name.

Usually, when a search plugin is installed, it is then hidden from the list of offered plugins. I believe this is done by matching the names but since the names don't match, it results in useless bloating of the menu.

So, it's more of a cosmetic issue than a functionality one. And in the same vein, a suggested fix would be removing the " (indexed)" part from the search plugin XML files. At least I'm not aware of any added value it might have.

(This used to be easily fixed locally but Firefox has for some time now been using cached copies of search plugins instead of the source XML files, so I thought getting to the source of the problem might be worth the while.)



Sort Order: Ascending order - Click to sort in descending order
Nicolás Tamargo added a comment - 20/Dec/11 08:08 AM

While I don't know about the rest, "indexed" is there to indicate it uses the indexed database search, as opposed to the direct one, see http://musicbrainz.org/search and http://wiki.musicbrainz.org/Indexed_Search_Syntax


Lasse Mattila added a comment - 20/Dec/11 08:38 AM

Ok, I thought that might be the case but I don't see the harm if there are no direct search plugins in the mix. Alternatively, I can live with "(indexed)" in the offered plugin names as well.


Aurélien Mino added a comment - 20/Dec/11 08:56 AM

Nicolás Tamargo: Most users just want to do a search.
They don't care about direct or indexed search, these are technical details, that are mainly useful for power users.


Paul Taylor added a comment - 20/Dec/11 10:54 AM

This isn't part of the Search Server codebase


Oliver Charles added a comment - 20/Dec/11 04:33 PM

I am happy only supporting indexed searches from this list.


Aurélien Mino added a comment - 20/Dec/11 07:40 PM

I agree with only supporting indexed searches.


Oliver Charles added a comment - 25/Sep/12 05:00 PM