mambax7 / smartobject

class/Plugin.php

Indentation +55 added lines, -55 removed lines

@@ -25,66 +25,66 @@
  */
 class Plugin
 {
-    public $_infoArray;
+	public $_infoArray;
 
-    /**
-     * SmartPlugin constructor.
-     * @param $array
-     */
-    public function __construct($array)
-    {
-        $this->_infoArray = $array;
-    }
+	/**
+	 * SmartPlugin constructor.
+	 * @param $array
+	 */
+	public function __construct($array)
+	{
+		$this->_infoArray = $array;
+	}
 
-    /**
-     * @param $item
-     * @return bool
-     */
-    public function getItemInfo($item)
-    {
-        if (isset($this->_infoArray['items'][$item])) {
-            return $this->_infoArray['items'][$item];
-        } else {
-            return false;
-        }
-    }
+	/**
+	 * @param $item
+	 * @return bool
+	 */
+	public function getItemInfo($item)
+	{
+		if (isset($this->_infoArray['items'][$item])) {
+			return $this->_infoArray['items'][$item];
+		} else {
+			return false;
+		}
+	}
 
-    /**
-     * @return mixed
-     */
-    public function getItemList()
-    {
-        $itemsArray = $this->_infoArray['items'];
-        foreach ($itemsArray as $k => $v) {
-            $ret[$k] = $v['caption'];
-        }
+	/**
+	 * @return mixed
+	 */
+	public function getItemList()
+	{
+		$itemsArray = $this->_infoArray['items'];
+		foreach ($itemsArray as $k => $v) {
+			$ret[$k] = $v['caption'];
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 
-    /**
-     * @return bool|int|string
-     */
-    public function getItem()
-    {
-        $ret = false;
-        foreach ($this->_infoArray['items'] as $k => $v) {
-            $search_str = str_replace('%u', '', $v['url']);
-            if (strpos($_SERVER['PHP_SELF'] . '?' . $_SERVER['QUERY_STRING'], $search_str) > 0) {
-                $ret = $k;
-                break;
-            }
-        }
+	/**
+	 * @return bool|int|string
+	 */
+	public function getItem()
+	{
+		$ret = false;
+		foreach ($this->_infoArray['items'] as $k => $v) {
+			$search_str = str_replace('%u', '', $v['url']);
+			if (strpos($_SERVER['PHP_SELF'] . '?' . $_SERVER['QUERY_STRING'], $search_str) > 0) {
+				$ret = $k;
+				break;
+			}
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 
-    /**
-     * @param $item
-     * @return mixed
-     */
-    public function getItemIdForItem($item)
-    {
-        return $_REQUEST[$this->_infoArray['items'][$item]['request']];
-    }
+	/**
+	 * @param $item
+	 * @return mixed
+	 */
+	public function getItemIdForItem($item)
+	{
+		return $_REQUEST[$this->_infoArray['items'][$item]['request']];
+	}
 }

class/MlObject.php

Indentation +69 added lines, -69 removed lines

@@ -26,80 +26,80 @@
  */
 class MlObject extends Smartobject\BaseSmartObject
 {
-    /**
-     * SmartMlObject constructor.
-     */
-    public function __construct()
-    {
-        $this->initVar('language', XOBJ_DTYPE_TXTBOX, 'english', false, null, '', true, _CO_SOBJECT_LANGUAGE_CAPTION, _CO_SOBJECT_LANGUAGE_DSC, true, true);
-        $this->setControl('language', 'language');
-    }
+	/**
+	 * SmartMlObject constructor.
+	 */
+	public function __construct()
+	{
+		$this->initVar('language', XOBJ_DTYPE_TXTBOX, 'english', false, null, '', true, _CO_SOBJECT_LANGUAGE_CAPTION, _CO_SOBJECT_LANGUAGE_DSC, true, true);
+		$this->setControl('language', 'language');
+	}
 
-    /**
-     * If object is not new, change the control of the not-multilanguage fields
-     *
-     * We need to intercept this function from SmartObject because if the object is not new...
-     */
-    // function getForm() {
+	/**
+	 * If object is not new, change the control of the not-multilanguage fields
+	 *
+	 * We need to intercept this function from SmartObject because if the object is not new...
+	 */
+	// function getForm() {
 
-    //}
+	//}
 
-    /**
-     * Strip Multilanguage Fields
-     *
-     * Get rid of all the multilanguage fields to have an object with only global fields.
-     * This will be usefull when creating the ML object for the first time. Then we will be able
-     * to create translations.
-     */
-    public function stripMultilanguageFields()
-    {
-        $objectVars    =& $this->getVars();
-        $newObjectVars = [];
-        foreach ($objectVars as $key => $var) {
-            if (!$var['multilingual']) {
-                $newObjectVars[$key] = $var;
-            }
-        }
-        $this->vars = $newObjectVars;
-    }
+	/**
+	 * Strip Multilanguage Fields
+	 *
+	 * Get rid of all the multilanguage fields to have an object with only global fields.
+	 * This will be usefull when creating the ML object for the first time. Then we will be able
+	 * to create translations.
+	 */
+	public function stripMultilanguageFields()
+	{
+		$objectVars    =& $this->getVars();
+		$newObjectVars = [];
+		foreach ($objectVars as $key => $var) {
+			if (!$var['multilingual']) {
+				$newObjectVars[$key] = $var;
+			}
+		}
+		$this->vars = $newObjectVars;
+	}
 
-    public function stripNonMultilanguageFields()
-    {
-        $objectVars    =& $this->getVars();
-        $newObjectVars = [];
-        foreach ($objectVars as $key => $var) {
-            if ($var['multilingual'] || $key == $this->handler->keyName) {
-                $newObjectVars[$key] = $var;
-            }
-        }
-        $this->vars = $newObjectVars;
-    }
+	public function stripNonMultilanguageFields()
+	{
+		$objectVars    =& $this->getVars();
+		$newObjectVars = [];
+		foreach ($objectVars as $key => $var) {
+			if ($var['multilingual'] || $key == $this->handler->keyName) {
+				$newObjectVars[$key] = $var;
+			}
+		}
+		$this->vars = $newObjectVars;
+	}
 
-    /**
-     * Make non multilanguage fields read only
-     *
-     * This is used when we are creating/editing a translation.
-     * We only want to edit the multilanguag fields, not the global one.
-     */
-    public function makeNonMLFieldReadOnly()
-    {
-        foreach ($this->getVars() as $key => $var) {
-            //if (($key == 'language') || (!$var['multilingual'] && $key <> $this->handler->keyName)) {
-            if (!$var['multilingual'] && $key != $this->handler->keyName) {
-                $this->setControl($key, 'label');
-            }
-        }
-    }
+	/**
+	 * Make non multilanguage fields read only
+	 *
+	 * This is used when we are creating/editing a translation.
+	 * We only want to edit the multilanguag fields, not the global one.
+	 */
+	public function makeNonMLFieldReadOnly()
+	{
+		foreach ($this->getVars() as $key => $var) {
+			//if (($key == 'language') || (!$var['multilingual'] && $key <> $this->handler->keyName)) {
+			if (!$var['multilingual'] && $key != $this->handler->keyName) {
+				$this->setControl($key, 'label');
+			}
+		}
+	}
 
-    /**
-     * @param  bool $onlyUrl
-     * @param  bool $withimage
-     * @return string
-     */
-    public function getEditLanguageLink($onlyUrl = false, $withimage = true)
-    {
-        $controller = new ObjectController($this->handler);
+	/**
+	 * @param  bool $onlyUrl
+	 * @param  bool $withimage
+	 * @return string
+	 */
+	public function getEditLanguageLink($onlyUrl = false, $withimage = true)
+	{
+		$controller = new ObjectController($this->handler);
 
-        return $controller->getEditLanguageLink($this, $onlyUrl, $withimage);
-    }
+		return $controller->getEditLanguageLink($this, $onlyUrl, $withimage);
+	}
 }

class/Uploader.php

Indentation +42 added lines, -42 removed lines

@@ -81,51 +81,51 @@
  */
 class Uploader 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;
+	}
 }

class/HookHandler.php

Indentation +37 added lines, -37 removed lines

@@ -27,44 +27,44 @@
  */
 class HookHandler
 {
-    /**
-     * SmartHookHandler constructor.
-     */
-    public function __construct()
-    {
-    }
+	/**
+	 * SmartHookHandler constructor.
+	 */
+	public function __construct()
+	{
+	}
 
-    /**
-     * Access the only instance of this class
-     *
-     * @return \XoopsModules\Smartobject\HookHandler
-     *
-     * @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\HookHandler
+	 *
+	 * @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();
+			}
+		}
+	}
 }

class/Tip.php

Indentation +39 added lines, -39 removed lines

@@ -17,45 +17,45 @@
  */
 class Tip
 {
-    public $id;
-    public $caption;
-    public $message;
-    public $visible;
-    public $_tpl;
+	public $id;
+	public $caption;
+	public $message;
+	public $visible;
+	public $_tpl;
 
-    /**
-     * SmartTip constructor.
-     * @param      $id
-     * @param      $caption
-     * @param      $message
-     * @param bool $visible
-     */
-    public function __construct($id, $caption, $message, $visible = false)
-    {
-        $this->id      = $id;
-        $this->caption = $caption;
-        $this->message = $message;
-        $this->visible = $visible;
-        $this->_tpl    = new \XoopsTpl();
-    }
+	/**
+	 * SmartTip constructor.
+	 * @param      $id
+	 * @param      $caption
+	 * @param      $message
+	 * @param bool $visible
+	 */
+	public function __construct($id, $caption, $message, $visible = false)
+	{
+		$this->id      = $id;
+		$this->caption = $caption;
+		$this->message = $message;
+		$this->visible = $visible;
+		$this->_tpl    = new \XoopsTpl();
+	}
 
-    /**
-     * @param  bool $outputNow
-     * @return mixed|string|void
-     */
-    public function render($outputNow = true)
-    {
-        $aTip = [
-            'id'      => $this->id,
-            'caption' => $this->caption,
-            'message' => $this->message,
-            'visible' => $this->visible ? 'block' : 'none'
-        ];
-        $this->_tpl->assign('tip', $aTip);
-        if ($outputNow) {
-            $this->_tpl->display('db:smartobject_tip.tpl');
-        } else {
-            return $this->_tpl->fetch('db:smartobject_tip.tpl');
-        }
-    }
+	/**
+	 * @param  bool $outputNow
+	 * @return mixed|string|void
+	 */
+	public function render($outputNow = true)
+	{
+		$aTip = [
+			'id'      => $this->id,
+			'caption' => $this->caption,
+			'message' => $this->message,
+			'visible' => $this->visible ? 'block' : 'none'
+		];
+		$this->_tpl->assign('tip', $aTip);
+		if ($outputNow) {
+			$this->_tpl->display('db:smartobject_tip.tpl');
+		} else {
+			return $this->_tpl->fetch('db:smartobject_tip.tpl');
+		}
+	}
 }

class/PluginHandler.php

Indentation +46 added lines, -46 removed lines

@@ -27,57 +27,57 @@
  */
 class PluginHandler
 {
-    public $pluginPatterns = false;
+	public $pluginPatterns = false;
 
-    /**
-     * @param $dirname
-     * @return bool|Plugin
-     */
-    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 Plugin($array);
+	/**
+	 * @param $dirname
+	 * @return bool|Plugin
+	 */
+	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 Plugin($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;
+	}
 }

class/Export.php

Indentation +105 added lines, -105 removed lines

@@ -41,116 +41,116 @@
  */
 class Export
 {
-    public $handler;
-    public $criteria;
-    public $fields;
-    public $format;
-    public $filename;
-    public $filepath;
-    public $options;
-    public $outputMethods = false;
-    public $notDisplayFields;
+	public $handler;
+	public $criteria;
+	public $fields;
+	public $format;
+	public $filename;
+	public $filepath;
+	public $options;
+	public $outputMethods = false;
+	public $notDisplayFields;
 
-    /**
-     * Constructor
-     *
-     * @param PersistableObjectHandler $objectHandler SmartObjectHandler handling the data we want to export
-     * @param \CriteriaElement         $criteria      containing the criteria of the query fetching the objects to be exported
-     * @param array|bool               $fields        fields to be exported. If FALSE then all fields will be exported
-     * @param bool|string              $filename      name of the file to be created
-     * @param bool|string              $filepath      path where the file will be saved
-     * @param string                   $format        format of the ouputed export. Currently only supports CSV
-     * @param array|bool               $options       options of the format to be exported in
-     */
-    public function __construct(
-        PersistableObjectHandler $objectHandler,
-        \CriteriaElement $criteria = null,
-        $fields = false,
-        $filename = false,
-        $filepath = false,
-        $format = 'csv',
-        $options = false
-    ) {
-        $this->handler          = $objectHandler;
-        $this->criteria         = $criteria;
-        $this->fields           = $fields;
-        $this->filename         = $filename;
-        $this->format           = $format;
-        $this->options          = $options;
-        $this->notDisplayFields = false;
-    }
+	/**
+	 * Constructor
+	 *
+	 * @param PersistableObjectHandler $objectHandler SmartObjectHandler handling the data we want to export
+	 * @param \CriteriaElement         $criteria      containing the criteria of the query fetching the objects to be exported
+	 * @param array|bool               $fields        fields to be exported. If FALSE then all fields will be exported
+	 * @param bool|string              $filename      name of the file to be created
+	 * @param bool|string              $filepath      path where the file will be saved
+	 * @param string                   $format        format of the ouputed export. Currently only supports CSV
+	 * @param array|bool               $options       options of the format to be exported in
+	 */
+	public function __construct(
+		PersistableObjectHandler $objectHandler,
+		\CriteriaElement $criteria = null,
+		$fields = false,
+		$filename = false,
+		$filepath = false,
+		$format = 'csv',
+		$options = false
+	) {
+		$this->handler          = $objectHandler;
+		$this->criteria         = $criteria;
+		$this->fields           = $fields;
+		$this->filename         = $filename;
+		$this->format           = $format;
+		$this->options          = $options;
+		$this->notDisplayFields = false;
+	}
 
-    /**
-     * Renders the export
-     * @param $filename
-     */
-    public function render($filename)
-    {
-        $this->filename = $filename;
+	/**
+	 * Renders the export
+	 * @param $filename
+	 */
+	public function render($filename)
+	{
+		$this->filename = $filename;
 
-        $objects        = $this->handler->getObjects($this->criteria);
-        $rows           = [];
-        $columnsHeaders = [];
-        $firstObject    = true;
-        foreach ($objects as $object) {
-            $row = [];
-            foreach ($object->vars as $key => $var) {
-                if ((!$this->fields || in_array($key, $this->fields)) && !in_array($key, $this->notDisplayFields)) {
-                    if ($this->outputMethods && isset($this->outputMethods[$key])
-                        && method_exists($object, $this->outputMethods[$key])) {
-                        $method    = $this->outputMethods[$key];
-                        $row[$key] = $object->$method();
-                    } else {
-                        $row[$key] = $object->getVar($key);
-                    }
-                    if ($firstObject) {
-                        // then set the columnsHeaders array as well
-                        $columnsHeaders[$key] = $var['form_caption'];
-                    }
-                }
-            }
-            $firstObject = false;
-            $rows[]      = $row;
-            unset($row);
-        }
-        $data                   = [];
-        $data['rows']           = $rows;
-        $data['columnsHeaders'] = $columnsHeaders;
-        $smartExportRenderer    = new ExportRenderer($data, $this->filename, $this->filepath, $this->format, $this->options);
-        $smartExportRenderer->execute();
-    }
+		$objects        = $this->handler->getObjects($this->criteria);
+		$rows           = [];
+		$columnsHeaders = [];
+		$firstObject    = true;
+		foreach ($objects as $object) {
+			$row = [];
+			foreach ($object->vars as $key => $var) {
+				if ((!$this->fields || in_array($key, $this->fields)) && !in_array($key, $this->notDisplayFields)) {
+					if ($this->outputMethods && isset($this->outputMethods[$key])
+						&& method_exists($object, $this->outputMethods[$key])) {
+						$method    = $this->outputMethods[$key];
+						$row[$key] = $object->$method();
+					} else {
+						$row[$key] = $object->getVar($key);
+					}
+					if ($firstObject) {
+						// then set the columnsHeaders array as well
+						$columnsHeaders[$key] = $var['form_caption'];
+					}
+				}
+			}
+			$firstObject = false;
+			$rows[]      = $row;
+			unset($row);
+		}
+		$data                   = [];
+		$data['rows']           = $rows;
+		$data['columnsHeaders'] = $columnsHeaders;
+		$smartExportRenderer    = new ExportRenderer($data, $this->filename, $this->filepath, $this->format, $this->options);
+		$smartExportRenderer->execute();
+	}
 
-    /**
-     * Set an array contaning the alternate methods to use instead of the default getVar()
-     *
-     * $outputMethods array example: 'uid' => 'getUserName'...
-     * @param $outputMethods
-     */
-    public function setOuptutMethods($outputMethods)
-    {
-        $this->outputMethods = $outputMethods;
-    }
+	/**
+	 * Set an array contaning the alternate methods to use instead of the default getVar()
+	 *
+	 * $outputMethods array example: 'uid' => 'getUserName'...
+	 * @param $outputMethods
+	 */
+	public function setOuptutMethods($outputMethods)
+	{
+		$this->outputMethods = $outputMethods;
+	}
 
-    /*
+	/*
      * Set an array of fields that we don't want in export
      */
-    /**
-     * @param $fields
-     */
-    public function setNotDisplayFields($fields)
-    {
-        if (!$this->notDisplayFields) {
-            if (is_array($fields)) {
-                $this->notDisplayFields = $fields;
-            } else {
-                $this->notDisplayFields = [$fields];
-            }
-        } else {
-            if (is_array($fields)) {
-                $this->notDisplayFields = array_merge($this->notDisplayFields, $fields);
-            } else {
-                $this->notDisplayFields[] = $fields;
-            }
-        }
-    }
+	/**
+	 * @param $fields
+	 */
+	public function setNotDisplayFields($fields)
+	{
+		if (!$this->notDisplayFields) {
+			if (is_array($fields)) {
+				$this->notDisplayFields = $fields;
+			} else {
+				$this->notDisplayFields = [$fields];
+			}
+		} else {
+			if (is_array($fields)) {
+				$this->notDisplayFields = array_merge($this->notDisplayFields, $fields);
+			} else {
+				$this->notDisplayFields[] = $fields;
+			}
+		}
+	}
 }

class/Tag.php

Indentation +12 added lines, -12 removed lines

@@ -28,17 +28,17 @@
  */
 class Tag extends MlObject
 {
-    /**
-     * SmartobjectTag constructor.
-     */
-    public function __construct()
-    {
-        $this->initVar('tagid', XOBJ_DTYPE_INT, '', true);
-        $this->initVar('name', XOBJ_DTYPE_TXTBOX, '', true, 255, '', false, _CO_SOBJECT_TAG_TAGID_CAPTION, _CO_SOBJECT_TAG_TAGID_DSC, true);
-        $this->initVar('description', XOBJ_DTYPE_TXTAREA, '', true, null, '', false, _CO_SOBJECT_TAG_DESCRIPTION_CAPTION, _CO_SOBJECT_TAG_DESCRIPTION_DSC);
-        $this->initVar('value', XOBJ_DTYPE_TXTAREA, '', true, null, '', true, _CO_SOBJECT_TAG_VALUE_CAPTION, _CO_SOBJECT_TAG_VALUE_DSC);
+	/**
+	 * SmartobjectTag constructor.
+	 */
+	public function __construct()
+	{
+		$this->initVar('tagid', XOBJ_DTYPE_INT, '', true);
+		$this->initVar('name', XOBJ_DTYPE_TXTBOX, '', true, 255, '', false, _CO_SOBJECT_TAG_TAGID_CAPTION, _CO_SOBJECT_TAG_TAGID_DSC, true);
+		$this->initVar('description', XOBJ_DTYPE_TXTAREA, '', true, null, '', false, _CO_SOBJECT_TAG_DESCRIPTION_CAPTION, _CO_SOBJECT_TAG_DESCRIPTION_DSC);
+		$this->initVar('value', XOBJ_DTYPE_TXTAREA, '', true, null, '', true, _CO_SOBJECT_TAG_VALUE_CAPTION, _CO_SOBJECT_TAG_VALUE_DSC);
 
-        // call parent constructor to get Multilanguage field initiated
-        $this->SmartMlObject();
-    }
+		// call parent constructor to get Multilanguage field initiated
+		$this->SmartMlObject();
+	}
 }

class/Tree.php

Indentation +13 added lines, -13 removed lines

@@ -30,17 +30,17 @@
  */
 class Tree extends \XoopsObjectTree
 {
-    public function _initialize()
-    {
-        foreach (array_keys($this->_objects) as $i) {
-            $key1                         = $this->_objects[$i]->getVar($this->_myId);
-            $this->tree[$key1]['obj']     = $this->_objects[$i];
-            $key2                         = $this->_objects[$i]->getVar($this->_parentId, 'e');
-            $this->tree[$key1]['parent']  = $key2;
-            $this->tree[$key2]['child'][] = $key1;
-            if (isset($this->_rootId)) {
-                $this->tree[$key1]['root'] = $this->_objects[$i]->getVar($this->_rootId);
-            }
-        }
-    }
+	public function _initialize()
+	{
+		foreach (array_keys($this->_objects) as $i) {
+			$key1                         = $this->_objects[$i]->getVar($this->_myId);
+			$this->tree[$key1]['obj']     = $this->_objects[$i];
+			$key2                         = $this->_objects[$i]->getVar($this->_parentId, 'e');
+			$this->tree[$key1]['parent']  = $key2;
+			$this->tree[$key2]['child'][] = $key1;
+			if (isset($this->_rootId)) {
+				$this->tree[$key1]['root'] = $this->_objects[$i]->getVar($this->_rootId);
+			}
+		}
+	}
 }