PHP 7.2 auf PHP7.4 -> Wordpress -> Fehler

[verzw (Gast)]

Hallo Leute,

Folgendes Szenario:
Mein Hoster dreht bald den PHP 7.2 Support ab.
Also schnell mal die PHP Einstellung auf 7.4 geändert.

Resultat:

Fatal error: Unparenthesized `a ? b : c ? d : e` is not supported. Use either `(a ? b : c) ? d : e` or `a ? b : (c ? d : e)` in /homepages/5/e635618891/htdocs/XXXXXXXXX/wp-content/plugins/user-access-manager/src/Util/Util.php on line 100

Das Plugin heisst "User Access Manager" und wird leider nicht mehr 
gepflegt.
Es ist sehr einfach zu bedienen und damit mein "Traumplugin", was die 
Zuriffsbeschränkung auf eine Seite darstellt.

Daher wollte ich mir mal das PHP Script anschauen.

Zeile 100 sieht wie folgt aus:

 $secure = empty($_SERVER['HTTPS']) ? '' : ($_SERVER['HTTPS'] === 'on') ? 's' : '';

Die ganze Funktion wie folgt:

 /**
     * Returns the current url.
     *
     * @return string
     */
    public function getCurrentUrl()
    {
        if (isset($_SERVER['REQUEST_URI']) === false) {
            $serverRequestUri = $_SERVER['PHP_SELF'];
        } else {
            $serverRequestUri = $_SERVER['REQUEST_URI'];
        }

        $secure = empty($_SERVER['HTTPS']) ? '' : ($_SERVER['HTTPS'] === 'on') ? 's' : '';
        $protocols = explode('/', strtolower($_SERVER['SERVER_PROTOCOL']));
        $protocol = $protocols[0].$secure;
        $port = ((int)$_SERVER['SERVER_PORT'] === 80) ? '' : (':'.$_SERVER['SERVER_PORT']);

        return $protocol.'://'.$_SERVER['SERVER_NAME'].$port.$serverRequestUri;
    }

Hat jemand eine Idee, wie ich das wieder zum laufen bekomme?

Sollte >7.4 kompatibel sein und auch mal PHP 8.0 können.

Würde mich über ein Feedback freuen und
verbleibe mfg,
verzw

[Andreas (Gast)]

Meine PHP-Zeit ist lange her, aber dieser Teil des Codes scheint mir 
überflüssig zu sein und sollte einfach entfernt werden können: 
ˋempty($_SERVER['HTTPS']) ? '' : ˋ

[Karl Klammer (Gast)]

Setz Klammern. Steht ja exakt da in der Fehlermeldung.

[verzw (Gast)]

Karl Klammer schrieb:
> Setz Klammern. Steht ja exakt da in der Fehlermeldung.

Und wie sollte das dann aussehen?

[🐧 DPA 🐧 (Gast)]

verzw schrieb:
> Und wie sollte das dann aussehen?

$secure = @$_SERVER['HTTPS'] === 'on' ? 's' : '';

Aber ich würde den "User Access Manager" entfernen. Ist die einzige 
sichere & permanente Lösung. Noch besser, das ganze Wordpress 
entfernen/ersetzen. Ich höre static pages sind wieder in.

[verzw (Gast)]

🐧 DPA 🐧 schrieb:
> $secure = @$_SERVER['HTTPS'] === 'on' ? 's' : '';

GEHT!
Sehr gut. Dankeschön :-)

[verzw (Gast)]

Brauche doch nochmals eure Hilfe.
Der Fehler diesmal:

Deprecated: Function create_function() is deprecated in /homepages/5/e635618891/htdocs/XXXXXXXXX/wp-content/plugins/above-the-fold-optimization/includes/proxy.class.php on line 152

In Zeile 152 steht folgendes:

create_function('$value', 'return trim($value) !== "";')

Die ganze Funktion sieht wie folgt aus:

 // merge default include / exclude list with settings
            $this->$key = array_unique(
                array_filter(
                    array_merge($this->$key, ((isset($this->CTRL->options[$params[0] . '_proxy_' . $params[1]]) && is_array($this->CTRL->options[$params[0] . '_proxy_' . $params[1]])) ? $this->CTRL->options[$params[0] . '_proxy_' . $params[1]] : array())),
                    create_function('$value', 'return trim($value) !== "";')
                )

[Andreas (Gast)]

Also wenn du den Code komplett "ent-deprecaten" willst, dann kommst du 
nicht darum dich intensiver mit PHP auseinanderzusetzen. Das sieht 
allerdings echt nach ziemlichem Spaghetticode aus, vielleicht ist es 
besser sich da nach einer neuen Loesung umzusehen die noch gewartet 
wird.

[quotendepp (Gast)]

ich muss nur die paar zeilen hier gesehen haben, um meinen jahrzehnte 
alten hass auf wordpress wieder aufleben zu lassen. der code ist einfach 
zum kotzen hässlich...

der kurzen rede noch kürzerer sinn: wenn du das problem beheben willst, 
dann musst du:
- entweder die plugins selber umbauen/neu schreiben oder
- die plugins wegwerfen und durch andere (aktuellere) ersetzen

[Weinbauer (Gast)]

https://stackoverflow.com/questions/48161526/php-7-2-function-create-function-is-deprecated

[werists (Gast)]

verzw schrieb:
> Das Plugin heisst "User Access Manager" und wird leider nicht mehr
> gepflegt.  Es ist sehr einfach zu bedienen
Vermutlich auch einfach zu hacken.

[der andere Klaus (Gast)]

Wordpress ist sowieso ein einziges Sicherheitsrisiko, in den letzten 
Jahren hunderte Schwachstellen, die ständig angegriffen werden: ich habe 
das länsgt entsorgt

[ThomasW (Gast)]

Dann ändere Zeile 100 doch wie gefordert:

$secure = empty($_SERVER['HTTPS']) ? '' : (($_SERVER['HTTPS'] === 'on') 
? 's' : '');

wobei ich das anders schreiben würde:

$secure = (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] === 'on') ? 
's' : '';

[Uwe D. (monkye)]

der andere Klaus schrieb:
> Wordpress ist sowieso ein einziges Sicherheitsrisiko, in den letzten
> Jahren hunderte Schwachstellen, die ständig angegriffen werden: ich habe
> das länsgt entsorgt

Das halte ich für ein Gerücht und es ist einfach pauschaliert. Aber als 
Betreiber muss ich mich selbstverständlich um zeitnahes Schließen von 
bekanntgewordenen Lücken kümmern. 
(https://www.heise.de/security/meldung/Contact-Form-7-Datepicker-Gefaehrliches-WordPress-Plugin-ohne-Support-4696045.html)

Was gefährlich ist: PHP Code Schreiben ohne die Folgen abschätzen zu 
können...