Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.2.3

Bug #1729 With the new Anchor parsing ColorText is not shown
Submitted: 2004-06-27 22:43 UTC
From: paul at kungfoocoder dot org Assigned: pmjones
Status: Closed Package: Text_Wiki
PHP Version: 4.3.4 OS:
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 11 + 47 = ?

 
 [2004-06-27 22:43 UTC] paul at kungfoocoder dot org
Description: ------------ Basically the problem is simply that the URL parsing is done before the ColorTewxt parsing in the default rules list, this means that ##red|red text## is interpreted as a link to the anchor #red|red... this can be easily fixed by moving the ColorText Parser before the URL parser in the rules array.

Comments

 [2004-06-27 23:31 UTC] pmjones
Kungfoocoder is correct in describing the source of the bug. However, changing the rule order (which is an easy fix) will probably have negative consequences on parsing of other rules; wiki rule parsing is pretty dynamic, and changing one usually has effects on the others. Thus, the "better" fix is to work with the URL parsing regex, which I will do.
 [2004-06-28 15:41 UTC] pmjones
REQUEST FOR COMMENTS It turns out there is no good fix for this problem. The use of a single hash-mark for same-page anchors has a number of problems: (1) easy to confuse with numbered- list markup, (2) confuses the color text rule, (3) tries to make links out of normal-use numbers, e.g., "Rule #12" or "Reason #88". So my proposed "fix" is to not allow the shorthand method of "#anchor" to refer to a same-page named anchor target. Instead, you'll have to use the previous markup of "PageName#anchorname" (even if it's on the same page). Do any of the Text_Wiki users have a serious problem with this? If so, please email me directly, pmjones@ciaweb.net; if I don't hear back in 24 hours I'll commit my fix as described above.
 [2004-06-28 18:43 UTC] paul at kungfoocoder dot org
Well, for my own thoughts, I think that this is a perfectly acceptable solution. What may also be usefulis some way to easily automatically generate a list of overlapping regexes, so that this sort of thing can be more easily detected, since (as you say) the order of matching should not be important.
 [2004-06-28 23:14 UTC] pmjones
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. Kungfoocoder suggested that an automated test of the overlapping regexes would be useful; I agree. However, I am not bright enough to quickly figure out how that would work. I would be very happy to hear suggestions for automated testing of the parsing rules. :-)