summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrey Andreev <narf@bofh.bg>2013-01-22 01:06:27 +0100
committerAndrey Andreev <narf@bofh.bg>2013-01-22 01:06:27 +0100
commit8dbc9993b5dcd2a045dbf193a3ed198a55649bea (patch)
tree27257bf9ea895d5198c0bebb00855ffaa4aa2ce6
parent9f690f190f1aa503dfc6270e3a97d96196ae3cff (diff)
parent4de1198ccf88d4a448ff752f35630228b60c53a8 (diff)
Merge pull request #2164 from cryode/bug/auto_link
Fix and optimize auto_link() URL helper function.
-rw-r--r--system/helpers/url_helper.php45
1 files changed, 18 insertions, 27 deletions
diff --git a/system/helpers/url_helper.php b/system/helpers/url_helper.php
index 130f6f962..a6536cf81 100644
--- a/system/helpers/url_helper.php
+++ b/system/helpers/url_helper.php
@@ -383,43 +383,34 @@ if ( ! function_exists('auto_link'))
*/
function auto_link($str, $type = 'both', $popup = FALSE)
{
- if ($type !== 'email' && preg_match_all('#(^|\s|\(|\b)((http(s?)://)|(www\.))(\w+[^\s\)\<]+)#i', $str, $matches))
+ // Find and replace any URLs.
+ if ($type !== 'email' && preg_match_all('#\b(([\w-]+://?|www\.)[^\s()<>]+(?:\([\w\d]+\)|([^[:punct:]\s]|/)))#', $str, $matches, PREG_OFFSET_CAPTURE))
{
- $pop = ($popup) ? ' target="_blank" ' : '';
+ // Set our target HTML if using popup links.
+ $target = ($popup) ? 'target="_blank"' : '';
- for ($i = 0, $c = count($matches[0]); $i < $c; $i++)
+ // We process the links in reverse order (last -> first) so that
+ // the returned string offsets from preg_match_all() are not
+ // moved as we add more HTML.
+ foreach (array_reverse($matches[0]) as $match)
{
- if (preg_match('/(\.|\,)$/i', $matches[6][$i], $m))
- {
- $punct = $m[1];
- $matches[6][$i] = substr($matches[6][$i], 0, -1);
- }
- else
- {
- $punct = '';
- }
+ // $match is an array generated by the PREG_OFFSET_CAPTURE flag.
+ // $match[0] is the matched string, $match[1] is the string offset.
- $str = str_replace($matches[0][$i],
- $matches[1][$i].'<a href="http'.$matches[4][$i].'://'
- .$matches[5][$i].$matches[6][$i].'"'.$pop.'>http'
- .$matches[4][$i].'://'.$matches[5][$i]
- .$matches[6][$i].'</a>'.$punct,
- $str);
+ $anchor = anchor($match[0], '', $target);
+
+ $str = substr_replace($str, $anchor, $match[1], strlen($match[0]));
}
}
- if ($type !== 'url' && preg_match_all('/([a-zA-Z0-9_\.\-\+]+)@([a-zA-Z0-9\-]+)\.([a-zA-Z0-9\-\.]+)/i', $str, $matches, PREG_OFFSET_CAPTURE))
+ // Find and replace any emails.
+ if ($type !== 'url' && preg_match_all('#([\w\.\-\+]+@[a-z0-9\-]+\.[a-z0-9\-\.]+[^[:punct:]\s])#i', $str, $matches, PREG_OFFSET_CAPTURE))
{
- for ($i = count($matches[0]) - 1; $i > -1; $i--)
+ foreach (array_reverse($matches[0]) as $match)
{
- if (preg_match('/(\.|\,)$/i', $matches[3][$i][0], $m))
- {
- $matches[3][$i][0] = substr($matches[3][$i][0], 0, -1);
- }
-
- if (filter_var(($m = $matches[1][$i][0].'@'.$matches[2][$i][0].'.'.$matches[3][$i][0]), FILTER_VALIDATE_EMAIL) !== FALSE)
+ if (filter_var($match[0], FILTER_VALIDATE_EMAIL) !== FALSE)
{
- $str = substr_replace($str, safe_mailto($m), $matches[0][$i][1], strlen($m));
+ $str = substr_replace($str, safe_mailto($match[0]), $match[1], strlen($match[0]));
}
}
}