txmodxoops / tdmcreate

class/files/user/UserXoopsVersion.php

Spacing +1 added lines, -1 removed lines

@@ -1022,7 +1022,7 @@
      */
     private function getXoopsVersionNotificationCategory($language, $type, $name, $title, $from, $item, $allow)
     {
-        $title = strtoupper($title);//{$from}
+        $title = strtoupper($title); //{$from}
         $ret = "
 \$modversion['notification']['{$type}'][] = array(
     'name' => '{$name}',

class/files/admin/AdminIndex.php

Doc Comments +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     /**
      * @param $module
      * @param $tables
-     * @param $filename
+     * @param string $filename
      */
     public function write($module, $tables, $filename)
     {
@@ -146,7 +146,7 @@ 
     *  @param null
     */
     /**
-     * @return bool|string
+     * @return null|string
      */
     public function render()
     {

class/files/admin/AdminMenu.php

Doc Comments +4 added lines, -4 removed lines

@@ -69,7 +69,7 @@ 
      * @param $module
      * @param $table
      * @param $tables
-     * @param $filename
+     * @param string $filename
      */
     public function write($module, $tables, $filename)
     {
@@ -131,8 +131,8 @@ 
     *  @param integer $menu
     */
     /**
-     * @param $language
-     * @param $menu
+     * @param string $language
+     * @param integer $menu
      *
      * @return string
      */
@@ -217,7 +217,7 @@ 
     *  @param null
     */
     /**
-     * @return bool|string
+     * @return null|string
      */
     public function render()
     {

class/files/admin/AdminPermissions.php

Doc Comments +11 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
     /**
      * @param $module
      * @param $tables
-     * @param $filename
+     * @param string $filename
      */
     public function write($module, $tables, $filename)
     {
@@ -85,6 +85,10 @@ 
      *
      * @return string
      */
+
+    /**
+     * @param string $language
+     */
     private function getPermissionsHeader($module, $language)
     {
         $moduleDirname = $module->getVar('mod_dirname');
@@ -117,6 +121,10 @@ 
      *
      * @return string
      */
+
+    /**
+     * @param string $language
+     */
     private function getPermissionsSwitch($moduleDirname, $language)
     {        		
 		$cases = array('global' => array("\$formTitle = {$language}PERMISSIONS_GLOBAL;",
@@ -145,7 +153,7 @@ 
     */
     /**
      * @param $module
-     * @param $language
+     * @param string $language
      *
      * @return string
      */
@@ -200,7 +208,7 @@ 
     *  @param null
     */
     /**
-     * @return bool|string
+     * @return null|string
      */
     public function render()
     {

Indentation +42 added lines, -42 removed lines

@@ -95,17 +95,17 @@ 
             }
         }
         $ret = $this->getInclude('header');		
-		$ret .= $this->phpcode->getPhpCodeIncludeDir('XOOPS_ROOT_PATH', 'class/xoopsform/grouppermform', true);		
+        $ret .= $this->phpcode->getPhpCodeIncludeDir('XOOPS_ROOT_PATH', 'class/xoopsform/grouppermform', true);		
         $ret .= $this->xoopscode->getXoopsHandlerLine($moduleDirname, $tableName);
-		$ret .= $this->getCommentLine('Check admin have access to this page');
-		$ret .= $this->adminxoopscode->getAdminTemplateMain($moduleDirname, 'permissions');		
-		$ret .= $this->xoopscode->getXoopsCodeTplAssign('navigation', "\$adminMenu->addNavigation('permissions.php')");
-		$ret .= $this->xoopscode->getXoopsCodeXoopsRequest('op', 'op', 'global');
-		$ret .= $this->xoopscode->getXoopsCodeLoad('XoopsFormLoader');
-		$optionsSelect = array('global' => "{$language}PERMISSIONS_GLOBAL", 'approve' => "{$language}PERMISSIONS_GLOBAL", 
-								'submit' => "{$language}PERMISSIONS_GLOBAL", 'view' => "{$language}PERMISSIONS_GLOBAL");
-		$formSelect = $this->xoopscode->getXoopsFormSelect('formSelect', '\'\'', 'op', $optionsSelect, 'onchange="document.fselperm.submit()"');
-		$ret .= $this->xoopscode->getXoopsSimpleForm('permTableForm', 'formSelect', $formSelect, '\'\'', 'fselperm', 'permissions');
+        $ret .= $this->getCommentLine('Check admin have access to this page');
+        $ret .= $this->adminxoopscode->getAdminTemplateMain($moduleDirname, 'permissions');		
+        $ret .= $this->xoopscode->getXoopsCodeTplAssign('navigation', "\$adminMenu->addNavigation('permissions.php')");
+        $ret .= $this->xoopscode->getXoopsCodeXoopsRequest('op', 'op', 'global');
+        $ret .= $this->xoopscode->getXoopsCodeLoad('XoopsFormLoader');
+        $optionsSelect = array('global' => "{$language}PERMISSIONS_GLOBAL", 'approve' => "{$language}PERMISSIONS_GLOBAL", 
+                                'submit' => "{$language}PERMISSIONS_GLOBAL", 'view' => "{$language}PERMISSIONS_GLOBAL");
+        $formSelect = $this->xoopscode->getXoopsFormSelect('formSelect', '\'\'', 'op', $optionsSelect, 'onchange="document.fselperm.submit()"');
+        $ret .= $this->xoopscode->getXoopsSimpleForm('permTableForm', 'formSelect', $formSelect, '\'\'', 'fselperm', 'permissions');
 
         return $ret;
     }
@@ -119,23 +119,23 @@ 
      */
     private function getPermissionsSwitch($moduleDirname, $language)
     {        		
-		$cases = array('global' => array("\$formTitle = {$language}PERMISSIONS_GLOBAL;",
-										"\$permName = '{$moduleDirname}_ac';",
-										"\$permDesc = {$language}PERMISSIONS_GLOBAL_DESC;",
-										"\$globalPerms = array( '4' => {$language}PERMISSIONS_GLOBAL_4, '8' => {$language}PERMISSIONS_GLOBAL_8, '16' => {$language}PERMISSIONS_GLOBAL_16 );"), 
-						'approve' => array("\$formTitle = {$language}PERMISSIONS_APPROVE;",
-										"\$permName = '{$moduleDirname}_approve';",
-										"\$permDesc = {$language}PERMISSIONS_APPROVE_DESC;"),
-						'submit' => array("\$formTitle = {$language}PERMISSIONS_SUBMIT;",
-										"\$permName = '{$moduleDirname}_submit';",
-										"\$permDesc = {$language}PERMISSIONS_SUBMIT_DESC;"),
-						'view' => array("\$formTitle = {$language}PERMISSIONS_VIEW;", 
-										"\$permName = '{$moduleDirname}_view';", 
-										"\$permDesc = {$language}PERMISSIONS_VIEW_DESC;"));
+        $cases = array('global' => array("\$formTitle = {$language}PERMISSIONS_GLOBAL;",
+                                        "\$permName = '{$moduleDirname}_ac';",
+                                        "\$permDesc = {$language}PERMISSIONS_GLOBAL_DESC;",
+                                        "\$globalPerms = array( '4' => {$language}PERMISSIONS_GLOBAL_4, '8' => {$language}PERMISSIONS_GLOBAL_8, '16' => {$language}PERMISSIONS_GLOBAL_16 );"), 
+                        'approve' => array("\$formTitle = {$language}PERMISSIONS_APPROVE;",
+                                        "\$permName = '{$moduleDirname}_approve';",
+                                        "\$permDesc = {$language}PERMISSIONS_APPROVE_DESC;"),
+                        'submit' => array("\$formTitle = {$language}PERMISSIONS_SUBMIT;",
+                                        "\$permName = '{$moduleDirname}_submit';",
+                                        "\$permDesc = {$language}PERMISSIONS_SUBMIT_DESC;"),
+                        'view' => array("\$formTitle = {$language}PERMISSIONS_VIEW;", 
+                                        "\$permName = '{$moduleDirname}_view';", 
+                                        "\$permDesc = {$language}PERMISSIONS_VIEW_DESC;"));
 		
-		$contentSwitch = $this->phpcode->getPhpCodeCaseSwitch($cases, true);
+        $contentSwitch = $this->phpcode->getPhpCodeCaseSwitch($cases, true);
         
-		return $this->phpcode->getPhpCodeSwitch('op', $contentSwitch);
+        return $this->phpcode->getPhpCodeSwitch('op', $contentSwitch);
     }
 
     /*
@@ -173,26 +173,26 @@ 
             }
         }		
         
-		$ret = $this->xoopscode->getXoopsCodeGetVar('moduleId', 'xoopsModule', 'mid');
-		$ret .= $this->xoopscode->getXoopsCodeGroupPermForm('permform', '$formTitle', '$moduleId', '$permName', '$permDesc', "'admin/permissions.php'");
-		$foreach1 = "\t".$this->xoopscode->getXoopsCodeAddItem('permform', '$gPermId', '$gPermName');
+        $ret = $this->xoopscode->getXoopsCodeGetVar('moduleId', 'xoopsModule', 'mid');
+        $ret .= $this->xoopscode->getXoopsCodeGroupPermForm('permform', '$formTitle', '$moduleId', '$permName', '$permDesc', "'admin/permissions.php'");
+        $foreach1 = "\t".$this->xoopscode->getXoopsCodeAddItem('permform', '$gPermId', '$gPermName');
         $if1 = $this->phpcode->getPhpCodeForeach('globalPerms', false, 'gPermId', 'gPermName', $foreach1);
-		$if1 .= "\t".$this->xoopscode->getXoopsCodeTplAssign('form', '$permform->render()');
-		$else = $this->xoopscode->getXoopsCodeObjHandlerCount($tableName);
-		$else .= $this->xoopscode->getXoopsCodeObjHandlerAll($tableName, $fieldMain);
-		$getVar1 = $this->xoopscode->getXoopsCodeGetVar('', "{$tableName}All[\$i]", $fieldId, true);
-		$getVar2 = $this->xoopscode->getXoopsCodeGetVar('', "{$tableName}All[\$i]", $fieldMain, true);
-		$foreach2 = $this->xoopscode->getXoopsCodeAddItem('permform', $getVar1, $getVar2);
-		$else .=  $this->phpcode->getPhpCodeForeach("{$tableName}All", true, false, 'i', $foreach2);
-		$if2 = $this->xoopscode->getXoopsCodeTplAssign('form', '$permform->render()');
-		$elseInter = $this->xoopscode->getXoopsCodeRedirectHeader($tableName, '?op=new', '3', "{$language}NO_PERMISSIONS_SET");
-		$elseInter .= $this->getSimpleString("\texit();");
-		$else .= "\t".$this->phpcode->getPhpCodeConditions("\${$tableName}Count", ' > ', '0', $if2, $elseInter);
+        $if1 .= "\t".$this->xoopscode->getXoopsCodeTplAssign('form', '$permform->render()');
+        $else = $this->xoopscode->getXoopsCodeObjHandlerCount($tableName);
+        $else .= $this->xoopscode->getXoopsCodeObjHandlerAll($tableName, $fieldMain);
+        $getVar1 = $this->xoopscode->getXoopsCodeGetVar('', "{$tableName}All[\$i]", $fieldId, true);
+        $getVar2 = $this->xoopscode->getXoopsCodeGetVar('', "{$tableName}All[\$i]", $fieldMain, true);
+        $foreach2 = $this->xoopscode->getXoopsCodeAddItem('permform', $getVar1, $getVar2);
+        $else .=  $this->phpcode->getPhpCodeForeach("{$tableName}All", true, false, 'i', $foreach2);
+        $if2 = $this->xoopscode->getXoopsCodeTplAssign('form', '$permform->render()');
+        $elseInter = $this->xoopscode->getXoopsCodeRedirectHeader($tableName, '?op=new', '3', "{$language}NO_PERMISSIONS_SET");
+        $elseInter .= $this->getSimpleString("\texit();");
+        $else .= "\t".$this->phpcode->getPhpCodeConditions("\${$tableName}Count", ' > ', '0', $if2, $elseInter);
 		
-		$ret .= "\t".$this->phpcode->getPhpCodeConditions('$op', ' == ', "'global'", $if1, $else);
-		$ret .= $this->phpcode->getPhpCodeUnset('permform');
+        $ret .= "\t".$this->phpcode->getPhpCodeConditions('$op', ' == ', "'global'", $if1, $else);
+        $ret .= $this->phpcode->getPhpCodeUnset('permform');
 		
-		return $ret;
+        return $ret;
     }
 
     /*

Spacing +1 added lines, -1 removed lines

@@ -183,7 +183,7 @@
 		$getVar1 = $this->xoopscode->getXoopsCodeGetVar('', "{$tableName}All[\$i]", $fieldId, true);
 		$getVar2 = $this->xoopscode->getXoopsCodeGetVar('', "{$tableName}All[\$i]", $fieldMain, true);
 		$foreach2 = $this->xoopscode->getXoopsCodeAddItem('permform', $getVar1, $getVar2);
-		$else .=  $this->phpcode->getPhpCodeForeach("{$tableName}All", true, false, 'i', $foreach2);
+		$else .= $this->phpcode->getPhpCodeForeach("{$tableName}All", true, false, 'i', $foreach2);
 		$if2 = $this->xoopscode->getXoopsCodeTplAssign('form', '$permform->render()');
 		$elseInter = $this->xoopscode->getXoopsCodeRedirectHeader($tableName, '?op=new', '3', "{$language}NO_PERMISSIONS_SET");
 		$elseInter .= $this->getSimpleString("\texit();");

class/files/TDMCreatePhpCode.php

Indentation +41 added lines, -41 removed lines

@@ -185,7 +185,7 @@ 
     public function getPhpCodeForeach($array = '', $arrayKey = false, $key = false, $value = false, $content = '')
     {
         $vars = '';
-		if ((false === $arrayKey) && (false === $key)) {
+        if ((false === $arrayKey) && (false === $key)) {
             $vars = "\${$array} as \${$value}";
         } elseif ((false === $arrayKey) && (false !== $key)) {
             $vars = "\${$array} as \${$key} => \${$value}";
@@ -266,35 +266,35 @@ 
     /**
      *  @public function getPhpCodeCaseSwitch     
      *  @param $cases
-	 *  @param $defaultAfterCase
-	 *  @param $default
+     *  @param $defaultAfterCase
+     *  @param $default
      *
      *  @return string
      */
     public function getPhpCodeCaseSwitch($cases = array(), $defaultAfterCase = false, $default = false)
     {        
         $ret = '';
-		$def = "\tdefault:\n";
+        $def = "\tdefault:\n";
         foreach($cases as $case => $value) {
-			$case = is_string($case) ? "'{$case}'" : $case;
-			if (!empty($case)) {
-				$ret .= "\tcase {$case}:\n";
-				if($defaultAfterCase != false){
-					$ret .= $def;
-				}
-				if (is_array($value)) {
-					foreach($value as $content) {
-						$ret .= "\t\t{$content}\n";
-					}
-				}
-				$ret .= "\tbreak;\n";
-			}	
-			$defaultAfterCase = false;
-		}		
+            $case = is_string($case) ? "'{$case}'" : $case;
+            if (!empty($case)) {
+                $ret .= "\tcase {$case}:\n";
+                if($defaultAfterCase != false){
+                    $ret .= $def;
+                }
+                if (is_array($value)) {
+                    foreach($value as $content) {
+                        $ret .= "\t\t{$content}\n";
+                    }
+                }
+                $ret .= "\tbreak;\n";
+            }	
+            $defaultAfterCase = false;
+        }		
         if ($default !== false) {
             $ret .= $def;
-			$ret .= "\t\t{$default}\n";
-			$ret .= "\tbreak;\n";			
+            $ret .= "\t\t{$default}\n";
+            $ret .= "\tbreak;\n";			
         }
 
         return $ret;
@@ -310,7 +310,7 @@ 
         return "isset(\${$var})";
     }
 	
-	/*
+    /*
     *  @public function getPhpCodeUnset
     *  @param $var
     *  @return string
@@ -372,7 +372,7 @@ 
         return "rawurlencode({$var})";
     }
 	
-	/*
+    /*
     *  @public function getPhpCodePregReplace
     *  @param $return
 	*  @param $exp
@@ -384,15 +384,15 @@ 
     */
     public function getPhpCodePregReplace($return, $exp, $str, $val, $isParam = false)
     {
-		if ($isParam === false) {	
-			$ret = "\${$return} = preg_replace( '{$exp}' , '{$str}' , {$val});\n";
-		} else {
-			$ret = "preg_replace( '{$exp}' , '{$str}' , {$val})";
-		}
-		return $ret;
+        if ($isParam === false) {	
+            $ret = "\${$return} = preg_replace( '{$exp}' , '{$str}' , {$val});\n";
+        } else {
+            $ret = "preg_replace( '{$exp}' , '{$str}' , {$val})";
+        }
+        return $ret;
     }
 	
-	/*
+    /*
     *  @public function getPhpCodeStrReplace
     *  @param $left
 	*  @param $var
@@ -405,14 +405,14 @@ 
     public function getPhpCodeStrReplace($left, $var, $str, $value, $isParam = false)
     {
         if ($isParam === false) {	
-			$ret = "\${$left} = str_replace( '{$var}' , '{$str}' , {$value});\n";
-		} else {
-			$ret = "str_replace( '{$var}' , '{$str}' , {$value})";
-		}
-		return $ret;
+            $ret = "\${$left} = str_replace( '{$var}' , '{$str}' , {$value});\n";
+        } else {
+            $ret = "str_replace( '{$var}' , '{$str}' , {$value})";
+        }
+        return $ret;
     }
 	
-	/*
+    /*
     *  @public function getPhpCodeStripTags
     *  @param $left
 	*  @param $value
@@ -423,11 +423,11 @@ 
     public function getPhpCodeStripTags($left, $value, $isParam = false)
     {
         if ($isParam === false) {	
-			$ret = "\${$left} = strip_tags({$value});\n";
-		} else {
-			$ret = "strip_tags({$value})";
-		}
-		return $ret;
+            $ret = "\${$left} = strip_tags({$value});\n";
+        } else {
+            $ret = "strip_tags({$value})";
+        }
+        return $ret;
     }
 
     /*

Spacing +3 added lines, -3 removed lines

@@ -275,15 +275,15 @@
     {        
         $ret = '';
 		$def = "\tdefault:\n";
-        foreach($cases as $case => $value) {
+        foreach ($cases as $case => $value) {
 			$case = is_string($case) ? "'{$case}'" : $case;
 			if (!empty($case)) {
 				$ret .= "\tcase {$case}:\n";
-				if($defaultAfterCase != false){
+				if ($defaultAfterCase != false) {
 					$ret .= $def;
 				}
 				if (is_array($value)) {
-					foreach($value as $content) {
+					foreach ($value as $content) {
 						$ret .= "\t\t{$content}\n";
 					}
 				}

class/files/TDMCreateXoopsCode.php

Indentation +36 added lines, -36 removed lines

@@ -79,7 +79,7 @@ 
         return $ret;
     }
 	
-	/*
+    /*
      *  @private function getXoopsCodeSwitch
      *  @param $op
      *  @param $listCases
@@ -89,9 +89,9 @@ 
      */
     private function getXoopsCodeSwitch($op = 'op', $listCases = array(), $defaultList = false)
     {        		
-		$switch = $this->phpcode->getPhpCodeCaseSwitch(array($listCases), $defaultList);
+        $switch = $this->phpcode->getPhpCodeCaseSwitch(array($listCases), $defaultList);
         
-		return $this->phpcode->getPhpCodeSwitch($op, $switch);
+        return $this->phpcode->getPhpCodeSwitch($op, $switch);
     }
 
     /*
@@ -126,7 +126,7 @@ 
         return "xoops_load('{$var}');\n";
     }
 	
-	/**
+    /**
      *  @public function getXoopsCodeLoadLanguage
      *
      *  @param $lang
@@ -150,7 +150,7 @@ 
         return "\${$tableName}Obj->setVar('{$fieldName}', {$var});\n";
     }
 	
-	/*
+    /*
     *  @public function getXoopsCodeGetVar
     *  @param string $varLeft
 	*  @param string $handle
@@ -162,15 +162,15 @@ 
     public function getXoopsCodeGetVar($varLeft = '', $handle = '', $var = '', $isParam = false)
     {
         if($isParam === false) {
-			$ret = "\${$varLeft} = \${$handle}->getVar('{$var}');\n";
-		} else {	
-			$ret = "\${$handle}->getVar('{$var}')";
-		}
+            $ret = "\${$varLeft} = \${$handle}->getVar('{$var}');\n";
+        } else {	
+            $ret = "\${$handle}->getVar('{$var}')";
+        }
 		
-		return $ret; 
+        return $ret; 
     }
 	
-	/*
+    /*
     *  @public function getXoopsCodeGroupPermForm
     *  @param string $varLeft
 	*  @param string $formTitle
@@ -183,10 +183,10 @@ 
     */
     public function getXoopsCodeGroupPermForm($varLeft = '', $formTitle = '', $moduleId = '', $permName = '', $permDesc = '', $filename = '')
     {        
-		return "\${$varLeft} = new XoopsGroupPermForm({$formTitle}, {$moduleId}, {$permName}, {$permDesc}, {$filename});\n";
+        return "\${$varLeft} = new XoopsGroupPermForm({$formTitle}, {$moduleId}, {$permName}, {$permDesc}, {$filename});\n";
     }
 	
-	/*
+    /*
     *  @public function getXoopsCodeAddItem
     *  @param string $varLeft
 	*  @param string $paramLeft
@@ -196,7 +196,7 @@ 
     */
     public function getXoopsCodeAddItem($varLeft = '', $paramLeft = '', $paramRight = '')
     {		
-		return "\${$varLeft}->addItem({$paramLeft}, {$paramRight});\n";
+        return "\${$varLeft}->addItem({$paramLeft}, {$paramRight});\n";
     }	
 
     /*
@@ -257,7 +257,7 @@ 
         return "\${$lpFieldName}['{$rpFieldName}'] = \${$tableName}All[\$i]->getVar('{$fieldName}');\n";
     }
 	
-	/*
+    /*
     *  @public function getXoopsHandlerLine
     *  @param $moduleDirname
     *  @param $tableName
@@ -268,7 +268,7 @@ 
         return "\${$tableName}Handler =& \${$moduleDirname}->getHandler('{$tableName}');\n";
     }
 	
-	/*
+    /*
     *  @public function getXoopsSimpleForm
     *  @param $left
 	*  @param $element
@@ -283,16 +283,16 @@ 
     public function getXoopsSimpleForm($left = '', $element = '', $elementsContent = '', $caption = '', $var = '', $filename = '', $type = 'post')
     {
         $ret = "\${$left} = new XoopsSimpleForm({$caption}, '{$var}', '{$filename}.php', '{$type}');\n";
-		if(!empty($elementsContent)) {
-			$ret .= "{$elementsContent}";
-		}
-		$ret .= "\${$left}->addElement(\${$element})\n";
-		$ret .= "\${$left}->display();\n";
+        if(!empty($elementsContent)) {
+            $ret .= "{$elementsContent}";
+        }
+        $ret .= "\${$left}->addElement(\${$element})\n";
+        $ret .= "\${$left}->display();\n";
 
-		return $ret;
+        return $ret;
     }
 	
-	/*
+    /*
     *  @public function getXoopsFormSelect
     *  @param $varSelect
 	*  @param $caption
@@ -302,17 +302,17 @@ 
     *  
     *  @return string
     */
-	public function getXoopsFormSelect($varSelect = '', $caption = '', $var = '', $options = array(), $setExtra = true)
+    public function getXoopsFormSelect($varSelect = '', $caption = '', $var = '', $options = array(), $setExtra = true)
     {
         $ret = "\${$varSelect} = new XoopsFormSelect({$caption}, '{$var}', \${$var});\n";
-		if(false !== $setExtra) {
-			$ret .= "\${$varSelect}->setExtra('{$setExtra}');\n";
-		}
-		foreach($options as $key => $value) {
-			$ret .= "\${$varSelect}->addOption('{$key}', {$value});\n";
-		}
+        if(false !== $setExtra) {
+            $ret .= "\${$varSelect}->setExtra('{$setExtra}');\n";
+        }
+        foreach($options as $key => $value) {
+            $ret .= "\${$varSelect}->addOption('{$key}', {$value});\n";
+        }
 		
-		return $ret;
+        return $ret;
     }	
 	
     /*
@@ -691,7 +691,7 @@ 
         return $ret;
     }
 	
-	/**
+    /**
      *  @public function getXoopsCodeObjHandlerCount
      *
      *  @param string $tableName
@@ -701,23 +701,23 @@ 
     public function getXoopsCodeObjHandlerCount($tableName)
     {
         $ucfTableName = ucfirst($tableName);
-		$ret = "\${$tableName}Count = \${$tableName}Handler->getCount{$ucfTableName}();\n";
+        $ret = "\${$tableName}Count = \${$tableName}Handler->getCount{$ucfTableName}();\n";
 
         return $ret;
     }
 	
-	/**
+    /**
      *  @public function getXoopsCodeObjHandlerAll
      *
      *  @param string $tableName
-	 *  @param string $fieldMain
+     *  @param string $fieldMain
      *
      *  @return string
      */
     public function getXoopsCodeObjHandlerAll($tableName, $fieldMain)
     {
         $ucfTableName = ucfirst($tableName);
-		$ret = "\${$tableName}All = \${$tableName}Handler->getAll{$ucfTableName}(0, 0, '{$fieldMain}');\n";
+        $ret = "\${$tableName}All = \${$tableName}Handler->getAll{$ucfTableName}(0, 0, '{$fieldMain}');\n";
 
         return $ret;
     }

Spacing +4 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
     */
     public function getXoopsCodeGetVar($varLeft = '', $handle = '', $var = '', $isParam = false)
     {
-        if($isParam === false) {
+        if ($isParam === false) {
 			$ret = "\${$varLeft} = \${$handle}->getVar('{$var}');\n";
 		} else {	
 			$ret = "\${$handle}->getVar('{$var}')";
@@ -283,7 +283,7 @@ 
     public function getXoopsSimpleForm($left = '', $element = '', $elementsContent = '', $caption = '', $var = '', $filename = '', $type = 'post')
     {
         $ret = "\${$left} = new XoopsSimpleForm({$caption}, '{$var}', '{$filename}.php', '{$type}');\n";
-		if(!empty($elementsContent)) {
+		if (!empty($elementsContent)) {
 			$ret .= "{$elementsContent}";
 		}
 		$ret .= "\${$left}->addElement(\${$element})\n";
@@ -305,10 +305,10 @@ 
 	public function getXoopsFormSelect($varSelect = '', $caption = '', $var = '', $options = array(), $setExtra = true)
     {
         $ret = "\${$varSelect} = new XoopsFormSelect({$caption}, '{$var}', \${$var});\n";
-		if(false !== $setExtra) {
+		if (false !== $setExtra) {
 			$ret .= "\${$varSelect}->setExtra('{$setExtra}');\n";
 		}
-		foreach($options as $key => $value) {
+		foreach ($options as $key => $value) {
 			$ret .= "\${$varSelect}->addOption('{$key}', {$value});\n";
 		}
 		

class/files/admin/AdminPages.php

Indentation +7 added lines, -7 removed lines

@@ -94,15 +94,15 @@ 
         $ret .= $this->xoopscode->getXoopsCodeEqualsOperator("\${$ccFieldId}", "XoopsRequest::getInt('{$fieldId}')");
         
         
-		// This must be removed
-		$ret .= $this->getCommentLine('Switch options');
-		$ret .= $this->getSimpleString('switch($op)');
+        // This must be removed
+        $ret .= $this->getCommentLine('Switch options');
+        $ret .= $this->getSimpleString('switch($op)');
         $ret .= $this->getSimpleString('{');
 
         return $ret;
     }
 	
-	/*
+    /*
      *  @private function getAdminPagesSwitch
      *  @param $moduleDirname
      *  @param $language
@@ -111,11 +111,11 @@ 
      */
     private function getAdminPagesSwitch($moduleDirname, $language)
     {        		
-		$cases = array('list' => array(), 'new' => array(),	'save' => array(), 'edit' => array(), 'update' => array());
+        $cases = array('list' => array(), 'new' => array(),	'save' => array(), 'edit' => array(), 'update' => array());
 		
-		$contentSwitch = $this->phpcode->getPhpCodeCaseSwitch($cases, true);
+        $contentSwitch = $this->phpcode->getPhpCodeCaseSwitch($cases, true);
         
-		return $this->phpcode->getPhpCodeSwitch('op', $contentSwitch);
+        return $this->phpcode->getPhpCodeSwitch('op', $contentSwitch);
     }
 
     /*

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
      */
     private function getAdminPagesSwitch($moduleDirname, $language)
     {        		
-		$cases = array('list' => array(), 'new' => array(),	'save' => array(), 'edit' => array(), 'update' => array());
+		$cases = array('list' => array(), 'new' => array(), 'save' => array(), 'edit' => array(), 'update' => array());
 		
 		$contentSwitch = $this->phpcode->getPhpCodeCaseSwitch($cases, true);
         

class/files/admin/AdminAbout.php

Doc Comments +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     /**
      * @param $module
      * @param $tables
-     * @param $filename
+     * @param string $filename
      */
     public function write($module, $tables, $filename)
     {
@@ -146,7 +146,7 @@ 
     *  @param null
     */
     /**
-     * @return bool|string
+     * @return null|string
      */
     public function render()
     {

class/files/admin/AdminFooter.php

Doc Comments +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     /**
      * @param $module
      * @param $tables
-     * @param $filename
+     * @param string $filename
      */
     public function write($module, $tables, $filename)
     {
@@ -146,7 +146,7 @@ 
     *  @param null
     */
     /**
-     * @return bool|string
+     * @return null|string
      */
     public function render()
     {