zytzagoo / smtp-validate-email

src/Validator.php

Spacing +32 added lines, -32 removed lines

@@ -223,8 +223,8 @@ 
             return false;
         }
 
-        $test     = 'catch-all-test-' . time();
-        $accepted = $this->rcpt($test . '@' . $domain);
+        $test     = 'catch-all-test-'.time();
+        $accepted = $this->rcpt($test.'@'.$domain);
         if ($accepted) {
             // Success on a non-existing address is a "catch-all"
             $this->domains_info[$domain]['catchall'] = true;
@@ -234,7 +234,7 @@ 
         // Log when we get disconnected while trying catchall detection
         $this->noop();
         if (!$this->connected()) {
-            $this->debug('Disconnected after trying a non-existing recipient on ' . $domain);
+            $this->debug('Disconnected after trying a non-existing recipient on '.$domain);
         }
 
         /**
@@ -283,7 +283,7 @@ 
             // Add the hostname itself with 0 weight (RFC 2821)
             $mxs[$domain] = 0;
 
-            $this->debug('MX records (' . $domain . '): ' . print_r($mxs, true));
+            $this->debug('MX records ('.$domain.'): '.print_r($mxs, true));
             $this->domains_info[$domain]          = [];
             $this->domains_info[$domain]['users'] = $users;
             $this->domains_info[$domain]['mxs']   = $mxs;
@@ -298,7 +298,7 @@ 
                     }
                 } catch (NoConnectionException $e) {
                     // Unable to connect to host, so these addresses are invalid?
-                    $this->debug('Unable to connect. Exception caught: ' . $e->getMessage());
+                    $this->debug('Unable to connect. Exception caught: '.$e->getMessage());
                     $this->setDomainResults($users, $domain, $this->no_conn_is_valid);
                 }
             }
@@ -309,7 +309,7 @@ 
                     // Say helo, and continue if we can talk
                     if ($this->helo()) {
                         // try issuing MAIL FROM
-                        if (!$this->mail($this->from_user . '@' . $this->from_domain)) {
+                        if (!$this->mail($this->from_user.'@'.$this->from_domain)) {
                             // MAIL FROM not accepted, we can't talk
                             $this->setDomainResults($users, $domain, $this->no_comm_is_valid);
                         }
@@ -340,7 +340,7 @@ 
                                 $this->noop();
                                 // RCPT for each user
                                 foreach ($users as $user) {
-                                    $address                 = $user . '@' . $domain;
+                                    $address                 = $user.'@'.$domain;
                                     $this->results[$address] = $this->rcpt($address);
                                     $this->noop();
                                 }
@@ -402,7 +402,7 @@ 
     private function setDomainResults(array $users, $domain, $val)
     {
         foreach ($users as $user) {
-            $this->results[$user . '@' . $domain] = $val;
+            $this->results[$user.'@'.$domain] = $val;
         }
     }
 
@@ -428,13 +428,13 @@ 
      */
     protected function connect($host)
     {
-        $remote_socket = $host . ':' . $this->connect_port;
+        $remote_socket = $host.':'.$this->connect_port;
         $errnum        = 0;
         $errstr        = '';
         $this->host    = $remote_socket;
 
         // Open connection
-        $this->debug('Connecting to ' . $this->host);
+        $this->debug('Connecting to '.$this->host);
         // @codingStandardsIgnoreLine
         $this->socket = /** @scrutinizer ignore-unhandled */ @stream_socket_client(
             $this->host,
@@ -447,8 +447,8 @@ 
 
         // Check and throw if not connected
         if (!$this->connected()) {
-            $this->debug('Connect failed: ' . $errstr . ', error number: ' . $errnum . ', host: ' . $this->host);
-            throw new NoConnectionException('Cannot open a connection to remote host (' . $this->host . ')');
+            $this->debug('Connect failed: '.$errstr.', error number: '.$errnum.', host: '.$this->host);
+            throw new NoConnectionException('Cannot open a connection to remote host ('.$this->host.')');
         }
 
         $result = stream_set_timeout($this->socket, $this->connect_timeout);
@@ -456,7 +456,7 @@ 
             throw new NoTimeoutException('Cannot set timeout');
         }
 
-        $this->debug('Connected to ' . $this->host . ' successfully');
+        $this->debug('Connected to '.$this->host.' successfully');
     }
 
     /**
@@ -473,7 +473,7 @@ 
         }
 
         if ($this->connected()) {
-            $this->debug('Closing socket to ' . $this->host);
+            $this->debug('Closing socket to '.$this->host);
             fclose($this->socket);
         }
 
@@ -533,7 +533,7 @@ 
         } catch (UnexpectedResponseException $e) {
             // Connected, but got an unexpected response, so disconnect
             $result = false;
-            $this->debug('Unexpected response after connecting: ' . $e->getMessage());
+            $this->debug('Unexpected response after connecting: '.$e->getMessage());
             $this->disconnect(false);
         }
 
@@ -549,11 +549,11 @@ 
     {
         try {
             // Modern
-            $this->send('EHLO ' . $this->from_domain);
+            $this->send('EHLO '.$this->from_domain);
             $this->expect(self::SMTP_GENERIC_SUCCESS, $this->command_timeouts['ehlo']);
         } catch (UnexpectedResponseException $e) {
             // Legacy
-            $this->send('HELO ' . $this->from_domain);
+            $this->send('HELO '.$this->from_domain);
             $this->expect(self::SMTP_GENERIC_SUCCESS, $this->command_timeouts['helo']);
         }
     }
@@ -574,7 +574,7 @@ 
         }
 
         // Issue MAIL FROM, 5 minute timeout
-        $this->send('MAIL FROM:<' . $from . '>');
+        $this->send('MAIL FROM:<'.$from.'>');
 
         try {
             $this->expect(self::SMTP_GENERIC_SUCCESS, $this->command_timeouts['mail']);
@@ -588,7 +588,7 @@ 
             $result = false;
 
             // Got something unexpected in response to MAIL FROM
-            $this->debug("Unexpected response to MAIL FROM\n:" . $e->getMessage());
+            $this->debug("Unexpected response to MAIL FROM\n:".$e->getMessage());
 
             // Hotmail has been known to do this + was closing the connection
             // forcibly on their end, so we're killing the socket here too
@@ -625,17 +625,17 @@ 
 
         // Issue RCPT TO, 5 minute timeout
         try {
-            $this->send('RCPT TO:<' . $to . '>');
+            $this->send('RCPT TO:<'.$to.'>');
             // Handle response
             try {
                 $this->expect($expected_codes, $this->command_timeouts['rcpt']);
                 $this->state['rcpt'] = true;
                 $valid               = true;
             } catch (UnexpectedResponseException $e) {
-                $this->debug('Unexpected response to RCPT TO: ' . $e->getMessage());
+                $this->debug('Unexpected response to RCPT TO: '.$e->getMessage());
             }
         } catch (Exception $e) {
-            $this->debug('Sending RCPT TO failed: ' . $e->getMessage());
+            $this->debug('Sending RCPT TO failed: '.$e->getMessage());
         }
 
         return $valid;
@@ -674,7 +674,7 @@ 
         if ($this->state['helo']) {
             $this->send('QUIT');
             $this->expect(
-                [self::SMTP_GENERIC_SUCCESS,self::SMTP_QUIT_SUCCESS],
+                [self::SMTP_GENERIC_SUCCESS, self::SMTP_QUIT_SUCCESS],
                 $this->command_timeouts['quit'],
                 true
             );
@@ -719,13 +719,13 @@ 
         // Must be connected
         $this->throwIfNotConnected();
 
-        $this->debug('send>>>: ' . $cmd);
+        $this->debug('send>>>: '.$cmd);
         // Write the cmd to the connection stream
-        $result = fwrite($this->socket, $cmd . self::CRLF);
+        $result = fwrite($this->socket, $cmd.self::CRLF);
 
         // Did it work?
         if (false === $result) {
-            throw new SendFailedException('Send failed on: ' . $this->host);
+            throw new SendFailedException('Send failed on: '.$this->host);
         }
 
         return $result;
@@ -754,7 +754,7 @@ 
 
         // Retrieve response
         $line = fgets($this->socket, 1024);
-        $this->debug('<<<recv: ' . $line);
+        $this->debug('<<<recv: '.$line);
 
         // Have we timed out?
         $info = stream_get_meta_data($this->socket);
@@ -808,7 +808,7 @@ 
              * No response in expect() probably means that the remote server
              * forcibly closed the connection so lets clean up on our end as well?
              */
-            $this->debug('No response in expect(): ' . $e->getMessage());
+            $this->debug('No response in expect(): '.$e->getMessage());
             $this->disconnect(false);
         }
 
@@ -912,9 +912,9 @@ 
         $this->log($str);
         if ($this->debug) {
             if ('cli' !== PHP_SAPI) {
-                $str = '<br/><pre>' . htmlspecialchars($str) . '</pre>';
+                $str = '<br/><pre>'.htmlspecialchars($str).'</pre>';
             }
-            echo "\n" . $str;
+            echo "\n".$str;
         }
     }
 
@@ -940,7 +940,7 @@ 
     private function stamp($msg)
     {
         $date = \DateTime::createFromFormat('U.u', sprintf('%.f', microtime(true)))->format('Y-m-d\TH:i:s.uO');
-        $line = '[' . $date . '] ' . $msg;
+        $line = '['.$date.'] '.$msg;
 
         return $line;
     }
@@ -979,7 +979,7 @@ 
         if (\method_exists($this, $camelized)) {
             return \call_user_func_array([$this, $camelized], $args);
         } else {
-            trigger_error('Fatal error: Call to undefined method ' . self::class . '::' . $name . '()', E_USER_ERROR);
+            trigger_error('Fatal error: Call to undefined method '.self::class.'::'.$name.'()', E_USER_ERROR);
         }
     }
 

Indentation +4 added lines, -4 removed lines

@@ -70,10 +70,10 @@
      */
     public $greylisted_considered_valid = true;
 
-     /**
-     * stream context arguments for connection socket, necessary to initiate Server IP (in case reverse IP)
-     * https://stackoverflow.com/a/8968016
-     */
+        /**
+         * stream context arguments for connection socket, necessary to initiate Server IP (in case reverse IP)
+         * https://stackoverflow.com/a/8968016
+         */
     public $stream_context_args = [];
     
     /**