mambax7 / instruction

class/Tree.php

Indentation +364 added lines, -364 removed lines

@@ -18,96 +18,96 @@ 
  */
 class Tree extends \XoopsObjectTree
 {
-    //    public function __construct()
-    //    {
-    //    }
-
-    /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
-     * @param        $objInsinstr
-     * @param string $class
-     * @param string $prefix_curr
-     */
-    public function _makePagesAdminOptions($key, &$ret, $prefix_orig, $objInsinstr, $class = 'odd', $prefix_curr = '')
-    {
-        if ($key > 0) {
-
-            //
-            $class = ('even' === $class) ? 'odd' : 'even';
-            // ID инструкции ( Можно сделать статической )
-            $instrid = $objInsinstr->getVar('instrid');
-
-            // ID страницы
-            $pageid = $this->tree[$key]['obj']->getVar('pageid');
-            // Название страницы
-            $pagetitle = $this->tree[$key]['obj']->getVar('title');
-            // Вес
-            $pageweight = $this->tree[$key]['obj']->getVar('weight');
-            // Статус
-            $pagestatus = $this->tree[$key]['obj']->getVar('status');
-            // Тип страницы
-            $pagetype = $this->tree[$key]['obj']->getVar('type');
-
-            // Дочернии страницы
-            $page_childs = $this->getAllChild($pageid);
-            // Число дочерних страниц
-            $num_childs = count($page_childs);
-
-            // Действие - удаление
-            $act_del = ($num_childs > 0) ? '<img src="../assets/images/icons/16/application_key.png" alt="' . _AM_INSTR_NODELPAGE . '" title="' . _AM_INSTR_NODELPAGE . '" />' : '<a href="instr.php?op=delpage&pageid='
-                                                                                                                                                                      . $pageid
-                                                                                                                                                                      . '"><img src="../assets/icons/delete_mini.png" alt="'
-                                                                                                                                                                      . _AM_INSTRUCTION_DEL
-                                                                                                                                                                      . '" title="'
-                                                                                                                                                                      . _AM_INSTRUCTION_DEL
-                                                                                                                                                                      . '"></a>';
-            //
-            $page_link = '<a name="pageid_' . $pageid . '" ' . ($pagetype ? 'href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext"' : '') . '>' . $pagetitle . '</a>';
-
-            $ret .= '<tr class="' . $class . '">
+	//    public function __construct()
+	//    {
+	//    }
+
+	/**
+	 * @param        $key
+	 * @param        $ret
+	 * @param        $prefix_orig
+	 * @param        $objInsinstr
+	 * @param string $class
+	 * @param string $prefix_curr
+	 */
+	public function _makePagesAdminOptions($key, &$ret, $prefix_orig, $objInsinstr, $class = 'odd', $prefix_curr = '')
+	{
+		if ($key > 0) {
+
+			//
+			$class = ('even' === $class) ? 'odd' : 'even';
+			// ID инструкции ( Можно сделать статической )
+			$instrid = $objInsinstr->getVar('instrid');
+
+			// ID страницы
+			$pageid = $this->tree[$key]['obj']->getVar('pageid');
+			// Название страницы
+			$pagetitle = $this->tree[$key]['obj']->getVar('title');
+			// Вес
+			$pageweight = $this->tree[$key]['obj']->getVar('weight');
+			// Статус
+			$pagestatus = $this->tree[$key]['obj']->getVar('status');
+			// Тип страницы
+			$pagetype = $this->tree[$key]['obj']->getVar('type');
+
+			// Дочернии страницы
+			$page_childs = $this->getAllChild($pageid);
+			// Число дочерних страниц
+			$num_childs = count($page_childs);
+
+			// Действие - удаление
+			$act_del = ($num_childs > 0) ? '<img src="../assets/images/icons/16/application_key.png" alt="' . _AM_INSTR_NODELPAGE . '" title="' . _AM_INSTR_NODELPAGE . '" />' : '<a href="instr.php?op=delpage&pageid='
+																																									  . $pageid
+																																									  . '"><img src="../assets/icons/delete_mini.png" alt="'
+																																									  . _AM_INSTRUCTION_DEL
+																																									  . '" title="'
+																																									  . _AM_INSTRUCTION_DEL
+																																									  . '"></a>';
+			//
+			$page_link = '<a name="pageid_' . $pageid . '" ' . ($pagetype ? 'href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext"' : '') . '>' . $pagetitle . '</a>';
+
+			$ret .= '<tr class="' . $class . '">
       <td>' . $prefix_curr . ' ' . $page_link . '</td>
       <td align="center" width="50">
         <input type="text" name="weights[]" size="2" value="' . $pageweight . '" />
         <input type="hidden" name="pageids[]" value="' . $pageid . '" />
       </td>
       <td align="center" width="180">';
-            // Просмотре без кэша
-            $ret .= ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '&amp;nocache=1"><img src="../assets/images/icons/16/database_black.png" alt="' . _AM_INSTR_DISPLAY_NOCACHE . '" title="' . _AM_INSTR_DISPLAY_NOCACHE . '" /></a> ';
-            // Добавить подстраницу
-            $ret .= ' <a href="instr.php?op=editpage&instrid=' . $instrid . '&pid=' . $pageid . '"><img src="'. $pathIcon16 . '/add.png" alt="' . _AM_INSTRUCTION_ADDSUBPAGE . '" title="' . _AM_INSTRUCTION_ADDSUBPAGE . '" /></a> ';
-
-            if ($pagestatus) {
-                $ret .= ' <img src="../assets/images/icons/16/lock.png" alt="' . _AM_INSTRUCTION_LOCK . '" title="' . _AM_INSTRUCTION_LOCK . '"> ';
-            } else {
-                $ret .= ' <img src="../assets/images/icons/16/lock_open.png" alt="' . _AM_INSTRUCTION_UNLOCK . '" title="' . _AM_INSTRUCTION_UNLOCK . '"> ';
-            }
-
-            $ret .= ' <a href="instr.php?op=editpage&pageid=' . $pageid . '"><img src="'. $pathIcon16 . '/edit.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a> ' . $act_del . '
+			// Просмотре без кэша
+			$ret .= ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '&amp;nocache=1"><img src="../assets/images/icons/16/database_black.png" alt="' . _AM_INSTR_DISPLAY_NOCACHE . '" title="' . _AM_INSTR_DISPLAY_NOCACHE . '" /></a> ';
+			// Добавить подстраницу
+			$ret .= ' <a href="instr.php?op=editpage&instrid=' . $instrid . '&pid=' . $pageid . '"><img src="'. $pathIcon16 . '/add.png" alt="' . _AM_INSTRUCTION_ADDSUBPAGE . '" title="' . _AM_INSTRUCTION_ADDSUBPAGE . '" /></a> ';
+
+			if ($pagestatus) {
+				$ret .= ' <img src="../assets/images/icons/16/lock.png" alt="' . _AM_INSTRUCTION_LOCK . '" title="' . _AM_INSTRUCTION_LOCK . '"> ';
+			} else {
+				$ret .= ' <img src="../assets/images/icons/16/lock_open.png" alt="' . _AM_INSTRUCTION_UNLOCK . '" title="' . _AM_INSTRUCTION_UNLOCK . '"> ';
+			}
+
+			$ret .= ' <a href="instr.php?op=editpage&pageid=' . $pageid . '"><img src="'. $pathIcon16 . '/edit.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a> ' . $act_del . '
       </td>
     </tr>';
 
-            // Устанавливаем префикс
-            $prefix_curr .= $prefix_orig;
-        }
-
-        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);
-            }
-        }
-    }
-
-    /**
-     * @param        $objInsinstr
-     * @param string $prefix
-     * @param int    $key
-     * @return string
-     */
-    public function makePagesAdmin(&$objInsinstr, $prefix = '-', $key = 0)
-    {
-        $ret = '<form name="inspages" action="instr.php" method="post">
+			// Устанавливаем префикс
+			$prefix_curr .= $prefix_orig;
+		}
+
+		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);
+			}
+		}
+	}
+
+	/**
+	 * @param        $objInsinstr
+	 * @param string $prefix
+	 * @param int    $key
+	 * @return string
+	 */
+	public function makePagesAdmin(&$objInsinstr, $prefix = '-', $key = 0)
+	{
+		$ret = '<form name="inspages" action="instr.php" method="post">
   <table width="100%" cellspacing="1" class="outer">
     <tr>
       <th align="center" colspan="3">' . sprintf(_AM_INSTRUCTION_LISTPAGESININSTR, $objInsinstr->getVar('title')) . '</th>
@@ -118,10 +118,10 @@ 
       <td class="head" align="center" width="180">' . _AM_INSTRUCTION_ACTION . '</td>
     </tr>';
 
-        // Выводим все страницы
-        $this->_makePagesAdminOptions($key, $ret, $prefix, $objInsinstr);
+		// Выводим все страницы
+		$this->_makePagesAdminOptions($key, $ret, $prefix, $objInsinstr);
 
-        $ret .= '<tr class="foot">
+		$ret .= '<tr class="foot">
       <td><a href="instr.php?op=editpage&instrid=' . $objInsinstr->getVar('instrid') . '"><img src="'. $pathIcon16 . '/add.png" alt="' . _AM_INSTRUCTION_ADDPAGE . '" title="' . _AM_INSTRUCTION_ADDPAGE . '"></a></td>
       <td colspan="2">
         <input type="hidden" name="instrid" value="' . $objInsinstr->getVar('instrid') . '" />
@@ -132,61 +132,61 @@ 
   </table>
 </form>';
 
-        return $ret;
-    }
-
-    // ==================================
-    // === Дерево категорий в админке ===
-    // ==================================
-
-    /**
-     * @param        $key
-     * @param        $ret
-     * @param        $prefix_orig
-     * @param array  $cidinstrids
-     * @param string $class
-     * @param string $prefix_curr
-     */
-    public function _makeCatsAdminOptions($key, &$ret, $prefix_orig, $cidinstrids = [], &$class = 'odd', $prefix_curr = '')
-    {
-        $pathIcon16    = \Xmf\Module\Admin::iconUrl('', 16);
-        if ($key > 0) {
-
-            //
-            $class = ('even' === $class) ? 'odd' : 'even';
-
-            // ID категории
-            $catid = $this->tree[$key]['obj']->getVar('cid');
-            // Название категории
-            $cattitle = $this->tree[$key]['obj']->getVar('title');
-            // Вес
-            $catweight = $this->tree[$key]['obj']->getVar('weight');
-            // Статус
-            $pagestatus = $this->tree[$key]['obj']->getVar('status');
-
-            // Дочернии категории
-            $cat_childs = $this->getAllChild($catid);
-            // Число дочерних категорий
-            $num_childs = count($cat_childs);
-            // Число инструкций
-            $num_instrs = isset($cidinstrids[$catid]) ? $cidinstrids[$catid] : 0;
-
-            // Действие - удаление
-            $act_del = (($num_instrs > 0) || ($num_childs > 0)) ? '<img src="'. $pathIcon16 . '/delete.png" alt="' . _AM_INSTR_NODELCAT . '" title="' . _AM_INSTR_NODELCAT . '" />' : '<a href="cat.php?op=delcat&cid='
-                                                                                                                                                                                           . $catid
-                                                                                                                                                                                           . '"><img src="'. $pathIcon16 . '/delete.png" alt="'
-                                                                                                                                                                                           . _AM_INSTRUCTION_DEL
-                                                                                                                                                                                           . '" title="'
-                                                                                                                                                                                           . _AM_INSTRUCTION_DEL
-                                                                                                                                                                                           . '" /></a>';
-            // Действие - просмотр
-            $act_view = ($num_instrs > 0) ? '<a href="instr.php?cid=' . $catid . '"><img src="'. $pathIcon16 . '/view.png" alt="' . _AM_INSTR_VIEWINSTR . '" title="' . _AM_INSTR_VIEWINSTR . '" /></a>' : '<img src="../assets/images/icons/16/document_protect.png" alt="'
-                                                                                                                                                                                                             . _AM_INSTR_NOVIEWINSTR
-                                                                                                                                                                                                             . '" title="'
-                                                                                                                                                                                                             . _AM_INSTR_NOVIEWINSTR
-                                                                                                                                                                                                             . '" />';
-
-            $ret .= '<tr class="' . $class . '">
+		return $ret;
+	}
+
+	// ==================================
+	// === Дерево категорий в админке ===
+	// ==================================
+
+	/**
+	 * @param        $key
+	 * @param        $ret
+	 * @param        $prefix_orig
+	 * @param array  $cidinstrids
+	 * @param string $class
+	 * @param string $prefix_curr
+	 */
+	public function _makeCatsAdminOptions($key, &$ret, $prefix_orig, $cidinstrids = [], &$class = 'odd', $prefix_curr = '')
+	{
+		$pathIcon16    = \Xmf\Module\Admin::iconUrl('', 16);
+		if ($key > 0) {
+
+			//
+			$class = ('even' === $class) ? 'odd' : 'even';
+
+			// ID категории
+			$catid = $this->tree[$key]['obj']->getVar('cid');
+			// Название категории
+			$cattitle = $this->tree[$key]['obj']->getVar('title');
+			// Вес
+			$catweight = $this->tree[$key]['obj']->getVar('weight');
+			// Статус
+			$pagestatus = $this->tree[$key]['obj']->getVar('status');
+
+			// Дочернии категории
+			$cat_childs = $this->getAllChild($catid);
+			// Число дочерних категорий
+			$num_childs = count($cat_childs);
+			// Число инструкций
+			$num_instrs = isset($cidinstrids[$catid]) ? $cidinstrids[$catid] : 0;
+
+			// Действие - удаление
+			$act_del = (($num_instrs > 0) || ($num_childs > 0)) ? '<img src="'. $pathIcon16 . '/delete.png" alt="' . _AM_INSTR_NODELCAT . '" title="' . _AM_INSTR_NODELCAT . '" />' : '<a href="cat.php?op=delcat&cid='
+																																														   . $catid
+																																														   . '"><img src="'. $pathIcon16 . '/delete.png" alt="'
+																																														   . _AM_INSTRUCTION_DEL
+																																														   . '" title="'
+																																														   . _AM_INSTRUCTION_DEL
+																																														   . '" /></a>';
+			// Действие - просмотр
+			$act_view = ($num_instrs > 0) ? '<a href="instr.php?cid=' . $catid . '"><img src="'. $pathIcon16 . '/view.png" alt="' . _AM_INSTR_VIEWINSTR . '" title="' . _AM_INSTR_VIEWINSTR . '" /></a>' : '<img src="../assets/images/icons/16/document_protect.png" alt="'
+																																																			 . _AM_INSTR_NOVIEWINSTR
+																																																			 . '" title="'
+																																																			 . _AM_INSTR_NOVIEWINSTR
+																																																			 . '" />';
+
+			$ret .= '<tr class="' . $class . '">
       <td>' . $prefix_curr . ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/index.php?cid=' . $catid . '">' . $cattitle . '</a></td>
       <td align="center" width="50">' . $catweight . '</td>
       <td align="center" width="100">' . $num_instrs . '</td>
@@ -197,26 +197,26 @@ 
       </td>
     </tr>';
 
-            // Устанавливаем префикс
-            $prefix_curr .= $prefix_orig;
-        }
-
-        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);
-            }
-        }
-    }
-
-    /**
-     * @param string $prefix
-     * @param array  $cidinstrids
-     * @param int    $key
-     * @return string
-     */
-    public function makeCatsAdmin($prefix = '-', $cidinstrids = [], $key = 0)
-    {
-        $ret = '<table width="100%" cellspacing="1" class="outer">
+			// Устанавливаем префикс
+			$prefix_curr .= $prefix_orig;
+		}
+
+		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);
+			}
+		}
+	}
+
+	/**
+	 * @param string $prefix
+	 * @param array  $cidinstrids
+	 * @param int    $key
+	 * @return string
+	 */
+	public function makeCatsAdmin($prefix = '-', $cidinstrids = [], $key = 0)
+	{
+		$ret = '<table width="100%" cellspacing="1" class="outer">
     <tr>
       <th align="center" colspan="4">' . _AM_INSTR_LISTALLCATS . '</th>
     </tr>
@@ -227,216 +227,216 @@ 
       <td class="head" align="center" width="150">' . _AM_INSTRUCTION_ACTION . '</td>
     </tr>';
 
-        // Выводим все страницы
-        $this->_makeCatsAdminOptions($key, $ret, $prefix, $cidinstrids);
-
-        $ret .= '</table>';
-
-        return $ret;
-    }
-
-    // ======================================
-    // Список страниц на стороне пользователя
-    // ======================================
-
-    /**
-     * @param       $key
-     * @param       $ret
-     * @param int   $currpageid
-     * @param array $lastpageids
-     * @param int   $level
-     */
-    public function _makePagesUserTree($key, &$ret, $currpageid = 0, &$lastpageids = [], $level = 0)
-    {
-
-        // Сохраняем значение предыдущей страницы
-        //static $stat_prevpages;
-
-        if ($key > 0) {
-
-            // ID страницы
-            $pageid = $this->tree[$key]['obj']->getVar('pageid');
-            // Название страницы
-            $pagetitle = $this->tree[$key]['obj']->getVar('title');
-            // Тип страницы
-            $pagetype = $this->tree[$key]['obj']->getVar('type');
-
-            // Дочернии категории
-            $page_childs = $this->getAllChild($pageid);
-            // Число дочерних страниц
-            $num_childs = count($page_childs);
-
-            // Генерируем класс
-            // InstrTreeNode InstrTreeIsRoot InstrTreeExpandClosed InstrTreeIsLast
-            $class = [];
-            // Данный класс должен быть у любого узла
-            $class[] = 'InstrTreeNode';
-            // Если узел нулевого уровня, добавляем InstrTreeIsRoot
-            if (0 === $level) {
-                $class[] = 'InstrTreeIsRoot';
-            }
-            // Тип узла InstrTreeExpandClosed|InstrTreeExpandLeaf
-            // Если у узла нет потомков - InstrTreeExpandLeaf
-            if (0 == $num_childs) {
-                $class[] = 'InstrTreeExpandLeaf';
-                // Если у искомого элемента есть потомки - открываем список
-            } elseif ($currpageid == $pageid) {
-                $class[] = 'InstrTreeExpandOpen';
-                // Если искомый элемент есть в потомках текущего, то ставим класс InstrTreeExpandOpen
-            } elseif (array_key_exists($currpageid, $page_childs)) {
-                $class[] = 'InstrTreeExpandOpen';
-                //
-            } else {
-                $class[] = 'InstrTreeExpandClosed';
-            }
-
-            // Данный класс нужно добавлять последнему узлу в каждом уровне
-
-            if (isset($lastpageids[$level]) && ($pageid == $lastpageids[$level])) {
-                $class[] = 'InstrTreeIsLast';
-            }
-
-            //$class[] = 'InstrTreeIsLast';
-
-            // Test
-            //$ret .= '<div id="' . $pageid . '">';
-
-            // Создаём запись
-            $ret .= '<li class="' . implode(' ', $class) . '">';
-            //
-            $ret .= '<div class="InstrTreeExpand"></div>';
-            //
-            $ret .= '<div class="InstrTreeContent">';
-
-            // Если это лист дерева
-            if (0 == $pagetype) {
-                $ret .= '<span class="InstrTreeEmptyPage">' . $pagetitle . '</span>';
-                //
-            } elseif ($currpageid == $pageid) {
-                $ret .= $pagetitle;
-                //
-            } else {
-                $ret .= '<a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext">' . $pagetitle . '</a>';
-            }
-
-            $ret .= '</div>';
-
-            // Если есть потомки
-            if ($num_childs > 0) {
-                $ret .= '<ul class="InstrTreeContainer">';
-            }
-
-            // Инкримент уровня
-            $level++;
-        }
-
-        // Рекурсия
-        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);
-            }
-        }
-
-        // Test
-        if ($key > 0) {
-            // Если есть потомки
-            if ($num_childs > 0) {
-                $ret .= '</ul>';
-            }
-            // Конец текущей записи
-            $ret .= '</li>';
-        }
-    }
-
-    // Находим предыдущую и следующую страницы.
-    // Находим последнии страницы на каждом уровне.
-    /**
-     * @param       $key
-     * @param int   $currpageid
-     * @param array $prevpages
-     * @param array $nextpages
-     * @param array $lastpageids
-     * @param int   $level
-     */
-    public function _makePagesUserCalc($key, $currpageid = 0, &$prevpages = [], &$nextpages = [], &$lastpageids = [], $level = 0)
-    {
-
-        // Сохраняем значение предыдущей страницы
-        static $stat_prevpages;
-
-        if ($key > 0) {
-            // ID страницы
-            $pageid = $this->tree[$key]['obj']->getVar('pageid');
-            // Название страницы
-            $pagetitle = $this->tree[$key]['obj']->getVar('title');
-            // Тип страницы
-            $pagetype = $this->tree[$key]['obj']->getVar('type');
-
-            // Если мы передали ID текущей страницы, то находить предыдудую и следующую страницы
-            // Не находить предыдущие и следующие для "Пустой страницы"
-            if ($currpageid && $pagetype) {
-                // Если элемент равен текущей странице
-                if (null !== $stat_prevpages && ($currpageid == $pageid)) {
-                    // Забиваем массив предыдущей страницы
-                    $prevpages['pageid'] = $stat_prevpages['pageid'];
-                    $prevpages['title']  = $stat_prevpages['title'];
-
-                    // Если предыдущий равен текущей странице
-                } elseif (null !== $stat_prevpages && ($currpageid == $stat_prevpages['pageid'])) {
-                    // Забиваем массив следующей страницы
-                    $nextpages['pageid'] = $pageid;
-                    $nextpages['title']  = $pagetitle;
-                }
-                // Заносим текущие данные в массив предыдущей страницы
-                $stat_prevpages['pageid'] = $pageid;
-                $stat_prevpages['title']  = $pagetitle;
-            }
-
-            // Заносим текущую страницу в массив "последних страний"
-            $lastpageids[$level] = $pageid;
-
-            // Инкримент уровня
-            $level++;
-        }
-
-        // Рекурсия
-        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);
-            }
-        }
-    }
-
-    //
-
-    /**
-     * @param int   $currpageid
-     * @param array $prevpages
-     * @param array $nextpages
-     * @param int   $key
-     * @return string
-     */
-    public function makePagesUser($currpageid = 0, &$prevpages = [], &$nextpages = [], $key = 0)
-    {
-
-        // Массив последней страницы на каждом уровне
-        // level => pageid
-        $lastpageids = [];
-
-        // Расчёт
-        $this->_makePagesUserCalc($key, $currpageid, $prevpages, $nextpages, $lastpageids);
-
-        $ret = '<div onclick="instr_tree_toggle(arguments[0])">
+		// Выводим все страницы
+		$this->_makeCatsAdminOptions($key, $ret, $prefix, $cidinstrids);
+
+		$ret .= '</table>';
+
+		return $ret;
+	}
+
+	// ======================================
+	// Список страниц на стороне пользователя
+	// ======================================
+
+	/**
+	 * @param       $key
+	 * @param       $ret
+	 * @param int   $currpageid
+	 * @param array $lastpageids
+	 * @param int   $level
+	 */
+	public function _makePagesUserTree($key, &$ret, $currpageid = 0, &$lastpageids = [], $level = 0)
+	{
+
+		// Сохраняем значение предыдущей страницы
+		//static $stat_prevpages;
+
+		if ($key > 0) {
+
+			// ID страницы
+			$pageid = $this->tree[$key]['obj']->getVar('pageid');
+			// Название страницы
+			$pagetitle = $this->tree[$key]['obj']->getVar('title');
+			// Тип страницы
+			$pagetype = $this->tree[$key]['obj']->getVar('type');
+
+			// Дочернии категории
+			$page_childs = $this->getAllChild($pageid);
+			// Число дочерних страниц
+			$num_childs = count($page_childs);
+
+			// Генерируем класс
+			// InstrTreeNode InstrTreeIsRoot InstrTreeExpandClosed InstrTreeIsLast
+			$class = [];
+			// Данный класс должен быть у любого узла
+			$class[] = 'InstrTreeNode';
+			// Если узел нулевого уровня, добавляем InstrTreeIsRoot
+			if (0 === $level) {
+				$class[] = 'InstrTreeIsRoot';
+			}
+			// Тип узла InstrTreeExpandClosed|InstrTreeExpandLeaf
+			// Если у узла нет потомков - InstrTreeExpandLeaf
+			if (0 == $num_childs) {
+				$class[] = 'InstrTreeExpandLeaf';
+				// Если у искомого элемента есть потомки - открываем список
+			} elseif ($currpageid == $pageid) {
+				$class[] = 'InstrTreeExpandOpen';
+				// Если искомый элемент есть в потомках текущего, то ставим класс InstrTreeExpandOpen
+			} elseif (array_key_exists($currpageid, $page_childs)) {
+				$class[] = 'InstrTreeExpandOpen';
+				//
+			} else {
+				$class[] = 'InstrTreeExpandClosed';
+			}
+
+			// Данный класс нужно добавлять последнему узлу в каждом уровне
+
+			if (isset($lastpageids[$level]) && ($pageid == $lastpageids[$level])) {
+				$class[] = 'InstrTreeIsLast';
+			}
+
+			//$class[] = 'InstrTreeIsLast';
+
+			// Test
+			//$ret .= '<div id="' . $pageid . '">';
+
+			// Создаём запись
+			$ret .= '<li class="' . implode(' ', $class) . '">';
+			//
+			$ret .= '<div class="InstrTreeExpand"></div>';
+			//
+			$ret .= '<div class="InstrTreeContent">';
+
+			// Если это лист дерева
+			if (0 == $pagetype) {
+				$ret .= '<span class="InstrTreeEmptyPage">' . $pagetitle . '</span>';
+				//
+			} elseif ($currpageid == $pageid) {
+				$ret .= $pagetitle;
+				//
+			} else {
+				$ret .= '<a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext">' . $pagetitle . '</a>';
+			}
+
+			$ret .= '</div>';
+
+			// Если есть потомки
+			if ($num_childs > 0) {
+				$ret .= '<ul class="InstrTreeContainer">';
+			}
+
+			// Инкримент уровня
+			$level++;
+		}
+
+		// Рекурсия
+		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);
+			}
+		}
+
+		// Test
+		if ($key > 0) {
+			// Если есть потомки
+			if ($num_childs > 0) {
+				$ret .= '</ul>';
+			}
+			// Конец текущей записи
+			$ret .= '</li>';
+		}
+	}
+
+	// Находим предыдущую и следующую страницы.
+	// Находим последнии страницы на каждом уровне.
+	/**
+	 * @param       $key
+	 * @param int   $currpageid
+	 * @param array $prevpages
+	 * @param array $nextpages
+	 * @param array $lastpageids
+	 * @param int   $level
+	 */
+	public function _makePagesUserCalc($key, $currpageid = 0, &$prevpages = [], &$nextpages = [], &$lastpageids = [], $level = 0)
+	{
+
+		// Сохраняем значение предыдущей страницы
+		static $stat_prevpages;
+
+		if ($key > 0) {
+			// ID страницы
+			$pageid = $this->tree[$key]['obj']->getVar('pageid');
+			// Название страницы
+			$pagetitle = $this->tree[$key]['obj']->getVar('title');
+			// Тип страницы
+			$pagetype = $this->tree[$key]['obj']->getVar('type');
+
+			// Если мы передали ID текущей страницы, то находить предыдудую и следующую страницы
+			// Не находить предыдущие и следующие для "Пустой страницы"
+			if ($currpageid && $pagetype) {
+				// Если элемент равен текущей странице
+				if (null !== $stat_prevpages && ($currpageid == $pageid)) {
+					// Забиваем массив предыдущей страницы
+					$prevpages['pageid'] = $stat_prevpages['pageid'];
+					$prevpages['title']  = $stat_prevpages['title'];
+
+					// Если предыдущий равен текущей странице
+				} elseif (null !== $stat_prevpages && ($currpageid == $stat_prevpages['pageid'])) {
+					// Забиваем массив следующей страницы
+					$nextpages['pageid'] = $pageid;
+					$nextpages['title']  = $pagetitle;
+				}
+				// Заносим текущие данные в массив предыдущей страницы
+				$stat_prevpages['pageid'] = $pageid;
+				$stat_prevpages['title']  = $pagetitle;
+			}
+
+			// Заносим текущую страницу в массив "последних страний"
+			$lastpageids[$level] = $pageid;
+
+			// Инкримент уровня
+			$level++;
+		}
+
+		// Рекурсия
+		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);
+			}
+		}
+	}
+
+	//
+
+	/**
+	 * @param int   $currpageid
+	 * @param array $prevpages
+	 * @param array $nextpages
+	 * @param int   $key
+	 * @return string
+	 */
+	public function makePagesUser($currpageid = 0, &$prevpages = [], &$nextpages = [], $key = 0)
+	{
+
+		// Массив последней страницы на каждом уровне
+		// level => pageid
+		$lastpageids = [];
+
+		// Расчёт
+		$this->_makePagesUserCalc($key, $currpageid, $prevpages, $nextpages, $lastpageids);
+
+		$ret = '<div onclick="instr_tree_toggle(arguments[0])">
 <div>' . _MD_INSTRUCTION_LISTPAGES . '</div>
 <div><ul class="InstrTreeContainer">';
 
-        // Генерируем дерево
-        $this->_makePagesUserTree($key, $ret, $currpageid, $lastpageids);
+		// Генерируем дерево
+		$this->_makePagesUserTree($key, $ret, $currpageid, $lastpageids);
 
-        $ret .= '</ul>
+		$ret .= '</ul>
 </div>
 </div>';
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

Spacing +12 added lines, -12 removed lines

@@ -76,7 +76,7 @@ 
             // Просмотре без кэша
             $ret .= ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '&amp;nocache=1"><img src="../assets/images/icons/16/database_black.png" alt="' . _AM_INSTR_DISPLAY_NOCACHE . '" title="' . _AM_INSTR_DISPLAY_NOCACHE . '" /></a> ';
             // Добавить подстраницу
-            $ret .= ' <a href="instr.php?op=editpage&instrid=' . $instrid . '&pid=' . $pageid . '"><img src="'. $pathIcon16 . '/add.png" alt="' . _AM_INSTRUCTION_ADDSUBPAGE . '" title="' . _AM_INSTRUCTION_ADDSUBPAGE . '" /></a> ';
+            $ret .= ' <a href="instr.php?op=editpage&instrid=' . $instrid . '&pid=' . $pageid . '"><img src="' . $pathIcon16 . '/add.png" alt="' . _AM_INSTRUCTION_ADDSUBPAGE . '" title="' . _AM_INSTRUCTION_ADDSUBPAGE . '" /></a> ';
 
             if ($pagestatus) {
                 $ret .= ' <img src="../assets/images/icons/16/lock.png" alt="' . _AM_INSTRUCTION_LOCK . '" title="' . _AM_INSTRUCTION_LOCK . '"> ';
@@ -84,7 +84,7 @@ 
                 $ret .= ' <img src="../assets/images/icons/16/lock_open.png" alt="' . _AM_INSTRUCTION_UNLOCK . '" title="' . _AM_INSTRUCTION_UNLOCK . '"> ';
             }
 
-            $ret .= ' <a href="instr.php?op=editpage&pageid=' . $pageid . '"><img src="'. $pathIcon16 . '/edit.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a> ' . $act_del . '
+            $ret .= ' <a href="instr.php?op=editpage&pageid=' . $pageid . '"><img src="' . $pathIcon16 . '/edit.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a> ' . $act_del . '
       </td>
     </tr>';
 
@@ -92,7 +92,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);
             }
@@ -122,7 +122,7 @@ 
         $this->_makePagesAdminOptions($key, $ret, $prefix, $objInsinstr);
 
         $ret .= '<tr class="foot">
-      <td><a href="instr.php?op=editpage&instrid=' . $objInsinstr->getVar('instrid') . '"><img src="'. $pathIcon16 . '/add.png" alt="' . _AM_INSTRUCTION_ADDPAGE . '" title="' . _AM_INSTRUCTION_ADDPAGE . '"></a></td>
+      <td><a href="instr.php?op=editpage&instrid=' . $objInsinstr->getVar('instrid') . '"><img src="' . $pathIcon16 . '/add.png" alt="' . _AM_INSTRUCTION_ADDPAGE . '" title="' . _AM_INSTRUCTION_ADDPAGE . '"></a></td>
       <td colspan="2">
         <input type="hidden" name="instrid" value="' . $objInsinstr->getVar('instrid') . '" />
         <input type="hidden" name="op" value="updpage" />
@@ -149,7 +149,7 @@ 
      */
     public function _makeCatsAdminOptions($key, &$ret, $prefix_orig, $cidinstrids = [], &$class = 'odd', $prefix_curr = '')
     {
-        $pathIcon16    = \Xmf\Module\Admin::iconUrl('', 16);
+        $pathIcon16 = \Xmf\Module\Admin::iconUrl('', 16);
         if ($key > 0) {
 
             //
@@ -172,15 +172,15 @@ 
             $num_instrs = isset($cidinstrids[$catid]) ? $cidinstrids[$catid] : 0;
 
             // Действие - удаление
-            $act_del = (($num_instrs > 0) || ($num_childs > 0)) ? '<img src="'. $pathIcon16 . '/delete.png" alt="' . _AM_INSTR_NODELCAT . '" title="' . _AM_INSTR_NODELCAT . '" />' : '<a href="cat.php?op=delcat&cid='
+            $act_del = (($num_instrs > 0) || ($num_childs > 0)) ? '<img src="' . $pathIcon16 . '/delete.png" alt="' . _AM_INSTR_NODELCAT . '" title="' . _AM_INSTR_NODELCAT . '" />' : '<a href="cat.php?op=delcat&cid='
                                                                                                                                                                                            . $catid
-                                                                                                                                                                                           . '"><img src="'. $pathIcon16 . '/delete.png" alt="'
+                                                                                                                                                                                           . '"><img src="' . $pathIcon16 . '/delete.png" alt="'
                                                                                                                                                                                            . _AM_INSTRUCTION_DEL
                                                                                                                                                                                            . '" title="'
                                                                                                                                                                                            . _AM_INSTRUCTION_DEL
                                                                                                                                                                                            . '" /></a>';
             // Действие - просмотр
-            $act_view = ($num_instrs > 0) ? '<a href="instr.php?cid=' . $catid . '"><img src="'. $pathIcon16 . '/view.png" alt="' . _AM_INSTR_VIEWINSTR . '" title="' . _AM_INSTR_VIEWINSTR . '" /></a>' : '<img src="../assets/images/icons/16/document_protect.png" alt="'
+            $act_view = ($num_instrs > 0) ? '<a href="instr.php?cid=' . $catid . '"><img src="' . $pathIcon16 . '/view.png" alt="' . _AM_INSTR_VIEWINSTR . '" title="' . _AM_INSTR_VIEWINSTR . '" /></a>' : '<img src="../assets/images/icons/16/document_protect.png" alt="'
                                                                                                                                                                                                              . _AM_INSTR_NOVIEWINSTR
                                                                                                                                                                                                              . '" title="'
                                                                                                                                                                                                              . _AM_INSTR_NOVIEWINSTR
@@ -192,7 +192,7 @@ 
       <td align="center" width="100">' . $num_instrs . '</td>
       <td align="center" width="150">
         ' . $act_view . '
-        <a href="cat.php?op=editcat&cid=' . $catid . '"><img src="'. $pathIcon16 . '/edit.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a>
+        <a href="cat.php?op=editcat&cid=' . $catid . '"><img src="' . $pathIcon16 . '/edit.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a>
         ' . $act_del . '
       </td>
     </tr>';
@@ -201,7 +201,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);
             }
@@ -331,7 +331,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);
             }
@@ -400,7 +400,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);
             }

page.php

Indentation +43 added lines, -43 removed lines

@@ -29,8 +29,8 @@ 
 $criteria->add(new \Criteria('pageid ', $pageid));
 $criteria->add(new \Criteria('status ', '0', '>'));
 if (0 == $pageHandler->getCount($criteria)) {
-    redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
-    exit();
+	redirect_header('index.php', 3, _MD_INSTRUCTION_PAGENOTEXIST);
+	exit();
 }
 //
 unset($criteria);
@@ -42,8 +42,8 @@ 
 
 // Если админ и ссылка на отключение кэша
 if (($GLOBALS['xoopsUser'] instanceof \XoopsUser) && $GLOBALS['xoopsUser']->isAdmin() && $nocache) {
-    // Отключаем кэш
-    $GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
+	// Отключаем кэш
+	$GLOBALS['xoopsConfig']['module_cache'][$GLOBALS['xoopsModule']->getVar('mid')] = 0;
 }
 
 // Задание тайтла
@@ -60,8 +60,8 @@ 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::getItemIds();
 if (!in_array($objInsinstr->getVar('cid'), $categories)) {
-    redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
-    exit();
+	redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
+	exit();
 }
 
 // Массив данных о странице
@@ -78,9 +78,9 @@ 
 $footnote = $objInspage->getVar('footnote');
 // Если есть сноски
 if ($footnote) {
-    $pages['footnotes'] = explode('|', $objInspage->getVar('footnote'));
+	$pages['footnotes'] = explode('|', $objInspage->getVar('footnote'));
 } else {
-    $pages['footnotes'] = false;
+	$pages['footnotes'] = false;
 }
 // Мета-теги ключевых слов
 $pages['keywords'] = $objInspage->getVar('keywords');
@@ -89,39 +89,39 @@ 
 //
 // Если админ, рисуем админлинк
 if (($GLOBALS['xoopsUser'] instanceof \XoopsUser) && $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid())) {
-    $pages['adminlink'] = '&nbsp;<a href="'
-                          . XOOPS_URL
-                          . '/modules/'
-                          . $moduleDirName
-                          . '/admin/instr.php?op=editpage&pageid='
-                          . $pages['pageid']
-                          . '"><img style="width:16px;" src="'. $pathModIcon16 . '/assets/images/icons/16/edit.png" alt='
-                          . _EDIT
-                          . ' title='
-                          . _EDIT
-                          . '></a>&nbsp;<a href="'
-                          . XOOPS_URL
-                          . '/modules/'
-                          . $moduleDirName
-                          . '/admin/instr.php?op=delpage&pageid='
-                          . $pages['pageid']
-                          . '"><img style="width:16px;" src="./assets/icons/delete_mini.png" alt='
-                          . _DELETE
-                          . ' title='
-                          . _DELETE
-                          . '></a>&nbsp;';
+	$pages['adminlink'] = '&nbsp;<a href="'
+						  . XOOPS_URL
+						  . '/modules/'
+						  . $moduleDirName
+						  . '/admin/instr.php?op=editpage&pageid='
+						  . $pages['pageid']
+						  . '"><img style="width:16px;" src="'. $pathModIcon16 . '/assets/images/icons/16/edit.png" alt='
+						  . _EDIT
+						  . ' title='
+						  . _EDIT
+						  . '></a>&nbsp;<a href="'
+						  . XOOPS_URL
+						  . '/modules/'
+						  . $moduleDirName
+						  . '/admin/instr.php?op=delpage&pageid='
+						  . $pages['pageid']
+						  . '"><img style="width:16px;" src="./assets/icons/delete_mini.png" alt='
+						  . _DELETE
+						  . ' title='
+						  . _DELETE
+						  . '></a>&nbsp;';
 } else {
-    $pages['adminlink'] = '&nbsp;';
-    // Если можно редактировать
-    if ($gpermHandler->checkRight($moduleDirName . '_edit', $objInsinstr->getVar('cid'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
-        $pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
-    }
-
-    $pages['adminlink'] .= '&nbsp;';
-    // Если нет админлика
-    if ('[&nbsp;&nbsp;]' === $pages['adminlink']) {
-        $pages['adminlink'] = '';
-    }
+	$pages['adminlink'] = '&nbsp;';
+	// Если можно редактировать
+	if ($gpermHandler->checkRight($moduleDirName . '_edit', $objInsinstr->getVar('cid'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
+		$pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
+	}
+
+	$pages['adminlink'] .= '&nbsp;';
+	// Если нет админлика
+	if ('[&nbsp;&nbsp;]' === $pages['adminlink']) {
+		$pages['adminlink'] = '';
+	}
 }
 // Выводим в шаблон
 $GLOBALS['xoopsTpl']->assign('insPage', $pages);
@@ -140,7 +140,7 @@ 
 $nav_parent_id = array_reverse($nav_parent_id);
 $navigation    = '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/">' . $GLOBALS['xoopsModule']->name() . '</a>&nbsp;:&nbsp;';
 foreach (array_keys($nav_parent_id) as $i) {
-    $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
+	$navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
 }
 $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $objInscat->getVar('cid') . '">' . $objInscat->getVar('title') . '</a>&nbsp;:&nbsp;';
 $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/instr.php?id=' . $pages['instrid'] . '">' . $objInsinstr->getVar('title') . '</a>';
@@ -172,9 +172,9 @@ 
 
 // Рейтинг
 if (xoops_getModuleOption('userat', 'instruction')) {
-    $xoopsTpl->assign('insUserat', true);
+	$xoopsTpl->assign('insUserat', true);
 } else {
-    $xoopsTpl->assign('insUserat', false);
+	$xoopsTpl->assign('insUserat', false);
 }
 
 // Мета теги

Spacing +3 added lines, -3 removed lines

@@ -59,7 +59,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();
 }
@@ -95,7 +95,7 @@ 
                           . $moduleDirName
                           . '/admin/instr.php?op=editpage&pageid='
                           . $pages['pageid']
-                          . '"><img style="width:16px;" src="'. $pathModIcon16 . '/assets/images/icons/16/edit.png" alt='
+                          . '"><img style="width:16px;" src="' . $pathModIcon16 . '/assets/images/icons/16/edit.png" alt='
                           . _EDIT
                           . ' title='
                           . _EDIT
@@ -114,7 +114,7 @@ 
     $pages['adminlink'] = '&nbsp;';
     // Если можно редактировать
     if ($gpermHandler->checkRight($moduleDirName . '_edit', $objInsinstr->getVar('cid'), $groups, $GLOBALS['xoopsModule']->getVar('mid'))) {
-        $pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
+        $pages['adminlink'] .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/submit.php?op=editpage&pageid=' . $pages['pageid'] . '"><img style="width:16px;" src="' . $pathIcon16 . '/edit.png" alt=' . _EDIT . ' title=' . _EDIT . '></a>';
     }
 
     $pages['adminlink'] .= '&nbsp;';

admin/admin_header.php

Indentation +6 added lines, -6 removed lines

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

instr.php

Indentation +33 added lines, -33 removed lines

@@ -19,8 +19,8 @@ 
 $criteria->add(new \Criteria('instrid', $instrid));
 $criteria->add(new \Criteria('status ', '0', '>'));
 if (0 == $instructionHandler->getCount($criteria)) {
-    redirect_header('index.php', 3, _MD_INSTRUCTION_INSTRNOTEXIST);
-    exit();
+	redirect_header('index.php', 3, _MD_INSTRUCTION_INSTRNOTEXIST);
+	exit();
 }
 //
 unset($criteria);
@@ -42,8 +42,8 @@ 
 // Права на просмотр инструкции
 $categories = Xoopsmodules\instruction\Utility::getItemIds();
 if (!in_array($objInsinstr->getVar('cid'), $categories)) {
-    redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
-    exit();
+	redirect_header(XOOPS_URL . '/modules/' . $moduleDirName . '/', 3, _NOPERM);
+	exit();
 }
 
 // Массив данных об инструкции
@@ -56,29 +56,29 @@ 
 $instrs['description'] = $objInsinstr->getVar('description');
 // Если админ, рисуем админлинк
 if (($GLOBALS['xoopsUser'] instanceof \XoopsUser) && $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid())) {
-    $instrs['adminlink'] = '&nbsp;<a href="'
-                           . XOOPS_URL
-                           . '/modules/'
-                           . $moduleDirName
-                           . '/admin/instr.php?op=editinstr&instrid='
-                           . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt='
-                           . _EDIT
-                           . ' title='
-                           . _EDIT
-                           . '></a>&nbsp;<a href="'
-                           . XOOPS_URL
-                           . '/modules/'
-                           . $moduleDirName
-                           . '/admin/instr.php?op=delinstr&instrid='
-                           . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/delete.png" alt='
-                           . _DELETE
-                           . ' title='
-                           . _DELETE
-                           . '></a>&nbsp;';
+	$instrs['adminlink'] = '&nbsp;<a href="'
+						   . XOOPS_URL
+						   . '/modules/'
+						   . $moduleDirName
+						   . '/admin/instr.php?op=editinstr&instrid='
+						   . $instrid
+						   . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt='
+						   . _EDIT
+						   . ' title='
+						   . _EDIT
+						   . '></a>&nbsp;<a href="'
+						   . XOOPS_URL
+						   . '/modules/'
+						   . $moduleDirName
+						   . '/admin/instr.php?op=delinstr&instrid='
+						   . $instrid
+						   . '"><img style="width:16px;" src="'. $pathIcon16 . '/delete.png" alt='
+						   . _DELETE
+						   . ' title='
+						   . _DELETE
+						   . '></a>&nbsp;';
 } else {
-    $instrs['adminlink'] = '';
+	$instrs['adminlink'] = '';
 }
 
 // Выводим в шаблон
@@ -102,7 +102,7 @@ 
 $nav_parent_id = array_reverse($nav_parent_id);
 $navigation    = '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/">' . $GLOBALS['xoopsModule']->name() . '</a>&nbsp;:&nbsp;';
 foreach (array_keys($nav_parent_id) as $i) {
-    $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
+	$navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $nav_parent_id[$i]->getVar('cid') . '">' . $nav_parent_id[$i]->getVar('title') . '</a>&nbsp;:&nbsp;';
 }
 $navigation .= '<a href="' . XOOPS_URL . '/modules/' . $moduleDirName . '/index.php?cid=' . $objInscat->getVar('cid') . '">' . $objInscat->getVar('title') . '</a>&nbsp;:&nbsp;';
 $navigation .= $objInsinstr->getVar('title');
@@ -129,18 +129,18 @@ 
 
 // Теги
 if (xoops_getModuleOption('usetag', 'instruction')) {
-    include_once $GLOBALS['xoops']->path('modules/tag/include/tagbar.php');
-    $xoopsTpl->assign('tags', true);
-    $xoopsTpl->assign('tagbar', tagBar($instrid, 0));
+	include_once $GLOBALS['xoops']->path('modules/tag/include/tagbar.php');
+	$xoopsTpl->assign('tags', true);
+	$xoopsTpl->assign('tagbar', tagBar($instrid, 0));
 } else {
-    $xoopsTpl->assign('tags', false);
+	$xoopsTpl->assign('tags', false);
 }
 
 // Рейтинг
 if (xoops_getModuleOption('userat', 'instruction')) {
-    $xoopsTpl->assign('insUserat', true);
+	$xoopsTpl->assign('insUserat', true);
 } else {
-    $xoopsTpl->assign('insUserat', false);
+	$xoopsTpl->assign('insUserat', false);
 }
 
 // Подвал

Spacing +3 added lines, -3 removed lines

@@ -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();
 }
@@ -62,7 +62,7 @@ 
                            . $moduleDirName
                            . '/admin/instr.php?op=editinstr&instrid='
                            . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/edit.png" alt='
+                           . '"><img style="width:16px;" src="' . $pathIcon16 . '/edit.png" alt='
                            . _EDIT
                            . ' title='
                            . _EDIT
@@ -72,7 +72,7 @@ 
                            . $moduleDirName
                            . '/admin/instr.php?op=delinstr&instrid='
                            . $instrid
-                           . '"><img style="width:16px;" src="'. $pathIcon16 . '/delete.png" alt='
+                           . '"><img style="width:16px;" src="' . $pathIcon16 . '/delete.png" alt='
                            . _DELETE
                            . ' title='
                            . _DELETE