Bug #79

HTML-Purifier und TinyMCE filtern zuviel weg

Added by following about 6 years ago. Updated over 3 years ago.

Status:erledigt% Done:

100%

Priority name:2 mittelEstimated time: (Total: 0.00 h)
Assignee:following
Target version:Version 14
Ticket Referenz:3307 mic@ 1/2012 Kategorien:listing logs editor

Description

mic@:

Wenn man beim Erstellen der Cachebeschreibung
in der HTML-Lasche ein strike-out eingibt (also bla),
dann in den Editor wechselt (wo es auch durchgestrichen
angezeigt wird, dann speichert... dann wird dieses HTML-
Attribut automatisch gefiltert, und der Text sieht wieder normal aus :-(


Subtasks

Bug #89: Umlautdomainnamen in Logs erledigtfollowing


Related issues

Related to OC Entwicklung - Orga #83: Defektes Alt-HTML korrigieren erledigt
Related to OC Entwicklung - Change #780: Update TinyMCE offen

Associated revisions

Revision b6bdbc44
Added by following almost 4 years ago

use lib2 HTML purifier for all code; updates #79, updates #130

Revision 1786f57c
Added by following almost 4 years ago

refined HTML purifier settings and updated HTML docs; updates #79

Revision 7a3821f4
Added by mambofive almost 4 years ago

Merge pull request #188 from following5/79-purifier

refined HTML purifier settings and updated HTML docs; updates #79

History

#1 Updated by following about 6 years ago

HTML-Kommentare werden wohl auch weggefiltert, was laut einer Nutzeranfrage (RT 3127) früher mal anders war.

#2 Updated by following about 6 years ago

  • Subject changed from Fehler / Probleme im HTML-Purifier to HTML-Purifier filtert zuviel weg

#3 Updated by following about 6 years ago

Nutzeranfrage / RT-Ticket 4934:

"Laut Liste der erlaubten Tags sind MAP, AREA, USEMAP sowie das Attribut NAME erlaubt. Leider werden sie alle herausgefiltert."

...

"Mein Ziel ist mittels

<a name="ZIEL"/>

und

<a href=#ZIEL"/>

an eine bestimmte Stelle in der Seite zu springen"

... und das ist anscheinend nicht machbar.

#4 Updated by following about 6 years ago

  • Status changed from neu to offen

#5 Updated by following about 6 years ago

Achtung: Der Purifier wird auch von der OKAPI-Methode services/log/submit verwendet! Also Vorsicht bei Änderungen, und die OKAPI testen. (Logsenden kann man auch ohne OAuth testen, indem man vorübergehend in submit.php, function options(), eine Option a la "'DEBUG_AS_USERNAME' => 'following'" einfügt. Das wirkt dann wie eine OAuth-Anmeldung unter diesem User.)

#6 Updated by following about 6 years ago

  • Kategorien set to listing logs

#7 Updated by following about 6 years ago

Ein Nutzer hatte sich eine einfache Möglichkeit gewünscht um Elemente auszublenden, als Teil eines Rätselcaches (RT Nr. 3127 vom 12.10.2011). <div style="display:none"> wurde früher mal akzeptiert, wird inzwischen aber gefiltert. Kommentare gehen (<!-- -->) im Gegensatz zu GC auch nicht.

#8 Updated by following almost 6 years ago

  • Kategorien changed from listing logs to listing logs editor

#9 Updated by following almost 6 years ago

#10 Updated by following almost 6 years ago

Für die lib2 wird eine neue HTML-Purifier-Installation verwendet, Einbindung via lib2/OcHTMLPurifier.class.php (bislang nur für neue Logs). Diese Klasse sollte dann überall eingesetzt werden und entsprechend konfiguriert, dass die erlaubten Tags und Attribute funktionieren.

#11 Updated by following over 5 years ago

  • Target version set to Version 11

#12 Updated by bohrsty almost 5 years ago

  • Target version changed from Version 11 to Version 13

#13 Updated by bohrsty almost 5 years ago

  • Target version deleted (Version 13)

#14 Updated by mic@ almost 5 years ago

Kleiner Hinweis:
Während das s-Tag für strikeout weggefiltert wird,
klappt das del-Tag (ebenfalls für strikeout) noch tadellos.
Details siehe mein Newbie-Listing:
http://www.opencaching.de/viewcache.php?cacheid=131335

#15 Updated by following almost 4 years ago

Wenn man direkt im HTML-Modus speichert, bleibt das Strikeout erhalten.

#18 Updated by following almost 4 years ago

  • Subject changed from HTML-Purifier filtert zuviel weg to HTML-Purifier und TinyMCE filtern zuviel weg

Inzwischen ist mir aufgefallen, dass auch der Editor manche Dinge wegfiltert. Müsste also auch besser konfiguriert werden, wenn möglich. Evtl. braucht's auch ein Update?

#19 Updated by following almost 4 years ago

  • Assignee set to following
  • Target version set to Version 14

#20 Updated by following almost 4 years ago

  • Subject changed from HTML-Purifier und TinyMCE filtern zuviel weg to HTML-Purifier
  • Status changed from offen to in Arbeit

zunächst mal der Purifier; ggf. neues Ticket für TinyMCE

#21 Updated by following almost 4 years ago

  • Subject changed from HTML-Purifier to HTML-Purifier filtert zuviel weg

#22 Updated by following almost 4 years ago

  • Status changed from in Arbeit to in Arbeit 30%
  • % Done changed from 0 to 30

#23 Updated by following almost 4 years ago

  • Subject changed from HTML-Purifier filtert zuviel weg to HTML-Purifier und TinyMCE filtern zuviel weg

#24 Updated by following almost 4 years ago

TinyMCE filtert
HTML4-Tags: <s>, <center>, <figure>
HTML4-Attribute li:value, ol:start, ul:type, p:align, thead/tbody/tfoot:align,valign
HTML5-Tags: <aside>, <details>, <figcaption>, <footer>, <mark>, <meter>, <nav>, <ruby>/<rp>/<rt>, <section>, <summary>, <time>

#25 Updated by following almost 4 years ago

  • Status changed from in Arbeit 30% to in Arbeit 50%
  • % Done changed from 30 to 50

#26 Updated by following almost 4 years ago

Alle gemeldeten Probleme mit Ausnahme von <s> waren per HTMLPurifier-Config behebbar. Zusätzlich aktiviert habe ich die Tags <q> und <strike>.

Nach einem Update von TinyMCE (#780) können evtl. weitere Tags und Attribute aktiviert werden, inklusive <s> und <center>. Beides ist aber auch durch CSS-Stile ersetzbar. TinyMCE enthält einen eigenen HTML-Filter, der anscheinend nicht konfigurierbar ist.

#27 Updated by following almost 4 years ago

  • Status changed from in Arbeit 50% to im Test
  • % Done changed from 50 to 90

#28 Updated by following over 3 years ago

  • Status changed from im Test to erledigt
  • % Done changed from 90 to 100

Also available in: Atom PDF