code refactor: nested condition -> guard clause; $response var initia…

…lization to null
yurplan · Oct 14, 2019 · 04a4b4a · 04a4b4a
1 parent a53dab2
commit 04a4b4a
Showing 1 changed file with 16 additions and 13 deletions.
diff --git a/lib/classes/Swift/Transport/AbstractSmtpTransport.php b/lib/classes/Swift/Transport/AbstractSmtpTransport.php
@@ -331,20  331,23 @@ public function executeCommand($command, $codes = [], &$failures = null, $pipeli
         }
 
         $this->pipeline[] = [$command, $seq, $codes, $address];
 
         if ($pipeline && $this->pipelining) {
-            $response = null;
-        } else {
-            while ($this->pipeline) {
-                list($command, $seq, $codes, $address) = array_shift($this->pipeline);
-                $response = $this->getFullResponse($seq);
-                try {
-                    $this->assertResponseCode($response, $codes);
-                } catch (Swift_TransportException $e) {
-                    if ($this->pipeline && $address) {
-                        $failures[] = $address;
-                    } else {
-                        $this->throwException($e);
-                    }
             return null;
         }
 
         $response = null;
 
         while ($this->pipeline) {
             list($command, $seq, $codes, $address) = array_shift($this->pipeline);
             $response = $this->getFullResponse($seq);
             try {
                 $this->assertResponseCode($response, $codes);
             } catch (Swift_TransportException $e) {
                 if ($this->pipeline && $address) {
                     $failures[] = $address;
                 } else {
                     $this->throwException($e);
                 }
             }
         }