Page 1 of 1

Bug in Podcast plugin

Posted: Mon Sep 26, 2005 4:49 pm
by ReneSchmidt
Hello people,

there is a bug in the Podcast plugin v1.10 (serendipity_event_podcast.php) that makes it render enclosure tags incorrectly.

Example:

Code: Select all

Im Netz gibt es recht viel kostenlose Musik: man muss nur wissen, wo sie zu finden ist. Allerdings ist das meiste davon nicht nach meinem Geschmack und vieles davon nicht gut produziert.

Eine Ausnahme ist die "Swiss Circus in Stereo"-Compilation von <a href="http://www.alpinechic.net">alpinechic.net</a>. Eine Vorstellung der Zusammenstellung ist auf Phlow.net verfuegbar - dort ist dann auch der Link zum Download zu finden: <a href="http://www.phlow.net/arc/001251.php">www.Phlow.net</a>

Anspieltipp: <a href="http://www.alpinechic.net/ac/mp3/ac011-01_env-world.mp3">World von Env</a>


The enclosure tag is then looking like this:

Code: Select all

<enclosure url="http://www.alpinechic.net&quot;&gt;alpinechic.net&lt;/a&gt;. Eine Vorstellung der Zusammenstellung ist auf Phlow.net verfuegbar &amp;#8211; dort ist dann auch der Link zum Download zu finden: &lt;a href=&quot;http://www.phlow.net/arc/001251.php&quot;&gt;www.Phlow.net&lt;/a&gt;&lt;/p&gt;

 &lt;p&gt;Anspieltipp: &lt;a href=&quot;http://www.alpinechic.net/ac/mp3/ac011-01_env-world.mp3" type="audio/mp3"  />


The regex for media link recognition is flawed and this is the cure:

Code: Select all

189c189
<             $mediaLinkPattern = '@href\s*=\s*(\'|")(.+\.(' . implode('|', array_keys($this->mediaFileTypes)) . '))\1@Usie';
---
>             $mediaLinkPattern = '@href\s*=\s*(\'|")([^>]+\.(' . implode('|', array_keys($this->mediaFileTypes)) . '))\1@Usie';


Thanks for your attention.

Rene Schmidt

Re: Bug in Podcast plugin

Posted: Mon Sep 26, 2005 5:39 pm
by garvinhicking
Thanks a lot! Just fixed this bug thanks to your patch :)

Regards,
Garvin