Verwendung von strstr() mit strpos() ersetzen

[Torsten Riemer]

Es gibt diverse Stellen, die wir mit der schnelleren und weniger speicherintensiven Funktion strpos() anstelle von strstr() umbauen können.

[Gerhard Waldemair]

In 13072:

partly fix #1805 - replaced function strstr() with strpos() where possible

[Gerhard Waldemair]

In 13078:

partly fix #1805 - update paypal - replaced function strstr() with strpos() where possible

[Gerhard Waldemair]

In 13079:

partly fix #1805 - update easycredit - replaced function strstr() with strpos() where possible

[Gerhard Waldemair]

In 13080:

partly fix #1805 - update billpay - replaced function strstr() with strpos() where possible

[Gerhard Waldemair]

In 13082:

fix #1805 - replaced function strstr() with strpos() where possible in templates

[Gerhard Waldemair]

In 13513:

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

[Gerhard Waldemair]

In 13514:

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

[Torsten Riemer]

Karl hat sich die Mühe gemacht und die Ersetzungen in den Templates nochmal überprüft:

[...]
Die Templates sind nicht im changeset enthalten, deshalb habe ich meine Änderungsvorschläge mal angehängt.

Schwierigkeiten beim Durchsehen habe ich mit solchen Codestellen, wie z.B.
if (!strpos($sOrdersComment, 'ExtendedOrderID')) {
oder
if (strpos($key, 'customers_') === false
Man kann an vielen Stellen nicht genau nachvollziehen, ob es dem Programmierer klar war, dass der Rückgabewert "0" auch als "false" gewertet wird (im 2. Beispiel wird der Rückgabewert "0" bewusst als "false" verwendet).

Anders ist es meiner Ansicht nach aber in inc/xtc_php_mail.inc.php
$path_to_attachments[$i] = ((strpos($path_to_attachments[$i], DIR_FS_DOCUMENT_ROOT)===false) ? DIR_FS_DOCUMENT_ROOT:'') . trim($path_to_attachments[$i]);
Wenn hier ein Mailanhang beginnend mit dem absoluten Shoppfad ankommt, wird im nochmal der absolute Pfad vorangestellt - hier müsste man eher auf "=== 0" prüfen.

Leider kann ich nicht mehr liefern, da mir an so vielen Passagen im Code das "Know how" fehlt.
Mal [b]schnellb den Code von "strstr()" nach "strpos()" umzustellen ist bestimmt nicht leicht gewesen.
[...]

Quelle: ​modified eCommerce Shopsoftware 2.0.6.0 rev 13500 veröffentlicht

Vielleicht sollten wir in dem Zusammenhang wirklich alle zugehörigen Changesets nochmal unter die Lupe nehmen: r13072, r13078 (für PayPal PLUS Module relevant!), r13079, r13080 & r13082

Nachtrag: Hier noch weitere Hinweise von noRiddle: ​modified eCommerce Shopsoftware 2.0.6.0 rev 13500 veröffentlicht

[Gerhard Waldemair]

In 13539:

revised r13078 (partly fix #1805 - update paypal - replaced function strstr() with strpos() where possible) - fix strpos usage for paypal