Disabling liveSearch does not work

[kelesi]

In v2.25.3 setting "filter_liveSearch : false" does not disable liveSearch. It was working in v2.23.5.

[kelesi]

Seems like the issue is in

.bind( 'search change keypress '.split( ' ' ).join( namespace + ' ' ), function( event )

where this condition

if ( wo.filter_initialized && ( event.which === 13 || event.type === 'search' ||
    event.type === 'change' && this.value !== c.lastSearch[column] ) ) {

has been changed to this

if ( wo.filter_initialized && ( event.which === tskeyCodes.enter || event.type === 'search' ||
    // only "input" event fires in MS Edge when clicking the "x" to clear the search
   ( event.type === 'change' || event.type === 'input' ) && this.value !== c.lastSearch[column] ) ) {

When removing the added condition

event.type === 'input'

the disabling of liveSearch works.

Or replacing it with

( event.type === 'change' || (event.type === 'input' && wo.filter_liveSearch === true))

I am not sure why the event.type === 'input' condition has been added to the function, so I am not sure what is the correct way for fixing this.

[Mottie]

Hi @kelesi!

I'm looking into this now. Thanks for reporting the issue and doing some troubleshooting!

[Mottie]

Ok, I have a fix available in the master branch... I was trying to write a unit test, but sadly I've run out of time for messing around this morning 😉

[Mottie]

Version 2.25.4 is now available with this fix!

[kelesi]

Hi @Mottie,
Seems like disabling live search does not work again since v2.27.7.

[Mottie]

Hi @kelesi!

Sorry for the delay in responding, I've just returned from a much needed extended vacation.

It appears that the liveSearch setting was bypassed when an "input" event listener was included to fix issue #1280. I'll need to figure out the best way to deal with this new issue... either ignore the "input" event when the filter_liveSearch option is false, or check for a mobile device and only include in that case.