zbateson / mail-mime-parser

src/Message.php

Doc Comments -4 removed lines

@@ -502,7 +502,6 @@ 
      * Creates and returns a new MimePart for the signature part of a
      * multipart/signed message and assigns it to $this->signedSignaturePart.
      * 
-     * @param string $protocol
      * @param string $body
      */
     public function createSignaturePart($body)
@@ -522,7 +521,6 @@ 
      * Creates a multipart/mixed MimePart assigns it to $this->signedMixedPart
      * if the message contains attachments.
      * 
-     * @param array $parts
      */
     private function createMultipartMixedForSignedMessage()
     {
@@ -590,8 +588,6 @@ 
      * 
      * @param string $micalg The Message Integrity Check algorithm being used
      * @param string $protocol The mime-type of the signature body
-     * @param string $body The signature signed according to the value of
-     *        $protocol
      */
     public function setAsMultipartSigned($micalg, $protocol)
     {

Spacing +5 added lines, -5 removed lines

@@ -342,7 +342,7 @@ 
      */
     private function getUniqueBoundary()
     {
-        return uniqid('----=MMP-' . $this->objectId . '.', true);
+        return uniqid('----=MMP-'.$this->objectId.'.', true);
     }
     
     /**
@@ -357,7 +357,7 @@ 
         $part->setRawHeader(
             'Content-Type',
             "$mimeType;\r\n\tboundary=\"" 
-                . $this->getUniqueBoundary() . "\""
+                . $this->getUniqueBoundary()."\""
         );
     }
     
@@ -830,7 +830,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();
@@ -974,7 +974,7 @@ 
      * @param \ZBateson\MailMimeParser\MimePart $boundaryParent
      * @param string $boundary
      */
-    private function writePartBoundaries($handle, MimePart $part, MimePart $parent, MimePart &$boundaryParent, $boundary)
+    private function writePartBoundaries($handle, MimePart $part, MimePart $parent, MimePart & $boundaryParent, $boundary)
     {
         if ($boundaryParent !== $parent && $boundaryParent !== $part) {
             if ($boundaryParent !== null && $parent->getParent() !== $boundaryParent) {
@@ -997,7 +997,7 @@ 
      * @param \ZBateson\MailMimeParser\MimePart $parent
      * @param \ZBateson\MailMimeParser\MimePart $boundaryParent
      */
-    private function writePartTo($handle, MimePart $part, MimePart $parent, MimePart &$boundaryParent)
+    private function writePartTo($handle, MimePart $part, MimePart $parent, MimePart & $boundaryParent)
     {
         $boundary = $boundaryParent->getHeaderParameter('Content-Type', 'boundary');
         if (!empty($boundary)) {

src/MimePart.php

Doc Comments +2 added lines, -2 removed lines

@@ -174,7 +174,7 @@ 
     /**
      * Returns true if this part's mime type is multipart/*
      * 
-     * @return bool
+     * @return integer
      */
     public function isMultiPart()
     {
@@ -278,7 +278,7 @@ 
      * Note that mime headers aren't case sensitive.
      * 
      * @param string $name
-     * @return \ZBateson\MailMimeParser\Header\Header
+     * @return Header\AbstractHeader|null
      */
     public function getHeader($name)
     {

src/Stream/Base64EncodeStreamFilter.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
         $numBytes = strlen($converted);
         if ($this->numBytesWritten != 0) {
             $next = 76 - ($this->numBytesWritten % 76);
-            $converted = substr($converted, 0, $next) . "\r\n" . rtrim(chunk_split(substr($converted, $next)));
+            $converted = substr($converted, 0, $next)."\r\n".rtrim(chunk_split(substr($converted, $next)));
         } else {
             $converted = rtrim(chunk_split($converted));
         }
@@ -65,7 +65,7 @@ 
     private function readAndConvert($in, $out, &$consumed)
     {
         while ($bucket = stream_bucket_make_writeable($in)) {
-            $data = $this->leftovers->value . $bucket->data;
+            $data = $this->leftovers->value.$bucket->data;
             $consumed += $bucket->datalen;
             $nRemain = strlen($data) % 3;
             $toConvert = $data;
@@ -74,7 +74,7 @@ 
                 $this->leftovers->encodedValue = '';
             } else {
                 $this->leftovers->value = substr($data, -$nRemain);
-                $this->leftovers->encodedValue = base64_encode($this->leftovers->value) . "\r\n";
+                $this->leftovers->encodedValue = base64_encode($this->leftovers->value)."\r\n";
                 $toConvert = substr($data, 0, -$nRemain);
             }
             $this->convertAndAppend($toConvert, $out);

src/Stream/UUEncodeStreamFilter.php

Spacing +4 added lines, -4 removed lines

@@ -39,11 +39,11 @@ 
     private function convertAndAppend($data, $out)
     {
         $converted = convert_uuencode($data);
-        $cleaned = rtrim(substr(rtrim($converted), 0, -1));      // remove end line ` character
+        $cleaned = rtrim(substr(rtrim($converted), 0, -1)); // remove end line ` character
         if (empty($cleaned)) {
             return;
         }
-        $cleaned = "\r\n" . $cleaned;
+        $cleaned = "\r\n".$cleaned;
         stream_bucket_append($out, stream_bucket_new($this->stream, $cleaned));
     }
     
@@ -85,7 +85,7 @@ 
     private function readAndConvert($in, $out, &$consumed)
     {
         while ($bucket = stream_bucket_make_writeable($in)) {
-            $data = $this->leftovers->value . $bucket->data;
+            $data = $this->leftovers->value.$bucket->data;
             if (!$this->headerWritten) {
                 $this->writeUUEncodingHeader($out);
                 $this->headerWritten = true;
@@ -98,7 +98,7 @@ 
                 $this->leftovers->encodedValue = $this->getUUEncodingFooter();
             } else {
                 $this->leftovers->value = substr($data, -$nRemain);
-                $this->leftovers->encodedValue = "\r\n" .
+                $this->leftovers->encodedValue = "\r\n".
                     rtrim(substr(rtrim(convert_uuencode($this->leftovers->value)), 0, -1))
                     . $this->getUUEncodingFooter();
                 $toConvert = substr($data, 0, -$nRemain);

src/SimpleDi.php

Spacing +11 added lines, -11 removed lines

@@ -180,7 +180,7 @@ 
         if ($this->partStreamRegistry === null) {
             $this->registerStreamExtensions();
         }
-        return $this->getInstance('partStreamRegistry', __NAMESPACE__ . '\Stream\PartStreamRegistry');
+        return $this->getInstance('partStreamRegistry', __NAMESPACE__.'\Stream\PartStreamRegistry');
     }
     
     /**
@@ -190,7 +190,7 @@ 
      */
     public function getHeaderPartFactory()
     {
-        return $this->getInstance('headerPartFactory', __NAMESPACE__ . '\Header\Part\HeaderPartFactory');
+        return $this->getInstance('headerPartFactory', __NAMESPACE__.'\Header\Part\HeaderPartFactory');
     }
     
     /**
@@ -200,7 +200,7 @@ 
      */
     public function getMimeLiteralPartFactory()
     {
-        return $this->getInstance('mimeLiteralPartFactory', __NAMESPACE__ . '\Header\Part\MimeLiteralPartFactory');
+        return $this->getInstance('mimeLiteralPartFactory', __NAMESPACE__.'\Header\Part\MimeLiteralPartFactory');
     }
     
     /**
@@ -227,10 +227,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
@@ -242,21 +242,21 @@ 
         if (!in_array('convert.*', $filters)) {
             stream_filter_register(
                 'convert.quoted-printable-decode',
-                __NAMESPACE__ . '\Stream\ConvertStreamFilter'
+                __NAMESPACE__.'\Stream\ConvertStreamFilter'
             );
             stream_filter_register(
                 'convert.quoted-printable-encode',
-                __NAMESPACE__ . '\Stream\ConvertStreamFilter'
+                __NAMESPACE__.'\Stream\ConvertStreamFilter'
             );
         }
         // @codeCoverageIgnoreEnd
         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/MessageParser.php

Spacing +2 added lines, -3 removed lines

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