XoopsModules25x / songlist

class/Utf8mapHandler.php

Indentation +85 added lines, -85 removed lines

@@ -17,98 +17,98 @@
  */
 class Utf8mapHandler extends XoopsPersistableObjectHandler
 {
-    /**
-     * Utf8mapHandler constructor.
-     * @param \XoopsDatabase $db
-     */
-    public function __construct(XoopsDatabase $db)
-    {
-        parent::__construct($db, 'songlist_utf8map', Utf8map::class, 'utfid', 'from');
-    }
+	/**
+	 * Utf8mapHandler constructor.
+	 * @param \XoopsDatabase $db
+	 */
+	public function __construct(XoopsDatabase $db)
+	{
+		parent::__construct($db, 'songlist_utf8map', Utf8map::class, 'utfid', 'from');
+	}
 
-    /**
-     * @return array
-     */
-    public function filterFields(): array
-    {
-        return ['utfid', 'from', 'to', 'created', 'updated'];
-    }
+	/**
+	 * @return array
+	 */
+	public function filterFields(): array
+	{
+		return ['utfid', 'from', 'to', 'created', 'updated'];
+	}
 
-    /**
-     * @param $filter
-     * @return \CriteriaCompo
-     */
-    public function getFilterCriteria($filter): CriteriaCompo
-    {
-        $parts    = \explode('|', $filter);
-        $criteria = new CriteriaCompo();
-        foreach ($parts as $part) {
-            $var = \explode(',', $part);
-            if (!empty($var[1]) && !\is_numeric($var[0])) {
-                $object = $this->create();
-                if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
-                    || \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
-                } elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
-                }
-            } elseif (!empty($var[1]) && \is_numeric($var[0])) {
-                $criteria->add(new Criteria($var[0], $var[1]));
-            }
-        }
+	/**
+	 * @param $filter
+	 * @return \CriteriaCompo
+	 */
+	public function getFilterCriteria($filter): CriteriaCompo
+	{
+		$parts    = \explode('|', $filter);
+		$criteria = new CriteriaCompo();
+		foreach ($parts as $part) {
+			$var = \explode(',', $part);
+			if (!empty($var[1]) && !\is_numeric($var[0])) {
+				$object = $this->create();
+				if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
+					|| \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
+				} elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
+				} elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
+				} elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
+				}
+			} elseif (!empty($var[1]) && \is_numeric($var[0])) {
+				$criteria->add(new Criteria($var[0], $var[1]));
+			}
+		}
 
-        return $criteria;
-    }
+		return $criteria;
+	}
 
-    /**
-     * @param        $filter
-     * @param        $field
-     * @param string $sort
-     * @param string $op
-     * @param string $fct
-     * @return string
-     */
-    public function getFilterForm($filter, $field, $sort = 'created', $op = 'dashboard', $fct = 'list'): string
-    {
-        $ele = Utility::getFilterElement($filter, $field, $sort, $op, $fct);
-        if (\is_object($ele)) {
-            return $ele->render();
-        }
+	/**
+	 * @param        $filter
+	 * @param        $field
+	 * @param string $sort
+	 * @param string $op
+	 * @param string $fct
+	 * @return string
+	 */
+	public function getFilterForm($filter, $field, $sort = 'created', $op = 'dashboard', $fct = 'list'): string
+	{
+		$ele = Utility::getFilterElement($filter, $field, $sort, $op, $fct);
+		if (\is_object($ele)) {
+			return $ele->render();
+		}
 
-        return ' ';
-    }
+		return ' ';
+	}
 
-    /**
-     * @param bool $force
-     * @return mixed
-     */
-    public function insert(XoopsObject $obj, $force = true)
-    {
-        if ($obj->isNew()) {
-            $obj->setVar('created', \time());
-        } else {
-            $obj->setVar('updated', \time());
-        }
+	/**
+	 * @param bool $force
+	 * @return mixed
+	 */
+	public function insert(XoopsObject $obj, $force = true)
+	{
+		if ($obj->isNew()) {
+			$obj->setVar('created', \time());
+		} else {
+			$obj->setVar('updated', \time());
+		}
 
-        return parent::insert($obj, $force);
-    }
+		return parent::insert($obj, $force);
+	}
 
-    /**
-     * @param string $phrase
-     * @param null   $criteria
-     * @return string|string[]
-     */
-    public function convert($phrase = '', $criteria = null)
-    {
-        foreach ($this->getObjects($criteria, true) as $utfid => $utf8) {
-            $phrase = \str_replace(mb_strtolower($utf8->getVar('from')), \mb_strtolower($utf8->getVar('to')), $phrase);
-            $phrase = \str_replace(mb_strtoupper($utf8->getVar('from')), \mb_strtoupper($utf8->getVar('to')), $phrase);
-        }
+	/**
+	 * @param string $phrase
+	 * @param null   $criteria
+	 * @return string|string[]
+	 */
+	public function convert($phrase = '', $criteria = null)
+	{
+		foreach ($this->getObjects($criteria, true) as $utfid => $utf8) {
+			$phrase = \str_replace(mb_strtolower($utf8->getVar('from')), \mb_strtolower($utf8->getVar('to')), $phrase);
+			$phrase = \str_replace(mb_strtoupper($utf8->getVar('from')), \mb_strtoupper($utf8->getVar('to')), $phrase);
+		}
 
-        return $phrase;
-    }
+		return $phrase;
+	}
 }

Spacing +9 added lines, -9 removed lines

@@ -8,7 +8,7 @@ 
 use XoopsObject;
 use XoopsPersistableObjectHandler;
 
-require_once \dirname(__DIR__) . '/include/songlist.object.php';
+require_once \dirname(__DIR__).'/include/songlist.object.php';
 // require_once \dirname(__DIR__) . '/include/songlist.form.php';
 use  XoopsModules\Songlist\Form\FormController;
 
@@ -46,15 +46,15 @@ 
             $var = \explode(',', $part);
             if (!empty($var[1]) && !\is_numeric($var[0])) {
                 $object = $this->create();
-                if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
-                    || \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
+                if (\XOBJ_DTYPE_TXTBOX==$object->vars[$var[0]]['data_type']
+                    || \XOBJ_DTYPE_TXTAREA==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new Criteria('`'.$var[0].'`', '%'.$var[1].'%', ($var[2] ?? 'LIKE')));
                 } elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
+                    $criteria->add(new Criteria('`'.$var[0].'`', $var[1], ($var[2] ?? '=')));
+                } elseif (\XOBJ_DTYPE_ENUM==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new Criteria('`'.$var[0].'`', $var[1], ($var[2] ?? '=')));
+                } elseif (\XOBJ_DTYPE_ARRAY==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new Criteria('`'.$var[0].'`', '%"'.$var[1].'";%', ($var[2] ?? 'LIKE')));
                 }
             } elseif (!empty($var[1]) && \is_numeric($var[0])) {
                 $criteria->add(new Criteria($var[0], $var[1]));

class/Genre.php

Spacing +4 added lines, -4 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace XoopsModules\Songlist;
 
-require_once \dirname(__DIR__) . '/include/songlist.object.php';
+require_once \dirname(__DIR__).'/include/songlist.object.php';
 // require_once \dirname(__DIR__) . '/include/songlist.form.php';
 use  XoopsModules\Songlist\Form\FormController;
 
@@ -49,12 +49,12 @@ 
             $ret['form'][$key] = $element->render();
         }
         foreach (['created', 'updated'] as $key) {
-            if ($this->getVar($key) > 0) {
+            if ($this->getVar($key)>0) {
                 $ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
                 $ret[$key]         = \date(_DATESTRING, $this->getVar($key));
             }
         }
-        $ret['rank'] = \number_format(($this->getVar('rank') > 0 && $this->getVar('votes') > 0 ? $this->getVar('rank') / $this->getVar('votes') : 0), 2) . \_MI_SONGLIST_OFTEN;
+        $ret['rank'] = \number_format(($this->getVar('rank')>0 && $this->getVar('votes')>0 ? $this->getVar('rank')/$this->getVar('votes') : 0), 2).\_MI_SONGLIST_OFTEN;
 
         return $ret;
     }
@@ -88,6 +88,6 @@ 
                    . $GLOBALS['songlistModuleConfig']['endofurl'];
         }
 
-        return XOOPS_URL . '/modules/songlist/' . $file . '.php?op=' . $op . '&fct=' . $fct . '&id=' . $this->getVar('gid') . '&value=' . \urlencode($value ?? '') . '&gid=' . $gid . '&cid=' . $cid;
+        return XOOPS_URL.'/modules/songlist/'.$file.'.php?op='.$op.'&fct='.$fct.'&id='.$this->getVar('gid').'&value='.\urlencode($value ?? '').'&gid='.$gid.'&cid='.$cid;
     }
 }

Indentation +84 added lines, -84 removed lines

@@ -12,94 +12,94 @@
  */
 class Genre extends \XoopsObject
 {
-    public $gid;
-    public $name;
-    public $artists;
-    public $albums;
-    public $songs;
-    public $hits;
-    public $rank;
-    public $votes;
-    public $created;
-    public $updated;
+	public $gid;
+	public $name;
+	public $artists;
+	public $albums;
+	public $songs;
+	public $hits;
+	public $rank;
+	public $votes;
+	public $created;
+	public $updated;
 
-    /**
-     * Genre constructor.
-     * @param null $fid
-     */
-    public function __construct($fid = null)
-    {
-        $this->initVar('gid', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('name', \XOBJ_DTYPE_TXTBOX, null, false, 128);
-        $this->initVar('artists', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('albums', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('songs', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('hits', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('rank', \XOBJ_DTYPE_DECIMAL, 0, false);
-        $this->initVar('votes', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('created', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('updated', \XOBJ_DTYPE_INT, 0, false);
-    }
+	/**
+	 * Genre constructor.
+	 * @param null $fid
+	 */
+	public function __construct($fid = null)
+	{
+		$this->initVar('gid', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('name', \XOBJ_DTYPE_TXTBOX, null, false, 128);
+		$this->initVar('artists', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('albums', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('songs', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('hits', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('rank', \XOBJ_DTYPE_DECIMAL, 0, false);
+		$this->initVar('votes', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('created', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('updated', \XOBJ_DTYPE_INT, 0, false);
+	}
 
-    /**
-     * @param bool $as_array
-     * @return array|string
-     */
-    public function getForm($as_array = false)
-    {
-        return FormController::getFormGenre($this, $as_array);
-    }
+	/**
+	 * @param bool $as_array
+	 * @return array|string
+	 */
+	public function getForm($as_array = false)
+	{
+		return FormController::getFormGenre($this, $as_array);
+	}
 
-    /**
-     * @return array
-     */
-    public function toArray(): array
-    {
-        $ret  = parent::toArray();
-        $form = $this->getForm(true);
-        foreach ($form as $key => $element) {
-            $ret['form'][$key] = $element->render();
-        }
-        foreach (['created', 'updated'] as $key) {
-            if ($this->getVar($key) > 0) {
-                $ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
-                $ret[$key]         = \date(_DATESTRING, $this->getVar($key));
-            }
-        }
-        $ret['rank'] = \number_format(($this->getVar('rank') > 0 && $this->getVar('votes') > 0 ? $this->getVar('rank') / $this->getVar('votes') : 0), 2) . \_MI_SONGLIST_OFTEN;
+	/**
+	 * @return array
+	 */
+	public function toArray(): array
+	{
+		$ret  = parent::toArray();
+		$form = $this->getForm(true);
+		foreach ($form as $key => $element) {
+			$ret['form'][$key] = $element->render();
+		}
+		foreach (['created', 'updated'] as $key) {
+			if ($this->getVar($key) > 0) {
+				$ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
+				$ret[$key]         = \date(_DATESTRING, $this->getVar($key));
+			}
+		}
+		$ret['rank'] = \number_format(($this->getVar('rank') > 0 && $this->getVar('votes') > 0 ? $this->getVar('rank') / $this->getVar('votes') : 0), 2) . \_MI_SONGLIST_OFTEN;
 
-        return $ret;
-    }
+		return $ret;
+	}
 
-    /**
-     * @return string
-     */
-    public function getURL(): string
-    {
-        global $file, $op, $fct, $id, $value, $gid, $cid, $start, $limit;
-        if ($GLOBALS['songlistModuleConfig']['htaccess']) {
-            return XOOPS_URL
-                   . '/'
-                   . $GLOBALS['songlistModuleConfig']['baseurl']
-                   . '/'
-                   . $file
-                   . '/'
-                   . \urlencode(\str_replace([' ', \chr(9)], '-', $this->getVar('name')))
-                   . '/'
-                   . $op
-                   . '-'
-                   . $fct
-                   . '-'
-                   . $this->getVar('gid')
-                   . '-'
-                   . \urlencode($value)
-                   . '-'
-                   . $gid
-                   . '-'
-                   . $cid
-                   . $GLOBALS['songlistModuleConfig']['endofurl'];
-        }
+	/**
+	 * @return string
+	 */
+	public function getURL(): string
+	{
+		global $file, $op, $fct, $id, $value, $gid, $cid, $start, $limit;
+		if ($GLOBALS['songlistModuleConfig']['htaccess']) {
+			return XOOPS_URL
+				   . '/'
+				   . $GLOBALS['songlistModuleConfig']['baseurl']
+				   . '/'
+				   . $file
+				   . '/'
+				   . \urlencode(\str_replace([' ', \chr(9)], '-', $this->getVar('name')))
+				   . '/'
+				   . $op
+				   . '-'
+				   . $fct
+				   . '-'
+				   . $this->getVar('gid')
+				   . '-'
+				   . \urlencode($value)
+				   . '-'
+				   . $gid
+				   . '-'
+				   . $cid
+				   . $GLOBALS['songlistModuleConfig']['endofurl'];
+		}
 
-        return XOOPS_URL . '/modules/songlist/' . $file . '.php?op=' . $op . '&fct=' . $fct . '&id=' . $this->getVar('gid') . '&value=' . \urlencode($value ?? '') . '&gid=' . $gid . '&cid=' . $cid;
-    }
+		return XOOPS_URL . '/modules/songlist/' . $file . '.php?op=' . $op . '&fct=' . $fct . '&id=' . $this->getVar('gid') . '&value=' . \urlencode($value ?? '') . '&gid=' . $gid . '&cid=' . $cid;
+	}
 }

class/ExtrasHandler.php

Indentation +209 added lines, -209 removed lines

@@ -11,213 +11,213 @@
  */
 class ExtrasHandler extends XoopsPersistableObjectHandler
 {
-    /**
-     * holds reference to {@link ObjectsFieldHandler} object
-     */
-    public $_fHandler;
-    /**
-     * Array of {@link XoopsObjectsField} objects
-     * @var array
-     */
-    public $_fields = [];
-
-    /**
-     * ExtrasHandler constructor.
-     * @param \XoopsDatabase $db
-     */
-    public function __construct(XoopsDatabase $db)
-    {
-        parent::__construct($db, 'songlist_extra', Extras::class, 'sid');
-        $this->_fHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Field');
-    }
-
-    /**
-     * create a new {@link Extras}
-     *
-     * @param bool $isNew Flag the new objects as "new"?
-     *
-     * @return object {@link Extras}
-     */
-    public function &create($isNew = true): object
-    {
-        $obj          = new $this->className($this->loadFields());
-        $obj->handler = $this;
-        $obj->setNew();
-
-        return $obj;
-    }
-
-    /**
-     * Get a {@link Extras}
-     *
-     * @param null $id
-     * @param null $fields
-     * @return object <a href='psi_element://Extras'>Extras</a>
-     */
-    public function get($id = null, $fields = null): object //get($uid, $createOnFailure = true)
-    {
-        if (null === $fields) {
-            $fields = true;
-        }
-        $obj = parent::get($id);
-        if (!\is_object($obj) && $fields) {
-            $obj = $this->create();
-        }
-
-        return $obj;
-    }
-
-    /**
-     * Create new {@link ObjectsField} object
-     *
-     * @param bool $isNew
-     *
-     * @return object
-     */
-    public function &createField($isNew = true): object
-    {
-        $return = $this->_fHandler->create($isNew);
-
-        return $return;
-    }
-
-    /**
-     * Load field information
-     *
-     * @return array|false
-     */
-    public function loadFields()
-    {
-        if (0 == \count($this->_fields)) {
-            $this->_fields = $this->_fHandler->loadFields();
-        }
-
-        return (0 == $this->_fields || empty($this->_fields) ? false : $this->_fields);
-    }
-
-    /**
-     * @return array
-     */
-    public function getPostVars(): array
-    {
-        return ['sid', 'cid', 'gid', 'aids', 'abid', 'songid', 'title', 'lyrics', 'hits', 'rank', 'votes', 'tags', 'created', 'updated'];
-    }
-
-    /**
-     * @param      $criteria
-     * @param bool $id_as_key
-     * @param bool $as_object
-     * @return array
-     */
-    public function getFields($criteria, $id_as_key = true, $as_object = true): array
-    {
-        return $this->_fHandler->getObjects($criteria, $id_as_key, $as_object);
-    }
-
-    /**
-     * Insert a field in the database
-     *
-     * @param XoopsObject $field
-     * @param bool   $force
-     * @return mixed|void
-     */
-    public function insertField($field, $force = false)
-    {
-        return $this->_fHandler->insert($field, $force);
-    }
-
-    /**
-     * Delete a field from the database
-     *
-     * @param XoopsObject $field
-     * @param bool   $force
-     * @return bool
-     */
-    public function deleteField($field, $force = false)
-    {
-        return $this->_fHandler->delete($field, $force);
-    }
-
-    /**
-     * Save a new field in the database
-     *
-     * @param array $vars array of variables, taken from $module->loadInfo('Extras')['field']
-     * @param int   $weight
-     * @return string
-     * @internal param int $categoryid ID of the category to add it to
-     * @internal param int $type valuetype of the field
-     * @internal param int $moduleid ID of the module, this field belongs to
-     */
-    public function saveField($vars, $weight = 0): string
-    {
-        $field = $this->createField();
-        $field->setVar('field_name', $vars['name']);
-        $field->setVar('field_valuetype', $vars['valuetype']);
-        $field->setVar('field_type', $vars['type']);
-        $field->setVar('field_weight', $weight);
-        if (isset($vars['title'])) {
-            $field->setVar('field_title', $vars['title']);
-        }
-        if (isset($vars['description'])) {
-            $field->setVar('field_description', $vars['description']);
-        }
-        if (isset($vars['required'])) {
-            $field->setVar('field_required', $vars['required']); //0 = no, 1 = yes
-        }
-        if (isset($vars['maxlength'])) {
-            $field->setVar('field_maxlength', $vars['maxlength']);
-        }
-        if (isset($vars['default'])) {
-            $field->setVar('field_default', $vars['default']);
-        }
-        if (isset($vars['notnull'])) {
-            $field->setVar('field_notnull', $vars['notnull']);
-        }
-        if (isset($vars['show'])) {
-            $field->setVar('field_show', $vars['show']);
-        }
-        if (isset($vars['edit'])) {
-            $field->setVar('field_edit', $vars['edit']);
-        }
-        if (isset($vars['config'])) {
-            $field->setVar('field_config', $vars['config']);
-        }
-        if (isset($vars['options'])) {
-            $field->setVar('field_options', $vars['options']);
-        } else {
-            $field->setVar('field_options', []);
-        }
-        if ($this->insertField($field)) {
-            $msg = '&nbsp;&nbsp;Field <b>' . $vars['name'] . '</b> added to the database';
-        } else {
-            $msg = '&nbsp;&nbsp;<span style="color:#ff0000;">ERROR: Could not insert field <b>' . $vars['name'] . '</b> into the database. ' . \implode(' ', $field->getErrors()) . $this->db->error() . '</span>';
-        }
-        unset($field);
-
-        return $msg;
-    }
-
-    /**
-     * insert a new object in the database
-     *
-     * @param \XoopsObject $obj         reference to the object
-     * @param bool         $force       whether to force the query execution despite security settings
-     * @param bool         $checkObject check if the object is dirty and clean the attributes
-     *
-     * @return bool FALSE if failed, TRUE if already present and unchanged or successful
-     */
-    public function insert(XoopsObject $obj, $force = false, $checkObject = true): bool
-    {
-        $uservars = $this->getPostVars();
-        foreach ($uservars as $var) {
-            if ('sid' != $var) {
-                unset($obj->vars[$var]);
-            }
-        }
-        if (0 == \count($obj->vars)) {
-            return true;
-        }
-
-        return (bool)parent::insert($obj, $force, $checkObject);
-    }
+	/**
+	 * holds reference to {@link ObjectsFieldHandler} object
+	 */
+	public $_fHandler;
+	/**
+	 * Array of {@link XoopsObjectsField} objects
+	 * @var array
+	 */
+	public $_fields = [];
+
+	/**
+	 * ExtrasHandler constructor.
+	 * @param \XoopsDatabase $db
+	 */
+	public function __construct(XoopsDatabase $db)
+	{
+		parent::__construct($db, 'songlist_extra', Extras::class, 'sid');
+		$this->_fHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Field');
+	}
+
+	/**
+	 * create a new {@link Extras}
+	 *
+	 * @param bool $isNew Flag the new objects as "new"?
+	 *
+	 * @return object {@link Extras}
+	 */
+	public function &create($isNew = true): object
+	{
+		$obj          = new $this->className($this->loadFields());
+		$obj->handler = $this;
+		$obj->setNew();
+
+		return $obj;
+	}
+
+	/**
+	 * Get a {@link Extras}
+	 *
+	 * @param null $id
+	 * @param null $fields
+	 * @return object <a href='psi_element://Extras'>Extras</a>
+	 */
+	public function get($id = null, $fields = null): object //get($uid, $createOnFailure = true)
+	{
+		if (null === $fields) {
+			$fields = true;
+		}
+		$obj = parent::get($id);
+		if (!\is_object($obj) && $fields) {
+			$obj = $this->create();
+		}
+
+		return $obj;
+	}
+
+	/**
+	 * Create new {@link ObjectsField} object
+	 *
+	 * @param bool $isNew
+	 *
+	 * @return object
+	 */
+	public function &createField($isNew = true): object
+	{
+		$return = $this->_fHandler->create($isNew);
+
+		return $return;
+	}
+
+	/**
+	 * Load field information
+	 *
+	 * @return array|false
+	 */
+	public function loadFields()
+	{
+		if (0 == \count($this->_fields)) {
+			$this->_fields = $this->_fHandler->loadFields();
+		}
+
+		return (0 == $this->_fields || empty($this->_fields) ? false : $this->_fields);
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getPostVars(): array
+	{
+		return ['sid', 'cid', 'gid', 'aids', 'abid', 'songid', 'title', 'lyrics', 'hits', 'rank', 'votes', 'tags', 'created', 'updated'];
+	}
+
+	/**
+	 * @param      $criteria
+	 * @param bool $id_as_key
+	 * @param bool $as_object
+	 * @return array
+	 */
+	public function getFields($criteria, $id_as_key = true, $as_object = true): array
+	{
+		return $this->_fHandler->getObjects($criteria, $id_as_key, $as_object);
+	}
+
+	/**
+	 * Insert a field in the database
+	 *
+	 * @param XoopsObject $field
+	 * @param bool   $force
+	 * @return mixed|void
+	 */
+	public function insertField($field, $force = false)
+	{
+		return $this->_fHandler->insert($field, $force);
+	}
+
+	/**
+	 * Delete a field from the database
+	 *
+	 * @param XoopsObject $field
+	 * @param bool   $force
+	 * @return bool
+	 */
+	public function deleteField($field, $force = false)
+	{
+		return $this->_fHandler->delete($field, $force);
+	}
+
+	/**
+	 * Save a new field in the database
+	 *
+	 * @param array $vars array of variables, taken from $module->loadInfo('Extras')['field']
+	 * @param int   $weight
+	 * @return string
+	 * @internal param int $categoryid ID of the category to add it to
+	 * @internal param int $type valuetype of the field
+	 * @internal param int $moduleid ID of the module, this field belongs to
+	 */
+	public function saveField($vars, $weight = 0): string
+	{
+		$field = $this->createField();
+		$field->setVar('field_name', $vars['name']);
+		$field->setVar('field_valuetype', $vars['valuetype']);
+		$field->setVar('field_type', $vars['type']);
+		$field->setVar('field_weight', $weight);
+		if (isset($vars['title'])) {
+			$field->setVar('field_title', $vars['title']);
+		}
+		if (isset($vars['description'])) {
+			$field->setVar('field_description', $vars['description']);
+		}
+		if (isset($vars['required'])) {
+			$field->setVar('field_required', $vars['required']); //0 = no, 1 = yes
+		}
+		if (isset($vars['maxlength'])) {
+			$field->setVar('field_maxlength', $vars['maxlength']);
+		}
+		if (isset($vars['default'])) {
+			$field->setVar('field_default', $vars['default']);
+		}
+		if (isset($vars['notnull'])) {
+			$field->setVar('field_notnull', $vars['notnull']);
+		}
+		if (isset($vars['show'])) {
+			$field->setVar('field_show', $vars['show']);
+		}
+		if (isset($vars['edit'])) {
+			$field->setVar('field_edit', $vars['edit']);
+		}
+		if (isset($vars['config'])) {
+			$field->setVar('field_config', $vars['config']);
+		}
+		if (isset($vars['options'])) {
+			$field->setVar('field_options', $vars['options']);
+		} else {
+			$field->setVar('field_options', []);
+		}
+		if ($this->insertField($field)) {
+			$msg = '&nbsp;&nbsp;Field <b>' . $vars['name'] . '</b> added to the database';
+		} else {
+			$msg = '&nbsp;&nbsp;<span style="color:#ff0000;">ERROR: Could not insert field <b>' . $vars['name'] . '</b> into the database. ' . \implode(' ', $field->getErrors()) . $this->db->error() . '</span>';
+		}
+		unset($field);
+
+		return $msg;
+	}
+
+	/**
+	 * insert a new object in the database
+	 *
+	 * @param \XoopsObject $obj         reference to the object
+	 * @param bool         $force       whether to force the query execution despite security settings
+	 * @param bool         $checkObject check if the object is dirty and clean the attributes
+	 *
+	 * @return bool FALSE if failed, TRUE if already present and unchanged or successful
+	 */
+	public function insert(XoopsObject $obj, $force = false, $checkObject = true): bool
+	{
+		$uservars = $this->getPostVars();
+		foreach ($uservars as $var) {
+			if ('sid' != $var) {
+				unset($obj->vars[$var]);
+			}
+		}
+		if (0 == \count($obj->vars)) {
+			return true;
+		}
+
+		return (bool)parent::insert($obj, $force, $checkObject);
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -56,7 +56,7 @@ 
      */
     public function get($id = null, $fields = null): object //get($uid, $createOnFailure = true)
     {
-        if (null === $fields) {
+        if (null===$fields) {
             $fields = true;
         }
         $obj = parent::get($id);
@@ -88,11 +88,11 @@ 
      */
     public function loadFields()
     {
-        if (0 == \count($this->_fields)) {
+        if (0==\count($this->_fields)) {
             $this->_fields = $this->_fHandler->loadFields();
         }
 
-        return (0 == $this->_fields || empty($this->_fields) ? false : $this->_fields);
+        return (0==$this->_fields || empty($this->_fields) ? false : $this->_fields);
     }
 
     /**
@@ -188,9 +188,9 @@ 
             $field->setVar('field_options', []);
         }
         if ($this->insertField($field)) {
-            $msg = '&nbsp;&nbsp;Field <b>' . $vars['name'] . '</b> added to the database';
+            $msg = '&nbsp;&nbsp;Field <b>'.$vars['name'].'</b> added to the database';
         } else {
-            $msg = '&nbsp;&nbsp;<span style="color:#ff0000;">ERROR: Could not insert field <b>' . $vars['name'] . '</b> into the database. ' . \implode(' ', $field->getErrors()) . $this->db->error() . '</span>';
+            $msg = '&nbsp;&nbsp;<span style="color:#ff0000;">ERROR: Could not insert field <b>'.$vars['name'].'</b> into the database. '.\implode(' ', $field->getErrors()).$this->db->error().'</span>';
         }
         unset($field);
 
@@ -210,14 +210,14 @@ 
     {
         $uservars = $this->getPostVars();
         foreach ($uservars as $var) {
-            if ('sid' != $var) {
+            if ('sid'!=$var) {
                 unset($obj->vars[$var]);
             }
         }
-        if (0 == \count($obj->vars)) {
+        if (0==\count($obj->vars)) {
             return true;
         }
 
-        return (bool)parent::insert($obj, $force, $checkObject);
+        return (bool) parent::insert($obj, $force, $checkObject);
     }
 }

class/CategoryHandler.php

Indentation +180 added lines, -180 removed lines

@@ -7,165 +7,165 @@ 
  */
 class CategoryHandler extends \XoopsPersistableObjectHandler
 {
-    /**
-     * CategoryHandler constructor.
-     * @param \XoopsDatabase $db
-     */
-    public function __construct(\XoopsDatabase $db)
-    {
-        parent::__construct($db, 'songlist_category', Category::class, 'cid', 'name');
-    }
-
-    /**
-     * @return array
-     */
-    public function filterFields(): array
-    {
-        return ['cid', 'pid', 'weight', 'name', 'image', 'path', 'artists', 'albums', 'songs', 'hits', 'rank', 'votes', 'created', 'updated'];
-    }
-
-    /**
-     * @param $filter
-     * @return \CriteriaCompo
-     */
-    public function getFilterCriteria($filter): \CriteriaCompo
-    {
-        $parts    = \explode('|', $filter);
-        $criteria = new \CriteriaCompo();
-        foreach ($parts as $part) {
-            $var = \explode(',', $part);
-            if (!empty($var[1]) && !\is_numeric($var[0])) {
-                $object = $this->create();
-                if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
-                    || \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
-                } elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
-                }
-            } elseif (!empty($var[1]) && \is_numeric($var[0])) {
-                $criteria->add(new \Criteria($var[0], $var[1]));
-            }
-        }
-
-        return $criteria;
-    }
-
-    /**
-     * @param        $filter
-     * @param        $field
-     * @param string $sort
-     * @param string $op
-     * @param string $fct
-     * @return string
-     */
-    public function getFilterForm($filter, $field, $sort = 'created', $op = 'dashboard', $fct = 'list'): string
-    {
-        $ele = Utility::getFilterElement($filter, $field, $sort, $op, $fct);
-        if (\is_object($ele)) {
-            return $ele->render();
-        }
-
-        return ' ';
-    }
-
-    /**
-     * @param int $pid
-     * @return mixed
-     */
-    public function GetCatAndSubCat($pid = 0)
-    {
-        $categories  = $this->getObjects(new \Criteria('pid', $pid), true);
-        $langs_array = $this->TreeIDs([], $categories, -1);
-
-        return $langs_array;
-    }
-
-    /**
-     * @param $langs_array
-     * @param $categories
-     * @param $level
-     * @return mixed
-     */
-    private function TreeIDs($langs_array, $categories, $level)
-    {
-        foreach ($categories as $catid => $category) {
-            $langs_array[$catid] = $catid;
-            $categoriesb         = $this->getObjects(new \Criteria('pid', $catid), true);
-            if ($categoriesb) {
-                $langs_array = $this->TreeIDs($langs_array, $categoriesb, $level);
-            }
-        }
-
-        return $langs_array;
-    }
-
-    /**
-     * @param bool $force
-     * @return bool|mixed
-     */
-    public function insert(\XoopsObject $obj, $force = true)
-    {
-        if ($obj->isNew()) {
-            $obj->setVar('created', \time());
-        } else {
-            $obj->setVar('updated', \time());
-        }
-        if ('' == $obj->getVar('name')) {
-            return false;
-        }
-
-        return parent::insert($obj, $force);
-    }
-
-    public $_objects = ['object' => [], 'array' => []];
-
-    /**
-     * @param null $id
-     * @param null $fields
-     * @return \XoopsObject
-     */
-    public function get($id = null, $fields = null): \XoopsObject//get($id, $fields = '*')
-    {
-        $fields = $fields ?: '*';
-        if (!isset($this->_objects['object'][$id])) {
-            $this->_objects['object'][$id] = parent::get($id, $fields);
-            if (!isset($GLOBALS['songlistAdmin'])) {
-                $sql = 'UPDATE `' . $this->table . '` set hits=hits+1 where `' . $this->keyName . '` = ' . $id;
-                $GLOBALS['xoopsDB']->queryF($sql);
-            }
-        }
-
-        return $this->_objects['object'][$id];
-    }
-
-    /**
-     * @param \XoopsObject $object
-     * @param bool         $force
-     * @return bool
-     */
-    public function delete(\XoopsObject $object, $force = true): bool
-    {
-        parent::delete($object, $force);
-        $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('songlist_songs') . ' SET `cid` = 0 WHERE `cid` = ' . $object->getVar('cid');
-
-        return $GLOBALS['xoopsDB']->queryF($sql);
-    }
-
-    /**
-     * @param \CriteriaElement|\CriteriaCompo $criteria
-     * @param bool $id_as_key
-     * @param bool $as_object
-     * @return array
-     */
-    public function &getObjects($criteria = null, $id_as_key = false, $as_object = true): array
-    {
-        $ret = parent::getObjects($criteria, $id_as_key, $as_object);
-
-        /*if (!isset($GLOBALS['songlistAdmin'])) {
+	/**
+	 * CategoryHandler constructor.
+	 * @param \XoopsDatabase $db
+	 */
+	public function __construct(\XoopsDatabase $db)
+	{
+		parent::__construct($db, 'songlist_category', Category::class, 'cid', 'name');
+	}
+
+	/**
+	 * @return array
+	 */
+	public function filterFields(): array
+	{
+		return ['cid', 'pid', 'weight', 'name', 'image', 'path', 'artists', 'albums', 'songs', 'hits', 'rank', 'votes', 'created', 'updated'];
+	}
+
+	/**
+	 * @param $filter
+	 * @return \CriteriaCompo
+	 */
+	public function getFilterCriteria($filter): \CriteriaCompo
+	{
+		$parts    = \explode('|', $filter);
+		$criteria = new \CriteriaCompo();
+		foreach ($parts as $part) {
+			$var = \explode(',', $part);
+			if (!empty($var[1]) && !\is_numeric($var[0])) {
+				$object = $this->create();
+				if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
+					|| \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new \Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
+				} elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
+					$criteria->add(new \Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
+				} elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new \Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
+				} elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new \Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
+				}
+			} elseif (!empty($var[1]) && \is_numeric($var[0])) {
+				$criteria->add(new \Criteria($var[0], $var[1]));
+			}
+		}
+
+		return $criteria;
+	}
+
+	/**
+	 * @param        $filter
+	 * @param        $field
+	 * @param string $sort
+	 * @param string $op
+	 * @param string $fct
+	 * @return string
+	 */
+	public function getFilterForm($filter, $field, $sort = 'created', $op = 'dashboard', $fct = 'list'): string
+	{
+		$ele = Utility::getFilterElement($filter, $field, $sort, $op, $fct);
+		if (\is_object($ele)) {
+			return $ele->render();
+		}
+
+		return ' ';
+	}
+
+	/**
+	 * @param int $pid
+	 * @return mixed
+	 */
+	public function GetCatAndSubCat($pid = 0)
+	{
+		$categories  = $this->getObjects(new \Criteria('pid', $pid), true);
+		$langs_array = $this->TreeIDs([], $categories, -1);
+
+		return $langs_array;
+	}
+
+	/**
+	 * @param $langs_array
+	 * @param $categories
+	 * @param $level
+	 * @return mixed
+	 */
+	private function TreeIDs($langs_array, $categories, $level)
+	{
+		foreach ($categories as $catid => $category) {
+			$langs_array[$catid] = $catid;
+			$categoriesb         = $this->getObjects(new \Criteria('pid', $catid), true);
+			if ($categoriesb) {
+				$langs_array = $this->TreeIDs($langs_array, $categoriesb, $level);
+			}
+		}
+
+		return $langs_array;
+	}
+
+	/**
+	 * @param bool $force
+	 * @return bool|mixed
+	 */
+	public function insert(\XoopsObject $obj, $force = true)
+	{
+		if ($obj->isNew()) {
+			$obj->setVar('created', \time());
+		} else {
+			$obj->setVar('updated', \time());
+		}
+		if ('' == $obj->getVar('name')) {
+			return false;
+		}
+
+		return parent::insert($obj, $force);
+	}
+
+	public $_objects = ['object' => [], 'array' => []];
+
+	/**
+	 * @param null $id
+	 * @param null $fields
+	 * @return \XoopsObject
+	 */
+	public function get($id = null, $fields = null): \XoopsObject//get($id, $fields = '*')
+	{
+		$fields = $fields ?: '*';
+		if (!isset($this->_objects['object'][$id])) {
+			$this->_objects['object'][$id] = parent::get($id, $fields);
+			if (!isset($GLOBALS['songlistAdmin'])) {
+				$sql = 'UPDATE `' . $this->table . '` set hits=hits+1 where `' . $this->keyName . '` = ' . $id;
+				$GLOBALS['xoopsDB']->queryF($sql);
+			}
+		}
+
+		return $this->_objects['object'][$id];
+	}
+
+	/**
+	 * @param \XoopsObject $object
+	 * @param bool         $force
+	 * @return bool
+	 */
+	public function delete(\XoopsObject $object, $force = true): bool
+	{
+		parent::delete($object, $force);
+		$sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('songlist_songs') . ' SET `cid` = 0 WHERE `cid` = ' . $object->getVar('cid');
+
+		return $GLOBALS['xoopsDB']->queryF($sql);
+	}
+
+	/**
+	 * @param \CriteriaElement|\CriteriaCompo $criteria
+	 * @param bool $id_as_key
+	 * @param bool $as_object
+	 * @return array
+	 */
+	public function &getObjects($criteria = null, $id_as_key = false, $as_object = true): array
+	{
+		$ret = parent::getObjects($criteria, $id_as_key, $as_object);
+
+		/*if (!isset($GLOBALS['songlistAdmin'])) {
             $id = [];
             foreach($ret as $data) {
                 if ($as_object==true) {
@@ -186,25 +186,25 @@ 
             $GLOBALS['xoopsDB']->queryF($sql);
         }*/
 
-        return $ret;
-    }
-
-    /**
-     * @param int $limit
-     * @return array
-     */
-    public function getTop($limit = 1): array
-    {
-        $sql     = 'SELECT * FROM `' . $this->table . '` WHERE `rank`>=0 ORDER BY (`rank`/`votes`) DESC LIMIT ' . $limit;
-        $results = $GLOBALS['xoopsDB']->queryF($sql);
-        $ret     = [];
-        $i       = 0;
-        while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($results))) {
-            $ret[$i] = $this->create();
-            $ret[$i]->assignVars($row);
-            ++$i;
-        }
-
-        return $ret;
-    }
+		return $ret;
+	}
+
+	/**
+	 * @param int $limit
+	 * @return array
+	 */
+	public function getTop($limit = 1): array
+	{
+		$sql     = 'SELECT * FROM `' . $this->table . '` WHERE `rank`>=0 ORDER BY (`rank`/`votes`) DESC LIMIT ' . $limit;
+		$results = $GLOBALS['xoopsDB']->queryF($sql);
+		$ret     = [];
+		$i       = 0;
+		while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($results))) {
+			$ret[$i] = $this->create();
+			$ret[$i]->assignVars($row);
+			++$i;
+		}
+
+		return $ret;
+	}
 }

Spacing +13 added lines, -13 removed lines

@@ -36,15 +36,15 @@ 
             $var = \explode(',', $part);
             if (!empty($var[1]) && !\is_numeric($var[0])) {
                 $object = $this->create();
-                if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
-                    || \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
+                if (\XOBJ_DTYPE_TXTBOX==$object->vars[$var[0]]['data_type']
+                    || \XOBJ_DTYPE_TXTAREA==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new \Criteria('`'.$var[0].'`', '%'.$var[1].'%', ($var[2] ?? 'LIKE')));
                 } elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new \Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
+                    $criteria->add(new \Criteria('`'.$var[0].'`', $var[1], ($var[2] ?? '=')));
+                } elseif (\XOBJ_DTYPE_ENUM==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new \Criteria('`'.$var[0].'`', $var[1], ($var[2] ?? '=')));
+                } elseif (\XOBJ_DTYPE_ARRAY==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new \Criteria('`'.$var[0].'`', '%"'.$var[1].'";%', ($var[2] ?? 'LIKE')));
                 }
             } elseif (!empty($var[1]) && \is_numeric($var[0])) {
                 $criteria->add(new \Criteria($var[0], $var[1]));
@@ -114,7 +114,7 @@ 
         } else {
             $obj->setVar('updated', \time());
         }
-        if ('' == $obj->getVar('name')) {
+        if (''==$obj->getVar('name')) {
             return false;
         }
 
@@ -134,7 +134,7 @@ 
         if (!isset($this->_objects['object'][$id])) {
             $this->_objects['object'][$id] = parent::get($id, $fields);
             if (!isset($GLOBALS['songlistAdmin'])) {
-                $sql = 'UPDATE `' . $this->table . '` set hits=hits+1 where `' . $this->keyName . '` = ' . $id;
+                $sql = 'UPDATE `'.$this->table.'` set hits=hits+1 where `'.$this->keyName.'` = '.$id;
                 $GLOBALS['xoopsDB']->queryF($sql);
             }
         }
@@ -150,7 +150,7 @@ 
     public function delete(\XoopsObject $object, $force = true): bool
     {
         parent::delete($object, $force);
-        $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('songlist_songs') . ' SET `cid` = 0 WHERE `cid` = ' . $object->getVar('cid');
+        $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('songlist_songs').' SET `cid` = 0 WHERE `cid` = '.$object->getVar('cid');
 
         return $GLOBALS['xoopsDB']->queryF($sql);
     }
@@ -195,11 +195,11 @@ 
      */
     public function getTop($limit = 1): array
     {
-        $sql     = 'SELECT * FROM `' . $this->table . '` WHERE `rank`>=0 ORDER BY (`rank`/`votes`) DESC LIMIT ' . $limit;
+        $sql     = 'SELECT * FROM `'.$this->table.'` WHERE `rank`>=0 ORDER BY (`rank`/`votes`) DESC LIMIT '.$limit;
         $results = $GLOBALS['xoopsDB']->queryF($sql);
         $ret     = [];
         $i       = 0;
-        while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($results))) {
+        while (false!==($row = $GLOBALS['xoopsDB']->fetchArray($results))) {
             $ret[$i] = $this->create();
             $ret[$i]->assignVars($row);
             ++$i;

class/Requests.php

Spacing +3 added lines, -3 removed lines

@@ -5,7 +5,7 @@ 
 use XoopsFormText;
 use XoopsObject;
 
-require_once \dirname(__DIR__) . '/include/songlist.object.php';
+require_once \dirname(__DIR__).'/include/songlist.object.php';
 // require_once \dirname(__DIR__) . '/include/songlist.form.php';
 use  XoopsModules\Songlist\Form\FormController;
 
@@ -51,12 +51,12 @@ 
     {
         $ret            = parent::toArray();
         $form           = $this->getForm(true);
-        $form['songid'] = new XoopsFormText('', $this->getVar('rid') . '[songid]', 11, 32);
+        $form['songid'] = new XoopsFormText('', $this->getVar('rid').'[songid]', 11, 32);
         foreach ($form as $key => $element) {
             $ret['form'][$key] = $element->render();
         }
         foreach (['created', 'updated'] as $key) {
-            if ($this->getVar($key) > 0) {
+            if ($this->getVar($key)>0) {
                 $ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
                 $ret[$key]         = \date(_DATESTRING, $this->getVar($key));
             }

Indentation +60 added lines, -60 removed lines

@@ -15,68 +15,68 @@
  */
 class Requests extends XoopsObject
 {
-    public $rid;
-    public $aid;
-    public $artist;
-    public $album;
-    public $title;
-    public $lyrics;
-    public $uid;
-    public $name;
-    public $email;
-    public $songid;
-    public $sid;
-    public $created;
-    public $updated;
+	public $rid;
+	public $aid;
+	public $artist;
+	public $album;
+	public $title;
+	public $lyrics;
+	public $uid;
+	public $name;
+	public $email;
+	public $songid;
+	public $sid;
+	public $created;
+	public $updated;
 
-    /**
-     * Requests constructor.
-     * @param null $fid
-     */
-    public function __construct($fid = null)
-    {
-        $this->initVar('rid', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('aid', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('artist', \XOBJ_DTYPE_TXTBOX, null, false, 128);
-        $this->initVar('album', \XOBJ_DTYPE_TXTBOX, null, false, 128);
-        $this->initVar('title', \XOBJ_DTYPE_TXTBOX, null, false, 128);
-        $this->initVar('lyrics', \XOBJ_DTYPE_OTHER, null, false);
-        $this->initVar('uid', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('name', \XOBJ_DTYPE_TXTBOX, null, false, 128);
-        $this->initVar('email', \XOBJ_DTYPE_TXTBOX, null, false, 255);
-        $this->initVar('songid', \XOBJ_DTYPE_TXTBOX, null, false, 32);
-        $this->initVar('sid', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('created', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('updated', \XOBJ_DTYPE_INT, 0, false);
-    }
+	/**
+	 * Requests constructor.
+	 * @param null $fid
+	 */
+	public function __construct($fid = null)
+	{
+		$this->initVar('rid', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('aid', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('artist', \XOBJ_DTYPE_TXTBOX, null, false, 128);
+		$this->initVar('album', \XOBJ_DTYPE_TXTBOX, null, false, 128);
+		$this->initVar('title', \XOBJ_DTYPE_TXTBOX, null, false, 128);
+		$this->initVar('lyrics', \XOBJ_DTYPE_OTHER, null, false);
+		$this->initVar('uid', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('name', \XOBJ_DTYPE_TXTBOX, null, false, 128);
+		$this->initVar('email', \XOBJ_DTYPE_TXTBOX, null, false, 255);
+		$this->initVar('songid', \XOBJ_DTYPE_TXTBOX, null, false, 32);
+		$this->initVar('sid', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('created', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('updated', \XOBJ_DTYPE_INT, 0, false);
+	}
 
-    /**
-     * @param bool $as_array
-     * @return array|string
-     */
-    public function getForm($as_array = false)
-    {
-        return FormController::getFormRequests($this, $as_array);
-    }
+	/**
+	 * @param bool $as_array
+	 * @return array|string
+	 */
+	public function getForm($as_array = false)
+	{
+		return FormController::getFormRequests($this, $as_array);
+	}
 
-    /**
-     * @return array
-     */
-    public function toArray(): array
-    {
-        $ret            = parent::toArray();
-        $form           = $this->getForm(true);
-        $form['songid'] = new XoopsFormText('', $this->getVar('rid') . '[songid]', 11, 32);
-        foreach ($form as $key => $element) {
-            $ret['form'][$key] = $element->render();
-        }
-        foreach (['created', 'updated'] as $key) {
-            if ($this->getVar($key) > 0) {
-                $ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
-                $ret[$key]         = \date(_DATESTRING, $this->getVar($key));
-            }
-        }
+	/**
+	 * @return array
+	 */
+	public function toArray(): array
+	{
+		$ret            = parent::toArray();
+		$form           = $this->getForm(true);
+		$form['songid'] = new XoopsFormText('', $this->getVar('rid') . '[songid]', 11, 32);
+		foreach ($form as $key => $element) {
+			$ret['form'][$key] = $element->render();
+		}
+		foreach (['created', 'updated'] as $key) {
+			if ($this->getVar($key) > 0) {
+				$ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
+				$ret[$key]         = \date(_DATESTRING, $this->getVar($key));
+			}
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

class/VisibilityHandler.php

Indentation +73 added lines, -73 removed lines

@@ -27,81 +27,81 @@
  */
 class VisibilityHandler extends XoopsPersistableObjectHandler
 {
-    /**
-     * VisibilityHandler constructor.
-     * @param \XoopsDatabase $db
-     */
-    public function __construct(XoopsDatabase $db)
-    {
-        parent::__construct($db, 'songlist_visibility', Visibility::class, 'field_id');
-    }
+	/**
+	 * VisibilityHandler constructor.
+	 * @param \XoopsDatabase $db
+	 */
+	public function __construct(XoopsDatabase $db)
+	{
+		parent::__construct($db, 'songlist_visibility', Visibility::class, 'field_id');
+	}
 
-    /**
-     * get all objects matching a condition
-     *
-     * @param \CriteriaElement|null $criteria {@link \CriteriaElement} to match
-     * @param null                  $fields
-     * @param bool                  $asObject
-     * @param bool                  $id_as_key
-     * @return array of objects/array <a href='psi_element://XoopsObject'>XoopsObject</a>
-     * @internal param array $fields variables to fetch
-     * @internal param bool $asObject flag indicating as object, otherwise as array
-     * @internal param bool $id_as_key use the ID as key for the array
-     */
-    public function &getAll(CriteriaElement $criteria = null, $fields = null, $asObject = true, $id_as_key = true): array //getAll($criteria = null)
-    {
-        $limit            = null;
-        $GLOBALS['start'] = null;
-        $sql              = "SELECT * FROM `{$this->table}`";
-        if (($criteria instanceof \CriteriaCompo) || ($criteria instanceof \Criteria)) {
-            $sql     .= ' ' . $criteria->renderWhere();
-            $groupby = $criteria->getGroupby();
-            if ($groupby) {
-                $sql .= ' ' . $groupby;
-            }
-            $sort = $criteria->getSort();
-            if ($sort) {
-                $sql      .= " ORDER BY {$sort} " . $criteria->getOrder();
-                $orderSet = true;
-            }
-            $limit            = $criteria->getLimit();
-            $GLOBALS['start'] = $criteria->getStart();
-        }
-        if (empty($orderSet)) {
-            $sql .= " ORDER BY `{$this->keyName}` DESC";
-        }
-        $result = $this->db->query($sql, $limit, $GLOBALS['start']);
-        $ret    = [];
-        while (false !== ($row = $this->db->fetchArray($result))) {
-            $ret[$row['field_id']][] = $row;
-        }
+	/**
+	 * get all objects matching a condition
+	 *
+	 * @param \CriteriaElement|null $criteria {@link \CriteriaElement} to match
+	 * @param null                  $fields
+	 * @param bool                  $asObject
+	 * @param bool                  $id_as_key
+	 * @return array of objects/array <a href='psi_element://XoopsObject'>XoopsObject</a>
+	 * @internal param array $fields variables to fetch
+	 * @internal param bool $asObject flag indicating as object, otherwise as array
+	 * @internal param bool $id_as_key use the ID as key for the array
+	 */
+	public function &getAll(CriteriaElement $criteria = null, $fields = null, $asObject = true, $id_as_key = true): array //getAll($criteria = null)
+	{
+		$limit            = null;
+		$GLOBALS['start'] = null;
+		$sql              = "SELECT * FROM `{$this->table}`";
+		if (($criteria instanceof \CriteriaCompo) || ($criteria instanceof \Criteria)) {
+			$sql     .= ' ' . $criteria->renderWhere();
+			$groupby = $criteria->getGroupby();
+			if ($groupby) {
+				$sql .= ' ' . $groupby;
+			}
+			$sort = $criteria->getSort();
+			if ($sort) {
+				$sql      .= " ORDER BY {$sort} " . $criteria->getOrder();
+				$orderSet = true;
+			}
+			$limit            = $criteria->getLimit();
+			$GLOBALS['start'] = $criteria->getStart();
+		}
+		if (empty($orderSet)) {
+			$sql .= " ORDER BY `{$this->keyName}` DESC";
+		}
+		$result = $this->db->query($sql, $limit, $GLOBALS['start']);
+		$ret    = [];
+		while (false !== ($row = $this->db->fetchArray($result))) {
+			$ret[$row['field_id']][] = $row;
+		}
 
-        return $ret;
-    }
+		return $ret;
+	}
 
-    /**
-     * Get fields visible to the $user_groups on a $profile_groups profile
-     *
-     * @param array $profile_groups groups of the user to be accessed
-     * @param array $user_groups    groups of the visitor, default as $GLOBALS['xoopsUser']
-     *
-     * @return array
-     */
-    public function getVisibleFields($profile_groups, $user_groups = []): array
-    {
-        $profile_groups[] = '0';
-        $user_groups[]    = '0';
-        $profile_groups[] = $user_groups[] = 0;
-        $sql              = "SELECT field_id FROM {$this->table} WHERE profile_group IN (" . \implode(',', $profile_groups) . ')';
-        $sql              .= ' AND user_group IN (' . \implode(',', $user_groups) . ')';
-        $field_ids        = [];
-        $result           = $this->db->query($sql);
-        if ($result) {
-            while ([$field_id] = $this->db->fetchRow($result)) {
-                $field_ids[] = $field_id;
-            }
-        }
+	/**
+	 * Get fields visible to the $user_groups on a $profile_groups profile
+	 *
+	 * @param array $profile_groups groups of the user to be accessed
+	 * @param array $user_groups    groups of the visitor, default as $GLOBALS['xoopsUser']
+	 *
+	 * @return array
+	 */
+	public function getVisibleFields($profile_groups, $user_groups = []): array
+	{
+		$profile_groups[] = '0';
+		$user_groups[]    = '0';
+		$profile_groups[] = $user_groups[] = 0;
+		$sql              = "SELECT field_id FROM {$this->table} WHERE profile_group IN (" . \implode(',', $profile_groups) . ')';
+		$sql              .= ' AND user_group IN (' . \implode(',', $user_groups) . ')';
+		$field_ids        = [];
+		$result           = $this->db->query($sql);
+		if ($result) {
+			while ([$field_id] = $this->db->fetchRow($result)) {
+				$field_ids[] = $field_id;
+			}
+		}
 
-        return $field_ids;
-    }
+		return $field_ids;
+	}
 }

Spacing +6 added lines, -6 removed lines

@@ -54,14 +54,14 @@ 
         $GLOBALS['start'] = null;
         $sql              = "SELECT * FROM `{$this->table}`";
         if (($criteria instanceof \CriteriaCompo) || ($criteria instanceof \Criteria)) {
-            $sql     .= ' ' . $criteria->renderWhere();
+            $sql     .= ' '.$criteria->renderWhere();
             $groupby = $criteria->getGroupby();
             if ($groupby) {
-                $sql .= ' ' . $groupby;
+                $sql .= ' '.$groupby;
             }
             $sort = $criteria->getSort();
             if ($sort) {
-                $sql      .= " ORDER BY {$sort} " . $criteria->getOrder();
+                $sql .= " ORDER BY {$sort} ".$criteria->getOrder();
                 $orderSet = true;
             }
             $limit            = $criteria->getLimit();
@@ -72,7 +72,7 @@ 
         }
         $result = $this->db->query($sql, $limit, $GLOBALS['start']);
         $ret    = [];
-        while (false !== ($row = $this->db->fetchArray($result))) {
+        while (false!==($row = $this->db->fetchArray($result))) {
             $ret[$row['field_id']][] = $row;
         }
 
@@ -92,8 +92,8 @@ 
         $profile_groups[] = '0';
         $user_groups[]    = '0';
         $profile_groups[] = $user_groups[] = 0;
-        $sql              = "SELECT field_id FROM {$this->table} WHERE profile_group IN (" . \implode(',', $profile_groups) . ')';
-        $sql              .= ' AND user_group IN (' . \implode(',', $user_groups) . ')';
+        $sql              = "SELECT field_id FROM {$this->table} WHERE profile_group IN (".\implode(',', $profile_groups).')';
+        $sql .= ' AND user_group IN ('.\implode(',', $user_groups).')';
         $field_ids        = [];
         $result           = $this->db->query($sql);
         if ($result) {

class/VotesHandler.php

Indentation +136 added lines, -136 removed lines

@@ -17,140 +17,140 @@
  */
 class VotesHandler extends XoopsPersistableObjectHandler
 {
-    /**
-     * VotesHandler constructor.
-     * @param \XoopsDatabase $db
-     */
-    public function __construct(XoopsDatabase $db)
-    {
-        parent::__construct($db, 'songlist_votes', Votes::class, 'vid', 'ip');
-    }
-
-    /**
-     * @return array
-     */
-    public function filterFields(): array
-    {
-        return ['vid', 'sid', 'uid', 'ip', 'netaddy', 'rank'];
-    }
-
-    /**
-     * @param string $filter
-     * @return \CriteriaCompo
-     */
-    public function getFilterCriteria($filter): CriteriaCompo
-    {
-        $parts    = \explode('|', $filter);
-        $criteria = new CriteriaCompo();
-        foreach ($parts as $part) {
-            $var = \explode(',', $part);
-            if (!empty($var[1]) && !\is_numeric($var[0])) {
-                $object = $this->create();
-                if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
-                    || \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
-                } elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
-                }
-            } elseif (!empty($var[1]) && \is_numeric($var[0])) {
-                $criteria->add(new Criteria($var[0], $var[1]));
-            }
-        }
-
-        return $criteria;
-    }
-
-    /**
-     * @param string $filter
-     * @param string $field
-     * @param string $sort
-     * @param string $op
-     * @param string $fct
-     * @return string
-     */
-    public function getFilterForm($filter, $field, $sort = 'created', $op = 'dashboard', $fct = 'list'): string
-    {
-        $ele = Utility::getFilterElement($filter, $field, $sort, $op, $fct);
-        if (\is_object($ele)) {
-            return $ele->render();
-        }
-
-        return ' ';
-    }
-
-    /**
-     * @param bool $force
-     * @return mixed
-     */
-    public function insert(XoopsObject $obj, $force = true)
-    {
-        if ($obj->isNew()) {
-            $obj->setVar('created', \time());
-        } else {
-            $obj->setVar('updated', \time());
-        }
-
-        return parent::insert($obj, $force);
-    }
-
-    /**
-     * @param int $sid
-     * @param string$value
-     * @return bool
-     */
-    public function addVote($sid, $value): bool
-    {
-        $criteria = new CriteriaCompo(new Criteria('sid', $sid));
-
-        $ip = Utility::getIPData(false);
-        if ($ip['uid'] > 0) {
-            $criteria->add(new Criteria('uid', $ip['uid']));
-        } else {
-            $criteria->add(new Criteria('ip', $ip['ip']));
-            $criteria->add(new Criteria('netaddy', $ip['network-addy']));
-        }
-
-        if (0 == $this->getCount($criteria) && $sid > 0 && $value > 0) {
-            $vote = $this->create();
-            $vote->setVar('sid', $sid);
-            $vote->setVar('uid', $ip['uid']);
-            $vote->setVar('ip', $ip['ip']);
-            $vote->setVar('netaddy', $ip['network-addy']);
-            $vote->setVar('rank', $value);
-            if ($this->insert($vote)) {
-                $songsHandler    = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Songs');
-                $albumsHandler   = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Albums');
-                $artistsHandler  = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Artists');
-                $categoryHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Category');
-                $genreHandler    = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Genre');
-                $voiceHandler    = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Voice');
-
-                $song  = $songsHandler->get($sid);
-                $sql   = [];
-                $sql[] = 'UPDATE `' . $songsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $songsHandler->keyName . '` = ' . $sid;
-                $sql[] = 'UPDATE `' . $categoryHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $categoryHandler->keyName . '` = ' . $song->getVar($categoryHandler->keyName);
-                $sql[] = 'UPDATE `' . $genreHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $genreHandler->keyName . '` = ' . $song->getVar($genreHandler->keyName);
-                $sql[] = 'UPDATE `' . $voiceHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $voiceHandler->keyName . '` = ' . $song->getVar($voiceHandler->keyName);
-                $sql[] = 'UPDATE `' . $albumsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $albumsHandler->keyName . '` = ' . $song->getVar($albumsHandler->keyName);
-                foreach ($song->getVar('aids') as $aid) {
-                    $sql[] = 'UPDATE `' . $artistsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $artistsHandler->keyName . '` = ' . $aid;
-                }
-                foreach ($sql as $question) {
-                    $GLOBALS['xoopsDB']->queryF($question);
-                }
-                \redirect_header($_POST['uri'], 10, \_MD_SONGLIST_MSG_VOTED_FINISHED);
-                exit(0);
-            }
-            \redirect_header($_POST['uri'], 10, \_MD_SONGLIST_MSG_VOTED_ALREADY);
-            exit(0);
-        }
-        \redirect_header($_POST['uri'], 10, \_MD_SONGLIST_MSG_VOTED_SOMETHINGWRONG);
-        exit(0);
-
-        return false;
-    }
+	/**
+	 * VotesHandler constructor.
+	 * @param \XoopsDatabase $db
+	 */
+	public function __construct(XoopsDatabase $db)
+	{
+		parent::__construct($db, 'songlist_votes', Votes::class, 'vid', 'ip');
+	}
+
+	/**
+	 * @return array
+	 */
+	public function filterFields(): array
+	{
+		return ['vid', 'sid', 'uid', 'ip', 'netaddy', 'rank'];
+	}
+
+	/**
+	 * @param string $filter
+	 * @return \CriteriaCompo
+	 */
+	public function getFilterCriteria($filter): CriteriaCompo
+	{
+		$parts    = \explode('|', $filter);
+		$criteria = new CriteriaCompo();
+		foreach ($parts as $part) {
+			$var = \explode(',', $part);
+			if (!empty($var[1]) && !\is_numeric($var[0])) {
+				$object = $this->create();
+				if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
+					|| \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
+				} elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
+				} elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
+				} elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
+					$criteria->add(new Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
+				}
+			} elseif (!empty($var[1]) && \is_numeric($var[0])) {
+				$criteria->add(new Criteria($var[0], $var[1]));
+			}
+		}
+
+		return $criteria;
+	}
+
+	/**
+	 * @param string $filter
+	 * @param string $field
+	 * @param string $sort
+	 * @param string $op
+	 * @param string $fct
+	 * @return string
+	 */
+	public function getFilterForm($filter, $field, $sort = 'created', $op = 'dashboard', $fct = 'list'): string
+	{
+		$ele = Utility::getFilterElement($filter, $field, $sort, $op, $fct);
+		if (\is_object($ele)) {
+			return $ele->render();
+		}
+
+		return ' ';
+	}
+
+	/**
+	 * @param bool $force
+	 * @return mixed
+	 */
+	public function insert(XoopsObject $obj, $force = true)
+	{
+		if ($obj->isNew()) {
+			$obj->setVar('created', \time());
+		} else {
+			$obj->setVar('updated', \time());
+		}
+
+		return parent::insert($obj, $force);
+	}
+
+	/**
+	 * @param int $sid
+	 * @param string$value
+	 * @return bool
+	 */
+	public function addVote($sid, $value): bool
+	{
+		$criteria = new CriteriaCompo(new Criteria('sid', $sid));
+
+		$ip = Utility::getIPData(false);
+		if ($ip['uid'] > 0) {
+			$criteria->add(new Criteria('uid', $ip['uid']));
+		} else {
+			$criteria->add(new Criteria('ip', $ip['ip']));
+			$criteria->add(new Criteria('netaddy', $ip['network-addy']));
+		}
+
+		if (0 == $this->getCount($criteria) && $sid > 0 && $value > 0) {
+			$vote = $this->create();
+			$vote->setVar('sid', $sid);
+			$vote->setVar('uid', $ip['uid']);
+			$vote->setVar('ip', $ip['ip']);
+			$vote->setVar('netaddy', $ip['network-addy']);
+			$vote->setVar('rank', $value);
+			if ($this->insert($vote)) {
+				$songsHandler    = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Songs');
+				$albumsHandler   = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Albums');
+				$artistsHandler  = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Artists');
+				$categoryHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Category');
+				$genreHandler    = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Genre');
+				$voiceHandler    = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Voice');
+
+				$song  = $songsHandler->get($sid);
+				$sql   = [];
+				$sql[] = 'UPDATE `' . $songsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $songsHandler->keyName . '` = ' . $sid;
+				$sql[] = 'UPDATE `' . $categoryHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $categoryHandler->keyName . '` = ' . $song->getVar($categoryHandler->keyName);
+				$sql[] = 'UPDATE `' . $genreHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $genreHandler->keyName . '` = ' . $song->getVar($genreHandler->keyName);
+				$sql[] = 'UPDATE `' . $voiceHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $voiceHandler->keyName . '` = ' . $song->getVar($voiceHandler->keyName);
+				$sql[] = 'UPDATE `' . $albumsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $albumsHandler->keyName . '` = ' . $song->getVar($albumsHandler->keyName);
+				foreach ($song->getVar('aids') as $aid) {
+					$sql[] = 'UPDATE `' . $artistsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $artistsHandler->keyName . '` = ' . $aid;
+				}
+				foreach ($sql as $question) {
+					$GLOBALS['xoopsDB']->queryF($question);
+				}
+				\redirect_header($_POST['uri'], 10, \_MD_SONGLIST_MSG_VOTED_FINISHED);
+				exit(0);
+			}
+			\redirect_header($_POST['uri'], 10, \_MD_SONGLIST_MSG_VOTED_ALREADY);
+			exit(0);
+		}
+		\redirect_header($_POST['uri'], 10, \_MD_SONGLIST_MSG_VOTED_SOMETHINGWRONG);
+		exit(0);
+
+		return false;
+	}
 }

Spacing +17 added lines, -17 removed lines

@@ -8,7 +8,7 @@ 
 use XoopsObject;
 use XoopsPersistableObjectHandler;
 
-require_once \dirname(__DIR__) . '/include/songlist.object.php';
+require_once \dirname(__DIR__).'/include/songlist.object.php';
 // require_once \dirname(__DIR__) . '/include/songlist.form.php';
 use  XoopsModules\Songlist\Form\FormController;
 
@@ -46,15 +46,15 @@ 
             $var = \explode(',', $part);
             if (!empty($var[1]) && !\is_numeric($var[0])) {
                 $object = $this->create();
-                if (\XOBJ_DTYPE_TXTBOX == $object->vars[$var[0]]['data_type']
-                    || \XOBJ_DTYPE_TXTAREA == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%' . $var[1] . '%', ($var[2] ?? 'LIKE')));
+                if (\XOBJ_DTYPE_TXTBOX==$object->vars[$var[0]]['data_type']
+                    || \XOBJ_DTYPE_TXTAREA==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new Criteria('`'.$var[0].'`', '%'.$var[1].'%', ($var[2] ?? 'LIKE')));
                 } elseif (in_array($object->vars[$var[0]]['data_type'], [XOBJ_DTYPE_INT, XOBJ_DTYPE_DECIMAL, XOBJ_DTYPE_FLOAT])) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ENUM == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', $var[1], ($var[2] ?? '=')));
-                } elseif (\XOBJ_DTYPE_ARRAY == $object->vars[$var[0]]['data_type']) {
-                    $criteria->add(new Criteria('`' . $var[0] . '`', '%"' . $var[1] . '";%', ($var[2] ?? 'LIKE')));
+                    $criteria->add(new Criteria('`'.$var[0].'`', $var[1], ($var[2] ?? '=')));
+                } elseif (\XOBJ_DTYPE_ENUM==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new Criteria('`'.$var[0].'`', $var[1], ($var[2] ?? '=')));
+                } elseif (\XOBJ_DTYPE_ARRAY==$object->vars[$var[0]]['data_type']) {
+                    $criteria->add(new Criteria('`'.$var[0].'`', '%"'.$var[1].'";%', ($var[2] ?? 'LIKE')));
                 }
             } elseif (!empty($var[1]) && \is_numeric($var[0])) {
                 $criteria->add(new Criteria($var[0], $var[1]));
@@ -107,14 +107,14 @@ 
         $criteria = new CriteriaCompo(new Criteria('sid', $sid));
 
         $ip = Utility::getIPData(false);
-        if ($ip['uid'] > 0) {
+        if ($ip['uid']>0) {
             $criteria->add(new Criteria('uid', $ip['uid']));
         } else {
             $criteria->add(new Criteria('ip', $ip['ip']));
             $criteria->add(new Criteria('netaddy', $ip['network-addy']));
         }
 
-        if (0 == $this->getCount($criteria) && $sid > 0 && $value > 0) {
+        if (0==$this->getCount($criteria) && $sid>0 && $value>0) {
             $vote = $this->create();
             $vote->setVar('sid', $sid);
             $vote->setVar('uid', $ip['uid']);
@@ -131,13 +131,13 @@ 
 
                 $song  = $songsHandler->get($sid);
                 $sql   = [];
-                $sql[] = 'UPDATE `' . $songsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $songsHandler->keyName . '` = ' . $sid;
-                $sql[] = 'UPDATE `' . $categoryHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $categoryHandler->keyName . '` = ' . $song->getVar($categoryHandler->keyName);
-                $sql[] = 'UPDATE `' . $genreHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $genreHandler->keyName . '` = ' . $song->getVar($genreHandler->keyName);
-                $sql[] = 'UPDATE `' . $voiceHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $voiceHandler->keyName . '` = ' . $song->getVar($voiceHandler->keyName);
-                $sql[] = 'UPDATE `' . $albumsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $albumsHandler->keyName . '` = ' . $song->getVar($albumsHandler->keyName);
+                $sql[] = 'UPDATE `'.$songsHandler->table.'` SET `rank` = `rank` + '.$value.', `votes` = `votes` + 1 WHERE `'.$songsHandler->keyName.'` = '.$sid;
+                $sql[] = 'UPDATE `'.$categoryHandler->table.'` SET `rank` = `rank` + '.$value.', `votes` = `votes` + 1 WHERE `'.$categoryHandler->keyName.'` = '.$song->getVar($categoryHandler->keyName);
+                $sql[] = 'UPDATE `'.$genreHandler->table.'` SET `rank` = `rank` + '.$value.', `votes` = `votes` + 1 WHERE `'.$genreHandler->keyName.'` = '.$song->getVar($genreHandler->keyName);
+                $sql[] = 'UPDATE `'.$voiceHandler->table.'` SET `rank` = `rank` + '.$value.', `votes` = `votes` + 1 WHERE `'.$voiceHandler->keyName.'` = '.$song->getVar($voiceHandler->keyName);
+                $sql[] = 'UPDATE `'.$albumsHandler->table.'` SET `rank` = `rank` + '.$value.', `votes` = `votes` + 1 WHERE `'.$albumsHandler->keyName.'` = '.$song->getVar($albumsHandler->keyName);
                 foreach ($song->getVar('aids') as $aid) {
-                    $sql[] = 'UPDATE `' . $artistsHandler->table . '` SET `rank` = `rank` + ' . $value . ', `votes` = `votes` + 1 WHERE `' . $artistsHandler->keyName . '` = ' . $aid;
+                    $sql[] = 'UPDATE `'.$artistsHandler->table.'` SET `rank` = `rank` + '.$value.', `votes` = `votes` + 1 WHERE `'.$artistsHandler->keyName.'` = '.$aid;
                 }
                 foreach ($sql as $question) {
                     $GLOBALS['xoopsDB']->queryF($question);

class/Albums.php

Spacing +15 added lines, -15 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace XoopsModules\Songlist;
 
-require_once \dirname(__DIR__) . '/include/songlist.object.php';
+require_once \dirname(__DIR__).'/include/songlist.object.php';
 // require_once \dirname(__DIR__) . '/include/songlist.form.php';
 use  XoopsModules\Songlist\Form\FormController;
 
@@ -54,20 +54,20 @@ 
             $ret['form'][$key] = $element->render();
         }
         foreach (['created', 'updated'] as $key) {
-            if ($this->getVar($key) > 0) {
+            if ($this->getVar($key)>0) {
                 $ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
                 $ret[$key]         = \date(_DATESTRING, $this->getVar($key));
             }
         }
         $ret['picture'] = $this->getImage('image', false);
-        $ret['rank']    = \number_format(($this->getVar('rank') > 0 && $this->getVar('votes') > 0 ? $this->getVar('rank') / $this->getVar('votes') : 0), 2) . \_MI_SONGLIST_OFTEN;
+        $ret['rank']    = \number_format(($this->getVar('rank')>0 && $this->getVar('votes')>0 ? $this->getVar('rank')/$this->getVar('votes') : 0), 2).\_MI_SONGLIST_OFTEN;
         $ret['url']     = $this->getURL(true);
 
         if (!$extra) {
             return $ret;
         }
 
-        if (0 != $this->getVar('cid')) {
+        if (0!=$this->getVar('cid')) {
             $categoryHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Category');
             $category        = $categoryHandler->get($this->getVar('cid'));
             if (\is_object($category)) {
@@ -75,7 +75,7 @@ 
             }
         }
 
-        if (0 != \count($this->getVar('aids'))) {
+        if (0!=\count($this->getVar('aids'))) {
             $artistsHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Artists');
             foreach ($this->getVar('aids') as $aid) {
                 $artist = $artistsHandler->get($aid);
@@ -85,9 +85,9 @@ 
             }
         }
 
-        if (0 != \count($this->getVar('sids'))) {
+        if (0!=\count($this->getVar('sids'))) {
             $songsHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Songs');
-            $criteria     = new \Criteria('sid', '(' . \implode(',', $this->getVar('sids')) . ')', 'IN');
+            $criteria     = new \Criteria('sid', '('.\implode(',', $this->getVar('sids')).')', 'IN');
             $criteria->setSort('traxid');
             $criteria->setOrder('ASC');
             foreach ($songsHandler->getObjects($criteria, true) as $sid => $song) {
@@ -107,17 +107,17 @@ 
      */
     public function getImage($field = 'image', $local = false)
     {
-        if ('' == $this->getVar($field)) {
+        if (''==$this->getVar($field)) {
             return false;
         }
-        if (!\file_exists($GLOBALS['xoops']->path($this->getVar('path') . $this->getVar($field)))) {
+        if (!\file_exists($GLOBALS['xoops']->path($this->getVar('path').$this->getVar($field)))) {
             return false;
         }
         if (!$local) {
-            return XOOPS_URL . '/' . \str_replace(DS, '/', $this->getVar('path')) . $this->getVar($field);
+            return XOOPS_URL.'/'.\str_replace(DS, '/', $this->getVar('path')).$this->getVar($field);
         }
 
-        return XOOPS_ROOT_PATH . DS . $this->getVar('path') . $this->getVar($field);
+        return XOOPS_ROOT_PATH.DS.$this->getVar('path').$this->getVar($field);
     }
 
     /**
@@ -127,7 +127,7 @@ 
     {
         global $file, $op, $fct, $id, $value, $gid, $vid, $vcid, $cid, $start, $limit;
         if ($GLOBALS['songlistModuleConfig']['htaccess']) {
-            if (0 != $id) {
+            if (0!=$id) {
                 $artistHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Albums');
                 $artist        = $artistHandler->get($id);
                 if (\is_object($artist) && !$artist->isNew()) {
@@ -153,12 +153,12 @@ 
                            . $GLOBALS['songlistModuleConfig']['endofurl'];
                 }
 
-                return XOOPS_URL . '/' . $GLOBALS['songlistModuleConfig']['baseofurl'] . '/albums/' . $start . '-' . $id . '-' . $op . '-' . $fct . '-' . $gid . '-' . $cid . '/' . \urlencode($value) . $GLOBALS['songlistModuleConfig']['endofurl'];
+                return XOOPS_URL.'/'.$GLOBALS['songlistModuleConfig']['baseofurl'].'/albums/'.$start.'-'.$id.'-'.$op.'-'.$fct.'-'.$gid.'-'.$cid.'/'.\urlencode($value).$GLOBALS['songlistModuleConfig']['endofurl'];
             }
 
-            return XOOPS_URL . '/' . $GLOBALS['songlistModuleConfig']['baseofurl'] . '/albums/' . $start . '-' . $id . '-' . $op . '-' . $fct . '-' . $gid . '-' . $cid . '/' . \urlencode($value) . $GLOBALS['songlistModuleConfig']['endofurl'];
+            return XOOPS_URL.'/'.$GLOBALS['songlistModuleConfig']['baseofurl'].'/albums/'.$start.'-'.$id.'-'.$op.'-'.$fct.'-'.$gid.'-'.$cid.'/'.\urlencode($value).$GLOBALS['songlistModuleConfig']['endofurl'];
         }
 
-        return XOOPS_URL . '/modules/songlist/albums.php?op=' . $op . '&fct=' . $fct . '&id=' . $id . '&value=' . \urlencode($value ?? '') . '&gid=' . $gid . '&vid=' . $vid . '&cid=' . $cid . '&start=' . $start;
+        return XOOPS_URL.'/modules/songlist/albums.php?op='.$op.'&fct='.$fct.'&id='.$id.'&value='.\urlencode($value ?? '').'&gid='.$gid.'&vid='.$vid.'&cid='.$cid.'&start='.$start;
     }
 }

Indentation +165 added lines, -165 removed lines

@@ -12,169 +12,169 @@
  */
 class Albums extends \XoopsObject
 {
-    public $abid;
-    public $cid;
-    public $aids;
-    public $sids;
-    public $title;
-    public $image;
-    public $path;
-    public $artists;
-    public $songs;
-    public $hits;
-    public $rank;
-    public $votes;
-    public $created;
-    public $updated;
-
-    /**
-     * Albums constructor.
-     * @param null $fid
-     */
-    public function __construct($fid = null)
-    {
-        $this->initVar('abid', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('cid', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('aids', \XOBJ_DTYPE_ARRAY, [], false);
-        $this->initVar('sids', \XOBJ_DTYPE_ARRAY, [], false);
-        $this->initVar('title', \XOBJ_DTYPE_TXTBOX, null, false, 128);
-        $this->initVar('image', \XOBJ_DTYPE_TXTBOX, null, false, 255);
-        $this->initVar('path', \XOBJ_DTYPE_TXTBOX, null, false, 255);
-        $this->initVar('artists', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('songs', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('hits', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('rank', \XOBJ_DTYPE_DECIMAL, 0, false);
-        $this->initVar('votes', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('created', \XOBJ_DTYPE_INT, 0, false);
-        $this->initVar('updated', \XOBJ_DTYPE_INT, 0, false);
-    }
-
-    /**
-     * @param bool $as_array
-     * @return array|string
-     */
-    public function getForm($as_array = false)
-    {
-        return FormController::getFormAlbums($this, $as_array);
-    }
-
-    /**
-     * @param bool $extra
-     * @return array
-     */
-    public function toArray($extra = true): array
-    {
-        $ret  = parent::toArray();
-        $form = $this->getForm(true);
-        foreach ($form as $key => $element) {
-            $ret['form'][$key] = $element->render();
-        }
-        foreach (['created', 'updated'] as $key) {
-            if ($this->getVar($key) > 0) {
-                $ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
-                $ret[$key]         = \date(_DATESTRING, $this->getVar($key));
-            }
-        }
-        $ret['picture'] = $this->getImage('image', false);
-        $ret['rank']    = \number_format(($this->getVar('rank') > 0 && $this->getVar('votes') > 0 ? $this->getVar('rank') / $this->getVar('votes') : 0), 2) . \_MI_SONGLIST_OFTEN;
-        $ret['url']     = $this->getURL(true);
-
-        if (!$extra) {
-            return $ret;
-        }
-
-        if (0 != $this->getVar('cid')) {
-            $categoryHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Category');
-            $category        = $categoryHandler->get($this->getVar('cid'));
-            if (\is_object($category)) {
-                $ret['category'] = $category->toArray(false);
-            }
-        }
-
-        if (0 != \count($this->getVar('aids'))) {
-            $artistsHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Artists');
-            foreach ($this->getVar('aids') as $aid) {
-                $artist = $artistsHandler->get($aid);
-                if (\is_object($artist)) {
-                    $ret['artists_array'][$aid] = $artist->toArray(false);
-                }
-            }
-        }
-
-        if (0 != \count($this->getVar('sids'))) {
-            $songsHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Songs');
-            $criteria     = new \Criteria('sid', '(' . \implode(',', $this->getVar('sids')) . ')', 'IN');
-            $criteria->setSort('traxid');
-            $criteria->setOrder('ASC');
-            foreach ($songsHandler->getObjects($criteria, true) as $sid => $song) {
-                if (\is_object($song)) {
-                    $ret['songs_array'][$sid] = $song->toArray(false);
-                }
-            }
-        }
-
-        return $ret;
-    }
-
-    /**
-     * @param string $field
-     * @param bool   $local
-     * @return bool|string
-     */
-    public function getImage($field = 'image', $local = false)
-    {
-        if ('' == $this->getVar($field)) {
-            return false;
-        }
-        if (!\file_exists($GLOBALS['xoops']->path($this->getVar('path') . $this->getVar($field)))) {
-            return false;
-        }
-        if (!$local) {
-            return XOOPS_URL . '/' . \str_replace(DS, '/', $this->getVar('path')) . $this->getVar($field);
-        }
-
-        return XOOPS_ROOT_PATH . DS . $this->getVar('path') . $this->getVar($field);
-    }
-
-    /**
-     * @return string
-     */
-    public function getURL(): string
-    {
-        global $file, $op, $fct, $id, $value, $gid, $vid, $vcid, $cid, $start, $limit;
-        if ($GLOBALS['songlistModuleConfig']['htaccess']) {
-            if (0 != $id) {
-                $artistHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Albums');
-                $artist        = $artistHandler->get($id);
-                if (\is_object($artist) && !$artist->isNew()) {
-                    return XOOPS_URL
-                           . '/'
-                           . $GLOBALS['songlistModuleConfig']['baseofurl']
-                           . '/albums/'
-                           . \urlencode(\str_replace([' ', \chr(9)], '-', $artist->getVar('title')))
-                           . '/'
-                           . $start
-                           . '-'
-                           . $id
-                           . '-'
-                           . $op
-                           . '-'
-                           . $fct
-                           . '-'
-                           . $gid
-                           . '-'
-                           . $cid
-                           . '/'
-                           . \urlencode($value)
-                           . $GLOBALS['songlistModuleConfig']['endofurl'];
-                }
-
-                return XOOPS_URL . '/' . $GLOBALS['songlistModuleConfig']['baseofurl'] . '/albums/' . $start . '-' . $id . '-' . $op . '-' . $fct . '-' . $gid . '-' . $cid . '/' . \urlencode($value) . $GLOBALS['songlistModuleConfig']['endofurl'];
-            }
-
-            return XOOPS_URL . '/' . $GLOBALS['songlistModuleConfig']['baseofurl'] . '/albums/' . $start . '-' . $id . '-' . $op . '-' . $fct . '-' . $gid . '-' . $cid . '/' . \urlencode($value) . $GLOBALS['songlistModuleConfig']['endofurl'];
-        }
-
-        return XOOPS_URL . '/modules/songlist/albums.php?op=' . $op . '&fct=' . $fct . '&id=' . $id . '&value=' . \urlencode($value ?? '') . '&gid=' . $gid . '&vid=' . $vid . '&cid=' . $cid . '&start=' . $start;
-    }
+	public $abid;
+	public $cid;
+	public $aids;
+	public $sids;
+	public $title;
+	public $image;
+	public $path;
+	public $artists;
+	public $songs;
+	public $hits;
+	public $rank;
+	public $votes;
+	public $created;
+	public $updated;
+
+	/**
+	 * Albums constructor.
+	 * @param null $fid
+	 */
+	public function __construct($fid = null)
+	{
+		$this->initVar('abid', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('cid', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('aids', \XOBJ_DTYPE_ARRAY, [], false);
+		$this->initVar('sids', \XOBJ_DTYPE_ARRAY, [], false);
+		$this->initVar('title', \XOBJ_DTYPE_TXTBOX, null, false, 128);
+		$this->initVar('image', \XOBJ_DTYPE_TXTBOX, null, false, 255);
+		$this->initVar('path', \XOBJ_DTYPE_TXTBOX, null, false, 255);
+		$this->initVar('artists', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('songs', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('hits', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('rank', \XOBJ_DTYPE_DECIMAL, 0, false);
+		$this->initVar('votes', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('created', \XOBJ_DTYPE_INT, 0, false);
+		$this->initVar('updated', \XOBJ_DTYPE_INT, 0, false);
+	}
+
+	/**
+	 * @param bool $as_array
+	 * @return array|string
+	 */
+	public function getForm($as_array = false)
+	{
+		return FormController::getFormAlbums($this, $as_array);
+	}
+
+	/**
+	 * @param bool $extra
+	 * @return array
+	 */
+	public function toArray($extra = true): array
+	{
+		$ret  = parent::toArray();
+		$form = $this->getForm(true);
+		foreach ($form as $key => $element) {
+			$ret['form'][$key] = $element->render();
+		}
+		foreach (['created', 'updated'] as $key) {
+			if ($this->getVar($key) > 0) {
+				$ret['form'][$key] = \date(_DATESTRING, $this->getVar($key));
+				$ret[$key]         = \date(_DATESTRING, $this->getVar($key));
+			}
+		}
+		$ret['picture'] = $this->getImage('image', false);
+		$ret['rank']    = \number_format(($this->getVar('rank') > 0 && $this->getVar('votes') > 0 ? $this->getVar('rank') / $this->getVar('votes') : 0), 2) . \_MI_SONGLIST_OFTEN;
+		$ret['url']     = $this->getURL(true);
+
+		if (!$extra) {
+			return $ret;
+		}
+
+		if (0 != $this->getVar('cid')) {
+			$categoryHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Category');
+			$category        = $categoryHandler->get($this->getVar('cid'));
+			if (\is_object($category)) {
+				$ret['category'] = $category->toArray(false);
+			}
+		}
+
+		if (0 != \count($this->getVar('aids'))) {
+			$artistsHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Artists');
+			foreach ($this->getVar('aids') as $aid) {
+				$artist = $artistsHandler->get($aid);
+				if (\is_object($artist)) {
+					$ret['artists_array'][$aid] = $artist->toArray(false);
+				}
+			}
+		}
+
+		if (0 != \count($this->getVar('sids'))) {
+			$songsHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Songs');
+			$criteria     = new \Criteria('sid', '(' . \implode(',', $this->getVar('sids')) . ')', 'IN');
+			$criteria->setSort('traxid');
+			$criteria->setOrder('ASC');
+			foreach ($songsHandler->getObjects($criteria, true) as $sid => $song) {
+				if (\is_object($song)) {
+					$ret['songs_array'][$sid] = $song->toArray(false);
+				}
+			}
+		}
+
+		return $ret;
+	}
+
+	/**
+	 * @param string $field
+	 * @param bool   $local
+	 * @return bool|string
+	 */
+	public function getImage($field = 'image', $local = false)
+	{
+		if ('' == $this->getVar($field)) {
+			return false;
+		}
+		if (!\file_exists($GLOBALS['xoops']->path($this->getVar('path') . $this->getVar($field)))) {
+			return false;
+		}
+		if (!$local) {
+			return XOOPS_URL . '/' . \str_replace(DS, '/', $this->getVar('path')) . $this->getVar($field);
+		}
+
+		return XOOPS_ROOT_PATH . DS . $this->getVar('path') . $this->getVar($field);
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getURL(): string
+	{
+		global $file, $op, $fct, $id, $value, $gid, $vid, $vcid, $cid, $start, $limit;
+		if ($GLOBALS['songlistModuleConfig']['htaccess']) {
+			if (0 != $id) {
+				$artistHandler = \XoopsModules\Songlist\Helper::getInstance()->getHandler('Albums');
+				$artist        = $artistHandler->get($id);
+				if (\is_object($artist) && !$artist->isNew()) {
+					return XOOPS_URL
+						   . '/'
+						   . $GLOBALS['songlistModuleConfig']['baseofurl']
+						   . '/albums/'
+						   . \urlencode(\str_replace([' ', \chr(9)], '-', $artist->getVar('title')))
+						   . '/'
+						   . $start
+						   . '-'
+						   . $id
+						   . '-'
+						   . $op
+						   . '-'
+						   . $fct
+						   . '-'
+						   . $gid
+						   . '-'
+						   . $cid
+						   . '/'
+						   . \urlencode($value)
+						   . $GLOBALS['songlistModuleConfig']['endofurl'];
+				}
+
+				return XOOPS_URL . '/' . $GLOBALS['songlistModuleConfig']['baseofurl'] . '/albums/' . $start . '-' . $id . '-' . $op . '-' . $fct . '-' . $gid . '-' . $cid . '/' . \urlencode($value) . $GLOBALS['songlistModuleConfig']['endofurl'];
+			}
+
+			return XOOPS_URL . '/' . $GLOBALS['songlistModuleConfig']['baseofurl'] . '/albums/' . $start . '-' . $id . '-' . $op . '-' . $fct . '-' . $gid . '-' . $cid . '/' . \urlencode($value) . $GLOBALS['songlistModuleConfig']['endofurl'];
+		}
+
+		return XOOPS_URL . '/modules/songlist/albums.php?op=' . $op . '&fct=' . $fct . '&id=' . $id . '&value=' . \urlencode($value ?? '') . '&gid=' . $gid . '&vid=' . $vid . '&cid=' . $cid . '&start=' . $start;
+	}
 }

xoops_version.php

Indentation +301 added lines, -301 removed lines

@@ -12,7 +12,7 @@ 
 $configHandler             = xoops_getHandler('config');
 $GLOBALS['songlistModule'] = $moduleHandler->getByDirname('songlist');
 if (is_object($GLOBALS['songlistModule'])) {
-    $GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
+	$GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
 }
 
 $modversion['version']             = '2.0.0';
@@ -52,18 +52,18 @@ 
 
 // ------------------- Tables ------------------ //
 $modversion['tables'] = [
-    $moduleDirName . '_' . 'albums',
-    $moduleDirName . '_' . 'artists',
-    $moduleDirName . '_' . 'category',
-    $moduleDirName . '_' . 'extra',
-    $moduleDirName . '_' . 'field',
-    $moduleDirName . '_' . 'genre',
-    $moduleDirName . '_' . 'voice',
-    $moduleDirName . '_' . 'requests',
-    $moduleDirName . '_' . 'songs',
-    $moduleDirName . '_' . 'utf8map',
-    $moduleDirName . '_' . 'visibility',
-    $moduleDirName . '_' . 'votes',
+	$moduleDirName . '_' . 'albums',
+	$moduleDirName . '_' . 'artists',
+	$moduleDirName . '_' . 'category',
+	$moduleDirName . '_' . 'extra',
+	$moduleDirName . '_' . 'field',
+	$moduleDirName . '_' . 'genre',
+	$moduleDirName . '_' . 'voice',
+	$moduleDirName . '_' . 'requests',
+	$moduleDirName . '_' . 'songs',
+	$moduleDirName . '_' . 'utf8map',
+	$moduleDirName . '_' . 'visibility',
+	$moduleDirName . '_' . 'votes',
 ];
 
 // Admin things
@@ -95,52 +95,52 @@ 
 
 // ------------------- Help files ------------------- //
 $modversion['helpsection'] = [
-    ['name' => _MI_SONGLIST_OVERVIEW, 'link' => 'page=help'],
-    ['name' => _MI_SONGLIST_DISCLAIMER, 'link' => 'page=disclaimer'],
-    ['name' => _MI_SONGLIST_LICENSE, 'link' => 'page=license'],
-    ['name' => _MI_SONGLIST_SUPPORT, 'link' => 'page=support'],
+	['name' => _MI_SONGLIST_OVERVIEW, 'link' => 'page=help'],
+	['name' => _MI_SONGLIST_DISCLAIMER, 'link' => 'page=disclaimer'],
+	['name' => _MI_SONGLIST_LICENSE, 'link' => 'page=license'],
+	['name' => _MI_SONGLIST_SUPPORT, 'link' => 'page=support'],
 ];
 
 // ------------------- Templates ------------------- //
 $modversion['templates'] = [
-    ['file' => 'songlist_albums_index.tpl', 'description' => ''],
-    ['file' => 'songlist_albums_item.tpl', 'description' => ''],
-    ['file' => 'songlist_albums_list.tpl', 'description' => ''],
-    ['file' => 'songlist_alpha_browse.tpl', 'description' => ''],
-    ['file' => 'songlist_artists_index.tpl', 'description' => ''],
-    ['file' => 'songlist_artists_item.tpl', 'description' => ''],
-    ['file' => 'songlist_artists_list.tpl', 'description' => ''],
-    ['file' => 'songlist_category_item.tpl', 'description' => ''],
-    ['file' => 'songlist_category_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_albums_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_albums_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_artists_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_artists_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_category_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_category_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_fieldlist.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_genre_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_genre_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_import_actiona.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_import_actionb.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_requests_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_requests_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_songs_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_songs_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_utf8map_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_utf8map_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_visibility.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_voice_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_voice_list.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_votes_edit.tpl', 'description' => ''],
-    ['file' => 'songlist_cpanel_votes_list.tpl', 'description' => ''],
-    ['file' => 'songlist_requests_index.tpl', 'description' => ''],
-    ['file' => 'songlist_search_index.tpl', 'description' => ''],
-    ['file' => 'songlist_search_results.tpl', 'description' => ''],
-    ['file' => 'songlist_search_search.tpl', 'description' => ''],
-    ['file' => 'songlist_songs_index.tpl', 'description' => ''],
-    ['file' => 'songlist_songs_item.tpl', 'description' => ''],
-    ['file' => 'songlist_songs_list.tpl', 'description' => ''],
+	['file' => 'songlist_albums_index.tpl', 'description' => ''],
+	['file' => 'songlist_albums_item.tpl', 'description' => ''],
+	['file' => 'songlist_albums_list.tpl', 'description' => ''],
+	['file' => 'songlist_alpha_browse.tpl', 'description' => ''],
+	['file' => 'songlist_artists_index.tpl', 'description' => ''],
+	['file' => 'songlist_artists_item.tpl', 'description' => ''],
+	['file' => 'songlist_artists_list.tpl', 'description' => ''],
+	['file' => 'songlist_category_item.tpl', 'description' => ''],
+	['file' => 'songlist_category_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_albums_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_albums_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_artists_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_artists_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_category_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_category_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_fieldlist.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_genre_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_genre_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_import_actiona.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_import_actionb.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_requests_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_requests_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_songs_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_songs_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_utf8map_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_utf8map_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_visibility.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_voice_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_voice_list.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_votes_edit.tpl', 'description' => ''],
+	['file' => 'songlist_cpanel_votes_list.tpl', 'description' => ''],
+	['file' => 'songlist_requests_index.tpl', 'description' => ''],
+	['file' => 'songlist_search_index.tpl', 'description' => ''],
+	['file' => 'songlist_search_results.tpl', 'description' => ''],
+	['file' => 'songlist_search_search.tpl', 'description' => ''],
+	['file' => 'songlist_songs_index.tpl', 'description' => ''],
+	['file' => 'songlist_songs_item.tpl', 'description' => ''],
+	['file' => 'songlist_songs_list.tpl', 'description' => ''],
 ];
 
 // Submenus
@@ -149,11 +149,11 @@ 
 $modversion['sub'][$i]['url']  = 'artists.php';
 ++$i;
 if (isset($GLOBALS['songlistModuleConfig']['album'])) {
-    if ($GLOBALS['songlistModuleConfig']['album']) {
-        $modversion['sub'][$i]['name'] = _MI_SONGLIST_MENU_ALBUMS;
-        $modversion['sub'][$i]['url']  = 'albums.php';
-        ++$i;
-    }
+	if ($GLOBALS['songlistModuleConfig']['album']) {
+		$modversion['sub'][$i]['name'] = _MI_SONGLIST_MENU_ALBUMS;
+		$modversion['sub'][$i]['url']  = 'albums.php';
+		++$i;
+	}
 }
 $modversion['sub'][$i]['name'] = _MI_SONGLIST_MENU_SEARCH;
 $modversion['sub'][$i]['url']  = 'search.php';
@@ -165,90 +165,90 @@ 
 $i = 0;
 // ------------------- Blocks ------------------- //
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_artist.php',
-    'name'        => _MI_SONGLIST_BLOCK_TOP_ARTIST,
-    'description' => 'Shows top artist',
-    'show_func'   => 'b_songlist_popular_artist_show',
-    'options'     => '',
-    'edit_func'   => 'b_songlist_popular_artist_edit',
-    'template'    => 'songlist_popular_artist.tpl',
+	'file'        => 'songlist_popular_artist.php',
+	'name'        => _MI_SONGLIST_BLOCK_TOP_ARTIST,
+	'description' => 'Shows top artist',
+	'show_func'   => 'b_songlist_popular_artist_show',
+	'options'     => '',
+	'edit_func'   => 'b_songlist_popular_artist_edit',
+	'template'    => 'songlist_popular_artist.tpl',
 ];
 
 ++$i;
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_artists.php',
-    'name'        => _MI_SONGLIST_BLOCK_POPULAR_ARTISTS,
-    'description' => 'Shows popular artists',
-    'show_func'   => 'b_songlist_popular_artists_show',
-    'options'     => '6',
-    'edit_func'   => 'b_songlist_popular_artists_edit',
-    'template'    => 'songlist_popular_artists.tpl',
+	'file'        => 'songlist_popular_artists.php',
+	'name'        => _MI_SONGLIST_BLOCK_POPULAR_ARTISTS,
+	'description' => 'Shows popular artists',
+	'show_func'   => 'b_songlist_popular_artists_show',
+	'options'     => '6',
+	'edit_func'   => 'b_songlist_popular_artists_edit',
+	'template'    => 'songlist_popular_artists.tpl',
 ];
 
 ++$i;
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_album.php',
-    'name'        => _MI_SONGLIST_BLOCK_TOP_ALBUM,
-    'description' => 'Shows top album',
-    'show_func'   => 'b_songlist_popular_album_show',
-    'options'     => '',
-    'edit_func'   => 'b_songlist_popular_album_edit',
-    'template'    => 'songlist_popular_album.tpl',
+	'file'        => 'songlist_popular_album.php',
+	'name'        => _MI_SONGLIST_BLOCK_TOP_ALBUM,
+	'description' => 'Shows top album',
+	'show_func'   => 'b_songlist_popular_album_show',
+	'options'     => '',
+	'edit_func'   => 'b_songlist_popular_album_edit',
+	'template'    => 'songlist_popular_album.tpl',
 ];
 
 ++$i;
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_albums.php',
-    'name'        => _MI_SONGLIST_BLOCK_POPULAR_ALBUMS,
-    'description' => 'Shows popular albums',
-    'show_func'   => 'b_songlist_popular_albums_show',
-    'options'     => '6',
-    'edit_func'   => 'b_songlist_popular_albums_edit',
-    'template'    => 'songlist_popular_albums.tpl',
+	'file'        => 'songlist_popular_albums.php',
+	'name'        => _MI_SONGLIST_BLOCK_POPULAR_ALBUMS,
+	'description' => 'Shows popular albums',
+	'show_func'   => 'b_songlist_popular_albums_show',
+	'options'     => '6',
+	'edit_func'   => 'b_songlist_popular_albums_edit',
+	'template'    => 'songlist_popular_albums.tpl',
 ];
 
 ++$i;
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_genre.php',
-    'name'        => _MI_SONGLIST_BLOCK_TOP_GENRE,
-    'description' => 'Shows top genre',
-    'show_func'   => 'b_songlist_popular_genre_show',
-    'options'     => '',
-    'edit_func'   => 'b_songlist_popular_genre_edit',
-    'template'    => 'songlist_popular_genre.tpl',
+	'file'        => 'songlist_popular_genre.php',
+	'name'        => _MI_SONGLIST_BLOCK_TOP_GENRE,
+	'description' => 'Shows top genre',
+	'show_func'   => 'b_songlist_popular_genre_show',
+	'options'     => '',
+	'edit_func'   => 'b_songlist_popular_genre_edit',
+	'template'    => 'songlist_popular_genre.tpl',
 ];
 
 ++$i;
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_genres.php',
-    'name'        => _MI_SONGLIST_BLOCK_POPULAR_GENRES,
-    'description' => 'Shows popular genres',
-    'show_func'   => 'b_songlist_popular_genres_show',
-    'options'     => '6',
-    'edit_func'   => 'b_songlist_popular_genres_edit',
-    'template'    => 'songlist_popular_genres.tpl',
+	'file'        => 'songlist_popular_genres.php',
+	'name'        => _MI_SONGLIST_BLOCK_POPULAR_GENRES,
+	'description' => 'Shows popular genres',
+	'show_func'   => 'b_songlist_popular_genres_show',
+	'options'     => '6',
+	'edit_func'   => 'b_songlist_popular_genres_edit',
+	'template'    => 'songlist_popular_genres.tpl',
 ];
 
 ++$i;
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_song.php',
-    'name'        => _MI_SONGLIST_BLOCK_TOP_SONG,
-    'description' => 'Shows top song',
-    'show_func'   => 'b_songlist_popular_song_show',
-    'options'     => '',
-    'edit_func'   => 'b_songlist_popular_song_edit',
-    'template'    => 'songlist_popular_song.tpl',
+	'file'        => 'songlist_popular_song.php',
+	'name'        => _MI_SONGLIST_BLOCK_TOP_SONG,
+	'description' => 'Shows top song',
+	'show_func'   => 'b_songlist_popular_song_show',
+	'options'     => '',
+	'edit_func'   => 'b_songlist_popular_song_edit',
+	'template'    => 'songlist_popular_song.tpl',
 ];
 
 ++$i;
 $modversion['blocks'][$i] = [
-    'file'        => 'songlist_popular_songs.php',
-    'name'        => _MI_SONGLIST_BLOCK_POPULAR_SONGS,
-    'description' => 'Shows popular songs',
-    'show_func'   => 'b_songlist_popular_songs_show',
-    'options'     => '6',
-    'edit_func'   => 'b_songlist_popular_songs_edit',
-    'template'    => 'songlist_popular_songs.tpl',
+	'file'        => 'songlist_popular_songs.php',
+	'name'        => _MI_SONGLIST_BLOCK_POPULAR_SONGS,
+	'description' => 'Shows popular songs',
+	'show_func'   => 'b_songlist_popular_songs_show',
+	'options'     => '6',
+	'edit_func'   => 'b_songlist_popular_songs_edit',
+	'template'    => 'songlist_popular_songs.tpl',
 ];
 
 // default admin editor
@@ -257,127 +257,127 @@ 
 $editorList    = array_flip($editorHandler->getList());
 
 $modversion['config'][] = [
-    'name'        => 'editor',
-    'title'       => '_MI_SONGLIST_EDITORS',
-    'description' => '_MI_SONGLIST_EDITORS_DESC',
-    'formtype'    => 'select',
-    'valuetype'   => 'text',
-    'default'     => 'tinymce',
-    'options'     => $editorList,
+	'name'        => 'editor',
+	'title'       => '_MI_SONGLIST_EDITORS',
+	'description' => '_MI_SONGLIST_EDITORS_DESC',
+	'formtype'    => 'select',
+	'valuetype'   => 'text',
+	'default'     => 'tinymce',
+	'options'     => $editorList,
 ];
 
 $modversion['config'][] = [
-    'name'        => 'salt',
-    'title'       => '_MI_SONGLIST_SALT',
-    'description' => '_MI_SONGLIST_SALT_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'text',
-    'default'     => (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : ''),
-
-    'options' => [],
+	'name'        => 'salt',
+	'title'       => '_MI_SONGLIST_SALT',
+	'description' => '_MI_SONGLIST_SALT_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'text',
+	'default'     => (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
+					 . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : ''),
+
+	'options' => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'cols',
-    'title'       => '_MI_SONGLIST_COLS',
-    'description' => '_MI_SONGLIST_COLS_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'int',
-    'default'     => 4,
-    'options'     => [],
+	'name'        => 'cols',
+	'title'       => '_MI_SONGLIST_COLS',
+	'description' => '_MI_SONGLIST_COLS_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'int',
+	'default'     => 4,
+	'options'     => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'rows',
-    'title'       => '_MI_SONGLIST_ROWS',
-    'description' => '_MI_SONGLIST_ROWS_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'int',
-    'default'     => 4,
-    'options'     => [],
+	'name'        => 'rows',
+	'title'       => '_MI_SONGLIST_ROWS',
+	'description' => '_MI_SONGLIST_ROWS_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'int',
+	'default'     => 4,
+	'options'     => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'email',
-    'title'       => '_MI_SONGLIST_EMAIL',
-    'description' => '_MI_SONGLIST_EMAIL_DESC',
-    'formtype'    => 'textarea',
-    'valuetype'   => 'text',
-    'default'     => $GLOBALS['xoopsConfig']['adminmail'],
-    'options'     => [],
+	'name'        => 'email',
+	'title'       => '_MI_SONGLIST_EMAIL',
+	'description' => '_MI_SONGLIST_EMAIL_DESC',
+	'formtype'    => 'textarea',
+	'valuetype'   => 'text',
+	'default'     => $GLOBALS['xoopsConfig']['adminmail'],
+	'options'     => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'mp3_filesize',
-    'title'       => '_MI_SONGLIST_MP3FILESIZE',
-    'description' => '_MI_SONGLIST_MP3FILESIZE_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => '195035100',
+	'name'        => 'mp3_filesize',
+	'title'       => '_MI_SONGLIST_MP3FILESIZE',
+	'description' => '_MI_SONGLIST_MP3FILESIZE_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => '195035100',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'mp3_mimetype',
-    'title'       => '_MI_SONGLIST_MP3MIMETYPE',
-    'description' => '_MI_SONGLIST_MP3MIMETYPE_DESC',
-    'formtype'    => 'textarea',
-    'valuetype'   => 'text',
-    'default'     => 'audio/mpeg|audio/x-mpeg|audio/mp3|audio/x-mp3|audio/mpeg3|audio/x-mpeg3|audio/mpg|audio/x-mpg|audio/x-mpegaudio',
+	'name'        => 'mp3_mimetype',
+	'title'       => '_MI_SONGLIST_MP3MIMETYPE',
+	'description' => '_MI_SONGLIST_MP3MIMETYPE_DESC',
+	'formtype'    => 'textarea',
+	'valuetype'   => 'text',
+	'default'     => 'audio/mpeg|audio/x-mpeg|audio/mp3|audio/x-mp3|audio/mpeg3|audio/x-mpeg3|audio/mpg|audio/x-mpg|audio/x-mpegaudio',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'mp3_extensions',
-    'title'       => '_MI_SONGLIST_MP3EXTENSIONS',
-    'description' => '_MI_SONGLIST_MP3EXTENSIONS_DESC',
-    'formtype'    => 'textarea',
-    'valuetype'   => 'text',
-    'default'     => 'mp3',
+	'name'        => 'mp3_extensions',
+	'title'       => '_MI_SONGLIST_MP3EXTENSIONS',
+	'description' => '_MI_SONGLIST_MP3EXTENSIONS_DESC',
+	'formtype'    => 'textarea',
+	'valuetype'   => 'text',
+	'default'     => 'mp3',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'filesize_upload',
-    'title'       => '_MI_SONGLIST_FILESIZEUPLD',
-    'description' => '_MI_SONGLIST_FILESIZEUPLD_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => '1950351',
+	'name'        => 'filesize_upload',
+	'title'       => '_MI_SONGLIST_FILESIZEUPLD',
+	'description' => '_MI_SONGLIST_FILESIZEUPLD_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => '1950351',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'allowed_mimetype',
-    'title'       => '_MI_SONGLIST_ALLOWEDMIMETYPE',
-    'description' => '_MI_SONGLIST_ALLOWEDMIMETYPE_DESC',
-    'formtype'    => 'textarea',
-    'valuetype'   => 'text',
-    'default'     => 'image/gif|image/pjpeg|image/jpeg|image/x-png|image/png',
+	'name'        => 'allowed_mimetype',
+	'title'       => '_MI_SONGLIST_ALLOWEDMIMETYPE',
+	'description' => '_MI_SONGLIST_ALLOWEDMIMETYPE_DESC',
+	'formtype'    => 'textarea',
+	'valuetype'   => 'text',
+	'default'     => 'image/gif|image/pjpeg|image/jpeg|image/x-png|image/png',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'allowed_extensions',
-    'title'       => '_MI_SONGLIST_ALLOWEDEXTENSIONS',
-    'description' => '_MI_SONGLIST_ALLOWEDEXTENSIONS_DESC',
-    'formtype'    => 'textarea',
-    'valuetype'   => 'text',
-    'default'     => 'gif|pjpeg|jpeg|jpg|png',
+	'name'        => 'allowed_extensions',
+	'title'       => '_MI_SONGLIST_ALLOWEDEXTENSIONS',
+	'description' => '_MI_SONGLIST_ALLOWEDEXTENSIONS_DESC',
+	'formtype'    => 'textarea',
+	'valuetype'   => 'text',
+	'default'     => 'gif|pjpeg|jpeg|jpg|png',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'upload_areas',
-    'title'       => '_MI_SONGLIST_UPLOADAREAS',
-    'description' => '_MI_SONGLIST_UPLOADAREAS_DESC',
+	'name'        => 'upload_areas',
+	'title'       => '_MI_SONGLIST_UPLOADAREAS',
+	'description' => '_MI_SONGLIST_UPLOADAREAS_DESC',
 //    'formtype'    => 'select',
 //    'valuetype'   => 'text',
 //    'default'     => 'uploads' . DS . 'songlist' . DS,
@@ -385,45 +385,45 @@ 
 //        'uploads' . DS                   => 'uploads' . DS,
 //        'uploads' . DS . 'songlist' . DS => 'uploads' . DS . 'songlist' . DS,
 //],
-    'formtype'    => 'textbox',
-    'valuetype'   => 'text',
-    'default'     => '/uploads/songlist/',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'text',
+	'default'     => '/uploads/songlist/',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'album',
-    'title'       => '_MI_SONGLIST_ALBUM',
-    'description' => '_MI_SONGLIST_ALBUM_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => '0',
+	'name'        => 'album',
+	'title'       => '_MI_SONGLIST_ALBUM',
+	'description' => '_MI_SONGLIST_ALBUM_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => '0',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'genre',
-    'title'       => '_MI_SONGLIST_GENRE',
-    'description' => '_MI_SONGLIST_GENRE_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => '0',
+	'name'        => 'genre',
+	'title'       => '_MI_SONGLIST_GENRE',
+	'description' => '_MI_SONGLIST_GENRE_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => '0',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'voice',
-    'title'       => '_MI_SONGLIST_VOICE',
-    'description' => '_MI_SONGLIST_VOICE_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => '0',
+	'name'        => 'voice',
+	'title'       => '_MI_SONGLIST_VOICE',
+	'description' => '_MI_SONGLIST_VOICE_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => '0',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'lyrics',
-    'title'       => '_MI_SONGLIST_LYRICS',
-    'description' => '_MI_SONGLIST_LYRICS_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => '0',
+	'name'        => 'lyrics',
+	'title'       => '_MI_SONGLIST_LYRICS',
+	'description' => '_MI_SONGLIST_LYRICS_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => '0',
 ];
 
 /*
@@ -438,110 +438,110 @@ 
 */
 
 $modversion['config'][] = [
-    'name'        => 'htaccess',
-    'title'       => '_MI_SONGLIST_HTACCESS',
-    'description' => '_MI_SONGLIST_HTACCESS_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => '0',
+	'name'        => 'htaccess',
+	'title'       => '_MI_SONGLIST_HTACCESS',
+	'description' => '_MI_SONGLIST_HTACCESS_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => '0',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'baseofurl',
-    'title'       => '_MI_SONGLIST_HTACCESS_BASEOFURL',
-    'description' => '_MI_SONGLIST_HTACCESS_BASEOFURL_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'text',
-    'default'     => 'songlist',
+	'name'        => 'baseofurl',
+	'title'       => '_MI_SONGLIST_HTACCESS_BASEOFURL',
+	'description' => '_MI_SONGLIST_HTACCESS_BASEOFURL_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'text',
+	'default'     => 'songlist',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'endofurl',
-    'title'       => '_MI_SONGLIST_HTACCESS_ENDOFURL',
-    'description' => '_MI_SONGLIST_HTACCESS_ENDOFURL_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'text',
-    'default'     => '.tpl',
+	'name'        => 'endofurl',
+	'title'       => '_MI_SONGLIST_HTACCESS_ENDOFURL',
+	'description' => '_MI_SONGLIST_HTACCESS_ENDOFURL_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'text',
+	'default'     => '.tpl',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'tags',
-    'title'       => '_MI_SONGLIST_TAGS',
-    'description' => '_MI_SONGLIST_TAGS_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => '0',
+	'name'        => 'tags',
+	'title'       => '_MI_SONGLIST_TAGS',
+	'description' => '_MI_SONGLIST_TAGS_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => '0',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'force_jquery',
-    'title'       => '_MI_SONGLIST_FORCE_JQUERY',
-    'description' => '_MI_SONGLIST_FORCE_JQUERY_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => true,
+	'name'        => 'force_jquery',
+	'title'       => '_MI_SONGLIST_FORCE_JQUERY',
+	'description' => '_MI_SONGLIST_FORCE_JQUERY_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => true,
 ];
 
 $modversion['config'][] = [
-    'name'        => 'memory_admin',
-    'title'       => '_MI_SONGLIST_MEMORY_ADMIN',
-    'description' => '_MI_SONGLIST_MEMORY_ADMIN_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'int',
-    'default'     => 128,
-    'options'     => [],
+	'name'        => 'memory_admin',
+	'title'       => '_MI_SONGLIST_MEMORY_ADMIN',
+	'description' => '_MI_SONGLIST_MEMORY_ADMIN_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'int',
+	'default'     => 128,
+	'options'     => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'memory_user',
-    'title'       => '_MI_SONGLIST_MEMORY_USER',
-    'description' => '_MI_SONGLIST_MEMORY_USER_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'int',
-    'default'     => 128,
-    'options'     => [],
+	'name'        => 'memory_user',
+	'title'       => '_MI_SONGLIST_MEMORY_USER',
+	'description' => '_MI_SONGLIST_MEMORY_USER_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'int',
+	'default'     => 128,
+	'options'     => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'time_admin',
-    'title'       => '_MI_SONGLIST_TIME_ADMIN',
-    'description' => '_MI_SONGLIST_TIME_ADMIN_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'int',
-    'default'     => 360,
-    'options'     => [],
+	'name'        => 'time_admin',
+	'title'       => '_MI_SONGLIST_TIME_ADMIN',
+	'description' => '_MI_SONGLIST_TIME_ADMIN_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'int',
+	'default'     => 360,
+	'options'     => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'time_user',
-    'title'       => '_MI_SONGLIST_TIME_USER',
-    'description' => '_MI_SONGLIST_TIME_USER_DESC',
-    'formtype'    => 'text',
-    'valuetype'   => 'int',
-    'default'     => 360,
-    'options'     => [],
+	'name'        => 'time_user',
+	'title'       => '_MI_SONGLIST_TIME_USER',
+	'description' => '_MI_SONGLIST_TIME_USER_DESC',
+	'formtype'    => 'text',
+	'valuetype'   => 'int',
+	'default'     => 360,
+	'options'     => [],
 ];
 
 /**
  * Make Sample button visible?
  */
 $modversion['config'][] = [
-    'name'        => 'displaySampleButton',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 1,
+	'name'        => 'displaySampleButton',
+	'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON',
+	'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 1,
 ];
 
 /**
  * Show Developer Tools?
  */
 $modversion['config'][] = [
-    'name'        => 'displayDeveloperTools',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 0,
+	'name'        => 'displayDeveloperTools',
+	'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS',
+	'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 0,
 ];

Spacing +32 added lines, -32 removed lines

@@ -1,6 +1,6 @@ 
 <?php declare(strict_types=1);
 
-require_once __DIR__ . '/preloads/autoloader.php';
+require_once __DIR__.'/preloads/autoloader.php';
 
 error_reporting(E_ALL);
 $moduleDirName      = basename(__DIR__);
@@ -29,7 +29,7 @@ 
 $modversion['releasedate']         = 'Thursday, 03rd July, 2022';
 $modversion['website']             = 'www.chronolabs.coop';
 $modversion['release_info']        = 'Stable 2012/08/16';
-$modversion['release_file']        = XOOPS_URL . '/modules/songlist/docs/changelog.txt';
+$modversion['release_file']        = XOOPS_URL.'/modules/songlist/docs/changelog.txt';
 $modversion['author_realname']     = 'Wishcraft';
 $modversion['author_website_url']  = 'https://www.chronolabs.coop';
 $modversion['author_website_name'] = 'Chronolabs';
@@ -52,18 +52,18 @@ 
 
 // ------------------- Tables ------------------ //
 $modversion['tables'] = [
-    $moduleDirName . '_' . 'albums',
-    $moduleDirName . '_' . 'artists',
-    $moduleDirName . '_' . 'category',
-    $moduleDirName . '_' . 'extra',
-    $moduleDirName . '_' . 'field',
-    $moduleDirName . '_' . 'genre',
-    $moduleDirName . '_' . 'voice',
-    $moduleDirName . '_' . 'requests',
-    $moduleDirName . '_' . 'songs',
-    $moduleDirName . '_' . 'utf8map',
-    $moduleDirName . '_' . 'visibility',
-    $moduleDirName . '_' . 'votes',
+    $moduleDirName.'_'.'albums',
+    $moduleDirName.'_'.'artists',
+    $moduleDirName.'_'.'category',
+    $moduleDirName.'_'.'extra',
+    $moduleDirName.'_'.'field',
+    $moduleDirName.'_'.'genre',
+    $moduleDirName.'_'.'voice',
+    $moduleDirName.'_'.'requests',
+    $moduleDirName.'_'.'songs',
+    $moduleDirName.'_'.'utf8map',
+    $moduleDirName.'_'.'visibility',
+    $moduleDirName.'_'.'votes',
 ];
 
 // Admin things
@@ -272,20 +272,20 @@ 
     'description' => '_MI_SONGLIST_SALT_DESC',
     'formtype'    => 'text',
     'valuetype'   => 'text',
-    'default'     => (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : '')
-                     . (2 != random_int(0, 4) ? chr(random_int(32, 190)) : ''),
+    'default'     => (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : '')
+                     . (2!=random_int(0, 4) ? chr(random_int(32, 190)) : ''),
 
     'options' => [],
 ];
@@ -527,8 +527,8 @@ 
  */
 $modversion['config'][] = [
     'name'        => 'displaySampleButton',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC',
+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON',
+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON_DESC',
     'formtype'    => 'yesno',
     'valuetype'   => 'int',
     'default'     => 1,
@@ -539,8 +539,8 @@ 
  */
 $modversion['config'][] = [
     'name'        => 'displayDeveloperTools',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS_DESC',
+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_DEV_TOOLS',
+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_DEV_TOOLS_DESC',
     'formtype'    => 'yesno',
     'valuetype'   => 'int',
     'default'     => 0,