[#42474] - [4.4] Update Browser.php
- Pending
- Medium
- Build: 4.4-dev
- # 42474
- Diff
- BrainforgeUK:patch-22
- Pending continuous-integration/drone/pr Build is pending Details
User tests: Successful: Unsuccessful:
Now uses strpos instead of preg_match.
Summary of Changes
In isRobot() use strpos() instead of preg_match() as that is faster.
See https://www.php.net/manual/en/function.preg-match.php#refsect1-function.preg-match-notes
Testing Instructions
A simply test available in this plugin.
bfisrobottest.zip
Install and on enabling the front-end displays the following on success:
Test (false):
Test (true):1
Test (true):1
Actual result BEFORE applying this Pull Request
Robots detected.
Expected result AFTER applying this Pull Request
Robots detected.
Its a big list of robots - possibly a detectable performance improvement,
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Labels |
Added:
PR-4.4-dev
|
||
Please rebase for v5.1 as this is B/C break when extending the Browser class, and using $robots in it. Thanks.
I have tested this item ✅ successfully on 932b5cf
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42474.
I have tested this item ✅ successfully on 932b5cf
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42474.
The PR needs to be rebased, so I can't set RTC. See previous comment #42474 (comment) .
If we don't use already a 3rd party library to maintain our browser detection we should do this, for example https://github.com/hisorange/browser-detect (just the first I found not validated if it's useful)
| Title |
|
||||||
Lines 87, 95, 166, 250, 268, 391 should be changed.