Completed
Pull Request — master (#11)
by
unknown
01:57
created
src/BounceMailHandler/BounceMailHandler.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
  */
17 17
 namespace BounceMailHandler;
18 18
 
19
-require_once __DIR__ . '/phpmailer-bmh_rules.php';
19
+require_once __DIR__.'/phpmailer-bmh_rules.php';
20 20
 
21 21
 /**
22 22
  * BounceMailHandler class
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
   {
311 311
     // before starting the processing, let's check the delete flag and do global deletes if true
312 312
     if (trim($this->deleteMsgDate) != '') {
313
-      echo 'processing global delete based on date of ' . $this->deleteMsgDate . '<br />';
313
+      echo 'processing global delete based on date of '.$this->deleteMsgDate.'<br />';
314 314
       $this->globalDelete();
315 315
     }
316 316
 
@@ -320,24 +320,24 @@  discard block
 block discarded – undo
320 320
       $this->moveHard = false;
321 321
     }
322 322
 
323
-    $port = $this->port . '/' . $this->service . '/' . $this->serviceOption;
323
+    $port = $this->port.'/'.$this->service.'/'.$this->serviceOption;
324 324
 
325 325
     set_time_limit(6000);
326 326
 
327 327
     if (!$this->testMode) {
328
-      $this->mailboxLink = imap_open('{' . $this->mailhost . ':' . $port . '}' . $this->boxname, $this->mailboxUserName, $this->mailboxPassword, CL_EXPUNGE | ($this->testMode ? OP_READONLY : 0));
328
+      $this->mailboxLink = imap_open('{'.$this->mailhost.':'.$port.'}'.$this->boxname, $this->mailboxUserName, $this->mailboxPassword, CL_EXPUNGE | ($this->testMode ? OP_READONLY : 0));
329 329
     } else {
330
-      $this->mailboxLink = imap_open('{' . $this->mailhost . ':' . $port . '}' . $this->boxname, $this->mailboxUserName, $this->mailboxPassword, ($this->testMode ? OP_READONLY : 0));
330
+      $this->mailboxLink = imap_open('{'.$this->mailhost.':'.$port.'}'.$this->boxname, $this->mailboxUserName, $this->mailboxPassword, ($this->testMode ? OP_READONLY : 0));
331 331
     }
332 332
 
333 333
     if (!$this->mailboxLink) {
334
-      $this->errorMessage = 'Cannot create ' . $this->service . ' connection to ' . $this->mailhost . $this->bmhNewLine . 'Error MSG: ' . imap_last_error();
334
+      $this->errorMessage = 'Cannot create '.$this->service.' connection to '.$this->mailhost.$this->bmhNewLine.'Error MSG: '.imap_last_error();
335 335
       $this->output();
336 336
 
337 337
       return false;
338 338
     }
339 339
 
340
-    $this->output('Connected to: ' . $this->mailhost . ' (' . $this->mailboxUserName . ')');
340
+    $this->output('Connected to: '.$this->mailhost.' ('.$this->mailboxUserName.')');
341 341
 
342 342
     return true;
343 343
   }
@@ -352,14 +352,14 @@  discard block
 block discarded – undo
352 352
     $dateArr = explode('-', $this->deleteMsgDate); // date format is yyyy-mm-dd
353 353
     $delDate = mktime(0, 0, 0, $dateArr[1], $dateArr[2], $dateArr[0]);
354 354
 
355
-    $port = $this->port . '/' . $this->service . '/' . $this->serviceOption;
356
-    $mboxt = imap_open('{' . $this->mailhost . ':' . $port . '}', $this->mailboxUserName, $this->mailboxPassword, OP_HALFOPEN);
355
+    $port = $this->port.'/'.$this->service.'/'.$this->serviceOption;
356
+    $mboxt = imap_open('{'.$this->mailhost.':'.$port.'}', $this->mailboxUserName, $this->mailboxPassword, OP_HALFOPEN);
357 357
 
358 358
     if ($mboxt === false) {
359 359
       return false;
360 360
     }
361 361
 
362
-    $list = imap_getmailboxes($mboxt, '{' . $this->mailhost . ':' . $port . '}', '*');
362
+    $list = imap_getmailboxes($mboxt, '{'.$this->mailhost.':'.$port.'}', '*');
363 363
 
364 364
     if (is_array($list)) {
365 365
       foreach ($list as $key => $val) {
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
         $nameRaw = $nameArr[count($nameArr) - 1];
369 369
 
370 370
         if (false === stripos($nameRaw, 'sent')) {
371
-          $mboxd = imap_open('{' . $this->mailhost . ':' . $port . '}' . $nameRaw, $this->mailboxUserName, $this->mailboxPassword, CL_EXPUNGE);
371
+          $mboxd = imap_open('{'.$this->mailhost.':'.$port.'}'.$nameRaw, $this->mailboxUserName, $this->mailboxPassword, CL_EXPUNGE);
372 372
           $messages = imap_sort($mboxd, SORTDATE, 0);
373 373
           $i = 0;
374 374
 
@@ -408,9 +408,9 @@  discard block
 block discarded – undo
408 408
   {
409 409
     if ($this->verbose >= $verboseLevel) {
410 410
       if (empty($msg)) {
411
-        echo $this->errorMessage . $this->bmhNewLine;
411
+        echo $this->errorMessage.$this->bmhNewLine;
412 412
       } else {
413
-        echo $msg . $this->bmhNewLine;
413
+        echo $msg.$this->bmhNewLine;
414 414
       }
415 415
     }
416 416
   }
@@ -433,13 +433,13 @@  discard block
 block discarded – undo
433 433
     }
434 434
 
435 435
     if (!$this->mailboxLink) {
436
-      $this->errorMessage = 'Cannot open the mailbox file to ' . $filePath . $this->bmhNewLine . 'Error MSG: ' . imap_last_error();
436
+      $this->errorMessage = 'Cannot open the mailbox file to '.$filePath.$this->bmhNewLine.'Error MSG: '.imap_last_error();
437 437
       $this->output();
438 438
 
439 439
       return false;
440 440
     }
441 441
 
442
-    $this->output('Opened ' . $filePath);
442
+    $this->output('Opened '.$filePath);
443 443
 
444 444
     return true;
445 445
   }
@@ -476,12 +476,12 @@  discard block
 block discarded – undo
476 476
     $unprocessedCount = 0;
477 477
     $deletedCount = 0;
478 478
     $movedCount = 0;
479
-    $this->output('Total: ' . $totalCount . ' messages ');
479
+    $this->output('Total: '.$totalCount.' messages ');
480 480
 
481 481
     // process maximum number of messages
482 482
     if ($fetchedCount > $this->maxMessages) {
483 483
       $fetchedCount = $this->maxMessages;
484
-      $this->output('Processing first ' . $fetchedCount . ' messages ');
484
+      $this->output('Processing first '.$fetchedCount.' messages ');
485 485
     }
486 486
 
487 487
     if ($this->testMode) {
@@ -525,7 +525,7 @@  discard block
 block discarded – undo
525 525
           $processedResult = $this->processBounce($x, 'DSN', $totalCount);
526 526
         } else {
527 527
           // not standard DSN msg
528
-          $this->output('Msg #' . $x . ' is not a standard DSN message', self::VERBOSE_REPORT);
528
+          $this->output('Msg #'.$x.' is not a standard DSN message', self::VERBOSE_REPORT);
529 529
 
530 530
           if ($this->debugBodyRule) {
531 531
             if ($structure->ifdescription) {
@@ -551,7 +551,7 @@  discard block
 block discarded – undo
551 551
             $processedResult = $this->processBounce($x, 'DSN', $totalCount);
552 552
           } else {
553 553
             // not standard DSN msg
554
-            $this->output('Msg #' . $x . ' is not a standard DSN message', self::VERBOSE_REPORT);
554
+            $this->output('Msg #'.$x.' is not a standard DSN message', self::VERBOSE_REPORT);
555 555
 
556 556
             if ($this->debugBodyRule) {
557 557
               $this->output("  Content-Type : {$match[1]}", self::VERBOSE_DEBUG);
@@ -561,10 +561,10 @@  discard block
 block discarded – undo
561 561
           }
562 562
         } else {
563 563
           // didn't get content-type header
564
-          $this->output('Msg #' . $x . ' is not a well-formatted MIME mail, missing Content-Type', self::VERBOSE_REPORT);
564
+          $this->output('Msg #'.$x.' is not a well-formatted MIME mail, missing Content-Type', self::VERBOSE_REPORT);
565 565
 
566 566
           if ($this->debugBodyRule) {
567
-            $this->output('  Headers: ' . $this->bmhNewLine . $header . $this->bmhNewLine, self::VERBOSE_DEBUG);
567
+            $this->output('  Headers: '.$this->bmhNewLine.$header.$this->bmhNewLine, self::VERBOSE_DEBUG);
568 568
           }
569 569
 
570 570
           $processedResult = $this->processBounce($x, 'BODY', $totalCount);
@@ -645,17 +645,17 @@  discard block
 block discarded – undo
645 645
       flush();
646 646
     }
647 647
 
648
-    $this->output($this->bmhNewLine . 'Closing mailbox, and purging messages');
648
+    $this->output($this->bmhNewLine.'Closing mailbox, and purging messages');
649 649
 
650 650
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
651 651
     @imap_expunge($this->mailboxLink);
652 652
     imap_close($this->mailboxLink);
653 653
 
654
-    $this->output('Read: ' . $fetchedCountOutput . ' messages');
655
-    $this->output($processedCount . ' action taken');
656
-    $this->output($unprocessedCount . ' no action taken');
657
-    $this->output($deletedCount . ' messages deleted');
658
-    $this->output($movedCount . ' messages moved');
654
+    $this->output('Read: '.$fetchedCountOutput.' messages');
655
+    $this->output($processedCount.' action taken');
656
+    $this->output($unprocessedCount.' no action taken');
657
+    $this->output($deletedCount.' messages deleted');
658
+    $this->output($movedCount.' messages moved');
659 659
 
660 660
     return true;
661 661
   }
@@ -763,7 +763,7 @@  discard block
 block discarded – undo
763 763
           break;
764 764
 
765 765
         default: // un-support Content-type
766
-          $this->output('Msg #' . $pos . ' is unsupported Content-Type:' . $structure->type, self::VERBOSE_REPORT);
766
+          $this->output('Msg #'.$pos.' is unsupported Content-Type:'.$structure->type, self::VERBOSE_REPORT);
767 767
 
768 768
           return false;
769 769
       }
@@ -810,7 +810,7 @@  discard block
 block discarded – undo
810 810
       }
811 811
 
812 812
       if ($this->testMode) {
813
-        $this->output('Match: ' . $ruleNumber . ':' . $ruleCategory . '; ' . $bounceType . '; ' . $email);
813
+        $this->output('Match: '.$ruleNumber.':'.$ruleCategory.'; '.$bounceType.'; '.$email);
814 814
       } else {
815 815
         // code below will use the Callback function, but return no value
816 816
         $params = array(
@@ -835,7 +835,7 @@  discard block
 block discarded – undo
835 835
     } else {
836 836
       // match rule, do bounce action
837 837
       if ($this->testMode) {
838
-        $this->output('Match: ' . $ruleNumber . ':' . $ruleCategory . '; ' . $bounceType . '; ' . $email);
838
+        $this->output('Match: '.$ruleNumber.':'.$ruleCategory.'; '.$bounceType.'; '.$email);
839 839
 
840 840
         return true;
841 841
       }
@@ -878,19 +878,19 @@  discard block
 block discarded – undo
878 878
     if (trim($mailbox) == '') {
879 879
       // this is a critical error with either the mailbox name blank or an invalid mailbox name
880 880
       // need to stop processing and exit at this point
881
-      echo 'Invalid mailbox name for move operation. Cannot continue: ' . $mailbox . "<br />\n";
881
+      echo 'Invalid mailbox name for move operation. Cannot continue: '.$mailbox."<br />\n";
882 882
       exit();
883 883
     }
884 884
 
885
-    $port = $this->port . '/' . $this->service . '/' . $this->serviceOption;
885
+    $port = $this->port.'/'.$this->service.'/'.$this->serviceOption;
886 886
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
887
-    $mbox = @imap_open('{' . $this->mailhost . ':' . $port . '}', $this->mailboxUserName, $this->mailboxPassword, OP_HALFOPEN);
887
+    $mbox = @imap_open('{'.$this->mailhost.':'.$port.'}', $this->mailboxUserName, $this->mailboxPassword, OP_HALFOPEN);
888 888
 
889 889
     if ($mbox === false) {
890 890
       return false;
891 891
     }
892 892
 
893
-    $list = imap_getmailboxes($mbox, '{' . $this->mailhost . ':' . $port . '}', '*');
893
+    $list = imap_getmailboxes($mbox, '{'.$this->mailhost.':'.$port.'}', '*');
894 894
     $mailboxFound = false;
895 895
 
896 896
     if (is_array($list)) {
@@ -905,7 +905,7 @@  discard block
 block discarded – undo
905 905
 
906 906
       if (($mailboxFound === false) && $create) {
907 907
         /** @noinspection PhpUsageOfSilenceOperatorInspection */
908
-        @imap_createmailbox($mbox, imap_utf7_encode('{' . $this->mailhost . ':' . $port . '}' . $mailbox));
908
+        @imap_createmailbox($mbox, imap_utf7_encode('{'.$this->mailhost.':'.$port.'}'.$mailbox));
909 909
         imap_close($mbox);
910 910
 
911 911
         return true;
Please login to merge, or discard this patch.