mambax7 / smartobject

class/Form/Elements/SmartFormSelectElement.php

Indentation +49 added lines, -49 removed lines

@@ -18,59 +18,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/SmartFormUrlLinkElement.php

Indentation +27 added lines, -27 removed lines

@@ -18,31 +18,31 @@
  */
 class SmartFormUrlLinkElement extends \XoopsFormElementTray
 {
-    /**
-     * SmartFormUrlLinkElement constructor.
-     * @param string $form_caption
-     * @param string $key
-     * @param string $object
-     */
-    public function __construct($form_caption, $key, $object)
-    {
-        parent::__construct($form_caption, ' ');
-
-        $this->addElement(new \XoopsFormLabel('', '<br>' . _CO_SOBJECT_URLLINK_URL));
-        $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_URLLINK_TARGET));
-        $targ_val    = $object->getVar('target');
-        $targetRadio = new \XoopsFormRadio('', 'target_' . $key, '' !== $targ_val ? $targ_val : '_blank');
-        $control     = $object->getControl('target');
-        $targetRadio->addOptionArray($control['options']);
-
-        $this->addElement($targetRadio);
-    }
+	/**
+	 * SmartFormUrlLinkElement constructor.
+	 * @param string $form_caption
+	 * @param string $key
+	 * @param string $object
+	 */
+	public function __construct($form_caption, $key, $object)
+	{
+		parent::__construct($form_caption, '&nbsp;');
+
+		$this->addElement(new \XoopsFormLabel('', '<br>' . _CO_SOBJECT_URLLINK_URL));
+		$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_URLLINK_TARGET));
+		$targ_val    = $object->getVar('target');
+		$targetRadio = new \XoopsFormRadio('', 'target_' . $key, '' !== $targ_val ? $targ_val : '_blank');
+		$control     = $object->getControl('target');
+		$targetRadio->addOptionArray($control['options']);
+
+		$this->addElement($targetRadio);
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -28,18 +28,18 @@
     {
         parent::__construct($form_caption, ' ');
 
-        $this->addElement(new \XoopsFormLabel('', '<br>' . _CO_SOBJECT_URLLINK_URL));
-        $this->addElement(new SmartFormTextElement($object, 'url_' . $key));
+        $this->addElement(new \XoopsFormLabel('', '<br>'._CO_SOBJECT_URLLINK_URL));
+        $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_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_DESC.'<br>'));
+        $this->addElement(new \XoopsFormTextArea('', 'desc_'.$key, $object->getVar('description')));
 
-        $this->addElement(new \XoopsFormLabel('', '<br>' . _CO_SOBJECT_URLLINK_TARGET));
+        $this->addElement(new \XoopsFormLabel('', '<br>'._CO_SOBJECT_URLLINK_TARGET));
         $targ_val    = $object->getVar('target');
-        $targetRadio = new \XoopsFormRadio('', 'target_' . $key, '' !== $targ_val ? $targ_val : '_blank');
+        $targetRadio = new \XoopsFormRadio('', 'target_'.$key, '' !== $targ_val ? $targ_val : '_blank');
         $control     = $object->getControl('target');
         $targetRadio->addOptionArray($control['options']);
 

class/Form/Elements/SmartFormHidden.php

Indentation +15 added lines, -15 removed lines

@@ -26,20 +26,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/SmartFormImageElement.php

Indentation +29 added lines, -29 removed lines

@@ -18,39 +18,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'], ' ');
-
-        $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>"));
-        }
+	/**
+	 * 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']);
+
+		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));
+		$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>', ''));
+		$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 +7 added lines, -7 removed lines

@@ -34,23 +34,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';
         $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>', ''));
+        $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/SmartFormTextElement.php

Indentation +18 added lines, -18 removed lines

@@ -18,24 +18,24 @@
  */
 class SmartFormTextElement extends \XoopsFormText
 {
-    /**
-     * SmartFormTextElement constructor.
-     * @param string $object
-     * @param string $key
-     */
-    public function __construct($object, $key)
-    {
-        $var = $object->vars[$key];
+	/**
+	 * SmartFormTextElement constructor.
+	 * @param string $object
+	 * @param string $key
+	 */
+	public function __construct($object, $key)
+	{
+		$var = $object->vars[$key];
 
-        if (isset($object->controls[$key])) {
-            $control        = $object->controls[$key];
-            $form_maxlength = isset($control['maxlength']) ? $control['maxlength'] : (isset($var['maxlength']) ? $var['maxlength'] : 255);
-            $form_size      = isset($control['size']) ? $control['size'] : 50;
-        } else {
-            $form_maxlength = 255;
-            $form_size      = 50;
-        }
+		if (isset($object->controls[$key])) {
+			$control        = $object->controls[$key];
+			$form_maxlength = isset($control['maxlength']) ? $control['maxlength'] : (isset($var['maxlength']) ? $var['maxlength'] : 255);
+			$form_size      = isset($control['size']) ? $control['size'] : 50;
+		} else {
+			$form_maxlength = 255;
+			$form_size      = 50;
+		}
 
-        parent::__construct($var['form_caption'], $key, $form_size, $form_maxlength, $object->getVar($key, 'e'));
-    }
+		parent::__construct($var['form_caption'], $key, $form_size, $form_maxlength, $object->getVar($key, 'e'));
+	}
 }

class/SmartobjectRatingHandler.php

Indentation +89 added lines, -89 removed lines

@@ -31,93 +31,93 @@
  */
 class SmartobjectRatingHandler extends Smartobject\SmartPersistableObjectHandler
 {
-    public $_rateOptions = [];
-    public $_moduleList  = false;
-    public $pluginsObject;
-
-    /**
-     * SmartobjectRatingHandler constructor.
-     * @param \XoopsDatabase $db
-     */
-    public function __construct(\XoopsDatabase $db)
-    {
-        parent::__construct($db, 'rating', 'ratingid', 'rate', '', 'smartobject');
-        $this->generalSQL = 'SELECT * FROM ' . $this->table . ' AS ' . $this->_itemname . ' INNER JOIN ' . $this->db->prefix('users') . ' AS user ON ' . $this->_itemname . '.uid=user.uid';
-
-        $this->_rateOptions[1] = 1;
-        $this->_rateOptions[2] = 2;
-        $this->_rateOptions[3] = 3;
-        $this->_rateOptions[4] = 4;
-        $this->_rateOptions[5] = 5;
-
-        $this->pluginsObject = new SmartPluginHandler();
-    }
-
-    /**
-     * @return bool
-     */
-    public function getModuleList()
-    {
-        if (!$this->_moduleList) {
-            $moduleArray          = $this->pluginsObject->getPluginsArray();
-            $this->_moduleList[0] = _CO_SOBJECT_MAKE_SELECTION;
-            foreach ($moduleArray as $k => $v) {
-                $this->_moduleList[$k] = $v;
-            }
-        }
-
-        return $this->_moduleList;
-    }
-
-    /**
-     * @return array
-     */
-    public function getRateList()
-    {
-        return $this->_rateOptions;
-    }
-
-    /**
-     * @param $itemid
-     * @param $dirname
-     * @param $item
-     * @return int
-     */
-    public function getRatingAverageByItemId($itemid, $dirname, $item)
-    {
-        $sql    = 'SELECT AVG(rate), COUNT(ratingid) FROM ' . $this->table . " WHERE itemid=$itemid AND dirname='$dirname' AND item='$item' GROUP BY itemid";
-        $result = $this->db->query($sql);
-        if (!$result) {
-            return 0;
-        }
-        list($average, $sum) = $this->db->fetchRow($result);
-        $ret['average'] = isset($average) ? $average : 0;
-        $ret['sum']     = isset($sum) ? $sum : 0;
-
-        return $ret;
-    }
-
-    /**
-     * @param $item
-     * @param $itemid
-     * @param $dirname
-     * @param $uid
-     * @return bool
-     */
-    public function already_rated($item, $itemid, $dirname, $uid)
-    {
-        $criteria = new \CriteriaCompo();
-        $criteria->add(new \Criteria('item', $item));
-        $criteria->add(new \Criteria('itemid', $itemid));
-        $criteria->add(new \Criteria('dirname', $dirname));
-        $criteria->add(new \Criteria('user.uid', $uid));
-
-        $ret =& $this->getObjects($criteria);
-
-        if (!$ret) {
-            return false;
-        } else {
-            return $ret[0];
-        }
-    }
+	public $_rateOptions = [];
+	public $_moduleList  = false;
+	public $pluginsObject;
+
+	/**
+	 * SmartobjectRatingHandler constructor.
+	 * @param \XoopsDatabase $db
+	 */
+	public function __construct(\XoopsDatabase $db)
+	{
+		parent::__construct($db, 'rating', 'ratingid', 'rate', '', 'smartobject');
+		$this->generalSQL = 'SELECT * FROM ' . $this->table . ' AS ' . $this->_itemname . ' INNER JOIN ' . $this->db->prefix('users') . ' AS user ON ' . $this->_itemname . '.uid=user.uid';
+
+		$this->_rateOptions[1] = 1;
+		$this->_rateOptions[2] = 2;
+		$this->_rateOptions[3] = 3;
+		$this->_rateOptions[4] = 4;
+		$this->_rateOptions[5] = 5;
+
+		$this->pluginsObject = new SmartPluginHandler();
+	}
+
+	/**
+	 * @return bool
+	 */
+	public function getModuleList()
+	{
+		if (!$this->_moduleList) {
+			$moduleArray          = $this->pluginsObject->getPluginsArray();
+			$this->_moduleList[0] = _CO_SOBJECT_MAKE_SELECTION;
+			foreach ($moduleArray as $k => $v) {
+				$this->_moduleList[$k] = $v;
+			}
+		}
+
+		return $this->_moduleList;
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getRateList()
+	{
+		return $this->_rateOptions;
+	}
+
+	/**
+	 * @param $itemid
+	 * @param $dirname
+	 * @param $item
+	 * @return int
+	 */
+	public function getRatingAverageByItemId($itemid, $dirname, $item)
+	{
+		$sql    = 'SELECT AVG(rate), COUNT(ratingid) FROM ' . $this->table . " WHERE itemid=$itemid AND dirname='$dirname' AND item='$item' GROUP BY itemid";
+		$result = $this->db->query($sql);
+		if (!$result) {
+			return 0;
+		}
+		list($average, $sum) = $this->db->fetchRow($result);
+		$ret['average'] = isset($average) ? $average : 0;
+		$ret['sum']     = isset($sum) ? $sum : 0;
+
+		return $ret;
+	}
+
+	/**
+	 * @param $item
+	 * @param $itemid
+	 * @param $dirname
+	 * @param $uid
+	 * @return bool
+	 */
+	public function already_rated($item, $itemid, $dirname, $uid)
+	{
+		$criteria = new \CriteriaCompo();
+		$criteria->add(new \Criteria('item', $item));
+		$criteria->add(new \Criteria('itemid', $itemid));
+		$criteria->add(new \Criteria('dirname', $dirname));
+		$criteria->add(new \Criteria('user.uid', $uid));
+
+		$ret =& $this->getObjects($criteria);
+
+		if (!$ret) {
+			return false;
+		} else {
+			return $ret[0];
+		}
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
     public function __construct(\XoopsDatabase $db)
     {
         parent::__construct($db, 'rating', 'ratingid', 'rate', '', 'smartobject');
-        $this->generalSQL = 'SELECT * FROM ' . $this->table . ' AS ' . $this->_itemname . ' INNER JOIN ' . $this->db->prefix('users') . ' AS user ON ' . $this->_itemname . '.uid=user.uid';
+        $this->generalSQL = 'SELECT * FROM '.$this->table.' AS '.$this->_itemname.' INNER JOIN '.$this->db->prefix('users').' AS user ON '.$this->_itemname.'.uid=user.uid';
 
         $this->_rateOptions[1] = 1;
         $this->_rateOptions[2] = 2;
@@ -85,7 +85,7 @@ 
      */
     public function getRatingAverageByItemId($itemid, $dirname, $item)
     {
-        $sql    = 'SELECT AVG(rate), COUNT(ratingid) FROM ' . $this->table . " WHERE itemid=$itemid AND dirname='$dirname' AND item='$item' GROUP BY itemid";
+        $sql    = 'SELECT AVG(rate), COUNT(ratingid) FROM '.$this->table." WHERE itemid=$itemid AND dirname='$dirname' AND item='$item' GROUP BY itemid";
         $result = $this->db->query($sql);
         if (!$result) {
             return 0;
@@ -112,7 +112,7 @@ 
         $criteria->add(new \Criteria('dirname', $dirname));
         $criteria->add(new \Criteria('user.uid', $uid));
 
-        $ret =& $this->getObjects($criteria);
+        $ret = & $this->getObjects($criteria);
 
         if (!$ret) {
             return false;

class/SmartHookHandler.php

Indentation +37 added lines, -37 removed lines

@@ -27,44 +27,44 @@
  */
 class SmartHookHandler
 {
-    /**
-     * SmartHookHandler constructor.
-     */
-    public function __construct()
-    {
-    }
+	/**
+	 * SmartHookHandler constructor.
+	 */
+	public function __construct()
+	{
+	}
 
-    /**
-     * Access the only instance of this class
-     *
-     * @return \XoopsModules\Smartobject\SmartHookHandler
-     *
-     * @static
-     * @staticvar   object
-     */
-    public static function getInstance()
-    {
-        static $instance;
-        if (null === $instance) {
-            $instance = new static();
-        }
+	/**
+	 * Access the only instance of this class
+	 *
+	 * @return \XoopsModules\Smartobject\SmartHookHandler
+	 *
+	 * @static
+	 * @staticvar   object
+	 */
+	public static function getInstance()
+	{
+		static $instance;
+		if (null === $instance) {
+			$instance = new static();
+		}
 
-        return $instance;
-    }
+		return $instance;
+	}
 
-    /**
-     * @param $hook_name
-     */
-    public function executeHook($hook_name)
-    {
-        $lower_hook_name = strtolower($hook_name);
-        $filename        = SMARTOBJECT_ROOT_PATH . 'include/custom_code/' . $lower_hook_name . '.php';
-        if (file_exists($filename)) {
-            require_once $filename;
-            $function = 'smarthook_' . $lower_hook_name;
-            if (function_exists($function)) {
-                $function();
-            }
-        }
-    }
+	/**
+	 * @param $hook_name
+	 */
+	public function executeHook($hook_name)
+	{
+		$lower_hook_name = strtolower($hook_name);
+		$filename        = SMARTOBJECT_ROOT_PATH . 'include/custom_code/' . $lower_hook_name . '.php';
+		if (file_exists($filename)) {
+			require_once $filename;
+			$function = 'smarthook_' . $lower_hook_name;
+			if (function_exists($function)) {
+				$function();
+			}
+		}
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -37,10 +37,10 @@
     public function executeHook($hook_name)
     {
         $lower_hook_name = strtolower($hook_name);
-        $filename        = SMARTOBJECT_ROOT_PATH . 'include/custom_code/' . $lower_hook_name . '.php';
+        $filename        = SMARTOBJECT_ROOT_PATH.'include/custom_code/'.$lower_hook_name.'.php';
         if (file_exists($filename)) {
             require_once $filename;
-            $function = 'smarthook_' . $lower_hook_name;
+            $function = 'smarthook_'.$lower_hook_name;
             if (function_exists($function)) {
                 $function();
             }

class/SmartPluginHandler.php

Indentation +46 added lines, -46 removed lines

@@ -27,57 +27,57 @@
  */
 class SmartPluginHandler
 {
-    public $pluginPatterns = false;
+	public $pluginPatterns = false;
 
-    /**
-     * @param $dirname
-     * @return bool|SmartPlugin
-     */
-    public function getPlugin($dirname)
-    {
-        $pluginName = SMARTOBJECT_ROOT_PATH . 'plugins/' . $dirname . '.php';
-        if (file_exists($pluginName)) {
-            require_once $pluginName;
-            $function = 'smartobject_plugin_' . $dirname;
-            if (function_exists($function)) {
-                $array = $function();
-                $ret   = new SmartPlugin($array);
+	/**
+	 * @param $dirname
+	 * @return bool|SmartPlugin
+	 */
+	public function getPlugin($dirname)
+	{
+		$pluginName = SMARTOBJECT_ROOT_PATH . 'plugins/' . $dirname . '.php';
+		if (file_exists($pluginName)) {
+			require_once $pluginName;
+			$function = 'smartobject_plugin_' . $dirname;
+			if (function_exists($function)) {
+				$array = $function();
+				$ret   = new SmartPlugin($array);
 
-                return $ret;
-            }
-        }
+				return $ret;
+			}
+		}
 
-        return false;
-    }
+		return false;
+	}
 
-    /**
-     * @return array
-     */
-    public function getPluginsArray()
-    {
-        require_once XOOPS_ROOT_PATH . '/class/xoopslists.php';
+	/**
+	 * @return array
+	 */
+	public function getPluginsArray()
+	{
+		require_once XOOPS_ROOT_PATH . '/class/xoopslists.php';
 
-        $moduleHandler = xoops_getHandler('module');
-        $criteria      = new \CriteriaCompo();
-        $criteria->add(new \Criteria('isactive', 1));
-        $tempModulesObj = $moduleHandler->getObjects($criteria);
-        $modulesObj     = [];
-        foreach ($tempModulesObj as $moduleObj) {
-            $modulesObj[$moduleObj->getVar('dirname')] = $moduleObj;
-        }
+		$moduleHandler = xoops_getHandler('module');
+		$criteria      = new \CriteriaCompo();
+		$criteria->add(new \Criteria('isactive', 1));
+		$tempModulesObj = $moduleHandler->getObjects($criteria);
+		$modulesObj     = [];
+		foreach ($tempModulesObj as $moduleObj) {
+			$modulesObj[$moduleObj->getVar('dirname')] = $moduleObj;
+		}
 
-        $aFiles = XoopsLists::getFileListAsArray(SMARTOBJECT_ROOT_PATH . 'plugins/');
-        $ret    = [];
-        foreach ($aFiles as $file) {
-            if ('.php' === substr($file, strlen($file) - 4, 4)) {
-                $pluginName                = str_replace('.php', '', $file);
-                $module_xoops_version_file = XOOPS_ROOT_PATH . "/modules/$pluginName/xoops_version.php";
-                if (file_exists($module_xoops_version_file) && isset($modulesObj[$pluginName])) {
-                    $ret[$pluginName] = $modulesObj[$pluginName]->getVar('name');
-                }
-            }
-        }
+		$aFiles = XoopsLists::getFileListAsArray(SMARTOBJECT_ROOT_PATH . 'plugins/');
+		$ret    = [];
+		foreach ($aFiles as $file) {
+			if ('.php' === substr($file, strlen($file) - 4, 4)) {
+				$pluginName                = str_replace('.php', '', $file);
+				$module_xoops_version_file = XOOPS_ROOT_PATH . "/modules/$pluginName/xoops_version.php";
+				if (file_exists($module_xoops_version_file) && isset($modulesObj[$pluginName])) {
+					$ret[$pluginName] = $modulesObj[$pluginName]->getVar('name');
+				}
+			}
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

Spacing +5 added lines, -5 removed lines

@@ -35,10 +35,10 @@ 
      */
     public function getPlugin($dirname)
     {
-        $pluginName = SMARTOBJECT_ROOT_PATH . 'plugins/' . $dirname . '.php';
+        $pluginName = SMARTOBJECT_ROOT_PATH.'plugins/'.$dirname.'.php';
         if (file_exists($pluginName)) {
             require_once $pluginName;
-            $function = 'smartobject_plugin_' . $dirname;
+            $function = 'smartobject_plugin_'.$dirname;
             if (function_exists($function)) {
                 $array = $function();
                 $ret   = new SmartPlugin($array);
@@ -55,7 +55,7 @@ 
      */
     public function getPluginsArray()
     {
-        require_once XOOPS_ROOT_PATH . '/class/xoopslists.php';
+        require_once XOOPS_ROOT_PATH.'/class/xoopslists.php';
 
         $moduleHandler = xoops_getHandler('module');
         $criteria      = new \CriteriaCompo();
@@ -66,12 +66,12 @@ 
             $modulesObj[$moduleObj->getVar('dirname')] = $moduleObj;
         }
 
-        $aFiles = XoopsLists::getFileListAsArray(SMARTOBJECT_ROOT_PATH . 'plugins/');
+        $aFiles = XoopsLists::getFileListAsArray(SMARTOBJECT_ROOT_PATH.'plugins/');
         $ret    = [];
         foreach ($aFiles as $file) {
             if ('.php' === substr($file, strlen($file) - 4, 4)) {
                 $pluginName                = str_replace('.php', '', $file);
-                $module_xoops_version_file = XOOPS_ROOT_PATH . "/modules/$pluginName/xoops_version.php";
+                $module_xoops_version_file = XOOPS_ROOT_PATH."/modules/$pluginName/xoops_version.php";
                 if (file_exists($module_xoops_version_file) && isset($modulesObj[$pluginName])) {
                     $ret[$pluginName] = $modulesObj[$pluginName]->getVar('name');
                 }

class/SmartUploader.php

Indentation +42 added lines, -42 removed lines

@@ -81,51 +81,51 @@
  */
 class SmartUploader extends \XoopsMediaUploader
 {
-    public $ext;
-    public $dimension;
+	public $ext;
+	public $dimension;
 
-    /**
-     * No admin check for uploads
-     */
-    public $noAdminSizeCheck;
+	/**
+	 * No admin check for uploads
+	 */
+	public $noAdminSizeCheck;
 
-    /**
-     * Constructor
-     *
-     * @param string    $uploadDir
-     * @param array|int $allowedMimeTypes
-     * @param int       $maxFileSize
-     * @param int       $maxWidth
-     * @param int       $maxHeight
-     * @internal param int $cmodvalue
-     */
-    public function __construct($uploadDir, $allowedMimeTypes = 0, $maxFileSize, $maxWidth = 0, $maxHeight = 0)
-    {
-        parent::__construct($uploadDir, $allowedMimeTypes, $maxFileSize, $maxWidth, $maxHeight);
-    }
+	/**
+	 * Constructor
+	 *
+	 * @param string    $uploadDir
+	 * @param array|int $allowedMimeTypes
+	 * @param int       $maxFileSize
+	 * @param int       $maxWidth
+	 * @param int       $maxHeight
+	 * @internal param int $cmodvalue
+	 */
+	public function __construct($uploadDir, $allowedMimeTypes = 0, $maxFileSize, $maxWidth = 0, $maxHeight = 0)
+	{
+		parent::__construct($uploadDir, $allowedMimeTypes, $maxFileSize, $maxWidth, $maxHeight);
+	}
 
-    /**
-     * @param $value
-     */
-    public function noAdminSizeCheck($value)
-    {
-        $this->noAdminSizeCheck = $value;
-    }
+	/**
+	 * @param $value
+	 */
+	public function noAdminSizeCheck($value)
+	{
+		$this->noAdminSizeCheck = $value;
+	}
 
-    /**
-     * Is the file the right size?
-     *
-     * @return bool
-     */
-    public function checkMaxFileSize()
-    {
-        if ($this->noAdminSizeCheck) {
-            return true;
-        }
-        if ($this->mediaSize > $this->maxFileSize) {
-            return false;
-        }
+	/**
+	 * Is the file the right size?
+	 *
+	 * @return bool
+	 */
+	public function checkMaxFileSize()
+	{
+		if ($this->noAdminSizeCheck) {
+			return true;
+		}
+		if ($this->mediaSize > $this->maxFileSize) {
+			return false;
+		}
 
-        return true;
-    }
+		return true;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -69,9 +69,9 @@
  * @link    http://smartfactory.ca The SmartFactory
  * @package SmartObject
  */
-mt_srand((double)microtime() * 1000000);
+mt_srand((double) microtime() * 1000000);
 
-require_once XOOPS_ROOT_PATH . '/class/uploader.php';
+require_once XOOPS_ROOT_PATH.'/class/uploader.php';
 
 /**
  * Class SmartUploader