[SOLUCIONADO] Problemas con array_shift explode en PHP 5.4.39

[AlexServ]

Me sale el error "PHP Strict Standards: Only variables should be passed by reference in contentads.php on line 167" esto ocurre desde que el servidor actualizó la versión de PHP, la linea 167 contiene $ip = array_shift(explode(',',$ip));

le dejo el código para que me puedan ayudar:

Código PHP:

  //function from CBreCaptcha
function GetUserIp() {
   $ip = "";
   if(isset($_SERVER)) {
       if (!empty($_SERVER['HTTP_CLIENT_IP'])) {
         $ip=$_SERVER['HTTP_CLIENT_IP'];
       } elseif (!empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
         $ip=$_SERVER['HTTP_X_FORWARDED_FOR'];
       } else {
         $ip=$_SERVER['REMOTE_ADDR'];
       }
    } else {
       if ( getenv( 'HTTP_CLIENT_IP' ) ) {
          $ip = getenv( 'HTTP_CLIENT_IP' );
       } elseif ( getenv( 'HTTP_X_FORWARDED_FOR' ) ) {
          $ip = getenv( 'HTTP_X_FORWARDED_FOR' );
       } else {
          $ip = getenv( 'REMOTE_ADDR' );
        }
    }
    // In some weird cases the ip address returned is repeated twice separated by comma
    if(strstr($ip,',')){
      $ip = array_shift(explode(',',$ip));
    }
   return $ip;
 }
} 
   

[hhs]

El problema es que explode no regresa una referencia y array_shift lo requiere para hacer su trabajo
la solución es sencilla; asigna el resultado de la función explode a una variable y esa variable pasala a la función array_shift.

Código PHP:

if(strstr($ip,',')){
      $stack = explode(',',$ip);
      $ip = array_shift($stack);
    }
 

Procura leer el manual con frecuencia para estar al día con los cambios y siempre mantener la versión mas reciente de php

[AlexServ]

Gracias, ahora si todo bien.