mambax7 / XoopsCore25

htdocs/class/xoopsform/formselectcheckgroup.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     public function __construct($caption, $name, $value = null)
     {
         /* @var $member_handler XoopsMemberHandler */
-        $member_handler   = xoops_getHandler('member');
+        $member_handler = xoops_getHandler('member');
         $userGroups = $member_handler->getGroupList();
         parent::__construct($caption, $name, $value);
         $this->columns = 3;

htdocs/class/xoopsform/formelement.php

Spacing +6 added lines, -6 removed lines

@@ -193,7 +193,7 @@ 
     {
         $access = $this->getAccessKey();
         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);
+            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);
         }
 
         return htmlspecialchars($str, ENT_QUOTES);
@@ -260,7 +260,7 @@ 
     public function getTitle($encode = true)
     {
         if (strlen($this->_description) > 0) {
-            return $encode ? htmlspecialchars(strip_tags($this->_caption . ' - ' . $this->_description), ENT_QUOTES) : strip_tags($this->_caption . ' - ' . $this->_description);
+            return $encode ? htmlspecialchars(strip_tags($this->_caption.' - '.$this->_description), ENT_QUOTES) : strip_tags($this->_caption.' - '.$this->_description);
         } else {
             return $encode ? htmlspecialchars(strip_tags($this->_caption), ENT_QUOTES) : strip_tags($this->_caption);
         }
@@ -345,14 +345,14 @@ 
     public function getExtra($encode = false)
     {
         if (!$encode) {
-            return ' ' . implode(' ', $this->_extra);
+            return ' '.implode(' ', $this->_extra);
         }
         $value = array();
         foreach ($this->_extra as $val) {
             $value[] = str_replace('>', '&gt;', str_replace('<', '&lt;', $val));
         }
 
-        return empty($value) ? '' : ' ' . implode(' ', $value);
+        return empty($value) ? '' : ' '.implode(' ', $value);
     }
 
     /**
@@ -428,10 +428,10 @@ 
             echo $this->getFormType();
             switch ($this->getFormType()) {
                 case 'checkbox':
-                    return NWLINE . "if (!myform.{$eltname}.checked) { window.alert(\"{$eltmsg}\"); myform.{$eltname}.focus(); return false; }\n";
+                    return NWLINE."if (!myform.{$eltname}.checked) { window.alert(\"{$eltmsg}\"); myform.{$eltname}.focus(); return false; }\n";
                     break;
                 default:
-                    return NWLINE . "if (myform.{$eltname}.value == \"\") { window.alert(\"{$eltmsg}\"); myform.{$eltname}.focus(); return false; }\n";
+                    return NWLINE."if (myform.{$eltname}.value == \"\") { window.alert(\"{$eltmsg}\"); myform.{$eltname}.focus(); return false; }\n";
                     break;
             } // switch
         }

htdocs/class/xoopsform/grouppermform.php

Spacing +21 added lines, -21 removed lines

@@ -73,7 +73,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;
@@ -148,7 +148,7 @@ 
             }
             // get selected item id(s) for each group
             $selected = $gperm_handler->getItemIds($this->_permName, $i, $this->_modid);
-            $ele      = new XoopsGroupFormCheckBox($glist[$i], 'perms[' . $this->_permName . ']', $i, $selected);
+            $ele      = new XoopsGroupFormCheckBox($glist[$i], 'perms['.$this->_permName.']', $i, $selected);
             $ele->setOptionTree($this->_itemTree);
             $this->addElement($ele);
             unset($ele);
@@ -158,27 +158,27 @@ 
         $tray->addElement(new XoopsFormButton('', 'reset', _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() . '>' . '<table width="100%" class="outer" cellspacing="1" valign="top">';
-        $elements =& $this->getElements();
+        $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   = '';
         foreach (array_keys($elements) as $i) {
             if (!is_object($elements[$i])) {
                 $ret .= $elements[$i];
             } 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>';
+                    $ret .= "<br><br><span style='font-weight: normal;'>".$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;
@@ -273,19 +273,19 @@ 
             $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.']'."'";
             }
         }
-        $checkallbtn_id = $ele_name . '[checkallbtn][' . $this->_groupId . ']';
+        $checkallbtn_id = $ele_name.'[checkallbtn]['.$this->_groupId.']';
         $option_ids_str = implode(', ', $option_ids);
-        $ret .= _ALL . " <input id=\"" . $checkallbtn_id . "\" type=\"checkbox\" value=\"\" onclick=\"var optionids = new Array(" . $option_ids_str . "); xoopsCheckAllElements(optionids, '" . $checkallbtn_id . "');\" />";
+        $ret .= _ALL." <input id=\"".$checkallbtn_id."\" type=\"checkbox\" value=\"\" onclick=\"var optionids = new Array(".$option_ids_str."); xoopsCheckAllElements(optionids, '".$checkallbtn_id."');\" />";
         $ret .= '</td></tr></table>';
 
         return $ret;
@@ -303,31 +303,31 @@ 
     public function _renderOptionTree(&$tree, $option, $prefix, $parentIds = array())
     {
         $ele_name = $this->getName();
-        $tree .= $prefix . "<input type=\"checkbox\" name=\"" . $ele_name . '[groups][' . $this->_groupId . '][' . $option['id'] . "]\" id=\"" . $ele_name . '[groups][' . $this->_groupId . '][' . $option['id'] . "]\" onclick=\"";
+        $tree .= $prefix."<input type=\"checkbox\" name=\"".$ele_name.'[groups]['.$this->_groupId.']['.$option['id']."]\" id=\"".$ele_name.'[groups]['.$this->_groupId.']['.$option['id']."]\" onclick=\"";
         // If there are parent elements, add javascript that will
         // make them selecteded when this element is checked to make
         // sure permissions to parent items are added as well.
         foreach ($parentIds as $pid) {
-            $parent_ele = $ele_name . '[groups][' . $this->_groupId . '][' . $pid . ']';
-            $tree .= "var ele = xoopsGetElementById('" . $parent_ele . "'); if(ele.checked != true) {ele.checked = this.checked;}";
+            $parent_ele = $ele_name.'[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
         // make them unchecked when this element is unchecked to make
         // sure permissions to child items are not added when there
         // is no permission to this item.
         foreach ($option['allchild'] as $cid) {
-            $child_ele = $ele_name . '[groups][' . $this->_groupId . '][' . $cid . ']';
-            $tree .= "var ele = xoopsGetElementById('" . $child_ele . "'); if(this.checked != true) {ele.checked = false;}";
+            $child_ele = $ele_name.'[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->_value)) {
             $tree .= ' checked';
         }
-        $tree .= ' />' . $option['name'] . "<input type=\"hidden\" name=\"" . $ele_name . '[parents][' . $option['id'] . "]\" value=\"" . implode(':', $parentIds) . "\" /><input type=\"hidden\" name=\"" . $ele_name . '[itemname][' . $option['id'] . "]\" value=\"" . htmlspecialchars($option['name']) . "\" /><br>\n";
+        $tree .= ' />'.$option['name']."<input type=\"hidden\" name=\"".$ele_name.'[parents]['.$option['id']."]\" value=\"".implode(':', $parentIds)."\" /><input type=\"hidden\" name=\"".$ele_name.'[itemname]['.$option['id']."]\" value=\"".htmlspecialchars($option['name'])."\" /><br>\n";
         if (isset($option['children'])) {
             foreach ($option['children'] as $child) {
                 $parentIds[] = $option['id'];
-                $this->_renderOptionTree($tree, $this->_optionTree[$child], $prefix . '&nbsp;-', $parentIds);
+                $this->_renderOptionTree($tree, $this->_optionTree[$child], $prefix.'&nbsp;-', $parentIds);
             }
         }
     }

htdocs/class/xoopsform/formdhtmltextarea.php

Spacing +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
                 }
             } else {
                 list($class, $path) = $this->htmlEditor;
-                include_once XOOPS_ROOT_PATH . $path;
+                include_once XOOPS_ROOT_PATH.$path;
                 if (class_exists($class)) {
                     $this->htmlEditor = new $class($options);
                 }

htdocs/class/xoopsform/formdatetime.php

Spacing +6 added lines, -6 removed lines

@@ -46,7 +46,7 @@ 
     public function __construct($caption, $name, $size = 15, $value = 0, $showtime = true)
     {
         parent::__construct($caption, ' ');
-        switch ((int) $showtime) {
+        switch ((int)$showtime) {
             case static::SHOW_DATE:
                 $displayDate = true;
                 $displayTime = false;
@@ -64,12 +64,12 @@ 
         $value    = ($value > 0) ? $value : time();
         $datetime = getdate($value);
         if ($displayDate) {
-            $this->addElement(new XoopsFormTextDateSelect('', $name . '[date]', $size, $value));
+            $this->addElement(new XoopsFormTextDateSelect('', $name.'[date]', $size, $value));
         } else {
             $value = !is_numeric($value) ? time() : (int)$value;
             $value = ($value == 0) ? time() : $value;
             $displayValue = date(_SHORTDATESTRING, $value);
-            $this->addElement(new XoopsFormHidden($name . '[date]', $displayValue));
+            $this->addElement(new XoopsFormHidden($name.'[date]', $displayValue));
         }
 
         if ($displayTime) {
@@ -77,16 +77,16 @@ 
             for ($i = 0; $i < 24; ++$i) {
                 for ($j = 0; $j < 60; $j += 10) {
                     $key = ($i * 3600) + ($j * 60);
-                    $timearray[$key] = ($j != 0) ? $i . ':' . $j : $i . ':0' . $j;
+                    $timearray[$key] = ($j != 0) ? $i.':'.$j : $i.':0'.$j;
                 }
             }
             ksort($timearray);
 
-            $timeselect = new XoopsFormSelect('', $name . '[time]', $datetime['hours'] * 3600 + 600 * ceil($datetime['minutes'] / 10));
+            $timeselect = new XoopsFormSelect('', $name.'[time]', $datetime['hours'] * 3600 + 600 * ceil($datetime['minutes'] / 10));
             $timeselect->addOptionArray($timearray);
             $this->addElement($timeselect);
         } else {
-            $this->addElement(new XoopsFormHidden($name . '[time]', 0));
+            $this->addElement(new XoopsFormHidden($name.'[time]', 0));
         }
     }
 }

htdocs/class/xoopsform/tableform.php

Spacing +8 added lines, -8 removed lines

@@ -42,25 +42,25 @@
      */
     public function render()
     {
-        $ret    = $this->getTitle() . NWLINE . '<form name="' . $this->getName() . '" id="' . $this->getName() . '" action="' . $this->getAction() . '" method="' . $this->getMethod() . '"' . $this->getExtra() . '>' . NWLINE . '<table border="0" width="100%">' . NWLINE;
+        $ret    = $this->getTitle().NWLINE.'<form name="'.$this->getName().'" id="'.$this->getName().'" action="'.$this->getAction().'" method="'.$this->getMethod().'"'.$this->getExtra().'>'.NWLINE.'<table border="0" width="100%">'.NWLINE;
         $hidden = '';
         foreach ($this->getElements() as $ele) {
             if (!$ele->isHidden()) {
                 if (!$ele->getNocolspan()) {
-                    $ret .= '<tr valign="top" align="left"><td>' . $ele->getCaption();
+                    $ret .= '<tr valign="top" align="left"><td>'.$ele->getCaption();
                     if ($ele_desc = $ele->getDescription()) {
-                        $ret .= '<br><br><span style="font-weight: normal;">' . $ele_desc . '</span>';
+                        $ret .= '<br><br><span style="font-weight: normal;">'.$ele_desc.'</span>';
                     }
-                    $ret .= '</td><td>' . $ele->render() . '</td></tr>';
+                    $ret .= '</td><td>'.$ele->render().'</td></tr>';
                 } else {
-                    $ret .= '<tr valign="top" align="left"><td colspan="2">' . $ele->getCaption();
-                    $ret .= '</td></tr><tr valign="top" align="left"><td>' . $ele->render() . '</td></tr>';
+                    $ret .= '<tr valign="top" align="left"><td colspan="2">'.$ele->getCaption();
+                    $ret .= '</td></tr><tr valign="top" align="left"><td>'.$ele->render().'</td></tr>';
                 }
             } else {
-                $hidden .= $ele->render() . NWLINE;
+                $hidden .= $ele->render().NWLINE;
             }
         }
-        $ret .= '</table>' . NWLINE . ' ' . $hidden . '</form>' . NWLINE;
+        $ret .= '</table>'.NWLINE.' '.$hidden.'</form>'.NWLINE;
 
         return $ret;
     }

htdocs/class/xoopsform/formhiddentoken.php

Spacing +1 added lines, -1 removed lines

@@ -32,6 +32,6 @@
      */
     public function __construct($name = 'XOOPS_TOKEN', $timeout = 0)
     {
-        parent::__construct($name . '_REQUEST', $GLOBALS['xoopsSecurity']->createToken($timeout, $name));
+        parent::__construct($name.'_REQUEST', $GLOBALS['xoopsSecurity']->createToken($timeout, $name));
     }
 }

htdocs/class/xoopsform/formselecteditor.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         $editor_handler->allowed_editors = $this->allowed_editors;
         $option_select                   = new XoopsFormSelect('', $this->name, $this->value);
         $extra                           = 'onchange="if (this.options[this.selectedIndex].value.length > 0) {
-            window.document.forms.' . $this->form->getName() . '.submit();
+            window.document.forms.' . $this->form->getName().'.submit();
             }"';
         $option_select->setExtra($extra);
         $option_select->addOptionArray($editor_handler->getList($this->nohtml));

htdocs/class/xoopsform/formhidden.php

Spacing +1 added lines, -1 removed lines

@@ -75,6 +75,6 @@
      */
     public function render()
     {
-        return '<input type="hidden" name="' . $this->getName() . '" id="' . $this->getName() . '" value="' . $this->getValue() . '" />';
+        return '<input type="hidden" name="'.$this->getName().'" id="'.$this->getName().'" value="'.$this->getValue().'" />';
     }
 }