mambax7 / instruction

class/instruction.php

Spacing +3 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
         $form->addElement(new XoopsFormText(_AM_INSTRUCTION_METADESCRIPTIONC, 'metadescription', 50, 255, $this->getVar('metadescription')), false);
 
         // Если мы редактируем категорию
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $form->addElement(new XoopsFormHidden('instrid', $this->getVar('instrid')));
         }
         //
@@ -118,9 +118,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/page.php

Spacing +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
         // Находим все страницы данной инструкции
         $criteria->add(new Criteria('instrid', $instrid_page, '='));
         // Если мы редактируем, то убрать текущую страницу из списка выбора родительской
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $criteria->add(new Criteria('pageid', $this->getVar('pageid'), '<>'));
         }
         $criteria->setSort('weight');
@@ -128,7 +128,7 @@ 
         $form->addElement($option_tray);
 
         // Если мы редактируем страницу
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $form->addElement(new XoopsFormHidden('pageid', $this->getVar('pageid')));
         } else {
             $form->addElement(new XoopsFormHidden('pageid', 0));

class/category.php

Spacing +7 added lines, -7 removed lines

@@ -83,7 +83,7 @@ 
         $instructioncatHandler = xoops_getModuleHandler('category', 'instruction');
         $criteria              = new CriteriaCompo();
         // Если мы редактируем, то убрать текущую категорию из списка выбора родительской
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $criteria->add(new Criteria('cid', $this->getVar('cid'), '<>'));
         }
         $criteria->setSort('weight ASC, title');
@@ -128,7 +128,7 @@ 
         // Права на просмотр
         $groups_ids = [];
         // Если мы редактируем
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $groups_ids        = $gpermHandler->getGroupIds('instruction_view', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
             $groups_ids        = array_values($groups_ids);
             $groups_instr_view = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_VIEW, 'groups_instr_view', $groups_ids);
@@ -140,7 +140,7 @@ 
 
         // Права на отправку
         $groups_ids = [];
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $groups_ids          = $gpermHandler->getGroupIds('instruction_submit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
             $groups_ids          = array_values($groups_ids);
             $groups_instr_submit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_SUBMIT, 'groups_instr_submit', $groups_ids);
@@ -152,7 +152,7 @@ 
 
         // Права на редактирование
         $groups_ids = [];
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $groups_ids        = $gpermHandler->getGroupIds('instruction_edit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
             $groups_ids        = array_values($groups_ids);
             $groups_instr_edit = new XoopsFormCheckBox(_AM_INSTRUCTION_PERM_EDIT, 'groups_instr_edit', $groups_ids);
@@ -166,7 +166,7 @@ 
         // ==========================================================
 
         // Если мы редактируем категорию
-        if (!$this->isNew()) {
+        if ( ! $this->isNew()) {
             $form->addElement(new XoopsFormHidden('cid', $this->getVar('cid')));
             //$form->addElement( new XoopsFormHidden( 'catmodify', true));
         }
@@ -196,9 +196,9 @@ 
     public function updateDateupdated($cid = 0, $time = null)
     {
         // Если не передали время
-        $time = null === $time ? time() : (int)$time;
+        $time = null === $time ? time() : (int) $time;
         //
-        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `cid` = %u', $this->table, $time, (int)$cid);
+        $sql = sprintf('UPDATE `%s` SET `dateupdated` = %u WHERE `cid` = %u', $this->table, $time, (int) $cid);
         //
         return $this->db->query($sql);
     }

class/utility.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) {

class/tree.php

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

index.php

Spacing +3 added lines, -3 removed lines

@@ -21,9 +21,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;
 //
 $limit = xoops_getModuleOption('perpagemain', 'instruction');
 // Права на просмотр
@@ -52,7 +52,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, '='));

admin/admin_header.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 
 if ($xoopsUser) {
     $modulepermHandler = xoops_getHandler('groupperm');
-    if (!$modulepermHandler->checkRight('module_admin', $xoopsModule->getVar('mid'), $xoopsUser->getGroups())) {
+    if ( ! $modulepermHandler->checkRight('module_admin', $xoopsModule->getVar('mid'), $xoopsUser->getGroups())) {
         redirect_header(XOOPS_URL, 1, _NOPERM);
         exit();
     }
@@ -26,7 +26,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

@@ -1,6 +1,6 @@
 <?php
 
-if (!isset($moduleDirName)) {
+if ( ! isset($moduleDirName)) {
     $moduleDirName = basename(dirname(__DIR__));
 }
 

admin/ajax.php

Spacing +13 added lines, -13 removed lines

@@ -38,10 +38,10 @@ 
         $type        = InstructionUtility::cleanVars($_POST, 'type', 0, 'int');
         $keywords    = InstructionUtility::cleanVars($_POST, 'keywords', '', 'string');
         $description = InstructionUtility::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      = InstructionUtility::cleanVars( $_POST, 'dohtml', 0, 'int' );
         //$dosmiley    = InstructionUtility::cleanVars( $_POST, 'dosmiley', 0, 'int' );
         //$doxcode     = InstructionUtility::cleanVars( $_POST, 'doxcode', 0, 'int' );
@@ -50,7 +50,7 @@ 
         $instrid = InstructionUtility::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>';
@@ -110,28 +110,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>';
         }