Catch an null given error #2

robin.kluth · robin.kluth · commit 0e4c7bfe65f9 · 2022-01-13T10:47:43.000+01:00
diff --git a/src/LdapAuth.php b/src/LdapAuth.php
@@ -380,8 +380,9 @@ public function searchUser(string $searchFor, $attributes = "", $searchFilter =
                     // Something is wrong with the search query
                     if (is_null($this->_l)) {
                         Yii::warning('ldap_search_error: null', __FUNCTION__);
+                    } else {
+                        Yii::warning('ldap_search_error: ' . ldap_error($this->_l), __FUNCTION__);
                     }
-                    Yii::warning('ldap_search_error: ' . ldap_error($this->_l), __FUNCTION__);
                     break;
                 }
                 ldap_parse_result($this->_l, $result, $errcode, $matcheddn, $errmsg, $referrals, $controls);

Original file line number	Diff line number	Diff line change
`@@ -380,8 +380,9 @@ public function searchUser(string $searchFor, $attributes = "", $searchFilter =`
`380`	`380`	`// Something is wrong with the search query`
`381`	`381`	`if (is_null($this->_l)) {`
`382`	`382`	`Yii::warning('ldap_search_error: null', __FUNCTION__);`
	`383`	`+ } else {`
	`384`	`+ Yii::warning('ldap_search_error: ' . ldap_error($this->_l), __FUNCTION__);`
`383`	`385`	`}`
`384`		`- Yii::warning('ldap_search_error: ' . ldap_error($this->_l), __FUNCTION__);`
`385`	`386`	`break;`
`386`	`387`	`}`
`387`	`388`	`ldap_parse_result($this->_l, $result, $errcode, $matcheddn, $errmsg, $referrals, $controls);`