Bug when cache is set to false and cacheKey omitted in the options about jquery-ajax-localstorage-cache HOT 2 CLOSED

Good catch, although I have to admit to some curiosity as to what circumstances might lead a developer to disable browser-level caching, but enable caching in local storage.

In any case, it looks like your proposed regex is a bit too specific - it assumes the _ parameter that jQuery appends will appear as the first parameter, which is not necessarily the case. Let's instead use the same regex that jQuery uses to detect the presence of the _ param:

/([?&])_=[^&]*/

If you'd like to make a pull request with the relevant change, I'll review and merge it.

from jquery-ajax-localstorage-cache.

Thanks for the regex tip. I managed to locate it in ajax.js for those wondering where you found it. The capture group isn't not really needed, but I guess we should be consistent with jQuery's version.

from jquery-ajax-localstorage-cache.