Opencaching.de

Die Plattform opencaching.de => Entwicklung => Thema gestartet von: mic@ am 03. Juli 2015, 16:23:23

Titel: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 03. Juli 2015, 16:23:23
Hallo,
ich sammle in diesem Thread mal die schrägsten HTML-Ausreisser,
also wo ein Log gleich die Umgebung durcheinanderbringt.
http://www.opencaching.de/viewcache.php?wp=OC0063&log=A#log1054413
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: Slini11 am 03. Juli 2015, 16:30:36
Geht so ein wenig in die Richtung von http://redmine.opencaching.de/issues/83, oder ?
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: Siggiiiiii am 03. Juli 2015, 17:01:11
Sowohl dieser Log als auch der nächste sehen komisch aus. Frisst der Editor echt vorformatierten Text z.B. aus Word und übernimmt das?
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 03. Juli 2015, 17:01:32
Geht so ein wenig in die Richtung von http://redmine.opencaching.de/issues/83, oder ?

Jein. Ursache war ein float:left. Wenn man den rausfiltert schränkt es die Gestaltungsmöglichkeiten im Log ein. Dieser Workaround (https://github.com/OpencachingDeutschland/oc-server3/commit/46d6bd29b7f40ccb48a485f284d5489a9afe2985) sollte genügen.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 03. Juli 2015, 23:19:47
Sowohl dieser Log als auch der nächste sehen komisch aus. Frisst der Editor echt vorformatierten Text z.B. aus Word und übernimmt das?

Solche Probleme enstehen, wenn man in den HTML-Modus schaltet und HTML-Code eingibt, mit dem niemand gerechnet hatte. Es ist schon ein HTML-Filter eingebaut, aber der ist noch nicht perfekt.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 05. Juli 2015, 14:43:48
Hier ist auch noch einer:

http://www.opencaching.de/viewcache.php?cacheid=149492&log=A#log843688

Mag sich mal jemand anders an einem Workaround versuchen?
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: Siggiiiiii am 05. Juli 2015, 16:15:27
Hier ist auch noch einer:

http://www.opencaching.de/viewcache.php?cacheid=149492&log=A#log843688

Mag sich mal jemand anders an einem Workaround versuchen?

Bei beiden Logs stehen im HTML-Code styles, die man scheinbar in den Editor nicht ohne weiteres reinbekommt. Ich habe Logkopien von GC nach OC mit IE nach IE und FF nach FF und IE nach FF getestet. Entweder übernimmt der Editor nur Basistags wie <em> oder <p> oder bei etwas umfangreicheren Markierungen auch die css-Klassen. Aber die Styles wurden nicht kopiert. Auch nach dem Abspeichern steht davon nichts im HTML-Quellcode.

Die css-Klassen in den wirren OC logs sind die von GC verwendeten Klassen: class="LogsTable NoBottomSpacing" class="log-row" class="FloatLeft LogDisplayRight" class="Clear LogContent" class="LogText".

Ich habe mal beim Black-Jack-Team gefragt wegen http://www.opencaching.de/viewcache.php?wp=OC0063&log=A#log1054413. Es wurde nur per Katze&Paste kopiert und im Editor wurde nur z.B. <p> übernommen. Also quasi wie bei meinen Tests.

Ich habe über Tante Google die Klassen auch im Quellcode der Testsourcen von c:geo entdeckt. Ich hatte den Verdacht, dass eine App die Logs ala GC formatiert und über die Schnittstelle nach OC schickt, aber das widespricht sich mit der Info vom Black-Jack-Team.

Jetzt frage ich mich, wo kommen die langen style-Schwänze her?
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 05. Juli 2015, 16:22:27
Wo das herkommt ist wenig relevant, darauf haben wir keinen Einfluss. Die Frage ist, was man im OC-HTML-Code (https://github.com/OpencachingDeutschland/oc-server3/blob/master/htdocs/templates2/ocstyle/res_logentry_logitem.tpl) drumherum bauen kann, damit es nicht mehr aus dem Rahmen rausfließt - so wie bei dem anderen Fall oben.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: Siggiiiiii am 05. Juli 2015, 16:34:50
Und dem HTML-Purifier sagen: "Hau alles CSS weg, wir formatieren den Output selber." wäre zu radikal, oder?
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: dl6hbo am 05. Juli 2015, 16:37:32
Haben wir nicht einen "HTML-Purifier" in unserer Software ?   Ich erinnere dunkel, dass es darum Diskussionen gab.
Wenn der "zuschlägt" und auf etwas trifft, was zu der Zeit, als er geschrieben wurde, noch nicht einmal angedacht war, könnte leicht Unsinn oder eine fehlerhafte Formatierung dabei herauskommen.

Im Übrigen sahen die Logs zu obigem Link bei mir jetzt gut aus.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: Siggiiiiii am 05. Juli 2015, 16:47:12
Im Übrigen sahen die Logs zu obigem Link bei mir jetzt gut aus.

Da wurde ja jetzt was eingebaut. :)
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 05. Juli 2015, 17:00:01
Zitat von: following
Die Frage ist, was man im OC-HTML-Code (https://github.com/OpencachingDeutschland/oc-server3/blob/master/htdocs/templates2/ocstyle/res_logentry_logitem.tpl) drumherum bauen kann, damit es nicht mehr aus dem Rahmen rausfließt - so wie bei dem anderen Fall oben.

Was wäre, wenn man einfach ALLE möglichen Tags schliesst. Ist zwar absolut bescheuerter HTML-Code,
wenn man etwas schliesst, was nicht offen ist. Aber ich glaube, HTML verkraftet das...
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: bohrsty am 05. Juli 2015, 17:00:32
Wo das herkommt ist wenig relevant, darauf haben wir keinen Einfluss. Die Frage ist, was man im OC-HTML-Code (https://github.com/OpencachingDeutschland/oc-server3/blob/master/htdocs/templates2/ocstyle/res_logentry_logitem.tpl) drumherum bauen kann, damit es nicht mehr aus dem Rahmen rausfließt - so wie bei dem anderen Fall oben.

in diesem fall bleiben meine ich nur zwei radikalloesungen:
overflow-x: hiddenwas den inhalt komplett abschneidet und unlesbar macht, oder
overflow-x: auto
wobei ich eher zur zweiten variante tendiere, denn dann wird der browser (falls, wie in diesem fall noetig) scrollbalken anbieten und sonst alles lassen wie es ist...

das verhindert zwar nicht, dass es an diesen stellen bei so einer html-eingabe etwas doof aussieht, wenn man scrollen muss, aber ich finde es besser als solch einen "auswuchs" wie jetzt...
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: Siggiiiiii am 05. Juli 2015, 17:09:44
Haben wir nicht einen "HTML-Purifier" in unserer Software ?   Ich erinnere dunkel, dass es darum Diskussionen gab.
Wenn der "zuschlägt" und auf etwas trifft, was zu der Zeit, als er geschrieben wurde, noch nicht einmal angedacht war, könnte leicht Unsinn oder eine fehlerhafte Formatierung dabei herauskommen.

Ich denke da eher in die Richtung (wenn ich die Funktionsweise richtig verstanden habe):

Mit $config->set('CSS.AllowedProperties', array()); alle CSS Properties rauswerfen. Dann sind die styles weg. Genauer, htmlpurifier wirft alles raus, was er kennt. Man müsste dann auch nicht dauernd den purifier updaten und Neuerfindungen nachkonfigurieren, weil man ihm sagt: Hau alles weg, was Du kennst. Bleibt ein minimales Restrisiko.

Dann bleiben class, p, br etc, also die HTML Sachen im Quellcode übrig.
Da man die fremden Klassen auch nicht braucht, könnte man über HTML.Allowed nur noch die Elemente erlauben, die auch der Editor erlaubt.
Und wenn man den nutzt erhält man scheinbar nur Standardsachen wie <p>, <br>, <b>, usw. Das ist HTML 1.0. Gab es vor 20 Jahren schon.

Damit würde man dann auch der Schnittstelle verbieten, irgendwelchen CSS-Code einzuschleusen, denn Inline CSS wie in den angegebenen Fällen in den Logs gefunden, überschreibt den CSS Code von opencaching.de.

Es gibt dazu auch eine Online-Demo:
http://htmlpurifier.org/demo.php (http://htmlpurifier.org/demo.php)

Hier mal das Zeug aus dem Log einfügen:
<p><table class="LogsTable NoBottomSpacing" style="font-weight:inherit;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:middle;border-collapse:collapse;table-layout:fixed;background-color:#ffffff;width:950px;padding:0px;margin:0px;"><tbody style="font-weight:inherit;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:baseline;padding:0px;margin:0px;"><tr class="log-row" style="font-weight:inherit;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:baseline;padding:0px;margin:0px;"><td style="padding-top:4px;padding-right:10px;padding-bottom:4px;padding-left:5px;font-weight:normal;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:middle;border-top-width:1px;border-top-style:solid;border-top-color:#d7d7d7;border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:#d7d7d7;width:935px;margin:0px;">
<div class="FloatLeft LogDisplayRight" style="padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:1em;font-weight:inherit;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:baseline;width:790px;margin:0px;">
<div class="Clear LogContent" style="padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:1em;font-weight:inherit;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:baseline;clear:both;border-left-width:1px;border-left-style:solid;border-left-color:#d7d7d7;margin:0px;">
<p class="LogText" style="padding-top:.5em;padding-right:0px;padding-bottom:0px;padding-left:0px;font-weight:inherit;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:baseline;margin:0px;">Dieser Cache stand schon lange auf unserer Liste, hatten wir doch fleissig unsere acht Bücher umsortiert und die Möglichkeiten mitgezählt. Nun wollten wir wissen, ob wir uns auch nicht verzählt hatten. Es war alles im grünen Bereich und zur Belohnung gab es einen schönen Spaziergang durch den regenfeuchten Wald. <br />TFTC</p>
</div>
</div>
</td>
</tr></tbody></table></p>

Dann einen Haken bei AllowedPropeties rausnehmen und Felt leer lassen.
Dann bei HTML.Allowed Haken rausnehmen und p,b,br,ol,ul,i,em eintragen.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 27. August 2015, 10:04:14
Und hier wieder ein paar Listings, die unsere Webseite ordentlich durcheinanderwirbeln:

http://www.opencaching.de/viewcache.php?cacheid=121550
http://www.opencaching.de/viewcache.php?cacheid=121549
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: OConly am 27. August 2015, 10:50:32
Sind doch beide archiviert.  ???
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 27. August 2015, 10:53:36
Zitat von: OConly
Sind doch beide archiviert.  ???

Ich weiss, einen davon habe ich ja selber soeben ins Archiv geschickt.
Mir geht es auch eher darum zu zeigen, wo die Schwachstellen unseres Codes liegen.
Also welche Listings es schaffen, die OC-Seite aus den Angeln zu heben.
Dadurch kann man dann vielleicht lernen, wie man diese Verwirbelung verhindern kann.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 27. August 2015, 12:55:07
Das sind beides Altlistings aus der Zeit bis 2010, als es bei Opencaching.de noch keinen zuverlässigen "HTML-Purifier" gab. In den letzten Jahren sind meines Wissens keine kaputten Listings mehr entstanden.

Die archivierten Listings sollten aber auch vernünftig lesbar sein, daher danke für den Hinweis. Wird beim nächsten Reparaturdurchgang in Ordnung gebracht.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 27. August 2015, 13:32:20
Zitat von: following
In den letzten Jahren sind meines Wissens keine kaputten Listings mehr entstanden.

OK, dann werde ich in Zukunft mehr auf neue Verwirbelungen achten und melden (sofern diese überhaupt existieren).
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 18. Oktober 2015, 22:09:16
Zitat von: following
Die archivierten Listings sollten aber auch vernünftig lesbar sein, daher danke für den Hinweis.

Na dann hätte ich was für Dich: http://www.opencaching.de/viewcache.php?cacheid=142008
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: FlashCool am 20. Oktober 2015, 06:40:03
Zwar bereits archiviert, aber auch hier ist etwas Durcheinander geraten...

http://www.opencaching.de/viewcache.php?cacheid=143260
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: FlashCool am 20. Oktober 2015, 06:51:13
Noch ein Beispiel (siehe Logs):

http://www.opencaching.de/viewcache.php?cacheid=120398
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 21. Oktober 2015, 16:04:02
Das ist nun soweit korrigiert, danke für die Hinweise. (HTML repariert, Owner informiert)
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 04. November 2015, 23:36:12
Hier ein weiteres altes Listing:
http://www.opencaching.de/viewcache.php?cacheid=109572
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: FlashCool am 05. November 2015, 22:16:12
Auch hier sieht es etwas verwirbelt ab den Logs aus:
http://www.opencaching.de/viewcache.php?cacheid=134903
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 10. November 2015, 14:37:38
Noch eines: http://www.opencaching.de/viewcache.php?wp=OCAD04
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: vepeg am 20. Dezember 2015, 11:46:55
Hier ein Log von mir persönlich:
http://opencaching.de/OCFECF

Ich weiß auch nicht warum...
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mambofive am 20. Dezember 2015, 12:33:50
Das sieht danach aus, als hättest du per cut&paste den Namen des anderen Caches da rein kopiert, dabei wird dann Schriftart etc mitkopiert.
Da hilft wahrscheinlich, das Log zu bearbeiten, den kompletten Logtext zu markieren und dann Schriftart und Größe neu zu setzen.

Gesendet von meinem Nexus 5 mit Tapatalk

Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: vepeg am 20. Dezember 2015, 12:47:46
Danke! Nun sieht es normal aus. Mit markiern und Schriftart/-größe ändern funktionierte es nicht, dafür aber mit dem Löschen der Formatierung.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: dl6hbo am 20. Dezember 2015, 15:40:36
Ja, so ist es jetzt gut !
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 21. Dezember 2015, 18:39:38
Hier noch ein altes Listing, was ziemlich verwirbelt ist:
http://www.opencaching.de/viewcache.php?cacheid=127272
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: Slini11 am 15. Januar 2016, 16:02:47
Noch eins: http://opencaching.de/OC4C67
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 24. Januar 2016, 22:19:02
Soweit alles korrigiert und Owner informiert.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: FlashCool am 28. März 2016, 10:15:56
Hier noch ein weiteres Listing mit Verwirbelungen ab den Logs:

http://www.opencaching.de/viewcache.php?wp=OCA6FD&log=A#log1109726
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 10. April 2016, 10:51:17
Gerade entdeckt:
http://www.opencaching.de/viewcache.php?cacheid=134471
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 10. April 2016, 12:44:34
Okay. Es sind wieder acht beisammen, da lohnt sich der nächste Reparaturdurchlauf.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 27. Juli 2016, 07:17:55
Hier scheint etwas nicht zu stimmen, denn die folgenden Logs sind allesamt fett dargestellt:
http://www.opencaching.de/viewcache.php?cacheid=136209
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: kirstenko am 02. August 2016, 12:36:18
Hallo,
eine ganz schnelle Analyse meinerseits ergab, dass im HTML-Quellcode 13 Fettdruck-Tags (<b>) geöffnet, aber lediglich 12 wieder geschlossen (</b>) werden.
Mein Tipp wäre, dass zu einer der Zeilen 584 oder 588 kein schließendes Tag existiert.

Folgende Änderung an Zeile 585 hat das Problem insofern gelöst, dass das Listings komplett fett geschrieben bleibt, aber alles ab Zeile 647 (ab "Stages...") wieder normal geschrieben ist.
Zeile 585 mit zusätzlichem </b>-Tag:
Fragen</b></font></h2>

Leider weiß ich nicht, wer die Änderung am Listing bzw. in der DB vornehmen kann bzw. wie das Vorgehen in einem solchen Fall aussieht.

Viele Grüße
Jan
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 03. September 2016, 20:14:45
Auch wenn Peter die HTML-Verwirbelungen nicht mehr zurückdrehen kann (mangels Rechten),
so will ich die Liste erkannter Fehler weiter fortführen:

http://www.opencaching.de/viewcache.php?cacheid=124083
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: teiling88 am 07. September 2016, 23:00:47
Sobald ich einen komfortablen Zugang auf die Datenbank habe - fixe ich die Probleme gerne.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: dogesu am 07. September 2016, 23:43:50
ich habe auch noch ein paar Listings dazu beizutragen:

opencaching.de/OCB8F5 (http://opencaching.de/OCB8F5) 

opencaching.de/OC8171 (http://opencaching.de/OC8171) ?  alle Logeinträge erscheinen fett ?

opencaching.de/OCB978 (http://opencaching.de/OCB978) 

opencaching.de/OCB97F (http://opencaching.de/OCB97F)

opencaching.de/OCBEF8 (http://opencaching.de/OCBEF8) 

opencaching.de/OCAD8F (http://opencaching.de/OCAD8F) 

opencaching.de/OCA47C (http://opencaching.de/OCA47C) 

opencaching.de/OCA082 (http://opencaching.de/OCA082)
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: 4_Vs am 08. September 2016, 10:08:21
Heho,

geht das nur über die DB oder gibt es einen User, der alle Listings editieren kann?

VG
Micha
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: teiling88 am 08. September 2016, 16:39:54
geht das nur über die DB oder gibt es einen User, der alle Listings editieren kann?

Es ist mir aktuell nur der Weg über die Datenbank bekannt.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: FlashCool am 20. September 2016, 21:50:35
Einen hab ich noch...  ;)

http://opencaching.de/OC0D48
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: following am 08. Oktober 2016, 15:43:31
Heho,

geht das nur über die DB oder gibt es einen User, der alle Listings editieren kann?

VG
Micha

Es gibt ein spezielles Admin-Recht, mit dem man die Listings teilweise editieren kann. Anleitung dazu im internen Teamwiki unter "Cachebeschreibungen ändern". Das Recht sollte wirklich nur für diesen speziellen Fall genutzt werden.

Im Redmine wird dokumentiert, welche Listings geändert wurden: http://redmine.opencaching.de/issues/83
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: teiling88 am 26. Oktober 2016, 23:50:07
ich habe auch noch ein paar Listings dazu beizutragen:

opencaching.de/OCB8F5 (http://opencaching.de/OCB8F5) 

opencaching.de/OC8171 (http://opencaching.de/OC8171) ?  alle Logeinträge erscheinen fett ? - ist schon gefixed

opencaching.de/OCB978 (http://opencaching.de/OCB978) 

opencaching.de/OCB97F (http://opencaching.de/OCB97F)

opencaching.de/OCBEF8 (http://opencaching.de/OCBEF8) 

opencaching.de/OCAD8F (http://opencaching.de/OCAD8F) 

opencaching.de/OCA47C (http://opencaching.de/OCA47C) 

opencaching.de/OCA082 (http://opencaching.de/OCA082)
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: teiling88 am 26. Oktober 2016, 23:54:21
http://opencaching.de/OC0D48

So alle gemeldeten Caches habe ich angepasst.
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 05. September 2017, 17:02:16
Hier eine neue HTML-Verwirbelung:
https://www.opencaching.de/viewcache.php?cacheid=137479
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: teiling88 am 17. September 2017, 21:40:58
Hier eine neue HTML-Verwirbelung:
https://www.opencaching.de/viewcache.php?cacheid=137479

Erledigt!
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 28. April 2018, 16:21:40
Kleiner Hinweis auf eine besondere Liste:
https://www.opencaching.de/cachelist.php?id=1742&key=reparieren
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: teiling88 am 23. Mai 2018, 08:33:49
Erledigt!
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: mic@ am 27. August 2019, 11:12:21
Nach langer Zeit mal wieder eine HTML-Verwirbelung entdeckt.
Hier wabert die grüne Schriftfarbe über seine Grenzen...
https://www.opencaching.de/viewcache.php?cacheid=108292
Titel: Re: Bug: HTML - Verwirbelungen
Beitrag von: teiling88 am 28. August 2019, 11:12:55
Erledigt!