chamilo / chamilo-lms

main/inc/lib/pear/HTML/QuickForm/select.php

Doc Comments +5 added lines, -8 removed lines

@@ -145,7 +145,7 @@ 
      *
      * @since     1.0
      * @access    public
-     * @return    array of selected values
+     * @return    string of selected values
      */
     function getSelected()
     {
@@ -208,7 +208,7 @@ 
     /**
      * Sets the value of the form element
      *
-     * @param     mixed    $values  Array or comma delimited string of selected values
+     * @param     mixed    $value  Array or comma delimited string of selected values
      * @since     1.0
      * @access    public
      * @return    void
@@ -226,7 +226,7 @@ 
      *
      * @since     1.0
      * @access    public
-     * @return    array of selected values
+     * @return    string of selected values
      */
     function getValue()
     {
@@ -257,7 +257,7 @@ 
      *
      * @since     1.0
      * @access    public
-     * @return    int
+     * @return    string|null
      */
     function getSize()
     {
@@ -357,12 +357,9 @@ 
         /**
      * Adds a new OPTION to the SELECT
      *
-     * @param     string    $text       Display text for the OPTION
-     * @param     string    $value      Value for the OPTION
-     * @param     mixed     $attributes Either a typical HTML attribute string
-     *                                  or an associative array
      * @since     1.0
      * @access    public
+     * @param string $label
      * @return    void
      */
     function addOptGroup($options, $label)

Indentation +18 added lines, -18 removed lines

@@ -355,16 +355,16 @@ 
 
 
         /**
-     * Adds a new OPTION to the SELECT
-     *
-     * @param     string    $text       Display text for the OPTION
-     * @param     string    $value      Value for the OPTION
-     * @param     mixed     $attributes Either a typical HTML attribute string
-     *                                  or an associative array
-     * @since     1.0
-     * @access    public
-     * @return    void
-     */
+         * Adds a new OPTION to the SELECT
+         *
+         * @param     string    $text       Display text for the OPTION
+         * @param     string    $value      Value for the OPTION
+         * @param     mixed     $attributes Either a typical HTML attribute string
+         *                                  or an associative array
+         * @since     1.0
+         * @access    public
+         * @return    void
+         */
     function addOptGroup($options, $label)
     {
         foreach ($options as $option) {
@@ -614,10 +614,10 @@ 
             }
             foreach ($value as $key => $item) {
                 $html .= '<input' . $this->_getAttrString(array(
-                             'type'  => 'hidden',
-                             'name'  => $name,
-                             'value' => $this->_values[$key]
-                         ) + $idAttr) . ' />';
+                                'type'  => 'hidden',
+                                'name'  => $name,
+                                'value' => $this->_values[$key]
+                            ) + $idAttr) . ' />';
             }
         }
         return $html;
@@ -626,10 +626,10 @@ 
     // }}}
     // {{{ exportValue()
 
-   /**
-    * We check the options and return only the values that _could_ have been
-    * selected. We also return a scalar value if select is not "multiple"
-    */
+    /**
+     * We check the options and return only the values that _could_ have been
+     * selected. We also return a scalar value if select is not "multiple"
+     */
     function exportValue(&$submitValues, $assoc = false)
     {
         $value = $this->_findValue($submitValues);

Spacing +24 added lines, -24 removed lines

@@ -86,7 +86,7 @@ 
             if (!empty($attributes['class'])) {
                 $oldClass = $attributes['class'];
             }
-            $attributes['class'] = $oldClass . ' selectpicker show-tick form-control';
+            $attributes['class'] = $oldClass.' selectpicker show-tick form-control';
             $attributes['data-live-search'] = 'true';
         }
         $columnsSize = isset($attributes['cols-size']) ? $attributes['cols-size'] : null;
@@ -196,7 +196,7 @@ 
     function getPrivateName()
     {
         if ($this->getAttribute('multiple')) {
-            return $this->getName() . '[]';
+            return $this->getName().'[]';
         } else {
             return $this->getName();
         }
@@ -312,7 +312,7 @@ 
      */
     function getMultiple()
     {
-        return (bool)$this->getAttribute('multiple');
+        return (bool) $this->getAttribute('multiple');
     } //end func getMultiple
 
     // }}}
@@ -332,7 +332,7 @@ 
     function addOption($text, $value, $attributes = null, $return_array = false)
     {
         if (null === $attributes) {
-            $attributes = array('value' => (string)$value);
+            $attributes = array('value' => (string) $value);
         } else {
             $attributes = $this->_parseAttributes($attributes);
             if (isset($attributes['selected'])) {
@@ -344,7 +344,7 @@ 
                     $this->_values[] = $value;
                 }
             }
-            $this->_updateAttrArray($attributes, array('value' => (string)$value));
+            $this->_updateAttrArray($attributes, array('value' => (string) $value));
         }
         if ($return_array) {
             return array('text' => $text, 'attr' => $attributes);
@@ -383,7 +383,7 @@ 
      * @return    PEAR_Error on error or true
      * @throws    PEAR_Error
      */
-    function loadArray($arr, $values=null)
+    function loadArray($arr, $values = null)
     {
         if (!is_array($arr)) {
             return PEAR::raiseError('Argument 1 of HTML_Select::loadArray is not a valid array');
@@ -420,7 +420,7 @@ 
      * @return    PEAR_Error on error or true
      * @throws    PEAR_Error
      */
-    function loadDbResult(&$result, $textCol=null, $valueCol=null, $values=null)
+    function loadDbResult(&$result, $textCol = null, $valueCol = null, $values = null)
     {
         if (!is_object($result) || !is_a($result, 'db_result')) {
             return PEAR::raiseError('Argument 1 of HTML_Select::loadDbResult is not a valid DB_result');
@@ -429,7 +429,7 @@ 
             $this->setValue($values);
         }
         $fetchMode = ($textCol && $valueCol) ? DB_FETCHMODE_ASSOC : DB_FETCHMODE_ORDERED;
-        while (is_array($row = $result->fetchRow($fetchMode)) ) {
+        while (is_array($row = $result->fetchRow($fetchMode))) {
             if ($fetchMode == DB_FETCHMODE_ASSOC) {
                 $this->addOption($row[$textCol], $row[$valueCol]);
             } else {
@@ -455,7 +455,7 @@ 
      * @return    void
      * @throws    PEAR_Error
      */
-    function loadQuery(&$conn, $sql, $textCol=null, $valueCol=null, $values=null)
+    function loadQuery(&$conn, $sql, $textCol = null, $valueCol = null, $values = null)
     {
         if (is_string($conn)) {
             require_once('DB.php');
@@ -502,7 +502,7 @@ 
      * @return    PEAR_Error on error or true
      * @throws    PEAR_Error
      */
-    function load(&$options, $param1=null, $param2=null, $param3=null, $param4=null)
+    function load(&$options, $param1 = null, $param2 = null, $param3 = null, $param4 = null)
     {
         switch (true) {
             case is_array($options):
@@ -539,32 +539,32 @@ 
             $strHtml = '';
 
             if ($this->getComment() != '') {
-                $strHtml .= $tabs . '<!-- ' . $this->getComment() . " //-->\n";
+                $strHtml .= $tabs.'<!-- '.$this->getComment()." //-->\n";
             }
 
             if (!$this->getMultiple()) {
                 $attrString = $this->_getAttrString($this->_attributes);
             } else {
                 $myName = $this->getName();
-                $this->setName($myName . '[]');
+                $this->setName($myName.'[]');
                 $attrString = $this->_getAttrString($this->_attributes);
                 $this->setName($myName);
             }
 
-            $strHtml .= $tabs . '<select ' . $attrString . ">\n";
+            $strHtml .= $tabs.'<select '.$attrString.">\n";
 
-            $strValues = is_array($this->_values)? array_map('strval', $this->_values): array();
+            $strValues = is_array($this->_values) ? array_map('strval', $this->_values) : array();
 
             foreach ($this->_options as $option) {
 
                 if (!empty($strValues) && in_array($option['attr']['value'], $strValues, true)) {
                     $option['attr']['selected'] = 'selected';
                 }
-                $strHtml .= $tabs . "<option" . $this->_getAttrString($option['attr']) . '>' .
-                    $option['text'] . "</option>";
+                $strHtml .= $tabs."<option".$this->_getAttrString($option['attr']).'>'.
+                    $option['text']."</option>";
             }
             foreach ($this->_optgroups as $optgroup) {
-                $strHtml .= $tabs . '<optgroup label="' . $optgroup['label'] . '">';
+                $strHtml .= $tabs.'<optgroup label="'.$optgroup['label'].'">';
                 foreach ($optgroup['options'] as $option) {
                     $text = $option['text'];
                     unset($option['text']);
@@ -573,11 +573,11 @@ 
                         $option['selected'] = 'selected';
                     }
 
-                    $strHtml .= $tabs . " <option" . $this->_getAttrString($option) . '>' .$text . "</option>";
+                    $strHtml .= $tabs." <option".$this->_getAttrString($option).'>'.$text."</option>";
                 }
                 $strHtml .= "</optgroup>";
             }
-            return $strHtml . $tabs . '</select>';
+            return $strHtml.$tabs.'</select>';
         }
     }
 
@@ -602,22 +602,22 @@ 
                 }
             }
         }
-        $html = empty($value)? '&nbsp;': join('<br />', $value);
+        $html = empty($value) ? '&nbsp;' : join('<br />', $value);
         if ($this->_persistantFreeze) {
             $name = $this->getPrivateName();
             // Only use id attribute if doing single hidden input
             if (1 == count($value)) {
                 $id     = $this->getAttribute('id');
-                $idAttr = isset($id)? array('id' => $id): array();
+                $idAttr = isset($id) ? array('id' => $id) : array();
             } else {
                 $idAttr = array();
             }
             foreach ($value as $key => $item) {
-                $html .= '<input' . $this->_getAttrString(array(
+                $html .= '<input'.$this->_getAttrString(array(
                              'type'  => 'hidden',
                              'name'  => $name,
                              'value' => $this->_values[$key]
-                         ) + $idAttr) . ' />';
+                         ) + $idAttr).' />';
             }
         }
         return $html;
@@ -635,7 +635,7 @@ 
         $value = $this->_findValue($submitValues);
         if (is_null($value)) {
             $value = $this->getValue();
-        } elseif(!is_array($value)) {
+        } elseif (!is_array($value)) {
             $value = array($value);
         }
         if (is_array($value) && !empty($this->_options)) {

main/inc/lib/pear/HTML/QuickForm/submit.php

Doc Comments +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
      * Freeze the element so that only its value is returned
      *
      * @access    public
-     * @return    void
+     * @return    boolean
      */
     function freeze()
     {

Indentation +4 added lines, -4 removed lines

@@ -70,10 +70,10 @@
     // }}}
     // {{{ exportValue()
 
-   /**
-    * Only return the value if it is found within $submitValues (i.e. if
-    * this particular submit button was clicked)
-    */
+    /**
+     * Only return the value if it is found within $submitValues (i.e. if
+     * this particular submit button was clicked)
+     */
     function exportValue(&$submitValues, $assoc = false)
     {
         return $this->_prepareValue($this->_findValue($submitValues), $assoc);

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
      * @access    public
      * @return    void
      */
-    public function __construct($elementName=null, $value=null, $attributes=null)
+    public function __construct($elementName = null, $value = null, $attributes = null)
     {
         parent::__construct($elementName, null, $attributes);
         $this->setValue($value);

main/inc/lib/pear/HTML/QuickForm/textarea.php

Doc Comments +2 added lines

@@ -54,6 +54,8 @@
      * @param     string    Input field name attribute
      * @param     mixed     Label(s) for a field
      * @param     mixed     Either a typical HTML attribute string or an associative array
+     * @param string $elementName
+     * @param string $elementLabel
      * @since     1.0
      * @access    public
      * @return    void

Indentation +7 added lines, -7 removed lines

@@ -194,13 +194,13 @@
             return $this->getFrozenHtml();
         } else {
             return $this->_getTabs() .
-                   '<textarea' . $this->_getAttrString($this->_attributes) . '>' .
-                   // because we wrap the form later we don't want the text indented
-                   // Modified by Ivan Tcholakov, 16-MAR-2010.
-                   //preg_replace("/(\r\n|\n|\r)/", '&#010;', htmlspecialchars($this->_value)) .
-                   preg_replace("/(\r\n|\n|\r)/", '&#010;', @htmlspecialchars($this->_value, ENT_COMPAT, HTML_Common::charset())) .
-                   //
-                   '</textarea>';
+                    '<textarea' . $this->_getAttrString($this->_attributes) . '>' .
+                    // because we wrap the form later we don't want the text indented
+                    // Modified by Ivan Tcholakov, 16-MAR-2010.
+                    //preg_replace("/(\r\n|\n|\r)/", '&#010;', htmlspecialchars($this->_value)) .
+                    preg_replace("/(\r\n|\n|\r)/", '&#010;', @htmlspecialchars($this->_value, ENT_COMPAT, HTML_Common::charset())) .
+                    //
+                    '</textarea>';
         }
     }
 

Spacing +6 added lines, -6 removed lines

@@ -58,7 +58,7 @@ 
      * @access    public
      * @return    void
      */
-    public function __construct($elementName=null, $elementLabel=null, $attributes=null)
+    public function __construct($elementName = null, $elementLabel = null, $attributes = null)
     {
         $attributes['class'] = isset($attributes['class']) ? $attributes['class'] : 'form-control';
         $columnsSize = isset($attributes['cols-size']) ? $attributes['cols-size'] : null;
@@ -193,12 +193,12 @@ 
         if ($this->_flagFrozen) {
             return $this->getFrozenHtml();
         } else {
-            return $this->_getTabs() .
-                   '<textarea' . $this->_getAttrString($this->_attributes) . '>' .
+            return $this->_getTabs().
+                   '<textarea'.$this->_getAttrString($this->_attributes).'>'.
                    // because we wrap the form later we don't want the text indented
                    // Modified by Ivan Tcholakov, 16-MAR-2010.
                    //preg_replace("/(\r\n|\n|\r)/", '&#010;', htmlspecialchars($this->_value)) .
-                   preg_replace("/(\r\n|\n|\r)/", '&#010;', @htmlspecialchars($this->_value, ENT_COMPAT, HTML_Common::charset())) .
+                   preg_replace("/(\r\n|\n|\r)/", '&#010;', @htmlspecialchars($this->_value, ENT_COMPAT, HTML_Common::charset())).
                    //
                    '</textarea>';
         }
@@ -218,11 +218,11 @@ 
         $value = @htmlspecialchars($this->getValue(), ENT_COMPAT, HTML_Common::charset());
         //
         if ($this->getAttribute('wrap') == 'off') {
-            $html = $this->_getTabs() . '<pre>' . $value."</pre>\n";
+            $html = $this->_getTabs().'<pre>'.$value."</pre>\n";
         } else {
             $html = nl2br($value)."\n";
         }
-        return $html . $this->_getPersistantData();
+        return $html.$this->_getPersistantData();
     }
     
     /**

main/inc/lib/pear/HTML/Table.php

Doc Comments +2 added lines, -2 removed lines

@@ -665,7 +665,7 @@ 
      * Adds a table row and returns the row identifier
      * @param    array     $contents     (optional) Must be a indexed array of
      *                                   valid cell contents
-     * @param    mixed     $attributes   (optional) Associative array or string
+     * @param    string     $attributes   (optional) Associative array or string
      *                                   of table row attributes. This can also
      *                                   be an array of attributes, in which
      *                                   case the attributes will be repeated
@@ -719,7 +719,7 @@ 
     /**
      * Updates the row attributes for an existing row
      * @param    int      $row          Row index
-     * @param    mixed    $attributes   Associative array or string of table row
+     * @param    string    $attributes   Associative array or string of table row
      *                                  attributes
      * @param    bool     $inTR         false if attributes are to be applied in
      *                                  TD tags; true if attributes are to be

Spacing +26 added lines, -26 removed lines

@@ -164,8 +164,8 @@ 
      */
     function HTML_Table($attributes = null, $tabOffset = 0, $useTGroups = false)
     {
-        parent::__construct($attributes, (int)$tabOffset);
-        $this->_useTGroups = (boolean)$useTGroups;
+        parent::__construct($attributes, (int) $tabOffset);
+        $this->_useTGroups = (boolean) $useTGroups;
         $this->addBody();
         if ($this->_useTGroups) {
             $this->_thead = new HTML_Table_Storage($tabOffset, $this->_useTGroups);
@@ -272,12 +272,12 @@ 
     function _adjustTbodyCount($body, $method)
     {
         if ($this->_autoGrow) {
-            while ($this->_tbodyCount <= (int)$body) {
+            while ($this->_tbodyCount <= (int) $body) {
                 $this->addBody();
             }
         } else {
-            return PEAR::raiseError('Invalid body reference[' .
-                $body . '] in HTML_Table::' . $method);
+            return PEAR::raiseError('Invalid body reference['.
+                $body.'] in HTML_Table::'.$method);
         }
     }
 
@@ -939,26 +939,26 @@ 
             $tBodyMaxColCount = max($tBodyColCounts);
         }
         if ($this->_comment) {
-            $strHtml .= $tabs . "<!-- $this->_comment -->" . $lnEnd;
+            $strHtml .= $tabs."<!-- $this->_comment -->".$lnEnd;
         }
         if ($this->getRowCount() > 0 && $tBodyMaxColCount > 0) {
             $strHtml .=
-                $tabs . '<table' . $this->_getAttrString($this->_attributes) . '>' . $lnEnd;
+                $tabs.'<table'.$this->_getAttrString($this->_attributes).'>'.$lnEnd;
             if (!empty($this->_caption)) {
                 $attr = $this->_caption['attr'];
                 $contents = $this->_caption['contents'];
-                $strHtml .= $tabs . $tab . '<caption' . $this->_getAttrString($attr) . '>';
+                $strHtml .= $tabs.$tab.'<caption'.$this->_getAttrString($attr).'>';
                 if (is_array($contents)) {
                     $contents = implode(', ', $contents);
                 }
                 $strHtml .= $contents;
-                $strHtml .= '</caption>' . $lnEnd;
+                $strHtml .= '</caption>'.$lnEnd;
             }
             if (!empty($this->_colgroup)) {
                 foreach ($this->_colgroup as $g => $col) {
                     $attr = $this->_colgroup[$g]['attr'];
                     $contents = $this->_colgroup[$g]['contents'];
-                    $strHtml .= $tabs . $tab . '<colgroup' . $this->_getAttrString($attr) . '>';
+                    $strHtml .= $tabs.$tab.'<colgroup'.$this->_getAttrString($attr).'>';
                     if (!empty($contents)) {
                         $strHtml .= $lnEnd;
                         if (!is_array($contents)) {
@@ -966,11 +966,11 @@ 
                         }
                         foreach ($contents as $a => $colAttr) {
                             $attr = $this->_parseAttributes($colAttr);
-                            $strHtml .= $tabs . $tab . $tab . '<col' . $this->_getAttrString($attr) . ' />' . $lnEnd;
+                            $strHtml .= $tabs.$tab.$tab.'<col'.$this->_getAttrString($attr).' />'.$lnEnd;
                         }
-                        $strHtml .= $tabs . $tab;
+                        $strHtml .= $tabs.$tab;
                     }
-                    $strHtml .= '</colgroup>' . $lnEnd;
+                    $strHtml .= '</colgroup>'.$lnEnd;
                 }
             }
             if ($this->_useTGroups) {
@@ -986,31 +986,31 @@ 
                 if ($this->_thead !== null) {
                     $this->_thead->setColCount($maxColCount);
                     if ($this->_thead->getRowCount() > 0) {
-                        $strHtml .= $tabs . $tab . '<thead' .
-                                    $this->_getAttrString($this->_thead->_attributes) .
-                                    '>' . $lnEnd;
+                        $strHtml .= $tabs.$tab.'<thead'.
+                                    $this->_getAttrString($this->_thead->_attributes).
+                                    '>'.$lnEnd;
                         $strHtml .= $this->_thead->toHtml($tabs, $tab);
-                        $strHtml .= $tabs . $tab . '</thead>' . $lnEnd;
+                        $strHtml .= $tabs.$tab.'</thead>'.$lnEnd;
                     }
                 }
                 if ($this->_tfoot !== null) {
                     $this->_tfoot->setColCount($maxColCount);
                     if ($this->_tfoot->getRowCount() > 0) {
-                        $strHtml .= $tabs . $tab . '<tfoot' .
-                                    $this->_getAttrString($this->_tfoot->_attributes) .
-                                    '>' . $lnEnd;
+                        $strHtml .= $tabs.$tab.'<tfoot'.
+                                    $this->_getAttrString($this->_tfoot->_attributes).
+                                    '>'.$lnEnd;
                         $strHtml .= $this->_tfoot->toHtml($tabs, $tab);
-                        $strHtml .= $tabs . $tab . '</tfoot>' . $lnEnd;
+                        $strHtml .= $tabs.$tab.'</tfoot>'.$lnEnd;
                     }
                 }
                 for ($i = 0; $i < $this->_tbodyCount; $i++) {
                     $this->_tbodies[$i]->setColCount($maxColCount);
                     if ($this->_tbodies[$i]->getRowCount() > 0) {
-                        $strHtml .= $tabs . $tab . '<tbody' .
-                                    $this->_getAttrString($this->_tbodies[$i]->_attributes) .
-                                    '>' . $lnEnd;
+                        $strHtml .= $tabs.$tab.'<tbody'.
+                                    $this->_getAttrString($this->_tbodies[$i]->_attributes).
+                                    '>'.$lnEnd;
                         $strHtml .= $this->_tbodies[$i]->toHtml($tabs, $tab);
-                        $strHtml .= $tabs . $tab . '</tbody>' . $lnEnd;
+                        $strHtml .= $tabs.$tab.'</tbody>'.$lnEnd;
                     }
                 }
             } else {
@@ -1019,7 +1019,7 @@ 
                     $strHtml .= $this->_tbodies[$i]->toHtml($tabs, $tab);
                 }
             }
-            $strHtml .= $tabs . '</table>' . $lnEnd;
+            $strHtml .= $tabs.'</table>'.$lnEnd;
         }
         return $strHtml;
     }

Indentation +2 added lines, -3 removed lines

@@ -48,7 +48,6 @@ 
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
  * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
  * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
  *
  * @category   HTML
  * @package    HTML_Table
@@ -256,7 +255,7 @@ 
 
         $body = $this->_tbodyCount++;
         $this->_tbodies[$body] = new HTML_Table_Storage($this->_tabOffset,
-                                                         $this->_useTGroups);
+                                                            $this->_useTGroups);
         $this->_tbodies[$body]->setAutoFill($this->_autoFill);
         $this->_tbodies[$body]->setAttributes($attributes);
         return $body;
@@ -308,7 +307,7 @@ 
         if (isset($colgroup)) {
             $attributes = $this->_parseAttributes($attributes);
             $this->_colgroup[] = array('attr' => $attributes,
-                                       'contents' => $colgroup);
+                                        'contents' => $colgroup);
         } else {
             $this->_colgroup = array();
         }

main/inc/lib/pear/HTML/Table/Storage.php

Doc Comments +2 added lines, -3 removed lines

@@ -165,7 +165,7 @@ 
     /**
      * Returns the autoFill value
      * @access   public
-     * @return   mixed
+     * @return   string
      */
     function getAutoFill()
     {
@@ -174,7 +174,6 @@ 
 
     /**
      * Sets the autoGrow value
-     * @param    bool   $fill
      * @access   public
      */
     function setAutoGrow($grow)
@@ -185,7 +184,7 @@ 
     /**
      * Returns the autoGrow value
      * @access   public
-     * @return   mixed
+     * @return   boolean
      */
     function getAutoGrow()
     {

Spacing +20 added lines, -20 removed lines

@@ -128,8 +128,8 @@ 
      */
     function HTML_Table_Storage($tabOffset = 0, $useTGroups = false)
     {
-        parent::__construct(null, (int)$tabOffset);
-        $this->_useTGroups = (boolean)$useTGroups;
+        parent::__construct(null, (int) $tabOffset);
+        $this->_useTGroups = (boolean) $useTGroups;
     }
 
     /**
@@ -288,7 +288,7 @@ 
      */
     function setCellAttributes($row, $col, $attributes)
     {
-        if (   isset($this->_structure[$row][$col])
+        if (isset($this->_structure[$row][$col])
             && $this->_structure[$row][$col] == '__SPANNED__'
         ) {
             return;
@@ -314,7 +314,7 @@ 
      */
     function updateCellAttributes($row, $col, $attributes)
     {
-        if (   isset($this->_structure[$row][$col])
+        if (isset($this->_structure[$row][$col])
             && $this->_structure[$row][$col] == '__SPANNED__'
         ) {
             return;
@@ -337,13 +337,13 @@ 
      */
     function getCellAttributes($row, $col)
     {
-        if (   isset($this->_structure[$row][$col])
+        if (isset($this->_structure[$row][$col])
             && $this->_structure[$row][$col] != '__SPANNED__'
         ) {
             return $this->_structure[$row][$col]['attr'];
         } elseif (!isset($this->_structure[$row][$col])) {
-            return PEAR::raiseError('Invalid table cell reference[' .
-                $row . '][' . $col . '] in HTML_Table::getCellAttributes');
+            return PEAR::raiseError('Invalid table cell reference['.
+                $row.']['.$col.'] in HTML_Table::getCellAttributes');
         }
         return;
     }
@@ -405,7 +405,7 @@ 
      */
     function _setSingleCellContents($row, $col, $contents, $type = 'TD')
     {
-        if (   isset($this->_structure[$row][$col])
+        if (isset($this->_structure[$row][$col])
             && $this->_structure[$row][$col] == '__SPANNED__'
         ) {
             return;
@@ -427,14 +427,14 @@ 
      */
     function getCellContents($row, $col)
     {
-        if (   isset($this->_structure[$row][$col])
+        if (isset($this->_structure[$row][$col])
             && $this->_structure[$row][$col] == '__SPANNED__'
         ) {
             return;
         }
         if (!isset($this->_structure[$row][$col])) {
-            return PEAR::raiseError('Invalid table cell reference[' .
-                $row . '][' . $col . '] in HTML_Table::getCellContents');
+            return PEAR::raiseError('Invalid table cell reference['.
+                $row.']['.$col.'] in HTML_Table::getCellContents');
         }
         return $this->_structure[$row][$col]['contents'];
     }
@@ -477,7 +477,7 @@ 
         $inTR = false)
     {
         if (isset($contents) && !is_array($contents)) {
-            return PEAR::raiseError('First parameter to HTML_Table::addRow ' .
+            return PEAR::raiseError('First parameter to HTML_Table::addRow '.
                 'must be an array');
         }
         if (is_null($contents)) {
@@ -620,7 +620,7 @@ 
     function addCol($contents = null, $attributes = null, $type = 'td')
     {
         if (isset($contents) && !is_array($contents)) {
-            return PEAR::raiseError('First parameter to HTML_Table::addCol ' .
+            return PEAR::raiseError('First parameter to HTML_Table::addCol '.
                 'must be an array');
         }
         if (is_null($contents)) {
@@ -727,13 +727,13 @@ 
             $extraTab = '';
         }
         if ($this->_cols > 0) {
-            for ($i = 0 ; $i < $this->_rows ; $i++) {
+            for ($i = 0; $i < $this->_rows; $i++) {
                 $attr = '';
                 if (isset($this->_structure[$i]['attr'])) {
                     $attr = $this->_getAttrString($this->_structure[$i]['attr']);
                 }
-                $strHtml .= $tabs .$tab . $extraTab . '<tr'.$attr.'>' . $lnEnd;
-                for ($j = 0 ; $j < $this->_cols ; $j++) {
+                $strHtml .= $tabs.$tab.$extraTab.'<tr'.$attr.'>'.$lnEnd;
+                for ($j = 0; $j < $this->_cols; $j++) {
                     $attr     = '';
                     $contents = '';
                     $type     = 'td';
@@ -753,7 +753,7 @@ 
                     if (is_object($contents)) {
                         // changes indent and line end settings on nested tables
                         if (is_subclass_of($contents, 'html_common')) {
-                            $contents->setTab($tab . $extraTab);
+                            $contents->setTab($tab.$extraTab);
                             $contents->setTabOffset($this->_tabOffset + 3);
                             $contents->_nestLevel = $this->_nestLevel + 1;
                             $contents->setLineEnd($this->_getLineEnd());
@@ -768,7 +768,7 @@ 
                         $contents = implode(', ', $contents);
                     }
 
-                    $typeContent = $tabs . $tab . $tab . $extraTab . "<$type" . $this->_getAttrString($attr) . '>';
+                    $typeContent = $tabs.$tab.$tab.$extraTab."<$type".$this->_getAttrString($attr).'>';
 
                     if (empty($contents)) {
                         if (isset($this->_autoFill) && $this->_autoFill) {
@@ -777,14 +777,14 @@ 
                     } else {
                         $typeContent .= $contents;
                     }
-                    $typeContent .= "</$type>" . $lnEnd;
+                    $typeContent .= "</$type>".$lnEnd;
 
                     if (!empty($contents)) {
                         $strHtml .= $typeContent;
                     }
 
                 }
-                $strHtml .= $tabs . $tab . $extraTab . '</tr>' . $lnEnd;
+                $strHtml .= $tabs.$tab.$extraTab.'</tr>'.$lnEnd;
             }
         }
         return $strHtml;

main/inc/lib/pear/Image/Text.php

Doc Comments +4 added lines, -4 removed lines

@@ -328,8 +328,8 @@ 
      * the Image_Text object after changing options. For possible options, please
      * take a look at the class options array!
      *
-     * @param array|string $option A single option name or the options array.
-     * @param mixed        $value  Option value if $option is string.
+     * @param string $option A single option name or the options array.
+     * @param string        $value  Option value if $option is string.
      *
      * @return void
      * @see    Image_Text::Image_Text()
@@ -982,7 +982,7 @@ 
      * @param bool $save Save or not the image on printout.
      * @param bool $free Free the image on exit.
      *
-     * @return  bool         True on success
+     * @return  boolean|null         True on success
      * @see Image_Text::save()
      * @throws Image_Text_Exception
      */
@@ -1133,7 +1133,7 @@ 
      * @param string $scolor string of colorcode.
      *
      * @see Image_Text::IMAGE_TEXT_REGEX_HTMLCOLOR
-     * @return bool|array false if string can't be converted to array
+     * @return resource false if string can't be converted to array
      */
     public static function convertString2RGB($scolor)
     {

Switch Indentation +94 added lines, -94 removed lines

@@ -346,19 +346,19 @@ 
         }
         foreach ($option as $opt => $val) {
             switch ($opt) {
-            case 'color':
-                $this->setColors($val);
-                break;
-            case 'text':
-                if (is_array($val)) {
-                    $this->_text = implode('\n', $val);
-                } else {
-                    $this->_text = $val;
-                }
-                break;
-            default:
-                $this->_options[$opt] = $val;
-                break;
+                case 'color':
+                    $this->setColors($val);
+                    break;
+                case 'text':
+                    if (is_array($val)) {
+                        $this->_text = implode('\n', $val);
+                    } else {
+                        $this->_text = $val;
+                    }
+                    break;
+                default:
+                    $this->_options[$opt] = $val;
+                    break;
             }
             if (isset($reInits[$opt])) {
                 $this->_init = false;
@@ -511,45 +511,45 @@ 
         $image_canvas = false;
         switch (true) {
 
-        case (empty($this->_options['canvas'])):
-            // Create new image from width && height of the clipping
-            $this->_img = imagecreatetruecolor(
-                $this->_options['width'], $this->_options['height']
-            );
-            if (!$this->_img) {
-                throw new Image_Text_Exception('Could not create image canvas.');
-            }
-            break;
+            case (empty($this->_options['canvas'])):
+                // Create new image from width && height of the clipping
+                $this->_img = imagecreatetruecolor(
+                    $this->_options['width'], $this->_options['height']
+                );
+                if (!$this->_img) {
+                    throw new Image_Text_Exception('Could not create image canvas.');
+                }
+                break;
 
-        case (is_resource($this->_options['canvas']) &&
-            get_resource_type($this->_options['canvas']) == 'gd'):
-            // The canvas is an image resource
-            $image_canvas = true;
-            $this->_img = $this->_options['canvas'];
-            break;
+            case (is_resource($this->_options['canvas']) &&
+                get_resource_type($this->_options['canvas']) == 'gd'):
+                // The canvas is an image resource
+                $image_canvas = true;
+                $this->_img = $this->_options['canvas'];
+                break;
 
-        case (is_array($this->_options['canvas']) &&
-            isset($this->_options['canvas']['width']) &&
-            isset($this->_options['canvas']['height'])):
+            case (is_array($this->_options['canvas']) &&
+                isset($this->_options['canvas']['width']) &&
+                isset($this->_options['canvas']['height'])):
 
-            // Canvas must be a width and height measure
-            $this->_img = imagecreatetruecolor(
-                $this->_options['canvas']['width'],
-                $this->_options['canvas']['height']
-            );
-            break;
+                // Canvas must be a width and height measure
+                $this->_img = imagecreatetruecolor(
+                    $this->_options['canvas']['width'],
+                    $this->_options['canvas']['height']
+                );
+                break;
 
-        case (is_array($this->_options['canvas']) &&
-            isset($this->_options['canvas']['size']) &&
-            ($this->_options['canvas']['size'] = 'auto')):
+            case (is_array($this->_options['canvas']) &&
+                isset($this->_options['canvas']['size']) &&
+                ($this->_options['canvas']['size'] = 'auto')):
 
-        case (is_string($this->_options['canvas']) &&
-            ($this->_options['canvas'] = 'auto')):
-            $this->_mode = 'auto';
-            break;
+            case (is_string($this->_options['canvas']) &&
+                ($this->_options['canvas'] = 'auto')):
+                $this->_mode = 'auto';
+                break;
 
-        default:
-            throw new Image_Text_Exception('Could not create image canvas.');
+            default:
+                throw new Image_Text_Exception('Could not create image canvas.');
         }
 
         if ($this->_img) {
@@ -889,19 +889,19 @@ 
         $cosR = cos($radians);
 
         switch ($this->_options['valign']) {
-        case self::IMAGE_TEXT_ALIGN_TOP:
-            $valign_space = 0;
-            break;
-        case self::IMAGE_TEXT_ALIGN_MIDDLE:
-            $valign_space = ($this->_options['height']
-                    - $this->_realTextSize['height']) / 2;
-            break;
-        case self::IMAGE_TEXT_ALIGN_BOTTOM:
-            $valign_space = $this->_options['height']
-                - $this->_realTextSize['height'];
-            break;
-        default:
-            $valign_space = 0;
+            case self::IMAGE_TEXT_ALIGN_TOP:
+                $valign_space = 0;
+                break;
+            case self::IMAGE_TEXT_ALIGN_MIDDLE:
+                $valign_space = ($this->_options['height']
+                        - $this->_realTextSize['height']) / 2;
+                break;
+            case self::IMAGE_TEXT_ALIGN_BOTTOM:
+                $valign_space = $this->_options['height']
+                    - $this->_realTextSize['height'];
+                break;
+            default:
+                $valign_space = 0;
         }
 
         $space = (1 + $line_spacing) * $size;
@@ -932,18 +932,18 @@ 
             // Calc the position using the block width, the current line width and
             // obviously the angle. That gives us the offset to slide the line.
             switch ($align) {
-            case self::IMAGE_TEXT_ALIGN_LEFT:
-                $hyp = 0;
-                break;
-            case self::IMAGE_TEXT_ALIGN_RIGHT:
-                $hyp = $block_width - $line_width - $left_margin;
-                break;
-            case self::IMAGE_TEXT_ALIGN_CENTER:
-                $hyp = ($block_width - $line_width) / 2 - $left_margin;
-                break;
-            default:
-                $hyp = 0;
-                break;
+                case self::IMAGE_TEXT_ALIGN_LEFT:
+                    $hyp = 0;
+                    break;
+                case self::IMAGE_TEXT_ALIGN_RIGHT:
+                    $hyp = $block_width - $line_width - $left_margin;
+                    break;
+                case self::IMAGE_TEXT_ALIGN_CENTER:
+                    $hyp = ($block_width - $line_width) / 2 - $left_margin;
+                    break;
+                default:
+                    $hyp = 0;
+                    break;
             }
 
             $posx = $new_posx + $cosR * $hyp;
@@ -997,17 +997,17 @@ 
             throw new Image_Text_Exception('Header already sent.');
         }
         switch ($this->_options['image_type']) {
-        case IMAGETYPE_PNG:
-            $imgout = 'imagepng';
-            break;
-        case IMAGETYPE_JPEG:
-            $imgout = 'imagejpeg';
-            break;
-        case IMAGETYPE_BMP:
-            $imgout = 'imagebmp';
-            break;
-        default:
-            throw new Image_Text_Exception('Unsupported image type.');
+            case IMAGETYPE_PNG:
+                $imgout = 'imagepng';
+                break;
+            case IMAGETYPE_JPEG:
+                $imgout = 'imagejpeg';
+                break;
+            case IMAGETYPE_BMP:
+                $imgout = 'imagebmp';
+                break;
+            default:
+                throw new Image_Text_Exception('Unsupported image type.');
         }
         if ($save) {
             $imgout($this->_img);
@@ -1048,18 +1048,18 @@ 
         }
 
         switch ($this->_options['image_type']) {
-        case IMAGETYPE_PNG:
-            $imgout = 'imagepng';
-            break;
-        case IMAGETYPE_JPEG:
-            $imgout = 'imagejpeg';
-            break;
-        case IMAGETYPE_BMP:
-            $imgout = 'imagebmp';
-            break;
-        default:
-            throw new Image_Text_Exception('Unsupported image type.');
-            break;
+            case IMAGETYPE_PNG:
+                $imgout = 'imagepng';
+                break;
+            case IMAGETYPE_JPEG:
+                $imgout = 'imagejpeg';
+                break;
+            case IMAGETYPE_BMP:
+                $imgout = 'imagebmp';
+                break;
+            default:
+                throw new Image_Text_Exception('Unsupported image type.');
+                break;
         }
 
         $res = $imgout($this->_img, $destFile);

Spacing +6 added lines, -6 removed lines

@@ -683,7 +683,7 @@ 
         $space = (1 + $this->_options['line_spacing'])
             * $this->_options['font_size'];
 
-        $max_lines = (int)$this->_options['max_lines'];
+        $max_lines = (int) $this->_options['max_lines'];
 
         if (($max_lines < 1) && !$force) {
             return false;
@@ -732,7 +732,7 @@ 
                     'color' => $c
                 );
                 $text_width = max($text_width, ($bounds[2] - $bounds[0]));
-                $text_height += (int)$space;
+                $text_height += (int) $space;
                 if (($text_height > $block_height) && !$force) {
                     return false;
                 }
@@ -749,7 +749,7 @@ 
                 $text_line_next = $token;
                 $beginning_of_line = false;
             } else {
-                $text_line_next = $text_line . ' ' . $token;
+                $text_line_next = $text_line.' '.$token;
             }
             $bounds = imagettfbbox($size, 0, $font, $text_line_next);
             $prev_width = isset($prev_width) ? $width : 0;
@@ -776,7 +776,7 @@ 
                     'color' => $c
                 );
                 $text_width = max($text_width, ($bounds[2] - $bounds[0]));
-                $text_height += (int)$space;
+                $text_height += (int) $space;
                 if (($text_height > $block_height) && !$force) {
                     return false;
                 }
@@ -990,7 +990,7 @@ 
     {
         if (!headers_sent()) {
             header(
-                "Content-type: " .
+                "Content-type: ".
                 image_type_to_mime_type($this->_options['image_type'])
             );
         } else {
@@ -1121,7 +1121,7 @@ 
                 $y += $vC[1];
             }
         }
-        return array('x' => (int)round($x, 0), 'y' => (int)round($y, 0));
+        return array('x' => (int) round($x, 0), 'y' => (int) round($y, 0));
     }
 
     /**

main/inc/lib/pear/OLE/OLE.php

Doc Comments +1 added lines, -1 removed lines

@@ -286,7 +286,7 @@
     *
     * @access private
     * @param integer $blockId the block id of the first block
-    * @return mixed true on success, PEAR_Error on failure
+    * @return boolean true on success, PEAR_Error on failure
     */
     function _readPpsWks($blockId)
     {

Indentation +104 added lines, -104 removed lines

@@ -49,74 +49,74 @@ 
 {
 
     /**
-    * The file handle for reading an OLE container
-    * @var resource
-    */
+     * The file handle for reading an OLE container
+     * @var resource
+     */
     var $_file_handle;
 
     /**
-    * Array of PPS's found on the OLE container
-    * @var array
-    */
+     * Array of PPS's found on the OLE container
+     * @var array
+     */
     var $_list;
 
     /**
-    * Root directory of OLE container
-    * @var OLE_PPS_Root
-    */
+     * Root directory of OLE container
+     * @var OLE_PPS_Root
+     */
     var $root;
 
     /**
-    * Big Block Allocation Table
-    * @var array  (blockId => nextBlockId)
-    */
+     * Big Block Allocation Table
+     * @var array  (blockId => nextBlockId)
+     */
     var $bbat;
 
     /**
-    * Short Block Allocation Table
-    * @var array  (blockId => nextBlockId)
-    */
+     * Short Block Allocation Table
+     * @var array  (blockId => nextBlockId)
+     */
     var $sbat;
 
     /**
-    * Size of big blocks. This is usually 512.
-    * @var  int  number of octets per block.
-    */
+     * Size of big blocks. This is usually 512.
+     * @var  int  number of octets per block.
+     */
     var $bigBlockSize;
 
     /**
-    * Size of small blocks. This is usually 64.
-    * @var  int  number of octets per block
-    */
+     * Size of small blocks. This is usually 64.
+     * @var  int  number of octets per block
+     */
     var $smallBlockSize;
 
     /**
-    * Creates a new OLE object
-    * @access public
-    */
+     * Creates a new OLE object
+     * @access public
+     */
     function OLE()
     {
         $this->_list = array();
     }
 
     /**
-    * Destructor (using PEAR)
-    * Just closes the file handle on the OLE file.
-    *
-    * @access private
-    */
+     * Destructor (using PEAR)
+     * Just closes the file handle on the OLE file.
+     *
+     * @access private
+     */
     function _OLE()
     {
         fclose($this->_file_handle);
     }
 
     /**
-    * Reads an OLE container from the contents of the file given.
-    *
-    * @access public
-    * @param string $file
-    * @return mixed true on success, PEAR_Error on failure
-    */
+     * Reads an OLE container from the contents of the file given.
+     *
+     * @access public
+     * @param string $file
+     * @return mixed true on success, PEAR_Error on failure
+     */
     function read($file)
     {
         $fh = @fopen($file, "r");
@@ -281,13 +281,13 @@ 
     }
 
     /**
-    * Gets information about all PPS's on the OLE container from the PPS WK's
-    * creates an OLE_PPS object for each one.
-    *
-    * @access private
-    * @param integer $blockId the block id of the first block
-    * @return mixed true on success, PEAR_Error on failure
-    */
+     * Gets information about all PPS's on the OLE container from the PPS WK's
+     * creates an OLE_PPS object for each one.
+     *
+     * @access private
+     * @param integer $blockId the block id of the first block
+     * @return mixed true on success, PEAR_Error on failure
+     */
     function _readPpsWks($blockId)
     {
         $fh = $this->getStream($blockId);
@@ -307,7 +307,7 @@ 
                 break;
             case OLE_PPS_TYPE_DIR:
                 $pps = new OLE_PPS(null, null, null, null, null,
-                                   null, null, null, null, array());
+                                    null, null, null, null, array());
                 break;
             case OLE_PPS_TYPE_FILE:
                 require_once 'OLE/PPS/File.php';
@@ -360,13 +360,13 @@ 
     }
 
     /**
-    * It checks whether the PPS tree is complete (all PPS's read)
-    * starting with the given PPS (not necessarily root)
-    *
-    * @access private
-    * @param integer $index The index of the PPS from which we are checking
-    * @return boolean Whether the PPS tree for the given PPS is complete
-    */
+     * It checks whether the PPS tree is complete (all PPS's read)
+     * starting with the given PPS (not necessarily root)
+     *
+     * @access private
+     * @param integer $index The index of the PPS from which we are checking
+     * @return boolean Whether the PPS tree for the given PPS is complete
+     */
     function _ppsTreeComplete($index)
     {
         return isset($this->_list[$index]) &&
@@ -380,12 +380,12 @@ 
     }
 
     /**
-    * Checks whether a PPS is a File PPS or not.
-    * If there is no PPS for the index given, it will return false.
-    * @param integer $index The index for the PPS
-    * @return bool true if it's a File PPS, false otherwise
-    * @access public
-    */
+     * Checks whether a PPS is a File PPS or not.
+     * If there is no PPS for the index given, it will return false.
+     * @param integer $index The index for the PPS
+     * @return bool true if it's a File PPS, false otherwise
+     * @access public
+     */
     function isFile($index)
     {
         if (isset($this->_list[$index])) {
@@ -395,12 +395,12 @@ 
     }
 
     /**
-    * Checks whether a PPS is a Root PPS or not.
-    * If there is no PPS for the index given, it will return false.
-    * @param integer $index The index for the PPS.
-    * @return bool true if it's a Root PPS, false otherwise
-    * @access public
-    */
+     * Checks whether a PPS is a Root PPS or not.
+     * If there is no PPS for the index given, it will return false.
+     * @param integer $index The index for the PPS.
+     * @return bool true if it's a Root PPS, false otherwise
+     * @access public
+     */
     function isRoot($index)
     {
         if (isset($this->_list[$index])) {
@@ -410,26 +410,26 @@ 
     }
 
     /**
-    * Gives the total number of PPS's found in the OLE container.
-    * @return integer The total number of PPS's found in the OLE container
-    * @access public
-    */
+     * Gives the total number of PPS's found in the OLE container.
+     * @return integer The total number of PPS's found in the OLE container
+     * @access public
+     */
     function ppsTotal()
     {
         return count($this->_list);
     }
 
     /**
-    * Gets data from a PPS
-    * If there is no PPS for the index given, it will return an empty string.
-    * @param integer $index    The index for the PPS
-    * @param integer $position The position from which to start reading
-    *                          (relative to the PPS)
-    * @param integer $length   The amount of bytes to read (at most)
-    * @return string The binary string containing the data requested
-    * @access public
-    * @see OLE_PPS_File::getStream()
-    */
+     * Gets data from a PPS
+     * If there is no PPS for the index given, it will return an empty string.
+     * @param integer $index    The index for the PPS
+     * @param integer $position The position from which to start reading
+     *                          (relative to the PPS)
+     * @param integer $length   The amount of bytes to read (at most)
+     * @return string The binary string containing the data requested
+     * @access public
+     * @see OLE_PPS_File::getStream()
+     */
     function getData($index, $position, $length)
     {
         // if position is not valid return empty string
@@ -446,12 +446,12 @@ 
     }
 
     /**
-    * Gets the data length from a PPS
-    * If there is no PPS for the index given, it will return 0.
-    * @param integer $index The index for the PPS
-    * @return integer The amount of bytes in data the PPS has
-    * @access public
-    */
+     * Gets the data length from a PPS
+     * If there is no PPS for the index given, it will return 0.
+     * @param integer $index The index for the PPS
+     * @return integer The amount of bytes in data the PPS has
+     * @access public
+     */
     function getDataLength($index)
     {
         if (isset($this->_list[$index])) {
@@ -461,13 +461,13 @@ 
     }
 
     /**
-    * Utility function to transform ASCII text to Unicode
-    *
-    * @access public
-    * @static
-    * @param string $ascii The ASCII string to transform
-    * @return string The string in Unicode
-    */
+     * Utility function to transform ASCII text to Unicode
+     *
+     * @access public
+     * @static
+     * @param string $ascii The ASCII string to transform
+     * @return string The string in Unicode
+     */
     function Asc2Ucs($ascii)
     {
         $rawname = '';
@@ -478,14 +478,14 @@ 
     }
 
     /**
-    * Utility function
-    * Returns a string for the OLE container with the date given
-    *
-    * @access public
-    * @static
-    * @param integer $date A timestamp
-    * @return string The string for the OLE container
-    */
+     * Utility function
+     * Returns a string for the OLE container with the date given
+     *
+     * @access public
+     * @static
+     * @param integer $date A timestamp
+     * @return string The string for the OLE container
+     */
     function LocalDate2OLE($date = null)
     {
         if (!isset($date)) {
@@ -500,7 +500,7 @@ 
         // calculate seconds
         $big_date = $days * 24 * 3600 +
             gmmktime(date("H",$date),date("i",$date),date("s",$date),
-                     date("m",$date),date("d",$date),date("Y",$date));
+                        date("m",$date),date("d",$date),date("Y",$date));
         // multiply just to make MS happy
         $big_date *= 10000000;
 
@@ -525,12 +525,12 @@ 
     }
 
     /**
-    * Returns a timestamp from an OLE container's date
-    * @param integer $string A binary string with the encoded date
-    * @return string The timestamp corresponding to the string
-    * @access public
-    * @static
-    */
+     * Returns a timestamp from an OLE container's date
+     * @param integer $string A binary string with the encoded date
+     * @return string The timestamp corresponding to the string
+     * @access public
+     * @static
+     */
     function OLE2LocalDate($string)
     {
         if (strlen($string) != 8) {

Switch Indentation +15 added lines, -15 removed lines

@@ -300,21 +300,21 @@
             $name = str_replace("\x00", "", $nameUtf16);
             $type = $this->_readInt1($fh);
             switch ($type) {
-            case OLE_PPS_TYPE_ROOT:
-                require_once 'OLE/PPS/Root.php';
-                $pps = new OLE_PPS_Root(null, null, array());
-                $this->root = $pps;
-                break;
-            case OLE_PPS_TYPE_DIR:
-                $pps = new OLE_PPS(null, null, null, null, null,
-                                   null, null, null, null, array());
-                break;
-            case OLE_PPS_TYPE_FILE:
-                require_once 'OLE/PPS/File.php';
-                $pps = new OLE_PPS_File($name);
-                break;
-            default:
-                continue;
+                case OLE_PPS_TYPE_ROOT:
+                    require_once 'OLE/PPS/Root.php';
+                    $pps = new OLE_PPS_Root(null, null, array());
+                    $this->root = $pps;
+                    break;
+                case OLE_PPS_TYPE_DIR:
+                    $pps = new OLE_PPS(null, null, null, null, null,
+                                       null, null, null, null, array());
+                    break;
+                case OLE_PPS_TYPE_FILE:
+                    require_once 'OLE/PPS/File.php';
+                    $pps = new OLE_PPS_File($name);
+                    break;
+                default:
+                    continue;
             }
             fseek($fh, 1, SEEK_CUR);
             $pps->Type    = $type;

Spacing +14 added lines, -14 removed lines

@@ -23,12 +23,12 @@ 
 /**
 * Constants for OLE package
 */
-define('OLE_PPS_TYPE_ROOT',        5);
-define('OLE_PPS_TYPE_DIR',         1);
-define('OLE_PPS_TYPE_FILE',        2);
+define('OLE_PPS_TYPE_ROOT', 5);
+define('OLE_PPS_TYPE_DIR', 1);
+define('OLE_PPS_TYPE_FILE', 2);
 define('OLE_DATA_SIZE_SMALL', 0x1000);
-define('OLE_LONG_INT_SIZE',        4);
-define('OLE_PPS_SIZE',          0x80);
+define('OLE_LONG_INT_SIZE', 4);
+define('OLE_PPS_SIZE', 0x80);
 
 /**
 * Array for storing OLE instances that are accessed from
@@ -183,7 +183,7 @@ 
         for ($i = 0; $i < $bbatBlockCount; $i++) {
             $pos = $this->_getBlockOffset($mbatBlocks[$i]);
             fseek($fh, $pos);
-            for ($j = 0 ; $j < $this->bigBlockSize / 4; $j++) {
+            for ($j = 0; $j < $this->bigBlockSize / 4; $j++) {
                 $this->bbat[] = $this->_readInt4($fh);
             }
         }
@@ -234,12 +234,12 @@ 
         $GLOBALS['_OLE_INSTANCES'][] = $this;
         $instanceId = end(array_keys($GLOBALS['_OLE_INSTANCES']));
 
-        $path = 'ole-chainedblockstream://oleInstanceId=' . $instanceId;
+        $path = 'ole-chainedblockstream://oleInstanceId='.$instanceId;
         if (is_a($blockIdOrPps, 'OLE_PPS')) {
-            $path .= '&blockId=' . $blockIdOrPps->_StartBlock;
-            $path .= '&size=' . $blockIdOrPps->Size;
+            $path .= '&blockId='.$blockIdOrPps->_StartBlock;
+            $path .= '&size='.$blockIdOrPps->Size;
         } else {
-            $path .= '&blockId=' . $blockIdOrPps;
+            $path .= '&blockId='.$blockIdOrPps;
         }
         return fopen($path, 'r');
     }
@@ -472,7 +472,7 @@ 
     {
         $rawname = '';
         for ($i = 0; $i < strlen($ascii); $i++) {
-            $rawname .= $ascii{$i} . "\x00";
+            $rawname .= $ascii{$i}."\x00";
         }
         return $rawname;
     }
@@ -499,8 +499,8 @@ 
         $days = 134774;
         // calculate seconds
         $big_date = $days * 24 * 3600 +
-            gmmktime(date("H",$date),date("i",$date),date("s",$date),
-                     date("m",$date),date("d",$date),date("Y",$date));
+            gmmktime(date("H", $date), date("i", $date), date("s", $date),
+                     date("m", $date), date("d", $date), date("Y", $date));
         // multiply just to make MS happy
         $big_date *= 10000000;
 
@@ -538,7 +538,7 @@ 
         }
 
         // factor used for separating numbers into 4 bytes parts
-        $factor = pow(2,32);
+        $factor = pow(2, 32);
         $high_part = 0;
         for ($i = 0; $i < 4; $i++) {
             list(, $high_part) = unpack('C', $string{(7 - $i)});

main/inc/lib/pear/OLE/PPS/File.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     *
     * @access public
     * @param string $dir The dir to be used as temp dir
-    * @return true if given dir is valid, false otherwise
+    * @return boolean if given dir is valid, false otherwise
     */
     function setTempDir($dir)
     {

Indentation +25 added lines, -25 removed lines

@@ -30,18 +30,18 @@ 
 class OLE_PPS_File extends OLE_PPS
 {
     /**
-    * The temporary dir for storing the OLE file
-    * @var string
-    */
+     * The temporary dir for storing the OLE file
+     * @var string
+     */
     var $_tmp_dir;
 
     /**
-    * The constructor
-    *
-    * @access public
-    * @param string $name The name of the file (in Unicode)
-    * @see OLE::Asc2Ucs()
-    */
+     * The constructor
+     *
+     * @access public
+     * @param string $name The name of the file (in Unicode)
+     * @see OLE::Asc2Ucs()
+     */
     function OLE_PPS_File($name)
     {
         $this->_tmp_dir = '';
@@ -59,12 +59,12 @@ 
     }
 
     /**
-    * Sets the temp dir used for storing the OLE file
-    *
-    * @access public
-    * @param string $dir The dir to be used as temp dir
-    * @return true if given dir is valid, false otherwise
-    */
+     * Sets the temp dir used for storing the OLE file
+     *
+     * @access public
+     * @param string $dir The dir to be used as temp dir
+     * @return true if given dir is valid, false otherwise
+     */
     function setTempDir($dir)
     {
         if (is_dir($dir)) {
@@ -75,11 +75,11 @@ 
     }
 
     /**
-    * Initialization method. Has to be called right after OLE_PPS_File().
-    *
-    * @access public
-    * @return mixed true on success. PEAR_Error on failure
-    */
+     * Initialization method. Has to be called right after OLE_PPS_File().
+     *
+     * @access public
+     * @return mixed true on success. PEAR_Error on failure
+     */
     function init()
     {
         $this->_tmp_filename = tempnam($this->_tmp_dir, "OLE_PPS_File");
@@ -96,11 +96,11 @@ 
     }
 
     /**
-    * Append data to PPS
-    *
-    * @access public
-    * @param string $data The data to append
-    */
+     * Append data to PPS
+     *
+     * @access public
+     * @param string $data The data to append
+     */
     function append($data)
     {
         if ($this->_PPS_FILE) {

main/inc/lib/pear/OLE/PPS/Root.php

Doc Comments +3 added lines, -3 removed lines

@@ -65,7 +65,7 @@ 
     *
     * @access public
     * @param string $dir The dir to be used as temp dir
-    * @return true if given dir is valid, false otherwise
+    * @return boolean if given dir is valid, false otherwise
     */
     function setTempDir($dir)
     {
@@ -142,7 +142,7 @@ 
     *
     * @access private
     * @param array $raList Reference to an array of PPS's
-    * @return array The array of numbers
+    * @return double[] The array of numbers
     */
     function _calcSize(&$raList)
     {
@@ -180,7 +180,7 @@ 
     * @access private
     * @param integer $i2 The argument
     * @see save()
-    * @return integer
+    * @return double
     */
     function _adjust2($i2)
     {

Indentation +101 added lines, -101 removed lines

@@ -32,41 +32,41 @@ 
 class OLE_PPS_Root extends OLE_PPS
 {
     /**
-    * The temporary dir for storing the OLE file
-    * @var string
-    */
+     * The temporary dir for storing the OLE file
+     * @var string
+     */
     var $_tmp_dir;
 
     /**
-    * Constructor
-    *
-    * @access public
-    * @param integer $time_1st A timestamp
-    * @param integer $time_2nd A timestamp
-    */
+     * Constructor
+     *
+     * @access public
+     * @param integer $time_1st A timestamp
+     * @param integer $time_2nd A timestamp
+     */
     function OLE_PPS_Root($time_1st, $time_2nd, $raChild)
     {
         $this->_tmp_dir = '';
         $this->OLE_PPS(
-           null,
-           OLE::Asc2Ucs('Root Entry'),
-           OLE_PPS_TYPE_ROOT,
-           null,
-           null,
-           null,
-           $time_1st,
-           $time_2nd,
-           null,
-           $raChild);
+            null,
+            OLE::Asc2Ucs('Root Entry'),
+            OLE_PPS_TYPE_ROOT,
+            null,
+            null,
+            null,
+            $time_1st,
+            $time_2nd,
+            null,
+            $raChild);
     }
 
     /**
-    * Sets the temp dir used for storing the OLE file
-    *
-    * @access public
-    * @param string $dir The dir to be used as temp dir
-    * @return true if given dir is valid, false otherwise
-    */
+     * Sets the temp dir used for storing the OLE file
+     *
+     * @access public
+     * @param string $dir The dir to be used as temp dir
+     * @return true if given dir is valid, false otherwise
+     */
     function setTempDir($dir)
     {
         if (is_dir($dir)) {
@@ -77,21 +77,21 @@ 
     }
 
     /**
-    * Method for saving the whole OLE container (including files).
-    * In fact, if called with an empty argument (or '-'), it saves to a
-    * temporary file and then outputs it's contents to stdout.
-    *
-    * @param string $filename The name of the file where to save the OLE container
-    * @access public
-    * @return mixed true on success, PEAR_Error on failure
-    */
+     * Method for saving the whole OLE container (including files).
+     * In fact, if called with an empty argument (or '-'), it saves to a
+     * temporary file and then outputs it's contents to stdout.
+     *
+     * @param string $filename The name of the file where to save the OLE container
+     * @access public
+     * @return mixed true on success, PEAR_Error on failure
+     */
     function save($filename)
     {
         // Initial Setting for saving
         $this->_BIG_BLOCK_SIZE  = pow(2,
-                      ((isset($this->_BIG_BLOCK_SIZE))? $this->_adjust2($this->_BIG_BLOCK_SIZE)  : 9));
+                        ((isset($this->_BIG_BLOCK_SIZE))? $this->_adjust2($this->_BIG_BLOCK_SIZE)  : 9));
         $this->_SMALL_BLOCK_SIZE= pow(2,
-                      ((isset($this->_SMALL_BLOCK_SIZE))?  $this->_adjust2($this->_SMALL_BLOCK_SIZE): 6));
+                        ((isset($this->_SMALL_BLOCK_SIZE))?  $this->_adjust2($this->_SMALL_BLOCK_SIZE): 6));
 
         // Open temp file if we are sending output to stdout
         if (($filename == '-') || ($filename == '')) {
@@ -138,12 +138,12 @@ 
     }
 
     /**
-    * Calculate some numbers
-    *
-    * @access private
-    * @param array $raList Reference to an array of PPS's
-    * @return array The array of numbers
-    */
+     * Calculate some numbers
+     *
+     * @access private
+     * @param array $raList Reference to an array of PPS's
+     * @return array The array of numbers
+     */
     function _calcSize(&$raList)
     {
         // Calculate Basic Setting
@@ -175,13 +175,13 @@ 
     }
 
     /**
-    * Helper function for caculating a magic value for block sizes
-    *
-    * @access private
-    * @param integer $i2 The argument
-    * @see save()
-    * @return integer
-    */
+     * Helper function for caculating a magic value for block sizes
+     *
+     * @access private
+     * @param integer $i2 The argument
+     * @see save()
+     * @return integer
+     */
     function _adjust2($i2)
     {
         $iWk = log($i2)/log(2);
@@ -189,13 +189,13 @@ 
     }
 
     /**
-    * Save OLE header
-    *
-    * @access private
-    * @param integer $iSBDcnt
-    * @param integer $iBBcnt
-    * @param integer $iPPScnt
-    */
+     * Save OLE header
+     *
+     * @access private
+     * @param integer $iSBDcnt
+     * @param integer $iBBcnt
+     * @param integer $iPPScnt
+     */
     function _saveHeader($iSBDcnt, $iBBcnt, $iPPScnt)
     {
         $FILE = $this->_FILEH_;
@@ -225,32 +225,32 @@ 
 
         // Save Header
         fwrite($FILE,
-                  "\xD0\xCF\x11\xE0\xA1\xB1\x1A\xE1"
-                  . "\x00\x00\x00\x00"
-                  . "\x00\x00\x00\x00"
-                  . "\x00\x00\x00\x00"
-                  . "\x00\x00\x00\x00"
-                  . pack("v", 0x3b)
-                  . pack("v", 0x03)
-                  . pack("v", -2)
-                  . pack("v", 9)
-                  . pack("v", 6)
-                  . pack("v", 0)
-                  . "\x00\x00\x00\x00"
-                  . "\x00\x00\x00\x00"
-                  . pack("V", $iBdCnt)
-                  . pack("V", $iBBcnt+$iSBDcnt) //ROOT START
-                  . pack("V", 0)
-                  . pack("V", 0x1000)
-                  . pack("V", 0)                  //Small Block Depot
-                  . pack("V", 1)
-          );
+                    "\xD0\xCF\x11\xE0\xA1\xB1\x1A\xE1"
+                    . "\x00\x00\x00\x00"
+                    . "\x00\x00\x00\x00"
+                    . "\x00\x00\x00\x00"
+                    . "\x00\x00\x00\x00"
+                    . pack("v", 0x3b)
+                    . pack("v", 0x03)
+                    . pack("v", -2)
+                    . pack("v", 9)
+                    . pack("v", 6)
+                    . pack("v", 0)
+                    . "\x00\x00\x00\x00"
+                    . "\x00\x00\x00\x00"
+                    . pack("V", $iBdCnt)
+                    . pack("V", $iBBcnt+$iSBDcnt) //ROOT START
+                    . pack("V", 0)
+                    . pack("V", 0x1000)
+                    . pack("V", 0)                  //Small Block Depot
+                    . pack("V", 1)
+            );
         // Extra BDList Start, Count
         if ($iBdCnt < $i1stBdL) {
             fwrite($FILE,
-                      pack("V", -2).      // Extra BDList Start
-                      pack("V", 0)        // Extra BDList Count
-                  );
+                        pack("V", -2).      // Extra BDList Start
+                        pack("V", 0)        // Extra BDList Count
+                    );
         } else {
             fwrite($FILE, pack("V", $iAll+$iBdCnt) . pack("V", $iBdExL));
         }
@@ -267,12 +267,12 @@ 
     }
 
     /**
-    * Saving big data (PPS's with data bigger than OLE_DATA_SIZE_SMALL)
-    *
-    * @access private
-    * @param integer $iStBlk
-    * @param array &$raList Reference to array of PPS's
-    */
+     * Saving big data (PPS's with data bigger than OLE_DATA_SIZE_SMALL)
+     *
+     * @access private
+     * @param integer $iStBlk
+     * @param array &$raList Reference to array of PPS's
+     */
     function _saveBigData($iStBlk, &$raList)
     {
         $FILE = $this->_FILEH_;
@@ -318,11 +318,11 @@ 
     }
 
     /**
-    * get small data (PPS's with data smaller than OLE_DATA_SIZE_SMALL)
-    *
-    * @access private
-    * @param array &$raList Reference to array of PPS's
-    */
+     * get small data (PPS's with data smaller than OLE_DATA_SIZE_SMALL)
+     *
+     * @access private
+     * @param array &$raList Reference to array of PPS's
+     */
     function _makeSmallData(&$raList)
     {
         $sRes = '';
@@ -374,11 +374,11 @@ 
     }
 
     /**
-    * Saves all the PPS's WKs
-    *
-    * @access private
-    * @param array $raList Reference to an array with all PPS's
-    */
+     * Saves all the PPS's WKs
+     *
+     * @access private
+     * @param array $raList Reference to an array with all PPS's
+     */
     function _savePps(&$raList)
     {
         // Save each PPS WK
@@ -396,13 +396,13 @@ 
     }
 
     /**
-    * Saving Big Block Depot
-    *
-    * @access private
-    * @param integer $iSbdSize
-    * @param integer $iBsize
-    * @param integer $iPpsCnt
-    */
+     * Saving Big Block Depot
+     *
+     * @access private
+     * @param integer $iSbdSize
+     * @param integer $iBsize
+     * @param integer $iPpsCnt
+     */
     function _saveBbd($iSbdSize, $iBsize, $iPpsCnt)
     {
         $FILE = $this->_FILEH_;

Spacing +43 added lines, -43 removed lines

@@ -89,14 +89,14 @@ 
     {
         // Initial Setting for saving
         $this->_BIG_BLOCK_SIZE  = pow(2,
-                      ((isset($this->_BIG_BLOCK_SIZE))? $this->_adjust2($this->_BIG_BLOCK_SIZE)  : 9));
-        $this->_SMALL_BLOCK_SIZE= pow(2,
-                      ((isset($this->_SMALL_BLOCK_SIZE))?  $this->_adjust2($this->_SMALL_BLOCK_SIZE): 6));
+                      ((isset($this->_BIG_BLOCK_SIZE)) ? $this->_adjust2($this->_BIG_BLOCK_SIZE) : 9));
+        $this->_SMALL_BLOCK_SIZE = pow(2,
+                      ((isset($this->_SMALL_BLOCK_SIZE)) ? $this->_adjust2($this->_SMALL_BLOCK_SIZE) : 6));
 
         // Open temp file if we are sending output to stdout
         if (($filename == '-') || ($filename == '')) {
             $this->_tmp_filename = tempnam($this->_tmp_dir, "OLE_PPS_Root");
-            $this->_FILEH_ = @fopen($this->_tmp_filename,"w+b");
+            $this->_FILEH_ = @fopen($this->_tmp_filename, "w+b");
             if ($this->_FILEH_ == false) {
                 return $this->raiseError("Can't create temporary file.");
             }
@@ -147,7 +147,7 @@ 
     function _calcSize(&$raList)
     {
         // Calculate Basic Setting
-        list($iSBDcnt, $iBBcnt, $iPPScnt) = array(0,0,0);
+        list($iSBDcnt, $iBBcnt, $iPPScnt) = array(0, 0, 0);
         $iSmallLen = 0;
         $iSBcnt = 0;
         for ($i = 0; $i < count($raList); $i++) {
@@ -155,21 +155,21 @@ 
                 $raList[$i]->Size = $raList[$i]->_DataLen();
                 if ($raList[$i]->Size < OLE_DATA_SIZE_SMALL) {
                     $iSBcnt += floor($raList[$i]->Size / $this->_SMALL_BLOCK_SIZE)
-                                  + (($raList[$i]->Size % $this->_SMALL_BLOCK_SIZE)? 1: 0);
+                                  + (($raList[$i]->Size % $this->_SMALL_BLOCK_SIZE) ? 1 : 0);
                 } else {
                     $iBBcnt += (floor($raList[$i]->Size / $this->_BIG_BLOCK_SIZE) +
-                        (($raList[$i]->Size % $this->_BIG_BLOCK_SIZE)? 1: 0));
+                        (($raList[$i]->Size % $this->_BIG_BLOCK_SIZE) ? 1 : 0));
                 }
             }
         }
         $iSmallLen = $iSBcnt * $this->_SMALL_BLOCK_SIZE;
         $iSlCnt = floor($this->_BIG_BLOCK_SIZE / OLE_LONG_INT_SIZE);
-        $iSBDcnt = floor($iSBcnt / $iSlCnt) + (($iSBcnt % $iSlCnt)? 1:0);
-        $iBBcnt +=  (floor($iSmallLen / $this->_BIG_BLOCK_SIZE) +
-                      (( $iSmallLen % $this->_BIG_BLOCK_SIZE)? 1: 0));
+        $iSBDcnt = floor($iSBcnt / $iSlCnt) + (($iSBcnt % $iSlCnt) ? 1 : 0);
+        $iBBcnt += (floor($iSmallLen / $this->_BIG_BLOCK_SIZE) +
+                      (($iSmallLen % $this->_BIG_BLOCK_SIZE) ? 1 : 0));
         $iCnt = count($raList);
         $iBdCnt = $this->_BIG_BLOCK_SIZE / OLE_PPS_SIZE;
-        $iPPScnt = (floor($iCnt/$iBdCnt) + (($iCnt % $iBdCnt)? 1: 0));
+        $iPPScnt = (floor($iCnt / $iBdCnt) + (($iCnt % $iBdCnt) ? 1 : 0));
 
         return array($iSBDcnt, $iBBcnt, $iPPScnt);
     }
@@ -184,8 +184,8 @@ 
     */
     function _adjust2($i2)
     {
-        $iWk = log($i2)/log(2);
-        return ($iWk > floor($iWk))? floor($iWk)+1:$iWk;
+        $iWk = log($i2) / log(2);
+        return ($iWk > floor($iWk)) ? floor($iWk) + 1 : $iWk;
     }
 
     /**
@@ -207,17 +207,17 @@ 
         $iBdExL = 0;
         $iAll = $iBBcnt + $iPPScnt + $iSBDcnt;
         $iAllW = $iAll;
-        $iBdCntW = floor($iAllW / $iBlCnt) + (($iAllW % $iBlCnt)? 1: 0);
-        $iBdCnt = floor(($iAll + $iBdCntW) / $iBlCnt) + ((($iAllW+$iBdCntW) % $iBlCnt)? 1: 0);
+        $iBdCntW = floor($iAllW / $iBlCnt) + (($iAllW % $iBlCnt) ? 1 : 0);
+        $iBdCnt = floor(($iAll + $iBdCntW) / $iBlCnt) + ((($iAllW + $iBdCntW) % $iBlCnt) ? 1 : 0);
 
         // Calculate BD count
         if ($iBdCnt > $i1stBdL) {
             while (1) {
                 $iBdExL++;
                 $iAllW++;
-                $iBdCntW = floor($iAllW / $iBlCnt) + (($iAllW % $iBlCnt)? 1: 0);
-                $iBdCnt = floor(($iAllW + $iBdCntW) / $iBlCnt) + ((($iAllW+$iBdCntW) % $iBlCnt)? 1: 0);
-                if ($iBdCnt <= ($iBdExL*$iBlCnt+ $i1stBdL)) {
+                $iBdCntW = floor($iAllW / $iBlCnt) + (($iAllW % $iBlCnt) ? 1 : 0);
+                $iBdCnt = floor(($iAllW + $iBdCntW) / $iBlCnt) + ((($iAllW + $iBdCntW) % $iBlCnt) ? 1 : 0);
+                if ($iBdCnt <= ($iBdExL * $iBlCnt + $i1stBdL)) {
                     break;
                 }
             }
@@ -239,7 +239,7 @@ 
                   . "\x00\x00\x00\x00"
                   . "\x00\x00\x00\x00"
                   . pack("V", $iBdCnt)
-                  . pack("V", $iBBcnt+$iSBDcnt) //ROOT START
+                  . pack("V", $iBBcnt + $iSBDcnt) //ROOT START
                   . pack("V", 0)
                   . pack("V", 0x1000)
                   . pack("V", 0)                  //Small Block Depot
@@ -248,19 +248,19 @@ 
         // Extra BDList Start, Count
         if ($iBdCnt < $i1stBdL) {
             fwrite($FILE,
-                      pack("V", -2).      // Extra BDList Start
+                      pack("V", -2).// Extra BDList Start
                       pack("V", 0)        // Extra BDList Count
                   );
         } else {
-            fwrite($FILE, pack("V", $iAll+$iBdCnt) . pack("V", $iBdExL));
+            fwrite($FILE, pack("V", $iAll + $iBdCnt).pack("V", $iBdExL));
         }
 
         // BDList
         for ($i = 0; $i < $i1stBdL && $i < $iBdCnt; $i++) {
-            fwrite($FILE, pack("V", $iAll+$i));
+            fwrite($FILE, pack("V", $iAll + $i));
         }
         if ($i < $i1stBdL) {
-            for ($j = 0; $j < ($i1stBdL-$i); $j++) {
+            for ($j = 0; $j < ($i1stBdL - $i); $j++) {
                 fwrite($FILE, (pack("V", -1)));
             }
         }
@@ -288,7 +288,7 @@ 
                     if (isset($raList[$i]->_PPS_FILE)) {
                         $iLen = 0;
                         fseek($raList[$i]->_PPS_FILE, 0); // To The Top
-                        while($sBuff = fread($raList[$i]->_PPS_FILE, 4096)) {
+                        while ($sBuff = fread($raList[$i]->_PPS_FILE, 4096)) {
                             $iLen += strlen($sBuff);
                             fwrite($FILE, $sBuff);
                         }
@@ -305,7 +305,7 @@ 
                     $raList[$i]->_StartBlock = $iStBlk;
                     $iStBlk +=
                             (floor($raList[$i]->Size / $this->_BIG_BLOCK_SIZE) +
-                                (($raList[$i]->Size % $this->_BIG_BLOCK_SIZE)? 1: 0));
+                                (($raList[$i]->Size % $this->_BIG_BLOCK_SIZE) ? 1 : 0));
                 }
                 // Close file for each PPS, and unlink it
                 if (isset($raList[$i]->_PPS_FILE)) {
@@ -337,10 +337,10 @@ 
                 }
                 if ($raList[$i]->Size < OLE_DATA_SIZE_SMALL) {
                     $iSmbCnt = floor($raList[$i]->Size / $this->_SMALL_BLOCK_SIZE)
-                                  + (($raList[$i]->Size % $this->_SMALL_BLOCK_SIZE)? 1: 0);
+                                  + (($raList[$i]->Size % $this->_SMALL_BLOCK_SIZE) ? 1 : 0);
                     // Add to SBD
-                    for ($j = 0; $j < ($iSmbCnt-1); $j++) {
-                        fwrite($FILE, pack("V", $j+$iSmBlk+1));
+                    for ($j = 0; $j < ($iSmbCnt - 1); $j++) {
+                        fwrite($FILE, pack("V", $j + $iSmBlk + 1));
                     }
                     fwrite($FILE, pack("V", -2));
 
@@ -413,16 +413,16 @@ 
         $iBdExL = 0;
         $iAll = $iBsize + $iPpsCnt + $iSbdSize;
         $iAllW = $iAll;
-        $iBdCntW = floor($iAllW / $iBbCnt) + (($iAllW % $iBbCnt)? 1: 0);
-        $iBdCnt = floor(($iAll + $iBdCntW) / $iBbCnt) + ((($iAllW+$iBdCntW) % $iBbCnt)? 1: 0);
+        $iBdCntW = floor($iAllW / $iBbCnt) + (($iAllW % $iBbCnt) ? 1 : 0);
+        $iBdCnt = floor(($iAll + $iBdCntW) / $iBbCnt) + ((($iAllW + $iBdCntW) % $iBbCnt) ? 1 : 0);
         // Calculate BD count
-        if ($iBdCnt >$i1stBdL) {
+        if ($iBdCnt > $i1stBdL) {
             while (1) {
                 $iBdExL++;
                 $iAllW++;
-                $iBdCntW = floor($iAllW / $iBbCnt) + (($iAllW % $iBbCnt)? 1: 0);
-                $iBdCnt = floor(($iAllW + $iBdCntW) / $iBbCnt) + ((($iAllW+$iBdCntW) % $iBbCnt)? 1: 0);
-                if ($iBdCnt <= ($iBdExL*$iBbCnt+ $i1stBdL)) {
+                $iBdCntW = floor($iAllW / $iBbCnt) + (($iAllW % $iBbCnt) ? 1 : 0);
+                $iBdCnt = floor(($iAllW + $iBdCntW) / $iBbCnt) + ((($iAllW + $iBdCntW) % $iBbCnt) ? 1 : 0);
+                if ($iBdCnt <= ($iBdExL * $iBbCnt + $i1stBdL)) {
                     break;
                 }
             }
@@ -432,19 +432,19 @@ 
         // Set for SBD
         if ($iSbdSize > 0) {
             for ($i = 0; $i < ($iSbdSize - 1); $i++) {
-                fwrite($FILE, pack("V", $i+1));
+                fwrite($FILE, pack("V", $i + 1));
             }
             fwrite($FILE, pack("V", -2));
         }
         // Set for B
         for ($i = 0; $i < ($iBsize - 1); $i++) {
-            fwrite($FILE, pack("V", $i+$iSbdSize+1));
+            fwrite($FILE, pack("V", $i + $iSbdSize + 1));
         }
         fwrite($FILE, pack("V", -2));
 
         // Set for PPS
         for ($i = 0; $i < ($iPpsCnt - 1); $i++) {
-            fwrite($FILE, pack("V", $i+$iSbdSize+$iBsize+1));
+            fwrite($FILE, pack("V", $i + $iSbdSize + $iBsize + 1));
         }
         fwrite($FILE, pack("V", -2));
         // Set for BBD itself ( 0xFFFFFFFD : BBD)
@@ -463,17 +463,17 @@ 
         }
         // Extra BDList
         if ($iBdCnt > $i1stBdL) {
-            $iN=0;
-            $iNb=0;
-            for ($i = $i1stBdL;$i < $iBdCnt; $i++, $iN++) {
+            $iN = 0;
+            $iNb = 0;
+            for ($i = $i1stBdL; $i < $iBdCnt; $i++, $iN++) {
                 if ($iN >= ($iBbCnt - 1)) {
                     $iN = 0;
                     $iNb++;
-                    fwrite($FILE, pack("V", $iAll+$iBdCnt+$iNb));
+                    fwrite($FILE, pack("V", $iAll + $iBdCnt + $iNb));
                 }
-                fwrite($FILE, pack("V", $iBsize+$iSbdSize+$iPpsCnt+$i));
+                fwrite($FILE, pack("V", $iBsize + $iSbdSize + $iPpsCnt + $i));
             }
-            if (($iBdCnt-$i1stBdL) % ($iBbCnt-1)) {
+            if (($iBdCnt - $i1stBdL) % ($iBbCnt - 1)) {
                 for ($i = 0; $i < (($iBbCnt - 1) - (($iBdCnt - $i1stBdL) % ($iBbCnt - 1))); $i++) {
                     fwrite($FILE, pack("V", -1));
                 }