PHP 7 a PHP 8

M

MissFortune #1 Jul '22

Buenas,

Me han encargado un proyecto que de momento no sé por donde cogerlo....

El enunciado es fácil: tengo una serie de archivos php en la versión 7 que tengo que pasar a la versión 8.

Para hacerlo más fácil, me he descargado el PHPStorm, pero no consigue detectarme muchos errores por no decir ninguno. He leído mucha documentación al respecto, pero no sé aplicar los conocimientos al proyecto. Para mí, todo está bien y no modificaría nada, pero hay cosas que seguramente se pueden mejorar a PHP 8.

No se si puedo pasar un ejemplo del código aquí y ver si alguien me podría dar un par de indicaciones que podría modificar para que sea PHP 8.

spoiler

<?php
error_reporting( E_ALL & ~( E_NOTICE | E_STRICT | E_DEPRECATED ) );
require_once "/usr/share/php/Mail.php";

header ('Content-type: text/html; charset=utf-8');

$nom = $_POST['nom'];
$email = $_POST['email'];
$tel = $_POST['tel'];
$num = $_POST['num'];
$evento = $_POST['evento'];
$modo = $_POST['modo'];

$chica1 = $_POST['CHICA1'];
$chica2 = $_POST['CHICA2'];
$chica3 = $_POST['CHICA3'];
$chica4 = $_POST['CHICA4'];
$chica5 = $_POST['CHICA5'];
$chica6 = $_POST['CHICA6'];
$chica7 = $_POST['CHICA7'];
$chica8 = $_POST['CHICA8'];
$chica9 = $_POST['CHICA9'];
$chica10 = $_POST['CHICA10'];
$chica11 = $_POST['CHICA11'];
$chica12 = $_POST['CHICA12'];
$chica13 = $_POST['CHICA13'];
$chica14 = $_POST['CHICA14'];
$chica15 = $_POST['CHICA15'];
$chica16 = $_POST['CHICA16'];
$chica17 = $_POST['CHICA17'];
$chica18 = $_POST['CHICA18'];
$chica19 = $_POST['CHICA19'];
$chica20 = $_POST['CHICA20'];
$chica21 = $_POST['CHICA21'];
$chica22 = $_POST['CHICA22'];
$chica23 = $_POST['CHICA23'];
$sugerencias = $_POST['sugerencias'];
$tchica1 = $_POST['tCHICA1'];
$tchica2 = $_POST['tCHICA2'];
$tchica3 = $_POST['tCHICA3'];
$tchica4 = $_POST['tCHICA4'];
$tchica5 = $_POST['tCHICA5'];
$tchica6 = $_POST['tCHICA6'];
$tchica7 = $_POST['tCHICA7'];
$tchica8 = $_POST['tCHICA8'];
$tchica9 = $_POST['tCHICA9'];
$tchica10 = $_POST['tCHICA10'];
$tchica11 = $_POST['tCHICA11'];
$tchica12 = $_POST['tCHICA12'];
$tchica13 = $_POST['tCHICA13'];
$tchica14 = $_POST['tCHICA14'];
$tchica15 = $_POST['tCHICA15'];
$tchica16 = $_POST['tCHICA16'];
$tchica17 = $_POST['tCHICA17'];
$tchica18 = $_POST['tCHICA18'];
$tchica19 = $_POST['tCHICA19'];
$tchica20 = $_POST['tCHICA20'];
$tchica21 = $_POST['tCHICA21'];
$tchica22 = $_POST['tCHICA22'];
$tchica23 = $_POST['tCHICA23'];


$to = '[email protected]'; 
$from = '';
$host     = '';
$username = '';
$password = '';
$subject = 'Confirmaci�n votaci�n Viernes'; 
$body = "Buenos d�as,\n".
"  \n".
"  \n".
"  \n".
"Saludos,\n".
"Jordi\n".
"  \n".
"---------------\n".
"Nombre: ".utf8_decode($nom)."\n".
"Soy el Num.: ".$num."\n".
"Email: ".$email."\n".
"Tel: ".$tel."\n"."\n".
"Mis votaciones:"."\n".
"1: ".utf8_decode($tchica1).$chica1."\n".
"2: ".utf8_decode($tchica2).$chica2."\n".
"3: ".utf8_decode($tchica3).$chica3."\n".
"4: ".utf8_decode($tchica4).$chica4."\n".
"5: ".utf8_decode($tchica5).$chica5."\n".
"6: ".utf8_decode($tchica6).$chica6."\n".
"7: ".utf8_decode($tchica7).$chica7."\n".
"8: ".utf8_decode($tchica8).$chica8."\n".
"9: ".utf8_decode($tchica9).$chica9."\n".
"10: ".utf8_decode($tchica10).$chica10."\n".
"11: ".utf8_decode($tchica11).$chica11."\n".
"12: ".utf8_decode($tchica12).$chica12."\n".
"13: ".utf8_decode($tchica13).$chica13."\n".
"14: ".utf8_decode($tchica14).$chica14."\n".
"15: ".utf8_decode($tchica15).$chica15."\n".
"16: ".utf8_decode($tchica16).$chica16."\n".
"17: ".utf8_decode($tchica17).$chica17."\n".
"18: ".utf8_decode($tchica18).$chica18."\n".
"19: ".utf8_decode($tchica19).$chica19."\n".
"20: ".utf8_decode($tchica20).$chica20."\n".
"21: ".utf8_decode($tchica21).$chica21."\n".
"22: ".utf8_decode($tchica22).$chica22."\n".
"23: ".utf8_decode($tchica23).$chica23."\n".
"  \n".
"Sugerencias: ".utf8_decode($sugerencias);

$headers = array('From'     => $from,
    'To'       => $to,
    'Subject'  => $subject,
    'Reply-To' => $_POST['email']);
$smtp    = Mail::factory('smtp', array('host'           => $host,
            'auth'           => "PLAIN",
            'socket_options' => array('ssl' => array('verify_peer_name' => false)),
            'username'       => $username,
            'password'       => $password));

$mail = $smtp->send($to, $headers, $body);

$usuario = '';
$contrasena = '';
$basededatos = "admin_votaciones";

$conexion = mysqli_connect($host,$usuario,$contrasena);
$db = mysqli_select_db($conexion,$basededatos);

$query = $conexion->query("Delete from votaciones where modo='$modo' and id='$evento' and telefono='$tel'");
$query = $conexion->query("INSERT INTO votaciones (nombre, email, telefono, identificador, vota1, vota2, vota3, vota4, vota5, vota6, vota7, vota8, vota9, vota10, vota11, vota12, vota13, vota14, vota15, vota16, vota17, vota18, vota19, vota20, vota21, vota22, vota23, sugerencias, id, modo) VALUES ('$nom','$email','$tel','$num','$chica1','$chica2','$chica3','$chica4','$chica5','$chica6','$chica7','$chica8','$chica9','$chica10','$chica11','$chica12','$chica13','$chica14','$chica15','$chica16','$chica17','$chica18','$chica19','$chica20','$chica21','$chica22','$chica23','$sugerencias','$evento','$modo')");


mysqli_close($conexion);

if (PEAR::isError($mail)) {
echo("
" . $mail->getMessage() . "

");
} else {
?>
<SCRIPT LANGUAGE="JavaScript">
  location.replace("https://.es/barcelona/online/votaciones.php");
 </SCRIPT>
    <?php

}

?>

Muchas gracias!

-Yepeto- #2 Jul '22

Eres trol poniendo las contraseñas o no es público del dominio? De todas formas, si función con PHP 8, lo mismo es que no tienes que cambiar nada.

1 respuesta

spud #3 Jul '22

Bastante por lo que veo...

1 respuesta

Mubris #4 Jul '22

Hay que ver la de gilipolleces que se piden a veces.

M

MissFortune #5 Jul '22

#2 sorry, no me ha dado tiempo a borrar las cosas importantes. Ya esta modificado correctamente.

#3 podrias indicarme mas o menos que cosas?

willy_chaos #6 Jul '22

wtf, no conocen el uso de arrays?

chica_n ?

un array de n y tienes lo mismo y menos mierda en el codigo xD

1 1 respuesta

Wei-Yu #7 Jul '22

#6 son deconstructed arrays, se hace por performance.

S

Sarwaz #8 Jul '22

Yo creo que si en el formulario pones de telefono:

' or 1=1 -- a

Se arreglan todos tus problemas

1 respuesta

M

MissFortune #9 Jul '22

#8 creo que hay mas cosas que modificar para que se adapte a php 8 pero no se por donde empezar

Sonos #10 Jul '22 MV Wizard

El código claro que es mejorable, pero lo que el quiere es cambiar de versión.

Normalmente cuando cambias de versión simplemente tirando de los errores/logs en ejecución ya ves que funciones están obsoletas, o no se recomienda su uso y las has de cambiar por nuevas, todo depende, no es lo mismo estar en un 7.0 que en un 7.4 para pasarse a 8/8.1.

Para ver todos los errores puedes dejar comentada la línea que tienes ahora, y poner
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);
error_reporting(E_ALL);

Eso si, yo cambiaría todo lo relacionado a la conexión de la base de datos de mysqli a pdo.

1 1 respuesta

M

MissFortune #11 Jul '22

#10 me he descargado phpstorm y sí que salen algunos warnings que no errores. Entiendo que son estas las mejoras?

1 respuesta

Sonos #12 Jul '22 MV Wizard

#11 No he usado PHPStorm, eso es un editor entiendo, que te avisará de errores de sintaxis o similar no? pero para saber si el php funciona tienes que ejecutarlo, te permite ejecutarlo con click derecho y RUN? tienes un entorno de pruebas en local?

1 respuesta

M

MissFortune #13 Jul '22

#12 tengo xaamp actualizado a php 8 y esta funcionando pefectamente

1 respuesta

Sonos #14 Jul '22 MV Wizard

#13 Pues básicamente es eso, si el código se ejecuta sin errores en el entorno 8.0, ya lo tendrías listo. En ocasiones toca cambiar mucho código, en otras funciona todo desde el momento 0.

Tunnecino #15 Jul '22

Aparte de que el código da bastante grima, y me ha trasladado a hace casi 18 años en el pasado, la mejor forma de "pasar" es si funciona, es que es compatible. PHP 7 y 8 sigue siendo el mismo lenguaje, salvo con los cambios y los deprecated, y las nuevas funciones.

PiradoIV #16 Jul '22 Brongster

Mira la documentación, siempre viene con guías para actualizar:
https://www.php.net/migration80

Soltrac #17 Jul '22 MV Wizard

Cuando se escribía así, desu ni había nacido....

1 respuesta

M

MissFortune #18 Jul '22

#17 ya se ha comentado esto más arriba y ya me ha quedado claro.

vivora #19 Jul '22

Nosotros en la empresa hemos hecho recientemente cambio a 8.1, y salvo alguna función de php que ahora no acepta como parámetro de entrada un string vacío, y algún que otro problema con nulls, poca cosa más hemos tenido que tocar.

1 respuesta

M

MissFortune #20 Jul '22

#19 eso es lo que he leido en la documentación de php. Tema funciones, classes, null, etc....

El tema es que los archivos que me han pasado no tiene nada de todo esto.

Me huelo que a lo mejor es una pregunta trampa.

JuAn4k4 #21 Jul '22

Lo que deberías hacer:

Leer la documentación oficial, Suelen poner que hay que cambiar.
No tocar más allá de lo justo y necesario.
Hacer muchas pruebas de regresión, y de load/performance testing.
Crear un plan de despliegue que vaya con cuidado, usando canaries e ir incrementando el % de tráfico poco a poco.
Fin.

Si empiezas a cambiar cosas porque en php8 se puede, acabas haciendo un Big Bang de la ostia que como algo vaya mal, la lías parda.

PD: nunca os fiéis de los cambios backwards compatibles.

PHP 7 a PHP 8

Usuarios habituales

Tags