Kundengruppen-Speicherung /admin/content_manager.php geht nicht korrekt

[noRiddle]

Grund: strpos() verkehrt verwendet, siehe bitte im Thread ​modified eCommerce Shopsoftware 2.0.6.0 rev 13500 veröffentlicht

Gruß,
noRiddle

[noRiddle]

*NACHTRAG*
Ich habe mal den Code durch-gescannt und finde einige Stellen wo strpos() ohne Typen-Operator benutzt wird (was auch geht wenn man genau weiß was das Ergebnis sein wird).
Ich würde allerdings dringend empfehlen

1. alle strpos()-Abfragen mit dem stricten Typ-Operator zu machen, also abzufragen

   !== false
   

   wenn bei der Abfrage lediglich ein Boolean als Rückgabe erwartet wird.

So kann man nicht versehentlich etwas übersehen.

2. sollte man auch darauf achten

Zitat PHP-Manual:

8.0.0 	Die Übergabe eines Integer als needle wird nicht mehr unterstützt.
7.3.0 	Die Übergabe eines Integer als needle wird missbilligt.
7.1.0 	Unterstützung für negative offset-Werte wurde hinzugefügt. 

Dies nur als Hinweis, habe im Code keine Stelle gefunden die einen INT als needle übergibt.

Gruß,
noRiddle

[Gerhard Waldemair]

In 13514:

fix #2012 & revised r13072 (partly fix #1805 - replaced function strstr() with strpos() where possible) - fix strpos usage

[jaktrip@…]

Der Fix funktioniert bei mir nicht. Ich habe die Datei ausgetauscht und den Cache geleert, aber trotzdem: bei jedem "aktualisieren" wird das oberste Häkchen bei den Kundengruppen gelöscht, genau wie vor dem Fix.

[Gerhard Waldemair]

Kannst du das hier nachstellen ?

​https://dev.modified-shop.org/

Wenn nicht, dann musst du den Fehler wo anders suchen...

[jaktrip@…]

Auf dev funktioniert es. Ich habe jetzt zum zweiten Mal die hier angehängte neue content_manager.php runtergeladen (vom 16.4. 08:31) und die alte Datei überschrieben. Weiter keine Besserung in meiner 2.0.6. Ist 16.4. die richtige Version? Kann man den dev-Shop auch irgendwo runterladen? Ich würde dann ausprobieren, ob es mit der Datei direkt aus dem dev-Shop geht.

[Gerhard Waldemair]

In 13513:

fix #2012 & revised r13072 (partly fix #1805 - replaced function strstr() with strpos() where possible) - fix strpos usage

[Torsten Riemer]

Ein dafür wichtiges Changeset war dem Ticket hier noch nicht zugeordnet, was ich soeben nachgeholt habe.

[jaktrip@…]

Danke! Ich habe alle Dateien übertragen und jetzt geht es bei mir in 2.0.6.!
Den Installer habe ich nicht zum laufen bekommen, da steht "you don't have permission to this resource", obwohl ich kein htaccess gesetzt habe.

Brauche ich den installer überhaupt?

[Torsten Riemer]

Argh... nein, den Installer brauchst du nicht! Dein Shop ist doch bereits installiert. ;-)

[svenyo]

Sind diese Changesets in der aktuellen Download-Zip bzw. Update-Zip bereits enthalten? Oder müssen diese zusätzlich heruntergeladen werden?

[Torsten Riemer]

Alle nach 2.0.6.0 veröffentlichten Changesets sind nicht in 2.0.6.0 enthalten, sondern werden erst in der folgenden Shopversion enthalten sein.