Fix Travis test errors

Author: ivanlanin

CHANGELOG.md

@@ -31,7 +31,7 @@ This release changed PHPWord license from LGPL 2.1 to LGPL 3.
 - Refactor: PHPMD recommendation: Change all `get...` method that returns `boolean` into `is...` or `has...` - @ivanlanin
 - Docs: Create gh-pages branch for API documentation - @Progi1984 GH-154
 - QA: Add `.scrutinizer.yml` and include `composer.lock` for preparation to Scrutinizer - @ivanlanin GH-186
-- Word2007 Writer: Refactor writer parts using composite pattern - @ivanlanin
+- Writer: Refactor writer parts using composite pattern - @ivanlanin
 - Docs: Show code quality and test code coverage badge on README
 
 ## 0.10.0 - 4 May 2014

composer.json

@@ -1,10 +1,10 @@
 {
     "name": "phpoffice/phpword",
-    "description": "PHPWord - Read, Create, and Write DOCX, ODT, and RTF documents in PHP",
+    "description": "PHPWord - A pure PHP library for reading and writing word processing documents (DOCX, ODT, RTF, HTML, PDF)",
     "keywords": [
         "PHP", "PhpOffice", "office", "PhpWord", "word", "template", "reader", "writer",
         "docx", "OOXML", "OpenXML", "Office Open XML", "ISO IEC 29500", "WordprocessingML",
-        "RTF", "Rich Text Format", "doc", "odt", "OpenDocument"
+        "RTF", "Rich Text Format", "doc", "odt", "OpenDocument", "PDF", "HTML"
     ],
     "homepage": "http://phpoffice.github.io",
     "type": "library",

src/PhpWord/Element/TextBox.php

@@ -36,8 +36,6 @@ class TextBox extends AbstractContainer
     /**
      * Create a new textbox
      *
-     * @param string $docPart
-     * @param integer $docPartId
      * @param mixed $style
      */
     public function __construct($style = null)

src/PhpWord/Style/AbstractStyle.php

@@ -230,9 +230,9 @@ protected function setFloatVal($value, $default = null)
      */
     protected function setEnumVal($value = null, $enum = array(), $default = null)
     {
-        if (!is_null($value) && !empty($enum) && !in_array($value, $enum)) {
+        if ($value != null && trim($value) != '' && !empty($enum) && !in_array($value, $enum)) {
             throw new \InvalidArgumentException('Invalid style value.');
-        } elseif (is_null($value)) {
+        } elseif (is_null($value) || trim($value) == '') {
             $value = $default;
         }
 

src/PhpWord/Style/Font.php

@@ -17,7 +17,6 @@
 
 namespace PhpOffice\PhpWord\Style;
 
-use PhpOffice\PhpWord\Exception\InvalidStyleException;
 use PhpOffice\PhpWord\PhpWord;
 
 /**
@@ -652,7 +651,7 @@ public function setShading($value = null)
     /**
      * Toggle $target property to false when $source true
      *
-     * @param mixed $target Target property
+     * @param bool $target Target property
      * @param bool $sourceValue
      */
     private function toggleFalse(&$target, $sourceValue)

src/PhpWord/Style/NumberingLevel.php

@@ -377,10 +377,11 @@ public function getHint()
      * @param string $value
      * @return self
      */
-    public function setHint($value)
+    public function setHint($value = null)
     {
         $enum = array('default', 'eastAsia', 'cs');
         $this->hint = $this->setEnumVal($value, $enum, $this->hint);
+
         return $this;
     }
 }

src/PhpWord/Style/TextBox.php

@@ -169,6 +169,24 @@ public function getInnerMargin()
         return array($this->innerMarginLeft, $this->innerMarginTop, $this->innerMarginRight, $this->innerMarginBottom);
     }
 
+    /**
+     * Has inner margin?
+     *
+     * @return bool
+     */
+    public function hasInnerMargins()
+    {
+        $hasInnerMargins = false;
+        $margins = $this->getInnerMargins();
+        for ($i = 0; $i < count($margins); $i++) {
+            if (!is_null($margins[$i])) {
+                $hasInnerMargins = true;
+            }
+        }
+
+        return $hasInnerMargins;
+    }
+
     /**
      * Set border size
      *

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

@@ -17,9 +17,6 @@
 
 namespace PhpOffice\PhpWord\Writer\ODText\Element;
 
-use PhpOffice\PhpWord\Element\TextBreak as TextBreakElement;
-use PhpOffice\PhpWord\Writer\ODText\Element\Element as ElementWriter;
-
 /**
  * Table element writer
  *

src/PhpWord/Writer/Word2007/Element/TextBox.php

@@ -17,7 +17,6 @@
 
 namespace PhpOffice\PhpWord\Writer\Word2007\Element;
 
-use PhpOffice\PhpWord\Style\TextBox as TextBoxStyle;
 use PhpOffice\PhpWord\Writer\Word2007\Style\TextBox as TextBoxStyleWriter;
 
 /**
@@ -35,11 +34,7 @@ public function write()
         $xmlWriter = $this->getXmlWriter();
         $element = $this->getElement();
         $style = $element->getStyle();
-
-        if ($style instanceof TextBoxStyle) {
-            $styleWriter = new TextBoxStyleWriter($xmlWriter, $style);
-            $styleWriter->write();
-        }
+        $styleWriter = new TextBoxStyleWriter($xmlWriter, $style);
 
         if (!$this->withoutP) {
             $xmlWriter->startElement('w:p');
@@ -57,20 +52,15 @@ public function write()
         $xmlWriter->startElement('v:shape');
         $xmlWriter->writeAttribute('type', '#_x0000_t0202');
         $styleWriter->write();
-
         $xmlWriter->startElement('v:textbox');
-        $margins = implode(', ', $style->getInnerMargin());
-        $xmlWriter->writeAttribute('inset', $margins);
-
+        $styleWriter->writeInnerMargin();
         $xmlWriter->startElement('w:txbxContent');
         $xmlWriter->startElement('w:p');
         $containerWriter = new Container($xmlWriter, $element);
         $containerWriter->write();
         $xmlWriter->endElement(); // w:p
         $xmlWriter->endElement(); // w:txbxContent
-
         $xmlWriter->endElement(); // v: textbox
-
         $styleWriter->writeW10Wrap();
         $xmlWriter->endElement(); // v:shape
         $xmlWriter->endElement(); // w:pict

src/PhpWord/Writer/Word2007/Element/TextRun.php

@@ -17,8 +17,6 @@
 
 namespace PhpOffice\PhpWord\Writer\Word2007\Element;
 
-use PhpOffice\PhpWord\Writer\Word2007\Style\Paragraph as ParagraphStyleWriter;
-
 /**
  * TextRun element writer
  *