XOOPS / XoopsCore

xoops_lib/Xoops/Form/Label.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     {
         $this->suppressRender(['name', 'value']);
         $attributes = $this->renderAttributeString();
-        $ret = '<div ' . $attributes . '>' . $this->getValue() . '</div>';
+        $ret = '<div '.$attributes.'>'.$this->getValue().'</div>';
         return $ret;
     }
 }

xoops_lib/Xoops/Form/ThemeForm.php

Spacing +4 added lines, -4 removed lines

@@ -35,14 +35,14 @@ 
      */
     public function insertBreak($extra = '', $class = '')
     {
-        $class = ($class != '' ? " class=\"" . $class . "\"" : " class=\"break\"");
+        $class = ($class != '' ? " class=\"".$class."\"" : " class=\"break\"");
         // Fix for $extra tag not showing
         if ($extra) {
-            $value = '<div' . $class . '>' . $extra . '</div>';
+            $value = '<div'.$class.'>'.$extra.'</div>';
             $ele = new Raw($value);
             $this->addElement($ele);
         } else {
-            $value = '<div' . $class . '>&nbsp;</div>';
+            $value = '<div'.$class.'>&nbsp;</div>';
             $ele = new Raw($value);
             $this->addElement($ele);
         }
@@ -95,7 +95,7 @@ 
                 $xoops->tpl()->appendByRef('xo_input', $input);
                 unset($input);
             } else {
-                $hidden .= $ele->render(). "\n";
+                $hidden .= $ele->render()."\n";
             }
 
         }

xoops_lib/Xoops/Form/DateTimeSelect.php

Spacing +9 added lines, -9 removed lines

@@ -47,14 +47,14 @@ 
         } else {
             parent::__construct($caption, '', $name);
             $this->set('value', $value);
-            $this->set(':showtime', (int) $showtime);
+            $this->set(':showtime', (int)$showtime);
         }
 
         $workingTime = \Xoops\Core\Locale\Time::cleanTime($this->get('value', 0));
 
         $displayDate = true;
         $displayTime = true;
-        switch ((int) $this->get(':showtime', static::SHOW_BOTH)) {
+        switch ((int)$this->get(':showtime', static::SHOW_BOTH)) {
             case static::SHOW_DATE:
                 $displayTime = false;
                 break;
@@ -65,8 +65,8 @@ 
 
         $dateDefinition = [
             'caption' => '',
-            'name' => $this->get('name') . '[date]',
-            'id' => $this->get('name') . '-date',
+            'name' => $this->get('name').'[date]',
+            'id' => $this->get('name').'-date',
             'size' => 15,
             'value' => $workingTime,
             ElementFactory::FORM_KEY => $this,
@@ -80,16 +80,16 @@ 
         }
 
         $minuteInterval = $this->get(':minuteinterval', 15);
-        $hours    = (int) ltrim($workingTime->format('H'), '0');
-        $minutes  = (int) ltrim($workingTime->format('i'), '0');
+        $hours    = (int)ltrim($workingTime->format('H'), '0');
+        $minutes  = (int)ltrim($workingTime->format('i'), '0');
 
         $timeDefinition = [
             'caption' => '',
-            'name' => $this->get('name') . '[time]',
-            'id' => $this->get('name') . '-time',
+            'name' => $this->get('name').'[time]',
+            'id' => $this->get('name').'-time',
             'size' => 1,
             'value' => \Xoops\Core\Locale\Time::formatTime(
-                $hours * 3600 + 60*$minuteInterval * ceil($minutes / $minuteInterval),
+                $hours * 3600 + 60 * $minuteInterval * ceil($minutes / $minuteInterval),
                 'short',
                 new \DateTimeZone('UTC')
             ),

xoops_lib/Xoops/Form/GroupPermissionForm.php

Spacing +11 added lines, -11 removed lines

@@ -72,7 +72,7 @@ 
      */
     public function __construct($title, $modid, $permname, $permdesc, $url = '', $anonymous = true)
     {
-        parent::__construct($title, 'groupperm_form', XOOPS_URL . '/modules/system/admin/groupperm.php', 'post');
+        parent::__construct($title, 'groupperm_form', XOOPS_URL.'/modules/system/admin/groupperm.php', 'post');
         $this->modid = (int)($modid);
         $this->permName = $permname;
         $this->permDesc = $permdesc;
@@ -143,7 +143,7 @@ 
             }
             // get selected item id(s) for each group
             $selected = $gperm_handler->getItemIds($this->permName, $i, $this->modid);
-            $ele = new GroupFormCheckbox($glist[$i], 'perms[' . $this->permName . ']', $i, $selected);
+            $ele = new GroupFormCheckbox($glist[$i], 'perms['.$this->permName.']', $i, $selected);
             $ele->setOptionTree($this->itemTree);
             $this->addElement($ele);
             unset($ele);
@@ -153,13 +153,13 @@ 
         $tray->addElement(new Button('', 'reset', \XoopsLocale::A_CANCEL, 'reset'));
         $this->addElement($tray);
 
-        $ret = '<h4>' . $this->getTitle() . '</h4>';
+        $ret = '<h4>'.$this->getTitle().'</h4>';
         if ($this->permDesc) {
-            $ret .= $this->permDesc . '<br /><br />';
+            $ret .= $this->permDesc.'<br /><br />';
         }
-        $ret .= '<form title="' . str_replace('"', '', $this->getTitle()) . '" name="'
-            . $this->getName() . '" id="' . $this->getName() . '" action="' . $this->getAction()
-            . '" method="' . $this->getMethod() . '"' . $this->getExtra() . '>'
+        $ret .= '<form title="'.str_replace('"', '', $this->getTitle()).'" name="'
+            . $this->getName().'" id="'.$this->getName().'" action="'.$this->getAction()
+            . '" method="'.$this->getMethod().'"'.$this->getExtra().'>'
             . '<table width="100%" class="outer" cellspacing="1" valign="top">';
         $elements = $this->getElements();
         $hidden = '';
@@ -167,17 +167,17 @@ 
             if ($elements[$i] instanceof Raw) {
                 $ret .= $elements[$i]->render();
             } elseif (!$elements[$i]->isHidden()) {
-                $ret .= '<tr valign="top" align="left"><td class="head">' . $elements[$i]->getCaption();
+                $ret .= '<tr valign="top" align="left"><td class="head">'.$elements[$i]->getCaption();
                 if ($elements[$i]->getDescription() != "") {
                     $ret .= "<br /><br /><span style='font-weight: normal;'>"
-                        . $elements[$i]->getDescription() . "</span>";
+                        . $elements[$i]->getDescription()."</span>";
                 }
-                $ret .= '</td>' . '<td class="even">' . $elements[$i]->render() . '</td></tr>' . '';
+                $ret .= '</td>'.'<td class="even">'.$elements[$i]->render().'</td></tr>'.'';
             } else {
                 $hidden .= $elements[$i]->render();
             }
         }
-        $ret .= '</table>' . $hidden . '</form>';
+        $ret .= '</table>'.$hidden.'</form>';
         $ret .= $this->renderValidationJS(true);
         return $ret;
     }

xoops_lib/Xoops/Form/Form.php

Spacing +3 added lines, -3 removed lines

@@ -368,7 +368,7 @@ 
      */
     public function getExtra()
     {
-        $extra = empty($this->extra) ? '' : ' ' . implode(' ', $this->extra);
+        $extra = empty($this->extra) ? '' : ' '.implode(' ', $this->extra);
         return $extra;
     }
 
@@ -479,7 +479,7 @@ 
         $elements = array();
         if (count($this->getRequired()) > 0) {
             $this->elements[] =
-                new Raw("<tr class='foot'><td colspan='2'>* = " . \XoopsLocale::REQUIRED . "</td></tr>");
+                new Raw("<tr class='foot'><td colspan='2'>* = ".\XoopsLocale::REQUIRED."</td></tr>");
         }
         foreach ($this->getElements() as $ele) {
             ++$i;
@@ -500,7 +500,7 @@ 
         $tpl->assign($this->getName(), array(
                 'title' => $this->getTitle(), 'name' => $this->getName(), 'action' => $this->getAction(),
                 'method' => $this->getMethod(),
-                'extra' => 'onsubmit="return xoopsFormValidate_' . $this->getName() . '();"' . $this->getExtra(),
+                'extra' => 'onsubmit="return xoopsFormValidate_'.$this->getName().'();"'.$this->getExtra(),
                 'javascript' => $js, 'elements' => $elements
             ));
     }

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/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/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);