Hi,
I dedicated some time to this today and found that the solution proposed isn't exactly equivalent to the current functionality. As the global search is right now, it ignores case, in other words it is a case insensitive search, but with the change proposed it is converted into a case sensitive search so it doesn't return the same results.
I made a minor change to make the functionality equivalent but it makes the search work slower and since in the majority of installs it really isn't necessary I added a global variable so it can be activated only if needed.
https://github.com/tsolucio/corebos/commit/5fc4fd795e9b5bba0bca9b3649bf698c836f4b2a
Thank you very much for sharing :-)
I dedicated some time to this today and found that the solution proposed isn't exactly equivalent to the current functionality. As the global search is right now, it ignores case, in other words it is a case insensitive search, but with the change proposed it is converted into a case sensitive search so it doesn't return the same results.
I made a minor change to make the functionality equivalent but it makes the search work slower and since in the majority of installs it really isn't necessary I added a global variable so it can be activated only if needed.
https://github.com/tsolucio/corebos/commit/5fc4fd795e9b5bba0bca9b3649bf698c836f4b2a
Thank you very much for sharing :-)
Joe
TSolucio
TSolucio