Feb 062007
 

FreshPorts often tries to help by creating HTML links. Most of the time, it gets this right. But for a recent port, it got it very wrong.

Have a look at the PostgreSQL 8.2 page. Note the commits for 05 Feb 2007 and 09 Jan 2007.

The code is taking what it thinks is a PR reference and linking to it.

The code is here: http://beta.freshports.org/htmlify.phps

The test cases are here: http://beta.freshports.org/htmlify-test.php and the source code for that is here.

I think the problem is the [\b]* just before the advocacy|alpha|bin|conf|docs|… in htmlify.php

Can you see the solution?

13 Feb

I changed the [\b]* to [\w\s]+ and that seems to work much better. Comments?

Website Pin Facebook Twitter Myspace Friendfeed Technorati del.icio.us Digg Google StumbleUpon Premium Responsive