mambax7 / smartobject

class/form/elements/smartautocompleteelement.php

Indentation +26 added lines, -26 removed lines

@@ -11,38 +11,38 @@
  */
 class SmartAutocompleteElement extends XoopsFormText
 {
-    public $_include_file;
+	public $_include_file;
 
-    /**
-     * SmartAutocompleteElement constructor.
-     * @param string $caption
-     * @param string $name
-     * @param int    $include_file
-     * @param int    $size
-     * @param string $maxlength
-     * @param string $value
-     */
-    public function __construct($caption, $name, $include_file, $size, $maxlength, $value = '')
-    {
-        $this->_include_file = $include_file;
-        parent::__construct($caption, $name, $size, $maxlength, $value);
-    }
+	/**
+	 * SmartAutocompleteElement constructor.
+	 * @param string $caption
+	 * @param string $name
+	 * @param int    $include_file
+	 * @param int    $size
+	 * @param string $maxlength
+	 * @param string $value
+	 */
+	public function __construct($caption, $name, $include_file, $size, $maxlength, $value = '')
+	{
+		$this->_include_file = $include_file;
+		parent::__construct($caption, $name, $size, $maxlength, $value);
+	}
 
-    /**
-     * Prepare HTML for output
-     *
-     * @return string HTML
-     */
-    public function render()
-    {
-        $ret = "<input type='text' name='" . $this->getName() . "' id='" . $this->getName() . "' size='" . $this->getSize() . "' maxlength='" . $this->getMaxlength() . "' value='" . $this->getValue() . "'" . $this->getExtra() . '>';
+	/**
+	 * Prepare HTML for output
+	 *
+	 * @return string HTML
+	 */
+	public function render()
+	{
+		$ret = "<input type='text' name='" . $this->getName() . "' id='" . $this->getName() . "' size='" . $this->getSize() . "' maxlength='" . $this->getMaxlength() . "' value='" . $this->getValue() . "'" . $this->getExtra() . '>';
 
-        $ret .= '   <div class="smartobject_autocomplete_hint" id="smartobject_autocomplete_hint' . $this->getName() . '"></div>
+		$ret .= '   <div class="smartobject_autocomplete_hint" id="smartobject_autocomplete_hint' . $this->getName() . '"></div>
 
     <script type="text/javascript">
         new Ajax.Autocompleter("' . $this->getName() . '","smartobject_autocomplete_hint' . $this->getName() . '","' . $this->_include_file . '?key=' . $this->getName() . '");
     </script>';
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -35,12 +35,12 @@
      */
     public function render()
     {
-        $ret = "<input type='text' name='" . $this->getName() . "' id='" . $this->getName() . "' size='" . $this->getSize() . "' maxlength='" . $this->getMaxlength() . "' value='" . $this->getValue() . "'" . $this->getExtra() . '>';
+        $ret = "<input type='text' name='".$this->getName()."' id='".$this->getName()."' size='".$this->getSize()."' maxlength='".$this->getMaxlength()."' value='".$this->getValue()."'".$this->getExtra().'>';
 
-        $ret .= '   <div class="smartobject_autocomplete_hint" id="smartobject_autocomplete_hint' . $this->getName() . '"></div>
+        $ret .= '   <div class="smartobject_autocomplete_hint" id="smartobject_autocomplete_hint'.$this->getName().'"></div>
 
     <script type="text/javascript">
-        new Ajax.Autocompleter("' . $this->getName() . '","smartobject_autocomplete_hint' . $this->getName() . '","' . $this->_include_file . '?key=' . $this->getName() . '");
+        new Ajax.Autocompleter("' . $this->getName().'","smartobject_autocomplete_hint'.$this->getName().'","'.$this->_include_file.'?key='.$this->getName().'");
     </script>';
 
         return $ret;

class/form/elements/smartformrichfileelement.php

Indentation +33 added lines, -33 removed lines

@@ -11,39 +11,39 @@
  */
 class SmartFormRichFileElement extends XoopsFormElementTray
 {
-    /**
-     * SmartFormRichFileElement constructor.
-     * @param string $form_caption
-     * @param string $key
-     * @param string $object
-     */
-    public function __construct($form_caption, $key, $object)
-    {
-        parent::__construct($form_caption, ' ');
-        if ($object->getVar('url') !== '') {
-            $caption = $object->getVar('caption') !== '' ? $object->getVar('caption') : $object->getVar('url');
-            $this->addElement(new XoopsFormLabel('', _CO_SOBJECT_CURRENT_FILE . "<a href='" . str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar('url')) . "' target='_blank' >" . $caption . '</a><br><br>'));
-            //$this->addElement( new XoopsFormLabel( '', "<br><a href = '".SMARTOBJECT_URL."admin/file.php?op=del&fileid=".$object->id()."'>"._CO_SOBJECT_DELETE_FILE."</a>"));
-        }
+	/**
+	 * SmartFormRichFileElement constructor.
+	 * @param string $form_caption
+	 * @param string $key
+	 * @param string $object
+	 */
+	public function __construct($form_caption, $key, $object)
+	{
+		parent::__construct($form_caption, '&nbsp;');
+		if ($object->getVar('url') !== '') {
+			$caption = $object->getVar('caption') !== '' ? $object->getVar('caption') : $object->getVar('url');
+			$this->addElement(new XoopsFormLabel('', _CO_SOBJECT_CURRENT_FILE . "<a href='" . str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar('url')) . "' target='_blank' >" . $caption . '</a><br><br>'));
+			//$this->addElement( new XoopsFormLabel( '', "<br><a href = '".SMARTOBJECT_URL."admin/file.php?op=del&fileid=".$object->id()."'>"._CO_SOBJECT_DELETE_FILE."</a>"));
+		}
 
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformfileuploadelement.php';
-        if ($object->isNew()) {
-            $this->addElement(new SmartFormFileUploadElement($object, $key));
-            $this->addElement(new XoopsFormLabel('', '<br><br><small>' . _CO_SOBJECT_URL_FILE_DSC . '</small>'));
-            $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
-            $this->addElement(new SmartFormTextElement($object, 'url_' . $key));
-        }
-        $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_CAPTION));
-        $this->addElement(new SmartFormTextElement($object, 'caption_' . $key));
-        $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_DESC . '<br>'));
-        $this->addElement(new XoopsFormTextArea('', 'desc_' . $key, $object->getVar('description')));
+		require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformfileuploadelement.php';
+		if ($object->isNew()) {
+			$this->addElement(new SmartFormFileUploadElement($object, $key));
+			$this->addElement(new XoopsFormLabel('', '<br><br><small>' . _CO_SOBJECT_URL_FILE_DSC . '</small>'));
+			$this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
+			$this->addElement(new SmartFormTextElement($object, 'url_' . $key));
+		}
+		$this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_CAPTION));
+		$this->addElement(new SmartFormTextElement($object, 'caption_' . $key));
+		$this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_DESC . '<br>'));
+		$this->addElement(new XoopsFormTextArea('', 'desc_' . $key, $object->getVar('description')));
 
-        if (!$object->isNew()) {
-            $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_CHANGE_FILE));
-            $this->addElement(new SmartFormFileUploadElement($object, $key));
-            $this->addElement(new XoopsFormLabel('', '<br><br><small>' . _CO_SOBJECT_URL_FILE_DSC . '</small>'));
-            $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
-            $this->addElement(new SmartFormTextElement($object, 'url_' . $key));
-        }
-    }
+		if (!$object->isNew()) {
+			$this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_CHANGE_FILE));
+			$this->addElement(new SmartFormFileUploadElement($object, $key));
+			$this->addElement(new XoopsFormLabel('', '<br><br><small>' . _CO_SOBJECT_URL_FILE_DSC . '</small>'));
+			$this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
+			$this->addElement(new SmartFormTextElement($object, 'url_' . $key));
+		}
+	}
 }

Spacing +13 added lines, -13 removed lines

@@ -22,28 +22,28 @@
         parent::__construct($form_caption, ' ');
         if ($object->getVar('url') !== '') {
             $caption = $object->getVar('caption') !== '' ? $object->getVar('caption') : $object->getVar('url');
-            $this->addElement(new XoopsFormLabel('', _CO_SOBJECT_CURRENT_FILE . "<a href='" . str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar('url')) . "' target='_blank' >" . $caption . '</a><br><br>'));
+            $this->addElement(new XoopsFormLabel('', _CO_SOBJECT_CURRENT_FILE."<a href='".str_replace('{XOOPS_URL}', XOOPS_URL, $object->getVar('url'))."' target='_blank' >".$caption.'</a><br><br>'));
             //$this->addElement( new XoopsFormLabel( '', "<br><a href = '".SMARTOBJECT_URL."admin/file.php?op=del&fileid=".$object->id()."'>"._CO_SOBJECT_DELETE_FILE."</a>"));
         }
 
-        require_once SMARTOBJECT_ROOT_PATH . 'class/form/elements/smartformfileuploadelement.php';
+        require_once SMARTOBJECT_ROOT_PATH.'class/form/elements/smartformfileuploadelement.php';
         if ($object->isNew()) {
             $this->addElement(new SmartFormFileUploadElement($object, $key));
-            $this->addElement(new XoopsFormLabel('', '<br><br><small>' . _CO_SOBJECT_URL_FILE_DSC . '</small>'));
-            $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
-            $this->addElement(new SmartFormTextElement($object, 'url_' . $key));
+            $this->addElement(new XoopsFormLabel('', '<br><br><small>'._CO_SOBJECT_URL_FILE_DSC.'</small>'));
+            $this->addElement(new XoopsFormLabel('', '<br>'._CO_SOBJECT_URL_FILE));
+            $this->addElement(new SmartFormTextElement($object, 'url_'.$key));
         }
-        $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_CAPTION));
-        $this->addElement(new SmartFormTextElement($object, 'caption_' . $key));
-        $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_DESC . '<br>'));
-        $this->addElement(new XoopsFormTextArea('', 'desc_' . $key, $object->getVar('description')));
+        $this->addElement(new XoopsFormLabel('', '<br>'._CO_SOBJECT_CAPTION));
+        $this->addElement(new SmartFormTextElement($object, 'caption_'.$key));
+        $this->addElement(new XoopsFormLabel('', '<br>'._CO_SOBJECT_DESC.'<br>'));
+        $this->addElement(new XoopsFormTextArea('', 'desc_'.$key, $object->getVar('description')));
 
         if (!$object->isNew()) {
-            $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_CHANGE_FILE));
+            $this->addElement(new XoopsFormLabel('', '<br>'._CO_SOBJECT_CHANGE_FILE));
             $this->addElement(new SmartFormFileUploadElement($object, $key));
-            $this->addElement(new XoopsFormLabel('', '<br><br><small>' . _CO_SOBJECT_URL_FILE_DSC . '</small>'));
-            $this->addElement(new XoopsFormLabel('', '<br>' . _CO_SOBJECT_URL_FILE));
-            $this->addElement(new SmartFormTextElement($object, 'url_' . $key));
+            $this->addElement(new XoopsFormLabel('', '<br><br><small>'._CO_SOBJECT_URL_FILE_DSC.'</small>'));
+            $this->addElement(new XoopsFormLabel('', '<br>'._CO_SOBJECT_URL_FILE));
+            $this->addElement(new SmartFormTextElement($object, 'url_'.$key));
         }
     }
 }

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