Bug in Podcast plugin

Found a bug? Tell us!!
Post Reply
User avatar
ReneSchmidt
Regular
Posts: 13
Joined: Sat Jan 22, 2005 7:34 pm
Contact:

Bug in Podcast plugin

Post by ReneSchmidt » Mon Sep 26, 2005 4:49 pm

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

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

Re: Bug in Podcast plugin

Post by garvinhicking » Mon Sep 26, 2005 5:39 pm

Thanks a lot! Just fixed this bug thanks to your patch :)

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