search

I-A-C / ExodusReduxRepo

GNU General Public License v3.0
29 stars 13 forks source link

Kodi 18 , Search Fix for Exodus Forks with working History #12

Closed SerpentDrago closed 6 years ago

SerpentDrago commented 6 years ago

https://old.reddit.com/r/Addons4Kodi/comments/9kgpau/alternative_xbox_one_kodi_18_search_fix/?context=10

As explained here

This fully fix's search not working in Kodi 18 Leia . and allows Search history

doko-desuka commented 6 years ago

The navigator.py pull removed function clearCacheAll, and now there's two copies of clearCacheSearch. Can you fix that? https://github.com/I-A-C/ExodusReduxRepo/pull/12/files#diff-583f8d2681f8c890f2dff4cc4a34ea91L331

SerpentDrago commented 6 years ago

umm i only see one ...

doko-desuka commented 6 years ago

I was linking to the master file, which keeps changing. I'll link to branches from now on, it's safer (as they're "frozen in time"). You can see the two repeated clearCacheSearch() functions here: https://github.com/SerpentDrago/ExodusReduxRepo/blob/20f51f6e31174a23b7e31e70a90ff3c22232ba57/script.module.exodusredux/lib/resources/lib/indexers/navigator.py#L323-L336

The original clearSearchAll() is in here: https://github.com/SerpentDrago/ExodusReduxRepo/blob/6ac753f1c8f4cb41fef53ea147458ed678c47eb2/script.module.exodusredux/lib/resources/lib/indexers/navigator.py#L323-L337

It comes right after clearCacheSearch. It needs to be restored at that position (i.e a new pull-request for that needs to be done).