XoopsModules25x / songlist

htdocs/modules/songlist/admin/header.php

Indentation +11 added lines, -11 removed lines

@@ -56,23 +56,23 @@
 	include_once $GLOBALS['xoops']->path('class'.DS.'xoopsmailer.php');
 	
 	if ( file_exists($GLOBALS['xoops']->path('/Frameworks/moduleclasses/moduleadmin/moduleadmin.php'))){
-        include_once $GLOBALS['xoops']->path('/Frameworks/moduleclasses/moduleadmin/moduleadmin.php');
-    }else{
-        echo xoops_error("Error: You don't use the Frameworks \"admin module\". Please install this Frameworks");
-    }
+		include_once $GLOBALS['xoops']->path('/Frameworks/moduleclasses/moduleadmin/moduleadmin.php');
+	}else{
+		echo xoops_error("Error: You don't use the Frameworks \"admin module\". Please install this Frameworks");
+	}
     
 	$GLOBALS['songlistImageIcon'] = XOOPS_URL .'/'. $GLOBALS['songlistModule']->getInfo('icons16');
 	$GLOBALS['songlistImageAdmin'] = XOOPS_URL .'/'. $GLOBALS['songlistModule']->getInfo('icons32');
 	
 	if ($GLOBALS['xoopsUser']) {
-	    $moduleperm_handler = xoops_gethandler('groupperm');
-	    if (!$moduleperm_handler->checkRight('module_admin', $GLOBALS['songlistModule']->getVar( 'mid' ), $GLOBALS['xoopsUser']->getGroups())) {
-	        redirect_header(XOOPS_URL, 1, _NOPERM);
-	        exit();
-	    }
+		$moduleperm_handler = xoops_gethandler('groupperm');
+		if (!$moduleperm_handler->checkRight('module_admin', $GLOBALS['songlistModule']->getVar( 'mid' ), $GLOBALS['xoopsUser']->getGroups())) {
+			redirect_header(XOOPS_URL, 1, _NOPERM);
+			exit();
+		}
 	} else {
-	    redirect_header(XOOPS_URL . "/user.php", 1, _NOPERM);
-	    exit();
+		redirect_header(XOOPS_URL . "/user.php", 1, _NOPERM);
+		exit();
 	}
 
 	xoops_loadLanguage('user');

htdocs/modules/songlist/admin/albums.php

Indentation +10 added lines, -10 removed lines

@@ -111,25 +111,25 @@
 						if ($uploader->fetchMedia('image')) {
 						  	if (!$uploader->upload()) {
 						  		
-						    	songlist_adminMenu(1);
-						    	echo $uploader->getErrors();
+								songlist_adminMenu(1);
+								echo $uploader->getErrors();
 								songlist_footer_adminMenu();
 								xoops_cp_footer();
 								exit(0);
-					  	    } else {
+					  		} 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());
-						      	@$albums_handler->insert($albums);
+							  	$albums->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
+							  	$albums->setVar('image', $uploader->getSavedFileName());
+							  	@$albums_handler->insert($albums);
 						      	
-						    }      	
+							}      	
 					  	} else {
 					  		
 					  		songlist_adminMenu(1);
-					       	echo $uploader->getErrors();
+						   	echo $uploader->getErrors();
 							songlist_footer_adminMenu();
 							xoops_cp_footer();
 							exit(0);

htdocs/modules/songlist/admin/export.php

Indentation +9 added lines, -9 removed lines

@@ -22,8 +22,8 @@ 
 			case "actiona":	
 
 				if (isset($_SESSION['xmlfile'])) {
-			      	redirect_header($_SERVER['PHP_SELF'].'?file='.$_SESSION['xmlfile'].'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
-			      	exit;
+				  	redirect_header($_SERVER['PHP_SELF'].'?file='.$_SESSION['xmlfile'].'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
+				  	exit;
 				}
 				$indexAdmin = new ModuleAdmin();
 				echo $indexAdmin->addNavigation(basename(__FILE__));
@@ -53,18 +53,18 @@ 
 					if ($uploader->fetchMedia('xmlfile')) {
 					  	if (!$uploader->upload()) {
 					  		
-					    	echo $uploader->getErrors();
+							echo $uploader->getErrors();
 							songlist_footer_adminMenu();
 							xoops_cp_footer();
 							exit(0);
-				  	    } else {
-					      	$_SESSION['xmlfile'] = $uploader->getSavedFileName();
-					      	redirect_header($_SERVER['PHP_SELF'].'?file='.$uploader->getSavedFileName().'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
-					      	exit;
-					    }      	
+				  		} else {
+						  	$_SESSION['xmlfile'] = $uploader->getSavedFileName();
+						  	redirect_header($_SERVER['PHP_SELF'].'?file='.$uploader->getSavedFileName().'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
+						  	exit;
+						}      	
 				  	} else {
 				  		
-				       	echo $uploader->getErrors();
+					   	echo $uploader->getErrors();
 						songlist_footer_adminMenu();
 						xoops_cp_footer();
 						exit(0);

htdocs/modules/songlist/admin/category.php

Indentation +10 added lines, -10 removed lines

@@ -111,26 +111,26 @@
 						if ($uploader->fetchMedia('image')) {
 						  	if (!$uploader->upload()) {
 						  		
-						    	songlist_adminMenu(1);
-						    	echo $uploader->getErrors();
+								songlist_adminMenu(1);
+								echo $uploader->getErrors();
 								songlist_footer_adminMenu();
 								xoops_cp_footer();
 								exit(0);
-					  	    } else {
+					  		} else {
 					  	    	
-						      	if (strlen($category->getVar('image')))
-						      		unlink($GLOBALS['xoops']->path($category->getVar('path')).$category->getVar('image'));
+							  	if (strlen($category->getVar('image')))
+							  		unlink($GLOBALS['xoops']->path($category->getVar('path')).$category->getVar('image'));
 						      	
-						      	$category->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
-						      	$category->setVar('image', $uploader->getSavedFileName());
-						      	@$category_handler->insert($category);
+							  	$category->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
+							  	$category->setVar('image', $uploader->getSavedFileName());
+							  	@$category_handler->insert($category);
 						      	
-						    }      	
+							}      	
 					  	} else {
 					  		
 							$indexAdmin = new ModuleAdmin();
 							echo $indexAdmin->addNavigation(basename(__FILE__));
-					       	echo $uploader->getErrors();
+						   	echo $uploader->getErrors();
 							songlist_footer_adminMenu();
 							xoops_cp_footer();
 							exit(0);

htdocs/modules/songlist/admin/votes.php

Indentation +10 added lines, -10 removed lines

@@ -111,25 +111,25 @@
 						if ($uploader->fetchMedia('image')) {
 						  	if (!$uploader->upload()) {
 						  		
-						    	songlist_adminMenu(1);
-						    	echo $uploader->getErrors();
+								songlist_adminMenu(1);
+								echo $uploader->getErrors();
 								songlist_footer_adminMenu();
 								xoops_cp_footer();
 								exit(0);
-					  	    } else {
+					  		} else {
 					  	    	
-						      	if (strlen($votes->getVar('image')))
-						      		unlink($GLOBALS['xoops']->path($votes->getVar('path')).$votes->getVar('image'));
+							  	if (strlen($votes->getVar('image')))
+							  		unlink($GLOBALS['xoops']->path($votes->getVar('path')).$votes->getVar('image'));
 						      	
-						      	$votes->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
-						      	$votes->setVar('image', $uploader->getSavedFileName());
-						      	@$votes_handler->insert($votes);
+							  	$votes->setVar('path', $GLOBALS['songlistModuleConfig']['upload_areas']);
+							  	$votes->setVar('image', $uploader->getSavedFileName());
+							  	@$votes_handler->insert($votes);
 						      	
-						    }      	
+							}      	
 					  	} else {
 					  		
 					  		songlist_adminMenu(1);
-					       	echo $uploader->getErrors();
+						   	echo $uploader->getErrors();
 							songlist_footer_adminMenu();
 							xoops_cp_footer();
 							exit(0);

htdocs/modules/songlist/admin/import.php

Indentation +13 added lines, -13 removed lines

@@ -23,8 +23,8 @@ 
 			case "actiona":	
 
 				if (isset($_SESSION['xmlfile'])) {
-			      	redirect_header($_SERVER['PHP_SELF'].'?file='.$_SESSION['xmlfile'].'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
-			      	exit;
+				  	redirect_header($_SERVER['PHP_SELF'].'?file='.$_SESSION['xmlfile'].'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
+				  	exit;
 				}
 				$indexAdmin = new ModuleAdmin();
 				echo $indexAdmin->addNavigation(basename(__FILE__));
@@ -38,9 +38,9 @@ 
 				if ($_POST['file']!='') {
 					$file = substr(md5(microtime(true)), mt_rand(0,20), 13).'.xml';
 					copy($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']).$_POST['file'], $GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']).$file);
-				    $_SESSION['xmlfile'] = $file;
-			      	redirect_header($_SERVER['PHP_SELF'].'?file='.$file.'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_COPIED);
-			      	exit;
+					$_SESSION['xmlfile'] = $file;
+				  	redirect_header($_SERVER['PHP_SELF'].'?file='.$file.'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_COPIED);
+				  	exit;
 				} elseif (isset($_FILES['xmlfile'])&&strlen($_FILES['xmlfile']['name'])) {
 						
 					if (!is_dir($GLOBALS['xoops']->path($GLOBALS['songlistModuleConfig']['upload_areas']))) {
@@ -59,25 +59,25 @@ 
 					  	if (!$uploader->upload()) {
 							$indexAdmin = new ModuleAdmin();
 							echo $indexAdmin->addNavigation(basename(__FILE__));
-					    	echo $uploader->getErrors();
+							echo $uploader->getErrors();
 							xoops_cp_footer();
 							exit(0);
-				  	    } else {
-					      	$_SESSION['xmlfile'] = $uploader->getSavedFileName();
-					      	redirect_header($_SERVER['PHP_SELF'].'?file='.$uploader->getSavedFileName().'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
-					      	exit;
-					    }      	
+				  		} else {
+						  	$_SESSION['xmlfile'] = $uploader->getSavedFileName();
+						  	redirect_header($_SERVER['PHP_SELF'].'?file='.$uploader->getSavedFileName().'&op=import&fct=actionb', 10, _AM_SONGLIST_XMLFILE_UPLOADED);
+						  	exit;
+						}      	
 				  	} else {
 						$indexAdmin = new ModuleAdmin();
 						echo $indexAdmin->addNavigation(basename(__FILE__));
-				       	echo $uploader->getErrors();
+					   	echo $uploader->getErrors();
 						xoops_cp_footer();
 						exit(0);
 				   	}
 				} else {
 					$indexAdmin = new ModuleAdmin();
 					echo $indexAdmin->addNavigation(basename(__FILE__));
-			       	echo _AM_SONGLIST_IMPORT_NOFILE;
+				   	echo _AM_SONGLIST_IMPORT_NOFILE;
 					xoops_cp_footer();
 					exit(0);
 				}

htdocs/modules/songlist/admin/dashboard.php

Indentation +13 added lines, -13 removed lines

@@ -37,8 +37,8 @@ 
 	$op = (!empty($_GET['op']) ? $_GET['op'] : (!empty($_POST['op']) ? $_POST['op'] : "default"));
 	
 	switch ($op) {
-	    case "default":
-	    default:
+		case "default":
+		default:
     	
 			$indexAdmin = new ModuleAdmin();
 			echo $indexAdmin->addNavigation(basename(__FILE__));
@@ -54,19 +54,19 @@ 
 			$requests_handler = xoops_getmodulehandler('requests', 'songlist');
 			$votes_handler = xoops_getmodulehandler('votes', 'songlist');
 			
-		    $indexAdmin->addInfoBox(_AM_SONGLIST_COUNT);
-		    $indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_CATEGORY."</label>", $category_handler->getCount(NULL, true), 'green');
-		    $indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_ARTISTS."</label>", $artists_handler->getCount(NULL, true), 'green');
-		    $indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_ALBUMS."</label>", $albums_handler->getCount(NULL, true), 'green');
-		    $indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_GENRE."</label>", $genre_handler->getCount(NULL, true), 'green');
+			$indexAdmin->addInfoBox(_AM_SONGLIST_COUNT);
+			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_CATEGORY."</label>", $category_handler->getCount(NULL, true), 'green');
+			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_ARTISTS."</label>", $artists_handler->getCount(NULL, true), 'green');
+			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_ALBUMS."</label>", $albums_handler->getCount(NULL, true), 'green');
+			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_GENRE."</label>", $genre_handler->getCount(NULL, true), 'green');
 			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_VOICE."</label>", $voice_handler->getCount(NULL, true), 'green');
-		    $indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_SONGS."</label>", $songs_handler->getCount(NULL, true), 'green');
-		    $indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_REQUESTS."</label>", $requests_handler->getCount(NULL, true), 'green');
-		    $indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_VOTES."</label>", $votes_handler->getCount(NULL, true), 'green');
-    		echo $indexAdmin->renderIndex();
+			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_SONGS."</label>", $songs_handler->getCount(NULL, true), 'green');
+			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_REQUESTS."</label>", $requests_handler->getCount(NULL, true), 'green');
+			$indexAdmin->addInfoBoxLine(_AM_SONGLIST_COUNT, "<label>"._AM_SONGLIST_NUMBER_OF_VOTES."</label>", $votes_handler->getCount(NULL, true), 'green');
+			echo $indexAdmin->renderIndex();
 
-	        xoops_cp_footer();
-	        break;
+			xoops_cp_footer();
+			break;
 	}
 
 ?>
\ No newline at end of file

htdocs/modules/songlist/admin/permissions.php

Indentation +131 added lines, -131 removed lines

@@ -45,71 +45,71 @@ 
   
 class forum_XoopsGroupPermForm extends XoopsGroupPermForm
 {
-    function forum_XoopsGroupPermForm($title, $modid, $permname, $permdesc, $url = "")
-    {
-	    $this->XoopsGroupPermForm($title, $modid, $permname, $permdesc, $url);
-    } 
+	function forum_XoopsGroupPermForm($title, $modid, $permname, $permdesc, $url = "")
+	{
+		$this->XoopsGroupPermForm($title, $modid, $permname, $permdesc, $url);
+	} 
     
-    function render()
-    { 
-        // load all child ids for javascript codes
-        foreach (array_keys($this->_itemTree)as $item_id) {
-            $this->_itemTree[$item_id]['allchild'] = array();
-            $this->_loadAllChildItemIds($item_id, $this->_itemTree[$item_id]['allchild']);
-        }
-        $gperm_handler = xoops_gethandler('groupperm');
-        $member_handler = xoops_gethandler('member');
-        $glist = $member_handler->getGroupList();
-        foreach (array_keys($glist) as $i) {
-            // get selected item id(s) for each group
-            $selected = $gperm_handler->getItemIds($this->_permName, $i, $this->_modid);
-            $ele = new forum_XoopsGroupFormCheckBox($glist[$i], 'perms[' . $this->_permName . ']', $i, $selected);
-            $ele->setOptionTree($this->_itemTree);
-            $this->addElement($ele);
-            unset($ele);
-        } 
-        $tray = new XoopsFormElementTray('');
-        $tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
-        $tray->addElement(new XoopsFormButton('', 'reset', _CANCEL, 'reset'));
-        $this->addElement($tray);
-        $ret = '<h4>' . $this->getTitle() . '</h4>' . $this->_permDesc . '<br />';
-        $ret .= "<form name='" . $this->getName() . "' id='" . $this->getName() . "' action='" . $this->getAction() . "' method='" . $this->getMethod() . "'" . $this->getExtra() . ">\n<table width='100%' class='outer' cellspacing='1' valign='top'>\n";
-        $elements = $this->getElements();
-        $hidden = '';
-        foreach (array_keys($elements) as $i) {
-            if (!is_object($elements[$i])) {
-                $ret .= $elements[$i];
-            } elseif (!$elements[$i]->isHidden()) {
-                $ret .= "<tr valign='top' align='left'><td class='head'>" . $elements[$i]->getCaption();
-                if ($elements[$i]->getDescription() != '') {
-                    $ret .= '<br /><br /><span style="font-weight: normal;">' . $elements[$i]->getDescription() . '</span>';
-                }
-                $ret .= "</td>\n<td class='even'>\n" . $elements[$i]->render() . "\n</td></tr>\n";
-            } else {
-                $hidden .= $elements[$i]->render();
-            }
-        }
-        $ret .= "</table>$hidden</form>";
-        $ret .= $this->renderValidationJS( true );
-        return $ret;
-    }
+	function render()
+	{ 
+		// load all child ids for javascript codes
+		foreach (array_keys($this->_itemTree)as $item_id) {
+			$this->_itemTree[$item_id]['allchild'] = array();
+			$this->_loadAllChildItemIds($item_id, $this->_itemTree[$item_id]['allchild']);
+		}
+		$gperm_handler = xoops_gethandler('groupperm');
+		$member_handler = xoops_gethandler('member');
+		$glist = $member_handler->getGroupList();
+		foreach (array_keys($glist) as $i) {
+			// get selected item id(s) for each group
+			$selected = $gperm_handler->getItemIds($this->_permName, $i, $this->_modid);
+			$ele = new forum_XoopsGroupFormCheckBox($glist[$i], 'perms[' . $this->_permName . ']', $i, $selected);
+			$ele->setOptionTree($this->_itemTree);
+			$this->addElement($ele);
+			unset($ele);
+		} 
+		$tray = new XoopsFormElementTray('');
+		$tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
+		$tray->addElement(new XoopsFormButton('', 'reset', _CANCEL, 'reset'));
+		$this->addElement($tray);
+		$ret = '<h4>' . $this->getTitle() . '</h4>' . $this->_permDesc . '<br />';
+		$ret .= "<form name='" . $this->getName() . "' id='" . $this->getName() . "' action='" . $this->getAction() . "' method='" . $this->getMethod() . "'" . $this->getExtra() . ">\n<table width='100%' class='outer' cellspacing='1' valign='top'>\n";
+		$elements = $this->getElements();
+		$hidden = '';
+		foreach (array_keys($elements) as $i) {
+			if (!is_object($elements[$i])) {
+				$ret .= $elements[$i];
+			} elseif (!$elements[$i]->isHidden()) {
+				$ret .= "<tr valign='top' align='left'><td class='head'>" . $elements[$i]->getCaption();
+				if ($elements[$i]->getDescription() != '') {
+					$ret .= '<br /><br /><span style="font-weight: normal;">' . $elements[$i]->getDescription() . '</span>';
+				}
+				$ret .= "</td>\n<td class='even'>\n" . $elements[$i]->render() . "\n</td></tr>\n";
+			} else {
+				$hidden .= $elements[$i]->render();
+			}
+		}
+		$ret .= "</table>$hidden</form>";
+		$ret .= $this->renderValidationJS( true );
+		return $ret;
+	}
 }
 
 class forum_XoopsGroupFormCheckBox extends XoopsGroupFormCheckBox
 {
-    function forum_XoopsGroupFormCheckBox($caption, $name, $groupId, $values = null)
-    {
-	    $this->XoopsGroupFormCheckBox($caption, $name, $groupId, $values);
-    }
+	function forum_XoopsGroupFormCheckBox($caption, $name, $groupId, $values = null)
+	{
+		$this->XoopsGroupFormCheckBox($caption, $name, $groupId, $values);
+	}
 
-    /**
-     * Renders checkbox options for this group
-     * 
-     * @return string 
-     * @access public 
-     */
-    function render()
-    {
+	/**
+	 * Renders checkbox options for this group
+	 * 
+	 * @return string 
+	 * @access public 
+	 */
+	function render()
+	{
 		$ret = '<table class="outer"><tr><td class="odd"><table><tr>';
 		$cols = 1;
 		foreach ($this->_optionTree[0]['children'] as $topitem) {
@@ -134,38 +134,38 @@ 
 		$ret .= _ALL." <input id=\"".$checkallbtn_id."\" type=\"checkbox\" value=\"\" onclick=\"var optionids = new Array(".$option_ids_str."); xoopsCheckAllElements(optionids, '".$checkallbtn_id."');\" />";
 		$ret .= '</td></tr></table>';
 		return $ret;
-    } 
+	} 
     
-    function _renderOptionTree(&$tree, $option, $prefix, $parentIds = array())
-    {
-	    if($option['id'] > 0):
-        $tree .= $prefix . "<input type=\"checkbox\" name=\"" . $this->getName() . "[groups][" . $this->_groupId . "][" . $option['id'] . "]\" id=\"" . $this->getName() . "[groups][" . $this->_groupId . "][" . $option['id'] . "]\" onclick=\""; 
-        foreach ($parentIds as $pid) {
-	        if($pid <= 0) continue;
-            $parent_ele = $this->getName() . '[groups][' . $this->_groupId . '][' . $pid . ']';
-            $tree .= "var ele = xoopsGetElementById('" . $parent_ele . "'); if(ele.checked != true) {ele.checked = this.checked;}";
-        } 
-        foreach ($option['allchild'] as $cid) {
-            $child_ele = $this->getName() . '[groups][' . $this->_groupId . '][' . $cid . ']';
-            $tree .= "var ele = xoopsGetElementById('" . $child_ele . "'); if(this.checked != true) {ele.checked = false;}";
-        } 
-        $tree .= '" value="1"';
-        if (in_array($option['id'], $this->_value)) {
-            $tree .= ' checked="checked"';
-        } 
-        $tree .= " />" . $option['name'] . "<input type=\"hidden\" name=\"" . $this->getName() . "[parents][" . $option['id'] . "]\" value=\"" . implode(':', $parentIds). "\" /><input type=\"hidden\" name=\"" . $this->getName() . "[itemname][" . $option['id'] . "]\" value=\"" . htmlspecialchars($option['name']). "\" /><br />\n";
-        else:
-        $tree .= $prefix . $option['name'] . "<input type=\"hidden\" id=\"" . $this->getName() . "[groups][" . $this->_groupId . "][" . $option['id'] . "]\" /><br />\n";
-        endif;
-        if (isset($option['children'])) {
-            foreach ($option['children'] as $child) {
-	            if($option['id'] > 0){
-	                array_push($parentIds, $option['id']);
-                }
-                $this->_renderOptionTree($tree, $this->_optionTree[$child], $prefix . '&nbsp;-', $parentIds);
-            }
-        }
-    }
+	function _renderOptionTree(&$tree, $option, $prefix, $parentIds = array())
+	{
+		if($option['id'] > 0):
+		$tree .= $prefix . "<input type=\"checkbox\" name=\"" . $this->getName() . "[groups][" . $this->_groupId . "][" . $option['id'] . "]\" id=\"" . $this->getName() . "[groups][" . $this->_groupId . "][" . $option['id'] . "]\" onclick=\""; 
+		foreach ($parentIds as $pid) {
+			if($pid <= 0) continue;
+			$parent_ele = $this->getName() . '[groups][' . $this->_groupId . '][' . $pid . ']';
+			$tree .= "var ele = xoopsGetElementById('" . $parent_ele . "'); if(ele.checked != true) {ele.checked = this.checked;}";
+		} 
+		foreach ($option['allchild'] as $cid) {
+			$child_ele = $this->getName() . '[groups][' . $this->_groupId . '][' . $cid . ']';
+			$tree .= "var ele = xoopsGetElementById('" . $child_ele . "'); if(this.checked != true) {ele.checked = false;}";
+		} 
+		$tree .= '" value="1"';
+		if (in_array($option['id'], $this->_value)) {
+			$tree .= ' checked="checked"';
+		} 
+		$tree .= " />" . $option['name'] . "<input type=\"hidden\" name=\"" . $this->getName() . "[parents][" . $option['id'] . "]\" value=\"" . implode(':', $parentIds). "\" /><input type=\"hidden\" name=\"" . $this->getName() . "[itemname][" . $option['id'] . "]\" value=\"" . htmlspecialchars($option['name']). "\" /><br />\n";
+		else:
+		$tree .= $prefix . $option['name'] . "<input type=\"hidden\" id=\"" . $this->getName() . "[groups][" . $this->_groupId . "][" . $option['id'] . "]\" /><br />\n";
+		endif;
+		if (isset($option['children'])) {
+			foreach ($option['children'] as $child) {
+				if($option['id'] > 0){
+					array_push($parentIds, $option['id']);
+				}
+				$this->_renderOptionTree($tree, $this->_optionTree[$child], $prefix . '&nbsp;-', $parentIds);
+			}
+		}
+	}
 }
 
 xoops_cp_header();
@@ -190,14 +190,14 @@ 
 		$opform->addElement($op_select);
 		$opform->display();
 		
-        $member_handler = xoops_gethandler('member');
-        $glist = $member_handler->getGroupList();
-        $elements = array();
-        $songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
-        $perm_template = $songlistperm_handler->getTemplate($groupid = 0);
-        foreach (array_keys($glist) as $i) {
-            $selected = !empty($perm_template[$i]) ? array_keys($perm_template[$i]) : array();
-            $ret_ele  = '<tr align="left" valign="top"><td class="head">'.$glist[$i].'</td>';
+		$member_handler = xoops_gethandler('member');
+		$glist = $member_handler->getGroupList();
+		$elements = array();
+		$songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
+		$perm_template = $songlistperm_handler->getTemplate($groupid = 0);
+		foreach (array_keys($glist) as $i) {
+			$selected = !empty($perm_template[$i]) ? array_keys($perm_template[$i]) : array();
+			$ret_ele  = '<tr align="left" valign="top"><td class="head">'.$glist[$i].'</td>';
 			$ret_ele .= '<td class="even">';
 			$ret_ele .= '<table class="outer"><tr><td class="odd"><table><tr>';
 			$ii = 0;
@@ -216,37 +216,37 @@ 
 			$ret_ele .= _ALL.' <input id="checkall['.$i.']" type="checkbox" value="" onclick="var optionids = new Array('.implode(", ", $option_ids).'); xoopsCheckAllElements(optionids, \'checkall['.$i.']\')" />';
 			$ret_ele .= '</td></tr></table>';
 			$ret_ele .= '</td></tr>';
-            $elements[] = $ret_ele;
-        }
-        $tray = new XoopsFormElementTray('');
-        $tray->addElement(new XoopsFormHidden('action', 'template_save'));
-        $tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
-        $tray->addElement(new XoopsFormButton('', 'reset', _CANCEL, 'reset'));
-     	$ret = '<h4>' . _AM_SONGLIST_PERM_TEMPLATE . '</h4>' . _AM_SONGLIST_PERM_TEMPLATE_DESC . '<br /><br /><br />';
-        $ret .= "<form name='template' id='template' method='post'>\n<table width='100%' class='outer' cellspacing='1'>\n";
-        $ret .= implode("\n",$elements);
+			$elements[] = $ret_ele;
+		}
+		$tray = new XoopsFormElementTray('');
+		$tray->addElement(new XoopsFormHidden('action', 'template_save'));
+		$tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
+		$tray->addElement(new XoopsFormButton('', 'reset', _CANCEL, 'reset'));
+	 	$ret = '<h4>' . _AM_SONGLIST_PERM_TEMPLATE . '</h4>' . _AM_SONGLIST_PERM_TEMPLATE_DESC . '<br /><br /><br />';
+		$ret .= "<form name='template' id='template' method='post'>\n<table width='100%' class='outer' cellspacing='1'>\n";
+		$ret .= implode("\n",$elements);
 		$ret .= '<tr align="left" valign="top"><td class="head"></td><td class="even">';
-        $ret .= $tray->render();
+		$ret .= $tray->render();
 		$ret .= '</td></tr>';
-        $ret .= '</table></form>';
-        echo $ret;
-        break;	
+		$ret .= '</table></form>';
+		echo $ret;
+		break;	
         
 	case "template_save":
-        $songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
-        $res = $songlistperm_handler->setTemplate($_POST['perms'], $groupid = 0);
-        if($res){
-	    	redirect_header("permissions.php?action=template", 2, _AM_SONGLIST_PERM_TEMPLATE_CREATED);
-        }else{
-	    	redirect_header("permissions.php?action=template", 2, _AM_SONGLIST_PERM_TEMPLATE_ERROR);
-        }
+		$songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
+		$res = $songlistperm_handler->setTemplate($_POST['perms'], $groupid = 0);
+		if($res){
+			redirect_header("permissions.php?action=template", 2, _AM_SONGLIST_PERM_TEMPLATE_CREATED);
+		}else{
+			redirect_header("permissions.php?action=template", 2, _AM_SONGLIST_PERM_TEMPLATE_ERROR);
+		}
 		break;
 		
 	case "apply":
-        $songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
-	    $perm_template = $songlistperm_handler->getTemplate();
+		$songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
+		$perm_template = $songlistperm_handler->getTemplate();
 		if($perm_template===null){
-	    	redirect_header("permissions.php?action=template", 2, _AM_SONGLIST_PERM_TEMPLATE);
+			redirect_header("permissions.php?action=template", 2, _AM_SONGLIST_PERM_TEMPLATE);
 		}
 		
 		$opform = new XoopsSimpleForm(_AM_SONGLIST_PERM_ACTION, 'actionform', 'permissions.php', "get");
@@ -266,7 +266,7 @@ 
 			$fm_options[-1*$c] = "[".$categories[$c]->getVar('cat_title')."]";
 			foreach(array_keys($songlists[$c]) as $f){
 				$fm_options[$f] = $songlists[$c][$f]["title"];
-		        if(!isset($songlists[$c][$f]["sub"])) continue;
+				if(!isset($songlists[$c][$f]["sub"])) continue;
 				foreach(array_keys($songlists[$c][$f]["sub"]) as $s){
 					$fm_options[$s] = "-- ".$songlists[$c][$f]["sub"][$s]["title"];
 				}
@@ -277,22 +277,22 @@ 
 		$fm_select = new XoopsFormSelect(_AM_SONGLIST_PERM_FORUMS, 'forums', null, 10, true);
 		$fm_select->addOptionArray($fm_options);
 		$fmform->addElement($fm_select);
-        $tray = new XoopsFormElementTray('');
-        $tray->addElement(new XoopsFormHidden('action', 'apply_save'));
-        $tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
-        $tray->addElement(new XoopsFormButton('', 'reset', _CANCEL, 'reset'));
+		$tray = new XoopsFormElementTray('');
+		$tray->addElement(new XoopsFormHidden('action', 'apply_save'));
+		$tray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
+		$tray->addElement(new XoopsFormButton('', 'reset', _CANCEL, 'reset'));
 		$fmform->addElement($tray);
 		$fmform->display();
 		break;
 		
 	case "apply_save":
 		if(empty($_POST["forums"])) break;
-	    $songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
+		$songlistperm_handler = xoops_getmodulehandler('permission', 'songlist');
 		foreach($_POST["forums"] as $songlist){
 			if($songlist < 1) continue;
 			$songlistperm_handler->applyTemplate($songlist, $module_id);
 		}
-	    redirect_header("permissions.php", 2, _AM_SONGLIST_PERM_TEMPLATE_APPLIED);
+		redirect_header("permissions.php", 2, _AM_SONGLIST_PERM_TEMPLATE_APPLIED);
 		break;
 		
 	default:
@@ -353,10 +353,10 @@ 
 				$key_c = -1 * $c;
 				$form->addItem($key_c, "<strong>[".$categories[$c]->getVar('cat_title')."]</strong>");
 				foreach(array_keys($songlists[$c]) as $f){
-			        $form->addItem($f, $songlists[$c][$f]["title"], $key_c);
-			        if(!isset($songlists[$c][$f]["sub"])) continue;
+					$form->addItem($f, $songlists[$c][$f]["title"], $key_c);
+					if(!isset($songlists[$c][$f]["sub"])) continue;
 					foreach(array_keys($songlists[$c][$f]["sub"]) as $s){
-			        	$form->addItem($s, "&rarr;".$songlists[$c][$f]["sub"][$s]["title"], $f);
+						$form->addItem($s, "&rarr;".$songlists[$c][$f]["sub"][$s]["title"], $f);
 					}
 				}
 			}

htdocs/modules/songlist/include/formselectalbum.php

Indentation +191 added lines, -191 removed lines

@@ -54,59 +54,59 @@ 
  */
 class SonglistFormSelectAlbum extends XoopsFormElement
 {
-    /**
-     * Options
-     *
-     * @var array
-     * @access private
-     */
-    var $_options = array();
+	/**
+	 * Options
+	 *
+	 * @var array
+	 * @access private
+	 */
+	var $_options = array();
 
-    /**
-     * Allow multiple selections?
-     *
-     * @var bool
-     * @access private
-     */
-    var $_multiple = false;
+	/**
+	 * Allow multiple selections?
+	 *
+	 * @var bool
+	 * @access private
+	 */
+	var $_multiple = false;
 
-    /**
-     * Number of rows. "1" makes a dropdown list.
-     *
-     * @var int
-     * @access private
-     */
-    var $_size;
+	/**
+	 * Number of rows. "1" makes a dropdown list.
+	 *
+	 * @var int
+	 * @access private
+	 */
+	var $_size;
 
-    /**
-     * Pre-selcted values
-     *
-     * @var array
-     * @access private
-     */
-    var $_value = array();
+	/**
+	 * Pre-selcted values
+	 *
+	 * @var array
+	 * @access private
+	 */
+	var $_value = array();
 
-    /**
-     * Constructor
-     *
-     * @param string $caption Caption
-     * @param string $name "name" attribute
-     * @param mixed $value Pre-selected value (or array of them).
-     * @param int $size Number or rows. "1" makes a drop-down-list
-     * @param bool $multiple Allow multiple selections?
-     */
-    function SonglistFormSelectAlbum($caption, $name, $value = null, $size = 1, $multiple = false, $ownid=0)
-    {
-    	global $_form_object_options;
-    	xoops_loadLanguage('modinfo', 'songlist');
+	/**
+	 * Constructor
+	 *
+	 * @param string $caption Caption
+	 * @param string $name "name" attribute
+	 * @param mixed $value Pre-selected value (or array of them).
+	 * @param int $size Number or rows. "1" makes a drop-down-list
+	 * @param bool $multiple Allow multiple selections?
+	 */
+	function SonglistFormSelectAlbum($caption, $name, $value = null, $size = 1, $multiple = false, $ownid=0)
+	{
+		global $_form_object_options;
+		xoops_loadLanguage('modinfo', 'songlist');
     	
-        $this->setCaption($caption);
-        $this->setName($name);
-        $this->_multiple = $multiple;
-        $this->_size = intval($size);
-        if (isset($value)) {
-            $this->setValue($value);
-        }
+		$this->setCaption($caption);
+		$this->setName($name);
+		$this->_multiple = $multiple;
+		$this->_size = intval($size);
+		if (isset($value)) {
+			$this->setValue($value);
+		}
 		$this->addOption('0', _MI_SONGLIST_NONE);
 		if (!isset($_form_object_options['albums'])) {
 			$albums_handler =& xoops_getmodulehandler('albums', 'songlist');
@@ -121,159 +121,159 @@ 
 			foreach($_form_object_options['albums'] as $id => $value)
 				$this->addOption($id, $value);
 		}
-    }
+	}
 
-    /**
-     * Are multiple selections allowed?
-     *
-     * @return bool
-     */
-    function isMultiple()
-    {
-        return $this->_multiple;
-    }
+	/**
+	 * Are multiple selections allowed?
+	 *
+	 * @return bool
+	 */
+	function isMultiple()
+	{
+		return $this->_multiple;
+	}
 
-    /**
-     * Get the size
-     *
-     * @return int
-     */
-    function getSize()
-    {
-        return $this->_size;
-    }
+	/**
+	 * Get the size
+	 *
+	 * @return int
+	 */
+	function getSize()
+	{
+		return $this->_size;
+	}
 
-    /**
-     * Get an array of pre-selected values
-     *
-     * @param bool $encode To sanitizer the text?
-     * @return array
-     */
-    function getValue($encode = false)
-    {
-        if (! $encode) {
-            return $this->_value;
-        }
-        $value = array();
-        foreach($this->_value as $val) {
-            $value[] = $val ? htmlspecialchars($val, ENT_QUOTES) : $val;
-        }
-        return $value;
-    }
+	/**
+	 * Get an array of pre-selected values
+	 *
+	 * @param bool $encode To sanitizer the text?
+	 * @return array
+	 */
+	function getValue($encode = false)
+	{
+		if (! $encode) {
+			return $this->_value;
+		}
+		$value = array();
+		foreach($this->_value as $val) {
+			$value[] = $val ? htmlspecialchars($val, ENT_QUOTES) : $val;
+		}
+		return $value;
+	}
 
-    /**
-     * Set pre-selected values
-     *
-     * @param  $value mixed
-     */
-    function setValue($value)
-    {
-        if (is_array($value)) {
-            foreach($value as $v) {
-                $this->_value[] = $v;
-            }
-        } elseif (isset($value)) {
-            $this->_value[] = $value;
-        }
-    }
+	/**
+	 * Set pre-selected values
+	 *
+	 * @param  $value mixed
+	 */
+	function setValue($value)
+	{
+		if (is_array($value)) {
+			foreach($value as $v) {
+				$this->_value[] = $v;
+			}
+		} elseif (isset($value)) {
+			$this->_value[] = $value;
+		}
+	}
 
-    /**
-     * Add an option
-     *
-     * @param string $value "value" attribute
-     * @param string $name "name" attribute
-     */
-    function addOption($value, $name = '')
-    {
-        if ($name != '') {
-            $this->_options[$value] = $name;
-        } else {
-            $this->_options[$value] = $value;
-        }
-    }
+	/**
+	 * Add an option
+	 *
+	 * @param string $value "value" attribute
+	 * @param string $name "name" attribute
+	 */
+	function addOption($value, $name = '')
+	{
+		if ($name != '') {
+			$this->_options[$value] = $name;
+		} else {
+			$this->_options[$value] = $value;
+		}
+	}
 
-    /**
-     * Add multiple options
-     *
-     * @param array $options Associative array of value->name pairs
-     */
-    function addOptionArray($options)
-    {
-        if (is_array($options)) {
-            foreach($options as $k => $v) {
-                $this->addOption($k, $v);
-            }
-        }
-    }
+	/**
+	 * Add multiple options
+	 *
+	 * @param array $options Associative array of value->name pairs
+	 */
+	function addOptionArray($options)
+	{
+		if (is_array($options)) {
+			foreach($options as $k => $v) {
+				$this->addOption($k, $v);
+			}
+		}
+	}
 
-    /**
-     * Get an array with all the options
-     *
-     * Note: both name and value should be sanitized. However for backward compatibility, only value is sanitized for now.
-     *
-     * @param int $encode To sanitizer the text? potential values: 0 - skip; 1 - only for value; 2 - for both value and name
-     * @return array Associative array of value->name pairs
-     */
-    function getOptions($encode = false)
-    {
-        if (! $encode) {
-            return $this->_options;
-        }
-        $value = array();
-        foreach($this->_options as $val => $name) {
-            $value[$encode ? htmlspecialchars($val, ENT_QUOTES) : $val] = ($encode > 1) ? htmlspecialchars($name, ENT_QUOTES) : $name;
-        }
-        return $value;
-    }
+	/**
+	 * Get an array with all the options
+	 *
+	 * Note: both name and value should be sanitized. However for backward compatibility, only value is sanitized for now.
+	 *
+	 * @param int $encode To sanitizer the text? potential values: 0 - skip; 1 - only for value; 2 - for both value and name
+	 * @return array Associative array of value->name pairs
+	 */
+	function getOptions($encode = false)
+	{
+		if (! $encode) {
+			return $this->_options;
+		}
+		$value = array();
+		foreach($this->_options as $val => $name) {
+			$value[$encode ? htmlspecialchars($val, ENT_QUOTES) : $val] = ($encode > 1) ? htmlspecialchars($name, ENT_QUOTES) : $name;
+		}
+		return $value;
+	}
 
-    /**
-     * Prepare HTML for output
-     *
-     * @return string HTML
-     */
-    function render()
-    {
-        $ele_name = $this->getName();
+	/**
+	 * Prepare HTML for output
+	 *
+	 * @return string HTML
+	 */
+	function render()
+	{
+		$ele_name = $this->getName();
 		$ele_title = $this->getTitle();
-        $ele_value = $this->getValue();
-        $ele_options = $this->getOptions();
-        $ret = '<select size="'.$this->getSize().'"'.$this->getExtra();
-        if ($this->isMultiple() != false) {
-            $ret .= ' name="'.$ele_name.'[]" id="'.$ele_name.'" title="'. $ele_title. '" multiple="multiple">' ;
-        } else {
-            $ret .= ' name="'.$ele_name.'" id="'.$ele_name.'" title="'. $ele_title. '">' ;
-        }
-        foreach($ele_options as $value => $name) {
-            $ret .= '<option value="'.htmlspecialchars($value, ENT_QUOTES).'"';
-            if (count($ele_value) > 0 && in_array($value, $ele_value)) {
-                $ret .= ' selected="selected"';
-            }
-            $ret .= '>'.$name.'</option>' ;
-        }
-        $ret .= '</select>';
-        return $ret;
-    }
+		$ele_value = $this->getValue();
+		$ele_options = $this->getOptions();
+		$ret = '<select size="'.$this->getSize().'"'.$this->getExtra();
+		if ($this->isMultiple() != false) {
+			$ret .= ' name="'.$ele_name.'[]" id="'.$ele_name.'" title="'. $ele_title. '" multiple="multiple">' ;
+		} else {
+			$ret .= ' name="'.$ele_name.'" id="'.$ele_name.'" title="'. $ele_title. '">' ;
+		}
+		foreach($ele_options as $value => $name) {
+			$ret .= '<option value="'.htmlspecialchars($value, ENT_QUOTES).'"';
+			if (count($ele_value) > 0 && in_array($value, $ele_value)) {
+				$ret .= ' selected="selected"';
+			}
+			$ret .= '>'.$name.'</option>' ;
+		}
+		$ret .= '</select>';
+		return $ret;
+	}
 
-    /**
-     * Render custom javascript validation code
-     *
-     * @seealso XoopsForm::renderValidationJS
-     */
-    function renderValidationJS()
-    {
-        // render custom validation code if any
-        if (! empty($this->customValidationCode)) {
-            return implode("\n", $this->customValidationCode);
-            // generate validation code if required
-        } elseif ($this->isRequired()) {
-            $eltname = $this->getName();
-            $eltcaption = $this->getCaption();
-            $eltmsg = empty($eltcaption) ? sprintf(_FORM_ENTER, $eltname) : sprintf(_FORM_ENTER, $eltcaption);
-            $eltmsg = str_replace('"', '\"', stripslashes($eltmsg));
-            return "\nvar hasSelected = false; var selectBox = myform.{$eltname};"."for (i = 0; i < selectBox.options.length; i++ ) { if (selectBox.options[i].selected == true) { hasSelected = true; break; } }"."if (!hasSelected) { window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }";
-        }
-        return '';
-    }
+	/**
+	 * Render custom javascript validation code
+	 *
+	 * @seealso XoopsForm::renderValidationJS
+	 */
+	function renderValidationJS()
+	{
+		// render custom validation code if any
+		if (! empty($this->customValidationCode)) {
+			return implode("\n", $this->customValidationCode);
+			// generate validation code if required
+		} elseif ($this->isRequired()) {
+			$eltname = $this->getName();
+			$eltcaption = $this->getCaption();
+			$eltmsg = empty($eltcaption) ? sprintf(_FORM_ENTER, $eltname) : sprintf(_FORM_ENTER, $eltcaption);
+			$eltmsg = str_replace('"', '\"', stripslashes($eltmsg));
+			return "\nvar hasSelected = false; var selectBox = myform.{$eltname};"."for (i = 0; i < selectBox.options.length; i++ ) { if (selectBox.options[i].selected == true) { hasSelected = true; break; } }"."if (!hasSelected) { window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }";
+		}
+		return '';
+	}
 }
 
 ?>
\ No newline at end of file