Opened 16 months ago

Closed 11 months ago

Last modified 10 months ago

#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 16 months ago by Tomcraft

  • Milestone set to modified-shop-2.0.5.0
  • Reporter changed from anonymous to noRiddle

comment:2 Changed 13 months ago by anonymous

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

Gruß,
noRiddle

Last edited 13 months ago by Tomcraft (previous) (diff)

comment:3 Changed 13 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.

Last edited 13 months ago by Tomcraft (previous) (diff)

comment:4 Changed 13 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 11 months ago by GTB

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

In 11904:

fix #136, #1598 & #1635

comment:6 Changed 11 months ago by GTB

In 11907:

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

comment:7 Changed 11 months ago by Tomcraft

In 11908:

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

comment:8 Changed 10 months ago by GTB

In 12036:

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

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.