zbateson / mail-mime-parser

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;
     }

src/Header/Consumer/AbstractConsumer.php

Spacing +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
     {
         $sChars = implode('|', $this->getAllTokenSeparators());
         $mimePartPattern = MimeLiteralPart::MIME_PART_PATTERN;
-        return '~(' . $mimePartPattern . '|\\\\.|' . $sChars . ')~';
+        return '~('.$mimePartPattern.'|\\\\.|'.$sChars.')~';
     }
     
     /**

src/SimpleDi.php

Spacing +11 added lines, -11 removed lines

@@ -205,7 +205,7 @@ 
         if ($this->partStreamRegistry === null) {
             $this->registerStreamExtensions();
         }
-        return $this->getInstance('partStreamRegistry', __NAMESPACE__ . '\Stream\PartStreamRegistry');
+        return $this->getInstance('partStreamRegistry', __NAMESPACE__.'\Stream\PartStreamRegistry');
     }
     
     /**
@@ -215,7 +215,7 @@ 
      */
     public function getHeaderPartFactory()
     {
-        return $this->getInstance('headerPartFactory', __NAMESPACE__ . '\Header\Part\HeaderPartFactory');
+        return $this->getInstance('headerPartFactory', __NAMESPACE__.'\Header\Part\HeaderPartFactory');
     }
     
     /**
@@ -225,7 +225,7 @@ 
      */
     public function getMimeLiteralPartFactory()
     {
-        return $this->getInstance('mimeLiteralPartFactory', __NAMESPACE__ . '\Header\Part\MimeLiteralPartFactory');
+        return $this->getInstance('mimeLiteralPartFactory', __NAMESPACE__.'\Header\Part\MimeLiteralPartFactory');
     }
     
     /**
@@ -252,10 +252,10 @@ 
      */
     protected function registerStreamExtensions()
     {
-        stream_filter_register(UUDecodeStreamFilter::STREAM_FILTER_NAME, __NAMESPACE__ . '\Stream\UUDecodeStreamFilter');
-        stream_filter_register(UUEncodeStreamFilter::STREAM_FILTER_NAME, __NAMESPACE__ . '\Stream\UUEncodeStreamFilter');
-        stream_filter_register(CharsetStreamFilter::STREAM_FILTER_NAME, __NAMESPACE__ . '\Stream\CharsetStreamFilter');
-        stream_wrapper_register(PartStream::STREAM_WRAPPER_PROTOCOL, __NAMESPACE__ . '\Stream\PartStream');
+        stream_filter_register(UUDecodeStreamFilter::STREAM_FILTER_NAME, __NAMESPACE__.'\Stream\UUDecodeStreamFilter');
+        stream_filter_register(UUEncodeStreamFilter::STREAM_FILTER_NAME, __NAMESPACE__.'\Stream\UUEncodeStreamFilter');
+        stream_filter_register(CharsetStreamFilter::STREAM_FILTER_NAME, __NAMESPACE__.'\Stream\CharsetStreamFilter');
+        stream_wrapper_register(PartStream::STREAM_WRAPPER_PROTOCOL, __NAMESPACE__.'\Stream\PartStream');
         
         // originally created for HHVM compatibility, but decided to use them
         // instead of built-in stream filters for reliability -- it seems the
@@ -269,19 +269,19 @@ 
         // filters, so now they're all named mmp-convert.*
         stream_filter_register(
             'mmp-convert.quoted-printable-decode',
-            __NAMESPACE__ . '\Stream\ConvertStreamFilter'
+            __NAMESPACE__.'\Stream\ConvertStreamFilter'
         );
         stream_filter_register(
             'mmp-convert.quoted-printable-encode',
-            __NAMESPACE__ . '\Stream\ConvertStreamFilter'
+            __NAMESPACE__.'\Stream\ConvertStreamFilter'
         );
         stream_filter_register(
             Base64EncodeStreamFilter::STREAM_FILTER_NAME,
-            __NAMESPACE__ . '\Stream\Base64EncodeStreamFilter'
+            __NAMESPACE__.'\Stream\Base64EncodeStreamFilter'
         );
         stream_filter_register(
             Base64DecodeStreamFilter::STREAM_FILTER_NAME,
-            __NAMESPACE__ . '\Stream\Base64DecodeStreamFilter'
+            __NAMESPACE__.'\Stream\Base64DecodeStreamFilter'
         );
     }
 }

src/Message/Writer/MimePartWriter.php

Spacing +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
             $first = false;
             $read = ltrim($read, "\r\n");
         }
-        $read = $lastChars . $read;
+        $read = $lastChars.$read;
         $lastChars = '';
         $matches = null;
         if (preg_match('/[\r\n]+$/', $read, $matches)) {

src/Stream/PartStreamRegistry.php

Spacing +5 added lines, -5 removed lines

@@ -139,7 +139,7 @@ 
                 $handle,
                 'mailmimeparser-encode',
                 STREAM_FILTER_READ,
-                [ 'charset' => $part->getHeaderParameter('Content-Type', 'charset') ]
+                ['charset' => $part->getHeaderParameter('Content-Type', 'charset')]
             );
         }
     }
@@ -159,8 +159,8 @@ 
         if (empty($this->registeredHandles[$id])) {
             return null;
         }
-        $handle = fopen('mmp-mime-message://' . $id . '?start=' .
-            $start . '&end=' . $end, 'r');
+        $handle = fopen('mmp-mime-message://'.$id.'?start='.
+            $start.'&end='.$end, 'r');
         
         $this->attachEncodingFilterToStream($part, $handle);
         $this->attachCharsetFilterToStream($part, $handle);
@@ -182,8 +182,8 @@ 
         if (empty($this->registeredHandles[$id])) {
             return null;
         }
-        $handle = fopen('mmp-mime-message://' . $id . '?start=' .
-            $start . '&end=' . $end, 'r');
+        $handle = fopen('mmp-mime-message://'.$id.'?start='.
+            $start.'&end='.$end, 'r');
         
         $part->attachOriginalStreamHandle($handle);
     }