search

ksyasuda / aniwrapper

A rofi wrapper around my fork of ani-cli: a command-line tool to stream, download, and play anime
GNU General Public License v3.0
19 stars 4 forks source link

Fixed multiword searches #10

Closed AdalynBlack closed 2 years ago

AdalynBlack commented 2 years ago

This is a simple fix that fixes #9 by using the same method used by ani-cli to filter out spaces when searching for anime

ksyasuda commented 2 years ago

Thanks for brining this up and taking a shot at it. While this solution does get the search query in the desired format, I believe the fix should be applied earlier so that the whole program will run with the correct anime_id

Since the search_anime function is called from process_search, in this solution, there is a portion of the program that will still run with an incorrect anime id (squashed together instead of separated by dashes). The reason this is an issue is that with the anime_id not in the correct format, we won't be able to successfully query and utilize the history database

image

See #11 for my solution, which addresses the issue after getting user input