XoopsModules25x / songlist

config/icons.php

Indentation +10 added lines, -10 removed lines

@@ -6,14 +6,14 @@
 $moduleDirName = \basename(\dirname(__DIR__));
 
 return [
-    'name'    => \mb_strtoupper($moduleDirName) . ' IconConfigurator',
-    'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . ' title=' . _EDIT . " align='middle'>",
-    'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt=" . _DELETE . ' title=' . _DELETE . " align='middle'>",
-    'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt=" . _CLONE . ' title=' . _CLONE . " align='middle'>",
-    'preview' => "<img src='" . $pathIcon16 . "/view.png' alt=" . _PREVIEW . ' title=' . _PREVIEW . " align='middle'>",
-    'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt=" . _PRINT . ' title=' . _PRINT . " align='middle'>",
-    'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt=" . _PDF . ' title=' . _PDF . " align='middle'>",
-    'add'     => "<img src='" . $pathIcon16 . "/add.png' alt=" . _ADD . ' title=' . _ADD . " align='middle'>",
-    '0'       => "<img src='" . $pathIcon16 . "/0.png' alt=" . 0 . ' title=' . _OFF . " align='middle'>",
-    '1'       => "<img src='" . $pathIcon16 . "/1.png' alt=" . 1 . ' title=' . _ON . " align='middle'>",
+	'name'    => \mb_strtoupper($moduleDirName) . ' IconConfigurator',
+	'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . ' title=' . _EDIT . " align='middle'>",
+	'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt=" . _DELETE . ' title=' . _DELETE . " align='middle'>",
+	'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt=" . _CLONE . ' title=' . _CLONE . " align='middle'>",
+	'preview' => "<img src='" . $pathIcon16 . "/view.png' alt=" . _PREVIEW . ' title=' . _PREVIEW . " align='middle'>",
+	'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt=" . _PRINT . ' title=' . _PRINT . " align='middle'>",
+	'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt=" . _PDF . ' title=' . _PDF . " align='middle'>",
+	'add'     => "<img src='" . $pathIcon16 . "/add.png' alt=" . _ADD . ' title=' . _ADD . " align='middle'>",
+	'0'       => "<img src='" . $pathIcon16 . "/0.png' alt=" . 0 . ' title=' . _OFF . " align='middle'>",
+	'1'       => "<img src='" . $pathIcon16 . "/1.png' alt=" . 1 . ' title=' . _ON . " align='middle'>",
 ];

Spacing +10 added lines, -10 removed lines

@@ -6,14 +6,14 @@
 $moduleDirName = \basename(\dirname(__DIR__));
 
 return [
-    'name'    => \mb_strtoupper($moduleDirName) . ' IconConfigurator',
-    'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . ' title=' . _EDIT . " align='middle'>",
-    'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt=" . _DELETE . ' title=' . _DELETE . " align='middle'>",
-    'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt=" . _CLONE . ' title=' . _CLONE . " align='middle'>",
-    'preview' => "<img src='" . $pathIcon16 . "/view.png' alt=" . _PREVIEW . ' title=' . _PREVIEW . " align='middle'>",
-    'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt=" . _PRINT . ' title=' . _PRINT . " align='middle'>",
-    'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt=" . _PDF . ' title=' . _PDF . " align='middle'>",
-    'add'     => "<img src='" . $pathIcon16 . "/add.png' alt=" . _ADD . ' title=' . _ADD . " align='middle'>",
-    '0'       => "<img src='" . $pathIcon16 . "/0.png' alt=" . 0 . ' title=' . _OFF . " align='middle'>",
-    '1'       => "<img src='" . $pathIcon16 . "/1.png' alt=" . 1 . ' title=' . _ON . " align='middle'>",
+    'name'    => \mb_strtoupper($moduleDirName).' IconConfigurator',
+    'edit'    => "<img src='".$pathIcon16."/edit.png'  alt="._EDIT.' title='._EDIT." align='middle'>",
+    'delete'  => "<img src='".$pathIcon16."/delete.png' alt="._DELETE.' title='._DELETE." align='middle'>",
+    'clone'   => "<img src='".$pathIcon16."/editcopy.png' alt="._CLONE.' title='._CLONE." align='middle'>",
+    'preview' => "<img src='".$pathIcon16."/view.png' alt="._PREVIEW.' title='._PREVIEW." align='middle'>",
+    'print'   => "<img src='".$pathIcon16."/printer.png' alt="._PRINT.' title='._PRINT." align='middle'>",
+    'pdf'     => "<img src='".$pathIcon16."/pdf.png' alt="._PDF.' title='._PDF." align='middle'>",
+    'add'     => "<img src='".$pathIcon16."/add.png' alt="._ADD.' title='._ADD." align='middle'>",
+    '0'       => "<img src='".$pathIcon16."/0.png' alt=".0.' title='._OFF." align='middle'>",
+    '1'       => "<img src='".$pathIcon16."/1.png' alt=".1.' title='._ON." align='middle'>",
 ];

config/config.php

Indentation +61 added lines, -61 removed lines

@@ -21,69 +21,69 @@
 $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
 
 return (object)[
-    'name'           => $moduleDirNameUpper . ' Module Configurator',
-    'paths'          => [
-        'dirname'    => $moduleDirName,
-        'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
-        'modPath'    => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
-        'modUrl'     => XOOPS_URL . '/modules/' . $moduleDirName,
-        'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
-    ],
-    'uploadFolders'  => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/albums',
-    ],
-    'copyBlankFiles' => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/albums',
-    ],
+	'name'           => $moduleDirNameUpper . ' Module Configurator',
+	'paths'          => [
+		'dirname'    => $moduleDirName,
+		'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
+		'modPath'    => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
+		'modUrl'     => XOOPS_URL . '/modules/' . $moduleDirName,
+		'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
+	],
+	'uploadFolders'  => [
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/albums',
+	],
+	'copyBlankFiles' => [
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/albums',
+	],
 
-    'copyTestFolders' => [
-        [
-            XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/uploads',
-            XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        ],
-        //            [
-        //                XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/thumbs',
-        //                XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/thumbs',
-        //            ],
-    ],
+	'copyTestFolders' => [
+		[
+			XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/uploads',
+			XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		],
+		//            [
+		//                XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/thumbs',
+		//                XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/thumbs',
+		//            ],
+	],
 
-    'templateFolders' => [
-        '/templates/',
-        //            '/templates/blocks/',
-        //            '/templates/admin/'
-    ],
-    'oldFiles'        => [
-        '/class/request.php',
-        '/class/registry.php',
-        '/class/utilities.php',
-        '/class/util.php',
-        //            '/include/constants.php',
-        //            '/include/functions.php',
-        '/ajaxrating.txt',
-    ],
-    'oldFolders'      => [
-        '/images',
-        '/css',
-        '/js',
-        '/tcpdf',
-    ],
+	'templateFolders' => [
+		'/templates/',
+		//            '/templates/blocks/',
+		//            '/templates/admin/'
+	],
+	'oldFiles'        => [
+		'/class/request.php',
+		'/class/registry.php',
+		'/class/utilities.php',
+		'/class/util.php',
+		//            '/include/constants.php',
+		//            '/include/functions.php',
+		'/ajaxrating.txt',
+	],
+	'oldFolders'      => [
+		'/images',
+		'/css',
+		'/js',
+		'/tcpdf',
+	],
 
-    'renameTables'  => [//         'XX_archive'     => 'ZZZZ_archive',
-    ],
-    'renameColumns' => [//        'extcal_event' => ['from' => 'event_etablissement', 'to' => 'event_location'],
-    ],
-    'moduleStats'   => [
-        //            'totalcategories' => $helper->getHandler('Category')->getCategoriesCount(-1),
-        //            'totalitems'      => $helper->getHandler('Item')->getItemsCount(),
-        //            'totalsubmitted'  => $helper->getHandler('Item')->getItemsCount(-1, [Constants::PUBLISHER_STATUS_SUBMITTED]),
-    ],
-    'modCopyright'  => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
+	'renameTables'  => [//         'XX_archive'     => 'ZZZZ_archive',
+	],
+	'renameColumns' => [//        'extcal_event' => ['from' => 'event_etablissement', 'to' => 'event_location'],
+	],
+	'moduleStats'   => [
+		//            'totalcategories' => $helper->getHandler('Category')->getCategoriesCount(-1),
+		//            'totalitems'      => $helper->getHandler('Item')->getItemsCount(),
+		//            'totalsubmitted'  => $helper->getHandler('Item')->getItemsCount(-1, [Constants::PUBLISHER_STATUS_SUBMITTED]),
+	],
+	'modCopyright'  => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
                      <img src='" . Admin::iconUrl('xoopsmicrobutton.gif') . "' alt='XOOPS Project'></a>",
 ];

Spacing +18 added lines, -18 removed lines

@@ -20,33 +20,33 @@ 
 $moduleDirName      = \basename(\dirname(__DIR__));
 $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
 
-return (object)[
-    'name'           => $moduleDirNameUpper . ' Module Configurator',
+return (object) [
+    'name'           => $moduleDirNameUpper.' Module Configurator',
     'paths'          => [
         'dirname'    => $moduleDirName,
-        'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
-        'modPath'    => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
-        'modUrl'     => XOOPS_URL . '/modules/' . $moduleDirName,
-        'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
+        'admin'      => XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/admin',
+        'modPath'    => XOOPS_ROOT_PATH.'/modules/'.$moduleDirName,
+        'modUrl'     => XOOPS_URL.'/modules/'.$moduleDirName,
+        'uploadPath' => XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+        'uploadUrl'  => XOOPS_UPLOAD_URL.'/'.$moduleDirName,
     ],
     'uploadFolders'  => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/albums',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/category',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/screenshots',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/albums',
     ],
     'copyBlankFiles' => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/albums',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/category',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/screenshots',
+        XOOPS_UPLOAD_PATH.'/'.$moduleDirName.'/albums',
     ],
 
     'copyTestFolders' => [
         [
-            XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/uploads',
-            XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+            XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/testdata/uploads',
+            XOOPS_UPLOAD_PATH.'/'.$moduleDirName,
         ],
         //            [
         //                XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/testdata/thumbs',
@@ -85,5 +85,5 @@ 
         //            'totalsubmitted'  => $helper->getHandler('Item')->getItemsCount(-1, [Constants::PUBLISHER_STATUS_SUBMITTED]),
     ],
     'modCopyright'  => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
-                     <img src='" . Admin::iconUrl('xoopsmicrobutton.gif') . "' alt='XOOPS Project'></a>",
+                     <img src='" . Admin::iconUrl('xoopsmicrobutton.gif')."' alt='XOOPS Project'></a>",
 ];

config/imageconfig.php

Indentation +24 added lines, -24 removed lines

@@ -5,37 +5,37 @@
 
 // extra module configs
 $modversion['config'][] = [
-    'name'        => 'imageConfigs',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG_DSC',
-    'formtype'    => 'line_break',
-    'valuetype'   => 'textbox',
-    'default'     => 'head',
+	'name'        => 'imageConfigs',
+	'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG',
+	'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG_DSC',
+	'formtype'    => 'line_break',
+	'valuetype'   => 'textbox',
+	'default'     => 'head',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'imageWidth',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH_DSC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 1200,
+	'name'        => 'imageWidth',
+	'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH',
+	'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH_DSC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 1200,
 ]; // =1024/16
 
 $modversion['config'][] = [
-    'name'        => 'imageHeight',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT_DSC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 800,
+	'name'        => 'imageHeight',
+	'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT',
+	'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT_DSC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 800,
 ]; // =768/16
 
 $modversion['config'][] = [
-    'name'        => 'imageUploadPath',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH_DSC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'text',
-    'default'     => 'uploads/' . $modversion['dirname'] . '/images',
+	'name'        => 'imageUploadPath',
+	'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH',
+	'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH_DSC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'text',
+	'default'     => 'uploads/' . $modversion['dirname'] . '/images',
 ];

Spacing +9 added lines, -9 removed lines

@@ -6,8 +6,8 @@ 
 // extra module configs
 $modversion['config'][] = [
     'name'        => 'imageConfigs',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG_DSC',
+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_CONFIG',
+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_CONFIG_DSC',
     'formtype'    => 'line_break',
     'valuetype'   => 'textbox',
     'default'     => 'head',
@@ -15,8 +15,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'imageWidth',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH_DSC',
+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_WIDTH',
+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_WIDTH_DSC',
     'formtype'    => 'textbox',
     'valuetype'   => 'int',
     'default'     => 1200,
@@ -24,8 +24,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'imageHeight',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT_DSC',
+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_HEIGHT',
+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_HEIGHT_DSC',
     'formtype'    => 'textbox',
     'valuetype'   => 'int',
     'default'     => 800,
@@ -33,9 +33,9 @@ 
 
 $modversion['config'][] = [
     'name'        => 'imageUploadPath',
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH',
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH_DSC',
+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_UPLOAD_PATH',
+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'IMAGE_UPLOAD_PATH_DSC',
     'formtype'    => 'textbox',
     'valuetype'   => 'text',
-    'default'     => 'uploads/' . $modversion['dirname'] . '/images',
+    'default'     => 'uploads/'.$modversion['dirname'].'/images',
 ];

search.php

Indentation +99 added lines, -99 removed lines

@@ -27,111 +27,111 @@
 $utf8mapHandler = Helper::getInstance()->getHandler('Utf8map');
 
 switch ($op) {
-    default:
-    case 'search':
-        $url = $songsHandler->getSearchURL();
-        if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
-            header('HTTP/1.1 301 Moved Permanently');
-            header('Location: ' . $url);
-            exit(0);
-        }
+	default:
+	case 'search':
+		$url = $songsHandler->getSearchURL();
+		if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
+			header('HTTP/1.1 301 Moved Permanently');
+			header('Location: ' . $url);
+			exit(0);
+		}
 
-        switch ($fct) {
-            default:
-            case 'titleandlyrics':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                break;
-            case 'albums':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                $albums   = $albumsHandler->getObjects($criteria, true);
-                $criteria = new \CriteriaCompo();
-                foreach ($albums as $abid => $album) {
-                    $criteria->add(new \Criteria('abid', $abid), 'OR');
-                }
-                break;
-            case 'artists':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('name', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                $artists  = $artistsHandler->getObjects($criteria, true);
-                $criteria = new \CriteriaCompo();
-                if (is_array($artists)) {
-                    foreach ($artists as $aid => $artist) {
-                        $criteria->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
-                    }
-                }
-                break;
-            case 'lyrics':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                break;
-            case 'title':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                break;
-        }
+		switch ($fct) {
+			default:
+			case 'titleandlyrics':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+					$criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				break;
+			case 'albums':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				$albums   = $albumsHandler->getObjects($criteria, true);
+				$criteria = new \CriteriaCompo();
+				foreach ($albums as $abid => $album) {
+					$criteria->add(new \Criteria('abid', $abid), 'OR');
+				}
+				break;
+			case 'artists':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('name', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				$artists  = $artistsHandler->getObjects($criteria, true);
+				$criteria = new \CriteriaCompo();
+				if (is_array($artists)) {
+					foreach ($artists as $aid => $artist) {
+						$criteria->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
+					}
+				}
+				break;
+			case 'lyrics':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				break;
+			case 'title':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				break;
+		}
 
-        if (0 != $gid && $GLOBALS['songlistModuleConfig']['genre']) {
-            $criteria->add(new \Criteria('gids', '%"' . $gid . '"%', 'LIKE'));
-        }
+		if (0 != $gid && $GLOBALS['songlistModuleConfig']['genre']) {
+			$criteria->add(new \Criteria('gids', '%"' . $gid . '"%', 'LIKE'));
+		}
 
-        if (0 != $vcid && $GLOBALS['songlistModuleConfig']['voice']) {
-            $criteria->add(new \Criteria('vcid', $vcid));
-        }
+		if (0 != $vcid && $GLOBALS['songlistModuleConfig']['voice']) {
+			$criteria->add(new \Criteria('vcid', $vcid));
+		}
 
-        if (0 != ($_GET['cid'] ?? $cid)) {
-            $criteria->add(new \Criteria('cid', ($_GET['cid'] ?? $cid)));
-        }
+		if (0 != ($_GET['cid'] ?? $cid)) {
+			$criteria->add(new \Criteria('cid', ($_GET['cid'] ?? $cid)));
+		}
 
-        $pagenav = new \XoopsPageNav($songsHandler->getCount($criteria), $limit, $start, 'start', "?op=$op&fct=$fct&gid=$gid&vcid=$vcid&value=$value&limit=$limit");
+		$pagenav = new \XoopsPageNav($songsHandler->getCount($criteria), $limit, $start, 'start', "?op=$op&fct=$fct&gid=$gid&vcid=$vcid&value=$value&limit=$limit");
 
-        $criteria->setLimit($limit);
-        $criteria->setStart($start);
+		$criteria->setLimit($limit);
+		$criteria->setStart($start);
 
-        $songs = $songsHandler->getObjects($criteria, false);
+		$songs = $songsHandler->getObjects($criteria, false);
 
-        $GLOBALS['xoopsOption']['template_main'] = 'songlist_search_index.tpl';
-        require_once $GLOBALS['xoops']->path('/header.php');
-        if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-            $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
-            $GLOBALS['loaded_jquery'] = true;
-        }
-        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
-        $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-        $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-        foreach ($songs as $song) {
-            $GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
-        }
-        $GLOBALS['xoopsTpl']->assign('songs', true);
-        $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
-        $GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
-        $GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
-        $GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
-        $GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
-        $GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
-        require_once $GLOBALS['xoops']->path('/footer.php');
-        break;
-    case 'category':
-        switch ($fct) {
-            default:
-            case 'set':
-                $_SESSION['cid'] = $id;
-                break;
-            case 'home':
-                unset($_SESSION['cid']);
-                break;
-        }
-        redirect_header($_SERVER['SCRIPT_NAME'] . "?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit", 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
+		$GLOBALS['xoopsOption']['template_main'] = 'songlist_search_index.tpl';
+		require_once $GLOBALS['xoops']->path('/header.php');
+		if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
+			$GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+			$GLOBALS['loaded_jquery'] = true;
+		}
+		$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+		$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+		$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+		foreach ($songs as $song) {
+			$GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
+		}
+		$GLOBALS['xoopsTpl']->assign('songs', true);
+		$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
+		$GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
+		$GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
+		$GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
+		$GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
+		$GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
+		require_once $GLOBALS['xoops']->path('/footer.php');
+		break;
+	case 'category':
+		switch ($fct) {
+			default:
+			case 'set':
+				$_SESSION['cid'] = $id;
+				break;
+			case 'home':
+				unset($_SESSION['cid']);
+				break;
+		}
+		redirect_header($_SERVER['SCRIPT_NAME'] . "?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit", 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
 }

Spacing +16 added lines, -16 removed lines

@@ -9,7 +9,7 @@ 
 use XoopsModules\Songlist\SongsHandler;
 use XoopsModules\Songlist\Utf8mapHandler;
 
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 global $file, $op, $fct, $id, $value, $gid, $cid, $start, $limit, $singer;
 
@@ -32,7 +32,7 @@ 
         $url = $songsHandler->getSearchURL();
         if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
             header('HTTP/1.1 301 Moved Permanently');
-            header('Location: ' . $url);
+            header('Location: '.$url);
             exit(0);
         }
 
@@ -41,14 +41,14 @@ 
             case 'titleandlyrics':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('title', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
+                    $criteria->add(new \Criteria('lyrics', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 break;
             case 'albums':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('title', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 $albums   = $albumsHandler->getObjects($criteria, true);
                 $criteria = new \CriteriaCompo();
@@ -59,39 +59,39 @@ 
             case 'artists':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('name', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('name', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 $artists  = $artistsHandler->getObjects($criteria, true);
                 $criteria = new \CriteriaCompo();
                 if (is_array($artists)) {
                     foreach ($artists as $aid => $artist) {
-                        $criteria->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
+                        $criteria->add(new \Criteria('aids', '%"'.$aid.'"%', 'LIKE'), 'OR');
                     }
                 }
                 break;
             case 'lyrics':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('lyrics', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 break;
             case 'title':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('title', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 break;
         }
 
-        if (0 != $gid && $GLOBALS['songlistModuleConfig']['genre']) {
-            $criteria->add(new \Criteria('gids', '%"' . $gid . '"%', 'LIKE'));
+        if (0!=$gid && $GLOBALS['songlistModuleConfig']['genre']) {
+            $criteria->add(new \Criteria('gids', '%"'.$gid.'"%', 'LIKE'));
         }
 
-        if (0 != $vcid && $GLOBALS['songlistModuleConfig']['voice']) {
+        if (0!=$vcid && $GLOBALS['songlistModuleConfig']['voice']) {
             $criteria->add(new \Criteria('vcid', $vcid));
         }
 
-        if (0 != ($_GET['cid'] ?? $cid)) {
+        if (0!=($_GET['cid'] ?? $cid)) {
             $criteria->add(new \Criteria('cid', ($_GET['cid'] ?? $cid)));
         }
 
@@ -105,10 +105,10 @@ 
         $GLOBALS['xoopsOption']['template_main'] = 'songlist_search_index.tpl';
         require_once $GLOBALS['xoops']->path('/header.php');
         if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-            $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+            $GLOBALS['xoTheme']->addScript(XOOPS_URL._MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
             $GLOBALS['loaded_jquery'] = true;
         }
-        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL._MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
         $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
         $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
         foreach ($songs as $song) {
@@ -133,5 +133,5 @@ 
                 unset($_SESSION['cid']);
                 break;
         }
-        redirect_header($_SERVER['SCRIPT_NAME'] . "?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit", 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
+        redirect_header($_SERVER['SCRIPT_NAME']."?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit", 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
 }

header.php

Indentation +3 added lines, -3 removed lines

@@ -20,10 +20,10 @@ 
 require_once \dirname(__DIR__, 2) . '/mainfile.php';
 
 if (!defined('_CHARSET')) {
-    define('_CHARSET', 'UTF-8');
+	define('_CHARSET', 'UTF-8');
 }
 if (!defined('_CHARSET_ISO')) {
-    define('_CHARSET_ISO', 'ISO-8859-1');
+	define('_CHARSET_ISO', 'ISO-8859-1');
 }
 
 $GLOBALS['myts'] = \MyTextSanitizer::getInstance();
@@ -64,5 +64,5 @@ 
 $GLOBALS['limit'] = $_REQUEST['limit'] ?? $GLOBALS['songlistModuleConfig']['cols'] * $GLOBALS['songlistModuleConfig']['rows'];
 
 if (!isset($_SESSION['cid'])) {
-    $_SESSION['cid'] = $GLOBALS['cid'];
+	$_SESSION['cid'] = $GLOBALS['cid'];
 }

Spacing +5 added lines, -5 removed lines

@@ -17,7 +17,7 @@ 
 
 use Xmf\Request;
 
-require_once \dirname(__DIR__, 2) . '/mainfile.php';
+require_once \dirname(__DIR__, 2).'/mainfile.php';
 
 if (!defined('_CHARSET')) {
     define('_CHARSET', 'UTF-8');
@@ -35,18 +35,18 @@ 
 $GLOBALS['songlistModule']       = $moduleHandler->getByDirname('songlist');
 $GLOBALS['songlistModuleConfig'] = $configHandler->getConfigList($GLOBALS['songlistModule']->getVar('mid'));
 
-ini_set('memory_limit', $GLOBALS['songlistModuleConfig']['memory_user'] . 'M');
+ini_set('memory_limit', $GLOBALS['songlistModuleConfig']['memory_user'].'M');
 set_time_limit($GLOBALS['songlistModuleConfig']['time_user']);
 
 xoops_load('pagenav');
 xoops_load('xoopslists');
 xoops_load('xoopsformloader');
-require_once $GLOBALS['xoops']->path('class' . DS . 'xoopsmailer.php');
+require_once $GLOBALS['xoops']->path('class'.DS.'xoopsmailer.php');
 
 xoops_loadLanguage('user');
 
 //require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['songlistModule']->getVar('dirname') . '/include/functions.php';
-require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['songlistModule']->getVar('dirname') . '/include/songlist.object.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['songlistModule']->getVar('dirname').'/include/songlist.object.php';
 //require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['songlistModule']->getVar('dirname') . '/include/songlist.form.php';
 
 xoops_loadLanguage('main', 'songlist');
@@ -61,7 +61,7 @@ 
 $GLOBALS['vcid']  = Request::getInt('vcid', 0, 'REQUEST');
 $GLOBALS['cid']   = ((($_POST['cid'] ?? isset($_GET['cid'])) ? $_GET['cid'] : isset($_SESSION['cid'])) ? $_SESSION['cid'] : 0);
 $GLOBALS['start'] = Request::getInt('start', 0, 'REQUEST');
-$GLOBALS['limit'] = $_REQUEST['limit'] ?? $GLOBALS['songlistModuleConfig']['cols'] * $GLOBALS['songlistModuleConfig']['rows'];
+$GLOBALS['limit'] = $_REQUEST['limit'] ?? $GLOBALS['songlistModuleConfig']['cols']*$GLOBALS['songlistModuleConfig']['rows'];
 
 if (!isset($_SESSION['cid'])) {
     $_SESSION['cid'] = $GLOBALS['cid'];

testdata/index.php

Indentation +152 added lines, -152 removed lines

@@ -37,122 +37,122 @@ 
 $helper->loadLanguage('common');
 
 switch ($op) {
-    case 'load':
-        if (Request::hasVar('ok', 'REQUEST') && 1 === Request::getInt('ok', 0)) {
-            if (!$GLOBALS['xoopsSecurity']->check()) {
-                redirect_header($helper->url('admin/index.php'), 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-            }
-            loadSampleData();
-        } else {
-            xoops_cp_header();
-            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', constant('CO_' . $moduleDirNameUpper . '_' . 'LOAD_SAMPLEDATA_CONFIRM'), constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
-            xoops_cp_footer();
-        }
-        break;
-    case 'save':
-        saveSampleData();
-        break;
-    case 'clear':
-        if (Request::hasVar('ok', 'REQUEST') && 1 === Request::getInt('ok', 0)) {
-            if (!$GLOBALS['xoopsSecurity']->check()) {
-                redirect_header($helper->url('admin/index.php'), 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-            }
-            clearSampleData();
-        } else {
-            xoops_cp_header();
-            xoops_confirm(['ok' => 1, 'op' => 'clear'], 'index.php', sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLEAR_SAMPLEDATA')), constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
-            xoops_cp_footer();
-        }
-        break;
+	case 'load':
+		if (Request::hasVar('ok', 'REQUEST') && 1 === Request::getInt('ok', 0)) {
+			if (!$GLOBALS['xoopsSecurity']->check()) {
+				redirect_header($helper->url('admin/index.php'), 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+			}
+			loadSampleData();
+		} else {
+			xoops_cp_header();
+			xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', constant('CO_' . $moduleDirNameUpper . '_' . 'LOAD_SAMPLEDATA_CONFIRM'), constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
+			xoops_cp_footer();
+		}
+		break;
+	case 'save':
+		saveSampleData();
+		break;
+	case 'clear':
+		if (Request::hasVar('ok', 'REQUEST') && 1 === Request::getInt('ok', 0)) {
+			if (!$GLOBALS['xoopsSecurity']->check()) {
+				redirect_header($helper->url('admin/index.php'), 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+			}
+			clearSampleData();
+		} else {
+			xoops_cp_header();
+			xoops_confirm(['ok' => 1, 'op' => 'clear'], 'index.php', sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLEAR_SAMPLEDATA')), constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
+			xoops_cp_footer();
+		}
+		break;
 }
 
 // XMF TableLoad for SAMPLE data
 
 function loadSampleData(): void
 {
-    global $xoopsConfig;
-    $moduleDirName      = \basename(\dirname(__DIR__));
-    $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
-
-    $utility      = new Utility();
-    $configurator = new Configurator();
-
-    $tables = \Xmf\Module\Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
-
-    $language = 'english/';
-    if (is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
-        $language = $xoopsConfig['language'] . '/';
-    }
-
-    // load module tables
-    foreach ($tables as $table) {
-        $tabledata = Yaml::readWrapped($language . $table . '.yml');
-        TableLoad::truncateTable($table);
-        TableLoad::loadTableFromArray($table, $tabledata);
-    }
-
-    // load permissions
-    $table     = 'group_permission';
-    $tabledata = Yaml::readWrapped($language . $table . '.yml');
-    $mid       = \Xmf\Module\Helper::getHelper($moduleDirName)->getModule()->getVar('mid');
-    loadTableFromArrayWithReplace($table, $tabledata, 'gperm_modid', $mid);
-
-    //  ---  COPY test folder files ---------------
-    if (is_array($configurator->copyTestFolders) && count($configurator->copyTestFolders) > 0) {
-        //        $file =  \dirname(__DIR__) . '/testdata/images/';
-        foreach (array_keys($configurator->copyTestFolders) as $i) {
-            $src  = $configurator->copyTestFolders[$i][0];
-            $dest = $configurator->copyTestFolders[$i][1];
-            $utility::rcopy($src, $dest);
-        }
-    }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'LOAD_SAMPLEDATA_SUCCESS'));
+	global $xoopsConfig;
+	$moduleDirName      = \basename(\dirname(__DIR__));
+	$moduleDirNameUpper = \mb_strtoupper($moduleDirName);
+
+	$utility      = new Utility();
+	$configurator = new Configurator();
+
+	$tables = \Xmf\Module\Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
+
+	$language = 'english/';
+	if (is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
+		$language = $xoopsConfig['language'] . '/';
+	}
+
+	// load module tables
+	foreach ($tables as $table) {
+		$tabledata = Yaml::readWrapped($language . $table . '.yml');
+		TableLoad::truncateTable($table);
+		TableLoad::loadTableFromArray($table, $tabledata);
+	}
+
+	// load permissions
+	$table     = 'group_permission';
+	$tabledata = Yaml::readWrapped($language . $table . '.yml');
+	$mid       = \Xmf\Module\Helper::getHelper($moduleDirName)->getModule()->getVar('mid');
+	loadTableFromArrayWithReplace($table, $tabledata, 'gperm_modid', $mid);
+
+	//  ---  COPY test folder files ---------------
+	if (is_array($configurator->copyTestFolders) && count($configurator->copyTestFolders) > 0) {
+		//        $file =  \dirname(__DIR__) . '/testdata/images/';
+		foreach (array_keys($configurator->copyTestFolders) as $i) {
+			$src  = $configurator->copyTestFolders[$i][0];
+			$dest = $configurator->copyTestFolders[$i][1];
+			$utility::rcopy($src, $dest);
+		}
+	}
+	\redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'LOAD_SAMPLEDATA_SUCCESS'));
 }
 
 function saveSampleData(): void
 {
-    global $xoopsConfig;
-    $moduleDirName      = \basename(\dirname(__DIR__));
-    $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
-    $helper             = Helper::getInstance();
-    $tables             = $helper->getModule()->getInfo('tables');
-
-    $languageFolder = __DIR__ . '/' . $xoopsConfig['language'];
-    if (!file_exists($languageFolder . '/')) {
-        Utility::createFolder($languageFolder . '/');
-    }
-    $exportFolder = $languageFolder . '/Exports-' . date('Y-m-d-H-i-s') . '/';
-    Utility::createFolder($exportFolder);
-
-    // save module tables
-    foreach ($tables as $table) {
-        TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
-    }
-
-    // save permissions
-    $criteria = new \CriteriaCompo();
-    $criteria->add(new \Criteria('gperm_modid', $helper->getModule()->getVar('mid')));
-    $skipColumns[] = 'gperm_id';
-    TableLoad::saveTableToYamlFile('group_permission', $exportFolder . 'group_permission.yml', $criteria, $skipColumns);
-    unset($criteria);
-
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA_SUCCESS'));
+	global $xoopsConfig;
+	$moduleDirName      = \basename(\dirname(__DIR__));
+	$moduleDirNameUpper = \mb_strtoupper($moduleDirName);
+	$helper             = Helper::getInstance();
+	$tables             = $helper->getModule()->getInfo('tables');
+
+	$languageFolder = __DIR__ . '/' . $xoopsConfig['language'];
+	if (!file_exists($languageFolder . '/')) {
+		Utility::createFolder($languageFolder . '/');
+	}
+	$exportFolder = $languageFolder . '/Exports-' . date('Y-m-d-H-i-s') . '/';
+	Utility::createFolder($exportFolder);
+
+	// save module tables
+	foreach ($tables as $table) {
+		TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
+	}
+
+	// save permissions
+	$criteria = new \CriteriaCompo();
+	$criteria->add(new \Criteria('gperm_modid', $helper->getModule()->getVar('mid')));
+	$skipColumns[] = 'gperm_id';
+	TableLoad::saveTableToYamlFile('group_permission', $exportFolder . 'group_permission.yml', $criteria, $skipColumns);
+	unset($criteria);
+
+	\redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA_SUCCESS'));
 }
 
 function exportSchema(): void
 {
-    $moduleDirName      = \basename(\dirname(__DIR__));
-    $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
-
-    try {
-        // TODO set exportSchema
-        //        $migrate = new Migrate($moduleDirName);
-        //        $migrate->saveCurrentSchema();
-        //
-        //        redirect_header('../admin/index.php', 1, constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
-    } catch (\Throwable $e) {
-        exit(constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
-    }
+	$moduleDirName      = \basename(\dirname(__DIR__));
+	$moduleDirNameUpper = \mb_strtoupper($moduleDirName);
+
+	try {
+		// TODO set exportSchema
+		//        $migrate = new Migrate($moduleDirName);
+		//        $migrate->saveCurrentSchema();
+		//
+		//        redirect_header('../admin/index.php', 1, constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
+	} catch (\Throwable $e) {
+		exit(constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
+	}
 }
 
 /**
@@ -169,58 +169,58 @@ 
  */
 function loadTableFromArrayWithReplace($table, $data, $search, $replace): int
 {
-    /** @var \XoopsMySQLDatabase $db */
-    $db = \XoopsDatabaseFactory::getDatabaseConnection();
-
-    $prefixedTable = $db->prefix($table);
-    $count         = 0;
-
-    $sql = 'DELETE FROM ' . $prefixedTable . ' WHERE `' . $search . '`=' . $db->quote($replace);
-
-    $result = $db->queryF($sql);
-
-    foreach ($data as $row) {
-        $insertInto  = 'INSERT INTO ' . $prefixedTable . ' (';
-        $valueClause = ' VALUES (';
-        $first       = true;
-        foreach ($row as $column => $value) {
-            if ($first) {
-                $first = false;
-            } else {
-                $insertInto  .= ', ';
-                $valueClause .= ', ';
-            }
-
-            $insertInto .= $column;
-            if ($search === $column) {
-                $valueClause .= $db->quote($replace);
-            } else {
-                $valueClause .= $db->quote($value);
-            }
-        }
-
-        $sql = $insertInto . ') ' . $valueClause . ')';
-
-        $result = $db->queryF($sql);
-        if (false !== $result) {
-            ++$count;
-        }
-    }
-
-    return $count;
+	/** @var \XoopsMySQLDatabase $db */
+	$db = \XoopsDatabaseFactory::getDatabaseConnection();
+
+	$prefixedTable = $db->prefix($table);
+	$count         = 0;
+
+	$sql = 'DELETE FROM ' . $prefixedTable . ' WHERE `' . $search . '`=' . $db->quote($replace);
+
+	$result = $db->queryF($sql);
+
+	foreach ($data as $row) {
+		$insertInto  = 'INSERT INTO ' . $prefixedTable . ' (';
+		$valueClause = ' VALUES (';
+		$first       = true;
+		foreach ($row as $column => $value) {
+			if ($first) {
+				$first = false;
+			} else {
+				$insertInto  .= ', ';
+				$valueClause .= ', ';
+			}
+
+			$insertInto .= $column;
+			if ($search === $column) {
+				$valueClause .= $db->quote($replace);
+			} else {
+				$valueClause .= $db->quote($value);
+			}
+		}
+
+		$sql = $insertInto . ') ' . $valueClause . ')';
+
+		$result = $db->queryF($sql);
+		if (false !== $result) {
+			++$count;
+		}
+	}
+
+	return $count;
 }
 
 function clearSampleData(): void
 {
-    $moduleDirName      = \basename(\dirname(__DIR__));
-    $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
-    $helper             = Helper::getInstance();
-    // Load language files
-    $helper->loadLanguage('common');
-    $tables = $helper->getModule()->getInfo('tables');
-    // truncate module tables
-    foreach ($tables as $table) {
-        \Xmf\Database\TableLoad::truncateTable($table);
-    }
-    redirect_header($helper->url('admin/index.php'), 1, constant('CO_' . $moduleDirNameUpper . '_' . 'CLEAR_SAMPLEDATA_OK'));
+	$moduleDirName      = \basename(\dirname(__DIR__));
+	$moduleDirNameUpper = \mb_strtoupper($moduleDirName);
+	$helper             = Helper::getInstance();
+	// Load language files
+	$helper->loadLanguage('common');
+	$tables = $helper->getModule()->getInfo('tables');
+	// truncate module tables
+	foreach ($tables as $table) {
+		\Xmf\Database\TableLoad::truncateTable($table);
+	}
+	redirect_header($helper->url('admin/index.php'), 1, constant('CO_' . $moduleDirNameUpper . '_' . 'CLEAR_SAMPLEDATA_OK'));
 }

Spacing +26 added lines, -26 removed lines

@@ -24,8 +24,8 @@ 
 /** @var Helper $helper */
 /** @var Utility $utility */
 /** @var Configurator $configurator */
-require \dirname(__DIR__, 3) . '/include/cp_header.php';
-require \dirname(__DIR__) . '/preloads/autoloader.php';
+require \dirname(__DIR__, 3).'/include/cp_header.php';
+require \dirname(__DIR__).'/preloads/autoloader.php';
 
 $op = Request::getCmd('op', '');
 
@@ -38,14 +38,14 @@ 
 
 switch ($op) {
     case 'load':
-        if (Request::hasVar('ok', 'REQUEST') && 1 === Request::getInt('ok', 0)) {
+        if (Request::hasVar('ok', 'REQUEST') && 1===Request::getInt('ok', 0)) {
             if (!$GLOBALS['xoopsSecurity']->check()) {
                 redirect_header($helper->url('admin/index.php'), 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             loadSampleData();
         } else {
             xoops_cp_header();
-            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', constant('CO_' . $moduleDirNameUpper . '_' . 'LOAD_SAMPLEDATA_CONFIRM'), constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
+            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', constant('CO_'.$moduleDirNameUpper.'_'.'LOAD_SAMPLEDATA_CONFIRM'), constant('CO_'.$moduleDirNameUpper.'_'.'CONFIRM'), true);
             xoops_cp_footer();
         }
         break;
@@ -53,14 +53,14 @@ 
         saveSampleData();
         break;
     case 'clear':
-        if (Request::hasVar('ok', 'REQUEST') && 1 === Request::getInt('ok', 0)) {
+        if (Request::hasVar('ok', 'REQUEST') && 1===Request::getInt('ok', 0)) {
             if (!$GLOBALS['xoopsSecurity']->check()) {
                 redirect_header($helper->url('admin/index.php'), 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             clearSampleData();
         } else {
             xoops_cp_header();
-            xoops_confirm(['ok' => 1, 'op' => 'clear'], 'index.php', sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'CLEAR_SAMPLEDATA')), constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
+            xoops_confirm(['ok' => 1, 'op' => 'clear'], 'index.php', sprintf(constant('CO_'.$moduleDirNameUpper.'_'.'CLEAR_SAMPLEDATA')), constant('CO_'.$moduleDirNameUpper.'_'.'CONFIRM'), true);
             xoops_cp_footer();
         }
         break;
@@ -80,25 +80,25 @@ 
     $tables = \Xmf\Module\Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
 
     $language = 'english/';
-    if (is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
-        $language = $xoopsConfig['language'] . '/';
+    if (is_dir(__DIR__.'/'.$xoopsConfig['language'])) {
+        $language = $xoopsConfig['language'].'/';
     }
 
     // load module tables
     foreach ($tables as $table) {
-        $tabledata = Yaml::readWrapped($language . $table . '.yml');
+        $tabledata = Yaml::readWrapped($language.$table.'.yml');
         TableLoad::truncateTable($table);
         TableLoad::loadTableFromArray($table, $tabledata);
     }
 
     // load permissions
     $table     = 'group_permission';
-    $tabledata = Yaml::readWrapped($language . $table . '.yml');
+    $tabledata = Yaml::readWrapped($language.$table.'.yml');
     $mid       = \Xmf\Module\Helper::getHelper($moduleDirName)->getModule()->getVar('mid');
     loadTableFromArrayWithReplace($table, $tabledata, 'gperm_modid', $mid);
 
     //  ---  COPY test folder files ---------------
-    if (is_array($configurator->copyTestFolders) && count($configurator->copyTestFolders) > 0) {
+    if (is_array($configurator->copyTestFolders) && count($configurator->copyTestFolders)>0) {
         //        $file =  \dirname(__DIR__) . '/testdata/images/';
         foreach (array_keys($configurator->copyTestFolders) as $i) {
             $src  = $configurator->copyTestFolders[$i][0];
@@ -106,7 +106,7 @@ 
             $utility::rcopy($src, $dest);
         }
     }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'LOAD_SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'LOAD_SAMPLEDATA_SUCCESS'));
 }
 
 function saveSampleData(): void
@@ -117,26 +117,26 @@ 
     $helper             = Helper::getInstance();
     $tables             = $helper->getModule()->getInfo('tables');
 
-    $languageFolder = __DIR__ . '/' . $xoopsConfig['language'];
-    if (!file_exists($languageFolder . '/')) {
-        Utility::createFolder($languageFolder . '/');
+    $languageFolder = __DIR__.'/'.$xoopsConfig['language'];
+    if (!file_exists($languageFolder.'/')) {
+        Utility::createFolder($languageFolder.'/');
     }
-    $exportFolder = $languageFolder . '/Exports-' . date('Y-m-d-H-i-s') . '/';
+    $exportFolder = $languageFolder.'/Exports-'.date('Y-m-d-H-i-s').'/';
     Utility::createFolder($exportFolder);
 
     // save module tables
     foreach ($tables as $table) {
-        TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
+        TableLoad::saveTableToYamlFile($table, $exportFolder.$table.'.yml');
     }
 
     // save permissions
     $criteria = new \CriteriaCompo();
     $criteria->add(new \Criteria('gperm_modid', $helper->getModule()->getVar('mid')));
     $skipColumns[] = 'gperm_id';
-    TableLoad::saveTableToYamlFile('group_permission', $exportFolder . 'group_permission.yml', $criteria, $skipColumns);
+    TableLoad::saveTableToYamlFile('group_permission', $exportFolder.'group_permission.yml', $criteria, $skipColumns);
     unset($criteria);
 
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'SAVE_SAMPLEDATA_SUCCESS'));
 }
 
 function exportSchema(): void
@@ -151,7 +151,7 @@ 
         //
         //        redirect_header('../admin/index.php', 1, constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
     } catch (\Throwable $e) {
-        exit(constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
+        exit(constant('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA_ERROR'));
     }
 }
 
@@ -175,12 +175,12 @@ 
     $prefixedTable = $db->prefix($table);
     $count         = 0;
 
-    $sql = 'DELETE FROM ' . $prefixedTable . ' WHERE `' . $search . '`=' . $db->quote($replace);
+    $sql = 'DELETE FROM '.$prefixedTable.' WHERE `'.$search.'`='.$db->quote($replace);
 
     $result = $db->queryF($sql);
 
     foreach ($data as $row) {
-        $insertInto  = 'INSERT INTO ' . $prefixedTable . ' (';
+        $insertInto  = 'INSERT INTO '.$prefixedTable.' (';
         $valueClause = ' VALUES (';
         $first       = true;
         foreach ($row as $column => $value) {
@@ -192,17 +192,17 @@ 
             }
 
             $insertInto .= $column;
-            if ($search === $column) {
+            if ($search===$column) {
                 $valueClause .= $db->quote($replace);
             } else {
                 $valueClause .= $db->quote($value);
             }
         }
 
-        $sql = $insertInto . ') ' . $valueClause . ')';
+        $sql = $insertInto.') '.$valueClause.')';
 
         $result = $db->queryF($sql);
-        if (false !== $result) {
+        if (false!==$result) {
             ++$count;
         }
     }
@@ -222,5 +222,5 @@ 
     foreach ($tables as $table) {
         \Xmf\Database\TableLoad::truncateTable($table);
     }
-    redirect_header($helper->url('admin/index.php'), 1, constant('CO_' . $moduleDirNameUpper . '_' . 'CLEAR_SAMPLEDATA_OK'));
+    redirect_header($helper->url('admin/index.php'), 1, constant('CO_'.$moduleDirNameUpper.'_'.'CLEAR_SAMPLEDATA_OK'));
 }

index.php

Indentation +314 added lines, -314 removed lines

@@ -19,12 +19,12 @@ 
 $criteria_cat    = new \CriteriaCompo();
 $cids            = $categoryHandler->GetCatAndSubCat($_SESSION['cid']);
 if ($_SESSION['cid'] > 0) {
-    $cids[$_SESSION['cid']] = $_SESSION['cid'];
+	$cids[$_SESSION['cid']] = $_SESSION['cid'];
 }
 if (count($cids) > 0 && 0 != $_SESSION['cid']) {
-    $criteria_cat->add(new \Criteria('cid', '(' . implode(',', $cids) . ')', 'IN'), 'OR');
+	$criteria_cat->add(new \Criteria('cid', '(' . implode(',', $cids) . ')', 'IN'), 'OR');
 } else {
-    $criteria_cat->add(new \Criteria(''), 'OR');
+	$criteria_cat->add(new \Criteria(''), 'OR');
 }
 $criteria_cat->setSort('created');
 $criteria_cat->setOrder('ASC');
@@ -38,20 +38,20 @@ 
 $col = 1;
 $row = 1;
 foreach ($categories as $category) {
-    $cat[$row][$col]          = $category->toArray(true);
-    $cat[$row][$col]['width'] = floor(100 / $GLOBALS['songlistModuleConfig']['cols']) . '%';
-    ++$col;
-    if ($col > $GLOBALS['songlistModuleConfig']['cols']) {
-        ++$row;
-        $col = 1;
-    }
+	$cat[$row][$col]          = $category->toArray(true);
+	$cat[$row][$col]['width'] = floor(100 / $GLOBALS['songlistModuleConfig']['cols']) . '%';
+	++$col;
+	if ($col > $GLOBALS['songlistModuleConfig']['cols']) {
+		++$row;
+		$col = 1;
+	}
 }
 if (1 != $col) {
-    $col--;
-    for ($j = $col; $j <= $GLOBALS['songlistModuleConfig']['cols']; ++$j) {
-        $cat[$row][$j][$categoryHandler->keyName] = 0;
-        $cat[$row][$j]['width']                   = floor(100 / $GLOBALS['songlistModuleConfig']['cols']) . '%';
-    }
+	$col--;
+	for ($j = $col; $j <= $GLOBALS['songlistModuleConfig']['cols']; ++$j) {
+		$cat[$row][$j][$categoryHandler->keyName] = 0;
+		$cat[$row][$j]['width']                   = floor(100 / $GLOBALS['songlistModuleConfig']['cols']) . '%';
+	}
 }
 
 $category_element = new SelectCategoryForm('', 'cid', ($_GET['cid'] ?? $cid));
@@ -61,324 +61,324 @@ 
 /** @var SongsHandler $songsHandler */
 $songsHandler = Helper::getInstance()->getHandler('Songs');
 switch ($op) {
-    case 'vote':
-        /** @var VotesHandler $votesHandler */
-        $votesHandler = Helper::getInstance()->getHandler('Votes');
-        $votesHandler->addVote($id, $value);
-        redirect_header($_POST['uri'], 10, _MD_SONGLIST_MSG_VOTED_ALREADY);
-    // no break
-    default:
-    case 'item':
-        switch ($fct) {
-            default:
-            case 'list':
-                $pagenav = new \XoopsPageNav($songsHandler->getCount($criteria_cat), $limit, $start, 'start', "op=$op&fct=$fct&id=$id&value=$value&limit=$limit");
+	case 'vote':
+		/** @var VotesHandler $votesHandler */
+		$votesHandler = Helper::getInstance()->getHandler('Votes');
+		$votesHandler->addVote($id, $value);
+		redirect_header($_POST['uri'], 10, _MD_SONGLIST_MSG_VOTED_ALREADY);
+	// no break
+	default:
+	case 'item':
+		switch ($fct) {
+			default:
+			case 'list':
+				$pagenav = new \XoopsPageNav($songsHandler->getCount($criteria_cat), $limit, $start, 'start', "op=$op&fct=$fct&id=$id&value=$value&limit=$limit");
 
-                $criteria_cat->setLimit($limit);
-                $criteria_cat->setStart($start);
+				$criteria_cat->setLimit($limit);
+				$criteria_cat->setStart($start);
 
-                $songs = $songsHandler->getObjects($criteria_cat, false);
+				$songs = $songsHandler->getObjects($criteria_cat, false);
 
-                $url = $songsHandler->getURL();
-                if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
-                    header('HTTP/1.1 301 Moved Permanently');
-                    header('Location: ' . $url);
-                    exit(0);
-                }
+				$url = $songsHandler->getURL();
+				if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
+					header('HTTP/1.1 301 Moved Permanently');
+					header('Location: ' . $url);
+					exit(0);
+				}
 
-                $GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_index.tpl';
-                require_once $GLOBALS['xoops']->path('/header.php');
-                if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-                    $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
-                    $GLOBALS['loaded_jquery'] = true;
-                }
-                $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
-                $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                foreach ($songs as $song) {
-                    $GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
-                }
-                $GLOBALS['xoopsTpl']->assign('songs', true);
-                $GLOBALS['xoopsTpl']->assign('categories', $cat);
-                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
-                $GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
-                $GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
-                $GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
-                $GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
-                if (0 != $_SESSION['cid']) {
-                    $category = $categoryHandler->get($_SESSION['cid']);
-                    if (is_object($category)) {
-                        $GLOBALS['xoopsTpl']->assign('category', $category->toArray(true));
-                    }
-                }
-                $GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
-                require_once $GLOBALS['xoops']->path('/footer.php');
-                break;
-            case 'item':
-                $song = $songsHandler->get($id);
+				$GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_index.tpl';
+				require_once $GLOBALS['xoops']->path('/header.php');
+				if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
+					$GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+					$GLOBALS['loaded_jquery'] = true;
+				}
+				$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+				$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				foreach ($songs as $song) {
+					$GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
+				}
+				$GLOBALS['xoopsTpl']->assign('songs', true);
+				$GLOBALS['xoopsTpl']->assign('categories', $cat);
+				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
+				$GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
+				$GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
+				$GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
+				$GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
+				if (0 != $_SESSION['cid']) {
+					$category = $categoryHandler->get($_SESSION['cid']);
+					if (is_object($category)) {
+						$GLOBALS['xoopsTpl']->assign('category', $category->toArray(true));
+					}
+				}
+				$GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
+				require_once $GLOBALS['xoops']->path('/footer.php');
+				break;
+			case 'item':
+				$song = $songsHandler->get($id);
 
-                $url = $song->getURL(true);
-                if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
-                    header('HTTP/1.1 301 Moved Permanently');
-                    header('Location: ' . $url);
-                    exit(0);
-                }
+				$url = $song->getURL(true);
+				if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
+					header('HTTP/1.1 301 Moved Permanently');
+					header('Location: ' . $url);
+					exit(0);
+				}
 
-                $GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_item.tpl';
-                require_once $GLOBALS['xoops']->path('/header.php');
-                if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-                    $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
-                    $GLOBALS['loaded_jquery'] = true;
-                }
-                $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
-                $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-                $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-                $GLOBALS['xoopsTpl']->assign('songs', false);
-                $GLOBALS['xoopsTpl']->assign('song', $song->toArray(true));
-                $GLOBALS['xoopsTpl']->assign('categories', $cat);
-                $GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
-                $GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
-                $GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
-                $GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
-                require_once $GLOBALS['xoops']->path('/footer.php');
-                break;
-        }
-        break;
-    case 'browseby':
-        switch ($fct) {
-            default:
-            case 'title':
-                $browse_criteria = new \CriteriaCompo();
-                switch ($value) {
-                    case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('title', $u . '%', 'LIKE'), 'OR');
-                        }
-                        break;
-                    default:
-                        $browse_criteria->add(new \Criteria('title', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('title', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
-                        break;
-                }
-                $criteria = new \CriteriaCompo($criteria_cat, 'AND');
-                $criteria->add($browse_criteria);
-                break;
-            case 'lyrics':
-                $browse_criteria = new \CriteriaCompo();
-                switch ($value) {
-                    case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('lyrics', $u . '%', 'LIKE'), 'OR');
-                        }
-                        break;
-                    default:
-                        $browse_criteria->add(new \Criteria('lyrics', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('lyrics', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
-                        break;
-                }
-                $criteria = new \CriteriaCompo($criteria_cat, 'AND');
-                $criteria->add($browse_criteria);
-                break;
-            case 'artist':
-                $browse_criteria = new \CriteriaCompo();
-                switch ($value) {
-                    case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('name', $u . '%', 'LIKE'), 'OR');
-                        }
-                        break;
-                    default:
-                        $browse_criteria->add(new \Criteria('name', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('name', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
-                        break;
-                }
-                $artistsHandler   = Helper::getInstance()->getHandler('Artists');
-                $browse_criteriab = new \CriteriaCompo();
-                foreach ($artistsHandler->getObjects($browse_criteria, true) as $aid => $obj) {
-                    $browse_criteriab->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
-                }
-                $criteria = new \CriteriaCompo($criteria_cat, 'AND');
-                $criteria->add($browse_criteriab);
-                break;
-            case 'album':
-                $browse_criteria = new \CriteriaCompo();
-                switch ($value) {
-                    case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('title', $u . '%', 'LIKE'), 'OR');
-                        }
-                        break;
-                    default:
-                        $browse_criteria->add(new \Criteria('title', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('title', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
-                        break;
-                }
-                $ids           = [];
-                $albumsHandler = Helper::getInstance()->getHandler('Albums');
-                foreach ($albumsHandler->getObjects($browse_criteria, true) as $id => $obj) {
-                    $ids[$id] = $id;
-                }
-                $criteria = new \CriteriaCompo($criteria_cat, 'AND');
-                if (count($ids) > 0) {
-                    $criteria->add(new \Criteria('abid', '(' . implode(',', $ids) . ')', 'IN'));
-                }
-                break;
-        }
+				$GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_item.tpl';
+				require_once $GLOBALS['xoops']->path('/header.php');
+				if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
+					$GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+					$GLOBALS['loaded_jquery'] = true;
+				}
+				$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+				$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+				$GLOBALS['xoopsTpl']->assign('songs', false);
+				$GLOBALS['xoopsTpl']->assign('song', $song->toArray(true));
+				$GLOBALS['xoopsTpl']->assign('categories', $cat);
+				$GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
+				$GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
+				$GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
+				$GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
+				require_once $GLOBALS['xoops']->path('/footer.php');
+				break;
+		}
+		break;
+	case 'browseby':
+		switch ($fct) {
+			default:
+			case 'title':
+				$browse_criteria = new \CriteriaCompo();
+				switch ($value) {
+					case '0':
+						for ($u = 0; $u < 10; ++$u) {
+							$browse_criteria->add(new \Criteria('title', $u . '%', 'LIKE'), 'OR');
+						}
+						break;
+					default:
+						$browse_criteria->add(new \Criteria('title', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
+						$browse_criteria->add(new \Criteria('title', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+						break;
+				}
+				$criteria = new \CriteriaCompo($criteria_cat, 'AND');
+				$criteria->add($browse_criteria);
+				break;
+			case 'lyrics':
+				$browse_criteria = new \CriteriaCompo();
+				switch ($value) {
+					case '0':
+						for ($u = 0; $u < 10; ++$u) {
+							$browse_criteria->add(new \Criteria('lyrics', $u . '%', 'LIKE'), 'OR');
+						}
+						break;
+					default:
+						$browse_criteria->add(new \Criteria('lyrics', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
+						$browse_criteria->add(new \Criteria('lyrics', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+						break;
+				}
+				$criteria = new \CriteriaCompo($criteria_cat, 'AND');
+				$criteria->add($browse_criteria);
+				break;
+			case 'artist':
+				$browse_criteria = new \CriteriaCompo();
+				switch ($value) {
+					case '0':
+						for ($u = 0; $u < 10; ++$u) {
+							$browse_criteria->add(new \Criteria('name', $u . '%', 'LIKE'), 'OR');
+						}
+						break;
+					default:
+						$browse_criteria->add(new \Criteria('name', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
+						$browse_criteria->add(new \Criteria('name', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+						break;
+				}
+				$artistsHandler   = Helper::getInstance()->getHandler('Artists');
+				$browse_criteriab = new \CriteriaCompo();
+				foreach ($artistsHandler->getObjects($browse_criteria, true) as $aid => $obj) {
+					$browse_criteriab->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
+				}
+				$criteria = new \CriteriaCompo($criteria_cat, 'AND');
+				$criteria->add($browse_criteriab);
+				break;
+			case 'album':
+				$browse_criteria = new \CriteriaCompo();
+				switch ($value) {
+					case '0':
+						for ($u = 0; $u < 10; ++$u) {
+							$browse_criteria->add(new \Criteria('title', $u . '%', 'LIKE'), 'OR');
+						}
+						break;
+					default:
+						$browse_criteria->add(new \Criteria('title', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
+						$browse_criteria->add(new \Criteria('title', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+						break;
+				}
+				$ids           = [];
+				$albumsHandler = Helper::getInstance()->getHandler('Albums');
+				foreach ($albumsHandler->getObjects($browse_criteria, true) as $id => $obj) {
+					$ids[$id] = $id;
+				}
+				$criteria = new \CriteriaCompo($criteria_cat, 'AND');
+				if (count($ids) > 0) {
+					$criteria->add(new \Criteria('abid', '(' . implode(',', $ids) . ')', 'IN'));
+				}
+				break;
+		}
 
-        $pagenav = new \XoopsPageNav($songsHandler->getCount($criteria), $limit, $start, 'start', "op=$op&fct=$fct&id=$id&value=$value&limit=$limit");
+		$pagenav = new \XoopsPageNav($songsHandler->getCount($criteria), $limit, $start, 'start', "op=$op&fct=$fct&id=$id&value=$value&limit=$limit");
 
-        $criteria->setLimit($limit);
-        $criteria->setStart($start);
+		$criteria->setLimit($limit);
+		$criteria->setStart($start);
 
-        $songs = $songsHandler->getObjects($criteria, false);
+		$songs = $songsHandler->getObjects($criteria, false);
 
-        $url = $songsHandler->getURL();
-        if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
-            header('HTTP/1.1 301 Moved Permanently');
-            header('Location: ' . $url);
-            exit(0);
-        }
+		$url = $songsHandler->getURL();
+		if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
+			header('HTTP/1.1 301 Moved Permanently');
+			header('Location: ' . $url);
+			exit(0);
+		}
 
-        $GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_index.tpl';
-        require_once $GLOBALS['xoops']->path('/header.php');
-        if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-            $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
-            $GLOBALS['loaded_jquery'] = true;
-        }
-        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
-        $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-        $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-        foreach ($songs as $song) {
-            $GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
-        }
-        $GLOBALS['xoopsTpl']->assign('songs', true);
-        $GLOBALS['xoopsTpl']->assign('categories', $cat);
-        $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
-        $GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
-        $GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
-        $GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
-        $GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
-        if (0 != $_SESSION['cid']) {
-            $category = $categoryHandler->get($_SESSION['cid']);
-            if (is_object($category)) {
-                $GLOBALS['xoopsTpl']->assign('category', $category->toArray(true));
-            }
-        }
-        $GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
-        require_once $GLOBALS['xoops']->path('/footer.php');
-        break;
-        break;
-    case 'search':
-        /** @var SongsHandler $songsHandler */
-        $songsHandler   = Helper::getInstance()->getHandler('Songs');
-        $artistsHandler = Helper::getInstance()->getHandler('Artists');
+		$GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_index.tpl';
+		require_once $GLOBALS['xoops']->path('/header.php');
+		if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
+			$GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+			$GLOBALS['loaded_jquery'] = true;
+		}
+		$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+		$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+		$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+		foreach ($songs as $song) {
+			$GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
+		}
+		$GLOBALS['xoopsTpl']->assign('songs', true);
+		$GLOBALS['xoopsTpl']->assign('categories', $cat);
+		$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
+		$GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
+		$GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
+		$GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
+		$GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
+		if (0 != $_SESSION['cid']) {
+			$category = $categoryHandler->get($_SESSION['cid']);
+			if (is_object($category)) {
+				$GLOBALS['xoopsTpl']->assign('category', $category->toArray(true));
+			}
+		}
+		$GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
+		require_once $GLOBALS['xoops']->path('/footer.php');
+		break;
+		break;
+	case 'search':
+		/** @var SongsHandler $songsHandler */
+		$songsHandler   = Helper::getInstance()->getHandler('Songs');
+		$artistsHandler = Helper::getInstance()->getHandler('Artists');
 
-        /** @var Utf8mapHandler $utf8mapHandler */
-        $utf8mapHandler = Helper::getInstance()->getHandler('Utf8map');
+		/** @var Utf8mapHandler $utf8mapHandler */
+		$utf8mapHandler = Helper::getInstance()->getHandler('Utf8map');
 
-        $url = $songsHandler->getSearchURL();
-        if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
-            header('HTTP/1.1 301 Moved Permanently');
-            header('Location: ' . $url);
-            exit(0);
-        }
+		$url = $songsHandler->getSearchURL();
+		if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
+			header('HTTP/1.1 301 Moved Permanently');
+			header('Location: ' . $url);
+			exit(0);
+		}
 
-        switch ($fct) {
-            default:
-            case 'titleandlyrics':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                break;
-            case 'albums':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                $albums   = $albumsHandler->getObjects($criteria, true);
-                $criteria = new \CriteriaCompo();
-                foreach ($albums as $abid => $album) {
-                    $criteria->add(new \Criteria('abid', $abid), 'OR');
-                }
-                break;
-            case 'artists':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('name', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                $artists  = $artistsHandler->getObjects($criteria, true);
-                $criteria = new \CriteriaCompo();
-                if (is_array($artists)) {
-                    foreach ($artists as $aid => $artist) {
-                        $criteria->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
-                    }
-                }
-                break;
-            case 'lyrics':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                break;
-            case 'title':
-                $criteria = new \CriteriaCompo();
-                foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                }
-                break;
-        }
+		switch ($fct) {
+			default:
+			case 'titleandlyrics':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+					$criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				break;
+			case 'albums':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				$albums   = $albumsHandler->getObjects($criteria, true);
+				$criteria = new \CriteriaCompo();
+				foreach ($albums as $abid => $album) {
+					$criteria->add(new \Criteria('abid', $abid), 'OR');
+				}
+				break;
+			case 'artists':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('name', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				$artists  = $artistsHandler->getObjects($criteria, true);
+				$criteria = new \CriteriaCompo();
+				if (is_array($artists)) {
+					foreach ($artists as $aid => $artist) {
+						$criteria->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
+					}
+				}
+				break;
+			case 'lyrics':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				break;
+			case 'title':
+				$criteria = new \CriteriaCompo();
+				foreach (explode(' ', $value) as $keyword) {
+					$criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+				}
+				break;
+		}
 
-        if (0 != $gid && $GLOBALS['songlistModuleConfig']['genre']) {
-            $criteria->add(new \Criteria('gids', '%"' . $gid . '"%', 'LIKE'));
-        }
+		if (0 != $gid && $GLOBALS['songlistModuleConfig']['genre']) {
+			$criteria->add(new \Criteria('gids', '%"' . $gid . '"%', 'LIKE'));
+		}
 
-        if (0 != $vcid && $GLOBALS['songlistModuleConfig']['voice']) {
-            $criteria->add(new \Criteria('vcid', $vcid));
-        }
+		if (0 != $vcid && $GLOBALS['songlistModuleConfig']['voice']) {
+			$criteria->add(new \Criteria('vcid', $vcid));
+		}
 
-        if (0 != ($_GET['cid'] ?? $cid)) {
-            $criteria->add(new \Criteria('cid', ($_GET['cid'] ?? $cid)));
-        }
+		if (0 != ($_GET['cid'] ?? $cid)) {
+			$criteria->add(new \Criteria('cid', ($_GET['cid'] ?? $cid)));
+		}
 
-        $pagenav = new \XoopsPageNav($songsHandler->getCount($criteria), $limit, $start, 'start', "op=$op&fct=$fct&gid=$gid&vcid=$vcid&value=$value&limit=$limit");
+		$pagenav = new \XoopsPageNav($songsHandler->getCount($criteria), $limit, $start, 'start', "op=$op&fct=$fct&gid=$gid&vcid=$vcid&value=$value&limit=$limit");
 
-        $criteria->setLimit($limit);
-        $criteria->setStart($start);
+		$criteria->setLimit($limit);
+		$criteria->setStart($start);
 
-        $songs = $songsHandler->getObjects($criteria, false);
+		$songs = $songsHandler->getObjects($criteria, false);
 
-        $GLOBALS['xoopsOption']['template_main'] = 'songlist_search_index.tpl';
-        require $GLOBALS['xoops']->path('/header.php');
-        if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-            $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
-            $GLOBALS['loaded_jquery'] = true;
-        }
-        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
-        $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
-        $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
-        foreach ($songs as $song) {
-            $GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
-        }
-        $GLOBALS['xoopsTpl']->assign('songs', true);
-        $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
-        $GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
-        $GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
-        $GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
-        $GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
-        $GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
-        require $GLOBALS['xoops']->path('/footer.php');
-        break;
-    case 'category':
-        switch ($fct) {
-            default:
-            case 'set':
-                $_SESSION['cid'] = $id;
-                break;
-            case 'home':
-                unset($_SESSION['cid']);
-                break;
-        }
-        redirect_header($_SERVER['SCRIPT_NAME'] . "?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit&cid=" . $_SESSION['cid'], 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
+		$GLOBALS['xoopsOption']['template_main'] = 'songlist_search_index.tpl';
+		require $GLOBALS['xoops']->path('/header.php');
+		if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
+			$GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+			$GLOBALS['loaded_jquery'] = true;
+		}
+		$GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+		$GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
+		$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
+		foreach ($songs as $song) {
+			$GLOBALS['xoopsTpl']->append('results', $song->toArray(true));
+		}
+		$GLOBALS['xoopsTpl']->assign('songs', true);
+		$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
+		$GLOBALS['xoopsTpl']->assign('category_element', $category_element->render());
+		$GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
+		$GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
+		$GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
+		$GLOBALS['xoopsTpl']->assign('uri', $_SERVER['REQUEST_URI']);
+		require $GLOBALS['xoops']->path('/footer.php');
+		break;
+	case 'category':
+		switch ($fct) {
+			default:
+			case 'set':
+				$_SESSION['cid'] = $id;
+				break;
+			case 'home':
+				unset($_SESSION['cid']);
+				break;
+		}
+		redirect_header($_SERVER['SCRIPT_NAME'] . "?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit&cid=" . $_SESSION['cid'], 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
 }

Spacing +54 added lines, -54 removed lines

@@ -10,7 +10,7 @@ 
 use XoopsModules\Songlist\Utf8mapHandler;
 
 
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 global $file, $op, $fct, $id, $value, $gid, $cid, $vcid, $start, $limit, $singer;
 
@@ -18,11 +18,11 @@ 
 $categoryHandler = Helper::getInstance()->getHandler('Category');
 $criteria_cat    = new \CriteriaCompo();
 $cids            = $categoryHandler->GetCatAndSubCat($_SESSION['cid']);
-if ($_SESSION['cid'] > 0) {
+if ($_SESSION['cid']>0) {
     $cids[$_SESSION['cid']] = $_SESSION['cid'];
 }
-if (count($cids) > 0 && 0 != $_SESSION['cid']) {
-    $criteria_cat->add(new \Criteria('cid', '(' . implode(',', $cids) . ')', 'IN'), 'OR');
+if (count($cids)>0 && 0!=$_SESSION['cid']) {
+    $criteria_cat->add(new \Criteria('cid', '('.implode(',', $cids).')', 'IN'), 'OR');
 } else {
     $criteria_cat->add(new \Criteria(''), 'OR');
 }
@@ -39,18 +39,18 @@ 
 $row = 1;
 foreach ($categories as $category) {
     $cat[$row][$col]          = $category->toArray(true);
-    $cat[$row][$col]['width'] = floor(100 / $GLOBALS['songlistModuleConfig']['cols']) . '%';
+    $cat[$row][$col]['width'] = floor(100/$GLOBALS['songlistModuleConfig']['cols']).'%';
     ++$col;
-    if ($col > $GLOBALS['songlistModuleConfig']['cols']) {
+    if ($col>$GLOBALS['songlistModuleConfig']['cols']) {
         ++$row;
         $col = 1;
     }
 }
-if (1 != $col) {
+if (1!=$col) {
     $col--;
-    for ($j = $col; $j <= $GLOBALS['songlistModuleConfig']['cols']; ++$j) {
+    for ($j = $col; $j<=$GLOBALS['songlistModuleConfig']['cols']; ++$j) {
         $cat[$row][$j][$categoryHandler->keyName] = 0;
-        $cat[$row][$j]['width']                   = floor(100 / $GLOBALS['songlistModuleConfig']['cols']) . '%';
+        $cat[$row][$j]['width']                   = floor(100/$GLOBALS['songlistModuleConfig']['cols']).'%';
     }
 }
 
@@ -82,17 +82,17 @@ 
                 $url = $songsHandler->getURL();
                 if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
                     header('HTTP/1.1 301 Moved Permanently');
-                    header('Location: ' . $url);
+                    header('Location: '.$url);
                     exit(0);
                 }
 
                 $GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_index.tpl';
                 require_once $GLOBALS['xoops']->path('/header.php');
                 if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-                    $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+                    $GLOBALS['xoTheme']->addScript(XOOPS_URL._MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
                     $GLOBALS['loaded_jquery'] = true;
                 }
-                $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+                $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL._MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
                 $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
                 $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
                 foreach ($songs as $song) {
@@ -105,7 +105,7 @@ 
                 $GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
                 $GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
                 $GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
-                if (0 != $_SESSION['cid']) {
+                if (0!=$_SESSION['cid']) {
                     $category = $categoryHandler->get($_SESSION['cid']);
                     if (is_object($category)) {
                         $GLOBALS['xoopsTpl']->assign('category', $category->toArray(true));
@@ -120,17 +120,17 @@ 
                 $url = $song->getURL(true);
                 if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
                     header('HTTP/1.1 301 Moved Permanently');
-                    header('Location: ' . $url);
+                    header('Location: '.$url);
                     exit(0);
                 }
 
                 $GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_item.tpl';
                 require_once $GLOBALS['xoops']->path('/header.php');
                 if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-                    $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+                    $GLOBALS['xoTheme']->addScript(XOOPS_URL._MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
                     $GLOBALS['loaded_jquery'] = true;
                 }
-                $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+                $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL._MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
                 $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
                 $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
                 $GLOBALS['xoopsTpl']->assign('songs', false);
@@ -151,13 +151,13 @@ 
                 $browse_criteria = new \CriteriaCompo();
                 switch ($value) {
                     case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('title', $u . '%', 'LIKE'), 'OR');
+                        for ($u = 0; $u<10; ++$u) {
+                            $browse_criteria->add(new \Criteria('title', $u.'%', 'LIKE'), 'OR');
                         }
                         break;
                     default:
-                        $browse_criteria->add(new \Criteria('title', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('title', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('title', \mb_strtoupper($value).'%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('title', \mb_strtolower($value).'%', 'LIKE'), 'OR');
                         break;
                 }
                 $criteria = new \CriteriaCompo($criteria_cat, 'AND');
@@ -167,13 +167,13 @@ 
                 $browse_criteria = new \CriteriaCompo();
                 switch ($value) {
                     case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('lyrics', $u . '%', 'LIKE'), 'OR');
+                        for ($u = 0; $u<10; ++$u) {
+                            $browse_criteria->add(new \Criteria('lyrics', $u.'%', 'LIKE'), 'OR');
                         }
                         break;
                     default:
-                        $browse_criteria->add(new \Criteria('lyrics', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('lyrics', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('lyrics', \mb_strtoupper($value).'%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('lyrics', \mb_strtolower($value).'%', 'LIKE'), 'OR');
                         break;
                 }
                 $criteria = new \CriteriaCompo($criteria_cat, 'AND');
@@ -183,19 +183,19 @@ 
                 $browse_criteria = new \CriteriaCompo();
                 switch ($value) {
                     case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('name', $u . '%', 'LIKE'), 'OR');
+                        for ($u = 0; $u<10; ++$u) {
+                            $browse_criteria->add(new \Criteria('name', $u.'%', 'LIKE'), 'OR');
                         }
                         break;
                     default:
-                        $browse_criteria->add(new \Criteria('name', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('name', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('name', \mb_strtoupper($value).'%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('name', \mb_strtolower($value).'%', 'LIKE'), 'OR');
                         break;
                 }
                 $artistsHandler   = Helper::getInstance()->getHandler('Artists');
                 $browse_criteriab = new \CriteriaCompo();
                 foreach ($artistsHandler->getObjects($browse_criteria, true) as $aid => $obj) {
-                    $browse_criteriab->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
+                    $browse_criteriab->add(new \Criteria('aids', '%"'.$aid.'"%', 'LIKE'), 'OR');
                 }
                 $criteria = new \CriteriaCompo($criteria_cat, 'AND');
                 $criteria->add($browse_criteriab);
@@ -204,13 +204,13 @@ 
                 $browse_criteria = new \CriteriaCompo();
                 switch ($value) {
                     case '0':
-                        for ($u = 0; $u < 10; ++$u) {
-                            $browse_criteria->add(new \Criteria('title', $u . '%', 'LIKE'), 'OR');
+                        for ($u = 0; $u<10; ++$u) {
+                            $browse_criteria->add(new \Criteria('title', $u.'%', 'LIKE'), 'OR');
                         }
                         break;
                     default:
-                        $browse_criteria->add(new \Criteria('title', \mb_strtoupper($value) . '%', 'LIKE'), 'OR');
-                        $browse_criteria->add(new \Criteria('title', \mb_strtolower($value) . '%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('title', \mb_strtoupper($value).'%', 'LIKE'), 'OR');
+                        $browse_criteria->add(new \Criteria('title', \mb_strtolower($value).'%', 'LIKE'), 'OR');
                         break;
                 }
                 $ids           = [];
@@ -219,8 +219,8 @@ 
                     $ids[$id] = $id;
                 }
                 $criteria = new \CriteriaCompo($criteria_cat, 'AND');
-                if (count($ids) > 0) {
-                    $criteria->add(new \Criteria('abid', '(' . implode(',', $ids) . ')', 'IN'));
+                if (count($ids)>0) {
+                    $criteria->add(new \Criteria('abid', '('.implode(',', $ids).')', 'IN'));
                 }
                 break;
         }
@@ -235,17 +235,17 @@ 
         $url = $songsHandler->getURL();
         if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
             header('HTTP/1.1 301 Moved Permanently');
-            header('Location: ' . $url);
+            header('Location: '.$url);
             exit(0);
         }
 
         $GLOBALS['xoopsOption']['template_main'] = 'songlist_songs_index.tpl';
         require_once $GLOBALS['xoops']->path('/header.php');
         if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-            $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+            $GLOBALS['xoTheme']->addScript(XOOPS_URL._MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
             $GLOBALS['loaded_jquery'] = true;
         }
-        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL._MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
         $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
         $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
         foreach ($songs as $song) {
@@ -258,7 +258,7 @@ 
         $GLOBALS['xoopsTpl']->assign('genre_element', $genre_element->render());
         $GLOBALS['xoopsTpl']->assign('voice_element', $voice_element->render());
         $GLOBALS['xoopsTpl']->assign('cid', $_SESSION['cid']);
-        if (0 != $_SESSION['cid']) {
+        if (0!=$_SESSION['cid']) {
             $category = $categoryHandler->get($_SESSION['cid']);
             if (is_object($category)) {
                 $GLOBALS['xoopsTpl']->assign('category', $category->toArray(true));
@@ -279,7 +279,7 @@ 
         $url = $songsHandler->getSearchURL();
         if (!mb_strpos($url, $_SERVER['REQUEST_URI'])) {
             header('HTTP/1.1 301 Moved Permanently');
-            header('Location: ' . $url);
+            header('Location: '.$url);
             exit(0);
         }
 
@@ -288,14 +288,14 @@ 
             case 'titleandlyrics':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('title', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
+                    $criteria->add(new \Criteria('lyrics', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 break;
             case 'albums':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('title', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 $albums   = $albumsHandler->getObjects($criteria, true);
                 $criteria = new \CriteriaCompo();
@@ -306,39 +306,39 @@ 
             case 'artists':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('name', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('name', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 $artists  = $artistsHandler->getObjects($criteria, true);
                 $criteria = new \CriteriaCompo();
                 if (is_array($artists)) {
                     foreach ($artists as $aid => $artist) {
-                        $criteria->add(new \Criteria('aids', '%"' . $aid . '"%', 'LIKE'), 'OR');
+                        $criteria->add(new \Criteria('aids', '%"'.$aid.'"%', 'LIKE'), 'OR');
                     }
                 }
                 break;
             case 'lyrics':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('lyrics', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('lyrics', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 break;
             case 'title':
                 $criteria = new \CriteriaCompo();
                 foreach (explode(' ', $value) as $keyword) {
-                    $criteria->add(new \Criteria('title', '%' . $utf8mapHandler->convert($keyword) . '%', 'LIKE'));
+                    $criteria->add(new \Criteria('title', '%'.$utf8mapHandler->convert($keyword).'%', 'LIKE'));
                 }
                 break;
         }
 
-        if (0 != $gid && $GLOBALS['songlistModuleConfig']['genre']) {
-            $criteria->add(new \Criteria('gids', '%"' . $gid . '"%', 'LIKE'));
+        if (0!=$gid && $GLOBALS['songlistModuleConfig']['genre']) {
+            $criteria->add(new \Criteria('gids', '%"'.$gid.'"%', 'LIKE'));
         }
 
-        if (0 != $vcid && $GLOBALS['songlistModuleConfig']['voice']) {
+        if (0!=$vcid && $GLOBALS['songlistModuleConfig']['voice']) {
             $criteria->add(new \Criteria('vcid', $vcid));
         }
 
-        if (0 != ($_GET['cid'] ?? $cid)) {
+        if (0!=($_GET['cid'] ?? $cid)) {
             $criteria->add(new \Criteria('cid', ($_GET['cid'] ?? $cid)));
         }
 
@@ -352,10 +352,10 @@ 
         $GLOBALS['xoopsOption']['template_main'] = 'songlist_search_index.tpl';
         require $GLOBALS['xoops']->path('/header.php');
         if ($GLOBALS['songlistModuleConfig']['force_jquery'] && !isset($GLOBALS['loaded_jquery'])) {
-            $GLOBALS['xoTheme']->addScript(XOOPS_URL . _MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
+            $GLOBALS['xoTheme']->addScript(XOOPS_URL._MI_SONGLIST_JQUERY, ['type' => 'text/javascript']);
             $GLOBALS['loaded_jquery'] = true;
         }
-        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL . _MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
+        $GLOBALS['xoTheme']->addStylesheet(XOOPS_URL._MI_SONGLIST_STYLESHEET, ['type' => 'text/css']);
         $GLOBALS['xoopsTpl']->assign('xoConfig', $GLOBALS['songlistModuleConfig']);
         $GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['SCRIPT_NAME']);
         foreach ($songs as $song) {
@@ -380,5 +380,5 @@ 
                 unset($_SESSION['cid']);
                 break;
         }
-        redirect_header($_SERVER['SCRIPT_NAME'] . "?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit&cid=" . $_SESSION['cid'], 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
+        redirect_header($_SERVER['SCRIPT_NAME']."?op=item&fct=list&id=$id&value=$value&start=$start&limit=$limit&cid=".$_SESSION['cid'], 10, _MD_SONGLIST_MSG_CATEGORYCHANGED);
 }

language/english/directorychecker.php

Spacing +9 added lines, -9 removed lines

@@ -14,12 +14,12 @@
 $moduleDirName      = basename(dirname(__DIR__, 2));
 $moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_AVAILABLE', "<span style='color: green;'>Available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTAVAILABLE', "<span style='color: red;'>Not available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE', "<span style='color: red;'>Should have permission ( %d ), but it has ( %d )</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_CREATETHEDIR', 'Create it');
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM', 'Set the permission');
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRCREATED', 'The directory has been created');
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRNOTCREATED', 'The directory cannot be created');
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMSET', 'The permission has been set');
-define('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMNOTSET', 'The permission cannot be set');
+define('CO_'.$moduleDirNameUpper.'_'.'DC_AVAILABLE', "<span style='color: green;'>Available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'DC_NOTAVAILABLE', "<span style='color: red;'>Not available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'DC_NOTWRITABLE', "<span style='color: red;'>Should have permission ( %d ), but it has ( %d )</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'DC_CREATETHEDIR', 'Create it');
+define('CO_'.$moduleDirNameUpper.'_'.'DC_SETMPERM', 'Set the permission');
+define('CO_'.$moduleDirNameUpper.'_'.'DC_DIRCREATED', 'The directory has been created');
+define('CO_'.$moduleDirNameUpper.'_'.'DC_DIRNOTCREATED', 'The directory cannot be created');
+define('CO_'.$moduleDirNameUpper.'_'.'DC_PERMSET', 'The permission has been set');
+define('CO_'.$moduleDirNameUpper.'_'.'DC_PERMNOTSET', 'The permission cannot be set');

language/english/forms.php

Spacing +1 added lines, -1 removed lines

@@ -201,7 +201,7 @@
 define('_FRM_SONGLIST_FORM_IMPORT_UPLOAD_XML', 'XML File to import');
 define('_FRM_SONGLIST_FORM_IMPORT_UPLOAD_XML_DESC', 'SELECT the XML File you wish TO IMPORT FROM your SYSTEM.');
 define('_FRM_SONGLIST_FORM_IMPORT_EXISTING_XML', 'Use Existing File instead of Uploading');
-define('_FRM_SONGLIST_FORM_IMPORT_EXISTING_XML_DESC', 'Upload Existing XML file to: ' . $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']));
+define('_FRM_SONGLIST_FORM_IMPORT_EXISTING_XML_DESC', 'Upload Existing XML file to: '.$GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']));
 
 //Import Wizard B
 define('_FRM_SONGLIST_FORM_ISNEW_ELEMENTS', 'Set XML Elements to Import');