mambax7 / instruction

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

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
      */

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