autocomplete und strlen()

[noRiddle]

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.

[anonymous]

Eventuell ist das bereits mit r11334 zu Ticket #1490 gefixt. Bitte prüfen.

Gruß,
noRiddle

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

[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

[GTB]

In 11904:

fix #136, #1598 & #1635

[GTB]

In 11907:

revised r11904 (fix #136, #1598 & #1635)

[Tomcraft]

In 11908:

revised r11904 (fix #136, #1598 & #1635)

[GTB]

In 12036:

revised r11904 (fix #136, #1598 & #1635)