mambax7 / instruction

admin/index.php

Indentation +2 added lines, -2 removed lines

@@ -11,8 +11,8 @@
 
 $configurator = include __DIR__ . '/../include/config.php';
 foreach (array_keys($configurator->uploadFolders) as $i) {
-    $utility::createFolder($configurator->uploadFolders[$i]);
-    $adminObject->addConfigBoxLine($configurator->uploadFolders[$i], 'folder');
+	$utility::createFolder($configurator->uploadFolders[$i]);
+	$adminObject->addConfigBoxLine($configurator->uploadFolders[$i], 'folder');
 }
 
 //$instructionHandler = xoops_getModuleHandler('instruction', 'instruction');

admin/menu.php

Indentation +31 added lines, -31 removed lines

@@ -3,7 +3,7 @@ 
 use Xoopsmodules\instruction;
 
 if (!isset($moduleDirName)) {
-    $moduleDirName = basename(dirname(__DIR__));
+	$moduleDirName = basename(dirname(__DIR__));
 }
 
 /** @var Xmf\Module\Helper $moduleHelper */
@@ -17,34 +17,34 @@ 
 
 // Административное меню
 $adminmenu = [
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_HOME,
-        'link'  => 'admin/index.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_HOME_DESC,
-        'icon'  => $pathIcon32 . '/home.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_CAT,
-        'link'  => 'admin/cat.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_CAT_DESC,
-        'icon'  => $pathIcon32 . '/category.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_INSTR,
-        'link'  => 'admin/instr.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_INSTR_DESC,
-        'icon'  => $pathModIcon32 . '/nav_book.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_PERM,
-        'link'  => 'admin/perm.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_PERM_DESC,
-        'icon'  => $pathIcon32 . '/permissions.png'
-    ],
-    [
-        'title' => _MI_INSTRUCTION_ADMIN_ABOUT,
-        'link'  => 'admin/about.php',
-        'desc'  => _MI_INSTRUCTION_ADMIN_ABOUT_DESC,
-        'icon'  => $pathIcon32 . '/about.png'
-    ]
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_HOME,
+		'link'  => 'admin/index.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_HOME_DESC,
+		'icon'  => $pathIcon32 . '/home.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_CAT,
+		'link'  => 'admin/cat.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_CAT_DESC,
+		'icon'  => $pathIcon32 . '/category.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_INSTR,
+		'link'  => 'admin/instr.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_INSTR_DESC,
+		'icon'  => $pathModIcon32 . '/nav_book.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_PERM,
+		'link'  => 'admin/perm.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_PERM_DESC,
+		'icon'  => $pathIcon32 . '/permissions.png'
+	],
+	[
+		'title' => _MI_INSTRUCTION_ADMIN_ABOUT,
+		'link'  => 'admin/about.php',
+		'desc'  => _MI_INSTRUCTION_ADMIN_ABOUT_DESC,
+		'icon'  => $pathIcon32 . '/about.png'
+	]
 ];

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 use Xoopsmodules\instruction;
 
-if (!isset($moduleDirName)) {
+if ( ! isset($moduleDirName)) {
     $moduleDirName = basename(dirname(__DIR__));
 }
 

admin/ajax.php

Indentation +174 added lines, -174 removed lines

@@ -22,178 +22,178 @@
 
 // Выбор
 switch ($op) {
-    // Сохранение страницы
-    case 'savepage':
-        // Выходной массив
-        $ret = [];
-
-        // Ошибки
-        $err         = false;
-        $message_err = '';
-
-        //
-        $title       = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'title', '', 'string');
-        $pid         = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'pid', 0, 'int');
-        $weight      = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'weight', 0, 'int');
-        $hometext    = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'hometext', '', 'string');
-        $footnote    = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'footnote', '', 'string');
-        $status      = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'status', 0, 'int');
-        $type        = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'type', 0, 'int');
-        $keywords    = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'keywords', '', 'string');
-        $description = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'description', '', 'string');
-        $dosmiley    = (isset($_POST['dosmiley']) && (int)$_POST['dosmiley'] > 0) ? 1 : 0;
-        $doxcode     = (isset($_POST['doxcode']) && (int)$_POST['doxcode'] > 0) ? 1 : 0;
-        $dobr        = (isset($_POST['dobr']) && (int)$_POST['dobr'] > 0) ? 1 : 0;
-        $dohtml      = (isset($_POST['dohtml']) && (int)$_POST['dohtml'] > 0) ? 1 : 0;
-        //$dohtml      = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dohtml', 0, 'int' );
-        //$dosmiley    = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dosmiley', 0, 'int' );
-        //$doxcode     = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'doxcode', 0, 'int' );
-        //$dobr        = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dobr', 0, 'int' );
-        $pageid  = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'pageid', 0, 'int');
-        $instrid = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'instrid', 0, 'int');
-
-        // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            $err         = true;
-            $err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
-            $message_err .= $err_txt . '<br>' . _AM_INSTR_TRY_AGAIN . '<br>';
-        }
-        // =========================
-        // Устанавливаем новый token
-        $token_name    = 'XOOPS_TOKEN';
-        $token_timeout = 0;
-        // $token_name . '_REQUEST' - название элемента формы
-        $ret['toket'] = $GLOBALS['xoopsSecurity']->createToken($token_timeout, $token_name);
-        // =========================
-
-        // Если мы редактируем
-        if ($pageid) {
-            $objInspage = $pageHandler->get($pageid);
-        } elseif ($instrid) {
-            $objInspage = $pageHandler->create();
-            // Если мы создаём страницу необходимо указать к какой инструкции
-            $objInspage->setVar('instrid', $instrid);
-            // Указываем дату создания
-            $objInspage->setVar('datecreated', $time);
-            // Указываем пользователя
-            $objInspage->setVar('uid', $uid);
-        } else {
-            // Устанавливаем сообщение
-            $ret['message'] = _AM_INSTRUCTION_BADREQUEST;
-            // Возвращаем ответ скрипту через JSON
-            echo json_encode($ret);
-            // Прерываем выполнение
-            exit();
-        }
-
-        // Родительская страница
-        $objInspage->setVar('pid', $pid);
-        // Дата обновления
-        $objInspage->setVar('dateupdated', $time);
-        // Название
-        $objInspage->setVar('title', $title);
-        // Вес
-        $objInspage->setVar('weight', $weight);
-        // Текст
-        $objInspage->setVar('hometext', $hometext);
-        // Сноска
-        $objInspage->setVar('footnote', $footnote);
-        // Статус
-        $objInspage->setVar('status', $status);
-        // Тип
-        $objInspage->setVar('type', $type);
-        // Мета-теги ключевых слов
-        $objInspage->setVar('keywords', $keywords);
-        // Мета-теги описания
-        $objInspage->setVar('description', $description);
-        //
-        $objInspage->setVar('dohtml', $dohtml);
-        $objInspage->setVar('dosmiley', $dosmiley);
-        $objInspage->setVar('doxcode', $doxcode);
-        $objInspage->setVar('dobr', $dobr);
-
-        // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
-        }
-        // Проверка веса
-        if (0 == $weight) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
-        }
-        // Проверка родительской страницы
-        if ($pageid && ($pageid == $pid)) {
-            $err         = true;
-            $message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
-        }
-        // Проверка названия
-        if (!$title) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
-        }
-        // Проверка основного текста
-        if (!$hometext) {
-            $err         = true;
-            $message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
-        }
-
-        // Если были ошибки
-        if (true === $err) {
-            //
-            $message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
-            // Устанавливаем сообщение
-            $ret['message'] = $message_err;
-            // Возвращаем ответ скрипту через JSON
-            echo json_encode($ret);
-            // Прерываем выполнение
-            exit();
-            // Если небыло ошибок
-        } else {
-            // Вставляем данные в БД
-            if ($pageHandler->insert($objInspage)) {
-                // Находим ID созданной записи
-                $pageid_new = $pageid ?: $objInspage->get_new_enreg();
-                //
-                $ret['pageid'] = $pageid_new;
-                // Получаем ID инструкции
-                $instrid = $objInspage->getInstrid();
-                // Обновляем в инструкции число страниц и дату
-                $instructionHandler->updatePages($instrid);
-                // Если мы редактируем
-                if ($pageid) {
-                    // Устанавливаем сообщение
-                    $ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEMODIFY . '</div>';
-                    // Возвращаем ответ скрипту через JSON
-                    echo json_encode($ret);
-                    // Прерываем выполнение
-                    exit();
-
-                    // Если мы добавляем
-                } else {
-                    // Инкримент комментов
-                    $pageHandler->updateposts($uid, $status, 'add');
-
-                    // Устанавливаем сообщение
-                    $ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEADDED . '</div>';
-                    // Возвращаем ответ скрипту через JSON
-                    echo json_encode($ret);
-                    // Прерываем выполнение
-                    exit();
-                }
-                // Если не получилось вставить данные
-            } else {
-
-                //
-                $message_err = '<div class="errorMsg" style="text-align: left;">' . $objInspage->getHtmlErrors() . '</div>';
-                // Устанавливаем сообщение
-                $ret['message'] = $message_err;
-                // Возвращаем ответ скрипту через JSON
-                echo json_encode($ret);
-                // Прерываем выполнение
-                exit();
-            }
-        }
-
-        break;
+	// Сохранение страницы
+	case 'savepage':
+		// Выходной массив
+		$ret = [];
+
+		// Ошибки
+		$err         = false;
+		$message_err = '';
+
+		//
+		$title       = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'title', '', 'string');
+		$pid         = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'pid', 0, 'int');
+		$weight      = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'weight', 0, 'int');
+		$hometext    = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'hometext', '', 'string');
+		$footnote    = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'footnote', '', 'string');
+		$status      = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'status', 0, 'int');
+		$type        = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'type', 0, 'int');
+		$keywords    = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'keywords', '', 'string');
+		$description = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'description', '', 'string');
+		$dosmiley    = (isset($_POST['dosmiley']) && (int)$_POST['dosmiley'] > 0) ? 1 : 0;
+		$doxcode     = (isset($_POST['doxcode']) && (int)$_POST['doxcode'] > 0) ? 1 : 0;
+		$dobr        = (isset($_POST['dobr']) && (int)$_POST['dobr'] > 0) ? 1 : 0;
+		$dohtml      = (isset($_POST['dohtml']) && (int)$_POST['dohtml'] > 0) ? 1 : 0;
+		//$dohtml      = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dohtml', 0, 'int' );
+		//$dosmiley    = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dosmiley', 0, 'int' );
+		//$doxcode     = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'doxcode', 0, 'int' );
+		//$dobr        = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dobr', 0, 'int' );
+		$pageid  = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'pageid', 0, 'int');
+		$instrid = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'instrid', 0, 'int');
+
+		// Проверка
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			$err         = true;
+			$err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
+			$message_err .= $err_txt . '<br>' . _AM_INSTR_TRY_AGAIN . '<br>';
+		}
+		// =========================
+		// Устанавливаем новый token
+		$token_name    = 'XOOPS_TOKEN';
+		$token_timeout = 0;
+		// $token_name . '_REQUEST' - название элемента формы
+		$ret['toket'] = $GLOBALS['xoopsSecurity']->createToken($token_timeout, $token_name);
+		// =========================
+
+		// Если мы редактируем
+		if ($pageid) {
+			$objInspage = $pageHandler->get($pageid);
+		} elseif ($instrid) {
+			$objInspage = $pageHandler->create();
+			// Если мы создаём страницу необходимо указать к какой инструкции
+			$objInspage->setVar('instrid', $instrid);
+			// Указываем дату создания
+			$objInspage->setVar('datecreated', $time);
+			// Указываем пользователя
+			$objInspage->setVar('uid', $uid);
+		} else {
+			// Устанавливаем сообщение
+			$ret['message'] = _AM_INSTRUCTION_BADREQUEST;
+			// Возвращаем ответ скрипту через JSON
+			echo json_encode($ret);
+			// Прерываем выполнение
+			exit();
+		}
+
+		// Родительская страница
+		$objInspage->setVar('pid', $pid);
+		// Дата обновления
+		$objInspage->setVar('dateupdated', $time);
+		// Название
+		$objInspage->setVar('title', $title);
+		// Вес
+		$objInspage->setVar('weight', $weight);
+		// Текст
+		$objInspage->setVar('hometext', $hometext);
+		// Сноска
+		$objInspage->setVar('footnote', $footnote);
+		// Статус
+		$objInspage->setVar('status', $status);
+		// Тип
+		$objInspage->setVar('type', $type);
+		// Мета-теги ключевых слов
+		$objInspage->setVar('keywords', $keywords);
+		// Мета-теги описания
+		$objInspage->setVar('description', $description);
+		//
+		$objInspage->setVar('dohtml', $dohtml);
+		$objInspage->setVar('dosmiley', $dosmiley);
+		$objInspage->setVar('doxcode', $doxcode);
+		$objInspage->setVar('dobr', $dobr);
+
+		// Проверка категорий
+		if (!$pageid && !$instrid) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
+		}
+		// Проверка веса
+		if (0 == $weight) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
+		}
+		// Проверка родительской страницы
+		if ($pageid && ($pageid == $pid)) {
+			$err         = true;
+			$message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
+		}
+		// Проверка названия
+		if (!$title) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_TITLE . '<br>';
+		}
+		// Проверка основного текста
+		if (!$hometext) {
+			$err         = true;
+			$message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
+		}
+
+		// Если были ошибки
+		if (true === $err) {
+			//
+			$message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
+			// Устанавливаем сообщение
+			$ret['message'] = $message_err;
+			// Возвращаем ответ скрипту через JSON
+			echo json_encode($ret);
+			// Прерываем выполнение
+			exit();
+			// Если небыло ошибок
+		} else {
+			// Вставляем данные в БД
+			if ($pageHandler->insert($objInspage)) {
+				// Находим ID созданной записи
+				$pageid_new = $pageid ?: $objInspage->get_new_enreg();
+				//
+				$ret['pageid'] = $pageid_new;
+				// Получаем ID инструкции
+				$instrid = $objInspage->getInstrid();
+				// Обновляем в инструкции число страниц и дату
+				$instructionHandler->updatePages($instrid);
+				// Если мы редактируем
+				if ($pageid) {
+					// Устанавливаем сообщение
+					$ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEMODIFY . '</div>';
+					// Возвращаем ответ скрипту через JSON
+					echo json_encode($ret);
+					// Прерываем выполнение
+					exit();
+
+					// Если мы добавляем
+				} else {
+					// Инкримент комментов
+					$pageHandler->updateposts($uid, $status, 'add');
+
+					// Устанавливаем сообщение
+					$ret['message'] = '<div class="successMsg" style="text-align: left;">' . _AM_INSTRUCTION_PAGEADDED . '</div>';
+					// Возвращаем ответ скрипту через JSON
+					echo json_encode($ret);
+					// Прерываем выполнение
+					exit();
+				}
+				// Если не получилось вставить данные
+			} else {
+
+				//
+				$message_err = '<div class="errorMsg" style="text-align: left;">' . $objInspage->getHtmlErrors() . '</div>';
+				// Устанавливаем сообщение
+				$ret['message'] = $message_err;
+				// Возвращаем ответ скрипту через JSON
+				echo json_encode($ret);
+				// Прерываем выполнение
+				exit();
+			}
+		}
+
+		break;
 }

Spacing +13 added lines, -13 removed lines

@@ -41,10 +41,10 @@ 
         $type        = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'type', 0, 'int');
         $keywords    = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'keywords', '', 'string');
         $description = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'description', '', 'string');
-        $dosmiley    = (isset($_POST['dosmiley']) && (int)$_POST['dosmiley'] > 0) ? 1 : 0;
-        $doxcode     = (isset($_POST['doxcode']) && (int)$_POST['doxcode'] > 0) ? 1 : 0;
-        $dobr        = (isset($_POST['dobr']) && (int)$_POST['dobr'] > 0) ? 1 : 0;
-        $dohtml      = (isset($_POST['dohtml']) && (int)$_POST['dohtml'] > 0) ? 1 : 0;
+        $dosmiley    = (isset($_POST['dosmiley']) && (int) $_POST['dosmiley'] > 0) ? 1 : 0;
+        $doxcode     = (isset($_POST['doxcode']) && (int) $_POST['doxcode'] > 0) ? 1 : 0;
+        $dobr        = (isset($_POST['dobr']) && (int) $_POST['dobr'] > 0) ? 1 : 0;
+        $dohtml      = (isset($_POST['dohtml']) && (int) $_POST['dohtml'] > 0) ? 1 : 0;
         //$dohtml      = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dohtml', 0, 'int' );
         //$dosmiley    = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'dosmiley', 0, 'int' );
         //$doxcode     = Xoopsmodules\instruction\Utility::cleanVars( $_POST, 'doxcode', 0, 'int' );
@@ -53,7 +53,7 @@ 
         $instrid = Xoopsmodules\instruction\Utility::cleanVars($_POST, 'instrid', 0, 'int');
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             $err         = true;
             $err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
             $message_err .= $err_txt . '<br>' . _AM_INSTR_TRY_AGAIN . '<br>';
@@ -113,28 +113,28 @@ 
         $objInspage->setVar('dobr', $dobr);
 
         // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
+        if ( ! $pageid && ! $instrid) {
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_INSTR . '<br>';
         }
         // Проверка веса
         if (0 == $weight) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
         }
         // Проверка родительской страницы
         if ($pageid && ($pageid == $pid)) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_PPAGE . '<br>';
         }
         // Проверка названия
-        if (!$title) {
-            $err         = true;
+        if ( ! $title) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
         }
         // Проверка основного текста
-        if (!$hometext) {
-            $err         = true;
+        if ( ! $hometext) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
         }
 

submit.php

Indentation +191 added lines, -191 removed lines

@@ -37,195 +37,195 @@
 
 switch ($op) {
 
-    case 'editpage':
-
-        // Задание тайтла
-        $xoopsOption['xoops_pagetitle'] = '';
-        // Шаблон
-        $GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_editpage.tpl';
-        // Заголовок
-        include_once $GLOBALS['xoops']->path('header.php');
-
-        // Если мы редактируем страницу
-        if ($pageid) {
-            // Получаем объект страницы
-            $objInspage = $pageHandler->get($pageid);
-            // ID инструкции
-            $instrid = $objInspage->getVar('instrid');
-            // Объект инструкции
-            $objInsinstr = $instructionHandler->get($instrid);
-            // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
-            }
-            // Создание новой страницы
-        } elseif ($instrid) {
-
-            // Если нельзя добавлять не в одну категорию
-            //if( ! count( $cat_submit ) ) redirect_header( 'index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMIT_PAGE );
-            // Создаём объект страницы
-            $objInspage = $pageHandler->create();
-            // Объект инструкции
-            $objInsinstr = $instructionHandler->get($instrid);
-            // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
-            }
-        } else {
-            redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
-        }
-
-        // Информация об инструкции
-
-        // Массив данных об инструкции
-        $instrs = [];
-        // ID инструкции
-        $instrs['instrid'] = $objInsinstr->getVar('instrid');
-        // Название страницы
-        $instrs['title'] = $objInsinstr->getVar('title');
-        // Описание
-        $instrs['description'] = $objInsinstr->getVar('description');
-
-        // Выводим в шаблон
-        $GLOBALS['xoopsTpl']->assign('insInstr', $instrs);
-
-        //
-
-        $form = $objInspage->getForm('submit.php', $instrid);
-        // Форма
-        $GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
-
-        // Подвал
-        include_once $GLOBALS['xoops']->path('footer.php');
-
-        break;
-    // Сохранение страницы
-    case 'savepage':
-
-        // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
-            redirect_header('index.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-        }
-
-        $err         = false;
-        $message_err = '';
-
-        // Если мы редактируем
-        if ($pageid) {
-            $objInspage = $pageHandler->get($pageid);
-            // Объект инструкции
-            $objInsinstr = $instructionHandler->get($objInspage->getVar('instrid'));
-            // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
-            }
-        } elseif ($instrid) {
-            $objInspage = $pageHandler->create();
-            // Объект инструкции
-            $objInsinstr = $instructionHandler->get($instrid);
-            // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
-                redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
-            }
-
-            // Если мы создаём страницу необходимо указать к какой инструкции
-            $objInspage->setVar('instrid', $instrid);
-            // Указываем дату создания
-            $objInspage->setVar('datecreated', $time);
-            // Указываем пользователя
-            $objInspage->setVar('uid', $uid);
-        } else {
-            redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
-        }
-
-        // Родительская страница
-        $objInspage->setVar('pid', $pid);
-        // Дата обновления
-        $objInspage->setVar('dateupdated', $time);
-        //
-        $objInspage->setVar('title', $_POST['title']);
-        $objInspage->setVar('weight', $weight);
-        $objInspage->setVar('hometext', $_POST['hometext']);
-        // Сноска
-        $objInspage->setVar('footnote', $_POST['footnote']);
-        $objInspage->setVar('status', $_POST['status']);
-        $objInspage->setVar('keywords', $_POST['keywords']);
-        $objInspage->setVar('description', $_POST['description']);
-
-        // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
-            $message_err .= _MD_INSTRUCTION_ERR_INSTR . '<br>';
-        }
-        // Проверка веса
-        if (0 == $weight) {
-            $err         = true;
-            $message_err .= _MD_INSTRUCTION_ERR_WEIGHT . '<br>';
-        }
-        // Проверка родительской страницы
-        if ($pageid && ($pageid == $pid)) {
-            $err         = true;
-            $message_err .= _MD_INSTRUCTION_ERR_PPAGE . '<br>';
-        }
-        // Если были ошибки
-        if (true === $err) {
-            // Задание тайтла
-            $xoopsOption['xoops_pagetitle'] = '';
-            // Шаблон
-            $GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
-            // Заголовок
-            include_once $GLOBALS['xoops']->path('header.php');
-            // Сообщение об ошибке
-            $message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
-            // Если небыло ошибок
-        } else {
-            // Вставляем данные в БД
-            if ($pageHandler->insert($objInspage)) {
-                // Если мы редактируем
-                if ($pageid) {
-                    // Обновление даты
-                    $sql = sprintf('UPDATE %s SET `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
-                    $GLOBALS['xoopsDB']->query($sql);
-                    // Запись в лог
-                    xoops_loadLanguage('main', 'userslog');
-                    //userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_MODIFY_PAGE );
-                    //
-                    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEMODIFY);
-                    // Если мы добавляем
-                } else {
-                    // Инкримент комментов
-                    $pageHandler->updateposts($uid, $_POST['status'], 'add');
-                    // Инкремент страниц и обновление даты
-                    $sql = sprintf('UPDATE %s SET `pages` = `pages` + 1, `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
-                    $GLOBALS['xoopsDB']->query($sql);
-                    // Запись в лог
-                    xoops_loadLanguage('main', 'userslog');
-                    //userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_SUBMIT_PAGE );
-                    //
-                    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEADDED);
-                }
-            }
-
-            // Задание тайтла
-            $xoopsOption['xoops_pagetitle'] = '';
-            // Шаблон
-            $GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
-            // Заголовок
-            include_once $GLOBALS['xoops']->path('header.php');
-
-            // Выводим ошибки в шаблон
-            $GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInspage->getHtmlErrors());
-        }
-        // Получаем форму
-        $form = $objInspage->getForm('submit.php', $instrid);
-
-        // Форма
-        $GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
-
-        // Подвал
-        include_once $GLOBALS['xoops']->path('footer.php');
-
-        break;
+	case 'editpage':
+
+		// Задание тайтла
+		$xoopsOption['xoops_pagetitle'] = '';
+		// Шаблон
+		$GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_editpage.tpl';
+		// Заголовок
+		include_once $GLOBALS['xoops']->path('header.php');
+
+		// Если мы редактируем страницу
+		if ($pageid) {
+			// Получаем объект страницы
+			$objInspage = $pageHandler->get($pageid);
+			// ID инструкции
+			$instrid = $objInspage->getVar('instrid');
+			// Объект инструкции
+			$objInsinstr = $instructionHandler->get($instrid);
+			// Можно ли редактировать инструкцию в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
+			}
+			// Создание новой страницы
+		} elseif ($instrid) {
+
+			// Если нельзя добавлять не в одну категорию
+			//if( ! count( $cat_submit ) ) redirect_header( 'index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMIT_PAGE );
+			// Создаём объект страницы
+			$objInspage = $pageHandler->create();
+			// Объект инструкции
+			$objInsinstr = $instructionHandler->get($instrid);
+			// Можно ли добавлять инструкции в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
+			}
+		} else {
+			redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
+		}
+
+		// Информация об инструкции
+
+		// Массив данных об инструкции
+		$instrs = [];
+		// ID инструкции
+		$instrs['instrid'] = $objInsinstr->getVar('instrid');
+		// Название страницы
+		$instrs['title'] = $objInsinstr->getVar('title');
+		// Описание
+		$instrs['description'] = $objInsinstr->getVar('description');
+
+		// Выводим в шаблон
+		$GLOBALS['xoopsTpl']->assign('insInstr', $instrs);
+
+		//
+
+		$form = $objInspage->getForm('submit.php', $instrid);
+		// Форма
+		$GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
+
+		// Подвал
+		include_once $GLOBALS['xoops']->path('footer.php');
+
+		break;
+	// Сохранение страницы
+	case 'savepage':
+
+		// Проверка
+		if (!$GLOBALS['xoopsSecurity']->check()) {
+			redirect_header('index.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+		}
+
+		$err         = false;
+		$message_err = '';
+
+		// Если мы редактируем
+		if ($pageid) {
+			$objInspage = $pageHandler->get($pageid);
+			// Объект инструкции
+			$objInsinstr = $instructionHandler->get($objInspage->getVar('instrid'));
+			// Можно ли редактировать инструкцию в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
+			}
+		} elseif ($instrid) {
+			$objInspage = $pageHandler->create();
+			// Объект инструкции
+			$objInsinstr = $instructionHandler->get($instrid);
+			// Можно ли добавлять инструкции в данной категории
+			if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+				redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
+			}
+
+			// Если мы создаём страницу необходимо указать к какой инструкции
+			$objInspage->setVar('instrid', $instrid);
+			// Указываем дату создания
+			$objInspage->setVar('datecreated', $time);
+			// Указываем пользователя
+			$objInspage->setVar('uid', $uid);
+		} else {
+			redirect_header('index.php', 3, _MD_INSTRUCTION_BADREQUEST);
+		}
+
+		// Родительская страница
+		$objInspage->setVar('pid', $pid);
+		// Дата обновления
+		$objInspage->setVar('dateupdated', $time);
+		//
+		$objInspage->setVar('title', $_POST['title']);
+		$objInspage->setVar('weight', $weight);
+		$objInspage->setVar('hometext', $_POST['hometext']);
+		// Сноска
+		$objInspage->setVar('footnote', $_POST['footnote']);
+		$objInspage->setVar('status', $_POST['status']);
+		$objInspage->setVar('keywords', $_POST['keywords']);
+		$objInspage->setVar('description', $_POST['description']);
+
+		// Проверка категорий
+		if (!$pageid && !$instrid) {
+			$err         = true;
+			$message_err .= _MD_INSTRUCTION_ERR_INSTR . '<br>';
+		}
+		// Проверка веса
+		if (0 == $weight) {
+			$err         = true;
+			$message_err .= _MD_INSTRUCTION_ERR_WEIGHT . '<br>';
+		}
+		// Проверка родительской страницы
+		if ($pageid && ($pageid == $pid)) {
+			$err         = true;
+			$message_err .= _MD_INSTRUCTION_ERR_PPAGE . '<br>';
+		}
+		// Если были ошибки
+		if (true === $err) {
+			// Задание тайтла
+			$xoopsOption['xoops_pagetitle'] = '';
+			// Шаблон
+			$GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
+			// Заголовок
+			include_once $GLOBALS['xoops']->path('header.php');
+			// Сообщение об ошибке
+			$message_err = '<div class="errorMsg" style="text-align: left;">' . $message_err . '</div>';
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $message_err);
+			// Если небыло ошибок
+		} else {
+			// Вставляем данные в БД
+			if ($pageHandler->insert($objInspage)) {
+				// Если мы редактируем
+				if ($pageid) {
+					// Обновление даты
+					$sql = sprintf('UPDATE %s SET `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
+					$GLOBALS['xoopsDB']->query($sql);
+					// Запись в лог
+					xoops_loadLanguage('main', 'userslog');
+					//userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_MODIFY_PAGE );
+					//
+					redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEMODIFY);
+					// Если мы добавляем
+				} else {
+					// Инкримент комментов
+					$pageHandler->updateposts($uid, $_POST['status'], 'add');
+					// Инкремент страниц и обновление даты
+					$sql = sprintf('UPDATE %s SET `pages` = `pages` + 1, `dateupdated` = %u WHERE `instrid` = %u', $GLOBALS['xoopsDB']->prefix($moduleDirName . '_instr'), $time, $instrid);
+					$GLOBALS['xoopsDB']->query($sql);
+					// Запись в лог
+					xoops_loadLanguage('main', 'userslog');
+					//userslog_insert( $objInsinstr->getVar('title') . ': ' . $objInspage->getVar('title'), _MD_USERSLOG_SUBMIT_PAGE );
+					//
+					redirect_header('index.php', 3, _MD_INSTRUCTION_PAGEADDED);
+				}
+			}
+
+			// Задание тайтла
+			$xoopsOption['xoops_pagetitle'] = '';
+			// Шаблон
+			$GLOBALS['xoopsOption']['template_main'] = $moduleDirName . '_savepage.tpl';
+			// Заголовок
+			include_once $GLOBALS['xoops']->path('header.php');
+
+			// Выводим ошибки в шаблон
+			$GLOBALS['xoopsTpl']->assign('insErrorMsg', $objInspage->getHtmlErrors());
+		}
+		// Получаем форму
+		$form = $objInspage->getForm('submit.php', $instrid);
+
+		// Форма
+		$GLOBALS['xoopsTpl']->assign('insFormPage', $form->render());
+
+		// Подвал
+		include_once $GLOBALS['xoops']->path('footer.php');
+
+		break;
 }

Spacing +16 added lines, -16 removed lines

@@ -15,17 +15,17 @@ 
 $time = time();
 
 // ID инструкции
-$instrid = isset($_GET['instrid']) ? (int)$_GET['instrid'] : 0;
-$instrid = isset($_POST['instrid']) ? (int)$_POST['instrid'] : $instrid;
+$instrid = isset($_GET['instrid']) ? (int) $_GET['instrid'] : 0;
+$instrid = isset($_POST['instrid']) ? (int) $_POST['instrid'] : $instrid;
 // ID страницы
-$pageid = isset($_GET['pageid']) ? (int)$_GET['pageid'] : 0;
-$pageid = isset($_POST['pageid']) ? (int)$_POST['pageid'] : $pageid;
+$pageid = isset($_GET['pageid']) ? (int) $_GET['pageid'] : 0;
+$pageid = isset($_POST['pageid']) ? (int) $_POST['pageid'] : $pageid;
 // ID категории
-$cid = isset($_POST['cid']) ? (int)$_POST['cid'] : 0;
+$cid = isset($_POST['cid']) ? (int) $_POST['cid'] : 0;
 // Вес
-$weight = isset($_POST['weight']) ? (int)$_POST['weight'] : 0;
+$weight = isset($_POST['weight']) ? (int) $_POST['weight'] : 0;
 //
-$pid = isset($_POST['pid']) ? (int)$_POST['pid'] : 0;
+$pid = isset($_POST['pid']) ? (int) $_POST['pid'] : 0;
 
 // Права на добавление
 $cat_submit = Xoopsmodules\instruction\Utility::getItemIds($moduleDirName . '_submit');
@@ -55,7 +55,7 @@ 
             // Объект инструкции
             $objInsinstr = $instructionHandler->get($instrid);
             // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_edit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
             }
             // Создание новой страницы
@@ -68,7 +68,7 @@ 
             // Объект инструкции
             $objInsinstr = $instructionHandler->get($instrid);
             // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_submit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
             }
         } else {
@@ -103,7 +103,7 @@ 
     case 'savepage':
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             redirect_header('index.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
 
@@ -116,7 +116,7 @@ 
             // Объект инструкции
             $objInsinstr = $instructionHandler->get($objInspage->getVar('instrid'));
             // Можно ли редактировать инструкцию в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_edit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_edit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_EDITPAGE);
             }
         } elseif ($instrid) {
@@ -124,7 +124,7 @@ 
             // Объект инструкции
             $objInsinstr = $instructionHandler->get($instrid);
             // Можно ли добавлять инструкции в данной категории
-            if (!in_array($objInsinstr->getVar('cid'), $cat_submit)) {
+            if ( ! in_array($objInsinstr->getVar('cid'), $cat_submit)) {
                 redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_SUBMITPAGE);
             }
 
@@ -153,18 +153,18 @@ 
         $objInspage->setVar('description', $_POST['description']);
 
         // Проверка категорий
-        if (!$pageid && !$instrid) {
-            $err         = true;
+        if ( ! $pageid && ! $instrid) {
+            $err = true;
             $message_err .= _MD_INSTRUCTION_ERR_INSTR . '<br>';
         }
         // Проверка веса
         if (0 == $weight) {
-            $err         = true;
+            $err = true;
             $message_err .= _MD_INSTRUCTION_ERR_WEIGHT . '<br>';
         }
         // Проверка родительской страницы
         if ($pageid && ($pageid == $pid)) {
-            $err         = true;
+            $err = true;
             $message_err .= _MD_INSTRUCTION_ERR_PPAGE . '<br>';
         }
         // Если были ошибки

page.php

Indentation +43 added lines, -43 removed lines

@@ -29,8 +29,8 @@ 
 $criteria->add(new \Criteria('pageid ', $pageid));
 $criteria->add(new \Criteria('status ', '0', '>'));
 if (0 == $pageHandler->getCount($criteria)) {
-    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
-    exit();
+	redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
+	exit();
 }
 //
 unset($criteria);
@@ -42,8 +42,8 @@ 
 
 // Если админ и ссылка на отключение кэша
 if (is_object($GLOBALS['xoopsUser']) && $GLOBALS['xoopsUser']->isAdmin() && $nocache) {
-    // Отключаем кэш
-    $GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
+	// Отключаем кэш
+	$GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
 }
 
 // Задание тайтла
@@ -60,8 +60,8 @@ 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::getItemIds();
 if (!in_array($objInsinstr->getVar('cid'), $categories)) {
-    redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
-    exit();
+	redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
+	exit();
 }
 
 // Массив данных о странице
@@ -78,9 +78,9 @@ 
 $footnote = $objInspage->getVar('footnote');
 // Если есть сноски
 if ($footnote) {
-    $pages['footnotes'] = explode('|', $objInspage->getVar('footnote'));
+	$pages['footnotes'] = explode('|', $objInspage->getVar('footnote'));
 } else {
-    $pages['footnotes'] = false;
+	$pages['footnotes'] = false;
 }
 // Мета-теги ключевых слов
 $pages['keywords'] = $objInspage->getVar('keywords');
@@ -89,39 +89,39 @@ 
 //
 // Если админ, рисуем админлинк
 if (is_object($GLOBALS['xoopsUser']) && $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid())) {
-    $pages['adminlink'] = '&nbsp;<a href="'
-                          . XOOPS_URL
-                          . '/modules/'
-                          . $moduleDirName
-                          . '/admin/instr.php?op=editpage&pageid='
-                          . $pages['pageid']
-                          . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt='
-                          . _EDIT
-                          . ' title='
-                          . _EDIT
-                          . '></a>&nbsp;<a href="'
-                          . XOOPS_URL
-                          . '/modules/'
-                          . $moduleDirName
-                          . '/admin/instr.php?op=delpage&pageid='
-                          . $pages['pageid']
-                          . '"><img style="width:16px;" src="./assets/icons/delete_mini.png" alt='
-                          . _DELETE
-                          . ' title='
-                          . _DELETE
-                          . '></a>&nbsp;';
+	$pages['adminlink'] = '&nbsp;<a href="'
+						  . XOOPS_URL
+						  . '/modules/'
+						  . $moduleDirName
+						  . '/admin/instr.php?op=editpage&pageid='
+						  . $pages['pageid']
+						  . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt='
+						  . _EDIT
+						  . ' title='
+						  . _EDIT
+						  . '></a>&nbsp;<a href="'
+						  . XOOPS_URL
+						  . '/modules/'
+						  . $moduleDirName
+						  . '/admin/instr.php?op=delpage&pageid='
+						  . $pages['pageid']
+						  . '"><img style="width:16px;" src="./assets/icons/delete_mini.png" alt='
+						  . _DELETE
+						  . ' title='
+						  . _DELETE
+						  . '></a>&nbsp;';
 } else {
-    $pages['adminlink'] = '&nbsp;';
-    // Если можно редактировать
-    if ($gpermHandler->checkRight($moduleDirName . '_edit', $objInsinstr->getVar('cid'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
-        $pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
-    }
-
-    $pages['adminlink'] .= '&nbsp;';
-    // Если нет админлика
-    if ('[&nbsp;&nbsp;]' === $pages['adminlink']) {
-        $pages['adminlink'] = '';
-    }
+	$pages['adminlink'] = '&nbsp;';
+	// Если можно редактировать
+	if ($gpermHandler->checkRight($moduleDirName . '_edit', $objInsinstr->getVar('cid'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
+		$pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="./assets/icons/edit_mini.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
+	}
+
+	$pages['adminlink'] .= '&nbsp;';
+	// Если нет админлика
+	if ('[&nbsp;&nbsp;]' === $pages['adminlink']) {
+		$pages['adminlink'] = '';
+	}
 }
 // Выводим в шаблон
 $GLOBALS['xoopsTpl']->assign('insPage', $pages);
@@ -140,7 +140,7 @@ 
 $nav_parent_id = array_reverse($nav_parent_id);
 $navigation    = '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/">' . $GLOBALS['xoopsModule']->name() . '</a>&nbsp;:&nbsp;';
 foreach (array_keys($nav_parent_id) as $i) {
-    $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
+	$navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
 }
 $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $objInscat->getVar('cid') . '">' . $objInscat->getVar('title') . '</a>&nbsp;:&nbsp;';
 $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/instr.php?id=' . $pages['instrid'] . '">' . $objInsinstr->getVar('title') . '</a>';
@@ -172,9 +172,9 @@ 
 
 // Рейтинг
 if (xoops_getModuleOption('userat', 'instruction')) {
-    $xoopsTpl->assign('insUserat', true);
+	$xoopsTpl->assign('insUserat', true);
 } else {
-    $xoopsTpl->assign('insUserat', false);
+	$xoopsTpl->assign('insUserat', false);
 }
 
 // Мета теги

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
 
 // Получаем данные
 // ID страницы
-$pageid = isset($_GET['id']) ? (int)$_GET['id'] : 0;
+$pageid = isset($_GET['id']) ? (int) $_GET['id'] : 0;
 // Без кэша
 $nocache = Xoopsmodules\instruction\Utility::cleanVars($_GET, 'nocache', 0, 'int');
 
@@ -59,7 +59,7 @@ 
 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::getItemIds();
-if (!in_array($objInsinstr->getVar('cid'), $categories)) {
+if ( ! in_array($objInsinstr->getVar('cid'), $categories)) {
     redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
     exit();
 }

preloads/core.php

Indentation +8 added lines, -8 removed lines

@@ -22,12 +22,12 @@
  */
 class InstructionCorePreload extends XoopsPreloadItem
 {
-    // to add PSR-4 autoloader
-    /**
-     * @param $args
-     */
-    public static function eventCoreIncludeCommonEnd($args)
-    {
-        include __DIR__ . '/autoloader.php';
-    }
+	// to add PSR-4 autoloader
+	/**
+	 * @param $args
+	 */
+	public static function eventCoreIncludeCommonEnd($args)
+	{
+		include __DIR__ . '/autoloader.php';
+	}
 }

preloads/autoloader.php

Indentation +19 added lines, -19 removed lines

@@ -4,29 +4,29 @@
  * @see http://www.php-fig.org/psr/psr-4/examples/
  */
 spl_autoload_register(function ($class) {
-    // project-specific namespace prefix
-    $prefix = 'Xoopsmodules\\instruction\\';
+	// project-specific namespace prefix
+	$prefix = 'Xoopsmodules\\instruction\\';
 
-    // base directory for the namespace prefix
-    $base_dir = __DIR__ . '/../class/';
+	// base directory for the namespace prefix
+	$base_dir = __DIR__ . '/../class/';
 
-    // does the class use the namespace prefix?
-    $len = strlen($prefix);
+	// does the class use the namespace prefix?
+	$len = strlen($prefix);
 
-    if (0 !== strncmp($prefix, $class, $len)) {
-        return;
-    }
+	if (0 !== strncmp($prefix, $class, $len)) {
+		return;
+	}
 
-    // get the relative class name
-    $relative_class = substr($class, $len);
+	// get the relative class name
+	$relative_class = substr($class, $len);
 
-    // replace the namespace prefix with the base directory, replace namespace
-    // separators with directory separators in the relative class name, append
-    // with .php
-    $file = $base_dir . str_replace('\\', '/', $relative_class) . '.php';
+	// replace the namespace prefix with the base directory, replace namespace
+	// separators with directory separators in the relative class name, append
+	// with .php
+	$file = $base_dir . str_replace('\\', '/', $relative_class) . '.php';
 
-    // if the file exists, require it
-    if (file_exists($file)) {
-        require $file;
-    }
+	// if the file exists, require it
+	if (file_exists($file)) {
+		require $file;
+	}
 });

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 /**
  * @see http://www.php-fig.org/psr/psr-4/examples/
  */
-spl_autoload_register(function ($class) {
+spl_autoload_register(function($class) {
     // project-specific namespace prefix
     $prefix = 'Xoopsmodules\\instruction\\';
 

index.php

Indentation +55 added lines, -55 removed lines

@@ -48,11 +48,11 @@ 
 $module = $helper->getModule();
 
 if (Xoopsmodules\instruction\Utility::checkVerXoops($module, '2.5.9')) {
-    $cat_select = $mytree->makeSelectElement('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'", '');
-    $GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select->render());
+	$cat_select = $mytree->makeSelectElement('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'", '');
+	$GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select->render());
 } else {
-    $cat_select = $mytree->makeSelBox('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'");
-    $GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select);
+	$cat_select = $mytree->makeSelBox('cid', 'title', '--', $cid, true, 0, "onChange='javascript: document.insformselcat.submit()'");
+	$GLOBALS['xoopsTpl']->assign('insFormSelCat', $cat_select);
 }
 
 // Находим список всех инструкций
@@ -62,14 +62,14 @@ 
 $criteria->add(new \Criteria('status', '0', '>'));
 // Если есть категория
 if ($cid) {
-    // Если нельзя просматривать эту категорию
-    if (!in_array($cid, $categories)) {
-        redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_CAT);
-    }
-    $criteria->add(new \Criteria('cid', $cid, '='));
-    // Иначе находим список всех
+	// Если нельзя просматривать эту категорию
+	if (!in_array($cid, $categories)) {
+		redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_CAT);
+	}
+	$criteria->add(new \Criteria('cid', $cid, '='));
+	// Иначе находим список всех
 } else {
-    $criteria->add(new \Criteria('cid', '( ' . implode(', ', $categories) . ' )', 'IN'));
+	$criteria->add(new \Criteria('cid', '( ' . implode(', ', $categories) . ' )', 'IN'));
 }
 
 // Число инструкций, удовлетворяющих данному условию
@@ -86,10 +86,10 @@ 
 $instr_arr = $instructionHandler->getall($criteria);
 // Если записей больше чем $limit, то выводим пагинатор
 if ($numrows > $limit) {
-    $pagenav = new \XoopsPageNav($numrows, $limit, $start, 'start', 'cid=' . $cid);
-    $pagenav = $pagenav->renderNav(4);
+	$pagenav = new \XoopsPageNav($numrows, $limit, $start, 'start', 'cid=' . $cid);
+	$pagenav = $pagenav->renderNav(4);
 } else {
-    $pagenav = '';
+	$pagenav = '';
 }
 // Выводим пагинатор в шаблон
 $GLOBALS['xoopsTpl']->assign('insPagenav', $pagenav);
@@ -100,53 +100,53 @@ 
 
 // Если есть записи
 if ($numrows > 0) {
-    $class = 'odd';
-    foreach (array_keys($instr_arr) as $i) {
-
-        //
-        $class = ('even' === $class) ? 'odd' : 'even';
-        // ID
-        $insinstr_instrid = $instr_arr[$i]->getVar('instrid');
-        // Название
-        $insinstr_title = $instr_arr[$i]->getVar('title');
-        // Статус
-        $insinstr_status = $instr_arr[$i]->getVar('status');
-        // Количество страниц
-        $insinstr_pages = $instr_arr[$i]->getVar('pages');
-        // Категория
-        $insinstr_cid = $instr_arr[$i]->getVar('cid');
-        $insinstr_cat = $categoryHandler->get($insinstr_cid);
-        // Права на добавление
-        $perm_submit = in_array($insinstr_cid, $cat_submit) ? true : false;
-        // Права на редактирование
-        $perm_edit = in_array($insinstr_cid, $cat_edit) ? true : false;
-        //Мета-теги ключевых слов
-        $insinstr_metakeywords = $instr_arr[$i]->getVar('metakeywords');
-        // Если есть - добавляем в мета-теги страницы
-        if ($insinstr_metakeywords) {
-            $index_metakeywords[] = $insinstr_metakeywords;
-        }
-        // Мета-теги описания
-        $insinstr_metadescript = $instr_arr[$i]->getVar('metadescription');
-        // Если есть - добавляем в мета-теги страницы
-        if ($insinstr_metadescript) {
-            $index_metadescript[] = $insinstr_metadescript;
-        }
-
-        // Выводим в шаблон
-        $GLOBALS['xoopsTpl']->append('insListInstr',
-                                     ['instrid' => $insinstr_instrid, 'title' => $insinstr_title, 'status' => $insinstr_status, 'pages' => $insinstr_pages, 'ctitle' => $insinstr_cat->getVar('title'), 'cid' => $insinstr_cid, 'permsubmit' => $perm_submit, 'permedit' => $perm_edit, 'class' => $class]);
-    }
-
-    // Языковые константы
+	$class = 'odd';
+	foreach (array_keys($instr_arr) as $i) {
+
+		//
+		$class = ('even' === $class) ? 'odd' : 'even';
+		// ID
+		$insinstr_instrid = $instr_arr[$i]->getVar('instrid');
+		// Название
+		$insinstr_title = $instr_arr[$i]->getVar('title');
+		// Статус
+		$insinstr_status = $instr_arr[$i]->getVar('status');
+		// Количество страниц
+		$insinstr_pages = $instr_arr[$i]->getVar('pages');
+		// Категория
+		$insinstr_cid = $instr_arr[$i]->getVar('cid');
+		$insinstr_cat = $categoryHandler->get($insinstr_cid);
+		// Права на добавление
+		$perm_submit = in_array($insinstr_cid, $cat_submit) ? true : false;
+		// Права на редактирование
+		$perm_edit = in_array($insinstr_cid, $cat_edit) ? true : false;
+		//Мета-теги ключевых слов
+		$insinstr_metakeywords = $instr_arr[$i]->getVar('metakeywords');
+		// Если есть - добавляем в мета-теги страницы
+		if ($insinstr_metakeywords) {
+			$index_metakeywords[] = $insinstr_metakeywords;
+		}
+		// Мета-теги описания
+		$insinstr_metadescript = $instr_arr[$i]->getVar('metadescription');
+		// Если есть - добавляем в мета-теги страницы
+		if ($insinstr_metadescript) {
+			$index_metadescript[] = $insinstr_metadescript;
+		}
+
+		// Выводим в шаблон
+		$GLOBALS['xoopsTpl']->append('insListInstr',
+									 ['instrid' => $insinstr_instrid, 'title' => $insinstr_title, 'status' => $insinstr_status, 'pages' => $insinstr_pages, 'ctitle' => $insinstr_cat->getVar('title'), 'cid' => $insinstr_cid, 'permsubmit' => $perm_submit, 'permedit' => $perm_edit, 'class' => $class]);
+	}
+
+	// Языковые константы
 }
 
 // Если есть мета-теги
 if (count($index_metakeywords)) {
-    $xoTheme->addMeta('meta', 'keywords', implode(', ', $index_metakeywords));
+	$xoTheme->addMeta('meta', 'keywords', implode(', ', $index_metakeywords));
 }
 if (count($index_metadescript)) {
-    $xoTheme->addMeta('meta', 'description', implode(', ', $index_metadescript));
+	$xoTheme->addMeta('meta', 'description', implode(', ', $index_metadescript));
 }
 
 // Подвал

Spacing +3 added lines, -3 removed lines

@@ -17,9 +17,9 @@ 
 // Стили
 $xoTheme->addStylesheet(XOOPS_URL . '/modules/' . $moduleDirName . '/assets/css/style.css');
 //
-$cid = isset($_GET['cid']) ? (int)$_GET['cid'] : 0;
+$cid = isset($_GET['cid']) ? (int) $_GET['cid'] : 0;
 //
-$start = isset($_GET['start']) ? (int)$_GET['start'] : 0;
+$start = isset($_GET['start']) ? (int) $_GET['start'] : 0;
 //
 
 //mb $limit = xoops_getModuleOption('perpagemain', 'instruction');
@@ -63,7 +63,7 @@ 
 // Если есть категория
 if ($cid) {
     // Если нельзя просматривать эту категорию
-    if (!in_array($cid, $categories)) {
+    if ( ! in_array($cid, $categories)) {
         redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_CAT);
     }
     $criteria->add(new \Criteria('cid', $cid, '='));

class/common/VersionChecks.php

Indentation +69 added lines, -69 removed lines

@@ -17,77 +17,77 @@
  */
 trait VersionChecks
 {
-    /**
-     *
-     * Verifies XOOPS version meets minimum requirements for this module
-     * @static
-     * @param \XoopsModule|null $module
-     *
-     * @param null|string       $requiredVer
-     * @return bool true if meets requirements, false if not
-     */
-    public static function checkVerXoops(\XoopsModule $module = null, $requiredVer = null)
-    {
-        $moduleDirName = basename(dirname(dirname(__DIR__)));
-        if (null === $module) {
-            $module = \XoopsModule::getByDirname($moduleDirName);
-        }
-        xoops_loadLanguage('admin', $moduleDirName);
-        //check for minimum XOOPS version
-        $currentVer = substr(XOOPS_VERSION, 6); // get the numeric part of string
-        $currArray  = explode('.', $currentVer);
-        if (null === $requiredVer) {
-            $requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
-        }
-        $reqArray = explode('.', $requiredVer);
-        $success  = true;
-        foreach ($reqArray as $k => $v) {
-            if (isset($currArray[$k])) {
-                if ($currArray[$k] > $v) {
-                    break;
-                } elseif ($currArray[$k] == $v) {
-                    continue;
-                } else {
-                    $success = false;
-                    break;
-                }
-            } else {
-                if ((int)$v > 0) { // handles versions like x.x.x.0_RC2
-                    $success = false;
-                    break;
-                }
-            }
-        }
+	/**
+	 *
+	 * Verifies XOOPS version meets minimum requirements for this module
+	 * @static
+	 * @param \XoopsModule|null $module
+	 *
+	 * @param null|string       $requiredVer
+	 * @return bool true if meets requirements, false if not
+	 */
+	public static function checkVerXoops(\XoopsModule $module = null, $requiredVer = null)
+	{
+		$moduleDirName = basename(dirname(dirname(__DIR__)));
+		if (null === $module) {
+			$module = \XoopsModule::getByDirname($moduleDirName);
+		}
+		xoops_loadLanguage('admin', $moduleDirName);
+		//check for minimum XOOPS version
+		$currentVer = substr(XOOPS_VERSION, 6); // get the numeric part of string
+		$currArray  = explode('.', $currentVer);
+		if (null === $requiredVer) {
+			$requiredVer = '' . $module->getInfo('min_xoops'); //making sure it's a string
+		}
+		$reqArray = explode('.', $requiredVer);
+		$success  = true;
+		foreach ($reqArray as $k => $v) {
+			if (isset($currArray[$k])) {
+				if ($currArray[$k] > $v) {
+					break;
+				} elseif ($currArray[$k] == $v) {
+					continue;
+				} else {
+					$success = false;
+					break;
+				}
+			} else {
+				if ((int)$v > 0) { // handles versions like x.x.x.0_RC2
+					$success = false;
+					break;
+				}
+			}
+		}
 
-        if (false === $success) {
-            $module->setErrors(sprintf(_AM_WFL_ERROR_BAD_XOOPS, $requiredVer, $currentVer));
-        }
+		if (false === $success) {
+			$module->setErrors(sprintf(_AM_WFL_ERROR_BAD_XOOPS, $requiredVer, $currentVer));
+		}
 
-        return $success;
-    }
+		return $success;
+	}
 
-    /**
-     *
-     * Verifies PHP version meets minimum requirements for this module
-     * @static
-     * @param \XoopsModule $module
-     *
-     * @return bool true if meets requirements, false if not
-     */
-    public static function checkVerPhp(\XoopsModule $module)
-    {
-        xoops_loadLanguage('admin', $module->dirname());
-        // check for minimum PHP version
-        $success = true;
-        $verNum  = PHP_VERSION;
-        $reqVer  = $module->getInfo('min_php');
-        if (false !== $reqVer && '' !== $reqVer) {
-            if (version_compare($verNum, $reqVer, '<')) {
-                $module->setErrors(sprintf(_AM_WFL_ERROR_BAD_PHP, $reqVer, $verNum));
-                $success = false;
-            }
-        }
+	/**
+	 *
+	 * Verifies PHP version meets minimum requirements for this module
+	 * @static
+	 * @param \XoopsModule $module
+	 *
+	 * @return bool true if meets requirements, false if not
+	 */
+	public static function checkVerPhp(\XoopsModule $module)
+	{
+		xoops_loadLanguage('admin', $module->dirname());
+		// check for minimum PHP version
+		$success = true;
+		$verNum  = PHP_VERSION;
+		$reqVer  = $module->getInfo('min_php');
+		if (false !== $reqVer && '' !== $reqVer) {
+			if (version_compare($verNum, $reqVer, '<')) {
+				$module->setErrors(sprintf(_AM_WFL_ERROR_BAD_PHP, $reqVer, $verNum));
+				$success = false;
+			}
+		}
 
-        return $success;
-    }
+		return $success;
+	}
 }