zbateson / mail-mime-parser

src/Message/Part/NonMimePartFactory.php

Doc Comments -1 removed lines

@@ -18,7 +18,6 @@
      * 
      * @param string $messageObjectId
      * @param PartBuilder $partBuilder
-     * @param PartStreamFilterManager $partStreamFilterManager
      * @return \ZBateson\MailMimeParser\Message\Part\NonMimePart
      */
     public function newInstance(

src/Message/Part/PartBuilder.php

Doc Comments +4 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
      * Sets the specified property denoted by $name to $value.
      * 
      * @param string $name
-     * @param mixed $value
+     * @param string $value
      */
     public function setProperty($name, $value)
     {
@@ -172,7 +172,7 @@ 
      * Returns the value of the property with the given $name.
      * 
      * @param string $name
-     * @return mixed
+     * @return null|string
      */
     public function getProperty($name)
     {
@@ -185,7 +185,7 @@ 
     /**
      * Registers the passed PartBuilder as a child of the current PartBuilder.
      * 
-     * @param \ZBateson\MailMimeParser\Message\PartBuilder $partBuilder
+     * @param PartBuilder $partBuilder
      */
     public function addChild(PartBuilder $partBuilder)
     {
@@ -196,7 +196,7 @@ 
     /**
      * Returns all children PartBuilder objects.
      * 
-     * @return \ZBateson\MailMimeParser\Message\PartBuilder[]
+     * @return PartBuilder[]
      */
     public function getChildren()
     {

Spacing +4 added lines, -4 removed lines

@@ -339,8 +339,8 @@ 
         if ($this->streamPartEndPos === 0) {
             return null;
         }
-        return $this->streamWrapperProtocol . '://' . $messageObjectId
-            . '?start=' . $this->streamPartStartPos . '&end='
+        return $this->streamWrapperProtocol.'://'.$messageObjectId
+            . '?start='.$this->streamPartStartPos.'&end='
             . $this->streamPartEndPos;
     }
     
@@ -356,8 +356,8 @@ 
         if ($this->streamContentEndPos === 0) {
             return null;
         }
-        return $this->streamWrapperProtocol . '://' . $messageObjectId
-            . '?start=' . $this->streamContentStartPos . '&end='
+        return $this->streamWrapperProtocol.'://'.$messageObjectId
+            . '?start='.$this->streamContentStartPos.'&end='
             . $this->streamContentEndPos;
     }
     

src/Message/WritableMessage.php

Doc Comments +2 added lines, -2 removed lines

@@ -228,8 +228,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)
@@ -342,7 +342,7 @@ 
      * used for something else (e.g. changing a non-mime message to a multipart
      * mime message).
      * 
-     * @param MimePart $part
+     * @param WritableMessage $part
      * @return MimePart the newly-created MimePart   
     */
     private function createNewContentPartFromPart(MimePart $part)

Unused Use Statements -4 removed lines

@@ -6,10 +6,6 @@
  */
 namespace ZBateson\MailMimeParser\Message;
 
-use ZBateson\MailMimeParser\Header\HeaderFactory;
-use ZBateson\MailMimeParser\Header\ParameterHeader;
-use ZBateson\MailMimeParser\Message\Writer\MimePartWriter;
-
 /**
  * Represents a single part of a multi-part mime message.
  *

Spacing +6 added lines, -6 removed lines

@@ -49,7 +49,7 @@ 
         $part->setRawHeader(
             'Content-Type',
             "$mimeType;\r\n\tboundary=\"" 
-                . $this->getUniqueBoundary($mimeType) . '"'
+                . $this->getUniqueBoundary($mimeType).'"'
         );
     }
 
@@ -132,7 +132,7 @@ 
     private function getUniqueBoundary($mimeType)
     {
         $type = ltrim(strtoupper(preg_replace('/^(multipart\/(.{3}).*|.*)$/i', '$2-', $mimeType)), '-');
-        return uniqid('----=MMP-' . $type . $this->messageObjectId . '.', true);
+        return uniqid('----=MMP-'.$type.$this->messageObjectId.'.', true);
     }
     
     /**
@@ -425,9 +425,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'));
@@ -491,7 +491,7 @@ 
      */
     public function getSignaturePart()
     {
-        return $this->getChild(0, new PartFilter([ 'signedpart' => PartFilter::FILTER_INCLUDE ]));
+        return $this->getChild(0, new PartFilter(['signedpart' => PartFilter::FILTER_INCLUDE]));
     }
     
     /**
@@ -730,7 +730,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/SimpleDi.php

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
     
     /**
      * 
-     * @return type
+     * @return PartFactoryService
      */
     public function getPartFactoryService()
     {

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 namespace ZBateson\MailMimeParser;
 
 use ZBateson\MailMimeParser\Message\MessageParser;
-use ZBateson\MailMimeParser\Message\Part\MimePartFactory;
 use ZBateson\MailMimeParser\Message\Part\PartBuilderFactory;
 use ZBateson\MailMimeParser\Message\Part\PartFactoryService;
 use ZBateson\MailMimeParser\Message\Writer\MessageWriterService;

Spacing +13 added lines, -13 removed lines

@@ -173,7 +173,7 @@ 
     {
         return $this->getInstance(
             'partFilterFactory',
-            __NAMESPACE__ . '\Message\PartFilterFactory'
+            __NAMESPACE__.'\Message\PartFilterFactory'
         );
     }
     
@@ -221,7 +221,7 @@ 
     {
         return $this->getInstance(
             'partStreamFilterManagerFactory',
-            __NAMESPACE__ . '\Message\Part\PartStreamFilterManagerFactory'
+            __NAMESPACE__.'\Message\Part\PartStreamFilterManagerFactory'
         );
     }
     
@@ -236,7 +236,7 @@ 
         if ($this->partStreamRegistry === null) {
             $this->registerStreamExtensions();
         }
-        return $this->getInstance('partStreamRegistry', __NAMESPACE__ . '\Stream\PartStreamRegistry');
+        return $this->getInstance('partStreamRegistry', __NAMESPACE__.'\Stream\PartStreamRegistry');
     }
     
     /**
@@ -246,7 +246,7 @@ 
      */
     public function getHeaderPartFactory()
     {
-        return $this->getInstance('headerPartFactory', __NAMESPACE__ . '\Header\Part\HeaderPartFactory');
+        return $this->getInstance('headerPartFactory', __NAMESPACE__.'\Header\Part\HeaderPartFactory');
     }
     
     /**
@@ -256,7 +256,7 @@ 
      */
     public function getMimeLiteralPartFactory()
     {
-        return $this->getInstance('mimeLiteralPartFactory', __NAMESPACE__ . '\Header\Part\MimeLiteralPartFactory');
+        return $this->getInstance('mimeLiteralPartFactory', __NAMESPACE__.'\Header\Part\MimeLiteralPartFactory');
     }
     
     /**
@@ -283,10 +283,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
@@ -300,19 +300,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/MessageParser.php

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
                 $this->addRawHeaderToPart($header, $partBuilder);
                 $header = '';
             } else {
-                $line = "\r\n" . $line;
+                $line = "\r\n".$line;
             }
             $header .= rtrim($line, "\r\n");
         } while ($header !== '');

src/Message/Part/PartStreamFilterManager.php

Spacing +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
                     $handle,
                     'mailmimeparser-encode',
                     STREAM_FILTER_READ,
-                    [ 'charset' => $charset ]
+                    ['charset' => $charset]
                 );
             }
             $this->charset['type'] = $charset;

src/Message/Part/WritableMimePart.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,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]);
         }
     }
 

src/Message/Part/MimePart.php

Spacing +4 added lines, -4 removed lines

@@ -80,7 +80,7 @@ 
 
         $pbChildren = $partBuilder->getChildren();
         if (!empty($pbChildren)) {
-            $this->children = array_map(function ($child) use ($messageObjectId) {
+            $this->children = array_map(function($child) use ($messageObjectId) {
                 $childPart = $child->createMessagePart($messageObjectId);
                 $childPart->parent = $this;
                 return $childPart;
@@ -123,7 +123,7 @@ 
      */
     public function getAllParts(PartFilter $filter = null)
     {
-        $aParts = [ $this ];
+        $aParts = [$this];
         foreach ($this->children as $part) {
             if ($part instanceof MimePart) {
                 $aParts = array_merge($aParts, $part->getAllParts(null, true));
@@ -134,7 +134,7 @@ 
         if (!empty($filter)) {
             return array_values(array_filter(
                 $aParts,
-                [ $filter, 'filter' ]
+                [$filter, 'filter']
             ));
         }
         return $aParts;
@@ -182,7 +182,7 @@ 
     public function getChildParts(PartFilter $filter = null)
     {
         if ($filter !== null) {
-            return array_values(array_filter($this->children, [ $filter, 'filter' ]));
+            return array_values(array_filter($this->children, [$filter, 'filter']));
         }
         return $this->children;
     }

src/Message.php

Spacing +1 added lines, -1 removed lines

@@ -181,7 +181,7 @@
         );
         return array_values(array_filter(
             $parts,
-            function ($part) {
+            function($part) {
                 return !(
                     $part->isTextPart()
                     && $part->getContentDisposition() === 'inline'