mambax7 / smartobject

class/form/elements/smartformsection.php

Indentation +34 added lines, -34 removed lines

@@ -6,41 +6,41 @@
  */
 class SmartFormSection extends XoopsFormElement
 {
-    /**
-     * Text
-     * @var string
-     * @access  private
-     */
-    public $_value;
+	/**
+	 * Text
+	 * @var string
+	 * @access  private
+	 */
+	public $_value;
 
-    /**
-     * SmartFormSection constructor.
-     * @param      $sectionname
-     * @param bool $value
-     */
-    public function __construct($sectionname, $value = false)
-    {
-        $this->setName($sectionname);
-        $this->_value = $value;
-    }
+	/**
+	 * SmartFormSection constructor.
+	 * @param      $sectionname
+	 * @param bool $value
+	 */
+	public function __construct($sectionname, $value = false)
+	{
+		$this->setName($sectionname);
+		$this->_value = $value;
+	}
 
-    /**
-     * Get the text
-     *
-     * @return string
-     */
-    public function getValue()
-    {
-        return $this->_value;
-    }
+	/**
+	 * Get the text
+	 *
+	 * @return string
+	 */
+	public function getValue()
+	{
+		return $this->_value;
+	}
 
-    /**
-     * Prepare HTML for output
-     *
-     * @return string
-     */
-    public function render()
-    {
-        return $this->getValue();
-    }
+	/**
+	 * Prepare HTML for output
+	 *
+	 * @return string
+	 */
+	public function render()
+	{
+		return $this->getValue();
+	}
 }

class/form/elements/smartformemailelement.php

Indentation +9 added lines, -9 removed lines

@@ -35,14 +35,14 @@
  */
 class SmartFormSelectElement extends XoopsFormElement
 {
-    /**
-     * @param  bool $withtags
-     * @return string
-     */
-    public function renderValidationJS($withtags = true)
-    {
-        $js = '';
+	/**
+	 * @param  bool $withtags
+	 * @return string
+	 */
+	public function renderValidationJS($withtags = true)
+	{
+		$js = '';
 
-        return $js;
-    }
+		return $js;
+	}
 }

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='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='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;
+	}
 }

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;
+	}
 }

class/form/elements/smartformhidden.php

Indentation +15 added lines, -15 removed lines

@@ -36,20 +36,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;
+	}
 }

class/form/elements/smartformdate_timeelement.php

Indentation +9 added lines, -9 removed lines

@@ -11,13 +11,13 @@
  */
 class SmartFormDate_timeElement extends XoopsFormDateTime
 {
-    /**
-     * SmartFormDate_timeElement constructor.
-     * @param mixed $object
-     * @param mixed $key
-     */
-    public function __construct($object, $key)
-    {
-        parent::__construct($object->vars[$key]['form_caption'], $key, 15, $object->getVar($key, 'e'));
-    }
+	/**
+	 * SmartFormDate_timeElement constructor.
+	 * @param mixed $object
+	 * @param mixed $key
+	 */
+	public function __construct($object, $key)
+	{
+		parent::__construct($object->vars[$key]['form_caption'], $key, 15, $object->getVar($key, 'e'));
+	}
 }

class/form/elements/smartformsectionclose.php

Indentation +34 added lines, -34 removed lines

@@ -6,41 +6,41 @@
  */
 class SmartFormSectionClose extends XoopsFormElement
 {
-    /**
-     * Text
-     * @var string
-     * @access  private
-     */
-    public $_value;
+	/**
+	 * Text
+	 * @var string
+	 * @access  private
+	 */
+	public $_value;
 
-    /**
-     * SmartFormSectionClose constructor.
-     * @param      $sectionname
-     * @param bool $value
-     */
-    public function __construct($sectionname, $value = false)
-    {
-        $this->setName($sectionname);
-        $this->_value = $value;
-    }
+	/**
+	 * SmartFormSectionClose constructor.
+	 * @param      $sectionname
+	 * @param bool $value
+	 */
+	public function __construct($sectionname, $value = false)
+	{
+		$this->setName($sectionname);
+		$this->_value = $value;
+	}
 
-    /**
-     * Get the text
-     *
-     * @return string
-     */
-    public function getValue()
-    {
-        return $this->_value;
-    }
+	/**
+	 * Get the text
+	 *
+	 * @return string
+	 */
+	public function getValue()
+	{
+		return $this->_value;
+	}
 
-    /**
-     * Prepare HTML for output
-     *
-     * @return string
-     */
-    public function render()
-    {
-        return $this->getValue();
-    }
+	/**
+	 * Prepare HTML for output
+	 *
+	 * @return string
+	 */
+	public function render()
+	{
+		return $this->getValue();
+	}
 }

class/form/elements/smartformimageelement.php

Indentation +28 added lines, -28 removed lines

@@ -11,37 +11,37 @@
  */
 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/>"));
+		}
 
-        include_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformfileuploadelement.php';
-        $this->addElement(new SmartFormFileUploadElement($object, $key));
+		include_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>', ''));
-        include_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformtextelement.php';
-        include_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>', ''));
+		include_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformtextelement.php';
+		include_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);
+	}
 }

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);
+					}
+				}
+			}
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
                         $controlHandler = xoops_getModuleHandler($control['itemHandler'], $control['module']);
                     }
                 } else {
-                    $controlHandler =& $object->handler;
+                    $controlHandler = & $object->handler;
                 }
 
                 // Checking if the specified method exists