Comments on: Auto Content Links http://redyellow.co.uk/blog Sport, Mobiles, Development and did I mention Sport? Thu, 26 Jan 2012 09:10:45 +0000 hourly 1 http://wordpress.org/?v=3.3.1 By: Rodger Zeisler http://redyellow.co.uk/blog/plugins/auto-content-links/comment-page-1/#comment-1740 Rodger Zeisler Sat, 04 Sep 2010 22:56:57 +0000 http://redyellow.co.uk/#comment-1740 I see you released version 1.2 that includes an option to open the link in a new window. Awesome! One suggestion...instead of using target=_new, I would recommend using target=_blank. This will always open the link in a new window instead of opening a new window once and then refreshing that same window. Read this post for some insight: http://www.russellheimlich.com/blog/target_blank-vs-target_new/ Thanks for all your efforts. I see you released version 1.2 that includes an option to open the link in a new window. Awesome! One suggestion…instead of using target=_new, I would recommend using target=_blank. This will always open the link in a new window instead of opening a new window once and then refreshing that same window. Read this post for some insight:

http://www.russellheimlich.com/blog/target_blank-vs-target_new/

Thanks for all your efforts.

]]> By: Rich Gubby http://redyellow.co.uk/blog/plugins/auto-content-links/comment-page-1/#comment-1603 Rich Gubby Tue, 24 Aug 2010 20:22:21 +0000 http://redyellow.co.uk/#comment-1603 Latest version has your new regular expression in - many many thanks! Latest version has your new regular expression in – many many thanks!

]]> By: Rodger Zeisler http://redyellow.co.uk/blog/plugins/auto-content-links/comment-page-1/#comment-1599 Rodger Zeisler Tue, 24 Aug 2010 17:11:12 +0000 http://redyellow.co.uk/#comment-1599 I just emailed you an update to the auto-content-links plugin. We found a situation where wordpress short codes were being mangled so we added a little to the regular expression so it would not try to link matching words inside brackets, just like we had done for HTML tags. I just emailed you an update to the auto-content-links plugin. We found a situation where wordpress short codes were being mangled so we added a little to the regular expression so it would not try to link matching words inside brackets, just like we had done for HTML tags.

]]> By: Rich Gubby http://redyellow.co.uk/blog/plugins/auto-content-links/comment-page-1/#comment-1500 Rich Gubby Fri, 13 Aug 2010 13:58:42 +0000 http://redyellow.co.uk/#comment-1500 That's awesome - please be sure to thank your son for the regular expression - very much appreciated! I'll try and get the new code implemented as soon as I can! That’s awesome – please be sure to thank your son for the regular expression – very much appreciated!

I’ll try and get the new code implemented as soon as I can!

]]> By: Rodger Zeisler http://redyellow.co.uk/blog/plugins/auto-content-links/comment-page-1/#comment-1484 Rodger Zeisler Wed, 11 Aug 2010 21:44:41 +0000 http://redyellow.co.uk/#comment-1484 I like your Auto Content Links plugin. It worked in most cases but I had some problems with text being modified inside anchor and image HTML tags (which corrupted the HTML). I also had problems with text that had been linked being relinked (overridden) by this plugin (which made the link not go where it had originally been linked). My son, Ryan, looked at your code and made a change to auto-content-links.php to not allow content to be auto-linked if it was within an HTML tag (like an anchor or img tag) and to not relink text that is already linked. Within function autoContentLinksContent, right before the return you assign a value to $content using preg_replace. That line was modified as follows: $content = preg_replace('@('.$wordBoundary.$link['name'].$wordBoundary.')(?!([^]*<[^Aa]+>)* ([^]+)?)(?!([^)@', '<a href="'.$link['url'].'" rel="nofollow">'.$link['name'].'</a>', $content, $link['instances']); If you would incorporate this into your plugin, I won't have to worry about updates and you will have a more bulletproof plugin. If what I pasted in doesn't make it thru this comment post, you can email me and I will send the updated file directly to you. I like your Auto Content Links plugin. It worked in most cases but I had some problems with text being modified inside anchor and image HTML tags (which corrupted the HTML). I also had problems with text that had been linked being relinked (overridden) by this plugin (which made the link not go where it had originally been linked).

My son, Ryan, looked at your code and made a change to auto-content-links.php to not allow content to be auto-linked if it was within an HTML tag (like an anchor or img tag) and to not relink text that is already linked. Within function autoContentLinksContent, right before the return you assign a value to $content using preg_replace. That line was modified as follows:

$content = preg_replace(‘@(‘.$wordBoundary.$link['name'].$wordBoundary.’)(?!([^]*<[^Aa]+>)* ([^]+)?)(?!([^)@', '<a href="'.$link['url'].’” rel=”nofollow”>’.$link['name'].’</a>’, $content, $link['instances']);

If you would incorporate this into your plugin, I won’t have to worry about updates and you will have a more bulletproof plugin. If what I pasted in doesn’t make it thru this comment post, you can email me and I will send the updated file directly to you.

]]>