Geeklog Forums

I noticed a problem with the "What's Related" box - it would only add links up to the first image. So I took a look at the function, made it more efficient, and fixed the problem.

The main difference in the match is that this one doesn't recognize any tags in between <a href=""> and </a>. So, for example, <a href="..."><b>foo</b></a> will not be matched. Maybe I'll futz around with it to handle this if anyone's interested.


function COM_extractLinks( $fulltext, $maxlength = 26 )
{
$rel = array();

preg_match_all( &quot;/(&lt;a href=[^>]+&gt([^<]*)(&lt;/a&gt/i&quot;, $fulltext, $matches );

for ( $i=0; $i&lt; count( $matches[0] ); $i++ )
{
// if link is too long, shorten it and add ... at the end
if ( ( $maxlength &gt; 0 ) &amp;&amp; ( strlen( $matches[2][$i] ) &gt; $maxlength ) )
{
$matches[2][$i] = substr( $matches[2][$i], 0, $maxlength - 3 ) . '...';
$matches[0][$i] = $matches[1][$i] . $matches[2][$i] . $matches[3][$i];
}

$rel[] = COM_checkHTML( $matches[0][$i] );
}

return $rel;
}


[Note I did not post this in HTML mode because it changed some of the code into smilies...]

In case you're curious, this is what we went with:



Or you can take a look at it in lib-common.php (without the smilely faces) at: lib-common.php.

I'm just curious Is there any reason you are not using the [ code ] bb tags when posting code in the forum?

If it is not working well, I'd like to know.

---

Geeklog components by PortalParts -- www.portalparts.com

Vinny - thanks for posting that. Don't we want a case-insensitive match though?

Blaine - I did that because posting it using CODE translates smilies

e.g.

Yeh, it appear that updates of recent to GL have effected this feature.

---

Geeklog components by PortalParts -- www.portalparts.com

Blaine,

I put my code snippet in the code tags, but it put the smilely's in there anyway.

Also, I'm sure you noticed but the QUOTE tags are acting funny as well.

-Vinny

With the geeklog.net upgrade, the allowable HTML was changed. I need the pre tags for the code block formatting. Dirk fixed it a few hours ago. Let's see if that fixed both the quotes and code formatting.

---

Geeklog components by PortalParts -- www.portalparts.com

I added the case insensitive flag to the regex for COM_extractLinks. (Good catch asmaloney). It should show up in -rc2, and if not there then in the final release of 1.3.9.

-Vinny

GL 1.3.9sr1 still has the problem that links are only added up to the first image. Perhaps someone could finally get this messy preg_match_all() sorted out?

I've just tested this is Gl 1.3.9sr1 and it works with the exception of when you have a link like this:



which just won't show up in the whats related field, though links after this image still will. I'll work on this last little bug related to COM_extractLinks(). If you can demonstrate another bug, please post a URL so I can see it.

Thanks,
Vinny

I still have this problem too. I have a story with images [which themselves are links to unscaled versions] and none of the links on the page show up in What's Related.

Here's an example from my site.

Your problem has nothing to do with images, you use single quotes instead of double quotes in your links i.e.



The HTML spec calls for the use of double quotes. I'll see about accepting both when 1.3.10 is relased though.

-Vinny

Heh. That was one of the first things I checked. Using Firefox if you select some text on the page and use the context menu to 'View Selection Source', it shows double quotes even though the page source shows single quotes... I guess I out-Foxed myself.



Yet the W3C validator validates them alright.

Thanks for catching that for me.

The next version of Geeklog (1.3.10) will have a COM_extractLinks that supports single quotes and also nested HTML tags (including images, i.e. [imageX]).

-Vinny