Completed
Pull Request — master (#4)
by Matthieu
03:46
created
src/Client.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -392,8 +392,8 @@
 block discarded – undo
392 392
         // Only tested with ROS pre 6.43, will test with post 6.43 => this could make legacy parameter obsolete ?
393 393
         if (count($response)>1 && $response[0] === '!done' && !$this->config('legacy'))
394 394
         {
395
-             $this->_config->set('legacy', true);
396
-             return $this->login();
395
+                $this->_config->set('legacy', true);
396
+                return $this->login();
397 397
         }
398 398
         // Return true if we have only one line from server and this line is !done
399 399
         return (1 === count($response)) &&  isset($response[0]) && ($response[0] === '!done');
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -390,13 +390,13 @@
 block discarded – undo
390 390
         //  - response is '!done'
391 391
         // => problem with legacy version, swap it and retry
392 392
         // Only tested with ROS pre 6.43, will test with post 6.43 => this could make legacy parameter obsolete ?
393
-        if (count($response)>1 && $response[0] === '!done' && !$this->config('legacy'))
393
+        if (count($response) > 1 && $response[0] === '!done' && !$this->config('legacy'))
394 394
         {
395 395
              $this->_config->set('legacy', true);
396 396
              return $this->login();
397 397
         }
398 398
         // Return true if we have only one line from server and this line is !done
399
-        return (1 === count($response)) &&  isset($response[0]) && ($response[0] === '!done');
399
+        return (1 === count($response)) && isset($response[0]) && ($response[0] === '!done');
400 400
     }
401 401
 
402 402
     /**
Please login to merge, or discard this patch.