diff options
| author | cash <cash.costello@gmail.com> | 2011-10-01 21:51:53 -0400 |
|---|---|---|
| committer | cash <cash.costello@gmail.com> | 2011-10-01 21:51:53 -0400 |
| commit | 3f3994647b238f05233da5f22f75a88582562308 (patch) | |
| tree | a867edc42c19f417a57da709b08d8ec96c1e1556 /mod/bookmarks/actions | |
| parent | 00433c37e49e1ca706809a1a5f1c515cd784fc22 (diff) | |
| download | elgg-3f3994647b238f05233da5f22f75a88582562308.tar.gz elgg-3f3994647b238f05233da5f22f75a88582562308.tar.bz2 | |
Fixes #3686 added work around for PHP bug when validating URLs with dashes
Diffstat (limited to 'mod/bookmarks/actions')
| -rw-r--r-- | mod/bookmarks/actions/bookmarks/save.php | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/mod/bookmarks/actions/bookmarks/save.php b/mod/bookmarks/actions/bookmarks/save.php index 19bb3c73a..f240c4b26 100644 --- a/mod/bookmarks/actions/bookmarks/save.php +++ b/mod/bookmarks/actions/bookmarks/save.php @@ -24,7 +24,24 @@ if ($address && !preg_match("#^((ht|f)tps?:)?//#i", $address)) { $address = "http://$address"; } -if (!$title || !$address || !filter_var($address, FILTER_VALIDATE_URL)) { +if (!$title || !$address) { + register_error(elgg_echo('bookmarks:save:failed')); + forward(REFERER); +} + +// see https://bugs.php.net/bug.php?id=51192 +$php_5_2_13_and_below = version_compare(PHP_VERSION, '5.2.14', '<'); +$php_5_3_0_to_5_3_2 = version_compare(PHP_VERSION, '5.3.0', '>=') && + version_compare(PHP_VERSION, '5.3.3', '<'); + +$validated = false; +if ($php_5_2_13_and_below || $php_5_3_0_to_5_3_2) { + $tmp_address = str_replace("-", "", $address); + $validated = filter_var($tmp_address, FILTER_VALIDATE_URL); +} else { + $validated = filter_var($address, FILTER_VALIDATE_URL); +} +if (!$validated) { register_error(elgg_echo('bookmarks:save:failed')); forward(REFERER); } |