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 im FE bei Wortbegrenzung in Artikelbasis #241

Closed
uwe367 opened this issue May 23, 2018 · 7 comments
Closed

Warning im FE bei Wortbegrenzung in Artikelbasis #241

uwe367 opened this issue May 23, 2018 · 7 comments
Assignees
Labels

Comments

@uwe367
Copy link

uwe367 commented May 23, 2018

Erstelle ich einen neuen Artikel und trage in max. Anzahl Worte einen Wert ein, so wird im FE eine Warning ausgegeben:

Warning: sizeof(): Parameter must be an array or an object that implements Countable in /www/htdocs/wxxxxx/muster/include/inc_ext/htmlfilter.php on line 112

Anbei ein Screenshot zum besseren Verständnis
screen shot 004

Lasse ich das Feld leer, wird kein Fehler ausgegeben.
Der Fehler tritt nur dann auf, wenn im Editor Text eingegeben ist. Bleibt der Editor leer und ich trage einen Wert ein, keine Fehlermeldung.
Weiterhin spielt es keine Rolle ob ich das Standarttemplate oder ein anderes Template verwende.

Version phpwcms 1.9.0-rc.2 (2018/03/19, r549)

@uwe367
Copy link
Author

uwe367 commented May 25, 2018

Dachte ich hätte eine Lösung gefunden, jedoch verursacht genau diese dann den gleichen Fehler im BE während der Fehler im FE verschwindet. Ist aber vielleicht ein Anhaltspunkt.
In general.inc.php
Zeile 993 original: if($sanitize !== NULL) {
Zeile 993 verändert: if($sanitize == NULL) {
Entfernt zwar den Fehler im FE, dafür hat man ihn dann im BE.

@uwe367
Copy link
Author

uwe367 commented Jun 2, 2018

Ich habe nun in der htmlfilter.php die Zeilen, die den Fehler verursachen, versuchsweise auskommentiert. Die Funktion, eine maximale Wortanzahl zu setzen ist nach wie vor gegeben und wird im FE auch entsprechend ausgegeben und die Fehlermeldung ist, im FE sowie im BE, weg.
Hier die betreffenden Zeilen (112-116)
Bleibt die Frage ob das der richtige Weg ist....

//if (sizeof($matches{1})){
//$count = strlen($matches{1});
//$offset += $count;
//}

slackero added a commit that referenced this issue Jun 11, 2018
@slackero slackero self-assigned this Jun 11, 2018
@slackero slackero added the bug label Jun 11, 2018
@uwe367
Copy link
Author

uwe367 commented Jun 12, 2018

Leider besteht der Fehler nach dem Fix noch immer. Es wird die gleiche Fehlermeldung ausgegeben.
Spielt hierbei keine Rolle ob Neuistallation oder Update einer älteren Version. Bei beidem gestestet.

slackero added a commit that referenced this issue Jun 16, 2018
@slackero
Copy link
Owner

Schau noch mal, ich bekomme keinen Fehler. Habe htmlfilter.php weiter angepasst – etwas outdated.

@uwe367
Copy link
Author

uwe367 commented Jun 16, 2018

Ja, das wars. Keine Fehlermeldungen mehr. Verstehe allerdings nicht warum bei dir kein Fehler ausgegeben wird.
Aktuell verwende ich nun, nach dem letzten Fix der den Fehler nicht beseitigte, phpwcms 1.9.1-dev (2018/05/26, r549)
PHP Version 7.2.1
Hoster ist all-inkl

Danke fürs schnelle Bearbeiten und Erledigen.

@slackero
Copy link
Owner

Meinte nach der Anpassung kein Fehler, vorher habe ich nicht probiert. Aber so wie der Code war, erklärt sich die Meldung.

@uwe367
Copy link
Author

uwe367 commented Jun 16, 2018

So war das gemeint.
Nein, nach der heutigen Anpassung wird keine Fehlermeldung mehr ausgegeben.
Danke nochmals :)

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

No branches or pull requests

2 participants