Opened 2 weeks ago

Closed 6 days 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 (1)

changeset_13514.zip (4.6 KB) - added by Tomcraft 3 days ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 2 weeks ago by Tomcraft

  • Milestone set to modified-shop-2.0.6.1

comment:2 Changed 2 weeks 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 6 days 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 days ago by Tomcraft

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.