Scrutinizer fixes

Author: troosan

run_tests.sh

@@ -1,14 +1,20 @@
 #!/bin/bash
+echo "Running composer update"
+composer update
 
 ## PHP_CodeSniffer
+echo "Running CodeSniffer"
 ./vendor/bin/phpcs src/ tests/ --standard=PSR2 -n --ignore=src/PhpWord/Shared/PCLZip
 
 ## PHP-CS-Fixer
+echo "Running CS Fixer"
 ./vendor/bin/php-cs-fixer fix --diff --verbose --dry-run
 
 ## PHP Mess Detector
+echo "Running Mess Detector"
 ./vendor/bin/phpmd src/,tests/ text ./phpmd.xml.dist --exclude pclzip.lib.php
 
 ## PHPUnit
-./vendor/bin/phpunit -c ./ --no-coverage
+echo "Running PHPUnit"
+./vendor/bin/phpunit -c ./
 

src/PhpWord/Element/Field.php

@@ -74,7 +74,7 @@ class Field extends AbstractElement
     /**
      * Field text
      *
-     * @var TextRun | string
+     * @var TextRun|string
      */
     protected $text;
 
@@ -98,7 +98,7 @@ class Field extends AbstractElement
      * @param string $type
      * @param array $properties
      * @param array $options
-     * @param TextRun | string $text
+     * @param TextRun|string $text
      */
     public function __construct($type = null, $properties = array(), $options = array(), $text = null)
     {

src/PhpWord/Element/TrackChange.php

@@ -37,7 +37,7 @@ class TrackChange extends AbstractContainer
     /**
      * Date
      *
-     * @var DateTime
+     * @var \DateTime
      */
     private $date;
 

src/PhpWord/Reader/Word2007/Settings.php

@@ -150,11 +150,11 @@ protected function setZoom(XMLReader $xmlReader, PhpWord $phpWord, \DOMElement $
     protected function setRevisionView(XMLReader $xmlReader, PhpWord $phpWord, \DOMElement $node)
     {
         $revisionView = new TrackChangesView();
-        $revisionView->setMarkup($xmlReader->getAttribute('w:markup', $node));
+        $revisionView->setMarkup(filter_var($xmlReader->getAttribute('w:markup', $node), FILTER_VALIDATE_BOOLEAN));
         $revisionView->setComments($xmlReader->getAttribute('w:comments', $node));
         $revisionView->setInsDel($xmlReader->getAttribute('w:insDel', $node));
-        $revisionView->setFormatting($xmlReader->getAttribute('w:formatting', $node));
-        $revisionView->setInkAnnotations($xmlReader->getAttribute('w:inkAnnotations', $node));
+        $revisionView->setFormatting(filter_var($xmlReader->getAttribute('w:formatting', $node), FILTER_VALIDATE_BOOLEAN));
+        $revisionView->setInkAnnotations(filter_var($xmlReader->getAttribute('w:inkAnnotations', $node), FILTER_VALIDATE_BOOLEAN));
         $phpWord->getSettings()->setRevisionView($revisionView);
     }
 }

src/PhpWord/Shared/AbstractEnum.php

@@ -48,7 +48,7 @@ public static function values()
     /**
      * Returns true the value is valid for this enum
      *
-     * @param strign $value
+     * @param string $value
      * @return bool true if value is valid
      */
     public static function isValid($value)

src/PhpWord/Shared/ZipArchive.php

@@ -351,7 +351,7 @@ public function pclzipGetFromName($filename)
      * Returns the name of an entry using its index (emulate \ZipArchive)
      *
      * @param int $index
-     * @return string
+     * @return string|bool
      * @since 0.10.0
      */
     public function pclzipGetNameIndex($index)

src/PhpWord/Writer/ODText/Element/Text.php

@@ -18,7 +18,6 @@
 namespace PhpOffice\PhpWord\Writer\ODText\Element;
 
 use PhpOffice\PhpWord\Exception\Exception;
-use PhpOffice\PhpWord\Settings;
 
 /**
  * Text element writer