| 
| 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:22It 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énezThis 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 ElizondoIf 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). |