XoopsModules25x / songlist

admin/albums.php

Indentation +189 added lines, -189 removed lines

@@ -22,102 +22,102 @@ 
 $filter = !empty($_REQUEST['filter']) ? '' . $_REQUEST['filter'] . '' : '1,1';
 
 switch ($op) {
-    default:
-    case 'albums':
-        switch ($fct) {
-            default:
-            case 'list':
-                $adminObject = Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-
-                /** @var AlbumsHandler $albumsHandler */
-                $albumsHandler = Helper::getInstance()->getHandler('Albums');
-
-                $criteria        = $albumsHandler->getFilterCriteria($GLOBALS['filter']);
-                $ttl             = $albumsHandler->getCount($criteria);
-                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
-
-                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
-                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
-
-                foreach ($albumsHandler->filterFields() as $id => $key) {
-                    $GLOBALS['xoopsTpl']->assign(
-                        \mb_strtolower(str_replace('-', '_', $key) . '_th'),
-                        '<a href="'
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?start='
-                        . $GLOBALS['start']
-                        . '&limit='
-                        . $GLOBALS['limit']
-                        . '&sort='
-                        . $key
-                        . '&order='
-                        . (($key == $GLOBALS['sort']) ? ('DESC' == $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
-                        . '&op='
-                        . $GLOBALS['op']
-                        . '&filter='
-                        . $GLOBALS['filter']
-                        . '">'
-                        . (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
-                        . '</a>'
-                    );
-                    $GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $albumsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
-                }
-
-                $GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
-                $GLOBALS['xoopsTpl']->assign('start', $GLOBALS['start']);
-                $GLOBALS['xoopsTpl']->assign('order', $GLOBALS['order']);
-                $GLOBALS['xoopsTpl']->assign('sort', $GLOBALS['sort']);
-                $GLOBALS['xoopsTpl']->assign('filter', $GLOBALS['filter']);
-                $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-
-                $criteria->setStart($GLOBALS['start']);
-                $criteria->setLimit($GLOBALS['limit']);
-                $criteria->setSort('`' . $GLOBALS['sort'] . '`');
-                $criteria->setOrder($GLOBALS['order']);
-
-                $albumsArray = $albumsHandler->getObjects($criteria, true);
-                foreach ($albumsArray as $cid => $album) {
-                    if (is_object($album)) {
-                        $GLOBALS['xoopsTpl']->append('albums', $album->toArray());
-                    }
-                }
-                $GLOBALS['xoopsTpl']->assign('form', FormController::getFormAlbums(false));
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                $GLOBALS['xoopsTpl']->display('db:songlist_cpanel_albums_list.tpl');
-                break;
-            case 'new':
-            case 'edit':
-                $adminObject = Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-
-                $albumsHandler = Helper::getInstance()->getHandler('Albums');
-                if (Request::hasVar('id', 'REQUEST')) {
-                    $album = $albumsHandler->get(Request::getInt('id', 0, 'REQUEST'));
-                } else {
-                    $album = $albumsHandler->create();
-                }
-
-                $GLOBALS['xoopsTpl']->assign('form', $album->getForm());
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                $GLOBALS['xoopsTpl']->display('db:songlist_cpanel_albums_edit.tpl');
-                break;
-            case 'save':
-                $albumsHandler = Helper::getInstance()->getHandler('Albums');
-                $id            = 0;
-                $id            = Request::getInt('id', 0, 'REQUEST');
-                if ($id) {
-                    $album = $albumsHandler->get($id);
-                } else {
-                    $album = $albumsHandler->create();
-                }
-                $album->setVars($_POST[$id]);
-
-                if (!$id = $albumsHandler->insert($album)) {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_FAILEDTOSAVE);
-                    exit(0);
-                }
-                if (Request::hasVar('image', 'FILES') && !empty($_FILES['image']['name'])) {
+	default:
+	case 'albums':
+		switch ($fct) {
+			default:
+			case 'list':
+				$adminObject = Admin::getInstance();
+				$adminObject->displayNavigation(basename(__FILE__));
+
+				/** @var AlbumsHandler $albumsHandler */
+				$albumsHandler = Helper::getInstance()->getHandler('Albums');
+
+				$criteria        = $albumsHandler->getFilterCriteria($GLOBALS['filter']);
+				$ttl             = $albumsHandler->getCount($criteria);
+				$GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
+
+				$pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
+				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
+
+				foreach ($albumsHandler->filterFields() as $id => $key) {
+					$GLOBALS['xoopsTpl']->assign(
+						\mb_strtolower(str_replace('-', '_', $key) . '_th'),
+						'<a href="'
+						. $_SERVER['SCRIPT_NAME']
+						. '?start='
+						. $GLOBALS['start']
+						. '&limit='
+						. $GLOBALS['limit']
+						. '&sort='
+						. $key
+						. '&order='
+						. (($key == $GLOBALS['sort']) ? ('DESC' == $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
+						. '&op='
+						. $GLOBALS['op']
+						. '&filter='
+						. $GLOBALS['filter']
+						. '">'
+						. (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
+						. '</a>'
+					);
+					$GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $albumsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
+				}
+
+				$GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
+				$GLOBALS['xoopsTpl']->assign('start', $GLOBALS['start']);
+				$GLOBALS['xoopsTpl']->assign('order', $GLOBALS['order']);
+				$GLOBALS['xoopsTpl']->assign('sort', $GLOBALS['sort']);
+				$GLOBALS['xoopsTpl']->assign('filter', $GLOBALS['filter']);
+				$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+
+				$criteria->setStart($GLOBALS['start']);
+				$criteria->setLimit($GLOBALS['limit']);
+				$criteria->setSort('`' . $GLOBALS['sort'] . '`');
+				$criteria->setOrder($GLOBALS['order']);
+
+				$albumsArray = $albumsHandler->getObjects($criteria, true);
+				foreach ($albumsArray as $cid => $album) {
+					if (is_object($album)) {
+						$GLOBALS['xoopsTpl']->append('albums', $album->toArray());
+					}
+				}
+				$GLOBALS['xoopsTpl']->assign('form', FormController::getFormAlbums(false));
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				$GLOBALS['xoopsTpl']->display('db:songlist_cpanel_albums_list.tpl');
+				break;
+			case 'new':
+			case 'edit':
+				$adminObject = Admin::getInstance();
+				$adminObject->displayNavigation(basename(__FILE__));
+
+				$albumsHandler = Helper::getInstance()->getHandler('Albums');
+				if (Request::hasVar('id', 'REQUEST')) {
+					$album = $albumsHandler->get(Request::getInt('id', 0, 'REQUEST'));
+				} else {
+					$album = $albumsHandler->create();
+				}
+
+				$GLOBALS['xoopsTpl']->assign('form', $album->getForm());
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				$GLOBALS['xoopsTpl']->display('db:songlist_cpanel_albums_edit.tpl');
+				break;
+			case 'save':
+				$albumsHandler = Helper::getInstance()->getHandler('Albums');
+				$id            = 0;
+				$id            = Request::getInt('id', 0, 'REQUEST');
+				if ($id) {
+					$album = $albumsHandler->get($id);
+				} else {
+					$album = $albumsHandler->create();
+				}
+				$album->setVars($_POST[$id]);
+
+				if (!$id = $albumsHandler->insert($album)) {
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_FAILEDTOSAVE);
+					exit(0);
+				}
+				if (Request::hasVar('image', 'FILES') && !empty($_FILES['image']['name'])) {
 //                    if (!is_dir($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']))) {
 //                        foreach (explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders) {
 //                            foreach (explode('/', $folders) as $folder) {
@@ -130,99 +130,99 @@ 
 //                    }
 
 //                    require_once $GLOBALS['xoops']->path('modules/songlist/include/uploader.php');
-                    $album    = $albumsHandler->get($id);
-                    $uploader = new Uploader(
-                        $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']. 'albums'),
-                        explode('|', $GLOBALS['songlistModuleConfig']['allowed_mimetype']),
-                        $GLOBALS['songlistModuleConfig']['filesize_upload'],
-                        0,
-                        0,
-                        explode('|', $GLOBALS['songlistModuleConfig']['allowed_extensions'])
-                    );
-                    try {
-                        $uploader->setPrefix(mb_substr(md5((string)microtime(true)), random_int(0, 20), 13));
-                    } catch (Exception $e) {
-                    }
-
-                    if ($uploader->fetchMedia('image')) {
-                        if (!$uploader->upload()) {
-                            $adminObject = Admin::getInstance();
-                            $adminObject->displayNavigation(basename(__FILE__));
-                            echo $uploader->getErrors();
-                            require __DIR__ . '/admin_footer.php';
-                            exit(0);
-                        }
-                        if (mb_strlen($album->getVar('image'))) {
-                            unlink($GLOBALS['xoops']->path($album->getVar('path')) . $album->getVar('image'));
-                        }
-
-                        $album->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']. 'albums/') ;
-                        $album->setVar('image', $uploader->getSavedFileName());
-                        @$albumsHandler->insert($album);
-                    } else {
-                        $adminObject = Admin::getInstance();
-                        $adminObject->displayNavigation(basename(__FILE__));
-                        echo $uploader->getErrors();
-                        require __DIR__ . '/admin_footer.php';
-                        exit(0);
-                    }
-                }
-
-                if ('new' == $_REQUEST['state'][$_REQUEST['id']]) {
-                    redirect_header(
-                        $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                        10,
-                        _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY
-                    );
-                } else {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
-                }
-                exit(0);
-
-                break;
-            case 'savelist':
-                $albumsHandler = Helper::getInstance()->getHandler('Albums');
-                foreach ($_REQUEST['id'] as $id) {
-                    $album = $albumsHandler->get($id);
-                    $album->setVars($_POST[$id]);
-                    if (!$albumsHandler->insert($album)) {
-                        redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                            10,
-                            _AM_SONGLIST_MSG_ALBUMS_FAILEDTOSAVE
-                        );
-                        exit(0);
-                    }
-                }
-                redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
-                exit(0);
-                break;
-            case 'delete':
-                $albumsHandler = Helper::getInstance()->getHandler('Albums');
-                $id            = 0;
-                if (Request::hasVar('id', 'POST') && $id = Request::getInt('id', 0, 'POST')) {
-                    $album = $albumsHandler->get($id);
-                    if (!$albumsHandler->delete($album)) {
-                        redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                            10,
-                            _AM_SONGLIST_MSG_ALBUMS_FAILEDTODELETE
-                        );
-                        exit(0);
-                    }
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_DELETED);
-                    exit(0);
-                }
-                $album = $albumsHandler->get(Request::getInt('id', 0, 'REQUEST'));
-                xoops_confirm(
-                    ['id' => $_REQUEST['id'], 'op' => $_REQUEST['op'], 'fct' => $_REQUEST['fct'], 'limit' => $_REQUEST['limit'], 'start' => $_REQUEST['start'], 'order' => $_REQUEST['order'], 'sort' => $_REQUEST['sort'], 'filter' => $_REQUEST['filter']],
-                    $_SERVER['SCRIPT_NAME'],
-                    sprintf(_AM_SONGLIST_MSG_ALBUMS_DELETE, $album->getVar('title'))
-                );
-
-                break;
-        }
-        break;
+					$album    = $albumsHandler->get($id);
+					$uploader = new Uploader(
+						$GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']. 'albums'),
+						explode('|', $GLOBALS['songlistModuleConfig']['allowed_mimetype']),
+						$GLOBALS['songlistModuleConfig']['filesize_upload'],
+						0,
+						0,
+						explode('|', $GLOBALS['songlistModuleConfig']['allowed_extensions'])
+					);
+					try {
+						$uploader->setPrefix(mb_substr(md5((string)microtime(true)), random_int(0, 20), 13));
+					} catch (Exception $e) {
+					}
+
+					if ($uploader->fetchMedia('image')) {
+						if (!$uploader->upload()) {
+							$adminObject = Admin::getInstance();
+							$adminObject->displayNavigation(basename(__FILE__));
+							echo $uploader->getErrors();
+							require __DIR__ . '/admin_footer.php';
+							exit(0);
+						}
+						if (mb_strlen($album->getVar('image'))) {
+							unlink($GLOBALS['xoops']->path($album->getVar('path')) . $album->getVar('image'));
+						}
+
+						$album->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']. 'albums/') ;
+						$album->setVar('image', $uploader->getSavedFileName());
+						@$albumsHandler->insert($album);
+					} else {
+						$adminObject = Admin::getInstance();
+						$adminObject->displayNavigation(basename(__FILE__));
+						echo $uploader->getErrors();
+						require __DIR__ . '/admin_footer.php';
+						exit(0);
+					}
+				}
+
+				if ('new' == $_REQUEST['state'][$_REQUEST['id']]) {
+					redirect_header(
+						$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+						10,
+						_AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY
+					);
+				} else {
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
+				}
+				exit(0);
+
+				break;
+			case 'savelist':
+				$albumsHandler = Helper::getInstance()->getHandler('Albums');
+				foreach ($_REQUEST['id'] as $id) {
+					$album = $albumsHandler->get($id);
+					$album->setVars($_POST[$id]);
+					if (!$albumsHandler->insert($album)) {
+						redirect_header(
+							$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+							10,
+							_AM_SONGLIST_MSG_ALBUMS_FAILEDTOSAVE
+						);
+						exit(0);
+					}
+				}
+				redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
+				exit(0);
+				break;
+			case 'delete':
+				$albumsHandler = Helper::getInstance()->getHandler('Albums');
+				$id            = 0;
+				if (Request::hasVar('id', 'POST') && $id = Request::getInt('id', 0, 'POST')) {
+					$album = $albumsHandler->get($id);
+					if (!$albumsHandler->delete($album)) {
+						redirect_header(
+							$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+							10,
+							_AM_SONGLIST_MSG_ALBUMS_FAILEDTODELETE
+						);
+						exit(0);
+					}
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_DELETED);
+					exit(0);
+				}
+				$album = $albumsHandler->get(Request::getInt('id', 0, 'REQUEST'));
+				xoops_confirm(
+					['id' => $_REQUEST['id'], 'op' => $_REQUEST['op'], 'fct' => $_REQUEST['fct'], 'limit' => $_REQUEST['limit'], 'start' => $_REQUEST['start'], 'order' => $_REQUEST['order'], 'sort' => $_REQUEST['sort'], 'filter' => $_REQUEST['filter']],
+					$_SERVER['SCRIPT_NAME'],
+					sprintf(_AM_SONGLIST_MSG_ALBUMS_DELETE, $album->getVar('title'))
+				);
+
+				break;
+		}
+		break;
 }
 
 xoops_cp_footer();

Spacing +24 added lines, -24 removed lines

@@ -7,7 +7,7 @@ 
 use XoopsModules\Songlist\AlbumsHandler;
 use XoopsModules\Songlist\Form\FormController;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
 xoops_loadLanguage('admin', 'songlist');
 
@@ -18,8 +18,8 @@ 
 $limit  = Request::getInt('limit', 30, 'REQUEST');
 $start  = Request::getInt('start', 0, 'REQUEST');
 $order  = !empty($_REQUEST['order']) ? $_REQUEST['order'] : 'DESC';
-$sort   = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
-$filter = !empty($_REQUEST['filter']) ? '' . $_REQUEST['filter'] . '' : '1,1';
+$sort   = !empty($_REQUEST['sort']) ? ''.$_REQUEST['sort'].'' : 'created';
+$filter = !empty($_REQUEST['filter']) ? ''.$_REQUEST['filter'].'' : '1,1';
 
 switch ($op) {
     default:
@@ -35,14 +35,14 @@ 
 
                 $criteria        = $albumsHandler->getFilterCriteria($GLOBALS['filter']);
                 $ttl             = $albumsHandler->getCount($criteria);
-                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
+                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? ''.$_REQUEST['sort'].'' : 'created';
 
-                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
+                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit='.$GLOBALS['limit'].'&sort='.$GLOBALS['sort'].'&order='.$GLOBALS['order'].'&op='.$GLOBALS['op'].'&fct='.$GLOBALS['fct'].'&filter='.$GLOBALS['filter']);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
 
                 foreach ($albumsHandler->filterFields() as $id => $key) {
                     $GLOBALS['xoopsTpl']->assign(
-                        \mb_strtolower(str_replace('-', '_', $key) . '_th'),
+                        \mb_strtolower(str_replace('-', '_', $key).'_th'),
                         '<a href="'
                         . $_SERVER['SCRIPT_NAME']
                         . '?start='
@@ -52,16 +52,16 @@ 
                         . '&sort='
                         . $key
                         . '&order='
-                        . (($key == $GLOBALS['sort']) ? ('DESC' == $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
+                        . (($key==$GLOBALS['sort']) ? ('DESC'==$GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
                         . '&op='
                         . $GLOBALS['op']
                         . '&filter='
                         . $GLOBALS['filter']
                         . '">'
-                        . (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
+                        . (defined('_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key)))
                         . '</a>'
                     );
-                    $GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $albumsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
+                    $GLOBALS['xoopsTpl']->assign('filter_'.\mb_strtolower(str_replace('-', '_', $key)).'_th', $albumsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
                 }
 
                 $GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
@@ -73,7 +73,7 @@ 
 
                 $criteria->setStart($GLOBALS['start']);
                 $criteria->setLimit($GLOBALS['limit']);
-                $criteria->setSort('`' . $GLOBALS['sort'] . '`');
+                $criteria->setSort('`'.$GLOBALS['sort'].'`');
                 $criteria->setOrder($GLOBALS['order']);
 
                 $albumsArray = $albumsHandler->getObjects($criteria, true);
@@ -114,7 +114,7 @@ 
                 $album->setVars($_POST[$id]);
 
                 if (!$id = $albumsHandler->insert($album)) {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_FAILEDTOSAVE);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_FAILEDTOSAVE);
                     exit(0);
                 }
                 if (Request::hasVar('image', 'FILES') && !empty($_FILES['image']['name'])) {
@@ -132,7 +132,7 @@ 
 //                    require_once $GLOBALS['xoops']->path('modules/songlist/include/uploader.php');
                     $album    = $albumsHandler->get($id);
                     $uploader = new Uploader(
-                        $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']. 'albums'),
+                        $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'].'albums'),
                         explode('|', $GLOBALS['songlistModuleConfig']['allowed_mimetype']),
                         $GLOBALS['songlistModuleConfig']['filesize_upload'],
                         0,
@@ -140,7 +140,7 @@ 
                         explode('|', $GLOBALS['songlistModuleConfig']['allowed_extensions'])
                     );
                     try {
-                        $uploader->setPrefix(mb_substr(md5((string)microtime(true)), random_int(0, 20), 13));
+                        $uploader->setPrefix(mb_substr(md5((string) microtime(true)), random_int(0, 20), 13));
                     } catch (Exception $e) {
                     }
 
@@ -149,33 +149,33 @@ 
                             $adminObject = Admin::getInstance();
                             $adminObject->displayNavigation(basename(__FILE__));
                             echo $uploader->getErrors();
-                            require __DIR__ . '/admin_footer.php';
+                            require __DIR__.'/admin_footer.php';
                             exit(0);
                         }
                         if (mb_strlen($album->getVar('image'))) {
-                            unlink($GLOBALS['xoops']->path($album->getVar('path')) . $album->getVar('image'));
+                            unlink($GLOBALS['xoops']->path($album->getVar('path')).$album->getVar('image'));
                         }
 
-                        $album->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']. 'albums/') ;
+                        $album->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas'].'albums/');
                         $album->setVar('image', $uploader->getSavedFileName());
                         @$albumsHandler->insert($album);
                     } else {
                         $adminObject = Admin::getInstance();
                         $adminObject->displayNavigation(basename(__FILE__));
                         echo $uploader->getErrors();
-                        require __DIR__ . '/admin_footer.php';
+                        require __DIR__.'/admin_footer.php';
                         exit(0);
                     }
                 }
 
-                if ('new' == $_REQUEST['state'][$_REQUEST['id']]) {
+                if ('new'==$_REQUEST['state'][$_REQUEST['id']]) {
                     redirect_header(
-                        $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                        $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'].'&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                         10,
                         _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY
                     );
                 } else {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
                 }
                 exit(0);
 
@@ -187,14 +187,14 @@ 
                     $album->setVars($_POST[$id]);
                     if (!$albumsHandler->insert($album)) {
                         redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                            $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                             10,
                             _AM_SONGLIST_MSG_ALBUMS_FAILEDTOSAVE
                         );
                         exit(0);
                     }
                 }
-                redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
+                redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
                 exit(0);
                 break;
             case 'delete':
@@ -204,13 +204,13 @@ 
                     $album = $albumsHandler->get($id);
                     if (!$albumsHandler->delete($album)) {
                         redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                            $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                             10,
                             _AM_SONGLIST_MSG_ALBUMS_FAILEDTODELETE
                         );
                         exit(0);
                     }
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_DELETED);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_DELETED);
                     exit(0);
                 }
                 $album = $albumsHandler->get(Request::getInt('id', 0, 'REQUEST'));

admin/votes.php

Indentation +189 added lines, -189 removed lines

@@ -22,102 +22,102 @@ 
 $filter = !empty($_REQUEST['filter']) ? '' . $_REQUEST['filter'] . '' : '1,1';
 
 switch ($op) {
-    default:
-    case 'votes':
-        switch ($fct) {
-            default:
-            case 'list':
-                $adminObject = Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-
-                /** @var VotesHandler $votesHandler */
-                $votesHandler = Helper::getInstance()->getHandler('Votes');
-
-                $criteria        = $votesHandler->getFilterCriteria($GLOBALS['filter']);
-                $ttl             = $votesHandler->getCount($criteria);
-                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
-
-                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
-                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
-
-                foreach ($votesHandler->filterFields() as $id => $key) {
-                    $GLOBALS['xoopsTpl']->assign(
-                        \mb_strtolower(str_replace('-', '_', $key) . '_th'),
-                        '<a href="'
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?start='
-                        . $GLOBALS['start']
-                        . '&limit='
-                        . $GLOBALS['limit']
-                        . '&sort='
-                        . $key
-                        . '&order='
-                        . (($key == $GLOBALS['sort']) ? ('DESC' === $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
-                        . '&op='
-                        . $GLOBALS['op']
-                        . '&filter='
-                        . $GLOBALS['filter']
-                        . '">'
-                        . (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
-                        . '</a>'
-                    );
-                    $GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $votesHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
-                }
-
-                $GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
-                $GLOBALS['xoopsTpl']->assign('start', $GLOBALS['start']);
-                $GLOBALS['xoopsTpl']->assign('order', $GLOBALS['order']);
-                $GLOBALS['xoopsTpl']->assign('sort', $GLOBALS['sort']);
-                $GLOBALS['xoopsTpl']->assign('filter', $GLOBALS['filter']);
-                $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-
-                $criteria->setStart($GLOBALS['start']);
-                $criteria->setLimit($GLOBALS['limit']);
-                $criteria->setSort('`' . $GLOBALS['sort'] . '`');
-                $criteria->setOrder($GLOBALS['order']);
-
-                $votess = $votesHandler->getObjects($criteria, true);
-                foreach ($votess as $cid => $votes) {
-                    if (is_object($votes)) {
-                        $GLOBALS['xoopsTpl']->append('categories', $votes->toArray());
-                    }
-                }
-                //$GLOBALS['xoopsTpl']->assign('form', FormController::votes_get_form(false));
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                $GLOBALS['xoopsTpl']->display('db:songlist_cpanel_votes_list.tpl');
-                break;
-            case 'new':
-            case 'edit':
-                $adminObject = Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-
-                $votesHandler = Helper::getInstance()->getHandler('Votes');
-                if (Request::hasVar('id', 'REQUEST')) {
-                    $votes = $votesHandler->get(Request::getInt('id', 0, 'REQUEST'));
-                } else {
-                    $votes = $votesHandler->create();
-                }
-
-                $GLOBALS['xoopsTpl']->assign('form', $votes->getForm());
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                $GLOBALS['xoopsTpl']->display('db:songlist_cpanel_votes_edit.tpl');
-                break;
-            case 'save':
-                $votesHandler = Helper::getInstance()->getHandler('Votes');
-                $id           = 0;
-                $id           = Request::getInt('id', 0, 'REQUEST');
-                if ($id) {
-                    $votes = $votesHandler->get($id);
-                } else {
-                    $votes = $votesHandler->create();
-                }
-                $votes->setVars($_POST[$id]);
-
-                if (!$id = $votesHandler->insert($votes)) {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_FAILEDTOSAVE);
-                    exit(0);
-                }
-                if (Request::hasVar('image', 'FILES') && !empty($_FILES['image']['name'])) {
+	default:
+	case 'votes':
+		switch ($fct) {
+			default:
+			case 'list':
+				$adminObject = Admin::getInstance();
+				$adminObject->displayNavigation(basename(__FILE__));
+
+				/** @var VotesHandler $votesHandler */
+				$votesHandler = Helper::getInstance()->getHandler('Votes');
+
+				$criteria        = $votesHandler->getFilterCriteria($GLOBALS['filter']);
+				$ttl             = $votesHandler->getCount($criteria);
+				$GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
+
+				$pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
+				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
+
+				foreach ($votesHandler->filterFields() as $id => $key) {
+					$GLOBALS['xoopsTpl']->assign(
+						\mb_strtolower(str_replace('-', '_', $key) . '_th'),
+						'<a href="'
+						. $_SERVER['SCRIPT_NAME']
+						. '?start='
+						. $GLOBALS['start']
+						. '&limit='
+						. $GLOBALS['limit']
+						. '&sort='
+						. $key
+						. '&order='
+						. (($key == $GLOBALS['sort']) ? ('DESC' === $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
+						. '&op='
+						. $GLOBALS['op']
+						. '&filter='
+						. $GLOBALS['filter']
+						. '">'
+						. (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
+						. '</a>'
+					);
+					$GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $votesHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
+				}
+
+				$GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
+				$GLOBALS['xoopsTpl']->assign('start', $GLOBALS['start']);
+				$GLOBALS['xoopsTpl']->assign('order', $GLOBALS['order']);
+				$GLOBALS['xoopsTpl']->assign('sort', $GLOBALS['sort']);
+				$GLOBALS['xoopsTpl']->assign('filter', $GLOBALS['filter']);
+				$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+
+				$criteria->setStart($GLOBALS['start']);
+				$criteria->setLimit($GLOBALS['limit']);
+				$criteria->setSort('`' . $GLOBALS['sort'] . '`');
+				$criteria->setOrder($GLOBALS['order']);
+
+				$votess = $votesHandler->getObjects($criteria, true);
+				foreach ($votess as $cid => $votes) {
+					if (is_object($votes)) {
+						$GLOBALS['xoopsTpl']->append('categories', $votes->toArray());
+					}
+				}
+				//$GLOBALS['xoopsTpl']->assign('form', FormController::votes_get_form(false));
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				$GLOBALS['xoopsTpl']->display('db:songlist_cpanel_votes_list.tpl');
+				break;
+			case 'new':
+			case 'edit':
+				$adminObject = Admin::getInstance();
+				$adminObject->displayNavigation(basename(__FILE__));
+
+				$votesHandler = Helper::getInstance()->getHandler('Votes');
+				if (Request::hasVar('id', 'REQUEST')) {
+					$votes = $votesHandler->get(Request::getInt('id', 0, 'REQUEST'));
+				} else {
+					$votes = $votesHandler->create();
+				}
+
+				$GLOBALS['xoopsTpl']->assign('form', $votes->getForm());
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				$GLOBALS['xoopsTpl']->display('db:songlist_cpanel_votes_edit.tpl');
+				break;
+			case 'save':
+				$votesHandler = Helper::getInstance()->getHandler('Votes');
+				$id           = 0;
+				$id           = Request::getInt('id', 0, 'REQUEST');
+				if ($id) {
+					$votes = $votesHandler->get($id);
+				} else {
+					$votes = $votesHandler->create();
+				}
+				$votes->setVars($_POST[$id]);
+
+				if (!$id = $votesHandler->insert($votes)) {
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_FAILEDTOSAVE);
+					exit(0);
+				}
+				if (Request::hasVar('image', 'FILES') && !empty($_FILES['image']['name'])) {
 //                    if (!is_dir($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']))) {
 //                        foreach (explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders) {
 //                            foreach (explode('/', $folders) as $folder) {
@@ -130,99 +130,99 @@ 
 //                    }
 
 //                    require_once $GLOBALS['xoops']->path('modules/songlist/include/uploader.php');
-                    $votes    = $votesHandler->get($id);
-                    $uploader = new Uploader(
-                        $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']),
-                        explode('|', $GLOBALS['songlistModuleConfig']['allowed_mimetype']),
-                        $GLOBALS['songlistModuleConfig']['filesize_upload'],
-                        0,
-                        0,
-                        explode('|', $GLOBALS['songlistModuleConfig']['allowed_extensions'])
-                    );
-                    try {
-                        $uploader->setPrefix(mb_substr(md5((string)microtime(true)), random_int(0, 20), 13));
-                    } catch (Exception $e) {
-                    }
-
-                    if ($uploader->fetchMedia('image')) {
-                        if (!$uploader->upload()) {
-                            $adminObject = Admin::getInstance();
-                            $adminObject->displayNavigation(basename(__FILE__));
-                            echo $uploader->getErrors();
-                            require __DIR__ . '/admin_footer.php';
-                            exit(0);
-                        }
-                        if (mb_strlen($votes->getVar('image'))) {
-                            unlink($GLOBALS['xoops']->path($votes->getVar('path')) . $votes->getVar('image'));
-                        }
-
-                        $votes->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
-                        $votes->setVar('image', $uploader->getSavedFileName());
-                        @$votesHandler->insert($votes);
-                    } else {
-                        $adminObject = Admin::getInstance();
-                        $adminObject->displayNavigation(basename(__FILE__));
-                        echo $uploader->getErrors();
-                        require __DIR__ . '/admin_footer.php';
-                        exit(0);
-                    }
-                }
-
-                if ('new' === $_REQUEST['state'][$_REQUEST['id']]) {
-                    redirect_header(
-                        $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                        10,
-                        _AM_SONGLIST_MSG_VOTES_SAVEDOKEY
-                    );
-                } else {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
-                }
-                exit(0);
-
-                break;
-            case 'savelist':
-                $votesHandler = Helper::getInstance()->getHandler('Votes');
-                foreach ($_REQUEST['id'] as $id) {
-                    $votes = $votesHandler->get($id);
-                    $votes->setVars($_POST[$id]);
-                    if (!$votesHandler->insert($votes)) {
-                        redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                            10,
-                            _AM_SONGLIST_MSG_VOTES_FAILEDTOSAVE
-                        );
-                        exit(0);
-                    }
-                }
-                redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
-                exit(0);
-                break;
-            case 'delete':
-                $votesHandler = Helper::getInstance()->getHandler('Votes');
-                $id           = 0;
-                if (Request::hasVar('id', 'POST') && $id = Request::getInt('id', 0, 'POST')) {
-                    $votes = $votesHandler->get($id);
-                    if (!$votesHandler->delete($votes)) {
-                        redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                            10,
-                            _AM_SONGLIST_MSG_VOTES_FAILEDTODELETE
-                        );
-                        exit(0);
-                    }
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_DELETED);
-                    exit(0);
-                }
-                $votes = $votesHandler->get(Request::getInt('id', 0, 'REQUEST'));
-                xoops_confirm(
-                    ['id' => $_REQUEST['id'], 'op' => $_REQUEST['op'], 'fct' => $_REQUEST['fct'], 'limit' => $_REQUEST['limit'], 'start' => $_REQUEST['start'], 'order' => $_REQUEST['order'], 'sort' => $_REQUEST['sort'], 'filter' => $_REQUEST['filter']],
-                    $_SERVER['SCRIPT_NAME'],
-                    sprintf(_AM_SONGLIST_MSG_VOTES_DELETE, $votes->getVar('name'))
-                );
-
-                break;
-        }
-        break;
+					$votes    = $votesHandler->get($id);
+					$uploader = new Uploader(
+						$GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']),
+						explode('|', $GLOBALS['songlistModuleConfig']['allowed_mimetype']),
+						$GLOBALS['songlistModuleConfig']['filesize_upload'],
+						0,
+						0,
+						explode('|', $GLOBALS['songlistModuleConfig']['allowed_extensions'])
+					);
+					try {
+						$uploader->setPrefix(mb_substr(md5((string)microtime(true)), random_int(0, 20), 13));
+					} catch (Exception $e) {
+					}
+
+					if ($uploader->fetchMedia('image')) {
+						if (!$uploader->upload()) {
+							$adminObject = Admin::getInstance();
+							$adminObject->displayNavigation(basename(__FILE__));
+							echo $uploader->getErrors();
+							require __DIR__ . '/admin_footer.php';
+							exit(0);
+						}
+						if (mb_strlen($votes->getVar('image'))) {
+							unlink($GLOBALS['xoops']->path($votes->getVar('path')) . $votes->getVar('image'));
+						}
+
+						$votes->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
+						$votes->setVar('image', $uploader->getSavedFileName());
+						@$votesHandler->insert($votes);
+					} else {
+						$adminObject = Admin::getInstance();
+						$adminObject->displayNavigation(basename(__FILE__));
+						echo $uploader->getErrors();
+						require __DIR__ . '/admin_footer.php';
+						exit(0);
+					}
+				}
+
+				if ('new' === $_REQUEST['state'][$_REQUEST['id']]) {
+					redirect_header(
+						$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+						10,
+						_AM_SONGLIST_MSG_VOTES_SAVEDOKEY
+					);
+				} else {
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
+				}
+				exit(0);
+
+				break;
+			case 'savelist':
+				$votesHandler = Helper::getInstance()->getHandler('Votes');
+				foreach ($_REQUEST['id'] as $id) {
+					$votes = $votesHandler->get($id);
+					$votes->setVars($_POST[$id]);
+					if (!$votesHandler->insert($votes)) {
+						redirect_header(
+							$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+							10,
+							_AM_SONGLIST_MSG_VOTES_FAILEDTOSAVE
+						);
+						exit(0);
+					}
+				}
+				redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
+				exit(0);
+				break;
+			case 'delete':
+				$votesHandler = Helper::getInstance()->getHandler('Votes');
+				$id           = 0;
+				if (Request::hasVar('id', 'POST') && $id = Request::getInt('id', 0, 'POST')) {
+					$votes = $votesHandler->get($id);
+					if (!$votesHandler->delete($votes)) {
+						redirect_header(
+							$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+							10,
+							_AM_SONGLIST_MSG_VOTES_FAILEDTODELETE
+						);
+						exit(0);
+					}
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_DELETED);
+					exit(0);
+				}
+				$votes = $votesHandler->get(Request::getInt('id', 0, 'REQUEST'));
+				xoops_confirm(
+					['id' => $_REQUEST['id'], 'op' => $_REQUEST['op'], 'fct' => $_REQUEST['fct'], 'limit' => $_REQUEST['limit'], 'start' => $_REQUEST['start'], 'order' => $_REQUEST['order'], 'sort' => $_REQUEST['sort'], 'filter' => $_REQUEST['filter']],
+					$_SERVER['SCRIPT_NAME'],
+					sprintf(_AM_SONGLIST_MSG_VOTES_DELETE, $votes->getVar('name'))
+				);
+
+				break;
+		}
+		break;
 }
 
 xoops_cp_footer();

Spacing +22 added lines, -22 removed lines

@@ -7,7 +7,7 @@ 
 use XoopsModules\Songlist\VotesHandler;
 use XoopsModules\Songlist\Form\FormController;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
 xoops_loadLanguage('admin', 'songlist');
 
@@ -18,8 +18,8 @@ 
 $limit  = Request::getInt('limit', 30, 'REQUEST');
 $start  = Request::getInt('start', 0, 'REQUEST');
 $order  = !empty($_REQUEST['order']) ? $_REQUEST['order'] : 'DESC';
-$sort   = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
-$filter = !empty($_REQUEST['filter']) ? '' . $_REQUEST['filter'] . '' : '1,1';
+$sort   = !empty($_REQUEST['sort']) ? ''.$_REQUEST['sort'].'' : 'created';
+$filter = !empty($_REQUEST['filter']) ? ''.$_REQUEST['filter'].'' : '1,1';
 
 switch ($op) {
     default:
@@ -35,14 +35,14 @@ 
 
                 $criteria        = $votesHandler->getFilterCriteria($GLOBALS['filter']);
                 $ttl             = $votesHandler->getCount($criteria);
-                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
+                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? ''.$_REQUEST['sort'].'' : 'created';
 
-                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
+                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit='.$GLOBALS['limit'].'&sort='.$GLOBALS['sort'].'&order='.$GLOBALS['order'].'&op='.$GLOBALS['op'].'&fct='.$GLOBALS['fct'].'&filter='.$GLOBALS['filter']);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
 
                 foreach ($votesHandler->filterFields() as $id => $key) {
                     $GLOBALS['xoopsTpl']->assign(
-                        \mb_strtolower(str_replace('-', '_', $key) . '_th'),
+                        \mb_strtolower(str_replace('-', '_', $key).'_th'),
                         '<a href="'
                         . $_SERVER['SCRIPT_NAME']
                         . '?start='
@@ -52,16 +52,16 @@ 
                         . '&sort='
                         . $key
                         . '&order='
-                        . (($key == $GLOBALS['sort']) ? ('DESC' === $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
+                        . (($key==$GLOBALS['sort']) ? ('DESC'===$GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
                         . '&op='
                         . $GLOBALS['op']
                         . '&filter='
                         . $GLOBALS['filter']
                         . '">'
-                        . (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
+                        . (defined('_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key)))
                         . '</a>'
                     );
-                    $GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $votesHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
+                    $GLOBALS['xoopsTpl']->assign('filter_'.\mb_strtolower(str_replace('-', '_', $key)).'_th', $votesHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
                 }
 
                 $GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
@@ -73,7 +73,7 @@ 
 
                 $criteria->setStart($GLOBALS['start']);
                 $criteria->setLimit($GLOBALS['limit']);
-                $criteria->setSort('`' . $GLOBALS['sort'] . '`');
+                $criteria->setSort('`'.$GLOBALS['sort'].'`');
                 $criteria->setOrder($GLOBALS['order']);
 
                 $votess = $votesHandler->getObjects($criteria, true);
@@ -114,7 +114,7 @@ 
                 $votes->setVars($_POST[$id]);
 
                 if (!$id = $votesHandler->insert($votes)) {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_FAILEDTOSAVE);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_FAILEDTOSAVE);
                     exit(0);
                 }
                 if (Request::hasVar('image', 'FILES') && !empty($_FILES['image']['name'])) {
@@ -140,7 +140,7 @@ 
                         explode('|', $GLOBALS['songlistModuleConfig']['allowed_extensions'])
                     );
                     try {
-                        $uploader->setPrefix(mb_substr(md5((string)microtime(true)), random_int(0, 20), 13));
+                        $uploader->setPrefix(mb_substr(md5((string) microtime(true)), random_int(0, 20), 13));
                     } catch (Exception $e) {
                     }
 
@@ -149,11 +149,11 @@ 
                             $adminObject = Admin::getInstance();
                             $adminObject->displayNavigation(basename(__FILE__));
                             echo $uploader->getErrors();
-                            require __DIR__ . '/admin_footer.php';
+                            require __DIR__.'/admin_footer.php';
                             exit(0);
                         }
                         if (mb_strlen($votes->getVar('image'))) {
-                            unlink($GLOBALS['xoops']->path($votes->getVar('path')) . $votes->getVar('image'));
+                            unlink($GLOBALS['xoops']->path($votes->getVar('path')).$votes->getVar('image'));
                         }
 
                         $votes->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
@@ -163,19 +163,19 @@ 
                         $adminObject = Admin::getInstance();
                         $adminObject->displayNavigation(basename(__FILE__));
                         echo $uploader->getErrors();
-                        require __DIR__ . '/admin_footer.php';
+                        require __DIR__.'/admin_footer.php';
                         exit(0);
                     }
                 }
 
-                if ('new' === $_REQUEST['state'][$_REQUEST['id']]) {
+                if ('new'===$_REQUEST['state'][$_REQUEST['id']]) {
                     redirect_header(
-                        $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                        $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'].'&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                         10,
                         _AM_SONGLIST_MSG_VOTES_SAVEDOKEY
                     );
                 } else {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
                 }
                 exit(0);
 
@@ -187,14 +187,14 @@ 
                     $votes->setVars($_POST[$id]);
                     if (!$votesHandler->insert($votes)) {
                         redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                            $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                             10,
                             _AM_SONGLIST_MSG_VOTES_FAILEDTOSAVE
                         );
                         exit(0);
                     }
                 }
-                redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
+                redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_SAVEDOKEY);
                 exit(0);
                 break;
             case 'delete':
@@ -204,13 +204,13 @@ 
                     $votes = $votesHandler->get($id);
                     if (!$votesHandler->delete($votes)) {
                         redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                            $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                             10,
                             _AM_SONGLIST_MSG_VOTES_FAILEDTODELETE
                         );
                         exit(0);
                     }
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_DELETED);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOTES_DELETED);
                     exit(0);
                 }
                 $votes = $votesHandler->get(Request::getInt('id', 0, 'REQUEST'));

admin/field_permissions.php

Indentation +58 added lines, -58 removed lines

@@ -38,28 +38,28 @@ 
 
 $perm_desc = '';
 switch ($op) {
-    case 'visibility':
-        //header("Location: visibility.php");
-        redirect_header('field_visibility.php', 0, _AM_SONGLIST_PROF_VISIBLE);
-        break;
-    case 'edit':
-        $title_of_form = _AM_SONGLIST_PROF_EDITABLE;
-        $perm_name     = 'songlist_edit';
-        $restriction   = 'field_edit';
-        $anonymous     = false;
-        break;
-    case 'post':
-        $title_of_form = _AM_SONGLIST_PROF_POST;
-        $perm_name     = 'songlist_post';
-        $restriction   = '';
-        $anonymous     = true;
-        break;
-    case 'search':
-        $title_of_form = _AM_SONGLIST_PROF_SEARCH;
-        $perm_name     = 'songlist_search';
-        $restriction   = '';
-        $anonymous     = true;
-        break;
+	case 'visibility':
+		//header("Location: visibility.php");
+		redirect_header('field_visibility.php', 0, _AM_SONGLIST_PROF_VISIBLE);
+		break;
+	case 'edit':
+		$title_of_form = _AM_SONGLIST_PROF_EDITABLE;
+		$perm_name     = 'songlist_edit';
+		$restriction   = 'field_edit';
+		$anonymous     = false;
+		break;
+	case 'post':
+		$title_of_form = _AM_SONGLIST_PROF_POST;
+		$perm_name     = 'songlist_post';
+		$restriction   = '';
+		$anonymous     = true;
+		break;
+	case 'search':
+		$title_of_form = _AM_SONGLIST_PROF_SEARCH;
+		$perm_name     = 'songlist_search';
+		$restriction   = '';
+		$anonymous     = true;
+		break;
 }
 
 $module_id = $GLOBALS['songlistModule']->getVar('mid');
@@ -67,43 +67,43 @@ 
 $form = new \XoopsGroupPermForm($title_of_form, $module_id, $perm_name, $perm_desc, 'admin/field_permissions.php', $anonymous);
 
 if ('access' === $op) {
-    /** @var \XoopsMemberHandler $memberHandler */
-    $memberHandler = xoops_getHandler('member');
-    $glist         = $memberHandler->getGroupList();
-    foreach (array_keys($glist) as $i) {
-        if (XOOPS_GROUP_ANONYMOUS != $i) {
-            $form->addItem($i, $glist[$i]);
-        }
-    }
+	/** @var \XoopsMemberHandler $memberHandler */
+	$memberHandler = xoops_getHandler('member');
+	$glist         = $memberHandler->getGroupList();
+	foreach (array_keys($glist) as $i) {
+		if (XOOPS_GROUP_ANONYMOUS != $i) {
+			$form->addItem($i, $glist[$i]);
+		}
+	}
 } else {
-    $extrasHandler = Helper::getInstance()->getHandler('Extras');
-    $tempFields    = $extrasHandler->loadFields();
-    if ($tempFields && is_array($tempFields)) {
-        $fields = array_merge([], $tempFields);
-        if ('search' !== $op) {
-            foreach (array_keys($fields) as $i) {
-                if ('' == $restriction || $fields[$i]->getVar($restriction)) {
-                    $form->addItem($fields[$i]->getVar('field_id'), xoops_substr($fields[$i]->getVar('field_title'), 0, 25));
-                }
-            }
-        } else {
-            $searchable_types = [
-                'textbox',
-                'select',
-                'radio',
-                'yesno',
-                'date',
-                'datetime',
-                'timezone',
-                'language',
-            ];
-            foreach (array_keys($fields) as $i) {
-                if (in_array($fields[$i]->getVar('field_type'), $searchable_types, true)) {
-                    $form->addItem($fields[$i]->getVar('field_id'), xoops_substr($fields[$i]->getVar('field_title'), 0, 25));
-                }
-            }
-        }
-    }
+	$extrasHandler = Helper::getInstance()->getHandler('Extras');
+	$tempFields    = $extrasHandler->loadFields();
+	if ($tempFields && is_array($tempFields)) {
+		$fields = array_merge([], $tempFields);
+		if ('search' !== $op) {
+			foreach (array_keys($fields) as $i) {
+				if ('' == $restriction || $fields[$i]->getVar($restriction)) {
+					$form->addItem($fields[$i]->getVar('field_id'), xoops_substr($fields[$i]->getVar('field_title'), 0, 25));
+				}
+			}
+		} else {
+			$searchable_types = [
+				'textbox',
+				'select',
+				'radio',
+				'yesno',
+				'date',
+				'datetime',
+				'timezone',
+				'language',
+			];
+			foreach (array_keys($fields) as $i) {
+				if (in_array($fields[$i]->getVar('field_type'), $searchable_types, true)) {
+					$form->addItem($fields[$i]->getVar('field_id'), xoops_substr($fields[$i]->getVar('field_title'), 0, 25));
+				}
+			}
+		}
+	}
 }
 $form->display();
 

Spacing +5 added lines, -5 removed lines

@@ -17,7 +17,7 @@ 
 use Xmf\Module\Admin;
 use XoopsModules\Songlist\Helper;
 
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 xoops_cp_header();
 
 $adminObject = Admin::getInstance();
@@ -66,12 +66,12 @@ 
 require_once $GLOBALS['xoops']->path('/class/xoopsform/grouppermform.php');
 $form = new \XoopsGroupPermForm($title_of_form, $module_id, $perm_name, $perm_desc, 'admin/field_permissions.php', $anonymous);
 
-if ('access' === $op) {
+if ('access'===$op) {
     /** @var \XoopsMemberHandler $memberHandler */
     $memberHandler = xoops_getHandler('member');
     $glist         = $memberHandler->getGroupList();
     foreach (array_keys($glist) as $i) {
-        if (XOOPS_GROUP_ANONYMOUS != $i) {
+        if (XOOPS_GROUP_ANONYMOUS!=$i) {
             $form->addItem($i, $glist[$i]);
         }
     }
@@ -80,9 +80,9 @@ 
     $tempFields    = $extrasHandler->loadFields();
     if ($tempFields && is_array($tempFields)) {
         $fields = array_merge([], $tempFields);
-        if ('search' !== $op) {
+        if ('search'!==$op) {
             foreach (array_keys($fields) as $i) {
-                if ('' == $restriction || $fields[$i]->getVar($restriction)) {
+                if (''==$restriction || $fields[$i]->getVar($restriction)) {
                     $form->addItem($fields[$i]->getVar('field_id'), xoops_substr($fields[$i]->getVar('field_title'), 0, 25));
                 }
             }

admin/clone.php

Indentation +43 added lines, -43 removed lines

@@ -20,9 +20,9 @@ 
 
 use Xmf\Request;
 use XoopsModules\Songlist\{
-    Common\Cloner,
-    Helper,
-    Utility
+	Common\Cloner,
+	Helper,
+	Utility
 };
 
 require_once __DIR__ . '/admin_header.php';
@@ -32,53 +32,53 @@ 
 $helper = Helper::getInstance();
 
 if ('submit' === Request::getString('op', '', 'POST')) {
-    if (!$GLOBALS['xoopsSecurity']->check()) {
-        redirect_header('clone.php', 3, implode('<br>', $GLOBALS['xoopsSecurity']->getErrors()));
-    }
+	if (!$GLOBALS['xoopsSecurity']->check()) {
+		redirect_header('clone.php', 3, implode('<br>', $GLOBALS['xoopsSecurity']->getErrors()));
+	}
 
-    //    $clone = $_POST['clone'];
-    $clone = Request::getString('clone', '', 'POST');
+	//    $clone = $_POST['clone'];
+	$clone = Request::getString('clone', '', 'POST');
 
-    //check if name is valid
-    if (empty($clone) || preg_match('/[^a-zA-Z0-9\_\-]/', $clone)) {
-        redirect_header('clone.php', 3, sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_INVALIDNAME'), $clone));
-    }
+	//check if name is valid
+	if (empty($clone) || preg_match('/[^a-zA-Z0-9\_\-]/', $clone)) {
+		redirect_header('clone.php', 3, sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_INVALIDNAME'), $clone));
+	}
 
-    // Check wether the cloned module exists or not
-    if ($clone && is_dir($GLOBALS['xoops']->path('modules/' . $clone))) {
-        redirect_header('clone.php', 3, sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_EXISTS'), $clone));
-    }
+	// Check wether the cloned module exists or not
+	if ($clone && is_dir($GLOBALS['xoops']->path('modules/' . $clone))) {
+		redirect_header('clone.php', 3, sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_EXISTS'), $clone));
+	}
 
-    $patterns = [
-        \mb_strtolower($helper->dirname())          => \mb_strtolower($clone),
-        \mb_strtoupper($helper->dirname())          => \mb_strtoupper($clone),
-        ucfirst(mb_strtolower($helper->dirname())) => ucfirst(mb_strtolower($clone)),
-    ];
+	$patterns = [
+		\mb_strtolower($helper->dirname())          => \mb_strtolower($clone),
+		\mb_strtoupper($helper->dirname())          => \mb_strtoupper($clone),
+		ucfirst(mb_strtolower($helper->dirname())) => ucfirst(mb_strtolower($clone)),
+	];
 
-    $patKeys   = array_keys($patterns);
-    $patValues = array_values($patterns);
-    Cloner::cloneFileFolder($helper->path());
-    $logocreated = Cloner::createLogo(mb_strtolower($clone));
+	$patKeys   = array_keys($patterns);
+	$patValues = array_values($patterns);
+	Cloner::cloneFileFolder($helper->path());
+	$logocreated = Cloner::createLogo(mb_strtolower($clone));
 
-    $msg = '';
-    if (is_dir($GLOBALS['xoops']->path('modules/' . \mb_strtolower($clone)))) {
-        $msg .= sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_CONGRAT'), "<a href='" . XOOPS_URL . "/modules/system/admin.php?fct=modulesadmin'>" . ucfirst(mb_strtolower($clone)) . '</a>') . "<br>\n";
-        if (!$logocreated) {
-            $msg .= constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_IMAGEFAIL');
-        }
-    } else {
-        $msg .= constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_FAIL');
-    }
-    echo $msg;
+	$msg = '';
+	if (is_dir($GLOBALS['xoops']->path('modules/' . \mb_strtolower($clone)))) {
+		$msg .= sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_CONGRAT'), "<a href='" . XOOPS_URL . "/modules/system/admin.php?fct=modulesadmin'>" . ucfirst(mb_strtolower($clone)) . '</a>') . "<br>\n";
+		if (!$logocreated) {
+			$msg .= constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_IMAGEFAIL');
+		}
+	} else {
+		$msg .= constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_FAIL');
+	}
+	echo $msg;
 } else {
-    require_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
-    $form  = new \XoopsThemeForm(sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_TITLE'), (string)$helper->getModule()->getVar('name', 'E')), 'clone', 'clone.php', 'post', true);
-    $clone = new \XoopsFormText(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_NAME'), 'clone', 20, 20, '');
-    $clone->setDescription(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_NAME_DSC'));
-    $form->addElement($clone, true);
-    $form->addElement(new \XoopsFormHidden('op', 'submit'));
-    $form->addElement(new \XoopsFormButton('', '', _SUBMIT, 'submit'));
-    $form->display();
+	require_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
+	$form  = new \XoopsThemeForm(sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_TITLE'), (string)$helper->getModule()->getVar('name', 'E')), 'clone', 'clone.php', 'post', true);
+	$clone = new \XoopsFormText(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_NAME'), 'clone', 20, 20, '');
+	$clone->setDescription(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_NAME_DSC'));
+	$form->addElement($clone, true);
+	$form->addElement(new \XoopsFormHidden('op', 'submit'));
+	$form->addElement(new \XoopsFormButton('', '', _SUBMIT, 'submit'));
+	$form->display();
 }
 
 require_once __DIR__ . '/admin_footer.php';

Spacing +13 added lines, -13 removed lines

@@ -25,13 +25,13 @@ 
     Utility
 };
 
-require_once __DIR__ . '/admin_header.php';
+require_once __DIR__.'/admin_header.php';
 
 xoops_cp_header();
 
 $helper = Helper::getInstance();
 
-if ('submit' === Request::getString('op', '', 'POST')) {
+if ('submit'===Request::getString('op', '', 'POST')) {
     if (!$GLOBALS['xoopsSecurity']->check()) {
         redirect_header('clone.php', 3, implode('<br>', $GLOBALS['xoopsSecurity']->getErrors()));
     }
@@ -41,12 +41,12 @@ 
 
     //check if name is valid
     if (empty($clone) || preg_match('/[^a-zA-Z0-9\_\-]/', $clone)) {
-        redirect_header('clone.php', 3, sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_INVALIDNAME'), $clone));
+        redirect_header('clone.php', 3, sprintf(constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_INVALIDNAME'), $clone));
     }
 
     // Check wether the cloned module exists or not
-    if ($clone && is_dir($GLOBALS['xoops']->path('modules/' . $clone))) {
-        redirect_header('clone.php', 3, sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_EXISTS'), $clone));
+    if ($clone && is_dir($GLOBALS['xoops']->path('modules/'.$clone))) {
+        redirect_header('clone.php', 3, sprintf(constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_EXISTS'), $clone));
     }
 
     $patterns = [
@@ -61,24 +61,24 @@ 
     $logocreated = Cloner::createLogo(mb_strtolower($clone));
 
     $msg = '';
-    if (is_dir($GLOBALS['xoops']->path('modules/' . \mb_strtolower($clone)))) {
-        $msg .= sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_CONGRAT'), "<a href='" . XOOPS_URL . "/modules/system/admin.php?fct=modulesadmin'>" . ucfirst(mb_strtolower($clone)) . '</a>') . "<br>\n";
+    if (is_dir($GLOBALS['xoops']->path('modules/'.\mb_strtolower($clone)))) {
+        $msg .= sprintf(constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_CONGRAT'), "<a href='".XOOPS_URL."/modules/system/admin.php?fct=modulesadmin'>".ucfirst(mb_strtolower($clone)).'</a>')."<br>\n";
         if (!$logocreated) {
-            $msg .= constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_IMAGEFAIL');
+            $msg .= constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_IMAGEFAIL');
         }
     } else {
-        $msg .= constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_FAIL');
+        $msg .= constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_FAIL');
     }
     echo $msg;
 } else {
     require_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
-    $form  = new \XoopsThemeForm(sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_TITLE'), (string)$helper->getModule()->getVar('name', 'E')), 'clone', 'clone.php', 'post', true);
-    $clone = new \XoopsFormText(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_NAME'), 'clone', 20, 20, '');
-    $clone->setDescription(constant('CO_' . $moduleDirNameUpper . '_' . 'CLONE_NAME_DSC'));
+    $form  = new \XoopsThemeForm(sprintf(constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_TITLE'), (string) $helper->getModule()->getVar('name', 'E')), 'clone', 'clone.php', 'post', true);
+    $clone = new \XoopsFormText(constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_NAME'), 'clone', 20, 20, '');
+    $clone->setDescription(constant('CO_'.$moduleDirNameUpper.'_'.'CLONE_NAME_DSC'));
     $form->addElement($clone, true);
     $form->addElement(new \XoopsFormHidden('op', 'submit'));
     $form->addElement(new \XoopsFormButton('', '', _SUBMIT, 'submit'));
     $form->display();
 }
 
-require_once __DIR__ . '/admin_footer.php';
+require_once __DIR__.'/admin_footer.php';

admin/requests.php

Indentation +150 added lines, -150 removed lines

@@ -21,156 +21,156 @@
 $filter = !empty($_REQUEST['filter']) ? '' . $_REQUEST['filter'] . '' : '1,1';
 
 switch ($op) {
-    default:
-    case 'requests':
-        switch ($fct) {
-            default:
-            case 'list':
-                $adminObject = Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-
-            /** @var RequestsHandler $requestsHandler */
-            $requestsHandler = Helper::getInstance()->getHandler('Requests');
-
-                $criteria        = $requestsHandler->getFilterCriteria($GLOBALS['filter']);
-                $ttl             = $requestsHandler->getCount($criteria);
-                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
-
-                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
-                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
-
-                foreach ($requestsHandler->filterFields() as $id => $key) {
-                    $GLOBALS['xoopsTpl']->assign(
-                        \mb_strtolower(str_replace('-', '_', $key) . '_th'),
-                        '<a href="'
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?start='
-                        . $GLOBALS['start']
-                        . '&limit='
-                        . $GLOBALS['limit']
-                        . '&sort='
-                        . $key
-                        . '&order='
-                        . (($key == $GLOBALS['sort']) ? ('DESC' === $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
-                        . '&op='
-                        . $GLOBALS['op']
-                        . '&filter='
-                        . $GLOBALS['filter']
-                        . '">'
-                        . (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
-                        . '</a>'
-                    );
-                    $GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $requestsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
-                }
-
-                $GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
-                $GLOBALS['xoopsTpl']->assign('start', $GLOBALS['start']);
-                $GLOBALS['xoopsTpl']->assign('order', $GLOBALS['order']);
-                $GLOBALS['xoopsTpl']->assign('sort', $GLOBALS['sort']);
-                $GLOBALS['xoopsTpl']->assign('filter', $GLOBALS['filter']);
-                $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-
-                $criteria->setStart($GLOBALS['start']);
-                $criteria->setLimit($GLOBALS['limit']);
-                $criteria->setSort('`' . $GLOBALS['sort'] . '`');
-                $criteria->setOrder($GLOBALS['order']);
-
-                $requestss = $requestsHandler->getObjects($criteria, true);
-                foreach ($requestss as $cid => $requests) {
-                    if (is_object($requests)) {
-                        $GLOBALS['xoopsTpl']->append('requests', $requests->toArray());
-                    }
-                }
-                $GLOBALS['xoopsTpl']->assign('form', FormController::getFormRequests(false));
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                $GLOBALS['xoopsTpl']->display('db:songlist_cpanel_requests_list.tpl');
-                break;
-            case 'new':
-            case 'edit':
-                $adminObject = Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-
-                $requestsHandler = Helper::getInstance()->getHandler('Requests');
-                if (Request::hasVar('id', 'REQUEST')) {
-                    $requests = $requestsHandler->get(Request::getInt('id', 0, 'REQUEST'));
-                } else {
-                    $requests = $requestsHandler->create();
-                }
-
-                $GLOBALS['xoopsTpl']->assign('form', $requests->getForm());
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                $GLOBALS['xoopsTpl']->display('db:songlist_cpanel_requests_edit.tpl');
-                break;
-            case 'save':
-                $requestsHandler = Helper::getInstance()->getHandler('Requests');
-                $id              = 0;
-                $id              = Request::getInt('id', 0, 'REQUEST');
-                if ($id) {
-                    $requests = $requestsHandler->get($id);
-                } else {
-                    $requests = $requestsHandler->create();
-                }
-                $requests->setVars($_POST[$id]);
-
-                if (!$id = $requestsHandler->insert($requests)) {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE);
-                    exit(0);
-                }
-                if ('new' === $_REQUEST['state'][$_REQUEST['id']]) {
-                    redirect_header(
-                        $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                        10,
-                        _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY
-                    );
-                } else {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
-                }
-                exit(0);
-
-                break;
-            case 'savelist':
-                $requestsHandler = Helper::getInstance()->getHandler('Requests');
-                foreach ($_REQUEST['id'] as $id) {
-                    $requests = $requestsHandler->get($id);
-                    $requests->setVars($_POST[$id]);
-                    if (!$requestsHandler->insert($requests)) {
-                        redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                            10,
-                            _AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE
-                        );
-                        exit(0);
-                    }
-                }
-                redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
-                exit(0);
-                break;
-            case 'delete':
-                $requestsHandler = Helper::getInstance()->getHandler('Requests');
-                $id              = 0;
-                if (Request::hasVar('id', 'POST') && $id = Request::getInt('id', 0, 'POST')) {
-                    $requests = $requestsHandler->get($id);
-                    if (!$requestsHandler->delete($requests)) {
-                        redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
-                            10,
-                            _AM_SONGLIST_MSG_REQUESTS_FAILEDTODELETE
-                        );
-                        exit(0);
-                    }
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_DELETED);
-                    exit(0);
-                }
-                $requests = $requestsHandler->get(Request::getInt('id', 0, 'REQUEST'));
-                xoops_confirm(
-                    ['id' => $_REQUEST['id'], 'op' => $_REQUEST['op'], 'fct' => $_REQUEST['fct'], 'limit' => $_REQUEST['limit'], 'start' => $_REQUEST['start'], 'order' => $_REQUEST['order'], 'sort' => $_REQUEST['sort'], 'filter' => $_REQUEST['filter']],
-                    $_SERVER['SCRIPT_NAME'],
-                    sprintf(_AM_SONGLIST_MSG_REQUESTS_DELETE, $requests->getVar('name'))
-                );
-
-                break;
-        }
-        break;
+	default:
+	case 'requests':
+		switch ($fct) {
+			default:
+			case 'list':
+				$adminObject = Admin::getInstance();
+				$adminObject->displayNavigation(basename(__FILE__));
+
+			/** @var RequestsHandler $requestsHandler */
+			$requestsHandler = Helper::getInstance()->getHandler('Requests');
+
+				$criteria        = $requestsHandler->getFilterCriteria($GLOBALS['filter']);
+				$ttl             = $requestsHandler->getCount($criteria);
+				$GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
+
+				$pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
+				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
+
+				foreach ($requestsHandler->filterFields() as $id => $key) {
+					$GLOBALS['xoopsTpl']->assign(
+						\mb_strtolower(str_replace('-', '_', $key) . '_th'),
+						'<a href="'
+						. $_SERVER['SCRIPT_NAME']
+						. '?start='
+						. $GLOBALS['start']
+						. '&limit='
+						. $GLOBALS['limit']
+						. '&sort='
+						. $key
+						. '&order='
+						. (($key == $GLOBALS['sort']) ? ('DESC' === $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
+						. '&op='
+						. $GLOBALS['op']
+						. '&filter='
+						. $GLOBALS['filter']
+						. '">'
+						. (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
+						. '</a>'
+					);
+					$GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $requestsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
+				}
+
+				$GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
+				$GLOBALS['xoopsTpl']->assign('start', $GLOBALS['start']);
+				$GLOBALS['xoopsTpl']->assign('order', $GLOBALS['order']);
+				$GLOBALS['xoopsTpl']->assign('sort', $GLOBALS['sort']);
+				$GLOBALS['xoopsTpl']->assign('filter', $GLOBALS['filter']);
+				$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+
+				$criteria->setStart($GLOBALS['start']);
+				$criteria->setLimit($GLOBALS['limit']);
+				$criteria->setSort('`' . $GLOBALS['sort'] . '`');
+				$criteria->setOrder($GLOBALS['order']);
+
+				$requestss = $requestsHandler->getObjects($criteria, true);
+				foreach ($requestss as $cid => $requests) {
+					if (is_object($requests)) {
+						$GLOBALS['xoopsTpl']->append('requests', $requests->toArray());
+					}
+				}
+				$GLOBALS['xoopsTpl']->assign('form', FormController::getFormRequests(false));
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				$GLOBALS['xoopsTpl']->display('db:songlist_cpanel_requests_list.tpl');
+				break;
+			case 'new':
+			case 'edit':
+				$adminObject = Admin::getInstance();
+				$adminObject->displayNavigation(basename(__FILE__));
+
+				$requestsHandler = Helper::getInstance()->getHandler('Requests');
+				if (Request::hasVar('id', 'REQUEST')) {
+					$requests = $requestsHandler->get(Request::getInt('id', 0, 'REQUEST'));
+				} else {
+					$requests = $requestsHandler->create();
+				}
+
+				$GLOBALS['xoopsTpl']->assign('form', $requests->getForm());
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				$GLOBALS['xoopsTpl']->display('db:songlist_cpanel_requests_edit.tpl');
+				break;
+			case 'save':
+				$requestsHandler = Helper::getInstance()->getHandler('Requests');
+				$id              = 0;
+				$id              = Request::getInt('id', 0, 'REQUEST');
+				if ($id) {
+					$requests = $requestsHandler->get($id);
+				} else {
+					$requests = $requestsHandler->create();
+				}
+				$requests->setVars($_POST[$id]);
+
+				if (!$id = $requestsHandler->insert($requests)) {
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE);
+					exit(0);
+				}
+				if ('new' === $_REQUEST['state'][$_REQUEST['id']]) {
+					redirect_header(
+						$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+						10,
+						_AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY
+					);
+				} else {
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
+				}
+				exit(0);
+
+				break;
+			case 'savelist':
+				$requestsHandler = Helper::getInstance()->getHandler('Requests');
+				foreach ($_REQUEST['id'] as $id) {
+					$requests = $requestsHandler->get($id);
+					$requests->setVars($_POST[$id]);
+					if (!$requestsHandler->insert($requests)) {
+						redirect_header(
+							$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+							10,
+							_AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE
+						);
+						exit(0);
+					}
+				}
+				redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
+				exit(0);
+				break;
+			case 'delete':
+				$requestsHandler = Helper::getInstance()->getHandler('Requests');
+				$id              = 0;
+				if (Request::hasVar('id', 'POST') && $id = Request::getInt('id', 0, 'POST')) {
+					$requests = $requestsHandler->get($id);
+					if (!$requestsHandler->delete($requests)) {
+						redirect_header(
+							$_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+							10,
+							_AM_SONGLIST_MSG_REQUESTS_FAILEDTODELETE
+						);
+						exit(0);
+					}
+					redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_DELETED);
+					exit(0);
+				}
+				$requests = $requestsHandler->get(Request::getInt('id', 0, 'REQUEST'));
+				xoops_confirm(
+					['id' => $_REQUEST['id'], 'op' => $_REQUEST['op'], 'fct' => $_REQUEST['fct'], 'limit' => $_REQUEST['limit'], 'start' => $_REQUEST['start'], 'order' => $_REQUEST['order'], 'sort' => $_REQUEST['sort'], 'filter' => $_REQUEST['filter']],
+					$_SERVER['SCRIPT_NAME'],
+					sprintf(_AM_SONGLIST_MSG_REQUESTS_DELETE, $requests->getVar('name'))
+				);
+
+				break;
+		}
+		break;
 }
 
 xoops_cp_footer();

Spacing +18 added lines, -18 removed lines

@@ -6,7 +6,7 @@ 
 use XoopsModules\Songlist\RequestsHandler;
 use XoopsModules\Songlist\Form\FormController;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
 xoops_loadLanguage('admin', 'songlist');
 
@@ -17,8 +17,8 @@ 
 $limit  = Request::getInt('limit', 30, 'REQUEST');
 $start  = Request::getInt('start', 0, 'REQUEST');
 $order  = !empty($_REQUEST['order']) ? $_REQUEST['order'] : 'DESC';
-$sort   = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
-$filter = !empty($_REQUEST['filter']) ? '' . $_REQUEST['filter'] . '' : '1,1';
+$sort   = !empty($_REQUEST['sort']) ? ''.$_REQUEST['sort'].'' : 'created';
+$filter = !empty($_REQUEST['filter']) ? ''.$_REQUEST['filter'].'' : '1,1';
 
 switch ($op) {
     default:
@@ -34,14 +34,14 @@ 
 
                 $criteria        = $requestsHandler->getFilterCriteria($GLOBALS['filter']);
                 $ttl             = $requestsHandler->getCount($criteria);
-                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : 'created';
+                $GLOBALS['sort'] = !empty($_REQUEST['sort']) ? ''.$_REQUEST['sort'].'' : 'created';
 
-                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit=' . $GLOBALS['limit'] . '&sort=' . $GLOBALS['sort'] . '&order=' . $GLOBALS['order'] . '&op=' . $GLOBALS['op'] . '&fct=' . $GLOBALS['fct'] . '&filter=' . $GLOBALS['filter']);
+                $pagenav = new \XoopsPageNav($ttl, $GLOBALS['limit'], $GLOBALS['start'], 'start', 'limit='.$GLOBALS['limit'].'&sort='.$GLOBALS['sort'].'&order='.$GLOBALS['order'].'&op='.$GLOBALS['op'].'&fct='.$GLOBALS['fct'].'&filter='.$GLOBALS['filter']);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
 
                 foreach ($requestsHandler->filterFields() as $id => $key) {
                     $GLOBALS['xoopsTpl']->assign(
-                        \mb_strtolower(str_replace('-', '_', $key) . '_th'),
+                        \mb_strtolower(str_replace('-', '_', $key).'_th'),
                         '<a href="'
                         . $_SERVER['SCRIPT_NAME']
                         . '?start='
@@ -51,16 +51,16 @@ 
                         . '&sort='
                         . $key
                         . '&order='
-                        . (($key == $GLOBALS['sort']) ? ('DESC' === $GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
+                        . (($key==$GLOBALS['sort']) ? ('DESC'===$GLOBALS['order'] ? 'ASC' : 'DESC') : $GLOBALS['order'])
                         . '&op='
                         . $GLOBALS['op']
                         . '&filter='
                         . $GLOBALS['filter']
                         . '">'
-                        . (defined('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_' . \mb_strtoupper(str_replace('-', '_', $key)))
+                        . (defined('_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key))) ? constant('_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key))) : '_AM_SONGLIST_TH_'.\mb_strtoupper(str_replace('-', '_', $key)))
                         . '</a>'
                     );
-                    $GLOBALS['xoopsTpl']->assign('filter_' . \mb_strtolower(str_replace('-', '_', $key)) . '_th', $requestsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
+                    $GLOBALS['xoopsTpl']->assign('filter_'.\mb_strtolower(str_replace('-', '_', $key)).'_th', $requestsHandler->getFilterForm($GLOBALS['filter'], $key, $GLOBALS['sort'], $GLOBALS['op'], $GLOBALS['fct']));
                 }
 
                 $GLOBALS['xoopsTpl']->assign('limit', $GLOBALS['limit']);
@@ -72,7 +72,7 @@ 
 
                 $criteria->setStart($GLOBALS['start']);
                 $criteria->setLimit($GLOBALS['limit']);
-                $criteria->setSort('`' . $GLOBALS['sort'] . '`');
+                $criteria->setSort('`'.$GLOBALS['sort'].'`');
                 $criteria->setOrder($GLOBALS['order']);
 
                 $requestss = $requestsHandler->getObjects($criteria, true);
@@ -113,17 +113,17 @@ 
                 $requests->setVars($_POST[$id]);
 
                 if (!$id = $requestsHandler->insert($requests)) {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE);
                     exit(0);
                 }
-                if ('new' === $_REQUEST['state'][$_REQUEST['id']]) {
+                if ('new'===$_REQUEST['state'][$_REQUEST['id']]) {
                     redirect_header(
-                        $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=edit&id=' . $_REQUEST['id'] . '&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                        $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'].'&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                         10,
                         _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY
                     );
                 } else {
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
                 }
                 exit(0);
 
@@ -135,14 +135,14 @@ 
                     $requests->setVars($_POST[$id]);
                     if (!$requestsHandler->insert($requests)) {
                         redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                            $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                             10,
                             _AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE
                         );
                         exit(0);
                     }
                 }
-                redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
+                redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
                 exit(0);
                 break;
             case 'delete':
@@ -152,13 +152,13 @@ 
                     $requests = $requestsHandler->get($id);
                     if (!$requestsHandler->delete($requests)) {
                         redirect_header(
-                            $_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'],
+                            $_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'],
                             10,
                             _AM_SONGLIST_MSG_REQUESTS_FAILEDTODELETE
                         );
                         exit(0);
                     }
-                    redirect_header($_SERVER['SCRIPT_NAME'] . '?op=' . $GLOBALS['op'] . '&fct=list&limit=' . $GLOBALS['limit'] . '&start=' . $GLOBALS['start'] . '&order=' . $GLOBALS['order'] . '&sort=' . $GLOBALS['sort'] . '&filter=' . $GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_DELETED);
+                    redirect_header($_SERVER['SCRIPT_NAME'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_DELETED);
                     exit(0);
                 }
                 $requests = $requestsHandler->get(Request::getInt('id', 0, 'REQUEST'));

list.tag.php

Spacing +2 added lines, -2 removed lines

@@ -1,4 +1,4 @@
 <?php declare(strict_types=1);
 
-require_once __DIR__ . '/header.php';
-require_once XOOPS_ROOT_PATH . '/modules/tag/list.tag.php';
+require_once __DIR__.'/header.php';
+require_once XOOPS_ROOT_PATH.'/modules/tag/list.tag.php';

include/oninstall.php

Indentation +50 added lines, -50 removed lines

@@ -29,19 +29,19 @@ 
  */
 function xoops_module_pre_install_songlist(\XoopsModule $module): bool
 {
-    require_once \dirname(__DIR__) . '/preloads/autoloader.php';
-    $utility      = new Utility();
-    $xoopsSuccess = $utility::checkVerXoops($module);
-    $phpSuccess   = $utility::checkVerPhp($module);
+	require_once \dirname(__DIR__) . '/preloads/autoloader.php';
+	$utility      = new Utility();
+	$xoopsSuccess = $utility::checkVerXoops($module);
+	$phpSuccess   = $utility::checkVerPhp($module);
 
-    if ($xoopsSuccess && $phpSuccess) {
-        $moduleTables = &$module->getInfo('tables');
-        foreach ($moduleTables as $table) {
-            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
-        }
-    }
+	if ($xoopsSuccess && $phpSuccess) {
+		$moduleTables = &$module->getInfo('tables');
+		foreach ($moduleTables as $table) {
+			$GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
+		}
+	}
 
-    return $xoopsSuccess && $phpSuccess;
+	return $xoopsSuccess && $phpSuccess;
 }
 
 /**
@@ -52,48 +52,48 @@ 
  */
 function xoops_module_install_songlist(\XoopsModule $module): bool
 {
-    require \dirname(__DIR__, 3) . '/mainfile.php';
+	require \dirname(__DIR__, 3) . '/mainfile.php';
 
-    $moduleDirName = \basename(\dirname(__DIR__));
-    $helper        = Helper::getInstance();
-    $utility       = new Utility();
-    $configurator  = new Songlist\Common\Configurator();
-    // Load language files
-    $helper->loadLanguage('admin');
-    $helper->loadLanguage('modinfo');
+	$moduleDirName = \basename(\dirname(__DIR__));
+	$helper        = Helper::getInstance();
+	$utility       = new Utility();
+	$configurator  = new Songlist\Common\Configurator();
+	// Load language files
+	$helper->loadLanguage('admin');
+	$helper->loadLanguage('modinfo');
 
-    // default Permission Settings ----------------------
-    global $xoopsModule;
-    $moduleId = $xoopsModule->getVar('mid');
-    // $moduleId2        = \XoopsModules\Songlist\Helper::getInstance()->getModule()->mid();
-    /** @var \XoopsGroupPermHandler $grouppermHandler */
-    $grouppermHandler = xoops_getHandler('groupperm');
-    // access rights ------------------------------------------
-    $grouppermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
+	// default Permission Settings ----------------------
+	global $xoopsModule;
+	$moduleId = $xoopsModule->getVar('mid');
+	// $moduleId2        = \XoopsModules\Songlist\Helper::getInstance()->getModule()->mid();
+	/** @var \XoopsGroupPermHandler $grouppermHandler */
+	$grouppermHandler = xoops_getHandler('groupperm');
+	// access rights ------------------------------------------
+	$grouppermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$grouppermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
+	$grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
 
-    //  ---  CREATE FOLDERS ---------------
-    if (count($configurator->uploadFolders) > 0) {
-        //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
-        foreach (array_keys($configurator->uploadFolders) as $i) {
-            $utility::createFolder($configurator->uploadFolders[$i]);
-        }
-    }
+	//  ---  CREATE FOLDERS ---------------
+	if (count($configurator->uploadFolders) > 0) {
+		//    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
+		foreach (array_keys($configurator->uploadFolders) as $i) {
+			$utility::createFolder($configurator->uploadFolders[$i]);
+		}
+	}
 
-    //  ---  COPY blank.png FILES ---------------
-    if (count($configurator->copyBlankFiles) > 0) {
-        $file = \dirname(__DIR__) . '/assets/images/blank.png';
-        foreach (array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
-            $utility::copyFile($file, $dest);
-        }
-    }
-    //delete .html entries from the tpl table
-    $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.tpl%'";
-    $GLOBALS['xoopsDB']->queryF($sql);
+	//  ---  COPY blank.png FILES ---------------
+	if (count($configurator->copyBlankFiles) > 0) {
+		$file = \dirname(__DIR__) . '/assets/images/blank.png';
+		foreach (array_keys($configurator->copyBlankFiles) as $i) {
+			$dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+			$utility::copyFile($file, $dest);
+		}
+	}
+	//delete .html entries from the tpl table
+	$sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.tpl%'";
+	$GLOBALS['xoopsDB']->queryF($sql);
 
-    return true;
+	return true;
 }

Spacing +13 added lines, -13 removed lines

@@ -29,7 +29,7 @@ 
  */
 function xoops_module_pre_install_songlist(\XoopsModule $module): bool
 {
-    require_once \dirname(__DIR__) . '/preloads/autoloader.php';
+    require_once \dirname(__DIR__).'/preloads/autoloader.php';
     $utility      = new Utility();
     $xoopsSuccess = $utility::checkVerXoops($module);
     $phpSuccess   = $utility::checkVerPhp($module);
@@ -37,7 +37,7 @@ 
     if ($xoopsSuccess && $phpSuccess) {
         $moduleTables = &$module->getInfo('tables');
         foreach ($moduleTables as $table) {
-            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
+            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS '.$GLOBALS['xoopsDB']->prefix($table).';');
         }
     }
 
@@ -52,7 +52,7 @@ 
  */
 function xoops_module_install_songlist(\XoopsModule $module): bool
 {
-    require \dirname(__DIR__, 3) . '/mainfile.php';
+    require \dirname(__DIR__, 3).'/mainfile.php';
 
     $moduleDirName = \basename(\dirname(__DIR__));
     $helper        = Helper::getInstance();
@@ -69,14 +69,14 @@ 
     /** @var \XoopsGroupPermHandler $grouppermHandler */
     $grouppermHandler = xoops_getHandler('groupperm');
     // access rights ------------------------------------------
-    $grouppermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
+    $grouppermHandler->addRight($moduleDirName.'_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
+    $grouppermHandler->addRight($moduleDirName.'_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
+    $grouppermHandler->addRight($moduleDirName.'_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
+    $grouppermHandler->addRight($moduleDirName.'_view', 1, XOOPS_GROUP_USERS, $moduleId);
+    $grouppermHandler->addRight($moduleDirName.'_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
 
     //  ---  CREATE FOLDERS ---------------
-    if (count($configurator->uploadFolders) > 0) {
+    if (count($configurator->uploadFolders)>0) {
         //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
         foreach (array_keys($configurator->uploadFolders) as $i) {
             $utility::createFolder($configurator->uploadFolders[$i]);
@@ -84,15 +84,15 @@ 
     }
 
     //  ---  COPY blank.png FILES ---------------
-    if (count($configurator->copyBlankFiles) > 0) {
-        $file = \dirname(__DIR__) . '/assets/images/blank.png';
+    if (count($configurator->copyBlankFiles)>0) {
+        $file = \dirname(__DIR__).'/assets/images/blank.png';
         foreach (array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+            $dest = $configurator->copyBlankFiles[$i].'/blank.png';
             $utility::copyFile($file, $dest);
         }
     }
     //delete .html entries from the tpl table
-    $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.tpl%'";
+    $sql = 'DELETE FROM '.$GLOBALS['xoopsDB']->prefix('tplfile')." WHERE `tpl_module` = '".$xoopsModule->getVar('dirname', 'n')."' AND `tpl_file` LIKE '%.tpl%'";
     $GLOBALS['xoopsDB']->queryF($sql);
 
     return true;

include/songlist.object.php

Indentation +4 added lines, -4 removed lines

@@ -5,12 +5,12 @@
 /** @var \XoopsConfigHandler $configHandler */
 $configHandler = xoops_getHandler('config');
 if (!isset($GLOBALS['songlistModule'])) {
-    $GLOBALS['songlistModule'] = $moduleHandler->getByDirname('songlist');
+	$GLOBALS['songlistModule'] = $moduleHandler->getByDirname('songlist');
 }
 if (is_object($GLOBALS['songlistModule'])) {
-    if (!isset($GLOBALS['songlistModuleConfig'])) {
-        $GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
-    }
+	if (!isset($GLOBALS['songlistModuleConfig'])) {
+		$GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
+	}
 }
 
 require_once $GLOBALS['xoops']->path('class/xoopsformloader.php');

include/functions.php

Indentation +580 added lines, -580 removed lines

@@ -9,611 +9,611 @@
 use XoopsModules\Songlist\Form\SelectVoiceForm;
 
 if (!function_exists('songlist_getToken')) {
-    /**
-     * @return mixed
-     */
-    function songlist_getToken()
-    {
-        $sql    = 'SELECT md5(rand()/rand()*rand()/rand()*rand()*rand()/rand()*rand()) as `salt`';
-        $result = $GLOBALS['xoopsDB']->queryF($sql);
-        [$salt] = $GLOBALS['xoopsDB']->fetchRow($result);
-
-        return $salt;
-    }
+	/**
+	 * @return mixed
+	 */
+	function songlist_getToken()
+	{
+		$sql    = 'SELECT md5(rand()/rand()*rand()/rand()*rand()*rand()/rand()*rand()) as `salt`';
+		$result = $GLOBALS['xoopsDB']->queryF($sql);
+		[$salt] = $GLOBALS['xoopsDB']->fetchRow($result);
+
+		return $salt;
+	}
 }
 
 if (!function_exists('ucword')) {
-    /**
-     * @param $string
-     * @return string
-     */
-    function ucword($string): string
-    {
-        $ret = [];
-        foreach (explode(' ', \mb_strtolower($string)) as $part) {
-            $ret[] = ucfirst($part);
-        }
-
-        return implode(' ', $ret);
-    }
+	/**
+	 * @param $string
+	 * @return string
+	 */
+	function ucword($string): string
+	{
+		$ret = [];
+		foreach (explode(' ', \mb_strtolower($string)) as $part) {
+			$ret[] = ucfirst($part);
+		}
+
+		return implode(' ', $ret);
+	}
 }
 
 if (!function_exists('songlist_getIPData')) {
-    /**
-     * @param bool|string $ip
-     * @return array
-     */
-    function songlist_getIPData($ip = false): array
-    {
-        $ret = [];
-        if (is_object($GLOBALS['xoopsUser'])) {
-            $ret['uid']   = $GLOBALS['xoopsUser']->getVar('uid');
-            $ret['uname'] = $GLOBALS['xoopsUser']->getVar('uname');
-            $ret['email'] = $GLOBALS['xoopsUser']->getVar('email');
-        } else {
-            $ret['uid']   = 0;
-            $ret['uname'] = ($_REQUEST['uname'] ?? '');
-            $ret['email'] = ($_REQUEST['email'] ?? '');
-        }
-        $ret['agent'] = $_SERVER['HTTP_USER_AGENT'];
-        if ($ip) {
-            $ret['is_proxied']   = false;
-            $ret['network-addy'] = @gethostbyaddr($ip);
-            $ret['long']         = @ip2long($ip);
-            if (is_ipv6($ip)) {
-                $ret['ip6'] = true;
-                $ret['ip4'] = false;
-            } else {
-                $ret['ip4'] = true;
-                $ret['ip6'] = false;
-            }
-            $ret['ip'] = $ip;
-        } elseif (Request::hasVar('HTTP_X_FORWARDED_FOR', 'SERVER')) {
-                $ip                  = $_SERVER['HTTP_X_FORWARDED_FOR'];
-                $ret['is_proxied']   = true;
-                $proxy_ip            = $_SERVER['REMOTE_ADDR'];
-                $ret['network-addy'] = @gethostbyaddr($ip);
-                $ret['long']         = @ip2long($ip);
-                if (is_ipv6($ip)) {
-                    $ret['ip6']       = true;
-                    $ret['proxy-ip6'] = true;
-                    $ret['ip4']       = false;
-                    $ret['proxy-ip4'] = false;
-                } else {
-                    $ret['ip4']       = true;
-                    $ret['proxy-ip4'] = true;
-                    $ret['ip6']       = false;
-                    $ret['proxy-ip6'] = false;
-                }
-                $ret['ip']       = $ip;
-                $ret['proxy-ip'] = $proxy_ip;
-            } else {
-                $ret['is_proxied']   = false;
-                $ip                  = $_SERVER['REMOTE_ADDR'];
-                $ret['network-addy'] = @gethostbyaddr($ip);
-                $ret['long']         = @ip2long($ip);
-                if (is_ipv6($ip)) {
-                    $ret['ip6'] = true;
-                    $ret['ip4'] = false;
-                } else {
-                    $ret['ip4'] = true;
-                    $ret['ip6'] = false;
-                }
-                $ret['ip'] = $ip;
-        }
-        $ret['made'] = time();
-
-        return $ret;
-    }
+	/**
+	 * @param bool|string $ip
+	 * @return array
+	 */
+	function songlist_getIPData($ip = false): array
+	{
+		$ret = [];
+		if (is_object($GLOBALS['xoopsUser'])) {
+			$ret['uid']   = $GLOBALS['xoopsUser']->getVar('uid');
+			$ret['uname'] = $GLOBALS['xoopsUser']->getVar('uname');
+			$ret['email'] = $GLOBALS['xoopsUser']->getVar('email');
+		} else {
+			$ret['uid']   = 0;
+			$ret['uname'] = ($_REQUEST['uname'] ?? '');
+			$ret['email'] = ($_REQUEST['email'] ?? '');
+		}
+		$ret['agent'] = $_SERVER['HTTP_USER_AGENT'];
+		if ($ip) {
+			$ret['is_proxied']   = false;
+			$ret['network-addy'] = @gethostbyaddr($ip);
+			$ret['long']         = @ip2long($ip);
+			if (is_ipv6($ip)) {
+				$ret['ip6'] = true;
+				$ret['ip4'] = false;
+			} else {
+				$ret['ip4'] = true;
+				$ret['ip6'] = false;
+			}
+			$ret['ip'] = $ip;
+		} elseif (Request::hasVar('HTTP_X_FORWARDED_FOR', 'SERVER')) {
+				$ip                  = $_SERVER['HTTP_X_FORWARDED_FOR'];
+				$ret['is_proxied']   = true;
+				$proxy_ip            = $_SERVER['REMOTE_ADDR'];
+				$ret['network-addy'] = @gethostbyaddr($ip);
+				$ret['long']         = @ip2long($ip);
+				if (is_ipv6($ip)) {
+					$ret['ip6']       = true;
+					$ret['proxy-ip6'] = true;
+					$ret['ip4']       = false;
+					$ret['proxy-ip4'] = false;
+				} else {
+					$ret['ip4']       = true;
+					$ret['proxy-ip4'] = true;
+					$ret['ip6']       = false;
+					$ret['proxy-ip6'] = false;
+				}
+				$ret['ip']       = $ip;
+				$ret['proxy-ip'] = $proxy_ip;
+			} else {
+				$ret['is_proxied']   = false;
+				$ip                  = $_SERVER['REMOTE_ADDR'];
+				$ret['network-addy'] = @gethostbyaddr($ip);
+				$ret['long']         = @ip2long($ip);
+				if (is_ipv6($ip)) {
+					$ret['ip6'] = true;
+					$ret['ip4'] = false;
+				} else {
+					$ret['ip4'] = true;
+					$ret['ip6'] = false;
+				}
+				$ret['ip'] = $ip;
+		}
+		$ret['made'] = time();
+
+		return $ret;
+	}
 }
 
 if (!function_exists('is_ipv6')) {
-    /**
-     * @param string $ip
-     * @return bool
-     */
-    function is_ipv6($ip = ''): bool
-    {
-        if ('' == $ip) {
-            return false;
-        }
-
-        if (mb_substr_count($ip, ':') > 0) {
-            return true;
-        }
-
-        return false;
-    }
+	/**
+	 * @param string $ip
+	 * @return bool
+	 */
+	function is_ipv6($ip = ''): bool
+	{
+		if ('' == $ip) {
+			return false;
+		}
+
+		if (mb_substr_count($ip, ':') > 0) {
+			return true;
+		}
+
+		return false;
+	}
 }
 
 if (!function_exists('songlist_getFilterElement')) {
-    /**
-     * @param        $filter
-     * @param        $field
-     * @param string $sort
-     * @param string $op
-     * @param string $fct
-     * @return bool|\XoopsModules\Songlist\Form\SelectAlbumForm|\XoopsModules\Songlist\Form\SelectArtistForm|\XoopsModules\Songlist\Form\SelectCategoryForm|\XoopsModules\Songlist\Form\SelectGenreForm|\XoopsModules\Songlist\Form\SelectVoiceForm
-     */
-    function songlist_getFilterElement($filter, $field, $sort = 'created', $op = '', $fct = '')
-    {
-        $components = songlist_getFilterURLComponents($filter, $field, $sort);
-        $ele        = false;
-        require_once __DIR__ . '/songlist.object.php';
-        switch ($field) {
-            case 'gid':
-                if ('genre' !== $op) {
-                    $ele = new SelectGenreForm('', 'filter_' . $field . '', $components['value'], 1, false);
-                    $ele->setExtra(
-                        'onchange="window.open(\''
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?'
-                        . $components['extra']
-                        . '&filter='
-                        . $components['filter']
-                        . (!empty($components['filter']) ? '|' : '')
-                        . $field
-                        . ',\'+this.options[this.selectedIndex].value'
-                        . (!empty($components['operator']) ? '+\','
-                                                             . $components['operator']
-                                                             . '\'' : '')
-                        . ',\'_self\')"'
-                    );
-                }
-                break;
-            case 'vcid':
-                if ('voice' !== $op) {
-                    $ele = new SelectVoiceForm('', 'filter_' . $field . '', $components['value'], 1, false);
-                    $ele->setExtra(
-                        'onchange="window.open(\''
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?'
-                        . $components['extra']
-                        . '&filter='
-                        . $components['filter']
-                        . (!empty($components['filter']) ? '|' : '')
-                        . $field
-                        . ',\'+this.options[this.selectedIndex].value'
-                        . (!empty($components['operator']) ? '+\','
-                                                             . $components['operator']
-                                                             . '\'' : '')
-                        . ',\'_self\')"'
-                    );
-                }
-                break;
-            case 'cid':
-                if ('category' !== $op) {
-                    $ele = new SelectCategoryForm('', 'filter_' . $field . '', $components['value'], 1, false);
-                    $ele->setExtra(
-                        'onchange="window.open(\''
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?'
-                        . $components['extra']
-                        . '&filter='
-                        . $components['filter']
-                        . (!empty($components['filter']) ? '|' : '')
-                        . $field
-                        . ',\'+this.options[this.selectedIndex].value'
-                        . (!empty($components['operator']) ? '+\','
-                                                             . $components['operator']
-                                                             . '\'' : '')
-                        . ',\'_self\')"'
-                    );
-                }
-                break;
-            case 'pid':
-                $ele = new SelectCategoryForm('', 'filter_' . $field . '', $components['value'], 1, false);
-                $ele->setExtra(
-                    'onchange="window.open(\''
-                    . $_SERVER['SCRIPT_NAME']
-                    . '?'
-                    . $components['extra']
-                    . '&filter='
-                    . $components['filter']
-                    . (!empty($components['filter']) ? '|' : '')
-                    . $field
-                    . ',\'+this.options[this.selectedIndex].value'
-                    . (!empty($components['operator']) ? '+\','
-                                                         . $components['operator']
-                                                         . '\'' : '')
-                    . ',\'_self\')"'
-                );
-                break;
-            case 'abid':
-                if ('albums' !== $op) {
-                    $ele = new SelectAlbumForm('', 'filter_' . $field . '', $components['value'], 1, false);
-                    $ele->setExtra(
-                        'onchange="window.open(\''
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?'
-                        . $components['extra']
-                        . '&filter='
-                        . $components['filter']
-                        . (!empty($components['filter']) ? '|' : '')
-                        . $field
-                        . ',\'+this.options[this.selectedIndex].value'
-                        . (!empty($components['operator']) ? '+\','
-                                                             . $components['operator']
-                                                             . '\'' : '')
-                        . ',\'_self\')"'
-                    );
-                }
-                break;
-            case 'aid':
-                if ('artists' !== $op) {
-                    $ele = new SelectArtistForm('', 'filter_' . $field . '', $components['value'], 1, false);
-                    $ele->setExtra(
-                        'onchange="window.open(\''
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?'
-                        . $components['extra']
-                        . '&filter='
-                        . $components['filter']
-                        . (!empty($components['filter']) ? '|' : '')
-                        . $field
-                        . ',\'+this.options[this.selectedIndex].value'
-                        . (!empty($components['operator']) ? '+\','
-                                                             . $components['operator']
-                                                             . '\'' : '')
-                        . ',\'_self\')"'
-                    );
-                }
-                break;
-            case 'sid':
-                if ('songs' !== $op) {
-                    $ele = new SelectSongForm('', 'filter_' . $field . '', $components['value'], 1, false);
-                    $ele->setExtra(
-                        'onchange="window.open(\''
-                        . $_SERVER['SCRIPT_NAME']
-                        . '?'
-                        . $components['extra']
-                        . '&filter='
-                        . $components['filter']
-                        . (!empty($components['filter']) ? '|' : '')
-                        . $field
-                        . ',\'+this.options[this.selectedIndex].value'
-                        . (!empty($components['operator']) ? '+\','
-                                                             . $components['operator']
-                                                             . '\'' : '')
-                        . ',\'_self\')"'
-                    );
-                }
-                break;
-            case 'name':
-            case 'title':
-            case 'artists':
-            case 'albums':
-            case 'songs':
-            case 'hits':
-            case 'rank':
-            case 'votes':
-            case 'description':
-            case 'lyrics':
-            case 'songid':
-            case 'tags':
-                $ele = new \XoopsFormElementTray('');
-                $ele->addElement(new \XoopsFormText('', 'filter_' . $field . '', 11, 40, $components['value']));
-                $button = new \XoopsFormButton('', 'button_' . $field . '', '[+]');
-                $button->setExtra(
-                    'onclick="window.open(\''
-                    . $_SERVER['SCRIPT_NAME']
-                    . '?'
-                    . $components['extra']
-                    . '&filter='
-                    . $components['filter']
-                    . (!empty($components['filter']) ? '|' : '')
-                    . $field
-                    . ',\'+$(\'#filter_'
-                    . $field
-                    . '\').val()'
-                    . (!empty($components['operator']) ? '+\','
-                                                         . $components['operator']
-                                                         . '\'' : '')
-                    . ',\'_self\')"'
-                );
-                $ele->addElement($button);
-                break;
-        }
-
-        return $ele;
-    }
+	/**
+	 * @param        $filter
+	 * @param        $field
+	 * @param string $sort
+	 * @param string $op
+	 * @param string $fct
+	 * @return bool|\XoopsModules\Songlist\Form\SelectAlbumForm|\XoopsModules\Songlist\Form\SelectArtistForm|\XoopsModules\Songlist\Form\SelectCategoryForm|\XoopsModules\Songlist\Form\SelectGenreForm|\XoopsModules\Songlist\Form\SelectVoiceForm
+	 */
+	function songlist_getFilterElement($filter, $field, $sort = 'created', $op = '', $fct = '')
+	{
+		$components = songlist_getFilterURLComponents($filter, $field, $sort);
+		$ele        = false;
+		require_once __DIR__ . '/songlist.object.php';
+		switch ($field) {
+			case 'gid':
+				if ('genre' !== $op) {
+					$ele = new SelectGenreForm('', 'filter_' . $field . '', $components['value'], 1, false);
+					$ele->setExtra(
+						'onchange="window.open(\''
+						. $_SERVER['SCRIPT_NAME']
+						. '?'
+						. $components['extra']
+						. '&filter='
+						. $components['filter']
+						. (!empty($components['filter']) ? '|' : '')
+						. $field
+						. ',\'+this.options[this.selectedIndex].value'
+						. (!empty($components['operator']) ? '+\','
+															 . $components['operator']
+															 . '\'' : '')
+						. ',\'_self\')"'
+					);
+				}
+				break;
+			case 'vcid':
+				if ('voice' !== $op) {
+					$ele = new SelectVoiceForm('', 'filter_' . $field . '', $components['value'], 1, false);
+					$ele->setExtra(
+						'onchange="window.open(\''
+						. $_SERVER['SCRIPT_NAME']
+						. '?'
+						. $components['extra']
+						. '&filter='
+						. $components['filter']
+						. (!empty($components['filter']) ? '|' : '')
+						. $field
+						. ',\'+this.options[this.selectedIndex].value'
+						. (!empty($components['operator']) ? '+\','
+															 . $components['operator']
+															 . '\'' : '')
+						. ',\'_self\')"'
+					);
+				}
+				break;
+			case 'cid':
+				if ('category' !== $op) {
+					$ele = new SelectCategoryForm('', 'filter_' . $field . '', $components['value'], 1, false);
+					$ele->setExtra(
+						'onchange="window.open(\''
+						. $_SERVER['SCRIPT_NAME']
+						. '?'
+						. $components['extra']
+						. '&filter='
+						. $components['filter']
+						. (!empty($components['filter']) ? '|' : '')
+						. $field
+						. ',\'+this.options[this.selectedIndex].value'
+						. (!empty($components['operator']) ? '+\','
+															 . $components['operator']
+															 . '\'' : '')
+						. ',\'_self\')"'
+					);
+				}
+				break;
+			case 'pid':
+				$ele = new SelectCategoryForm('', 'filter_' . $field . '', $components['value'], 1, false);
+				$ele->setExtra(
+					'onchange="window.open(\''
+					. $_SERVER['SCRIPT_NAME']
+					. '?'
+					. $components['extra']
+					. '&filter='
+					. $components['filter']
+					. (!empty($components['filter']) ? '|' : '')
+					. $field
+					. ',\'+this.options[this.selectedIndex].value'
+					. (!empty($components['operator']) ? '+\','
+														 . $components['operator']
+														 . '\'' : '')
+					. ',\'_self\')"'
+				);
+				break;
+			case 'abid':
+				if ('albums' !== $op) {
+					$ele = new SelectAlbumForm('', 'filter_' . $field . '', $components['value'], 1, false);
+					$ele->setExtra(
+						'onchange="window.open(\''
+						. $_SERVER['SCRIPT_NAME']
+						. '?'
+						. $components['extra']
+						. '&filter='
+						. $components['filter']
+						. (!empty($components['filter']) ? '|' : '')
+						. $field
+						. ',\'+this.options[this.selectedIndex].value'
+						. (!empty($components['operator']) ? '+\','
+															 . $components['operator']
+															 . '\'' : '')
+						. ',\'_self\')"'
+					);
+				}
+				break;
+			case 'aid':
+				if ('artists' !== $op) {
+					$ele = new SelectArtistForm('', 'filter_' . $field . '', $components['value'], 1, false);
+					$ele->setExtra(
+						'onchange="window.open(\''
+						. $_SERVER['SCRIPT_NAME']
+						. '?'
+						. $components['extra']
+						. '&filter='
+						. $components['filter']
+						. (!empty($components['filter']) ? '|' : '')
+						. $field
+						. ',\'+this.options[this.selectedIndex].value'
+						. (!empty($components['operator']) ? '+\','
+															 . $components['operator']
+															 . '\'' : '')
+						. ',\'_self\')"'
+					);
+				}
+				break;
+			case 'sid':
+				if ('songs' !== $op) {
+					$ele = new SelectSongForm('', 'filter_' . $field . '', $components['value'], 1, false);
+					$ele->setExtra(
+						'onchange="window.open(\''
+						. $_SERVER['SCRIPT_NAME']
+						. '?'
+						. $components['extra']
+						. '&filter='
+						. $components['filter']
+						. (!empty($components['filter']) ? '|' : '')
+						. $field
+						. ',\'+this.options[this.selectedIndex].value'
+						. (!empty($components['operator']) ? '+\','
+															 . $components['operator']
+															 . '\'' : '')
+						. ',\'_self\')"'
+					);
+				}
+				break;
+			case 'name':
+			case 'title':
+			case 'artists':
+			case 'albums':
+			case 'songs':
+			case 'hits':
+			case 'rank':
+			case 'votes':
+			case 'description':
+			case 'lyrics':
+			case 'songid':
+			case 'tags':
+				$ele = new \XoopsFormElementTray('');
+				$ele->addElement(new \XoopsFormText('', 'filter_' . $field . '', 11, 40, $components['value']));
+				$button = new \XoopsFormButton('', 'button_' . $field . '', '[+]');
+				$button->setExtra(
+					'onclick="window.open(\''
+					. $_SERVER['SCRIPT_NAME']
+					. '?'
+					. $components['extra']
+					. '&filter='
+					. $components['filter']
+					. (!empty($components['filter']) ? '|' : '')
+					. $field
+					. ',\'+$(\'#filter_'
+					. $field
+					. '\').val()'
+					. (!empty($components['operator']) ? '+\','
+														 . $components['operator']
+														 . '\'' : '')
+					. ',\'_self\')"'
+				);
+				$ele->addElement($button);
+				break;
+		}
+
+		return $ele;
+	}
 }
 
 if (!function_exists('songlist_getFilterURLComponents')) {
-    /**
-     * @param        $filter
-     * @param        $field
-     * @param string $sort
-     * @return array
-     */
-    function songlist_getFilterURLComponents($filter, $field, $sort = 'created'): array
-    {
-        $parts     = explode('|', $filter);
-        $ret       = [];
-        $value     = '';
-        $ele_value = '';
-        $operator  = '';
-        foreach ($parts as $part) {
-            $var = explode(',', $part);
-            if (count($var) > 1) {
-                if ($var[0] == $field) {
-                    $ele_value = $var[1];
-                    if (isset($var[2])) {
-                        $operator = $var[2];
-                    }
-                } elseif (1 != $var[0]) {
-                    $ret[] = implode(',', $var);
-                }
-            }
-        }
-        $pagenav          = [];
-        $pagenav['op']    = $_REQUEST['op'] ?? 'videos';
-        $pagenav['fct']   = $_REQUEST['fct'] ?? 'list';
-        $pagenav['limit'] = Request::getInt('limit', 30, 'REQUEST');
-        $pagenav['start'] = 0;
-        $pagenav['order'] = !empty($_REQUEST['order']) ? $_REQUEST['order'] : 'DESC';
-        $pagenav['sort']  = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : $sort;
-        $retb             = [];
-        foreach ($pagenav as $key => $value) {
-            $retb[] = "$key=$value";
-        }
-
-        return ['value' => $ele_value, 'field' => $field, 'operator' => $operator, 'filter' => implode('|', $ret), 'extra' => implode('&', $retb)];
-    }
+	/**
+	 * @param        $filter
+	 * @param        $field
+	 * @param string $sort
+	 * @return array
+	 */
+	function songlist_getFilterURLComponents($filter, $field, $sort = 'created'): array
+	{
+		$parts     = explode('|', $filter);
+		$ret       = [];
+		$value     = '';
+		$ele_value = '';
+		$operator  = '';
+		foreach ($parts as $part) {
+			$var = explode(',', $part);
+			if (count($var) > 1) {
+				if ($var[0] == $field) {
+					$ele_value = $var[1];
+					if (isset($var[2])) {
+						$operator = $var[2];
+					}
+				} elseif (1 != $var[0]) {
+					$ret[] = implode(',', $var);
+				}
+			}
+		}
+		$pagenav          = [];
+		$pagenav['op']    = $_REQUEST['op'] ?? 'videos';
+		$pagenav['fct']   = $_REQUEST['fct'] ?? 'list';
+		$pagenav['limit'] = Request::getInt('limit', 30, 'REQUEST');
+		$pagenav['start'] = 0;
+		$pagenav['order'] = !empty($_REQUEST['order']) ? $_REQUEST['order'] : 'DESC';
+		$pagenav['sort']  = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : $sort;
+		$retb             = [];
+		foreach ($pagenav as $key => $value) {
+			$retb[] = "$key=$value";
+		}
+
+		return ['value' => $ele_value, 'field' => $field, 'operator' => $operator, 'filter' => implode('|', $ret), 'extra' => implode('&', $retb)];
+	}
 }
 
 if (!function_exists('songlist_obj2array')) {
-    /**
-     * @param $objects
-     * @return array
-     */
-    function songlist_obj2array($objects): array
-    {
-        $ret = [];
-        foreach ((array)$objects as $key => $value) {
-            if (is_a($value, 'stdClass')) {
-                $ret[$key] = songlist_obj2array($value);
-            } elseif (is_array($value)) {
-                $ret[$key] = songlist_obj2array($value);
-            } else {
-                $ret[$key] = $value;
-            }
-        }
-
-        return $ret;
-    }
+	/**
+	 * @param $objects
+	 * @return array
+	 */
+	function songlist_obj2array($objects): array
+	{
+		$ret = [];
+		foreach ((array)$objects as $key => $value) {
+			if (is_a($value, 'stdClass')) {
+				$ret[$key] = songlist_obj2array($value);
+			} elseif (is_array($value)) {
+				$ret[$key] = songlist_obj2array($value);
+			} else {
+				$ret[$key] = $value;
+			}
+		}
+
+		return $ret;
+	}
 }
 
 if (!function_exists('songlist_shortenurl')) {
-    /**
-     * @param $url
-     * @return mixed
-     */
-    function songlist_shortenurl($url)
-    {
-        /** @var \XoopsModuleHandler $moduleHandler */
-        $moduleHandler = xoops_getHandler('module');
-        /** @var \XoopsConfigHandler $configHandler */
-        $configHandler                   = xoops_getHandler('config');
-        $GLOBALS['songlistModule']       = $moduleHandler->getByDirname('songlist');
-        $GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
-
-        if (!empty($GLOBALS['songlistModuleConfig']['bitly_username']) && !empty($GLOBALS['songlistModuleConfig']['bitly_apikey'])) {
-            $source_url = $GLOBALS['songlistModuleConfig']['bitly_apiurl'] . '/shorten?login=' . $GLOBALS['songlistModuleConfig']['bitly_username'] . '&apiKey=' . $GLOBALS['songlistModuleConfig']['bitly_apikey'] . '&format=json&longUrl=' . urlencode($url);
-            $cookies    = XOOPS_ROOT_PATH . '/uploads/songlist_' . md5($GLOBALS['songlistModuleConfig']['bitly_apikey']) . '.cookie';
-            if (!$ch = curl_init($source_url)) {
-                return $url;
-            }
-            curl_setopt($ch, CURLOPT_COOKIEJAR, $cookies);
-            curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
-            curl_setopt($ch, CURLOPT_USERAGENT, $GLOBALS['songlistModuleConfig']['user_agent']);
-            curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, $GLOBALS['songlistModuleConfig']['curl_connect_timeout']);
-            curl_setopt($ch, CURLOPT_TIMEOUT, $GLOBALS['songlistModuleConfig']['curl_timeout']);
-            $data = curl_exec($ch);
-            curl_close($ch);
-            $result                = songlist_object2array(json_decode($data));
-            $result['status_code'] = 200;
-            if ($result['status_code']) {
-                if (!empty($result['data']['url'])) {
-                    return $result['data']['url'];
-                }
-
-                return $url;
-            }
-
-            return $url;
-        }
-
-        return $url;
-    }
+	/**
+	 * @param $url
+	 * @return mixed
+	 */
+	function songlist_shortenurl($url)
+	{
+		/** @var \XoopsModuleHandler $moduleHandler */
+		$moduleHandler = xoops_getHandler('module');
+		/** @var \XoopsConfigHandler $configHandler */
+		$configHandler                   = xoops_getHandler('config');
+		$GLOBALS['songlistModule']       = $moduleHandler->getByDirname('songlist');
+		$GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
+
+		if (!empty($GLOBALS['songlistModuleConfig']['bitly_username']) && !empty($GLOBALS['songlistModuleConfig']['bitly_apikey'])) {
+			$source_url = $GLOBALS['songlistModuleConfig']['bitly_apiurl'] . '/shorten?login=' . $GLOBALS['songlistModuleConfig']['bitly_username'] . '&apiKey=' . $GLOBALS['songlistModuleConfig']['bitly_apikey'] . '&format=json&longUrl=' . urlencode($url);
+			$cookies    = XOOPS_ROOT_PATH . '/uploads/songlist_' . md5($GLOBALS['songlistModuleConfig']['bitly_apikey']) . '.cookie';
+			if (!$ch = curl_init($source_url)) {
+				return $url;
+			}
+			curl_setopt($ch, CURLOPT_COOKIEJAR, $cookies);
+			curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
+			curl_setopt($ch, CURLOPT_USERAGENT, $GLOBALS['songlistModuleConfig']['user_agent']);
+			curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, $GLOBALS['songlistModuleConfig']['curl_connect_timeout']);
+			curl_setopt($ch, CURLOPT_TIMEOUT, $GLOBALS['songlistModuleConfig']['curl_timeout']);
+			$data = curl_exec($ch);
+			curl_close($ch);
+			$result                = songlist_object2array(json_decode($data));
+			$result['status_code'] = 200;
+			if ($result['status_code']) {
+				if (!empty($result['data']['url'])) {
+					return $result['data']['url'];
+				}
+
+				return $url;
+			}
+
+			return $url;
+		}
+
+		return $url;
+	}
 }
 
 if (!function_exists('songlist_xml2array')) {
-    /**
-     * @param        $contents
-     * @param int    $get_attributes
-     * @param string $priority
-     * @return array|void
-     */
-    function songlist_xml2array($contents, $get_attributes = 1, $priority = 'tag')
-    {
-        if (!$contents) {
-            return [];
-        }
-
-        if (!function_exists('xml_parser_create')) {
-            return [];
-        }
-
-        //Get the XML parser of PHP - PHP must have this module for the parser to work
-        $parser = xml_parser_create('');
-        xml_parser_set_option($parser, XML_OPTION_TARGET_ENCODING, 'UTF-8'); # https://minutillo.com/steve/weblog/2004/6/17/php-xml-and-character-encodings-a-tale-of-sadness-rage-and-data-loss
-        xml_parser_set_option($parser, XML_OPTION_CASE_FOLDING, 0);
-        xml_parser_set_option($parser, XML_OPTION_SKIP_WHITE, 1);
-        xml_parse_into_struct($parser, trim($contents), $xml_values);
-        xml_parser_free($parser);
-
-        if (!$xml_values) {
-            return;
-        }//Hmm...
-
-        //Initializations
-        $xml_array   = [];
-        $parents     = [];
-        $opened_tags = [];
-        $arr         = [];
-
-        $current = &$xml_array; //Refference
-
-        //Go through the tags.
-        $repeated_tag_index = []; //Multiple tags with same name will be turned into an array
-        foreach ($xml_values as $data) {
-            unset($attributes, $value); //Remove existing values, or there will be trouble
-
-            //This command will extract these variables into the foreach scope
-            // tag(string), type(string), level(int), attributes(array).
-            extract($data); //We could use the array by itself, but this cooler.
-
-            $result          = [];
-            $attributes_data = [];
-
-            if (isset($value)) {
-                if ('tag' === $priority) {
-                    $result = $value;
-                } else {
-                    $result['value'] = $value;
-                } //Put the value in a assoc array if we are in the 'Attribute' mode
-            }
-
-            //Set the attributes too.
-            if (isset($attributes) and $get_attributes) {
-                foreach ($attributes as $attr => $val) {
-                    if ('tag' === $priority) {
-                        $attributes_data[$attr] = $val;
-                    } else {
-                        $result['attr'][$attr] = $val;
-                    } //Set all the attributes in a array called 'attr'
-                }
-            }
-
-            //See tag status and do the needed.
-            if ('open' === $type) {//The starting of the tag '<tag>'
-                $parent[$level - 1] = &$current;
-                if (!is_array($current) or (!array_key_exists($tag, $current))) { //Insert New tag
-                    $current[$tag] = $result;
-                    if ($attributes_data) {
-                        $current[$tag . '_attr'] = $attributes_data;
-                    }
-                    $repeated_tag_index[$tag . '_' . $level] = 1;
-
-                    $current = &$current[$tag];
-                } else { //There was another element with the same tag name
-                    if (isset($current[$tag][0])) {//If there is a 0th element it is already an array
-                        $current[$tag][$repeated_tag_index[$tag . '_' . $level]] = $result;
-                        $repeated_tag_index[$tag . '_' . $level]++;
-                    } else {//This section will make the value an array if multiple tags with the same name appear together
-                        $current[$tag]                           = [$current[$tag], $result]; //This will combine the existing item and the new item together to make an array
-                        $repeated_tag_index[$tag . '_' . $level] = 2;
-
-                        if (isset($current[$tag . '_attr'])) { //The attribute of the last(0th) tag must be moved as well
-                            $current[$tag]['0_attr'] = $current[$tag . '_attr'];
-                            unset($current[$tag . '_attr']);
-                        }
-                    }
-                    $last_item_index = $repeated_tag_index[$tag . '_' . $level] - 1;
-                    $current         = &$current[$tag][$last_item_index];
-                }
-            } elseif ('complete' === $type) { //Tags that ends in 1 line '<tag>'
-                //See if the key is already taken.
-                if (isset($current[$tag])) { //If taken, put all things inside a list(array)
-                    if (isset($current[$tag][0]) and is_array($current[$tag])) {//If it is already an array...
-                        // ...push the new element into that array.
-                        $current[$tag][$repeated_tag_index[$tag . '_' . $level]] = $result;
-
-                        if ('tag' === $priority and $get_attributes and $attributes_data) {
-                            $current[$tag][$repeated_tag_index[$tag . '_' . $level] . '_attr'] = $attributes_data;
-                        }
-                        $repeated_tag_index[$tag . '_' . $level]++;
-                    } else { //If it is not an array...
-                        $current[$tag]                           = [$current[$tag], $result]; //...Make it an array using using the existing value and the new value
-                        $repeated_tag_index[$tag . '_' . $level] = 1;
-                        if ('tag' === $priority and $get_attributes) {
-                            if (isset($current[$tag . '_attr'])) { //The attribute of the last(0th) tag must be moved as well
-                                $current[$tag]['0_attr'] = $current[$tag . '_attr'];
-                                unset($current[$tag . '_attr']);
-                            }
-
-                            if ($attributes_data) {
-                                $current[$tag][$repeated_tag_index[$tag . '_' . $level] . '_attr'] = $attributes_data;
-                            }
-                        }
-                        $repeated_tag_index[$tag . '_' . $level]++; //0 and 1 index is already taken
-                    }
-                } else { //New Key
-                    $current[$tag]                           = $result;
-                    $repeated_tag_index[$tag . '_' . $level] = 1;
-                    if ('tag' === $priority and $attributes_data) {
-                        $current[$tag . '_attr'] = $attributes_data;
-                    }
-                }
-            } elseif ('close' === $type) { //End of tag '</tag>'
-                $current = &$parent[$level - 1];
-            }
-        }
-
-        return $xml_array;
-    }
+	/**
+	 * @param        $contents
+	 * @param int    $get_attributes
+	 * @param string $priority
+	 * @return array|void
+	 */
+	function songlist_xml2array($contents, $get_attributes = 1, $priority = 'tag')
+	{
+		if (!$contents) {
+			return [];
+		}
+
+		if (!function_exists('xml_parser_create')) {
+			return [];
+		}
+
+		//Get the XML parser of PHP - PHP must have this module for the parser to work
+		$parser = xml_parser_create('');
+		xml_parser_set_option($parser, XML_OPTION_TARGET_ENCODING, 'UTF-8'); # https://minutillo.com/steve/weblog/2004/6/17/php-xml-and-character-encodings-a-tale-of-sadness-rage-and-data-loss
+		xml_parser_set_option($parser, XML_OPTION_CASE_FOLDING, 0);
+		xml_parser_set_option($parser, XML_OPTION_SKIP_WHITE, 1);
+		xml_parse_into_struct($parser, trim($contents), $xml_values);
+		xml_parser_free($parser);
+
+		if (!$xml_values) {
+			return;
+		}//Hmm...
+
+		//Initializations
+		$xml_array   = [];
+		$parents     = [];
+		$opened_tags = [];
+		$arr         = [];
+
+		$current = &$xml_array; //Refference
+
+		//Go through the tags.
+		$repeated_tag_index = []; //Multiple tags with same name will be turned into an array
+		foreach ($xml_values as $data) {
+			unset($attributes, $value); //Remove existing values, or there will be trouble
+
+			//This command will extract these variables into the foreach scope
+			// tag(string), type(string), level(int), attributes(array).
+			extract($data); //We could use the array by itself, but this cooler.
+
+			$result          = [];
+			$attributes_data = [];
+
+			if (isset($value)) {
+				if ('tag' === $priority) {
+					$result = $value;
+				} else {
+					$result['value'] = $value;
+				} //Put the value in a assoc array if we are in the 'Attribute' mode
+			}
+
+			//Set the attributes too.
+			if (isset($attributes) and $get_attributes) {
+				foreach ($attributes as $attr => $val) {
+					if ('tag' === $priority) {
+						$attributes_data[$attr] = $val;
+					} else {
+						$result['attr'][$attr] = $val;
+					} //Set all the attributes in a array called 'attr'
+				}
+			}
+
+			//See tag status and do the needed.
+			if ('open' === $type) {//The starting of the tag '<tag>'
+				$parent[$level - 1] = &$current;
+				if (!is_array($current) or (!array_key_exists($tag, $current))) { //Insert New tag
+					$current[$tag] = $result;
+					if ($attributes_data) {
+						$current[$tag . '_attr'] = $attributes_data;
+					}
+					$repeated_tag_index[$tag . '_' . $level] = 1;
+
+					$current = &$current[$tag];
+				} else { //There was another element with the same tag name
+					if (isset($current[$tag][0])) {//If there is a 0th element it is already an array
+						$current[$tag][$repeated_tag_index[$tag . '_' . $level]] = $result;
+						$repeated_tag_index[$tag . '_' . $level]++;
+					} else {//This section will make the value an array if multiple tags with the same name appear together
+						$current[$tag]                           = [$current[$tag], $result]; //This will combine the existing item and the new item together to make an array
+						$repeated_tag_index[$tag . '_' . $level] = 2;
+
+						if (isset($current[$tag . '_attr'])) { //The attribute of the last(0th) tag must be moved as well
+							$current[$tag]['0_attr'] = $current[$tag . '_attr'];
+							unset($current[$tag . '_attr']);
+						}
+					}
+					$last_item_index = $repeated_tag_index[$tag . '_' . $level] - 1;
+					$current         = &$current[$tag][$last_item_index];
+				}
+			} elseif ('complete' === $type) { //Tags that ends in 1 line '<tag>'
+				//See if the key is already taken.
+				if (isset($current[$tag])) { //If taken, put all things inside a list(array)
+					if (isset($current[$tag][0]) and is_array($current[$tag])) {//If it is already an array...
+						// ...push the new element into that array.
+						$current[$tag][$repeated_tag_index[$tag . '_' . $level]] = $result;
+
+						if ('tag' === $priority and $get_attributes and $attributes_data) {
+							$current[$tag][$repeated_tag_index[$tag . '_' . $level] . '_attr'] = $attributes_data;
+						}
+						$repeated_tag_index[$tag . '_' . $level]++;
+					} else { //If it is not an array...
+						$current[$tag]                           = [$current[$tag], $result]; //...Make it an array using using the existing value and the new value
+						$repeated_tag_index[$tag . '_' . $level] = 1;
+						if ('tag' === $priority and $get_attributes) {
+							if (isset($current[$tag . '_attr'])) { //The attribute of the last(0th) tag must be moved as well
+								$current[$tag]['0_attr'] = $current[$tag . '_attr'];
+								unset($current[$tag . '_attr']);
+							}
+
+							if ($attributes_data) {
+								$current[$tag][$repeated_tag_index[$tag . '_' . $level] . '_attr'] = $attributes_data;
+							}
+						}
+						$repeated_tag_index[$tag . '_' . $level]++; //0 and 1 index is already taken
+					}
+				} else { //New Key
+					$current[$tag]                           = $result;
+					$repeated_tag_index[$tag . '_' . $level] = 1;
+					if ('tag' === $priority and $attributes_data) {
+						$current[$tag . '_attr'] = $attributes_data;
+					}
+				}
+			} elseif ('close' === $type) { //End of tag '</tag>'
+				$current = &$parent[$level - 1];
+			}
+		}
+
+		return $xml_array;
+	}
 }
 
 if (!function_exists('songlist_toXml')) {
-    /**
-     * @param $array
-     * @param $name
-     * @param $standalone
-     * @param $beginning
-     * @param $nested
-     * @return string
-     */
-    function songlist_toXml($array, $name, $standalone, $beginning, $nested): string
-    {
-        $output = '';
-        if ($beginning) {
-            if ($standalone) {
-                header('content-type:text/xml;charset=' . _CHARSET);
-            }
-            $output .= '<' . '?' . 'xml version="1.0" encoding="' . _CHARSET . '"' . '?' . '>' . "\n";
-            $output .= '<' . $name . '>' . "\n";
-            $nested = 0;
-        }
-
-        if (is_array($array)) {
-            foreach ($array as $key => $value) {
-                ++$nested;
-                if (is_array($value)) {
-                    $output .= str_repeat("\t", (int)$nested) . '<' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
-                    ++$nested;
-                    $output .= songlist_toXml($value, $name, false, false, $nested);
-                    $nested--;
-                    $output .= str_repeat("\t", (int)$nested) . '</' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
-                } elseif ('' != $value) {
-                        ++$nested;
-                        $output .= str_repeat("\t", (int)$nested) . '  <' . (is_string($key) ? $key : $name . '_' . $key) . '>' . trim($value) . '</' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
-                        $nested--;
-                }
-                $nested--;
-            }
-        } elseif ('' != $array) {
-            ++$nested;
-            $output .= str_repeat("\t", (int)$nested) . trim($array) . "\n";
-            $nested--;
-        }
-
-        if ($beginning) {
-            $output .= '</' . $name . '>';
-
-            return $output;
-        }
-
-        return $output;
-    }
+	/**
+	 * @param $array
+	 * @param $name
+	 * @param $standalone
+	 * @param $beginning
+	 * @param $nested
+	 * @return string
+	 */
+	function songlist_toXml($array, $name, $standalone, $beginning, $nested): string
+	{
+		$output = '';
+		if ($beginning) {
+			if ($standalone) {
+				header('content-type:text/xml;charset=' . _CHARSET);
+			}
+			$output .= '<' . '?' . 'xml version="1.0" encoding="' . _CHARSET . '"' . '?' . '>' . "\n";
+			$output .= '<' . $name . '>' . "\n";
+			$nested = 0;
+		}
+
+		if (is_array($array)) {
+			foreach ($array as $key => $value) {
+				++$nested;
+				if (is_array($value)) {
+					$output .= str_repeat("\t", (int)$nested) . '<' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
+					++$nested;
+					$output .= songlist_toXml($value, $name, false, false, $nested);
+					$nested--;
+					$output .= str_repeat("\t", (int)$nested) . '</' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
+				} elseif ('' != $value) {
+						++$nested;
+						$output .= str_repeat("\t", (int)$nested) . '  <' . (is_string($key) ? $key : $name . '_' . $key) . '>' . trim($value) . '</' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
+						$nested--;
+				}
+				$nested--;
+			}
+		} elseif ('' != $array) {
+			++$nested;
+			$output .= str_repeat("\t", (int)$nested) . trim($array) . "\n";
+			$nested--;
+		}
+
+		if ($beginning) {
+			$output .= '</' . $name . '>';
+
+			return $output;
+		}
+
+		return $output;
+	}
 }

Spacing +65 added lines, -65 removed lines

@@ -114,11 +114,11 @@ 
      */
     function is_ipv6($ip = ''): bool
     {
-        if ('' == $ip) {
+        if (''==$ip) {
             return false;
         }
 
-        if (mb_substr_count($ip, ':') > 0) {
+        if (mb_substr_count($ip, ':')>0) {
             return true;
         }
 
@@ -139,11 +139,11 @@ 
     {
         $components = songlist_getFilterURLComponents($filter, $field, $sort);
         $ele        = false;
-        require_once __DIR__ . '/songlist.object.php';
+        require_once __DIR__.'/songlist.object.php';
         switch ($field) {
             case 'gid':
-                if ('genre' !== $op) {
-                    $ele = new SelectGenreForm('', 'filter_' . $field . '', $components['value'], 1, false);
+                if ('genre'!==$op) {
+                    $ele = new SelectGenreForm('', 'filter_'.$field.'', $components['value'], 1, false);
                     $ele->setExtra(
                         'onchange="window.open(\''
                         . $_SERVER['SCRIPT_NAME']
@@ -162,8 +162,8 @@ 
                 }
                 break;
             case 'vcid':
-                if ('voice' !== $op) {
-                    $ele = new SelectVoiceForm('', 'filter_' . $field . '', $components['value'], 1, false);
+                if ('voice'!==$op) {
+                    $ele = new SelectVoiceForm('', 'filter_'.$field.'', $components['value'], 1, false);
                     $ele->setExtra(
                         'onchange="window.open(\''
                         . $_SERVER['SCRIPT_NAME']
@@ -182,8 +182,8 @@ 
                 }
                 break;
             case 'cid':
-                if ('category' !== $op) {
-                    $ele = new SelectCategoryForm('', 'filter_' . $field . '', $components['value'], 1, false);
+                if ('category'!==$op) {
+                    $ele = new SelectCategoryForm('', 'filter_'.$field.'', $components['value'], 1, false);
                     $ele->setExtra(
                         'onchange="window.open(\''
                         . $_SERVER['SCRIPT_NAME']
@@ -202,7 +202,7 @@ 
                 }
                 break;
             case 'pid':
-                $ele = new SelectCategoryForm('', 'filter_' . $field . '', $components['value'], 1, false);
+                $ele = new SelectCategoryForm('', 'filter_'.$field.'', $components['value'], 1, false);
                 $ele->setExtra(
                     'onchange="window.open(\''
                     . $_SERVER['SCRIPT_NAME']
@@ -220,8 +220,8 @@ 
                 );
                 break;
             case 'abid':
-                if ('albums' !== $op) {
-                    $ele = new SelectAlbumForm('', 'filter_' . $field . '', $components['value'], 1, false);
+                if ('albums'!==$op) {
+                    $ele = new SelectAlbumForm('', 'filter_'.$field.'', $components['value'], 1, false);
                     $ele->setExtra(
                         'onchange="window.open(\''
                         . $_SERVER['SCRIPT_NAME']
@@ -240,8 +240,8 @@ 
                 }
                 break;
             case 'aid':
-                if ('artists' !== $op) {
-                    $ele = new SelectArtistForm('', 'filter_' . $field . '', $components['value'], 1, false);
+                if ('artists'!==$op) {
+                    $ele = new SelectArtistForm('', 'filter_'.$field.'', $components['value'], 1, false);
                     $ele->setExtra(
                         'onchange="window.open(\''
                         . $_SERVER['SCRIPT_NAME']
@@ -260,8 +260,8 @@ 
                 }
                 break;
             case 'sid':
-                if ('songs' !== $op) {
-                    $ele = new SelectSongForm('', 'filter_' . $field . '', $components['value'], 1, false);
+                if ('songs'!==$op) {
+                    $ele = new SelectSongForm('', 'filter_'.$field.'', $components['value'], 1, false);
                     $ele->setExtra(
                         'onchange="window.open(\''
                         . $_SERVER['SCRIPT_NAME']
@@ -292,8 +292,8 @@ 
             case 'songid':
             case 'tags':
                 $ele = new \XoopsFormElementTray('');
-                $ele->addElement(new \XoopsFormText('', 'filter_' . $field . '', 11, 40, $components['value']));
-                $button = new \XoopsFormButton('', 'button_' . $field . '', '[+]');
+                $ele->addElement(new \XoopsFormText('', 'filter_'.$field.'', 11, 40, $components['value']));
+                $button = new \XoopsFormButton('', 'button_'.$field.'', '[+]');
                 $button->setExtra(
                     'onclick="window.open(\''
                     . $_SERVER['SCRIPT_NAME']
@@ -335,13 +335,13 @@ 
         $operator  = '';
         foreach ($parts as $part) {
             $var = explode(',', $part);
-            if (count($var) > 1) {
-                if ($var[0] == $field) {
+            if (count($var)>1) {
+                if ($var[0]==$field) {
                     $ele_value = $var[1];
                     if (isset($var[2])) {
                         $operator = $var[2];
                     }
-                } elseif (1 != $var[0]) {
+                } elseif (1!=$var[0]) {
                     $ret[] = implode(',', $var);
                 }
             }
@@ -352,7 +352,7 @@ 
         $pagenav['limit'] = Request::getInt('limit', 30, 'REQUEST');
         $pagenav['start'] = 0;
         $pagenav['order'] = !empty($_REQUEST['order']) ? $_REQUEST['order'] : 'DESC';
-        $pagenav['sort']  = !empty($_REQUEST['sort']) ? '' . $_REQUEST['sort'] . '' : $sort;
+        $pagenav['sort']  = !empty($_REQUEST['sort']) ? ''.$_REQUEST['sort'].'' : $sort;
         $retb             = [];
         foreach ($pagenav as $key => $value) {
             $retb[] = "$key=$value";
@@ -370,7 +370,7 @@ 
     function songlist_obj2array($objects): array
     {
         $ret = [];
-        foreach ((array)$objects as $key => $value) {
+        foreach ((array) $objects as $key => $value) {
             if (is_a($value, 'stdClass')) {
                 $ret[$key] = songlist_obj2array($value);
             } elseif (is_array($value)) {
@@ -399,8 +399,8 @@ 
         $GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
 
         if (!empty($GLOBALS['songlistModuleConfig']['bitly_username']) && !empty($GLOBALS['songlistModuleConfig']['bitly_apikey'])) {
-            $source_url = $GLOBALS['songlistModuleConfig']['bitly_apiurl'] . '/shorten?login=' . $GLOBALS['songlistModuleConfig']['bitly_username'] . '&apiKey=' . $GLOBALS['songlistModuleConfig']['bitly_apikey'] . '&format=json&longUrl=' . urlencode($url);
-            $cookies    = XOOPS_ROOT_PATH . '/uploads/songlist_' . md5($GLOBALS['songlistModuleConfig']['bitly_apikey']) . '.cookie';
+            $source_url = $GLOBALS['songlistModuleConfig']['bitly_apiurl'].'/shorten?login='.$GLOBALS['songlistModuleConfig']['bitly_username'].'&apiKey='.$GLOBALS['songlistModuleConfig']['bitly_apikey'].'&format=json&longUrl='.urlencode($url);
+            $cookies    = XOOPS_ROOT_PATH.'/uploads/songlist_'.md5($GLOBALS['songlistModuleConfig']['bitly_apikey']).'.cookie';
             if (!$ch = curl_init($source_url)) {
                 return $url;
             }
@@ -478,7 +478,7 @@ 
             $attributes_data = [];
 
             if (isset($value)) {
-                if ('tag' === $priority) {
+                if ('tag'===$priority) {
                     $result = $value;
                 } else {
                     $result['value'] = $value;
@@ -488,7 +488,7 @@ 
             //Set the attributes too.
             if (isset($attributes) and $get_attributes) {
                 foreach ($attributes as $attr => $val) {
-                    if ('tag' === $priority) {
+                    if ('tag'===$priority) {
                         $attributes_data[$attr] = $val;
                     } else {
                         $result['attr'][$attr] = $val;
@@ -497,67 +497,67 @@ 
             }
 
             //See tag status and do the needed.
-            if ('open' === $type) {//The starting of the tag '<tag>'
-                $parent[$level - 1] = &$current;
+            if ('open'===$type) {//The starting of the tag '<tag>'
+                $parent[$level-1] = &$current;
                 if (!is_array($current) or (!array_key_exists($tag, $current))) { //Insert New tag
                     $current[$tag] = $result;
                     if ($attributes_data) {
-                        $current[$tag . '_attr'] = $attributes_data;
+                        $current[$tag.'_attr'] = $attributes_data;
                     }
-                    $repeated_tag_index[$tag . '_' . $level] = 1;
+                    $repeated_tag_index[$tag.'_'.$level] = 1;
 
                     $current = &$current[$tag];
                 } else { //There was another element with the same tag name
                     if (isset($current[$tag][0])) {//If there is a 0th element it is already an array
-                        $current[$tag][$repeated_tag_index[$tag . '_' . $level]] = $result;
-                        $repeated_tag_index[$tag . '_' . $level]++;
+                        $current[$tag][$repeated_tag_index[$tag.'_'.$level]] = $result;
+                        $repeated_tag_index[$tag.'_'.$level]++;
                     } else {//This section will make the value an array if multiple tags with the same name appear together
                         $current[$tag]                           = [$current[$tag], $result]; //This will combine the existing item and the new item together to make an array
-                        $repeated_tag_index[$tag . '_' . $level] = 2;
+                        $repeated_tag_index[$tag.'_'.$level] = 2;
 
-                        if (isset($current[$tag . '_attr'])) { //The attribute of the last(0th) tag must be moved as well
-                            $current[$tag]['0_attr'] = $current[$tag . '_attr'];
-                            unset($current[$tag . '_attr']);
+                        if (isset($current[$tag.'_attr'])) { //The attribute of the last(0th) tag must be moved as well
+                            $current[$tag]['0_attr'] = $current[$tag.'_attr'];
+                            unset($current[$tag.'_attr']);
                         }
                     }
-                    $last_item_index = $repeated_tag_index[$tag . '_' . $level] - 1;
+                    $last_item_index = $repeated_tag_index[$tag.'_'.$level]-1;
                     $current         = &$current[$tag][$last_item_index];
                 }
-            } elseif ('complete' === $type) { //Tags that ends in 1 line '<tag>'
+            } elseif ('complete'===$type) { //Tags that ends in 1 line '<tag>'
                 //See if the key is already taken.
                 if (isset($current[$tag])) { //If taken, put all things inside a list(array)
                     if (isset($current[$tag][0]) and is_array($current[$tag])) {//If it is already an array...
                         // ...push the new element into that array.
-                        $current[$tag][$repeated_tag_index[$tag . '_' . $level]] = $result;
+                        $current[$tag][$repeated_tag_index[$tag.'_'.$level]] = $result;
 
-                        if ('tag' === $priority and $get_attributes and $attributes_data) {
-                            $current[$tag][$repeated_tag_index[$tag . '_' . $level] . '_attr'] = $attributes_data;
+                        if ('tag'===$priority and $get_attributes and $attributes_data) {
+                            $current[$tag][$repeated_tag_index[$tag.'_'.$level].'_attr'] = $attributes_data;
                         }
-                        $repeated_tag_index[$tag . '_' . $level]++;
+                        $repeated_tag_index[$tag.'_'.$level]++;
                     } else { //If it is not an array...
                         $current[$tag]                           = [$current[$tag], $result]; //...Make it an array using using the existing value and the new value
-                        $repeated_tag_index[$tag . '_' . $level] = 1;
-                        if ('tag' === $priority and $get_attributes) {
-                            if (isset($current[$tag . '_attr'])) { //The attribute of the last(0th) tag must be moved as well
-                                $current[$tag]['0_attr'] = $current[$tag . '_attr'];
-                                unset($current[$tag . '_attr']);
+                        $repeated_tag_index[$tag.'_'.$level] = 1;
+                        if ('tag'===$priority and $get_attributes) {
+                            if (isset($current[$tag.'_attr'])) { //The attribute of the last(0th) tag must be moved as well
+                                $current[$tag]['0_attr'] = $current[$tag.'_attr'];
+                                unset($current[$tag.'_attr']);
                             }
 
                             if ($attributes_data) {
-                                $current[$tag][$repeated_tag_index[$tag . '_' . $level] . '_attr'] = $attributes_data;
+                                $current[$tag][$repeated_tag_index[$tag.'_'.$level].'_attr'] = $attributes_data;
                             }
                         }
-                        $repeated_tag_index[$tag . '_' . $level]++; //0 and 1 index is already taken
+                        $repeated_tag_index[$tag.'_'.$level]++; //0 and 1 index is already taken
                     }
                 } else { //New Key
                     $current[$tag]                           = $result;
-                    $repeated_tag_index[$tag . '_' . $level] = 1;
-                    if ('tag' === $priority and $attributes_data) {
-                        $current[$tag . '_attr'] = $attributes_data;
+                    $repeated_tag_index[$tag.'_'.$level] = 1;
+                    if ('tag'===$priority and $attributes_data) {
+                        $current[$tag.'_attr'] = $attributes_data;
                     }
                 }
-            } elseif ('close' === $type) { //End of tag '</tag>'
-                $current = &$parent[$level - 1];
+            } elseif ('close'===$type) { //End of tag '</tag>'
+                $current = &$parent[$level-1];
             }
         }
 
@@ -579,10 +579,10 @@ 
         $output = '';
         if ($beginning) {
             if ($standalone) {
-                header('content-type:text/xml;charset=' . _CHARSET);
+                header('content-type:text/xml;charset='._CHARSET);
             }
-            $output .= '<' . '?' . 'xml version="1.0" encoding="' . _CHARSET . '"' . '?' . '>' . "\n";
-            $output .= '<' . $name . '>' . "\n";
+            $output .= '<'.'?'.'xml version="1.0" encoding="'._CHARSET.'"'.'?'.'>'."\n";
+            $output .= '<'.$name.'>'."\n";
             $nested = 0;
         }
 
@@ -590,26 +590,26 @@ 
             foreach ($array as $key => $value) {
                 ++$nested;
                 if (is_array($value)) {
-                    $output .= str_repeat("\t", (int)$nested) . '<' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
+                    $output .= str_repeat("\t", (int) $nested).'<'.(is_string($key) ? $key : $name.'_'.$key).'>'."\n";
                     ++$nested;
                     $output .= songlist_toXml($value, $name, false, false, $nested);
                     $nested--;
-                    $output .= str_repeat("\t", (int)$nested) . '</' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
-                } elseif ('' != $value) {
+                    $output .= str_repeat("\t", (int) $nested).'</'.(is_string($key) ? $key : $name.'_'.$key).'>'."\n";
+                } elseif (''!=$value) {
                         ++$nested;
-                        $output .= str_repeat("\t", (int)$nested) . '  <' . (is_string($key) ? $key : $name . '_' . $key) . '>' . trim($value) . '</' . (is_string($key) ? $key : $name . '_' . $key) . '>' . "\n";
+                        $output .= str_repeat("\t", (int) $nested).'  <'.(is_string($key) ? $key : $name.'_'.$key).'>'.trim($value).'</'.(is_string($key) ? $key : $name.'_'.$key).'>'."\n";
                         $nested--;
                 }
                 $nested--;
             }
-        } elseif ('' != $array) {
+        } elseif (''!=$array) {
             ++$nested;
-            $output .= str_repeat("\t", (int)$nested) . trim($array) . "\n";
+            $output .= str_repeat("\t", (int) $nested).trim($array)."\n";
             $nested--;
         }
 
         if ($beginning) {
-            $output .= '</' . $name . '>';
+            $output .= '</'.$name.'>';
 
             return $output;
         }