function stripos() is only available on PHP5 or higher

Helmut Debes - August 10, 2009 - 00:13
Project:AdSense
Version:6.x-1.2
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

This version of the module calls the function stripos(), which is only available on PHP5 or higher. Updating a site running with PHP 4 results in WSoD.

#1

kbahey - August 10, 2009 - 00:25

Note that PHP4 is obsolete now.

Joao,

If this is not easy to get around, we should add:

php = 5.2.x

In the .info file to prevent this module from being used for PHP4.

#2

jcnventura - August 10, 2009 - 09:07

yes, but drupal 6 itself is still php4-compliant. I will try to make the module php4 compliant again.

#3

jcnventura - August 19, 2009 - 16:14

I have figured out that I can use stristr() for the same purpose that the stripos function was being used. I will test to make sure it's working and I will try to commit it soon.

#4

jcnventura - September 1, 2009 - 16:30
Status:active» fixed

I've committed to CVS a change to use stristr instead of stripos.

#5

System Message - September 15, 2009 - 16:40
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.