bogus output

samirnassar - April 16, 2008 - 17:21
Project:Pathologic
Version:5.x-1.1-beta2
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

I set up pathologic as indicated in Read Me.txt then edited the following post:

Perhaps I should say my brain is on <a href="http://wired.com/">Wired</a>. A couple of days ago I wrote about <a href="paradise-on-the-information-highway">why I like Radio Paradise so much</a>. Yesterday, while reading the <a href="http://www.wired.com/wired/issue/16-02">February issue of Wired Magazine</a>, I came across Brendan Koerner's <a href=""http://www.wired.com/culture/culturereviews/magazine/16-02/su_radio>Why Things Suck: Radio</a>. In five paragraphs her articulates why modern FM radio stations suck, concluding with:

The output comes out weird:

<p>Perhaps I should say my brain is on <a href="http://wired.com/">Wired</a>. A couple of days ago I wrote about <a href="/paradise-on-the-information-highway%22%3Ewhy+I+like+Radio+Paradise+so+much%3C/a%3E.+Yesterday%2C+while+reading+the+%3Ca+href%3D?Array"http://www.wired.com/wired/issue/16-02">February issue of Wired Magazine</a>, I came across Brendan Koerner's <a radio>Why Things Suck: Radio</a>. In five paragraphs her articulates why modern FM radio stations suck, concluding with:</p>

It looks like pathologic doesn't catch the closing anchor tag.

#1

Garrett Albright - April 16, 2008 - 20:26

Whoops. The Drupal 5 version of the module has the Drupal 6 version's code in it.

(Though it also doesn't help that your "Why Things Suck: Radio" link is malformed.)

#2

samirnassar - April 16, 2008 - 22:07

I'll wait for the 5.x version to come out again. The "Why Things Suck: Radio" link is fixed though, thanks. Change does not affect outcome.

#3

Garrett Albright - April 17, 2008 - 00:36
Status:active» fixed

Okay, with regards to your "why I like Radio Paradise so much" link, it looks like you found a case in which the regular expression I was using to find paths was broken. I'm not particularly sure why, but I did some tweaking and it now works for that case. Thanks for bringing that to my attention.

Beta 3 of the D5 Pathologic module should be online shortly. =]

#4

Anonymous (not verified) - May 1, 2008 - 00:42
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.