-
David Mudrák authored
The previous 2.9 implementation of is_web_crawler() used stripos() in certain cases. The unit tests re-added in the previous commit revealed that certain crawlers (such as BaiDuSpider) were not correctly detected in the new refactored implementation. It seems lesser evil and safe enough to use /i in the regex search even though it is not 100% same logic as before - as stripos() was used in some cases only, not always.
David Mudrák authoredThe previous 2.9 implementation of is_web_crawler() used stripos() in certain cases. The unit tests re-added in the previous commit revealed that certain crawlers (such as BaiDuSpider) were not correctly detected in the new refactored implementation. It seems lesser evil and safe enough to use /i in the regex search even though it is not 100% same logic as before - as stripos() was used in some cases only, not always.
Loading