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

Hier können Probleme und alles andere in Deutscher Sprache gelöst werden.
Post Reply
User avatar
Oliver
Regular
Posts: 178
Joined: Mon Jan 10, 2005 7:43 pm
Location: Herne, Germany
Contact:

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

Post by Oliver » Thu Apr 28, 2005 12:18 am

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
Image

User avatar
garvinhicking
Core Developer
Posts: 30020
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

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

Post by garvinhicking » Thu Apr 28, 2005 2:21 pm

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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/

User avatar
Oliver
Regular
Posts: 178
Joined: Mon Jan 10, 2005 7:43 pm
Location: Herne, Germany
Contact:

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

Post by Oliver » Thu Apr 28, 2005 5:27 pm

garvinhicking wrote: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

User avatar
garvinhicking
Core Developer
Posts: 30020
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

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

Post by garvinhicking » Fri Apr 29, 2005 1:19 pm

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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/

Post Reply