txmodxoops / tdmcreate

testdata/mymodule2/admin/header.php

Indentation +4 added lines, -4 removed lines

@@ -36,8 +36,8 @@ 
 $myts = MyTextSanitizer::getInstance();
 // 
 if (!isset($xoopsTpl) || !is_object($xoopsTpl)) {
-	include_once XOOPS_ROOT_PATH . '/class/template.php';
-	$xoopsTpl = new \XoopsTpl();
+    include_once XOOPS_ROOT_PATH . '/class/template.php';
+    $xoopsTpl = new \XoopsTpl();
 }
 
 // Load languages
@@ -46,9 +46,9 @@ 
 
 // Local admin menu class
 if (file_exists($GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php'))) {
-	include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
+    include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
 } else {
-	redirect_header('../../../admin.php', 5, _AM_MODULEADMIN_MISSING);
+    redirect_header('../../../admin.php', 5, _AM_MODULEADMIN_MISSING);
 }
 
 xoops_cp_header();

testdata/mymodule2/include/functions.php

Indentation +7 added lines, -7 removed lines

@@ -54,11 +54,11 @@ 
     if(is_array($permissions) && array_key_exists($permtype, $permissions)) {
         return $permissions[$permtype];
     }
-	$moduleHandler = xoops_getHandler('module');
-	$mymodule2Module = $moduleHandler->getByDirname($dirname);
-	$groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
-	$grouppermHandler = xoops_getHandler('groupperm');
-	$testfields = $grouppermHandler->getItemIds($permtype, $groups, $mymodule2Module->getVar('mid'));
+    $moduleHandler = xoops_getHandler('module');
+    $mymodule2Module = $moduleHandler->getByDirname($dirname);
+    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
+    $grouppermHandler = xoops_getHandler('groupperm');
+    $testfields = $grouppermHandler->getItemIds($permtype, $groups, $mymodule2Module->getVar('mid'));
     return $testfields;
 }
 
@@ -160,7 +160,7 @@ 
 
         case 'none':
             if($topic_name) {
-                 $topic_name = 'topic=' . $topic_name . '&';
+                    $topic_name = 'topic=' . $topic_name . '&';
             }
             $rewrite_base = '/modules/';
             $page = 'page=' . $array['content_alias'];
@@ -190,7 +190,7 @@ 
             return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name  . $id . $page . $rewrite_ext;
             break;
 
-         case 'short':
+            case 'short':
             if($topic_name) {
                 $topic_name .= '/';
             }

testdata/mymodule2/include/notifications.inc.php

Indentation +2 added lines, -2 removed lines

@@ -63,8 +63,8 @@
             $result = $xoopsDB->query($sql); // TODO: error check
             $result_array = $xoopsDB->fetchArray($result);
             $item['name'] = $result_array['tf_text'];
-			$item['url'] = MYMODULE2_URL . '/single.php?cid=' . $result_array['cid'] . '&tf_id=' . $item_id;
-			return $item;
+            $item['url'] = MYMODULE2_URL . '/single.php?cid=' . $result_array['cid'] . '&tf_id=' . $item_id;
+            return $item;
         break;
     }
     return null;

testdata/mymodule2/include/update.php

Indentation +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
     $ret = mymodule2_check_db($module);
 
     //check upload directory
-	include_once __DIR__ . '/install.php';
+    include_once __DIR__ . '/install.php';
     $ret = xoops_module_install_mymodule2($module);
 
     $errors = $module->getErrors();
@@ -113,7 +113,7 @@ 
 function mymodule2_check_db($module)
 {
     $ret = true;
-	//insert here code for database check
+    //insert here code for database check
 
     /*
     // Example: update table (add new field)

testdata/mymodule2/articles.php

Indentation +17 added lines, -17 removed lines

@@ -42,23 +42,23 @@
 $articlesAll = $articlesHandler->getAllArticles($start, $limit);
 $keywords = [];
 if ($articlesCount > 0) {
-	$articles = [];
-	// Get All Articles
-	foreach(array_keys($articlesAll) as $i) {
-		$articles[] = $articlesAll[$i]->getValuesArticles();
-		$keywords[] = $articlesAll[$i]->getVar('art_title');
-	}
-	$GLOBALS['xoopsTpl']->assign('articles', $articles);
-	unset($articles);
-	// Display Navigation
-	if ($articlesCount > $limit) {
-		include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-		$pagenav = new \XoopsPageNav($articlesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-		$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-	}
-	$GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
-	$GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
-	$GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
+    $articles = [];
+    // Get All Articles
+    foreach(array_keys($articlesAll) as $i) {
+        $articles[] = $articlesAll[$i]->getValuesArticles();
+        $keywords[] = $articlesAll[$i]->getVar('art_title');
+    }
+    $GLOBALS['xoopsTpl']->assign('articles', $articles);
+    unset($articles);
+    // Display Navigation
+    if ($articlesCount > $limit) {
+        include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+        $pagenav = new \XoopsPageNav($articlesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+        $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+    }
+    $GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
+    $GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
+    $GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
 }
 
 // Breadcrumbs

testdata/mymodule2/rate.php

Indentation +168 added lines, -168 removed lines

@@ -31,176 +31,176 @@
 include_once XOOPS_ROOT_PATH . '/header.php';
 // Define Stylesheet
 $GLOBALS['xoTheme']->addStylesheet( $style, null );
-	switch($op) {
-		case 'form':
-		default:
-				// Navigation
-		$navigation = _MA_MYMODULE2_RATE;
-		$GLOBALS['xoopsTpl']->assign('navigation', $navigation);
-		// Title of page
-		$title = _MA_MYMODULE2_RATE . ' - ';
-		$title .= $GLOBALS['xoopsModule']->name();
-		$GLOBALS['xoopsTpl']->assign('xoops_pagetitle', $title);
-		// Description
-		$GLOBALS['xoTheme']->addMeta( 'meta', 'description', strip_tags(_MA_MYMODULE2_RATE));
-		// Form Create
-		$testfieldsObj = $testfieldsHandler->create();
-		$form = $testfieldsObj->getFormTestfields();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
+    switch($op) {
+        case 'form':
+        default:
+                // Navigation
+        $navigation = _MA_MYMODULE2_RATE;
+        $GLOBALS['xoopsTpl']->assign('navigation', $navigation);
+        // Title of page
+        $title = _MA_MYMODULE2_RATE . ' - ';
+        $title .= $GLOBALS['xoopsModule']->name();
+        $GLOBALS['xoopsTpl']->assign('xoops_pagetitle', $title);
+        // Description
+        $GLOBALS['xoTheme']->addMeta( 'meta', 'description', strip_tags(_MA_MYMODULE2_RATE));
+        // Form Create
+        $testfieldsObj = $testfieldsHandler->create();
+        $form = $testfieldsObj->getFormTestfields();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
 
-		break;
-		case 'save':
-				// Security Check
-		if ($GLOBALS['xoopsSecurity']->check()) {
-			redirect_header('testfields.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		$testfieldsObj = $testfieldsHandler->create();
-		$testfieldsObj->setVar('tf_text', Request::getString('tf_text', ''));
-		$testfieldsObj->setVar('tf_textarea', Request::getString('tf_textarea', ''));
-		$testfieldsObj->setVar('tf_dhtml', Request::getString('tf_dhtml', ''));
-		$testfieldsObj->setVar('tf_checkbox', Request::getInt('tf_checkbox', 0));
-		$testfieldsObj->setVar('tf_yesno', Request::getInt('tf_yesno', 0));
-		$testfieldsObj->setVar('tf_selectbox', Request::getString('tf_selectbox', ''));
-		$testfieldsObj->setVar('tf_user', Request::getInt('tf_user', 0));
-		$testfieldsObj->setVar('tf_color', Request::getString('tf_color', ''));
-		// Set Var tf_imagelist
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$uploader = new \XoopsMediaUploader(XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32', 
-													$helper->getConfig('mimetypes_image'), 
-													$helper->getConfig('maxsize_image'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][0])) {
-			//$uploader->setPrefix(tf_imagelist_);
-			//$uploader->fetchMedia($_POST['xoops_upload_file'][0]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-				redirect_header('javascript:history.go(-1).php', 3, $errors);
-			} else {
-				$testfieldsObj->setVar('tf_imagelist', $uploader->getSavedFileName());
-			}
-		} else {
-			$testfieldsObj->setVar('tf_imagelist', Request::getString('tf_imagelist'));
-		}
-		$testfieldsObj->setVar('tf_urlfile', formatUrl($_REQUEST['tf_urlfile']));
-		// Set Var tf_urlfile
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_urlfile']['name'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_file'), 
-													$helper->getConfig('maxsize_file'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][1])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][1]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-			} else {
-				$testfieldsObj->setVar('tf_urlfile', $uploader->getSavedFileName());
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_urlfile', Request::getString('tf_urlfile'));
-		}
-		// Set Var tf_uplimage
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_uplimage']['name'];
-		$imgMimetype    = $_FILES['tf_uplimage']['type'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploaderErrors = '';
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_image'), 
-													$helper->getConfig('maxsize_image'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][2])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][2]);
-			if (!$uploader->upload()) {
-				$uploaderErrors = $uploader->getErrors();
-			} else {
-				$savedFilename = $uploader->getSavedFileName();
-				$maxwidth  = (int)$helper->getConfig('maxwidth_image');
-				$maxheight = (int)$helper->getConfig('maxheight_image');
-				if ($maxwidth > 0 && $maxheight > 0) {
-					// Resize image
-					$imgHandler                = new Mymodule2\Common\Resizer();
-					$imgHandler->sourceFile    = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
-					$imgHandler->endFile       = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
-					$imgHandler->imageMimetype = $imgMimetype;
-					$imgHandler->maxWidth      = $maxwidth;
-					$imgHandler->maxHeight     = $maxheight;
-					$result                    = $imgHandler->resizeImage();
-				}
-				$testfieldsObj->setVar('tf_uplimage', $savedFilename);
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_uplimage', Request::getString('tf_uplimage'));
-		}
-		// Set Var tf_uplfile
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_uplfile']['name'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_file'), 
-													$helper->getConfig('maxsize_file'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][3])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][3]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-			} else {
-				$testfieldsObj->setVar('tf_uplfile', $uploader->getSavedFileName());
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_uplfile', Request::getString('tf_uplfile'));
-		}
-		$testfieldTextdateselect = date_create_from_format(_SHORTDATESTRING, Request::getString('tf_textdateselect'));
-		$testfieldsObj->setVar('tf_textdateselect', $testfieldTextdateselect->getTimestamp());
-		// Set Var tf_selectfile
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_selectfile']['name'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_file'), 
-													$helper->getConfig('maxsize_file'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][4])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][4]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-			} else {
-				$testfieldsObj->setVar('tf_selectfile', $uploader->getSavedFileName());
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_selectfile', Request::getString('tf_selectfile'));
-		}
-		$testfieldsObj->setVar('tf_status', Request::getInt('tf_status', 0));
-		// Insert Data
-		if ($testfields1->insert($testfieldsObj)) {
-			redirect_header('index.php', 2, _MA_MYMODULE2_FORM_OK);
-		}
-		// Get Form Error
-		$GLOBALS['xoopsTpl']->assign('error', $testfieldsObj->getHtmlErrors());
-		$form = $testfieldsObj->getFormTestfields();
-		$GLOBALS['xoopsTpl']->assign('form', $form->display());
+        break;
+        case 'save':
+                // Security Check
+        if ($GLOBALS['xoopsSecurity']->check()) {
+            redirect_header('testfields.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        $testfieldsObj = $testfieldsHandler->create();
+        $testfieldsObj->setVar('tf_text', Request::getString('tf_text', ''));
+        $testfieldsObj->setVar('tf_textarea', Request::getString('tf_textarea', ''));
+        $testfieldsObj->setVar('tf_dhtml', Request::getString('tf_dhtml', ''));
+        $testfieldsObj->setVar('tf_checkbox', Request::getInt('tf_checkbox', 0));
+        $testfieldsObj->setVar('tf_yesno', Request::getInt('tf_yesno', 0));
+        $testfieldsObj->setVar('tf_selectbox', Request::getString('tf_selectbox', ''));
+        $testfieldsObj->setVar('tf_user', Request::getInt('tf_user', 0));
+        $testfieldsObj->setVar('tf_color', Request::getString('tf_color', ''));
+        // Set Var tf_imagelist
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $uploader = new \XoopsMediaUploader(XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32', 
+                                                    $helper->getConfig('mimetypes_image'), 
+                                                    $helper->getConfig('maxsize_image'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][0])) {
+            //$uploader->setPrefix(tf_imagelist_);
+            //$uploader->fetchMedia($_POST['xoops_upload_file'][0]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+                redirect_header('javascript:history.go(-1).php', 3, $errors);
+            } else {
+                $testfieldsObj->setVar('tf_imagelist', $uploader->getSavedFileName());
+            }
+        } else {
+            $testfieldsObj->setVar('tf_imagelist', Request::getString('tf_imagelist'));
+        }
+        $testfieldsObj->setVar('tf_urlfile', formatUrl($_REQUEST['tf_urlfile']));
+        // Set Var tf_urlfile
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_urlfile']['name'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_file'), 
+                                                    $helper->getConfig('maxsize_file'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][1])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][1]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+            } else {
+                $testfieldsObj->setVar('tf_urlfile', $uploader->getSavedFileName());
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_urlfile', Request::getString('tf_urlfile'));
+        }
+        // Set Var tf_uplimage
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_uplimage']['name'];
+        $imgMimetype    = $_FILES['tf_uplimage']['type'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploaderErrors = '';
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_image'), 
+                                                    $helper->getConfig('maxsize_image'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][2])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][2]);
+            if (!$uploader->upload()) {
+                $uploaderErrors = $uploader->getErrors();
+            } else {
+                $savedFilename = $uploader->getSavedFileName();
+                $maxwidth  = (int)$helper->getConfig('maxwidth_image');
+                $maxheight = (int)$helper->getConfig('maxheight_image');
+                if ($maxwidth > 0 && $maxheight > 0) {
+                    // Resize image
+                    $imgHandler                = new Mymodule2\Common\Resizer();
+                    $imgHandler->sourceFile    = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
+                    $imgHandler->endFile       = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
+                    $imgHandler->imageMimetype = $imgMimetype;
+                    $imgHandler->maxWidth      = $maxwidth;
+                    $imgHandler->maxHeight     = $maxheight;
+                    $result                    = $imgHandler->resizeImage();
+                }
+                $testfieldsObj->setVar('tf_uplimage', $savedFilename);
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_uplimage', Request::getString('tf_uplimage'));
+        }
+        // Set Var tf_uplfile
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_uplfile']['name'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_file'), 
+                                                    $helper->getConfig('maxsize_file'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][3])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][3]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+            } else {
+                $testfieldsObj->setVar('tf_uplfile', $uploader->getSavedFileName());
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_uplfile', Request::getString('tf_uplfile'));
+        }
+        $testfieldTextdateselect = date_create_from_format(_SHORTDATESTRING, Request::getString('tf_textdateselect'));
+        $testfieldsObj->setVar('tf_textdateselect', $testfieldTextdateselect->getTimestamp());
+        // Set Var tf_selectfile
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_selectfile']['name'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_file'), 
+                                                    $helper->getConfig('maxsize_file'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][4])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][4]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+            } else {
+                $testfieldsObj->setVar('tf_selectfile', $uploader->getSavedFileName());
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_selectfile', Request::getString('tf_selectfile'));
+        }
+        $testfieldsObj->setVar('tf_status', Request::getInt('tf_status', 0));
+        // Insert Data
+        if ($testfields1->insert($testfieldsObj)) {
+            redirect_header('index.php', 2, _MA_MYMODULE2_FORM_OK);
+        }
+        // Get Form Error
+        $GLOBALS['xoopsTpl']->assign('error', $testfieldsObj->getHtmlErrors());
+        $form = $testfieldsObj->getFormTestfields();
+        $GLOBALS['xoopsTpl']->assign('form', $form->display());
 
-		break;
+        break;
 }
 // Breadcrumbs
 $xoBreadcrumbs[] = ['title' => RATE_MA_MYMODULE2_];

testdata/mymodule2/broken.php

Indentation +182 added lines, -182 removed lines

@@ -33,191 +33,191 @@
 $GLOBALS['xoTheme']->addStylesheet( $style, null );
 // Redirection if not permissions
 if ($permSubmit === false) {
-	redirect_header('index.php', 2, _NOPERM);
-	exit();
+    redirect_header('index.php', 2, _NOPERM);
+    exit();
 }
 switch($op) {
-	case 'form':
-	default:
-		// Navigation
-		$navigation = _MA_MYMODULE2_SUBMIT_PROPOSER;
-		$GLOBALS['xoopsTpl']->assign('navigation', $navigation);
-		// Title of page
-		$title = _MA_MYMODULE2_SUBMIT_PROPOSER . ' - ';
-		$title .= $GLOBALS['xoopsModule']->name();
-		$GLOBALS['xoopsTpl']->assign('xoops_pagetitle', $title);
-		// Description
-		$GLOBALS['xoTheme']->addMeta( 'meta', 'description', strip_tags(_MA_MYMODULE2_SUBMIT_PROPOSER));
-		// Form Create
-		$testfieldsObj = $testfieldsHandler->create();
-		$form = $testfieldsObj->getFormTestfields();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
+    case 'form':
+    default:
+        // Navigation
+        $navigation = _MA_MYMODULE2_SUBMIT_PROPOSER;
+        $GLOBALS['xoopsTpl']->assign('navigation', $navigation);
+        // Title of page
+        $title = _MA_MYMODULE2_SUBMIT_PROPOSER . ' - ';
+        $title .= $GLOBALS['xoopsModule']->name();
+        $GLOBALS['xoopsTpl']->assign('xoops_pagetitle', $title);
+        // Description
+        $GLOBALS['xoTheme']->addMeta( 'meta', 'description', strip_tags(_MA_MYMODULE2_SUBMIT_PROPOSER));
+        // Form Create
+        $testfieldsObj = $testfieldsHandler->create();
+        $form = $testfieldsObj->getFormTestfields();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
 
-	break;
-	case 'save':
-		// Security Check
-		if ($GLOBALS['xoopsSecurity']->check()) {
-			redirect_header('testfields.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		$testfieldsObj = $testfieldsHandler->create();
-		$error = false;
-		$errorMessage = '';
-		// Test first the validation
-		xoops_load('captcha');
-		$xoopsCaptcha = \XoopsCaptcha::getInstance();
-		if (!$xoopsCaptcha->verify()) {
-			$errorMessage .= $xoopsCaptcha->getMessage().'<br>';
-			$error = true;
-		}
-		$testfieldsObj->setVar('tf_text', Request::getString('tf_text', ''));
-		$testfieldsObj->setVar('tf_textarea', Request::getString('tf_textarea', ''));
-		$testfieldsObj->setVar('tf_dhtml', Request::getString('tf_dhtml', ''));
-		$testfieldsObj->setVar('tf_checkbox', Request::getInt('tf_checkbox', 0));
-		$testfieldsObj->setVar('tf_yesno', Request::getInt('tf_yesno', 0));
-		$testfieldsObj->setVar('tf_selectbox', Request::getString('tf_selectbox', ''));
-		$testfieldsObj->setVar('tf_user', Request::getInt('tf_user', 0));
-		$testfieldsObj->setVar('tf_color', Request::getString('tf_color', ''));
-		// Set Var tf_imagelist
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$uploader = new \XoopsMediaUploader(XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32', 
-													$helper->getConfig('mimetypes_image'), 
-													$helper->getConfig('maxsize_image'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][0])) {
-			//$uploader->setPrefix(tf_imagelist_);
-			//$uploader->fetchMedia($_POST['xoops_upload_file'][0]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-				redirect_header('javascript:history.go(-1).php', 3, $errors);
-			} else {
-				$testfieldsObj->setVar('tf_imagelist', $uploader->getSavedFileName());
-			}
-		} else {
-			$testfieldsObj->setVar('tf_imagelist', Request::getString('tf_imagelist'));
-		}
-		$testfieldsObj->setVar('tf_urlfile', formatUrl($_REQUEST['tf_urlfile']));
-		// Set Var tf_urlfile
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_urlfile']['name'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_file'), 
-													$helper->getConfig('maxsize_file'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][1])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][1]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-			} else {
-				$testfieldsObj->setVar('tf_urlfile', $uploader->getSavedFileName());
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_urlfile', Request::getString('tf_urlfile'));
-		}
-		// Set Var tf_uplimage
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_uplimage']['name'];
-		$imgMimetype    = $_FILES['tf_uplimage']['type'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploaderErrors = '';
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_image'), 
-													$helper->getConfig('maxsize_image'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][2])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][2]);
-			if (!$uploader->upload()) {
-				$uploaderErrors = $uploader->getErrors();
-			} else {
-				$savedFilename = $uploader->getSavedFileName();
-				$maxwidth  = (int)$helper->getConfig('maxwidth_image');
-				$maxheight = (int)$helper->getConfig('maxheight_image');
-				if ($maxwidth > 0 && $maxheight > 0) {
-					// Resize image
-					$imgHandler                = new Mymodule2\Common\Resizer();
-					$imgHandler->sourceFile    = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
-					$imgHandler->endFile       = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
-					$imgHandler->imageMimetype = $imgMimetype;
-					$imgHandler->maxWidth      = $maxwidth;
-					$imgHandler->maxHeight     = $maxheight;
-					$result                    = $imgHandler->resizeImage();
-				}
-				$testfieldsObj->setVar('tf_uplimage', $savedFilename);
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_uplimage', Request::getString('tf_uplimage'));
-		}
-		// Set Var tf_uplfile
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_uplfile']['name'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_file'), 
-													$helper->getConfig('maxsize_file'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][3])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][3]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-			} else {
-				$testfieldsObj->setVar('tf_uplfile', $uploader->getSavedFileName());
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_uplfile', Request::getString('tf_uplfile'));
-		}
-		$testfieldTextdateselect = date_create_from_format(_SHORTDATESTRING, Request::getString('tf_textdateselect'));
-		$testfieldsObj->setVar('tf_textdateselect', $testfieldTextdateselect->getTimestamp());
-		// Set Var tf_selectfile
-		include_once XOOPS_ROOT_PATH . '/class/uploader.php';
-		$filename       = $_FILES['tf_selectfile']['name'];
-		$imgNameDef     = Request::getString('tf_text');
-		$uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
-													$helper->getConfig('mimetypes_file'), 
-													$helper->getConfig('maxsize_file'), null, null);
-		if ($uploader->fetchMedia($_POST['xoops_upload_file'][4])) {
-			$extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
-			$imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
-			$uploader->setPrefix($imgName);
-			$uploader->fetchMedia($_POST['xoops_upload_file'][4]);
-			if (!$uploader->upload()) {
-				$errors = $uploader->getErrors();
-			} else {
-				$testfieldsObj->setVar('tf_selectfile', $uploader->getSavedFileName());
-			}
-		} else {
-			if ($filename > '') {
-				$uploaderErrors = $uploader->getErrors();
-			}
-			$testfieldsObj->setVar('tf_selectfile', Request::getString('tf_selectfile'));
-		}
-		$testfieldsObj->setVar('tf_status', Request::getInt('tf_status', 0));
-		if ($error === true) {
-			$GLOBALS['xoopsTpl']->assign('error_message', $errorMessage);
-		} else {
-			// Insert Data
-			if ($testfields1->insert($testfieldsObj)) {
-				redirect_header('index.php', 2, _MA_MYMODULE2_FORM_OK);
-			}
-		}
-		// Get Form Error
-		$GLOBALS['xoopsTpl']->assign('error', $testfieldsObj->getHtmlErrors());
-		$form = $testfieldsObj->getFormTestfields();
-		$GLOBALS['xoopsTpl']->assign('form', $form->display());
+    break;
+    case 'save':
+        // Security Check
+        if ($GLOBALS['xoopsSecurity']->check()) {
+            redirect_header('testfields.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        $testfieldsObj = $testfieldsHandler->create();
+        $error = false;
+        $errorMessage = '';
+        // Test first the validation
+        xoops_load('captcha');
+        $xoopsCaptcha = \XoopsCaptcha::getInstance();
+        if (!$xoopsCaptcha->verify()) {
+            $errorMessage .= $xoopsCaptcha->getMessage().'<br>';
+            $error = true;
+        }
+        $testfieldsObj->setVar('tf_text', Request::getString('tf_text', ''));
+        $testfieldsObj->setVar('tf_textarea', Request::getString('tf_textarea', ''));
+        $testfieldsObj->setVar('tf_dhtml', Request::getString('tf_dhtml', ''));
+        $testfieldsObj->setVar('tf_checkbox', Request::getInt('tf_checkbox', 0));
+        $testfieldsObj->setVar('tf_yesno', Request::getInt('tf_yesno', 0));
+        $testfieldsObj->setVar('tf_selectbox', Request::getString('tf_selectbox', ''));
+        $testfieldsObj->setVar('tf_user', Request::getInt('tf_user', 0));
+        $testfieldsObj->setVar('tf_color', Request::getString('tf_color', ''));
+        // Set Var tf_imagelist
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $uploader = new \XoopsMediaUploader(XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32', 
+                                                    $helper->getConfig('mimetypes_image'), 
+                                                    $helper->getConfig('maxsize_image'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][0])) {
+            //$uploader->setPrefix(tf_imagelist_);
+            //$uploader->fetchMedia($_POST['xoops_upload_file'][0]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+                redirect_header('javascript:history.go(-1).php', 3, $errors);
+            } else {
+                $testfieldsObj->setVar('tf_imagelist', $uploader->getSavedFileName());
+            }
+        } else {
+            $testfieldsObj->setVar('tf_imagelist', Request::getString('tf_imagelist'));
+        }
+        $testfieldsObj->setVar('tf_urlfile', formatUrl($_REQUEST['tf_urlfile']));
+        // Set Var tf_urlfile
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_urlfile']['name'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_file'), 
+                                                    $helper->getConfig('maxsize_file'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][1])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][1]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+            } else {
+                $testfieldsObj->setVar('tf_urlfile', $uploader->getSavedFileName());
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_urlfile', Request::getString('tf_urlfile'));
+        }
+        // Set Var tf_uplimage
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_uplimage']['name'];
+        $imgMimetype    = $_FILES['tf_uplimage']['type'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploaderErrors = '';
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_image'), 
+                                                    $helper->getConfig('maxsize_image'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][2])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][2]);
+            if (!$uploader->upload()) {
+                $uploaderErrors = $uploader->getErrors();
+            } else {
+                $savedFilename = $uploader->getSavedFileName();
+                $maxwidth  = (int)$helper->getConfig('maxwidth_image');
+                $maxheight = (int)$helper->getConfig('maxheight_image');
+                if ($maxwidth > 0 && $maxheight > 0) {
+                    // Resize image
+                    $imgHandler                = new Mymodule2\Common\Resizer();
+                    $imgHandler->sourceFile    = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
+                    $imgHandler->endFile       = MYMODULE2_UPLOAD_IMAGE_PATH . '/testfields/' . $savedFilename;
+                    $imgHandler->imageMimetype = $imgMimetype;
+                    $imgHandler->maxWidth      = $maxwidth;
+                    $imgHandler->maxHeight     = $maxheight;
+                    $result                    = $imgHandler->resizeImage();
+                }
+                $testfieldsObj->setVar('tf_uplimage', $savedFilename);
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_uplimage', Request::getString('tf_uplimage'));
+        }
+        // Set Var tf_uplfile
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_uplfile']['name'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_file'), 
+                                                    $helper->getConfig('maxsize_file'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][3])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][3]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+            } else {
+                $testfieldsObj->setVar('tf_uplfile', $uploader->getSavedFileName());
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_uplfile', Request::getString('tf_uplfile'));
+        }
+        $testfieldTextdateselect = date_create_from_format(_SHORTDATESTRING, Request::getString('tf_textdateselect'));
+        $testfieldsObj->setVar('tf_textdateselect', $testfieldTextdateselect->getTimestamp());
+        // Set Var tf_selectfile
+        include_once XOOPS_ROOT_PATH . '/class/uploader.php';
+        $filename       = $_FILES['tf_selectfile']['name'];
+        $imgNameDef     = Request::getString('tf_text');
+        $uploader = new \XoopsMediaUploader(MYMODULE2_UPLOAD_FILES_PATH . '/testfields/', 
+                                                    $helper->getConfig('mimetypes_file'), 
+                                                    $helper->getConfig('maxsize_file'), null, null);
+        if ($uploader->fetchMedia($_POST['xoops_upload_file'][4])) {
+            $extension = preg_replace('/^.+\.([^.]+)$/sU', '', $filename);
+            $imgName = str_replace(' ', '', $imgNameDef) . '.' . $extension;
+            $uploader->setPrefix($imgName);
+            $uploader->fetchMedia($_POST['xoops_upload_file'][4]);
+            if (!$uploader->upload()) {
+                $errors = $uploader->getErrors();
+            } else {
+                $testfieldsObj->setVar('tf_selectfile', $uploader->getSavedFileName());
+            }
+        } else {
+            if ($filename > '') {
+                $uploaderErrors = $uploader->getErrors();
+            }
+            $testfieldsObj->setVar('tf_selectfile', Request::getString('tf_selectfile'));
+        }
+        $testfieldsObj->setVar('tf_status', Request::getInt('tf_status', 0));
+        if ($error === true) {
+            $GLOBALS['xoopsTpl']->assign('error_message', $errorMessage);
+        } else {
+            // Insert Data
+            if ($testfields1->insert($testfieldsObj)) {
+                redirect_header('index.php', 2, _MA_MYMODULE2_FORM_OK);
+            }
+        }
+        // Get Form Error
+        $GLOBALS['xoopsTpl']->assign('error', $testfieldsObj->getHtmlErrors());
+        $form = $testfieldsObj->getFormTestfields();
+        $GLOBALS['xoopsTpl']->assign('form', $form->display());
 
-	break;
+    break;
 }
 require __DIR__ . '/footer.php';

testdata/mymodule2/index.php

Indentation +46 added lines, -46 removed lines

@@ -38,29 +38,29 @@ 
 $GLOBALS['xoopsTpl']->assign('articlesCount', $articlesCount);
 $count = 1;
 if ($articlesCount > 0) {
-	$start = Request::getInt('start', 0);
-	$limit = Request::getInt('limit', $helper->getConfig('userpager'));
-	$articlesAll = $articlesHandler->getAllArticles($start, $limit);
-	// Get All Articles
-	$articles = [];
-	foreach(array_keys($articlesAll) as $i) {
-		$article = $articlesAll[$i]->getValuesArticles();
-		$acount = ['count', $count];
-		$articles[] = array_merge($article, $acount);
-		$keywords[] = $articlesAll[$i]->getVar('tf_text');
-		++$count;
-	}
-	$GLOBALS['xoopsTpl']->assign('articles', $articles);
-	unset($articles);
-	// Display Navigation
-	if ($articlesCount > $limit) {
-		include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-		$pagenav = new \XoopsPageNav($articlesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-		$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-	}
-	$GLOBALS['xoopsTpl']->assign('lang_thereare', sprintf(_MA_MYMODULE2_INDEX_THEREARE, $articlesCount));
-	$GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
-	$GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
+    $start = Request::getInt('start', 0);
+    $limit = Request::getInt('limit', $helper->getConfig('userpager'));
+    $articlesAll = $articlesHandler->getAllArticles($start, $limit);
+    // Get All Articles
+    $articles = [];
+    foreach(array_keys($articlesAll) as $i) {
+        $article = $articlesAll[$i]->getValuesArticles();
+        $acount = ['count', $count];
+        $articles[] = array_merge($article, $acount);
+        $keywords[] = $articlesAll[$i]->getVar('tf_text');
+        ++$count;
+    }
+    $GLOBALS['xoopsTpl']->assign('articles', $articles);
+    unset($articles);
+    // Display Navigation
+    if ($articlesCount > $limit) {
+        include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+        $pagenav = new \XoopsPageNav($articlesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+        $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+    }
+    $GLOBALS['xoopsTpl']->assign('lang_thereare', sprintf(_MA_MYMODULE2_INDEX_THEREARE, $articlesCount));
+    $GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
+    $GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
 }
 unset($count);
 $GLOBALS['xoopsTpl']->assign('table_type', $helper->getConfig('table_type'));
@@ -72,29 +72,29 @@ 
 $GLOBALS['xoopsTpl']->assign('testfieldsCount', $testfieldsCount);
 $count = 1;
 if ($testfieldsCount > 0) {
-	$start = Request::getInt('start', 0);
-	$limit = Request::getInt('limit', $helper->getConfig('userpager'));
-	$testfieldsAll = $testfieldsHandler->getAllTestfields($start, $limit);
-	// Get All Testfields
-	$testfields = [];
-	foreach(array_keys($testfieldsAll) as $i) {
-		$testfield = $testfieldsAll[$i]->getValuesTestfields();
-		$acount = ['count', $count];
-		$testfields[] = array_merge($testfield, $acount);
-		$keywords[] = $testfieldsAll[$i]->getVar('tf_text');
-		++$count;
-	}
-	$GLOBALS['xoopsTpl']->assign('testfields', $testfields);
-	unset($testfields);
-	// Display Navigation
-	if ($testfieldsCount > $limit) {
-		include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-		$pagenav = new \XoopsPageNav($testfieldsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-		$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-	}
-	$GLOBALS['xoopsTpl']->assign('lang_thereare', sprintf(_MA_MYMODULE2_INDEX_THEREARE, $testfieldsCount));
-	$GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
-	$GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
+    $start = Request::getInt('start', 0);
+    $limit = Request::getInt('limit', $helper->getConfig('userpager'));
+    $testfieldsAll = $testfieldsHandler->getAllTestfields($start, $limit);
+    // Get All Testfields
+    $testfields = [];
+    foreach(array_keys($testfieldsAll) as $i) {
+        $testfield = $testfieldsAll[$i]->getValuesTestfields();
+        $acount = ['count', $count];
+        $testfields[] = array_merge($testfield, $acount);
+        $keywords[] = $testfieldsAll[$i]->getVar('tf_text');
+        ++$count;
+    }
+    $GLOBALS['xoopsTpl']->assign('testfields', $testfields);
+    unset($testfields);
+    // Display Navigation
+    if ($testfieldsCount > $limit) {
+        include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+        $pagenav = new \XoopsPageNav($testfieldsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+        $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+    }
+    $GLOBALS['xoopsTpl']->assign('lang_thereare', sprintf(_MA_MYMODULE2_INDEX_THEREARE, $testfieldsCount));
+    $GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
+    $GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
 }
 unset($count);
 $GLOBALS['xoopsTpl']->assign('table_type', $helper->getConfig('table_type'));

testdata/mymodule2/print.php

Indentation +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
 // Define Stylesheet
 $GLOBALS['xoTheme']->addStylesheet( $style, null );
 if (empty($tfId)) {
-	redirect_header(MYMODULE2_URL . '/index.php', 2, _MA_MYMODULE2_NOTFID);
+    redirect_header(MYMODULE2_URL . '/index.php', 2, _MA_MYMODULE2_NOTFID);
 }
 // Get Instance of Handler
 $testfieldsHandler = $helper->getHandler('testfields');
@@ -37,12 +37,12 @@ 
 $testfields = $testfieldsHandler->get($tfId);
 // Verify permissions
 if (!$grouppermHandler->checkRight('mymodule2_view', $tfId->getVar('tf_id'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
-	redirect_header(MYMODULE2_URL . '/index.php', 3, _NOPERM);
-	exit();
+    redirect_header(MYMODULE2_URL . '/index.php', 3, _NOPERM);
+    exit();
 }
 $testfield = $testfields->getValuesTestfields();
 foreach($testfield as $k => $v) {
-	$GLOBALS['xoopsTpl']->append('"{$k}"', $v);
+    $GLOBALS['xoopsTpl']->append('"{$k}"', $v);
 }
 $GLOBALS['xoopsTpl']->assign('xoops_sitename', $GLOBALS['xoopsConfig']['sitename']);
 $GLOBALS['xoopsTpl']->assign('xoops_pagetitle', strip_tags($testfield->getVar('tf_text') - _MA_MYMODULE2_PRINT - $GLOBALS['xoopsModule']->name()));