Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Music search returns 0 results with illegal characters #40

Closed
reallistic opened this issue Aug 6, 2013 · 5 comments
Closed

Music search returns 0 results with illegal characters #40

reallistic opened this issue Aug 6, 2013 · 5 comments

Comments

@reallistic
Copy link

When xdm performs a search that contains illegal characters (.,/!') it does not pick these guys up. Seems as though the index sites do not allow these. For example, I dug through the log to find an api call on an indexer site searching for "The 20/20 Experience" and it returned 0 results. I took the exact url and plugged it directly into my browser and got the same result. I then took that url and removed the forward slash and it returned the expected results. The same occurred for "Magna Carta... Holy Grail."

@lad1337
Copy link
Owner

lad1337 commented Aug 6, 2013

mmhh i hope the / was url encoded if not this is a bug.
but as you explained the the search returned results from teh indexer after removing the /
and i guess replacing it with a space
you could use the Replace plugin (its a SearchTermFilter and should be in the core, i think)

the setting could look like this:
Replace This:

&;:;é;';/;!

With This:

; ;e;; ; 

(Note: there is a space behind the last ;)
of course you must let it run for music and enable it

this created this in the logs for me

Final search term set for Album(de.lad1337.music)[7772] The 20/20 Experience is: [u'Justin Timberlake The 20/20 Experience', u'Justin Timberlake The 20 20 Experience']

maybe this helps.
please give it a try

@reallistic
Copy link
Author

Sorry forgot to add that it was URL encoded. Also, I cannot remember what happened (or if I tried) when I used a space but the actual index sites had the 20/20 experience listed as 2020 experience.

I will try the replace plugin.

@lad1337
Copy link
Owner

lad1337 commented Aug 6, 2013

so the indexer had a replace rule too ... replace / with nothing ... you will of course have to do that too

@reallistic
Copy link
Author

This worked. I used:

&;:;é;';/;!;.
; ;e;;; ;

@lad1337
Copy link
Owner

lad1337 commented Aug 6, 2013

okay cool :)
i'll update the default settings in the replace plugin to also replace / with nothing
so future users don't have to add that

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants