Neu v5.1.5: TypeError beim Speichern von eigenen Seiten (Bildname)

[Xantiva]

Hi,

ich wollte nach der Migration vom Shop 4.06 auf den aktuellen 5.1.5 den Inhalt einer meiner eigenen Seiten speichern. Danach sehe ich nur noch eine weiße Seite. 🙁

/admin/links.php => Informationen => Versandinformationen bearbeiten und dann speichern. Die Seite beinhaltet zwei Bilder, die mir auch beim Editieren angezeigt werden.

Fatal error: Uncaught TypeError: Unsupported operand types: int + string in /var/www/vhosts/basteln-selbermachen.de/httpdocs/admin/links.php:248
Stack trace: #0 {main} thrown in /var/www/vhosts/basteln-selbermachen.de/httpdocs/admin/links.php on line 248

                if (is_array($_FILES['Bilder']['name']) && count($_FILES['Bilder']['name']) > 0) {
                    $lastImage = $linkAdmin->getLastImageNumber($kLink);
                    $counter   = 0;
                    if ($lastImage > 0) {
                        $counter = $lastImage;
                    }
248                 $imageCount = (count($_FILES['Bilder']['name']) + $counter);

Dann habe ich ein var_dump eingefügt:

var_dump("count ..", count($_FILES['Bilder']['name']), "counter", $counter, "lastImage", $lastImage, "kLink", $kLink);

Mit folgendem Ergebnis:

string(8) "count .." int(1) string(7) "counter" string(1) "p" string(9) "lastImage" string(1) "p" string(5) "kLink" int(50)

Also in /includes/src/Link/Admin/LinkAdmin.php reingeschaut, um zu sehen, wo das "p" herkommt:

        $max = 0;
        foreach ($images as $image) {
            $num = \mb_substr($image, 4, (\mb_strlen($image) - \mb_strpos($image, '.')) - 3);
            var_dump("image", $image, "num", $num, "max", $max);
            if ($num > $max) {
                $max = $num;
            }
        }
        var_dump("return max", $max);

        return $max;

Da kommt dann folgendes zum Vorschein:

string(5) "image" string(9) "Bild1.gif" string(3) "num" string(1) "1" string(3) "max" int(0)
string(5) "image" string(15) "dpd-predict.jpg" string(3) "num" string(1) "p" string(3) "max" string(1) "1"
string(10) "return max" string(1) "p"

Wieso erwartet Ihr hier plötzlich einen Dateinamen, der dem Schema "BildNNN.xyz" entspricht? 😱 Breaking Change!
Um bei den eigenen Seiten Bilder mit sprechenden Namen zu behalten, konnten die schon immer per FTP in den passenden Ordner hochgeladen und genutzt werden! 😉
Somit kann ich derzeit die meisten meiner eigenen Seite nicht mehr bearbeiten. ☹️

Ich erstelle noch ein Ticket ..

Ticket#2022122310002168

 

[Xantiva]

Antwort vom Support:

dies wurde in der 5.2 gefixed, diese ist aber noch nicht veröffentlicht. Der Fix für Ihr Problem würde aber wie folgt aussehen:

in /includes/src/Link/Admin/LinkAdmin.php können Sie die Zeile 753:

return $max;

durch

return (int)$max;

ersetzen.

Danach sollte es wieder funktionieren.

Noch nicht getestet ...