Bug fix; $_GET, $_POST, $_REQUEST, $_COOKIE, $_SESSION replacement codes. Improving and correcting regex pattern and callback namespace. See: #18

Author: JasWSInc

wp-redirects/wp-redirects.inc.php

@@ -217,7 +217,7 @@ public static function redirect_redirects()
 				$redirect_id = get_the_ID(); // Pull this one time only.
 
 				$to = (string)get_post_meta($redirect_id, 'wp_redirect_to', TRUE);
-				$to = preg_replace_callback('/%%\\\$([^\[]+?)(.+?)%%/i', 'wp_redirects\plugin::_url_e_gprcs_value', $to);
+				$to = preg_replace_callback('/%%\$([^[]+?)(\[.+?)%%/i', '\\wp_redirects\\plugin::_url_e_gprcs_value', $to);
 				$to = preg_replace('/%%(.+?)%%/i', '', $to); // Ditch any remaining replacement codes.
 
 				$to = // Cleanup any double slashes left over by replacement codes.
@@ -274,7 +274,7 @@ public function redirect_uri_patterns()
 						{
 							return urlencode((string)@$_is_regex_matches[$m[1]]);
 						}, $_to) : $_to);
-					$_to = preg_replace_callback('/%%\\\$([^\[]+?)(.+?)%%/i', 'wp_redirects\plugin::_url_e_gprcs_value', $_to);
+					$_to = preg_replace_callback('/%%\$([^[]+?)(\[.+?)%%/i', '\\wp_redirects\\plugin::_url_e_gprcs_value', $_to);
 					$_to = preg_replace('/%%(.+?)%%/i', '', $_to); // Ditch any remaining replacement codes.
 
 					$_to = // Cleanup any double slashes left over by replacement codes.
@@ -303,7 +303,7 @@ public static function _url_e_gprcs_value($m)
 					if(strlen($element_w_brackets = $m[2]) && preg_match('/^(?:(?:\[(["\'])[a-z0-9 \._\-]+?\\1\])|(?:\[[0-9]+\]))+$/i', $element_w_brackets))
 						eval('$value = urlencode(trim(stripslashes((string)@$'.$gprcs.$element_w_brackets.')));');
 
-				return (!empty($value)) ? $value : ''; // Default to empty string.
+				return !empty($value) ? $value : ''; // Default to empty string.
 			}
 
 			public static function get_redirect_hits($post_id) {