mambax7 / instruction

include/config.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 $moduleDirName = basename(dirname(__DIR__));
 $capsDirName   = strtoupper($moduleDirName);
 
-if (!defined($capsDirName . '_DIRNAME')) {
+if ( ! defined($capsDirName . '_DIRNAME')) {
     //if (!defined(constant($capsDirName . '_DIRNAME'))) {
     define($capsDirName . '_DIRNAME', $GLOBALS['xoopsModule']->dirname());
     define($capsDirName . '_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
@@ -35,7 +35,7 @@ 
 }
 
 //Configurator
-return (object)[
+return (object) [
     'name'          => strtoupper($moduleDirName) . ' Module Configurator',
     'paths'         => [
         'dirname'    => $moduleDirName,

include/search.inc.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
     $sql = 'SELECT p.pageid, p.title, p.uid, p.datecreated, i.title FROM ' . $GLOBALS['xoopsDB']->prefix('instruction_page') . ' p, ' . $GLOBALS['xoopsDB']->prefix('instruction_instr') . ' i WHERE i.instrid = p.instrid AND i.status > 0 AND p.status > 0 AND p.type > 0';
     if (0 != $userid) {
-        $sql .= ' AND p.uid = ' . (int)$userid . ' ';
+        $sql .= ' AND p.uid = ' . (int) $userid . ' ';
         //return NULL;
     }
 

instr.php

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
 //$categoryHandler   = xoops_getModuleHandler('category', 'instruction');
 //$pageHandler  = xoops_getModuleHandler('page', 'instruction');
 
-$instrid = isset($_GET['id']) ? (int)$_GET['id'] : 0;
+$instrid = isset($_GET['id']) ? (int) $_GET['id'] : 0;
 
 // Существует ли такая инструкция
 $criteria = new \CriteriaCompo();
@@ -41,7 +41,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();
 }

admin/cat.php

Spacing +6 added lines, -6 removed lines

@@ -100,7 +100,7 @@ 
     case 'savecat':
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
         // Если мы редактируем
@@ -126,12 +126,12 @@ 
 
         // Проверка веса
         if (0 == $weight) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_WEIGHT . '<br>';
         }
         // Проверка категорий
         if ($cid && ($cid == $pid)) {
-            $err         = true;
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_PCAT . '<br>';
         }
         // Если были ошибки
@@ -237,7 +237,7 @@ 
 
         $objInscat = $categoryHandler->get($cid);
         // Если нет такой категории
-        if (!is_object($objInscat)) {
+        if ( ! is_object($objInscat)) {
             redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
         }
 
@@ -253,12 +253,12 @@ 
         }
 
         // Нажали ли мы на кнопку OK
-        $ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
+        $ok = isset($_POST['ok']) ? (int) $_POST['ok'] : 0;
         // Если мы нажали на кнопку
         if ($ok) {
 
             // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
+            if ( ! $GLOBALS['xoopsSecurity']->check()) {
                 redirect_header('cat.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             // Пытаемся удалить категорию

admin/instr.php

Spacing +25 added lines, -25 removed lines

@@ -61,7 +61,7 @@ 
             // Получаем объект категории
             $objInscat = $categoryHandler->get($cid);
             // Если нет такой категории
-            if (!is_object($objInscat)) {
+            if ( ! is_object($objInscat)) {
                 redirect_header('cat.php', 3, _AM_INSTRUCTION_ERR_CATNOTSELECT);
             }
         }
@@ -167,7 +167,7 @@ 
     case 'saveinstr':
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
         // Если мы редактируем
@@ -198,18 +198,18 @@ 
         $objInsinstr->setVar('metadescription', $_POST['metadescription']);
 
         // Проверка категорий
-        if (!$cid) {
-            $err         = true;
+        if ( ! $cid) {
+            $err = true;
             $message_err .= _AM_INSTRUCTION_ERR_CAT . '<br>';
         }
         // Проверка названия
-        if (!$instr_title) {
-            $err         = true;
+        if ( ! $instr_title) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
         }
         // Проверка основного текста
-        if (!$instr_description) {
-            $err         = true;
+        if ( ! $instr_description) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_DESCRIPTION . '<br>';
         }
 
@@ -309,12 +309,12 @@ 
         $objInsinstr = $instructionHandler->get($instrid);
 
         // Нажали ли мы на кнопку OK
-        $ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
+        $ok = isset($_POST['ok']) ? (int) $_POST['ok'] : 0;
         //
         if ($ok) {
 
             // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
+            if ( ! $GLOBALS['xoopsSecurity']->check()) {
                 redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             // Находим все страницы, пренадлежащие этой инструкции
@@ -401,7 +401,7 @@ 
         $message_err = '';
 
         // Проверка сессии
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             $err         = true;
             $err_txt     = implode(', ', $GLOBALS['xoopsSecurity']->getErrors());
             $message_err .= $err_txt . '<br>';
@@ -447,10 +447,10 @@ 
         // Мета-теги ключевых слов
         $objInspage->setVar('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;
         //$doimage = ( isset( $_POST['doimage'] ) && intval( $_POST['doimage'] ) > 0 ) ? 1 : 0;
         $objInspage->setVar('dohtml', $dohtml);
         $objInspage->setVar('dosmiley', $dosmiley);
@@ -459,28 +459,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 (!$page_title) {
-            $err         = true;
+        if ( ! $page_title) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_TITLE . '<br>';
         }
         // Проверка основного текста
-        if (!$page_hometext) {
-            $err         = true;
+        if ( ! $page_hometext) {
+            $err = true;
             $message_err .= _AM_INSTR_ERR_HOMETEXT . '<br>';
         }
 
@@ -543,12 +543,12 @@ 
 
         $objInspage = $pageHandler->get($pageid);
         // Нажали ли мы на кнопку OK
-        $ok = isset($_POST['ok']) ? (int)$_POST['ok'] : 0;
+        $ok = isset($_POST['ok']) ? (int) $_POST['ok'] : 0;
         // Если мы нажали на кнопку
         if ($ok) {
 
             // Проверка
-            if (!$GLOBALS['xoopsSecurity']->check()) {
+            if ( ! $GLOBALS['xoopsSecurity']->check()) {
                 redirect_header('instr.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             // ID инструкции

admin/admin_header.php

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
 
 //if (is_object($GLOBALS['xoopsUser'])) {
 if ($GLOBALS['xoopsUser'] instanceof XoopsUser) {
-    if (!$helper->isUserAdmin()) {
+    if ( ! $helper->isUserAdmin()) {
         $helper->redirect(XOOPS_URL . '/', 3, _NOPERM);
     }
 } else {
@@ -22,7 +22,7 @@ 
 /** @var Xmf\Module\Admin $adminObject */
 $adminObject = \Xmf\Module\Admin::getInstance();
 
-if (!isset($GLOBALS['xoopsTpl']) || !($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
+if ( ! isset($GLOBALS['xoopsTpl']) || ! ($GLOBALS['xoopsTpl'] instanceof XoopsTpl)) {
     require_once $GLOBALS['xoops']->path('class/template.php');
     $xoopsTpl = new \XoopsTpl();
 }

admin/menu.php

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

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

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>';
         }
         // Если были ошибки