#1598 closed Bug/Fehler (fixed)
autocomplete und strlen()
Reported by: | noRiddle | Owned by: | somebody |
---|---|---|---|
Priority: | normal | Milestone: | modified-shop-2.0.5.0 |
Component: | Shop | Version: | 2.0.4.2 |
Description
In /api/autocomplete/autocomplete.php finden wir dies
if ($keywordcheck === true && strlen($queryString) > SEARCH_AC_MIN_LENGTH) {
Sollten wir dort nicht mb_strlen() verwenden ?
if ($keywordcheck === true && mb_strlen($queryString, $_SESSION['language_charset']) > SEARCH_AC_MIN_LENGTH) {
Ansonsten würden bei z.B. Default-Einstellung 3 für SEARCH_AC_MIN_LENGTH Eingaben dieser Art
Räd
bereits ein Ergebnis liefern.
Nicht super-schlimm aber inkorrekt.
Gruß,
noRiddle
*NACHTRAG*
An einigen anderen Stellen im Code gilt dasselbe, wie z.B. in der /create_account.php
if (strlen($customers_firstname) < ENTRY_FIRST_NAME_MIN_LENGTH) {
usw.
Attachments (0)
Change History (8)
comment:1 Changed 2 years ago by Tomcraft
- Milestone set to modified-shop-2.0.5.0
- Reporter changed from anonymous to noRiddle
comment:2 Changed 22 months ago by anonymous
comment:3 Changed 22 months ago by Tomcraft
Jein... ein Teil (Ersetzung von > SEARCH_AC_MIN_LENGTH mit >= SEARCH_AC_MIN_LENGTH) ist damit gefixt, aber was in der Tat noch geändert werden sollte ist die Ersetzung von strlen() mit mb_strlen(), da es sonst bei Umlauten wirklich zu merkwürdigen Ergebnissen kommt.
Danke, dass du das nochmal hoch geholt hast.
comment:4 Changed 22 months ago by anonymous
Okay, ich habe ja keinen Einblick in die Fixes.
In der /advanced_search_result.php gibt es das Problem mit strlen() übrigens ebenfalls, wobei dort außerdem die Min-Length hard-codiert mit 3 angegeben ist, sollte vielleicht ebenfalls im Backend einstellbar sein.
Gruß,
noRiddle
comment:5 Changed 20 months ago by GTB
- Resolution set to fixed
- Status changed from new to closed
In 11904:
comment:6 Changed 20 months ago by GTB
In 11907:
comment:7 Changed 20 months ago by Tomcraft
In 11908:
comment:8 Changed 19 months ago by GTB
In 12036:
Eventuell ist das bereits mit r11334 zu Ticket #1490 gefixt. Bitte prüfen.
Gruß,
noRiddle