0.7.1 --> 0.8 --> Anpassung von BBCode [URL]

[Oliver]

Hallo,

habe gerade ein Upgrade von 0.7.1 auf 0.8 gemacht, lief problemlos, bis auf einige Dateien, die ich manuell löschen muss, kein Ding (diesmal hab ich mir die 'Hinweise' ausgedruckt ).

Nach dem Upgrade funktionierte mein selbst erstelltes Template nicht mehr, auch kein Problem, layout.php in layout.php.old umbenannt, alles gut

Dann habe ich aber im Blog festgestellt, daß der [url=...]xxx[/url] nicht mehr funktionierte. D.h. es wurde

[url =http://cgi.ebay.de/ws/eBayISAPI.dll?ViewItem&category=9938&item=4545601587]DAS HIER[/url]

anstatt DAS HIER angezeigt. Das Ganze per <a href="...">xxx</a> einzubauen funktionierte nicht, der Atom-Feed konnte dann nicht mehr angezeigt werden.

Also habe ich im BBcode-Plugin folgenden Teil 'zurückgesetzt' auf den Stand von S9Y 0.7.1:

Original bbcode 0.8 für [url=...]

Code: Select all

// [url]
          '/(?<!\\\\)\[(google|search)\]'   . $pattern_query . '\[\/(google|search)\]/si'                 => "<a href=\"http://www.google.com/search?q=\\2\" target=\"_blank\" class=\"bb-url\">\\2</a>",
          '/(?<!\\\\)\[url(?::\w+)?\]www\.' . $pattern_url   . '\[\/url(?::\w+)?\]/si'                    => "<a href=\"http://www.\\1\" target=\"_blank\" class=\"bb-url\">\\1</a>",
          '/(?<!\\\\)\[url(?::\w+)?\]'      . $pattern_url   . '\[\/url(?::\w+)?\]/si'                    => "<a href=\"\\1\" target=\"_blank\" class=\"bb-url\">\\1</a>",
          '/(?<!\\\\)\[url(?::\w+)?='       . $pattern_url   . '?\](.*?)\[\/url(?::\w+)?\]/si'            => "<a href=\"\\1\" target=\"_blank\" class=\"bb-url\">\\2</a>",

bbcode 0.8 'Downgrade' auf 0.7:

Code: Select all

// [url]
          '/(?<!\\\\)\[(google|search)\]'   . $pattern_query . '\[\/(google|search)\]/si'                 => "<a href=\"http://www.google.com/search?q=\\2\" target=\"_blank\" class=\"bb-url\">\\2</a>",
          '/(?<!\\\\)\[url(?::\w+)?\]www\.(.*?)\[\/url(?::\w+)?\]/si'        => "<a href=\"http://www.\\1\" target=\"_blank\" class=\"bb-url\">\\1</a>",
          '/(?<!\\\\)\[url(?::\w+)?\](.*?)\[\/url(?::\w+)?\]/si'             => "<a href=\"\\1\" target=\"_blank\" class=\"bb-url\">\\1</a>",
          '/(?<!\\\\)\[url(?::\w+)?=(.*?)?\](.*?)\[\/url(?::\w+)?\]/si'      => "<a href=\"\\1\" target=\"_blank\" class=\"bb-url\">\\2</a>",

Meine Frage: Kann ich dadurch was 'kaputt machen'? Die Anzeige von [url=...]xxx[/url] funzt jetzt wieder ohne Probleme

Danke und viele Grüße

Oliver

[garvinhicking]

Oliver,

ja. Durch Deine Änderung hast Du Dir CrossSite Scripting Injections wieder reingeholt.

Probier stattdessen mal vom aktuellen 0.9 Snapshot die BBCode-DAtei zu nehmen. Dort haben wir noch ein paar mehr Zeichen erlaubt die in der URL genommen werden dürfen, dann sollte das klappen. Bitte drauf achten dass Du keine Leerzeichen oder so innerhalb des BBCodes eingibst...

Viele Grüße,
Garvin

[Oliver]

ja. Durch Deine Änderung hast Du Dir CrossSite Scripting Injections wieder reingeholt.

Probier stattdessen mal vom aktuellen 0.9 Snapshot die BBCode-DAtei zu nehmen. Dort haben wir noch ein paar mehr Zeichen erlaubt die in der URL genommen werden dürfen, dann sollte das klappen. Bitte drauf achten dass Du keine Leerzeichen oder so innerhalb des BBCodes eingibst...

Hui... Ok, das wusste ich nicht. Habe die BBCode-Datei direkt ausgetauscht. Nun funktionieren zwar die oben angegeben URLs nicht mehr, aber per <a href="..."></a> geht's dann doch, werde den ATOM-Feed ggf. abschalten.

Vielleicht noch 'ne nützliche BBCode-Erweiterung:

Code: Select all

'/(?<!\\\\)\[s(?::\w+)?\](.*?)\[\/s(?::\w+)?\]/si'                                              => "<span style="text-decoration:line-through">\\1</span>",

- für durchgestrichenen Text.

Danke nochmal für Deine Hilfe

Viele Grüße,

Oliver

[garvinhicking]

Hi Oliver!

Am besten Du nimmst den Code von SVN/Berlios und nutzt das. Da sollten die bestehenden Bugs behoben sein.

Ich persönlich bin kein so großer Freund von line-through text, da muss ich erstmal schauen ob ich über meinen schatten springen kann

Viele Grüße,
Garvin