mambax7 / instruction

class/utility.php

Unused Use Statements -2 removed lines

@@ -1,7 +1,5 @@
 <?php
 
-use Xmf\Request;
-
 require_once __DIR__ . '/common/traitversionchecks.php';
 require_once __DIR__ . '/common/traitserverstats.php';
 require_once __DIR__ . '/common/traitfilesmgmt.php';

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
                 break;
             case 'int':
             default:
-                $ret = isset($global[$key]) ? (int)$global[$key] : (int)$default;
+                $ret = isset($global[$key]) ? (int) $global[$key] : (int) $default;
                 break;
         }
         if (false === $ret) {

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/common.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
  * @author       XOOPS Development Team
  */
 
-if (!defined('INSTRUCTION_MODULE_PATH')) {
+if ( ! defined('INSTRUCTION_MODULE_PATH')) {
     define('INSTRUCTION_DIRNAME', basename(dirname(__DIR__)));
     define('INSTRUCTION_URL', XOOPS_URL . '/modules/' . INSTRUCTION_DIRNAME);
     define('INSTRUCTION_IMAGE_URL', INSTRUCTION_URL . '/assets/images/');
@@ -45,7 +45,7 @@ 
 $instruction = Instruction::getInstance();
 $instruction->loadLanguage('common');
 
-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();
 }

include/plugin.tag.php

Spacing +3 added lines, -3 removed lines

@@ -3,14 +3,14 @@ 
 // Информация об теге
 function instruction_tag_iteminfo(&$items)
 {
-    if (empty($items) || !is_array($items)) {
+    if (empty($items) || ! is_array($items)) {
         return false;
     }
 
     $items_id = [];
     foreach (array_keys($items) as $cat_id) {
         foreach (array_keys($items[$cat_id]) as $item_id) {
-            $items_id[] = (int)$item_id;
+            $items_id[] = (int) $item_id;
         }
     }
 
@@ -62,7 +62,7 @@ 
                . '            OR aa.status < 1'
                . '        )';
     endif;
-    if (!$result = $linkHandler->db->queryF($sql)) {
+    if ( ! $result = $linkHandler->db->queryF($sql)) {
         //xoops_error($linkHandler->db->error());
     }
 }

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;
     }
 

submit.php

Spacing +16 added lines, -16 removed lines

@@ -14,17 +14,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 = InstructionUtility::getItemIds($moduleDirName . '_submit');
@@ -54,7 +54,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->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);
             }
             // Создание новой страницы
@@ -67,7 +67,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->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 {
@@ -102,7 +102,7 @@ 
     case 'savepage':
 
         // Проверка
-        if (!$GLOBALS['xoopsSecurity']->check()) {
+        if ( ! $GLOBALS['xoopsSecurity']->check()) {
             redirect_header('index.php', 3, implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
 
@@ -115,7 +115,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->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) {
@@ -123,7 +123,7 @@ 
             // Объект инструкции
             $objInsinstr = $insinstrHandler->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);
             }
 
@@ -152,18 +152,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

Spacing +2 added lines, -2 removed lines

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

class/common/traitserverstats.php

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
         $html .= "<ul>\n";
         //
         $gdlib = function_exists('gd_info') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
-        $html  .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
+        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
         if (function_exists('gd_info')) {
             if ($gdlib = true === gd_info()) {
                 $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
@@ -56,7 +56,7 @@ 
         //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
         //
         $downloads = ini_get('file_uploads') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
-        $html      .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
+        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
         //
         $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: blue;">' . ini_get('upload_max_filesize') . "</span></b>\n";
         $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: blue;">' . ini_get('post_max_size') . "</span></b>\n";

class/common/traitversionchecks.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
                     break;
                 }
             } else {
-                if ((int)$v > 0) { // handles versions like x.x.x.0_RC2
+                if ((int) $v > 0) { // handles versions like x.x.x.0_RC2
                     $success = false;
                     break;
                 }