XOOPS / XoopsCore

xoops_lib/Xoops/Form/GroupFormCheckbox.php

Spacing +19 added lines, -19 removed lines

@@ -53,7 +53,7 @@ 
             parent::__construct([]);
             $this->setCaption($caption);
             $this->setName($name);
-            $this->setWithDefaults('value', (array) $values, []);
+            $this->setWithDefaults('value', (array)$values, []);
             $this->set(':groupid', $groupId);
         }
         $this->groupId = $this->get(':groupid', 0);
@@ -90,21 +90,21 @@ 
             $tree = '<td valign="top">';
             $prefix = '';
             $this->renderOptionTree($tree, $this->optionTree[$topitem], $prefix);
-            $ret .= $tree . '</td>';
+            $ret .= $tree.'</td>';
             ++$cols;
         }
         $ret .= '</tr></table></td><td class="even" valign="top">';
         $option_ids = array();
         foreach (array_keys($this->optionTree) as $id) {
             if (!empty($id)) {
-                $option_ids[] = "'" . $ele_name . '[groups][' . $this->groupId . '][' . $id . ']' . "'";
+                $option_ids[] = "'".$ele_name.'[groups]['.$this->groupId.']['.$id.']'."'";
             }
         }
-        $checkAllButtonId = $ele_name . '[checkallbtn][' . $this->groupId . ']';
+        $checkAllButtonId = $ele_name.'[checkallbtn]['.$this->groupId.']';
         $option_ids_str = implode(', ', $option_ids);
-        $ret .= \XoopsLocale::ALL . " <input id=\"" . $checkAllButtonId . "\" type=\"checkbox\" value=\"\" "
-            . "onclick=\"var optionids = new Array(" . $option_ids_str . "); "
-            . "xoopsCheckAllElements(optionids, '" . $checkAllButtonId . "');\" />";
+        $ret .= \XoopsLocale::ALL." <input id=\"".$checkAllButtonId."\" type=\"checkbox\" value=\"\" "
+            . "onclick=\"var optionids = new Array(".$option_ids_str."); "
+            . "xoopsCheckAllElements(optionids, '".$checkAllButtonId."');\" />";
         $ret .= '</td></tr></table>';
         return $ret;
     }
@@ -122,15 +122,15 @@ 
     private function renderOptionTree(&$tree, $option, $prefix, $parentIds = array())
     {
         $elementName = $this->getName();
-        $tree .= $prefix . "<input type=\"checkbox\" name=\"" . $elementName . "[groups]["
-            . $this->groupId . "][" . $option['id'] . "]\" id=\"" . $elementName
-            . "[groups][" . $this->groupId . "][" . $option['id'] . "]\" onclick=\"";
+        $tree .= $prefix."<input type=\"checkbox\" name=\"".$elementName."[groups]["
+            . $this->groupId."][".$option['id']."]\" id=\"".$elementName
+            . "[groups][".$this->groupId."][".$option['id']."]\" onclick=\"";
         // If there are parent elements, add javascript that will
         // make them selected when this element is checked to make
         // sure permissions to parent items are added as well.
         foreach ($parentIds as $pid) {
-            $parent_ele = $elementName . '[groups][' . $this->groupId . '][' . $pid . ']';
-            $tree .= "var ele = xoopsGetElementById('" . $parent_ele
+            $parent_ele = $elementName.'[groups]['.$this->groupId.']['.$pid.']';
+            $tree .= "var ele = xoopsGetElementById('".$parent_ele
                 . "'); if(ele.checked != true) {ele.checked = this.checked;}";
         }
         // If there are child elements, add javascript that will
@@ -138,22 +138,22 @@ 
         // sure permissions to child items are not added when there
         // is no permission to this item.
         foreach ($option['allchild'] as $cid) {
-            $child_ele = $elementName . '[groups][' . $this->groupId . '][' . $cid . ']';
-            $tree .= "var ele = xoopsGetElementById('" . $child_ele
+            $child_ele = $elementName.'[groups]['.$this->groupId.']['.$cid.']';
+            $tree .= "var ele = xoopsGetElementById('".$child_ele
                 . "'); if(this.checked != true) {ele.checked = false;}";
         }
         $tree .= '" value="1"';
         if (in_array($option['id'], $this->get('value', []))) {
             $tree .= ' checked="checked"';
         }
-        $tree .= " />" . $option['name'] . "<input type=\"hidden\" name=\"" . $elementName . "[parents]["
-            . $option['id'] . "]\" value=\"" . implode(':', $parentIds) . "\" /><input type=\"hidden\" name=\""
-            . $elementName . "[itemname][" . $option['id'] . "]\" value=\""
-            . htmlspecialchars($option['name']) . "\" /><br />\n";
+        $tree .= " />".$option['name']."<input type=\"hidden\" name=\"".$elementName."[parents]["
+            . $option['id']."]\" value=\"".implode(':', $parentIds)."\" /><input type=\"hidden\" name=\""
+            . $elementName."[itemname][".$option['id']."]\" value=\""
+            . htmlspecialchars($option['name'])."\" /><br />\n";
         if (isset($option['children'])) {
             foreach ($option['children'] as $child) {
                 array_push($parentIds, $option['id']);
-                $this->renderOptionTree($tree, $this->optionTree[$child], $prefix . '&nbsp;-', $parentIds);
+                $this->renderOptionTree($tree, $this->optionTree[$child], $prefix.'&nbsp;-', $parentIds);
             }
         }
     }

xoops_lib/Xoops/Form/Radio.php

Spacing +5 added lines, -5 removed lines

@@ -60,7 +60,7 @@ 
         $ele_options = $this->getOptions();
         $ele_value = $this->getValue();
         $ele_name = $this->getName();
-        $extra = ($this->getExtra() != '' ? " " . $this->getExtra() : '');
+        $extra = ($this->getExtra() != '' ? " ".$this->getExtra() : '');
         $ret = "";
         $inline = $this->has(':inline');
         if ($inline) {
@@ -74,15 +74,15 @@ 
             }
             $this->set('value', $value);
             ++$id_ele;
-            $this->set('id', $ele_name . $id_ele);
+            $this->set('id', $ele_name.$id_ele);
             if ($inline) {
                 $ret .= '<label class="radio-inline">';
-                $ret .= '<input ' . $this->renderAttributeString() . $extra . ">" . $buttonCaption . "\n";
+                $ret .= '<input '.$this->renderAttributeString().$extra.">".$buttonCaption."\n";
                 $ret .= "</label>\n";
             } else {
                 $ret .= "<div class=\"radio\">\n<label>";
-                $ret .= '<input ' . $this->renderAttributeString() . $extra . '>' . "\n";
-                $ret .= $buttonCaption . "\n";
+                $ret .= '<input '.$this->renderAttributeString().$extra.'>'."\n";
+                $ret .= $buttonCaption."\n";
                 $ret .= "</label>\n</div>\n";
             }
         }

xoops_lib/Xoops/Form/ButtonTray.php

Spacing +8 added lines, -8 removed lines

@@ -58,7 +58,7 @@ 
      */
     public function getType()
     {
-        return (string) $this->get('type', '');
+        return (string)$this->get('type', '');
     }
 
     /**
@@ -70,18 +70,18 @@ 
     {
         $ret = '';
         $this->add('class', 'btn');
-        $class = 'class="' . $this->getClass() . '"';
+        $class = 'class="'.$this->getClass().'"';
 
         $attributes = $this->renderAttributeString();
 
-        if ((bool) $this->get(':showdelete', false)) {
-            $ret .= '<input type="submit"' . $class . ' name="delete" id="delete" value="'
-                . \XoopsLocale::A_DELETE . '" onclick="this.form.elements.op.value=\'delete\'">';
+        if ((bool)$this->get(':showdelete', false)) {
+            $ret .= '<input type="submit"'.$class.' name="delete" id="delete" value="'
+                . \XoopsLocale::A_DELETE.'" onclick="this.form.elements.op.value=\'delete\'">';
         }
-        $ret .= ' <input type="button" ' . $class . ' value="' . \XoopsLocale::A_CANCEL
+        $ret .= ' <input type="button" '.$class.' value="'.\XoopsLocale::A_CANCEL
             . '" onclick="history.go(-1);return true;" />'
-            . ' <input type="reset"' . $class . ' name="reset"  id="reset" value="' . \XoopsLocale::A_RESET . '" />'
-            . ' <input ' . $attributes . $this->getExtra() . ' />';
+            . ' <input type="reset"'.$class.' name="reset"  id="reset" value="'.\XoopsLocale::A_RESET.'" />'
+            . ' <input '.$attributes.$this->getExtra().' />';
         return $ret;
     }
 }

xoops_lib/Xoops/Form/ElementFactory.php

Spacing +3 added lines, -3 removed lines

@@ -59,15 +59,15 @@
         }
         $elementClass = $specification[self::CLASS_KEY];
         if (false === strpos($elementClass, '\\')) {
-            $elementClass = '\Xoops\Form\\' . $elementClass;
+            $elementClass = '\Xoops\Form\\'.$elementClass;
         }
 
         if (!class_exists($elementClass)) {
-            throw new \DomainException('Unknown element class: ' . $specification[self::CLASS_KEY]);
+            throw new \DomainException('Unknown element class: '.$specification[self::CLASS_KEY]);
         }
 
         if (!is_a($elementClass, '\Xoops\Form\Element', true)) {
-            throw new \DomainException('Not an Element subclass: ' . $specification[self::CLASS_KEY]);
+            throw new \DomainException('Not an Element subclass: '.$specification[self::CLASS_KEY]);
         }
 
         $specification[self::CLASS_KEY] = $elementClass;

xoops_lib/Xoops/Form/ColorPicker.php

Spacing +6 added lines, -6 removed lines

@@ -36,7 +36,7 @@ 
     {
         if (is_array($caption)) {
             parent::__construct($caption);
-            $value = $this->get('value','');
+            $value = $this->get('value', '');
             if (empty($value)) {
                 $this->set('value', '#FFFFFF');
             }
@@ -64,21 +64,21 @@ 
         if ($xoops->theme()) {
             $xoops->theme()->addScript('include/color-picker.js');
         } else {
-            echo '<script type="text/javascript" src="' . $xoops->url('/include/color-picker.js') . '"></script>';
+            echo '<script type="text/javascript" src="'.$xoops->url('/include/color-picker.js').'"></script>';
         }
         $temp = $this->get('value', '');
         if (!empty($temp)) {
-            $this->set('style', 'background-color:' . $temp . ';');
+            $this->set('style', 'background-color:'.$temp.';');
         }
         $this->set('class', 'form-control');
         $ret = '<div class="input-group">';
         $attributes = $this->renderAttributeString();
-        $ret .= '<input ' . $attributes . ' ' . $this->getExtra() .' >';
+        $ret .= '<input '.$attributes.' '.$this->getExtra().' >';
         $ret .= '<span class="input-group-btn">';
         $ret .= '<button class="btn btn-default" type="button" ';
-        $ret .= 'data-toggle="tooltip" data-placement="left" title="' . \XoopsLocale::A_SELECT . '" ';
+        $ret .= 'data-toggle="tooltip" data-placement="left" title="'.\XoopsLocale::A_SELECT.'" ';
         $ret .= 'onclick="return TCP.popup(\'';
-        $ret .= $xoops->url('/include/') . '\',document.getElementById(\'' . $this->getName() . '\'));">';
+        $ret .= $xoops->url('/include/').'\',document.getElementById(\''.$this->getName().'\'));">';
         $ret .= '<span class="glyphicon glyphicon-option-horizontal" aria-hidden="true"></span></button>';
         $ret .= '</span></div>';
 

xoops_lib/Xoops/Form/SelectUser.php

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
         $user_count = $member_handler->getUserCount();
         $value = is_array($value) ? $value : (empty($value) ? array() : array($value));
         if ($user_count > $limit && count($value) > 0) {
-            $criteria = new CriteriaCompo(new Criteria('uid', '(' . implode(',', $value) . ')', 'IN'));
+            $criteria = new CriteriaCompo(new Criteria('uid', '('.implode(',', $value).')', 'IN'));
         } else {
             $criteria = new CriteriaCompo();
             $criteria->setLimit($limit);
@@ -72,7 +72,7 @@ 
             function addusers(opts){
                 var num = opts.substring(0, opts.indexOf(':'));
                 opts = opts.substring(opts.indexOf(':')+1, opts.length);
-                var sel = xoopsGetElementById('" . $name . "');
+                var sel = xoopsGetElementById('" . $name."');
                 var arr = new Array(num);
                 for (var n=0; n < num; n++) {
                     var nm = opts.substring(0, opts.indexOf(':'));
@@ -99,14 +99,14 @@ 
         $token = $xoops->security()->createToken();
         $action_tray = new Label(
             '',
-            '<a href="#" onclick="var sel = xoopsGetElementById(\'' . $name
+            '<a href="#" onclick="var sel = xoopsGetElementById(\''.$name
                 . '\');for (var i = sel.options.length-1; i >= 0; i--) {if (!sel.options[i].selected) '
-                .'{sel.options[i] = null;}}; return false;">' . \XoopsLocale::REMOVE_UNSELECTED_USERS . "</a>"
+                .'{sel.options[i] = null;}}; return false;">'.\XoopsLocale::REMOVE_UNSELECTED_USERS."</a>"
                 . ' | '
-                . '<a href="#" onclick="openWithSelfMain(\'' . \XoopsBaseConfig::get('url') . '/include/findusers.php?target='
-                . $name . '&amp;multiple=' . $multiple . '&amp;token=' . $token
+                . '<a href="#" onclick="openWithSelfMain(\''.\XoopsBaseConfig::get('url').'/include/findusers.php?target='
+                . $name.'&amp;multiple='.$multiple.'&amp;token='.$token
                 . '\', \'userselect\', 800, 600, null); return false;" >'
-                . \XoopsLocale::SEARCH_USERS . "</a>" . $js_addusers
+                . \XoopsLocale::SEARCH_USERS."</a>".$js_addusers
         );
         parent::__construct($caption, '', $name);
         $this->addElement($select_element);

xoops_lib/Xoops/Form/Element.php

Indentation +1 added lines, -1 removed lines

@@ -581,7 +581,7 @@
                 $class = 'form-control';
             }
         }
-     */
+         */
         $this->add('class', $class);
     }
 

Spacing +12 added lines, -12 removed lines

@@ -92,7 +92,7 @@ 
         if (!$this->has('id')) {
             $id = $this->get('name');
             if (substr($id, -2) === '[]') {
-                $id = substr($id, 0, strlen($id)-2);
+                $id = substr($id, 0, strlen($id) - 2);
             }
             $this->set('id', $id);
         }
@@ -150,7 +150,7 @@ 
      */
     public function getName()
     {
-        return (string) $this->get('name');
+        return (string)$this->get('name');
     }
 
     /**
@@ -172,7 +172,7 @@ 
      */
     public function getAccessKey()
     {
-        return (string) $this->get('accesskey');
+        return (string)$this->get('accesskey');
     }
 
     /**
@@ -188,7 +188,7 @@ 
         if (!empty($access) && (false !== ($pos = strpos($str, $access)))) {
             return htmlspecialchars(substr($str, 0, $pos), ENT_QUOTES)
                 . '<span style="text-decoration: underline;">'
-                . htmlspecialchars(substr($str, $pos, 1), ENT_QUOTES) . '</span>'
+                . htmlspecialchars(substr($str, $pos, 1), ENT_QUOTES).'</span>'
                 . htmlspecialchars(substr($str, $pos + 1), ENT_QUOTES);
         }
         return htmlspecialchars($str, ENT_QUOTES);
@@ -203,7 +203,7 @@ 
      */
     public function setClass($class)
     {
-        $this->add('class', (string) $class);
+        $this->add('class', (string)$class);
     }
 
     /**
@@ -241,7 +241,7 @@ 
      */
     public function getPattern()
     {
-        return (string) $this->get('pattern', '');
+        return (string)$this->get('pattern', '');
     }
 
     /**
@@ -251,7 +251,7 @@ 
      */
     public function getPatternDescription()
     {
-        return (string) $this->get(':pattern_description', '');
+        return (string)$this->get(':pattern_description', '');
     }
 
     /**
@@ -276,11 +276,11 @@ 
         if (!$this->isDatalist()) {
             return '';
         }
-        $ret = "\n" . '<datalist id="list_' . $this->getName() . '">' . "\n";
+        $ret = "\n".'<datalist id="list_'.$this->getName().'">'."\n";
         foreach ($this->get('datalist') as $datalist) {
-            $ret .= '<option value="' . htmlspecialchars($datalist, ENT_QUOTES) . '">' . "\n";
+            $ret .= '<option value="'.htmlspecialchars($datalist, ENT_QUOTES).'">'."\n";
         }
-        $ret .= '</datalist>' . "\n";
+        $ret .= '</datalist>'."\n";
         return $ret;
     }
 
@@ -341,7 +341,7 @@ 
         } else {
             if ($this->has(':pattern_description')) {
                 return htmlspecialchars(
-                    strip_tags($this->get('caption') . ' - ' . $this->get(':pattern_description')),
+                    strip_tags($this->get('caption').' - '.$this->get(':pattern_description')),
                     ENT_QUOTES
                 );
             } else {
@@ -526,7 +526,7 @@ 
         $class = $this->get('class');
         if ($class) {
             $length = strlen($pattern);
-            foreach ((array) $class as $i => $value) {
+            foreach ((array)$class as $i => $value) {
                 if (0 === strncmp($value, $pattern, $length)) {
                     return $i;
                 }

xoops_lib/Xoops/Form/BlockForm.php

Spacing +6 added lines, -6 removed lines

@@ -48,14 +48,14 @@
             }
             if (!$ele->isHidden()) {
                 $ret .= '<div class="form-group">';
-                $ret .= '<label>' . $ele->getCaption();
-                $ret .= ($ele->isRequired() ? '<span class="caption-required">*</span>' : '') . '</label>';
+                $ret .= '<label>'.$ele->getCaption();
+                $ret .= ($ele->isRequired() ? '<span class="caption-required">*</span>' : '').'</label>';
                 $ret .= $ele->render();
-                $ret .= '<small class="text-muted">' . $ele->getDescription() . '</small>';
-                $ret .= '<p class="dsc_pattern_vertical">' . $ele->getPatternDescription() . '</p>';
-                $ret .= '</div>' . "\n";
+                $ret .= '<small class="text-muted">'.$ele->getDescription().'</small>';
+                $ret .= '<p class="dsc_pattern_vertical">'.$ele->getPatternDescription().'</p>';
+                $ret .= '</div>'."\n";
             } else {
-                $ret .= $ele->render(). "\n";
+                $ret .= $ele->render()."\n";
             }
         }
         $ret .= '</div>';

xoops_lib/Xoops/Form/Select.php

Indentation +8 added lines, -8 removed lines

@@ -76,14 +76,14 @@
         return (int) $this->get('size');
     }
 
-     /**
-     * Add multiple optgroup
-     *
-     * @param string $name     name attribute
-     * @param array  $optgroup Associative array of value->name pairs
-     *
-     * @return void
-     */
+        /**
+         * Add multiple optgroup
+         *
+         * @param string $name     name attribute
+         * @param array  $optgroup Associative array of value->name pairs
+         *
+         * @return void
+         */
     public function addOptionGroup($name, $optgroup)
     {
         $this->setArrayItem('option', $name, $optgroup);

Spacing +9 added lines, -9 removed lines

@@ -73,7 +73,7 @@ 
      */
     public function getSize()
     {
-        return (int) $this->get('size');
+        return (int)$this->get('size');
     }
 
      /**
@@ -100,11 +100,11 @@ 
      */
     protected function renderOption($optionValue, $optionDisplay, $selected)
     {
-        $rendered = '<option value="' . htmlspecialchars($optionValue, ENT_QUOTES) . '"';
+        $rendered = '<option value="'.htmlspecialchars($optionValue, ENT_QUOTES).'"';
         if (in_array($optionValue, $selected)) {
             $rendered .= ' selected="selected"';
         }
-        $rendered .= '>' . $optionDisplay . '</option>' . "\n";
+        $rendered .= '>'.$optionDisplay.'</option>'."\n";
 
         return $rendered;
     }
@@ -116,19 +116,19 @@ 
      */
     public function render()
     {
-        $selected = (array) $this->getValue();
+        $selected = (array)$this->getValue();
 
         $ele_options = $this->getOptions();
 
-        $extra = ($this->getExtra() != '' ? " " . $this->getExtra() : '');
+        $extra = ($this->getExtra() != '' ? " ".$this->getExtra() : '');
         $this->themeDecorateElement();
         $attributes = $this->renderAttributeString();
-        $rendered = '<select ' . $attributes . $extra .' >' . "\n";
+        $rendered = '<select '.$attributes.$extra.' >'."\n";
 
         if (empty($ele_optgroup)) {
             foreach ($ele_options as $value => $display) {
                 if (is_array($display)) {
-                    $rendered .= '<optgroup label="' . $value . '">' . "\n";
+                    $rendered .= '<optgroup label="'.$value.'">'."\n";
                     foreach ($display as $optvalue => $optdisplay) {
                         $rendered .= $this->renderOption($optvalue, $optdisplay, $selected);
                     }
@@ -137,7 +137,7 @@ 
                 }
             }
         }
-        $rendered .= '</select>' . "\n";
+        $rendered .= '</select>'."\n";
 
         return $rendered;
     }
@@ -163,7 +163,7 @@ 
             return "\nvar hasSelected = false; var selectBox = myform.{$eltname};"
                 . "for (i = 0; i < selectBox.options.length; i++ ) { "
                 . "if (selectBox.options[i].selected == true && selectBox.options[i].value != '') "
-                . "{ hasSelected = true; break; } }" . "if (!hasSelected) "
+                . "{ hasSelected = true; break; } }"."if (!hasSelected) "
                 . "{ window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }";
         }
         return '';