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

class/InstructionHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 class InstructionHandler extends \XoopsPersistableObjectHandler
 {
     /**
-     * @param null|mixed $db
+     * @param null|\XoopsDatabase $db
      */
     public function __construct(\XoopsDatabase $db = null)
     {

Spacing +2 added lines, -2 removed lines

@@ -32,9 +32,9 @@
     public function updateDateupdated($instrid = 0, $time = null)
     {
         // Если не передали время
-        $time = null === $time ? time() : (int)$time;
+        $time = null === $time ? time() : (int) $time;
         //
-        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `instrid` = %u', $this->table, $time, (int)$instrid);
+        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `instrid` = %u', $this->table, $time, (int) $instrid);
         //
         return $this->db->query($sql);
     }

class/Tree.php

Doc Comments +9 added lines, -9 removed lines

@@ -23,9 +23,9 @@ 
     //    }
 
     /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
+     * @param        integer $key
+     * @param        string $ret
+     * @param        string $prefix_orig
      * @param        $objInsinstr
      * @param string $class
      * @param string $prefix_curr
@@ -140,9 +140,9 @@ 
     // ==================================
 
     /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
+     * @param        integer $key
+     * @param        string $ret
+     * @param        string $prefix_orig
      * @param array  $cidinstrids
      * @param string $class
      * @param string $prefix_curr
@@ -239,8 +239,8 @@ 
     // ======================================
 
     /**
-     * @param       $key
-     * @param       $ret
+     * @param       integer $key
+     * @param       string $ret
      * @param int   $currpageid
      * @param array $lastpageids
      * @param int   $level
@@ -350,7 +350,7 @@ 
     // Находим предыдущую и следующую страницы.
     // Находим последнии страницы на каждом уровне.
     /**
-     * @param       $key
+     * @param       integer $key
      * @param int   $currpageid
      * @param array $prevpages
      * @param array $nextpages

Spacing +4 added lines, -4 removed lines

@@ -78,7 +78,7 @@ 
             $prefix_curr .= $prefix_orig;
         }
 
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        if (isset($this->tree[$key]['child']) && ! empty($this->tree[$key]['child'])) {
             foreach ($this->tree[$key]['child'] as $childkey) {
                 $this->_makePagesAdminOptions($childkey, $ret, $prefix_orig, $objInsinstr, $class, $prefix_curr);
             }
@@ -172,7 +172,7 @@ 
             $prefix_curr .= $prefix_orig;
         }
 
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        if (isset($this->tree[$key]['child']) && ! empty($this->tree[$key]['child'])) {
             foreach ($this->tree[$key]['child'] as $childkey) {
                 $this->_makeCatsAdminOptions($childkey, $ret, $prefix_orig, $cidinstrids, $class, $prefix_curr);
             }
@@ -289,7 +289,7 @@ 
         }
 
         // Рекурсия
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        if (isset($this->tree[$key]['child']) && ! empty($this->tree[$key]['child'])) {
             foreach ($this->tree[$key]['child'] as $childkey) {
                 $this->_makePagesUserTree($childkey, $ret, $currpageid, $lastpageids, $level);
             }
@@ -350,7 +350,7 @@ 
         }
 
         // Рекурсия
-        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {
+        if (isset($this->tree[$key]['child']) && ! empty($this->tree[$key]['child'])) {
             foreach ($this->tree[$key]['child'] as $childkey) {
                 $this->_makePagesUserCalc($childkey, $currpageid, $prevpages, $nextpages, $lastpageids, $level);
             }

class/Utility.php

Doc Comments +2 added lines, -2 removed lines

@@ -46,8 +46,8 @@
     // Редактор
 
     /**
-     * @param        $caption
-     * @param        $name
+     * @param        string $caption
+     * @param        string $name
      * @param string $value
      * @return bool|\XoopsFormEditor
      */

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) {

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