BugFix für den Suchindex

[Scotty]

Code: Alles auswählen

# - BugFix für den Suchindex

#
#-----[ ÖFFNE ]-----
#
# includes/functions_search.php
#

#
#-----[ FINDE ]-----
#
		$entry = preg_replace('/[ ]([\S]{1,2}|[\S]{21,})[ ]/',' ', $entry);
#
#-----[ MIT FOLGENDEM ERSETZEN ]-----
#
		$entry = preg_replace('/(?<= )([\S]{1,2}|[\S]{21,})(?= )/',' ', $entry);

#
#-----[ HINWEIS ]-----
#
# Nach dem Einbau sollte man den den Suchindex neu aufbauen

Quellen: http://www.phpbb.de und http://www.phpbb.com
Autor: Unbekannt

[blondi]

welcher bug wird damit behoben?

kurze info zu den snippets wäre nicht schlecht...

liebe grüsse

[AmigaLink]

Das wurde hier schonmal an anderer stelle besprochen blondi (auch der Fix ist schonmal vorhanden).
Die Suchfunktion soll normalerweise keine Wörter indizieren die weniger als 3 oder mehr als 20 Buchstaben haben. Dieser Fix sorgt dafür das es auch wirklich so ist.

[Scotty]

Genau

Hier kann das dann auch jeder nachstellen, also je nachdem, kommt ja auch aufs Forenthema an. Im Original sollen Wörter mit weniger als 3 Zeichen nicht in den Index aufgenommen werden.

Ich würde aber auf jeden fall diesen wert erhöhen, also das Wörter mit weniger als 4 Zeichen nicht aufgenommen werden. Darunter fallen dann eben Wörter wie "der, die, das" und so , die müssen da ja nun wirklich nicht rein.

Man kann die sicher auch über die "search_stopwords.txt" ausschließen aber da gibt’s ja noch viel mehr und so ist das effektiver. Wenn man den wert erhöht, sollte man diese Wörter, also mit dieser Zeichenlänge dann auch aus der "search_stopwords.txt" löschen.

[blondi]

aha, sachen gibts danke euch beiden für den hinweis...

grüsse

[Christian_N]

Weiss jemand, ob der Bug noch aktuell ist in 2.0.18 oder wurde er schon behoben dort von der phpBB Group - falls denne der bekannt war?

[oxpus]

Naja, ein echter BUG ist es nicht, sondern nur die Möglichkeit, kurze Wörter nicht in den Suchindex mit aufzunehmen.
Das ist bislang noch nicht in der aktuellen Version enthalten.

[Christian_N]

oki kommt dann bei mir noch rein - danke der info