Merge pull request #131 from matt-h/fix-wp_redirect

chandra-patel · web-flow · commit 37d1d033edb8 · 2016-12-22T22:51:22.000+05:30
wp_redirect should return true on success The core WP function does this and it should for compatibility. https://developer.wordpress.org/reference/functions/wp_redirect/ Uncovered in this issue with Redirection johngodley/redirection#157
diff --git a/nginx-helper.php b/nginx-helper.php
@@ -374,6 +374,8 @@ function wp_redirect( $location, $status = 302 )
 				$location = wp_sanitize_redirect( $location );
 			}
 			header( 'Location: ' . $location, true, $status );
+
+			return true;
 		}
 
 	}

Original file line number	Diff line number	Diff line change
`@@ -374,6 +374,8 @@ function wp_redirect( $location, $status = 302 )`
`374`	`374`	`$location = wp_sanitize_redirect( $location );`
`375`	`375`	`}`
`376`	`376`	`header( 'Location: ' . $location, true, $status );`
	`377`	`+`
	`378`	`+ return true;`
`377`	`379`	`}`
`378`	`380`
`379`	`381`	`}`