mambax7 / smartobject

class/form/elements/smartformuserelement.php

Indentation +27 added lines, -27 removed lines

@@ -11,36 +11,36 @@
  */
 class SmartFormUserElement extends XoopsFormSelect
 {
-    public $multiple = false;
+	public $multiple = false;
 
-    /**
-     * SmartFormUserElement constructor.
-     * @param string $object
-     * @param string $key
-     */
-    public function __construct($object, $key)
-    {
-        $var  = $object->vars[$key];
-        $size = isset($var['size']) ? $var['size'] : ($this->multiple ? 5 : 1);
+	/**
+	 * SmartFormUserElement constructor.
+	 * @param string $object
+	 * @param string $key
+	 */
+	public function __construct($object, $key)
+	{
+		$var  = $object->vars[$key];
+		$size = isset($var['size']) ? $var['size'] : ($this->multiple ? 5 : 1);
 
-        parent::__construct($var['form_caption'], $key, $object->getVar($key, 'e'), $size, $this->multiple);
+		parent::__construct($var['form_caption'], $key, $object->getVar($key, 'e'), $size, $this->multiple);
 
-        // Adding the options inside this SelectBox
-        // If the custom method is not from a module, than it's from the core
-        $control = $object->getControl($key);
+		// Adding the options inside this SelectBox
+		// If the custom method is not from a module, than it's from the core
+		$control = $object->getControl($key);
 
-        global $xoopsDB;
-        $ret   = array();
-        $limit = $start = 0;
-        $sql   = 'SELECT uid, uname FROM ' . $xoopsDB->prefix('users');
-        $sql   .= ' ORDER BY uname ASC';
+		global $xoopsDB;
+		$ret   = array();
+		$limit = $start = 0;
+		$sql   = 'SELECT uid, uname FROM ' . $xoopsDB->prefix('users');
+		$sql   .= ' ORDER BY uname ASC';
 
-        $result = $xoopsDB->query($sql);
-        if ($result) {
-            while ($myrow = $xoopsDB->fetchArray($result)) {
-                $uArray[$myrow['uid']] = $myrow['uname'];
-            }
-        }
-        $this->addOptionArray($uArray);
-    }
+		$result = $xoopsDB->query($sql);
+		if ($result) {
+			while ($myrow = $xoopsDB->fetchArray($result)) {
+				$uArray[$myrow['uid']] = $myrow['uname'];
+			}
+		}
+		$this->addOptionArray($uArray);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
         global $xoopsDB;
         $ret   = array();
         $limit = $start = 0;
-        $sql   = 'SELECT uid, uname FROM ' . $xoopsDB->prefix('users');
+        $sql   = 'SELECT uid, uname FROM '.$xoopsDB->prefix('users');
         $sql   .= ' ORDER BY uname ASC';
 
         $result = $xoopsDB->query($sql);

class/form/elements/smartformuploadelement.php

Indentation +18 added lines, -18 removed lines

@@ -11,25 +11,25 @@
  */
 class SmartFormUploadElement extends XoopsFormFile
 {
-    /**
-     * @param $object
-     * @param $key
-     */
-    public function SmartFormFileElement($object, $key)
-    {
-        parent::__construct(_CO_SOBJECT_UPLOAD, $key, isset($object->vars[$key]['form_maxfilesize']) ? $object->vars[$key]['form_maxfilesize'] : 0);
-        $this->setExtra(' size=50');
-    }
+	/**
+	 * @param $object
+	 * @param $key
+	 */
+	public function SmartFormFileElement($object, $key)
+	{
+		parent::__construct(_CO_SOBJECT_UPLOAD, $key, isset($object->vars[$key]['form_maxfilesize']) ? $object->vars[$key]['form_maxfilesize'] : 0);
+		$this->setExtra(' size=50');
+	}
 
-    /**
-     * prepare HTML for output
-     *
-     * @return string HTML
-     */
-    public function render()
-    {
-        return "<input type='hidden' name='MAX_FILE_SIZE' value='" . $this->getMaxFileSize() . "'>
+	/**
+	 * prepare HTML for output
+	 *
+	 * @return string HTML
+	 */
+	public function render()
+	{
+		return "<input type='hidden' name='MAX_FILE_SIZE' value='" . $this->getMaxFileSize() . "'>
                 <input type='file' name='" . $this->getName() . "' id='" . $this->getName() . "'" . $this->getExtra() . ">
                 <input type='hidden' name='smart_upload_file[]' id='smart_upload_file[]' value='" . $this->getName() . "'>";
-    }
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -28,8 +28,8 @@
      */
     public function render()
     {
-        return "<input type='hidden' name='MAX_FILE_SIZE' value='" . $this->getMaxFileSize() . "'>
-                <input type='file' name='" . $this->getName() . "' id='" . $this->getName() . "'" . $this->getExtra() . ">
-                <input type='hidden' name='smart_upload_file[]' id='smart_upload_file[]' value='" . $this->getName() . "'>";
+        return "<input type='hidden' name='MAX_FILE_SIZE' value='".$this->getMaxFileSize()."'>
+                <input type='file' name='" . $this->getName()."' id='".$this->getName()."'".$this->getExtra().">
+                <input type='hidden' name='smart_upload_file[]' id='smart_upload_file[]' value='" . $this->getName()."'>";
     }
 }

class/form/elements/smartformfileelement.php

Indentation +28 added lines, -28 removed lines

@@ -5,38 +5,38 @@
  */
 class SmartFormFileElement extends XoopsFormFile
 {
-    public $object;
-    public $key;
+	public $object;
+	public $key;
 
-    /**
-     * SmartFormFileElement constructor.
-     * @param string $object
-     * @param string $key
-     */
-    public function __construct($object, $key)
-    {
-        $this->object = $object;
-        $this->key    = $key;
-        parent::__construct($object->vars[$key]['form_caption'], $key, isset($object->vars[$key]['form_maxfilesize']) ? $object->vars[$key]['form_maxfilesize'] : 0);
-        $this->setExtra(' size=50');
-    }
+	/**
+	 * SmartFormFileElement constructor.
+	 * @param string $object
+	 * @param string $key
+	 */
+	public function __construct($object, $key)
+	{
+		$this->object = $object;
+		$this->key    = $key;
+		parent::__construct($object->vars[$key]['form_caption'], $key, isset($object->vars[$key]['form_maxfilesize']) ? $object->vars[$key]['form_maxfilesize'] : 0);
+		$this->setExtra(' size=50');
+	}
 
-    /**
-     * prepare HTML for output
-     *
-     * @return string HTML
-     */
-    public function render()
-    {
-        $ret = '';
-        if ($this->object->getVar($this->key) !== '') {
-            $ret .= '<div>' . _CO_SOBJECT_CURRENT_FILE . $this->object->getVar($this->key) . '</div>';
-        }
+	/**
+	 * prepare HTML for output
+	 *
+	 * @return string HTML
+	 */
+	public function render()
+	{
+		$ret = '';
+		if ($this->object->getVar($this->key) !== '') {
+			$ret .= '<div>' . _CO_SOBJECT_CURRENT_FILE . $this->object->getVar($this->key) . '</div>';
+		}
 
-        $ret .= "<div><input type='hidden' name='MAX_FILE_SIZE' value='" . $this->getMaxFileSize() . "'>
+		$ret .= "<div><input type='hidden' name='MAX_FILE_SIZE' value='" . $this->getMaxFileSize() . "'>
                 <input type='file' name='" . $this->getName() . "' id='" . $this->getName() . "'" . $this->getExtra() . ">
                 <input type='hidden' name='smart_upload_file[]' id='smart_upload_file[]' value='" . $this->getName() . "'></div>";
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -30,12 +30,12 @@
     {
         $ret = '';
         if ($this->object->getVar($this->key) !== '') {
-            $ret .= '<div>' . _CO_SOBJECT_CURRENT_FILE . $this->object->getVar($this->key) . '</div>';
+            $ret .= '<div>'._CO_SOBJECT_CURRENT_FILE.$this->object->getVar($this->key).'</div>';
         }
 
-        $ret .= "<div><input type='hidden' name='MAX_FILE_SIZE' value='" . $this->getMaxFileSize() . "'>
-                <input type='file' name='" . $this->getName() . "' id='" . $this->getName() . "'" . $this->getExtra() . ">
-                <input type='hidden' name='smart_upload_file[]' id='smart_upload_file[]' value='" . $this->getName() . "'></div>";
+        $ret .= "<div><input type='hidden' name='MAX_FILE_SIZE' value='".$this->getMaxFileSize()."'>
+                <input type='file' name='" . $this->getName()."' id='".$this->getName()."'".$this->getExtra().">
+                <input type='hidden' name='smart_upload_file[]' id='smart_upload_file[]' value='" . $this->getName()."'></div>";
 
         return $ret;
     }

class/form/elements/smartformselectelement.php

Indentation +49 added lines, -49 removed lines

@@ -11,59 +11,59 @@
  */
 class SmartFormSelectElement extends XoopsFormSelect
 {
-    public $multiple = false;
+	public $multiple = false;
 
-    /**
-     * SmartFormSelectElement constructor.
-     * @param string $object
-     * @param string $key
-     */
-    public function __construct($object, $key)
-    {
-        $var  = $object->vars[$key];
-        $size = isset($var['size']) ? $var['size'] : ($this->multiple ? 5 : 1);
+	/**
+	 * SmartFormSelectElement constructor.
+	 * @param string $object
+	 * @param string $key
+	 */
+	public function __construct($object, $key)
+	{
+		$var  = $object->vars[$key];
+		$size = isset($var['size']) ? $var['size'] : ($this->multiple ? 5 : 1);
 
-        // Adding the options inside this SelectBox
-        // If the custom method is not from a module, than it's from the core
-        $control = $object->getControl($key);
+		// Adding the options inside this SelectBox
+		// If the custom method is not from a module, than it's from the core
+		$control = $object->getControl($key);
 
-        $value = isset($control['value']) ? $control['value'] : $object->getVar($key, 'e');
+		$value = isset($control['value']) ? $control['value'] : $object->getVar($key, 'e');
 
-        parent::__construct($var['form_caption'], $key, $value, $size, $this->multiple);
+		parent::__construct($var['form_caption'], $key, $value, $size, $this->multiple);
 
-        if (isset($control['options'])) {
-            $this->addOptionArray($control['options']);
-        } else {
-            // let's find if the method we need to call comes from an already defined object
-            if (isset($control['object'])) {
-                if (method_exists($control['object'], $control['method'])) {
-                    if ($option_array = $control['object']->{$control['method']}()) {
-                        // Adding the options array to the XoopsFormSelect
-                        $this->addOptionArray($option_array);
-                    }
-                }
-            } else {
-                // finding the itemHandler; if none, let's take the itemHandler of the $object
-                if (isset($control['itemHandler'])) {
-                    if (!$control['module']) {
-                        // Creating the specified core object handler
-                        $controlHandler = xoops_getHandler($control['itemHandler']);
-                    } else {
-                        $controlHandler = xoops_getModuleHandler($control['itemHandler'], $control['module']);
-                    }
-                } else {
-                    $controlHandler = $object->handler;
-                }
+		if (isset($control['options'])) {
+			$this->addOptionArray($control['options']);
+		} else {
+			// let's find if the method we need to call comes from an already defined object
+			if (isset($control['object'])) {
+				if (method_exists($control['object'], $control['method'])) {
+					if ($option_array = $control['object']->{$control['method']}()) {
+						// Adding the options array to the XoopsFormSelect
+						$this->addOptionArray($option_array);
+					}
+				}
+			} else {
+				// finding the itemHandler; if none, let's take the itemHandler of the $object
+				if (isset($control['itemHandler'])) {
+					if (!$control['module']) {
+						// Creating the specified core object handler
+						$controlHandler = xoops_getHandler($control['itemHandler']);
+					} else {
+						$controlHandler = xoops_getModuleHandler($control['itemHandler'], $control['module']);
+					}
+				} else {
+					$controlHandler = $object->handler;
+				}
 
-                // Checking if the specified method exists
-                if (method_exists($controlHandler, $control['method'])) {
-                    // TODO: How could I pass the parameters in the following call ...
-                    if ($option_array = $controlHandler->{$control['method']}()) {
-                        // Adding the options array to the XoopsFormSelect
-                        $this->addOptionArray($option_array);
-                    }
-                }
-            }
-        }
-    }
+				// Checking if the specified method exists
+				if (method_exists($controlHandler, $control['method'])) {
+					// TODO: How could I pass the parameters in the following call ...
+					if ($option_array = $controlHandler->{$control['method']}()) {
+						// Adding the options array to the XoopsFormSelect
+						$this->addOptionArray($option_array);
+					}
+				}
+			}
+		}
+	}
 }

class/form/elements/smartformimageelement.php

Indentation +30 added lines, -30 removed lines

@@ -11,39 +11,39 @@
  */
 class SmartFormImageElement extends XoopsFormElementTray
 {
-    /**
-     * SmartFormImageElement constructor.
-     * @param string $object
-     * @param string $key
-     */
-    public function __construct($object, $key)
-    {
-        $var             = $object->vars[$key];
-        $object_imageurl = $object->getImageDir();
-        parent::__construct($var['form_caption'], ' ');
+	/**
+	 * SmartFormImageElement constructor.
+	 * @param string $object
+	 * @param string $key
+	 */
+	public function __construct($object, $key)
+	{
+		$var             = $object->vars[$key];
+		$object_imageurl = $object->getImageDir();
+		parent::__construct($var['form_caption'], ' ');
 
-        $objectArray['image'] = str_replace('{XOOPS_URL}', XOOPS_URL, $objectArray['image']);
+		$objectArray['image'] = str_replace('{XOOPS_URL}', XOOPS_URL, $objectArray['image']);
 
-        if ($object->getVar($key) !== ''
-            && (0 === strpos($object->getVar($key), 'http')
-                || 0 === strpos($object->getVar($key), '{XOOPS_URL}'))) {
-            $this->addElement(new XoopsFormLabel('', "<img src='" . str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar($key)) . "' alt=''><br><br>"));
-        } elseif ($object->getVar($key) !== '') {
-            $this->addElement(new XoopsFormLabel('', "<img src='" . $object_imageurl . $object->getVar($key) . "' alt=''><br><br>"));
-        }
+		if ($object->getVar($key) !== ''
+			&& (0 === strpos($object->getVar($key), 'http')
+				|| 0 === strpos($object->getVar($key), '{XOOPS_URL}'))) {
+			$this->addElement(new XoopsFormLabel('', "<img src='" . str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar($key)) . "' alt=''><br><br>"));
+		} elseif ($object->getVar($key) !== '') {
+			$this->addElement(new XoopsFormLabel('', "<img src='" . $object_imageurl . $object->getVar($key) . "' alt=''><br><br>"));
+		}
 
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformfileuploadelement.php';
-        $this->addElement(new SmartFormFileUploadElement($object, $key));
+		require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformfileuploadelement.php';
+		$this->addElement(new SmartFormFileUploadElement($object, $key));
 
-        $this->addElement(new XoopsFormLabel('<div style="height: 10px; padding-top: 8px; font-size: 80%;">' . _CO_SOBJECT_URL_FILE_DSC . '</div>', ''));
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformtextelement.php';
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformcheckelement.php';
+		$this->addElement(new XoopsFormLabel('<div style="height: 10px; padding-top: 8px; font-size: 80%;">' . _CO_SOBJECT_URL_FILE_DSC . '</div>', ''));
+		require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformtextelement.php';
+		require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformcheckelement.php';
 
-        $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
-        $this->addElement(new SmartFormTextElement($object, 'url_' . $key));
-        $this->addElement(new XoopsFormLabel('', '<br><br>'));
-        $delete_check = new SmartFormCheckElement('', 'delete_' . $key);
-        $delete_check->addOption(1, '<span style="color:red;">' . _CO_SOBJECT_DELETE . '</span>');
-        $this->addElement($delete_check);
-    }
+		$this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
+		$this->addElement(new SmartFormTextElement($object, 'url_' . $key));
+		$this->addElement(new XoopsFormLabel('', '<br><br>'));
+		$delete_check = new SmartFormCheckElement('', 'delete_' . $key);
+		$delete_check->addOption(1, '<span style="color:red;">' . _CO_SOBJECT_DELETE . '</span>');
+		$this->addElement($delete_check);
+	}
 }

Spacing +10 added lines, -10 removed lines

@@ -27,23 +27,23 @@
         if ($object->getVar($key) !== ''
             && (0 === strpos($object->getVar($key), 'http')
                 || 0 === strpos($object->getVar($key), '{XOOPS_URL}'))) {
-            $this->addElement(new XoopsFormLabel('', "<img src='" . str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar($key)) . "' alt=''><br><br>"));
+            $this->addElement(new XoopsFormLabel('', "<img src='".str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar($key))."' alt=''><br><br>"));
         } elseif ($object->getVar($key) !== '') {
-            $this->addElement(new XoopsFormLabel('', "<img src='" . $object_imageurl . $object->getVar($key) . "' alt=''><br><br>"));
+            $this->addElement(new XoopsFormLabel('', "<img src='".$object_imageurl.$object->getVar($key)."' alt=''><br><br>"));
         }
 
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformfileuploadelement.php';
+        require_once SMARTOBJECT_ROOT_PATH.'class/form/elements/smartformfileuploadelement.php';
         $this->addElement(new SmartFormFileUploadElement($object, $key));
 
-        $this->addElement(new XoopsFormLabel('<div style="height: 10px; padding-top: 8px; font-size: 80%;">' . _CO_SOBJECT_URL_FILE_DSC . '</div>', ''));
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformtextelement.php';
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformcheckelement.php';
+        $this->addElement(new XoopsFormLabel('<div style="height: 10px; padding-top: 8px; font-size: 80%;">'._CO_SOBJECT_URL_FILE_DSC.'</div>', ''));
+        require_once SMARTOBJECT_ROOT_PATH.'class/form/elements/smartformtextelement.php';
+        require_once SMARTOBJECT_ROOT_PATH.'class/form/elements/smartformcheckelement.php';
 
-        $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
-        $this->addElement(new SmartFormTextElement($object, 'url_' . $key));
+        $this->addElement(new XoopsFormLabel('', '<br>'._CO_SOBJECT_URL_FILE));
+        $this->addElement(new SmartFormTextElement($object, 'url_'.$key));
         $this->addElement(new XoopsFormLabel('', '<br><br>'));
-        $delete_check = new SmartFormCheckElement('', 'delete_' . $key);
-        $delete_check->addOption(1, '<span style="color:red;">' . _CO_SOBJECT_DELETE . '</span>');
+        $delete_check = new SmartFormCheckElement('', 'delete_'.$key);
+        $delete_check->addOption(1, '<span style="color:red;">'._CO_SOBJECT_DELETE.'</span>');
         $this->addElement($delete_check);
     }
 }

class/form/elements/smartformcheckelement.php

Indentation +38 added lines, -38 removed lines

@@ -7,41 +7,41 @@ 
  */
 class SmartFormCheckElement extends XoopsFormCheckBox
 {
-    /**
-     *
-     * /**
-     * prepare HTML for output
-     *
-     * @return string
-     */
-    public function render()
-    {
-        $ret = '';
-        if (count($this->getOptions()) > 1 && substr($this->getName(), -2, 2) !== '[]') {
-            $newname = $this->getName() . '[]';
-            $this->setName($newname);
-        }
-        foreach ($this->getOptions() as $value => $name) {
-            $ret .= "<input type='checkbox' name='" . $this->getName() . "' value='" . $value . "'";
-            if (count($this->getValue()) > 0 && in_array($value, $this->getValue())) {
-                $ret .= ' checked';
-            }
-            $ret .= $this->getExtra() . '>' . $name . '<br>';
-        }
+	/**
+	 *
+	 * /**
+	 * prepare HTML for output
+	 *
+	 * @return string
+	 */
+	public function render()
+	{
+		$ret = '';
+		if (count($this->getOptions()) > 1 && substr($this->getName(), -2, 2) !== '[]') {
+			$newname = $this->getName() . '[]';
+			$this->setName($newname);
+		}
+		foreach ($this->getOptions() as $value => $name) {
+			$ret .= "<input type='checkbox' name='" . $this->getName() . "' value='" . $value . "'";
+			if (count($this->getValue()) > 0 && in_array($value, $this->getValue())) {
+				$ret .= ' checked';
+			}
+			$ret .= $this->getExtra() . '>' . $name . '<br>';
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 
-    /**
-     * @return string
-     */
-    public function renderValidationJS()
-    {
-        $js .= 'var hasSelections = false;';
-        //sometimes, there is an implicit '[]', sometimes not
-        $eltname = $this->getName();
-        if (strpos($eltname, '[') === false) {
-            $js .= "for (var i = 0; i < myform['{$eltname}[]'].length; i++) {
+	/**
+	 * @return string
+	 */
+	public function renderValidationJS()
+	{
+		$js .= 'var hasSelections = false;';
+		//sometimes, there is an implicit '[]', sometimes not
+		$eltname = $this->getName();
+		if (strpos($eltname, '[') === false) {
+			$js .= "for (var i = 0; i < myform['{$eltname}[]'].length; i++) {
                 if (myform['{$eltname}[]'][i].checked) {
                     hasSelections = true;
                 }
@@ -49,8 +49,8 @@ 
             }
             if (hasSelections === false) {
                 window.alert(\"{$eltmsg}\"); myform['{$eltname}[]'][0].focus(); return false; }\n";
-        } else {
-            $js .= "for (var i = 0; i < myform['" . $eltname . "'].length; i++) {
+		} else {
+			$js .= "for (var i = 0; i < myform['" . $eltname . "'].length; i++) {
                 if (myform['{$eltname}'][i].checked) {
                     hasSelections = true;
                 }
@@ -58,8 +58,8 @@ 
             }
             if (hasSelections === false) {
                 window.alert(\"{$eltmsg}\"); myform['{$eltname}'][0].focus(); return false; }\n";
-        }
+		}
 
-        return $js;
-    }
+		return $js;
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -18,15 +18,15 @@ 
     {
         $ret = '';
         if (count($this->getOptions()) > 1 && substr($this->getName(), -2, 2) !== '[]') {
-            $newname = $this->getName() . '[]';
+            $newname = $this->getName().'[]';
             $this->setName($newname);
         }
         foreach ($this->getOptions() as $value => $name) {
-            $ret .= "<input type='checkbox' name='" . $this->getName() . "' value='" . $value . "'";
+            $ret .= "<input type='checkbox' name='".$this->getName()."' value='".$value."'";
             if (count($this->getValue()) > 0 && in_array($value, $this->getValue())) {
                 $ret .= ' checked';
             }
-            $ret .= $this->getExtra() . '>' . $name . '<br>';
+            $ret .= $this->getExtra().'>'.$name.'<br>';
         }
 
         return $ret;
@@ -50,7 +50,7 @@ 
             if (hasSelections === false) {
                 window.alert(\"{$eltmsg}\"); myform['{$eltname}[]'][0].focus(); return false; }\n";
         } else {
-            $js .= "for (var i = 0; i < myform['" . $eltname . "'].length; i++) {
+            $js .= "for (var i = 0; i < myform['".$eltname."'].length; i++) {
                 if (myform['{$eltname}'][i].checked) {
                     hasSelections = true;
                 }

class/form/elements/smartformparentcategoryelement.php

Indentation +51 added lines, -51 removed lines

@@ -11,59 +11,59 @@
  */
 class SmartFormParentCategoryElement extends XoopsFormSelect
 {
-    /**
-     * SmartFormParentcategoryElement constructor.
-     * @param string $object
-     * @param string $key
-     */
-    public function __construct($object, $key)
-    {
-        $addNoParent = isset($object->controls[$key]['addNoParent']) ? $object->controls[$key]['addNoParent'] : true;
-        $criteria    = new CriteriaCompo();
-        $criteria->setSort('weight, name');
-        $categoryHandler = xoops_getModuleHandler('category', $object->handler->_moduleName);
-        $categories      = $categoryHandler->getObjects($criteria);
+	/**
+	 * SmartFormParentcategoryElement constructor.
+	 * @param string $object
+	 * @param string $key
+	 */
+	public function __construct($object, $key)
+	{
+		$addNoParent = isset($object->controls[$key]['addNoParent']) ? $object->controls[$key]['addNoParent'] : true;
+		$criteria    = new CriteriaCompo();
+		$criteria->setSort('weight, name');
+		$categoryHandler = xoops_getModuleHandler('category', $object->handler->_moduleName);
+		$categories      = $categoryHandler->getObjects($criteria);
 
-        require_once XOOPS_ROOT_PATH . '/class/tree.php';
-        $mytree = new XoopsObjectTree($categories, 'categoryid', 'parentid');
-        parent::__construct($object->vars[$key]['form_caption'], $key, $object->getVar($key, 'e'));
+		require_once XOOPS_ROOT_PATH . '/class/tree.php';
+		$mytree = new XoopsObjectTree($categories, 'categoryid', 'parentid');
+		parent::__construct($object->vars[$key]['form_caption'], $key, $object->getVar($key, 'e'));
 
-        $ret     = array();
-        $options = $this->getOptionArray($mytree, 'name', 0, '', $ret);
-        if ($addNoParent) {
-            $newOptions = array('0' => '----');
-            foreach ($options as $k => $v) {
-                $newOptions[$k] = $v;
-            }
-            $options = $newOptions;
-        }
-        $this->addOptionArray($options);
-    }
+		$ret     = array();
+		$options = $this->getOptionArray($mytree, 'name', 0, '', $ret);
+		if ($addNoParent) {
+			$newOptions = array('0' => '----');
+			foreach ($options as $k => $v) {
+				$newOptions[$k] = $v;
+			}
+			$options = $newOptions;
+		}
+		$this->addOptionArray($options);
+	}
 
-    /**
-     * Get options for a category select with hierarchy (recursive)
-     *
-     * @param XoopsObjectTree $tree
-     * @param string          $fieldName
-     * @param int             $key
-     * @param string          $prefix_curr
-     * @param array           $ret
-     *
-     * @return array
-     */
-    public function getOptionArray($tree, $fieldName, $key, $prefix_curr = '', &$ret)
-    {
-        if ($key > 0) {
-            $value       = $tree->tree[$key]['obj']->getVar($tree->_myId);
-            $ret[$key]   = $prefix_curr . $tree->tree[$key]['obj']->getVar($fieldName);
-            $prefix_curr .= '-';
-        }
-        if (isset($tree->tree[$key]['child']) && !empty($tree->tree[$key]['child'])) {
-            foreach ($tree->tree[$key]['child'] as $childkey) {
-                $this->getOptionArray($tree, $fieldName, $childkey, $prefix_curr, $ret);
-            }
-        }
+	/**
+	 * Get options for a category select with hierarchy (recursive)
+	 *
+	 * @param XoopsObjectTree $tree
+	 * @param string          $fieldName
+	 * @param int             $key
+	 * @param string          $prefix_curr
+	 * @param array           $ret
+	 *
+	 * @return array
+	 */
+	public function getOptionArray($tree, $fieldName, $key, $prefix_curr = '', &$ret)
+	{
+		if ($key > 0) {
+			$value       = $tree->tree[$key]['obj']->getVar($tree->_myId);
+			$ret[$key]   = $prefix_curr . $tree->tree[$key]['obj']->getVar($fieldName);
+			$prefix_curr .= '-';
+		}
+		if (isset($tree->tree[$key]['child']) && !empty($tree->tree[$key]['child'])) {
+			foreach ($tree->tree[$key]['child'] as $childkey) {
+				$this->getOptionArray($tree, $fieldName, $childkey, $prefix_curr, $ret);
+			}
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
         $categoryHandler = xoops_getModuleHandler('category', $object->handler->_moduleName);
         $categories      = $categoryHandler->getObjects($criteria);
 
-        require_once XOOPS_ROOT_PATH . '/class/tree.php';
+        require_once XOOPS_ROOT_PATH.'/class/tree.php';
         $mytree = new XoopsObjectTree($categories, 'categoryid', 'parentid');
         parent::__construct($object->vars[$key]['form_caption'], $key, $object->getVar($key, 'e'));
 
@@ -55,7 +55,7 @@ 
     {
         if ($key > 0) {
             $value       = $tree->tree[$key]['obj']->getVar($tree->_myId);
-            $ret[$key]   = $prefix_curr . $tree->tree[$key]['obj']->getVar($fieldName);
+            $ret[$key]   = $prefix_curr.$tree->tree[$key]['obj']->getVar($fieldName);
             $prefix_curr .= '-';
         }
         if (isset($tree->tree[$key]['child']) && !empty($tree->tree[$key]['child'])) {

class/form/elements/smartformhidden.php

Indentation +15 added lines, -15 removed lines

@@ -24,20 +24,20 @@
  */
 class SmartFormHidden extends XoopsFormHidden
 {
-    /**
-     * @return string
-     */
-    public function render()
-    {
-        if (is_array($this->getValue())) {
-            $ret = '';
-            foreach ($this->getValue() as $value) {
-                $ret .= "<input type='hidden' name='" . $this->getName() . "[]' id='" . $this->getName() . "' value='" . $value . "'>\n";
-            }
-        } else {
-            $ret = "<input type='hidden' name='" . $this->getName() . "' id='" . $this->getName() . "' value='" . $this->getValue() . "'>";
-        }
+	/**
+	 * @return string
+	 */
+	public function render()
+	{
+		if (is_array($this->getValue())) {
+			$ret = '';
+			foreach ($this->getValue() as $value) {
+				$ret .= "<input type='hidden' name='" . $this->getName() . "[]' id='" . $this->getName() . "' value='" . $value . "'>\n";
+			}
+		} else {
+			$ret = "<input type='hidden' name='" . $this->getName() . "' id='" . $this->getName() . "' value='" . $this->getValue() . "'>";
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -32,10 +32,10 @@
         if (is_array($this->getValue())) {
             $ret = '';
             foreach ($this->getValue() as $value) {
-                $ret .= "<input type='hidden' name='" . $this->getName() . "[]' id='" . $this->getName() . "' value='" . $value . "'>\n";
+                $ret .= "<input type='hidden' name='".$this->getName()."[]' id='".$this->getName()."' value='".$value."'>\n";
             }
         } else {
-            $ret = "<input type='hidden' name='" . $this->getName() . "' id='" . $this->getName() . "' value='" . $this->getValue() . "'>";
+            $ret = "<input type='hidden' name='".$this->getName()."' id='".$this->getName()."' value='".$this->getValue()."'>";
         }
 
         return $ret;

class/form/elements/smartformradioelement.php

Indentation +46 added lines, -46 removed lines

@@ -11,54 +11,54 @@
  */
 class SmartFormRadioElement extends XoopsFormRadio
 {
-    /**
-     * SmartFormRadioElement constructor.
-     * @param string $object
-     * @param string $key
-     */
-    public function __construct($object, $key)
-    {
-        $var = $object->vars[$key];
+	/**
+	 * SmartFormRadioElement constructor.
+	 * @param string $object
+	 * @param string $key
+	 */
+	public function __construct($object, $key)
+	{
+		$var = $object->vars[$key];
 
-        parent::__construct($var['form_caption'], $key, $object->getVar($key, 'e'));
+		parent::__construct($var['form_caption'], $key, $object->getVar($key, 'e'));
 
-        // Adding the options inside this SelectBox
-        // If the custom method is not from a module, than it's from the core
-        $control = $object->getControl($key);
+		// Adding the options inside this SelectBox
+		// If the custom method is not from a module, than it's from the core
+		$control = $object->getControl($key);
 
-        if (isset($control['options'])) {
-            $this->addOptionArray($control['options']);
-        } else {
-            // let's find if the method we need to call comes from an already defined object
-            if (isset($control['object'])) {
-                if (method_exists($control['object'], $control['method'])) {
-                    if ($option_array = $control['object']->{$control['method']}()) {
-                        // Adding the options array to the XoopsFormSelect
-                        $this->addOptionArray($option_array);
-                    }
-                }
-            } else {
-                // finding the itemHandler; if none, let's take the itemHandler of the $object
-                if (isset($control['itemHandler'])) {
-                    if (!$control['module']) {
-                        // Creating the specified core object handler
-                        $controlHandler = xoops_getHandler($control['itemHandler']);
-                    } else {
-                        $controlHandler = xoops_getModuleHandler($control['itemHandler'], $control['module']);
-                    }
-                } else {
-                    $controlHandler = $object->handler;
-                }
+		if (isset($control['options'])) {
+			$this->addOptionArray($control['options']);
+		} else {
+			// let's find if the method we need to call comes from an already defined object
+			if (isset($control['object'])) {
+				if (method_exists($control['object'], $control['method'])) {
+					if ($option_array = $control['object']->{$control['method']}()) {
+						// Adding the options array to the XoopsFormSelect
+						$this->addOptionArray($option_array);
+					}
+				}
+			} else {
+				// finding the itemHandler; if none, let's take the itemHandler of the $object
+				if (isset($control['itemHandler'])) {
+					if (!$control['module']) {
+						// Creating the specified core object handler
+						$controlHandler = xoops_getHandler($control['itemHandler']);
+					} else {
+						$controlHandler = xoops_getModuleHandler($control['itemHandler'], $control['module']);
+					}
+				} else {
+					$controlHandler = $object->handler;
+				}
 
-                // Checking if the specified method exists
-                if (method_exists($controlHandler, $control['method'])) {
-                    // TODO: How could I pass the parameters in the following call ...
-                    if ($option_array = $controlHandler->{$control['method']}()) {
-                        // Adding the options array to the XoopsFormSelect
-                        $this->addOptionArray($option_array);
-                    }
-                }
-            }
-        }
-    }
+				// Checking if the specified method exists
+				if (method_exists($controlHandler, $control['method'])) {
+					// TODO: How could I pass the parameters in the following call ...
+					if ($option_array = $controlHandler->{$control['method']}()) {
+						// Adding the options array to the XoopsFormSelect
+						$this->addOptionArray($option_array);
+					}
+				}
+			}
+		}
+	}
 }