Do not assign in `if()` in AbstractOauth2Action
authorTim Düsterhus <duesterhus@woltlab.com>
Thu, 1 Sep 2022 08:21:41 +0000 (10:21 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Thu, 1 Sep 2022 08:21:41 +0000 (10:21 +0200)
wcfsetup/install/files/lib/action/AbstractOauth2Action.class.php

index 64baf775b53bb059a3acd638a94d983fbd465364..64cf880ce889c1fd817c404ed9bd583930520f33 100644 (file)
@@ -229,7 +229,8 @@ abstract class AbstractOauth2Action extends AbstractAction
         $encodedParameters = \http_build_query($parameters, '', '&');
 
         $url = new Uri($this->getAuthorizeUrl());
-        if (($query = $url->getQuery())) {
+        $query = $url->getQuery();
+        if ($query !== '') {
             $url = $url->withQuery("{$query}&{$encodedParameters}");
         } else {
             $url = $url->withQuery($encodedParameters);