Opened 4 months ago

Closed 4 months ago

Last modified 5 weeks ago

#2012 closed Bug/Fehler (fixed)

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

Reported by: noRiddle Owned by: somebody
Priority: hoch Milestone: modified-shop-2.0.6.1
Component: Shop Version: 2.0.6.0

Description

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

Gruß,
noRiddle

Attachments (2)

changeset_13514.zip (4.6 KB) - added by Tomcraft 3 months ago.
changeset_13513.zip (127.0 KB) - added by Tomcraft 6 weeks ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 4 months ago by Tomcraft

  • Milestone set to modified-shop-2.0.6.1

comment:2 Changed 4 months ago by 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.

  1. 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

comment:3 Changed 4 months ago by GTB

  • Resolution set to fixed
  • Status changed from new to closed

In 13514:

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

Changed 3 months ago by Tomcraft

comment:4 Changed 6 weeks ago by 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.

comment:5 Changed 6 weeks ago by GTB

Kannst du das hier nachstellen ?

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

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

comment:6 Changed 6 weeks ago by 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.

comment:7 Changed 6 weeks ago by GTB

In 13513:

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

comment:8 Changed 6 weeks ago by Tomcraft

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

Changed 6 weeks ago by Tomcraft

comment:9 Changed 5 weeks ago by 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?

comment:10 Changed 5 weeks ago by Tomcraft

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

Add Comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.