Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Metaphone Zahlensuche entfernt #302

Merged
merged 9 commits into from
Nov 25, 2020
Merged

Metaphone Zahlensuche entfernt #302

merged 9 commits into from
Nov 25, 2020

Conversation

TobiasKrais
Copy link
Member

Nun wird auch Metaphone bei der Suche nach Zahlen nicht mehr aktiviert

Auch metaphone soll bei Nummern keine Ähnlichkeitssuche vornehmen.
@tyrant88
Copy link
Member

was ist das mit den einzelnen "&"? oder was verstehe ich gerade nicht?

@TobiasKrais
Copy link
Member Author

Um ehrlich zu sein verstehe ich die auch nicht. Die waren so im Code drin. Ich mach sie mal raus. Momentle...

@TobiasKrais
Copy link
Member Author

Ist geändert ;-)

@TobiasKrais
Copy link
Member Author

Warte nochmals mit mergen.

@tyrant88
Copy link
Member

Sorry, Kommando zurück.
Habe mich gerade noch mal aufklären lassen:
Es handelt sich um den bit-Operator.
Damit wird geprüft ob das "Bit" für SEARCH_IT_SIMILARWORDS_SOUNDEX in der Variable "similarword_mode" "gesetzt" ist.

@TobiasKrais
Copy link
Member Author

Alles klar. Ist geändert und zur Klarheit noch mit Klammern versehen.

Mit dem PR gibt es noch Probleme. Bei einem dem Suchbegriff "kbd 615" gibt es einen fatal error. Alle anderen getesteten Suchbegriffe laufen durch. Seltsam. Ich melde mich nochmals.

@TobiasKrais
Copy link
Member Author

So, jetzt ham' wers . Da war noch ein fatal error drin. Jetzt laufen alle Tests bei mir sauber durch.

@TobiasKrais
Copy link
Member Author

Heute Morgen nochmals alles getestet. Funktioniert einwandfrei: https://www.kaltenbach.com/de/suche/. Egal ob nur Zahl, Wort + Zahl, Treffer oder nicht Treffer. Kein Whoops mehr und immer das gesuchte Ergebnis.

@tyrant88 tyrant88 merged commit b016771 into master Nov 25, 2020
@delete-merged-branch delete-merged-branch bot deleted the metaphone_number branch November 25, 2020 08:21
@tyrant88
Copy link
Member

Hier gilt natürlich wieder: geile Aktion. Ding verbessert: THX.

@TobiasKrais
Copy link
Member Author

Danke. Bis ich die Ursache gefunden hatte, habe ich insgesamt ein halbes Jahr gesucht. Erst mit R5.11.1 wurde angezeigt, dass der Fehler in search_it zu suchen ist... Bin Mega froh darüber, de meine Suche nun erheblich performanter läuft wenn man nach eine Zahl sucht.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants