Subject: | It should return a boolean instead... |
Summary: | Package rating comment |
Messages: | 3 |
Author: | André Karell Contreras Jiménez |
Date: | 2015-08-17 03:00:22 |
|
|
|
André Karell Contreras Jiménez rated this package as follows:
Utility: | Bad |
Consistency: | Bad |
Examples: | Sufficient |
|
 André Karell Contreras Jiménez - 2015-08-17 03:00:22
It should return a boolean instead of a string. The format lack of consistency and while it could technically identify some bots, it should have more test to identify if it is a bot or not, maybe a better tool could be a JS oriented one, but server-side can do more than just search for 'bot' in the user agent.
 Ruben Elizondo - 2015-08-20 20:11:00 - In reply to message 1 from André Karell Contreras Jiménez
This little function works great for me....thats the reason i decided to share here....yes it should be a boolean....but it works this way....remember son....the simple, the better
 John Haywood - 2015-08-21 09:20:09 - In reply to message 2 from Ruben Elizondo
If you wanted simple, then you could have used a function which would then only be loaded when called, instead of the Class taking up memory all of the time (especially if auto-loaded).
|