Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't find anything from Wu-tang #343

Closed
hboetes opened this issue Jul 2, 2020 · 13 comments · Fixed by #368
Closed

Can't find anything from Wu-tang #343

hboetes opened this issue Jul 2, 2020 · 13 comments · Fixed by #368

Comments

@hboetes
Copy link
Member

hboetes commented Jul 2, 2020

I just searched for Legend Of The Wu-Tang which gave no results. Then I searched for Legend Of The Wu Tang — without the dash — and I did get results. What's going on here?

@hboetes hboetes added the bug label Jul 2, 2020
@mathiascode
Copy link
Member

I don't think there's much that can be done here. The same thing happens in the official SoulseekQt client.

@hboetes
Copy link
Member Author

hboetes commented Jul 2, 2020

Would it be an idea to replace - with spaces automatically? Maybe even replace some other characters like / and .

@mathiascode
Copy link
Member

I'm not sure if it's a good idea, considering how much the results seem to change depending on which characters you include.

@hboetes
Copy link
Member Author

hboetes commented Jul 2, 2020

Right, it's used to modify search results. https://www.slsknet.org/news/node/2855 as in search for everything wu but omit everything with tang. Rather sloppy coding if you'd ask me (on the slsk server side) since there is a clear difference between search -omit and jay-z, and any decent coder would fix that. Sigh closed source. :-(

Perhaps warn users if you'd search for [A-z]-[A-z]

@mathiascode
Copy link
Member

mathiascode commented Jul 7, 2020

@hboetes Could you test this PR? #368 (seems like SoulseekQt clients don't send search results if search terms include special characters, might be worth asking Nir what's up here)

git clone -b searchfilter https://github.com/mathiascode/nicotine-plus

@zniavre
Copy link
Contributor

zniavre commented Jul 7, 2020 via email

@mathiascode
Copy link
Member

mathiascode commented Jul 7, 2020

@zniavre Non-SoulseekQt clients send results if you include special characters, but SoulseekQt does not. If you search for something with a dash or apostrophe, you should notice that there are no results from SoulseekQt clients (their folders start with @@). This seems to be the behavior when searching inside the SoulseekQt client too.

@hboetes
Copy link
Member Author

hboetes commented Jul 7, 2020

I just searched for wu-tang with your branch and repo and I got results (a few albums) then I searched with the master branch and repo and I got different results( a few loose tracks).

I like your version better. :-)

@mathiascode
Copy link
Member

You can toggle the character filtering in settings -> searches, if desired. Searches without special characters in them should essentially be the same as searches with them, just with the addition of results from SoulseekQt clients.

@mathiascode
Copy link
Member

Excluding words by placing - in front of them should work as long as they are clearly separated words.

@hboetes
Copy link
Member Author

hboetes commented Jul 7, 2020

Yes, I like a lot. Could you sort search results by speed by default? :-) Or at least make clicking it sort decreasing instead of increasing? I always have to press that button twice.

slsk also sorts by speed by default.

@mathiascode
Copy link
Member

mathiascode commented Jul 8, 2020

I will defer the sorting change for now, since I had issues where the list would jump around a lot if you started scrolling when elements were still added.

@hboetes
Copy link
Member Author

hboetes commented Jul 8, 2020

OK, I'll open a new ticket for it, so we won't forget.

Man I really love how don't have to worry about if a special char is accepted or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

3 participants