isMobile doesn't cache matches

[npelov]

isMobile doesn't cache matches

[serbanghita]

Is there any reason isMobile() method doesn't cache matches in matchDetectionRulesAgainstUA()?

This is a mistake on my part, I should have cached this, but probably at that point I realized that my cache key should be based on userAgent string value. In matchUAAgainstKey I used the name of the method which is probably not that great to begin with. I probably avoided to create a cache key based on User Agent string + HTTP headers array

If you call isMobile() and then isIphone it'll run match again while it could be cached.
Also if you run isIPhone() the result will be cached, but then if you run isMoible() the cache won't be used.

As a side note - I'm not sure why match() method is public. it is not useful since by itself it doesn't detect anything.

Should be protected, probably I was thinking to allow people to use custom regexes.
Might be a good idea to make it protected and then whoever extends MobileDetect can work with it.
This is clearly a bug. setUserAgent resets the $cache.
I will fix this and cache based on User Agent string + HTTP headers array

[serbanghita]

I have fixed this in https://github.com/serbanghita/Mobile-Detect/releases/tag/4.8.01
This is a major refactor, I added support for PSR-16 cache

@npelov Nikolay, thank you for the detailed write-up, it helped me get motivated to release this new version with fixes.