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

Warning beim indexieren #361

Open
tbaddade opened this issue Jan 26, 2023 · 7 comments
Open

Warning beim indexieren #361

tbaddade opened this issue Jan 26, 2023 · 7 comments

Comments

@tbaddade
Copy link
Member

Description / Beschreibung
Folgende Warning erscheint während des schrittweise indexieren.

Warning: Invalid argument supplied for foreach() in /search_it/lib/search_it.php on line 835

Affected versions / Verwendete Versionen

REDAXO: 5.14.2
PHP: 7.4.29

@tyrant88
Copy link
Member

kannst du bitte mal die Zeile 831
ändern in
$plaintext = $this->getPlaintext($indexData['unchangedtext']);
und dann berichten, ob der Fehler weg ist?

@tbaddade
Copy link
Member Author

Fehler ist nach der Änderung immer noch vorhanden.

@tyrant88
Copy link
Member

Dann vielleicht
$plaintext = $this->getPlaintext($indexData['unchangedtext']) ?? '';
Hast du das plaintext-Plugin im Einsatz?

@tbaddade
Copy link
Member Author

Dann vielleicht
$plaintext = $this->getPlaintext($indexData['unchangedtext']) ?? '';

Das wird nicht helfen … $plaintext ist vor dem Array gefüllt. preg_split liefert in der foreach jedoch false, sodass die Warning erscheint.

Hast du das plaintext-Plugin im Einsatz?

Nein

@tyrant88
Copy link
Member

dass preg_split false liefert, hielt ich für unwahrscheinlich (passiert ja nur bei einem regex-error).
Wenn du weißt, dass plaintext gefüllt ist, kannst du es auch dumpen?
Ich könnte zwar den preg_split-Ausdruck auf !== false testen, aber da verliert man ja was, wenn dein $plaintext nicht null ist..

@tbaddade
Copy link
Member Author

Wenn ich diese Zeile lösche, dann klappt jedes preg_split.

Weshalb die Zeile mit html_entity_decode? Die Tags scheinen in der Methode noch vorhanden zu sein, da diese eine Zeile vorher via strip_tags gelöscht werden. Weshalb das Konvertieren von Entities zurück zu HTML Tags?

@tyrant88
Copy link
Member

tyrant88 commented Jan 27, 2023

ohne das decode könnte im String z.B. ein © stehen. da würde das preg_split dann das "copy" als keyword rausholen...

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

No branches or pull requests

2 participants