Droplets in 2.13.0 r63 unter PHP 8 - Problem!

[hillschmidt]

Mir ist heute ein Problem aufgefallen: meine Droplets laufen problemlos nach dem Update auf WB 2.13.0 r63 unter PHP 7.4 .
Sobald ich aber auf PHP 8 umstelle, werden die Inhalte nicht mehr aufgelöst - es erscheinen die Klammern [[ ]] und der Code, aber nicht die Auflösung.

URL: https://www.hillschmidt.de/wb - Startseite mit der englischen Flagge oben und der aktuellen Jahreszahl 2022 im Footer beim Copyright.

Sobald ich PHP 8 einstelle, erscheint an den Stellen

Code: [Select]

[[_englishVersion?url=277]]und

Code: [Select]

© 1999-[[SHOW_YEAR]] Andreas Schmidt
Hier die Codes der beiden Droplets (es sind aber auch andere betroffen)

SHOW_YEAR:

Code: [Select]

return date('Y');
englische Seite (nein, für mich ist eine echte zweisprachige Seite zu viel):

Code: [Select]

if (isset($url)) {
return '<p style="text-align:right;">
<a href="[wblink'.$url.']">english version
<img alt="" height="19" src="https://www.hillschmidt.de/wb/media/gif/flag_gbr.gif" width="33" /></a>
</p>';
} else {
return '<p style="text-align:right;">
<a>english version
<img alt="" height="19" src="https://www.hillschmidt.de/wb/media/gif/flag_gbr.gif" width="33" /></a>
</p>';
}
Hat jemand eine Idee?

[ruebenwurzel]

Hallo,

ein Upgrade auf 2.13.1 könnte das Problem beheben.

Matthias

[hillschmidt]

Update auf 2.13.1 bringt folgenden Fehler mit PHP 8 - zeigt aber auch die Droplets unter PHP 7.4 an:

Code: [Select]

There was an uncatched exception
preg_match(): Argument #3 ($matches) cannot be passed by reference
in line (57) of (/framework/Utils/IpAddress.php):
Dann lasse ich jetzt 2.13.1 r103 und mache ein neues Problem auf. OK?

[hgs]

Hast du den Patch 114 installiert? (hilft für das Problem wahrscheinlich nicht, aber uptodate ist immer gut )


Kannst du sagen welches Droplet es ist?
Evtl. hier posten
Sorry, den Anfangspost nicht ganz zu ende gelesen.

Alle von WB mitgelieferten Droplets sind php8 tauglich gefixt.

[hgs]

Hier die Codes der beiden Droplets (es sind aber auch andere betroffen)

SHOW_YEAR:

Code: [Select]

return date('Y');

Das Droplet mal bitte so ergänzen:

Code: [Select]

$datum = date("Y");
return "$datum";

[hillschmidt]

Code: [Select]

$datum = date("Y");
return "$datum";macht keinen Unterschied

[hgs]

Die Jahreszahl wird aber im Footer angezeigt, oder schau ich auf eine andere Domäne?

Auszug von deinem Link
© 1999-2022 Andreas Schmidt


Als Tester bin ich am Cache von diversen Browser schon verrückt geworden und die Entwickler haben Fehler gesucht, die es gar (nicht) mehr gab.
F5 wirkt da manchmal Wunder

[hillschmidt]

Ich bin ja wieder auf 7.4 zurückgegangen ... sobald ich PHP 8 aktiviere, taucht das Problem auf.
Und den Cache lösche ich immer bei solchen Aktionen ...

[hgs]

Ich habe alles auf  php8.0 und es läuft auch alles,


das Droplet "year" nutze ich auch aktiv auf allen Domänen im Footer.


Sehr komisch, aber es gab ja im Tages-Rhythmus Patche, mal sehn was die Entwickler da noch so finden und  fix fixen.

[crnogorac081]

Update auf 2.13.1 bringt folgenden Fehler mit PHP 8 - zeigt aber auch die Droplets unter PHP 7.4 an:

Code: [Select]

There was an uncatched exception
preg_match(): Argument #3 ($matches) cannot be passed by reference
in line (57) of (/framework/Utils/IpAddress.php):
Dann lasse ich jetzt 2.13.1 r103 und mache ein neues Problem auf. OK?

I am seing this after installing patch 114 , using wb portable with php 7.4

is there patch 115 published ?

[Luisehahne]

Patch R115 is under preparation, we will inform you

Dietmar

[crnogorac081]

Ok. Is there a quickfix for this line in IpAddress.php ?

[Luisehahne]

Ok. Is there a quickfix for this line in IpAddress.php ?

Fixed methode in /framework/Utils/IpAddress.php

Code: [Select]

<?php
    public static function sanitizeIp(string $sRawIp): string
    {
        // clean address from netmask/prefix and port
        $sPattern = '/^[{\[]?([.:a-f0-9]*)(?:\/[0-1]*)?(?:[\]}]?.*)$/im';
        $sIpAddress = \preg_replace($sPattern, '$1', $sRawIp);
        if (\strpos($sIpAddress, ':') === false) {
        // sanitize IPv4 -------------------------------------------------------------- //
            if (ip2long($sIpAddress) === false) {
                throw new \InvalidArgumentException('illegal IPv4 address given!');
            }
        } else {
            // sanitize IPv6 -------------------------------------------------------------- //
            // for backard compatibility it also check deprecated addressing where
            // IP includes a 32 bit IPv4 part and convert this into IPv6 format
            // sanitize IPv6 -------------------------------------------------------------- //
            $sPattern = '/^([:a-f0-9]*?)(?:(?:\:)([0-9]{1,3}(?:\.[0-9]{1,3}){3}))?$/is';
            $aMatches = [];
            if (\preg_match($sPattern, $sIpAddress, $aMatches)) {
                $sIpAddress = $aMatches[1].(isset($aMatches[2]) ? ':'.self::convertV4ToV6($aMatches[2]) : '');
            } else {
                throw new \InvalidArgumentException('illegal IPv6 address given!');
            }
            $sIpAddress = self::expandIpV6($sIpAddress);
        }
        return $sIpAddress;
    }

Dietmar

[hgs]

Hallo Dietmar
Der TeilCode, den du gepostet hast, ist laut "Diff Ceck" identisch mit dem TeilCode aus patch 114 in der besagten Datei.

Heißt das, dass Patch 114 das Problem fixen würde?

[crnogorac081]

I kind of solver it,

line 107

Code: [Select]

    public static function expandIpV6(string $sIpV6Address): string
    {
        $iPos = strpos($sIpV6Address, '::');
        if ($iPos !== false) {
//            $iInserts     = 8 - \count(\preg_split('/:/', $sIpV6Address, null, \PREG_SPLIT_NO_EMPTY));
// Replaced null with -1
            $iInserts     = 8 - \count(\preg_split('/:/', $sIpV6Address, -1, \PREG_SPLIT_NO_EMPTY));

I dont know if it is correct or not for further code I just saw preg_split documentation.

cheers

[DarkViper]

-1 is ok..     

[Luisehahne]

Hallo Dietmar
Der TeilCode, den du gepostet hast, ist laut "Diff Ceck" identisch mit dem TeilCode aus patch 114 in der besagten Datei.

Heißt das, dass Patch 114 das Problem fixen würde?

Correct

Dietmar

[crnogorac081]

It looks like patch and this fix fixed some memory leak I had...Did anyone elxe experienced memory leak ?