Markdown-Überschrift im extended entry im Feed nicht korrekt

Hier können Probleme und alles andere in Deutscher Sprache gelöst werden.
Post Reply
thh
Regular
Posts: 349
Joined: Thu Oct 26, 2006 2:38 pm
Location: Stuttgart, Germany
Contact:

Markdown-Überschrift im extended entry im Feed nicht korrekt

Post by thh » Sun Mar 29, 2015 11:02 am

Serendipity 2.0.1
PHP 5.4.39 (Debian Wheezy)

Ich verwende für die Eingabe meiner Beiträge Markdown. Das funktioniert (weitgehend) wunderbar (ich habe manchmal Schwierigkeiten mit Markdown in Kommentaren, und manche erweiterte Syntax von markdown-extra funktioniert nicht, aber das ist ein anderes Thema).

Schwierigkeiten gibt es aber im Feed, und zwar dann, wenn ich den erweiterten Eintrag nutze und diesen mit einer Überschrift beginne. Im Blogeintrag selbst wird das korrekt umgesetzt. Im Feed steht aber keine Überschrift, sondern das "rohe" Markdown, konkret beispielsweise

Code: Select all

### Eine Überschrift
Das betrifft auch nicht nur mich - ein aktuelles Beispiel wäre der Beitrag http://blog.sperrobjekt.de/content/1000 ... itHub.html im Sperrobjekt-Weblog. Die Überschrift "Results & Findings" ist im Blog korrekt. Im Feed unter http://feeds.feedburner.com/sperrobjekt steht stattdessen "### Results & Findings".

Als Ursache vermute ich eine fehlende Leerzeile vor der Überschrift; dann erkennt der Markdown-Parser sie nämlich nicht. Offenbar wird diese Leerzeile im Markdown-Quelltext beim Rendern des Blogeintrags eingefügt, im Feed aber nicht.

Ich kann offenbar auch manuell keine Leerzeile (am Ende des "Eintrags" oder am Anfang des "erweiterten Eintrags") einfügen, weil diese "geschluckt" wird.

Kann das jemand reproduzieren?

Ich hatte leider bisher keine Zeit, mir den Code anzugucken, möchte das aber gerne hier einwerfen, nachdem es mir eben nicht nur bei mir, sondern auch bei Matthias auffiel ...

mattsches
Regular
Posts: 440
Joined: Sat Nov 05, 2005 10:35 pm
Location: Wiesbaden, Germany
Contact:

Re: Markdown-Überschrift im extended entry im Feed nicht kor

Post by mattsches » Sun Mar 29, 2015 11:37 am

thh wrote:Das betrifft auch nicht nur mich - ein aktuelles Beispiel wäre der Beitrag http://blog.sperrobjekt.de/content/1000 ... itHub.html im Sperrobjekt-Weblog. Die Überschrift "Results & Findings" ist im Blog korrekt. Im Feed unter http://feeds.feedburner.com/sperrobjekt steht stattdessen "### Results & Findings".
Waaah! :wink:
Ich schaue mir das mal an. Vermutlich hast du recht mit der Leerzeile.

Update: Das Problem ist diese Zeile hier. Die Frage ist jetzt: Fixt man das an dieser Stelle oder im Markdown-Plugin? Ich neige dazu, in der functions_rss.inc.php ein

Code: Select all

$entry['body'] .= "\r\n" . $entry['extended'];
draus zu machen, aber ich kann nicht beurteilen, ob das evtl. andere Plugins (Textile, ...) beeinträchtigt.

I created an issue for this.

mattsches
Regular
Posts: 440
Joined: Sat Nov 05, 2005 10:35 pm
Location: Wiesbaden, Germany
Contact:

Re: Markdown-Überschrift im extended entry im Feed nicht kor

Post by mattsches » Sun Mar 29, 2015 9:12 pm

Sollte im nächsten Release von Serendipity gefixt sein.

thh
Regular
Posts: 349
Joined: Thu Oct 26, 2006 2:38 pm
Location: Stuttgart, Germany
Contact:

Re: Markdown-Überschrift im extended entry im Feed nicht kor

Post by thh » Sun Mar 29, 2015 9:52 pm

mattsches wrote:Sollte im nächsten Release von Serendipity gefixt sein.
Super, danke.

Ich habe den Patch direkt übernommen ... und dabei bemerkt, dass man vielleicht "\n" statt '\n' nehmen sollte, wenn man eine Leerzeile und nicht die Zeichen \n haben möchte. ;)

Ein etwas holpriger Pull request (mit der Live-Edit-Funktion von github) ist eingeworfen.

(Sorry, Du kannst gerne die Commit message ordentlicher machen oder einfach Deinen Patch updaten, mir fehlt momentan leider immer noch die Zeit, das sauber zu machen ...)

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

Re: Markdown-Überschrift im extended entry im Feed nicht kor

Post by garvinhicking » Mon Mar 30, 2015 11:10 am

Hi!

Super, danke. Sobald github wieder läuft (ich kann nicht pullen...) backporte ich den patch auch für 2.0...

LG,
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