XoopsModules25x / songlist

htdocs/modules/songlist/admin/field_permissions.php

Braces +6 added lines, -4 removed lines

@@ -79,10 +79,11 @@ 
 	$fields = array_merge(array(), $extras_handler->loadFields());
 	
 	if ( $op != "search" ) {
-		if (is_array($fields)&&count($fields)>0)
-			foreach (array_keys($fields) as $i ) {
+		if (is_array($fields)&&count($fields)>0) {
+					foreach (array_keys($fields) as $i ) {
 				if ( $restriction == "" || $fields[$i]->getVar($restriction)  ) {
 					$form->addItem($fields[$i]->getVar('field_id'), xoops_substr($fields[$i]->getVar('field_title'), 0, 25) );
+		}
 				}
 			}
 	} else {
@@ -94,10 +95,11 @@ 
 		'datetime',
 		'timezone',
 		'language');
-		if (is_array($fields)&&count($fields)>0)
-			foreach (array_keys($fields) as $i ) {
+		if (is_array($fields)&&count($fields)>0) {
+					foreach (array_keys($fields) as $i ) {
 				if ( in_array($fields[$i]->getVar('field_type'), $searchable_types)  ) {
 					$form->addItem($fields[$i]->getVar('field_id'), xoops_substr($fields[$i]->getVar('field_title'), 0, 25) );
+		}
 				}
 			}
 	}

htdocs/modules/songlist/admin/genre.php

Braces +8 added lines, -6 removed lines

@@ -52,8 +52,9 @@ 
 					
 				$genres = $genre_handler->getObjects($criteria, true);
 				foreach($genres as $cid => $genre) {
-					if (is_object($genre))					
-						$GLOBALS['xoopsTpl']->append('genre', $genre->toArray());
+					if (is_object($genre)) {
+											$GLOBALS['xoopsTpl']->append('genre', $genre->toArray());
+					}
 				}
 				$GLOBALS['xoopsTpl']->assign('form', songlist_genre_get_form(false));
 				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['PHP_SELF']);
@@ -92,10 +93,11 @@ 
 					redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_GENRE_FAILEDTOSAVE);
 					exit(0);
 				} else {
-					if ($_REQUEST['state'][$_REQUEST['id']]=='new')
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_GENRE_SAVEDOKEY);
-					else 
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_GENRE_SAVEDOKEY);
+					if ($_REQUEST['state'][$_REQUEST['id']]=='new') {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_GENRE_SAVEDOKEY);
+					} else {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_GENRE_SAVEDOKEY);
+					}
 					exit(0);
 				}
 				break;

htdocs/modules/songlist/admin/header.php

Braces +7 added lines, -5 removed lines

@@ -32,10 +32,12 @@ 
 	
 	require_once (dirname(dirname(dirname(dirname(__FILE__)))).'/include/cp_header.php');
 	
-	if (!defined('_CHARSET'))
-		define ("_CHARSET","UTF-8");
-	if (!defined('_CHARSET_ISO'))
-		define ("_CHARSET_ISO","ISO-8859-1");
+	if (!defined('_CHARSET')) {
+			define ("_CHARSET","UTF-8");
+	}
+	if (!defined('_CHARSET_ISO')) {
+			define ("_CHARSET_ISO","ISO-8859-1");
+	}
 	
 	$GLOBALS['songlistAdmin'] = true;
 	
@@ -57,7 +59,7 @@ 
 	
 	if ( file_exists($GLOBALS['xoops']->path('/Frameworks/moduleclasses/moduleadmin/moduleadmin.php'))){
         include_once $GLOBALS['xoops']->path('/Frameworks/moduleclasses/moduleadmin/moduleadmin.php');
-    }else{
+    } else{
         echo xoops_error("Error: You don't use the Frameworks \"admin module\". Please install this Frameworks");
     }
     

htdocs/modules/songlist/admin/albums.php

Braces +14 added lines, -10 removed lines

@@ -52,8 +52,9 @@ 
 					
 				$albumss = $albums_handler->getObjects($criteria, true);
 				foreach($albumss as $cid => $albums) {
-					if (is_object($albums))					
-						$GLOBALS['xoopsTpl']->append('albums', $albums->toArray());
+					if (is_object($albums)) {
+											$GLOBALS['xoopsTpl']->append('albums', $albums->toArray());
+					}
 				}
 				$GLOBALS['xoopsTpl']->assign('form', songlist_albums_get_form(false));
 				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['PHP_SELF']);
@@ -96,9 +97,10 @@ 
 					if (isset($_FILES['image'])&&!empty($_FILES['image']['title'])) {
 						
 						if (!is_dir($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']))) {
-							foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders)
-								foreach(explode('/', $folders) as $folder) {
+							foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders) {
+															foreach(explode('/', $folders) as $folder) {
 									$path .= DS . $folder;
+							}
 									mkdir($path, 0777);
 								}
 						}
@@ -118,8 +120,9 @@ 
 								exit(0);
 					  	    } else {
 					  	    	
-						      	if (strlen($albums->getVar('image')))
-						      		unlink($GLOBALS['xoops']->path($albums->getVar('path')).$albums->getVar('image'));
+						      	if (strlen($albums->getVar('image'))) {
+						      							      		unlink($GLOBALS['xoops']->path($albums->getVar('path')).$albums->getVar('image'));
+						      	}
 						      	
 						      	$albums->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
 						      	$albums->setVar('image', $uploader->getSavedFileName());
@@ -136,10 +139,11 @@ 
 					   	}
 					}
 					
-					if ($_REQUEST['state'][$_REQUEST['id']]=='new')
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
-					else 
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
+					if ($_REQUEST['state'][$_REQUEST['id']]=='new') {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
+					} else {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_ALBUMS_SAVEDOKEY);
+					}
 					exit(0);
 				}
 				break;

htdocs/modules/songlist/admin/export.php

Braces +3 added lines, -2 removed lines

@@ -38,9 +38,10 @@
 				if (isset($_FILES['xmlfile'])&&!empty($_FILES['xmlfile']['title'])) {
 						
 					if (!is_dir($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']))) {
-						foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders)
-							foreach(explode('/', $folders) as $folder) {
+						foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders) {
+													foreach(explode('/', $folders) as $folder) {
 								$path .= DS . $folder;
+						}
 								mkdir($path, 0777);
 							}
 					}

htdocs/modules/songlist/admin/songs.php

Braces +20 added lines, -14 removed lines

@@ -52,8 +52,9 @@ 
 					
 				$songss = $songs_handler->getObjects($criteria, true);
 				foreach($songss as $cid => $songs) {
-					if (is_object($songs))					
-						$GLOBALS['xoopsTpl']->append('songs', $songs->toArray());
+					if (is_object($songs)) {
+											$GLOBALS['xoopsTpl']->append('songs', $songs->toArray());
+					}
 				}
 				$GLOBALS['xoopsTpl']->assign('form', songlist_songs_get_form(false));
 				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['PHP_SELF']);
@@ -94,9 +95,10 @@ 
 				if (isset($_FILES['mp3'.$id])&&!empty($_FILES['mp3'.$id]['title'])) {
 						
 					if (!is_dir($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']))) {
-						foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders)
-							foreach(explode('/', $folders) as $folder) {
+						foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders) {
+													foreach(explode('/', $folders) as $folder) {
 							$path .= DS . $folder;
+						}
 							mkdir($path, 0777);
 						}
 					}
@@ -115,8 +117,9 @@ 
 							exit(0);
 						} else {
 								
-							if (strlen($songs->getVar('mp3')))
-								unlink($GLOBALS['xoops']->path($songs->getVar('path')).basename($songs->getVar('mp3')));
+							if (strlen($songs->getVar('mp3'))) {
+															unlink($GLOBALS['xoops']->path($songs->getVar('path')).basename($songs->getVar('mp3')));
+							}
 								
 							$songs->setVar('mp3', XOOPS_URL.'/'.str_replace(DS, '/', $GLOBALS['songlistModuleConfig']['upload_areas']).$uploader->getSavedFileName());
 								
@@ -145,10 +148,11 @@ 
 					}
 					
 					
-					if ($_REQUEST['state'][$_REQUEST['id']]=='new')
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_SONGS_SAVEDOKEY);
-					else 
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_SONGS_SAVEDOKEY);
+					if ($_REQUEST['state'][$_REQUEST['id']]=='new') {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_SONGS_SAVEDOKEY);
+					} else {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_SONGS_SAVEDOKEY);
+					}
 					exit(0);
 				}
 				break;
@@ -162,9 +166,10 @@ 
 					if (isset($_FILES['mp3'.$id])&&!empty($_FILES['mp3'.$id]['title'])) {
 					
 						if (!is_dir($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']))) {
-							foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders)
-								foreach(explode('/', $folders) as $folder) {
+							foreach(explode('\\', $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas'])) as $folders) {
+															foreach(explode('/', $folders) as $folder) {
 								$path .= DS . $folder;
+							}
 								mkdir($path, 0777);
 							}
 						}
@@ -183,8 +188,9 @@ 
 								exit(0);
 							} else {
 					
-								if (strlen($songs->getVar('mp3')))
-									unlink($GLOBALS['xoops']->path($songs->getVar('path')).basename($songs->getVar('mp3')));
+								if (strlen($songs->getVar('mp3'))) {
+																	unlink($GLOBALS['xoops']->path($songs->getVar('path')).basename($songs->getVar('mp3')));
+								}
 					
 								$songs->setVar('mp3', XOOPS_URL.'/'.str_replace(DS, '/', $GLOBALS['songlistModuleConfig']['upload_areas']).$uploader->getSavedFileName());
 					

htdocs/modules/songlist/admin/field.php

Braces +3 added lines, -1 removed lines

@@ -183,7 +183,9 @@
 		
 		if ( !empty($_REQUEST['addOption'])  ) {
 			foreach ($_REQUEST['addOption'] as $option ) {
-				if ( empty($option['value']) ) continue;
+				if ( empty($option['value']) ) {
+					continue;
+				}
 				$options[$option['key']] = $option['value'];
 				$redirect_to_edit = true;
 			}

htdocs/modules/songlist/admin/voice.php

Braces +8 added lines, -6 removed lines

@@ -52,8 +52,9 @@ 
 					
 				$voices = $voice_handler->getObjects($criteria, true);
 				foreach($voices as $cid => $voice) {
-					if (is_object($voice))					
-						$GLOBALS['xoopsTpl']->append('voice', $voice->toArray());
+					if (is_object($voice)) {
+											$GLOBALS['xoopsTpl']->append('voice', $voice->toArray());
+					}
 				}
 				$GLOBALS['xoopsTpl']->assign('form', songlist_voice_get_form(false));
 				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['PHP_SELF']);
@@ -92,10 +93,11 @@ 
 					redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOICE_FAILEDTOSAVE);
 					exit(0);
 				} else {
-					if ($_REQUEST['state'][$_REQUEST['id']]=='new')
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOICE_SAVEDOKEY);
-					else 
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOICE_SAVEDOKEY);
+					if ($_REQUEST['state'][$_REQUEST['id']]=='new') {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOICE_SAVEDOKEY);
+					} else {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_VOICE_SAVEDOKEY);
+					}
 					exit(0);
 				}
 				break;

htdocs/modules/songlist/admin/requests.php

Braces +8 added lines, -6 removed lines

@@ -52,8 +52,9 @@ 
 					
 				$requestss = $requests_handler->getObjects($criteria, true);
 				foreach($requestss as $cid => $requests) {
-					if (is_object($requests))					
-						$GLOBALS['xoopsTpl']->append('requests', $requests->toArray());
+					if (is_object($requests)) {
+											$GLOBALS['xoopsTpl']->append('requests', $requests->toArray());
+					}
 				}
 				$GLOBALS['xoopsTpl']->assign('form', songlist_requests_get_form(false));
 				$GLOBALS['xoopsTpl']->assign('php_self', $_SERVER['PHP_SELF']);
@@ -92,10 +93,11 @@ 
 					redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_FAILEDTOSAVE);
 					exit(0);
 				} else {
-					if ($_REQUEST['state'][$_REQUEST['id']]=='new')
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
-					else 
-						redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
+					if ($_REQUEST['state'][$_REQUEST['id']]=='new') {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=edit&id='.$_REQUEST['id'] . '&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
+					} else {
+											redirect_header($_SERVER['PHP_SELF'].'?op='.$GLOBALS['op'].'&fct=list&limit='.$GLOBALS['limit'].'&start='.$GLOBALS['start'].'&order='.$GLOBALS['order'].'&sort='.$GLOBALS['sort'].'&filter='.$GLOBALS['filter'], 10, _AM_SONGLIST_MSG_REQUESTS_SAVEDOKEY);
+					}
 					exit(0);
 				}
 				break;