Hotfix - General - Evitar errores en PHP 8

[jordiSTIC]

• Closes Incidencia - General - Evitar errores en PHP 8 por llamadas a funciones no estáticas como si lo fueran #476

Descripción

Este PR soluciona ciertos errores en PHP 8 encontrados en el core, al ejecutar la herramienta Rector en todo el código de SinergiaCRM.

Cambios realizados

Llamadas estáticas a funciones no estáticas

Se ha observado se realizan llamadas a funciones no estáticas como si éstas fueran estáticas, y esto provoca error en PHP 8: Deprecated: Non-static method should not be called statically:

• Commit 4c16a81: Corrección en la carpeta /custom
• Commit 76b8249: Corrección en el código de core

Uso de variable obsoleta $HTTP_SERVER_VARS

Hay accesos a la variable obsoleta $HTTP_SERVER_VARS. El soporte a esta variable se declaró obsoleto en la versión de php 5.4. Para corregir este error se debe substituir la variable $HTTP_SERVER_VARS por $_SERVER

• Commit d7301c7: Substitución de $HTTP_SERVER_VARS por $_SERVER

Operaciones ternarias encadenadas sin paréntesis

En PHP 7.4 se declaró obsoleta la opción de poder encadenar operaciones ternarias sin paréntesis explícitos. En include/SugarFields/Parsers/QuickCreateMetaParser.php se asignaba el resultado de múltiples ternarias sin aplicar paréntesis.

• Commit 6fc8c4f

Errores de sintaxis

Se observa una comparación errónea en include/Smarty/plugins/function.ext_includes.php:
Aparece la siguiente comparación:

if (strcasecmp(substr($file, -4), '.css' == 0))

En el código se evalúa '.css' == 0 lo que se traduce a 0 == 0: if (strcasecmp(substr($file, -4), 0 == 0))
El paréntesis está mal puesto. Tendría que ser:

if (strcasecmp(substr($file, -4), '.css') == 0)

• Commit: 4c85063

[github-actions]

Actions executed at: 2024-11-13 08:53:54.