zbateson / mail-mime-parser

src/Message/UUEncodedPart.php

Spacing +2 added lines, -2 removed lines

@@ -50,11 +50,11 @@
         
         $this->setRawHeader(
             'Content-Type',
-            'application/octet-stream; name="' . addcslashes($filename, '"') . '"'
+            'application/octet-stream; name="'.addcslashes($filename, '"').'"'
         );
         $this->setRawHeader(
             'Content-Disposition',
-            'attachment; filename="' . addcslashes($filename, '"') . '"'
+            'attachment; filename="'.addcslashes($filename, '"').'"'
         );
         $this->setRawHeader('Content-Transfer-Encoding', 'x-uuencode');
     }

src/Message/MessageParser.php

Spacing +2 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
                 $this->addRawHeaderToPart($header, $part);
                 $header = '';
             } else {
-                $line = "\r\n" . $line;
+                $line = "\r\n".$line;
             }
             $header .= rtrim($line, "\r\n");
         } while ($header !== '');
@@ -177,8 +177,7 @@ 
     private function getParentBoundary($curBoundary, MimePart $parent = null)
     {
         return $parent !== null ?
-            $parent->getHeaderParameter('Content-Type', 'boundary') :
-            $curBoundary;
+            $parent->getHeaderParameter('Content-Type', 'boundary') : $curBoundary;
     }
     
     /**

src/Message/Writer/MimePartWriter.php

Spacing +1 added lines, -1 removed lines

@@ -481,7 +481,7 @@
             $first = false;
             $read = ltrim($read, "\r\n");
         }
-        $read = $lastChars . $read;
+        $read = $lastChars.$read;
         $read = preg_replace('/\r\n|\r|\n/', "\r\n", $read);
         $lastChars = '';
         $matches = null;

src/Header/Consumer/SubjectConsumer.php

Spacing +1 added lines, -1 removed lines

@@ -63,6 +63,6 @@
     protected function getTokenSplitPattern()
     {
         $sChars = implode('|', $this->getAllTokenSeparators());
-        return '~(' . $sChars . ')~';
+        return '~('.$sChars.')~';
     }
 }

src/Message.php

Doc Comments +2 added lines, -2 removed lines

@@ -337,8 +337,8 @@ 
      * Removes the 'inline' part with the passed contentType, at the given index
      * defaulting to the first 
      * 
-     * @param string $contentType
      * @param int $index
+     * @param string $mimeType
      * @return boolean true on success
      */
     protected function removePartByMimeType($mimeType, $index = 0)
@@ -645,7 +645,7 @@ 
      * Otherwise searches for a text/plain part to return.
      * 
      * @param string $mimeType
-     * @return MimeType or null if not found
+     * @return MimePart|null or null if not found
      */
     private function findOtherContentPartFor($mimeType)
     {

Indentation +1 added lines, -1 removed lines

@@ -451,7 +451,7 @@
      * 
      * @param MimePart $part
      * @return MimePart the newly-created MimePart   
-    */
+     */
     private function createNewContentPartFromPart(MimePart $part)
     {
         $contPart = $this->mimePartFactory->newMimePart();

Spacing +7 added lines, -7 removed lines

@@ -186,7 +186,7 @@ 
     private function getUniqueBoundary($mimeType)
     {
         $type = ltrim(strtoupper(preg_replace('/^(multipart\/(.{3}).*|.*)$/i', '$2-', $mimeType)), '-');
-        return uniqid('----=MMP-' . $type . $this->objectId . '.', true);
+        return uniqid('----=MMP-'.$type.$this->objectId.'.', true);
     }
     
     /**
@@ -201,7 +201,7 @@ 
         $part->setRawHeader(
             'Content-Type',
             "$mimeType;\r\n\tboundary=\"" 
-                . $this->getUniqueBoundary($mimeType) . '"'
+                . $this->getUniqueBoundary($mimeType).'"'
         );
     }
     
@@ -532,9 +532,9 @@ 
     private function overwrite8bitContentEncoding()
     {
         $parts = $this->getAllParts(new PartFilter([
-            'headers' => [ PartFilter::FILTER_INCLUDE => [
+            'headers' => [PartFilter::FILTER_INCLUDE => [
                 'Content-Transfer-Encoding' => '8bit'
-            ] ]
+            ]]
         ]));
         foreach ($parts as $part) {
             $contentType = strtolower($part->getHeaderValue('Content-Type', 'text/plain'));
@@ -598,7 +598,7 @@ 
      */
     public function getSignaturePart()
     {
-        return $this->getChild(0, new PartFilter([ 'signedpart' => PartFilter::FILTER_INCLUDE ]));
+        return $this->getChild(0, new PartFilter(['signedpart' => PartFilter::FILTER_INCLUDE]));
     }
     
     /**
@@ -829,7 +829,7 @@ 
         );
         return array_values(array_filter(
             $parts,
-            function ($part) {
+            function($part) {
                 return !(
                     $part->isTextPart()
                     && $part->getHeaderValue('Content-Disposition', 'inline') === 'inline'
@@ -890,7 +890,7 @@ 
     public function addAttachmentPart($stringOrHandle, $mimeType, $filename = null, $disposition = 'attachment')
     {
         if ($filename === null) {
-            $filename = 'file' . uniqid();
+            $filename = 'file'.uniqid();
         }
         $filename = iconv('UTF-8', 'US-ASCII//translit//ignore', $filename);
         $part = $this->createPartForAttachment();

src/Message/PartFilter.php

Doc Comments +1 added lines, -1 removed lines

@@ -170,7 +170,7 @@
      * 
      * @param string $name Name of the member variable
      * @param string $value The value to test
-     * @param array $valid an array of valid values
+     * @param integer[] $valid an array of valid values
      * @throws InvalidArgumentException
      */
     private function validateArgument($name, $value, array $valid)

Spacing +6 added lines, -6 removed lines

@@ -156,7 +156,7 @@ 
      */
     public function __construct(array $filter = [])
     {
-        $params = [ 'multipart', 'textpart', 'signedpart', 'headers' ];
+        $params = ['multipart', 'textpart', 'signedpart', 'headers'];
         foreach ($params as $param) {
             if (isset($filter[$param])) {
                 $this->__set($param, $filter[$param]);
@@ -178,8 +178,8 @@ 
         if (!in_array($value, $valid)) {
             $last = array_pop($valid);
             throw new InvalidArgumentException(
-                '$value parameter for ' . $name . ' must be one of '
-                . join(', ', $valid) . ' or ' . $last . ' - "' . $value
+                '$value parameter for '.$name.' must be one of '
+                . join(', ', $valid).' or '.$last.' - "'.$value
                 . '" provided'
             );
         }
@@ -194,11 +194,11 @@ 
      */
     public function setHeaders(array $headers)
     {
-        array_walk($headers, function ($v, $k) {
+        array_walk($headers, function($v, $k) {
             $this->validateArgument(
                 'headers',
                 $k,
-                [ static::FILTER_EXCLUDE, static::FILTER_INCLUDE ]
+                [static::FILTER_EXCLUDE, static::FILTER_INCLUDE]
             );
             if (!is_array($v)) {
                 throw new InvalidArgumentException(
@@ -225,7 +225,7 @@ 
             $this->validateArgument(
                 $name,
                 $value,
-                [ static::FILTER_OFF, static::FILTER_EXCLUDE, static::FILTER_INCLUDE ]
+                [static::FILTER_OFF, static::FILTER_EXCLUDE, static::FILTER_INCLUDE]
             );
             $this->$name = $value;
         } elseif ($name === 'headers') {

src/Message/MimePart.php

Spacing +4 added lines, -4 removed lines

@@ -92,7 +92,7 @@ 
     {
         if ($part !== $this) {
             $part->setParent($this);
-            array_splice($this->parts, ($position === null) ? count($this->parts) : $position, 0, [ $part ]);
+            array_splice($this->parts, ($position === null) ? count($this->parts) : $position, 0, [$part]);
         }
     }
     
@@ -167,14 +167,14 @@ 
      */
     public function getAllParts(PartFilter $filter = null)
     {
-        $aParts = [ $this ];
+        $aParts = [$this];
         foreach ($this->parts as $part) {
             $aParts = array_merge($aParts, $part->getAllParts(null, true));
         }
         if (!empty($filter)) {
             return array_values(array_filter(
                 $aParts,
-                [ $filter, 'filter' ]
+                [$filter, 'filter']
             ));
         }
         return $aParts;
@@ -222,7 +222,7 @@ 
     public function getChildParts(PartFilter $filter = null)
     {
         if ($filter !== null) {
-            return array_values(array_filter($this->parts, [ $filter, 'filter' ]));
+            return array_values(array_filter($this->parts, [$filter, 'filter']));
         }
         return $this->parts;
     }