Merge pull request #70 from forevermatt/patch-2

pitbulk · web-flow · commit ca15871c0ecb · 2018-08-16T22:36:13.000+02:00
Stop using `$blog_id` before it's defined
diff --git a/onelogin-saml-sso/php/functions.php b/onelogin-saml-sso/php/functions.php
@@ -265,7 +265,7 @@ function saml_acs() {
 	}
 
 	if ($user_id) {
-		if (is_multisite() && !is_user_member_of_blog($user_id, $blog_id)) {
+		if (is_multisite() && !is_user_member_of_blog($user_id)) {
     	    if (get_option('onelogin_saml_autocreate')) {
     	    	//Exist's but is not user to the current blog id
     	    	$blog_id = get_current_blog_id();

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ function saml_acs() {`
`265`	`265`	`}`
`266`	`266`
`267`	`267`	`if ($user_id) {`
`268`		`- if (is_multisite() && !is_user_member_of_blog($user_id, $blog_id)) {`
	`268`	`+ if (is_multisite() && !is_user_member_of_blog($user_id)) {`
`269`	`269`	`if (get_option('onelogin_saml_autocreate')) {`
`270`	`270`	`//Exist's but is not user to the current blog id`
`271`	`271`	`$blog_id = get_current_blog_id();`