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,

Indentation +46 added lines, -46 removed lines

@@ -23,57 +23,57 @@
 $capsDirName   = strtoupper($moduleDirName);
 
 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'));
-    define($capsDirName . '_URL', XOOPS_URL . '/modules/' . constant($capsDirName . '_DIRNAME'));
-    define($capsDirName . '_ADMIN', constant($capsDirName . '_URL') . '/admin/index.php');
-    define($capsDirName . '_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
-    define($capsDirName . '_AUTHOR_LOGOIMG', constant($capsDirName . '_URL') . '/assets/images/logoModule.png');
-    define($capsDirName . '_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . $moduleDirName); // WITHOUT Trailing slash
-    define($capsDirName . '_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . $moduleDirName); // WITHOUT Trailing slash
+	//if (!defined(constant($capsDirName . '_DIRNAME'))) {
+	define($capsDirName . '_DIRNAME', $GLOBALS['xoopsModule']->dirname());
+	define($capsDirName . '_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
+	define($capsDirName . '_URL', XOOPS_URL . '/modules/' . constant($capsDirName . '_DIRNAME'));
+	define($capsDirName . '_ADMIN', constant($capsDirName . '_URL') . '/admin/index.php');
+	define($capsDirName . '_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . constant($capsDirName . '_DIRNAME'));
+	define($capsDirName . '_AUTHOR_LOGOIMG', constant($capsDirName . '_URL') . '/assets/images/logoModule.png');
+	define($capsDirName . '_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . $moduleDirName); // WITHOUT Trailing slash
+	define($capsDirName . '_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . $moduleDirName); // WITHOUT Trailing slash
 }
 
 //Configurator
 return (object)[
-    'name'          => strtoupper($moduleDirName) . ' Module Configurator',
-    'paths'         => [
-        'dirname'    => $moduleDirName,
-        'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
-        //        'path'       => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
-        //        'url'        => XOOPS_URL . '/modules/' . $moduleDirName,
-        'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
-    ],
-    'uploadFolders' => [
-        constant($capsDirName . '_UPLOAD_PATH'),
-        //constant($capsDirName . '_UPLOAD_PATH') . '/category',
-        //constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
-        //XOOPS_UPLOAD_PATH . '/flags'
-    ],
-    'blankFiles'    => [
-        constant($capsDirName . '_UPLOAD_PATH'),
-        //constant($capsDirName . '_UPLOAD_PATH') . '/category',
-        //constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
-        //XOOPS_UPLOAD_PATH . '/flags'
-    ],
+	'name'          => strtoupper($moduleDirName) . ' Module Configurator',
+	'paths'         => [
+		'dirname'    => $moduleDirName,
+		'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
+		//        'path'       => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
+		//        'url'        => XOOPS_URL . '/modules/' . $moduleDirName,
+		'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
+	],
+	'uploadFolders' => [
+		constant($capsDirName . '_UPLOAD_PATH'),
+		//constant($capsDirName . '_UPLOAD_PATH') . '/category',
+		//constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
+		//XOOPS_UPLOAD_PATH . '/flags'
+	],
+	'blankFiles'    => [
+		constant($capsDirName . '_UPLOAD_PATH'),
+		//constant($capsDirName . '_UPLOAD_PATH') . '/category',
+		//constant($capsDirName . '_UPLOAD_PATH') . '/screenshots',
+		//XOOPS_UPLOAD_PATH . '/flags'
+	],
 
-    'templateFolders' => [
-        '/templates/',
-        '/templates/blocks/',
-        '/templates/admin/'
+	'templateFolders' => [
+		'/templates/',
+		'/templates/blocks/',
+		'/templates/admin/'
 
-    ],
-    'oldFiles'        => [//        '/sql/mysql.sql',
-    ],
-    'oldFolders'      => [
-        '/images',
-        '/css',
-        '/js',
-        '/tree',
-        '/images/icons/16',
-        '/images/icons/32',
-    ],
-    'modCopyright'    => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
+	],
+	'oldFiles'        => [//        '/sql/mysql.sql',
+	],
+	'oldFolders'      => [
+		'/images',
+		'/css',
+		'/js',
+		'/tree',
+		'/images/icons/16',
+		'/images/icons/32',
+	],
+	'modCopyright'    => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
                      <img src='" . $capsDirName . '_AUTHOR_LOGOIMG' . '\' alt=\'XOOPS Project\' /></a>',
 ];

include/pagetypes.inc.php

Indentation +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@
 <?php
 // Типы страницы
 return [
-    0 => _AM_INSTR_PT_0,
-    1 => _AM_INSTR_PT_1
+	0 => _AM_INSTR_PT_0,
+	1 => _AM_INSTR_PT_1
 ];

include/plugin.tag.php

Braces +4 added lines, -2 removed lines

@@ -52,7 +52,8 @@ 
                . "                FROM {$itemHandler->table} "
                . "                WHERE {$itemHandler->table}.status > 0"
                . '            ) '
-               . '        )'; else:
+               . '        )'; else {

+               	:
         $sql = "    DELETE {$linkHandler->table} FROM {$linkHandler->table}"
                . "    LEFT JOIN {$itemHandler->table} AS aa ON {$linkHandler->table}.tag_itemid = aa.{$itemHandler->keyName} "
                . '    WHERE '
@@ -60,7 +61,8 @@ 
                . '        AND '
                . "        ( aa.{$itemHandler->keyName} IS NULL"
                . '            OR aa.status < 1'
-               . '        )';
+               . '        )';

+               }

     endif;
     if (!$result = $linkHandler->db->queryF($sql)) {
         //xoops_error($linkHandler->db->error());

Spacing +3 added lines, -3 removed lines

@@ -7,14 +7,14 @@ 
  */
 function instruction_tag_iteminfo(&$items)
 {
-    if (empty($items) || !is_array($items)) {
+    if (empty($items) || ! is_array($items)) {
         return false;
     }
 
     $items_id = [];
     foreach (array_keys($items) as $cat_id) {
         foreach (array_keys($items[$cat_id]) as $item_id) {
-            $items_id[] = (int)$item_id;
+            $items_id[] = (int) $item_id;
         }
     }
 
@@ -70,7 +70,7 @@ 
                . '            OR aa.status < 1'
                . '        )';
     endif;
-    if (!$result = $linkHandler->db->queryF($sql)) {
+    if ( ! $result = $linkHandler->db->queryF($sql)) {
         //xoops_error($linkHandler->db->error());
     }
 }

Indentation +54 added lines, -54 removed lines

@@ -7,37 +7,37 @@ 
  */
 function instruction_tag_iteminfo(&$items)
 {
-    if (empty($items) || !is_array($items)) {
-        return false;
-    }
+	if (empty($items) || !is_array($items)) {
+		return false;
+	}
 
-    $items_id = [];
-    foreach (array_keys($items) as $cat_id) {
-        foreach (array_keys($items[$cat_id]) as $item_id) {
-            $items_id[] = (int)$item_id;
-        }
-    }
+	$items_id = [];
+	foreach (array_keys($items) as $cat_id) {
+		foreach (array_keys($items[$cat_id]) as $item_id) {
+			$items_id[] = (int)$item_id;
+		}
+	}
 
-    $itemHandler = xoops_getModuleHandler('instruction', 'instruction');
-    $items_obj   = $itemHandler->getObjects(new \Criteria('instrid', '(' . implode(', ', $items_id) . ')', 'IN'), true);
+	$itemHandler = xoops_getModuleHandler('instruction', 'instruction');
+	$items_obj   = $itemHandler->getObjects(new \Criteria('instrid', '(' . implode(', ', $items_id) . ')', 'IN'), true);
 
-    foreach (array_keys($items) as $cat_id) {
-        foreach (array_keys($items[$cat_id]) as $item_id) {
-            if (isset($items_obj[$item_id])) {
-                $item_obj                 = $items_obj[$item_id];
-                $items[$cat_id][$item_id] = [
-                    'title'   => $item_obj->getVar('title'),
-                    'uid'     => $item_obj->getVar('uid'),
-                    'link'    => "instr.php?id={$item_id}",
-                    'time'    => $item_obj->getVar('datecreated'),
-                    'tags'    => '',
-                    'content' => '',
-                ];
-            }
-        }
-    }
-    unset($items_obj);
-    return '';
+	foreach (array_keys($items) as $cat_id) {
+		foreach (array_keys($items[$cat_id]) as $item_id) {
+			if (isset($items_obj[$item_id])) {
+				$item_obj                 = $items_obj[$item_id];
+				$items[$cat_id][$item_id] = [
+					'title'   => $item_obj->getVar('title'),
+					'uid'     => $item_obj->getVar('uid'),
+					'link'    => "instr.php?id={$item_id}",
+					'time'    => $item_obj->getVar('datecreated'),
+					'tags'    => '',
+					'content' => '',
+				];
+			}
+		}
+	}
+	unset($items_obj);
+	return '';
 }
 
 // Синхронизация тегов
@@ -46,31 +46,31 @@ 
  */
 function instruction_tag_synchronization($mid)
 {
-    $itemHandler = xoops_getModuleHandler('instruction', 'instruction');
-    $linkHandler = xoops_getModuleHandler('link', 'tag');
+	$itemHandler = xoops_getModuleHandler('instruction', 'instruction');
+	$linkHandler = xoops_getModuleHandler('link', 'tag');
 
-    /* clear tag-item links */
-    if (version_compare($GLOBALS['xoopsDB']->getServerVersion(), '4.1.0', 'ge')):
-        $sql = "    DELETE FROM {$linkHandler->table}"
-               . '    WHERE '
-               . "        tag_modid = {$mid}"
-               . '        AND '
-               . '        ( tag_itemid NOT IN '
-               . "            ( SELECT DISTINCT {$itemHandler->keyName} "
-               . "                FROM {$itemHandler->table} "
-               . "                WHERE {$itemHandler->table}.status > 0"
-               . '            ) '
-               . '        )'; else:
-        $sql = "    DELETE {$linkHandler->table} FROM {$linkHandler->table}"
-               . "    LEFT JOIN {$itemHandler->table} AS aa ON {$linkHandler->table}.tag_itemid = aa.{$itemHandler->keyName} "
-               . '    WHERE '
-               . "        tag_modid = {$mid}"
-               . '        AND '
-               . "        ( aa.{$itemHandler->keyName} IS NULL"
-               . '            OR aa.status < 1'
-               . '        )';
-    endif;
-    if (!$result = $linkHandler->db->queryF($sql)) {
-        //xoops_error($linkHandler->db->error());
-    }
+	/* clear tag-item links */
+	if (version_compare($GLOBALS['xoopsDB']->getServerVersion(), '4.1.0', 'ge')):
+		$sql = "    DELETE FROM {$linkHandler->table}"
+			   . '    WHERE '
+			   . "        tag_modid = {$mid}"
+			   . '        AND '
+			   . '        ( tag_itemid NOT IN '
+			   . "            ( SELECT DISTINCT {$itemHandler->keyName} "
+			   . "                FROM {$itemHandler->table} "
+			   . "                WHERE {$itemHandler->table}.status > 0"
+			   . '            ) '
+			   . '        )'; else:
+		$sql = "    DELETE {$linkHandler->table} FROM {$linkHandler->table}"
+			   . "    LEFT JOIN {$itemHandler->table} AS aa ON {$linkHandler->table}.tag_itemid = aa.{$itemHandler->keyName} "
+			   . '    WHERE '
+			   . "        tag_modid = {$mid}"
+			   . '        AND '
+			   . "        ( aa.{$itemHandler->keyName} IS NULL"
+			   . '            OR aa.status < 1'
+			   . '        )';
+	endif;
+	if (!$result = $linkHandler->db->queryF($sql)) {
+		//xoops_error($linkHandler->db->error());
+	}
 }

include/search.inc.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
     $sql = 'SELECT p.pageid, p.title, p.uid, p.datecreated, i.title FROM ' . $GLOBALS['xoopsDB']->prefix('instruction_page') . ' p, ' . $GLOBALS['xoopsDB']->prefix('instruction_instr') . ' i WHERE i.instrid = p.instrid AND i.status > 0 AND p.status > 0 AND p.type > 0';
     if (0 != $userid) {
-        $sql .= ' AND p.uid = ' . (int)$userid . ' ';
+        $sql .= ' AND p.uid = ' . (int) $userid . ' ';
         //return NULL;
     }
 

Indentation +39 added lines, -39 removed lines

@@ -13,46 +13,46 @@
  */
 function instruction_search($queryarray, $andor, $limit, $offset, $userid)
 {
-    // Подключаем функции
-    $moduleDirName = basename(dirname(__DIR__));
-    include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
+	// Подключаем функции
+	$moduleDirName = basename(dirname(__DIR__));
+	include_once $GLOBALS['xoops']->path('modules/' . $moduleDirName . '/class/utility.php');
 
-    $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 . ' ';
-        //return NULL;
-    }
+	$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 . ' ';
+		//return NULL;
+	}
 
-    // Права на просмотр
-    $categories = Xoopsmodules\instruction\Utility::getItemIds();
-    if (is_array($categories) && count($categories) > 0) {
-        $sql .= ' AND i.cid IN ( ' . implode(', ', $categories) . ' ) ';
-        // Если пользователь не имеет прав просмотра ни одной категории
-    } else {
-        return null;
-    }
+	// Права на просмотр
+	$categories = Xoopsmodules\instruction\Utility::getItemIds();
+	if (is_array($categories) && count($categories) > 0) {
+		$sql .= ' AND i.cid IN ( ' . implode(', ', $categories) . ' ) ';
+		// Если пользователь не имеет прав просмотра ни одной категории
+	} else {
+		return null;
+	}
 
-    // Добавляем в условие ключевые слова поиска
-    if (is_array($queryarray) && $count = count($queryarray)) {
-        $sql .= " AND ( ( p.title LIKE '%$queryarray[0]%' OR p.hometext LIKE '%$queryarray[0]%' )";
-        for ($i = 1; $i < $count; $i++) {
-            $sql .= " $andor ";
-            $sql .= "( p.title LIKE '%$queryarray[$i]%' OR p.hometext LIKE '%$queryarray[$i]%' )";
-        }
-        $sql .= ' ) ';
-    }
-    //$sql .= "ORDER BY date DESC";
-    $result = $GLOBALS['xoopsDB']->query($sql, $limit, $offset);
-    $ret    = [];
-    $i      = 0;
-    // Перебираем все результаты
-    while (list($pageid, $ptitle, $puid, $pdatecreated, $ititle) = $GLOBALS['xoopsDB']->fetchRow($result)) {
-        $ret[$i]['image'] = 'assets/images/size2.gif';
-        $ret[$i]['link']  = 'page.php?id=' . $pageid;
-        $ret[$i]['title'] = $ititle . ': ' . $ptitle;
-        $ret[$i]['time']  = $pdatecreated;
-        $ret[$i]['uid']   = $puid;
-        $i++;
-    }
-    return $ret;
+	// Добавляем в условие ключевые слова поиска
+	if (is_array($queryarray) && $count = count($queryarray)) {
+		$sql .= " AND ( ( p.title LIKE '%$queryarray[0]%' OR p.hometext LIKE '%$queryarray[0]%' )";
+		for ($i = 1; $i < $count; $i++) {
+			$sql .= " $andor ";
+			$sql .= "( p.title LIKE '%$queryarray[$i]%' OR p.hometext LIKE '%$queryarray[$i]%' )";
+		}
+		$sql .= ' ) ';
+	}
+	//$sql .= "ORDER BY date DESC";
+	$result = $GLOBALS['xoopsDB']->query($sql, $limit, $offset);
+	$ret    = [];
+	$i      = 0;
+	// Перебираем все результаты
+	while (list($pageid, $ptitle, $puid, $pdatecreated, $ititle) = $GLOBALS['xoopsDB']->fetchRow($result)) {
+		$ret[$i]['image'] = 'assets/images/size2.gif';
+		$ret[$i]['link']  = 'page.php?id=' . $pageid;
+		$ret[$i]['title'] = $ititle . ': ' . $ptitle;
+		$ret[$i]['time']  = $pdatecreated;
+		$ret[$i]['uid']   = $puid;
+		$i++;
+	}
+	return $ret;
 }

comment_new.php

Indentation +8 added lines, -8 removed lines

@@ -7,12 +7,12 @@
 
 $com_itemid = Request::getInt('com_itemid', 0, 'GET');
 if ($com_itemid > 0) {
-    $itemObj       = $publisher->getHandler('item')->get($com_itemid);
-    $com_replytext = _POSTEDBY . '&nbsp;<strong>' . $itemObj->getLinkedPosterName() . '</strong>&nbsp;' . _DATE . '&nbsp;<strong>' . $itemObj->dateSub() . '</strong><br><br>' . $itemObj->summary();
-    $bodytext      = $itemObj->body();
-    if ('' != $bodytext) {
-        $com_replytext .= '<br><br>' . $bodytext . '';
-    }
-    $com_replytitle = $itemObj->getTitle();
-    include_once $GLOBALS['xoops']->path('include/comment_new.php');
+	$itemObj       = $publisher->getHandler('item')->get($com_itemid);
+	$com_replytext = _POSTEDBY . '&nbsp;<strong>' . $itemObj->getLinkedPosterName() . '</strong>&nbsp;' . _DATE . '&nbsp;<strong>' . $itemObj->dateSub() . '</strong><br><br>' . $itemObj->summary();
+	$bodytext      = $itemObj->body();
+	if ('' != $bodytext) {
+		$com_replytext .= '<br><br>' . $bodytext . '';
+	}
+	$com_replytitle = $itemObj->getTitle();
+	include_once $GLOBALS['xoops']->path('include/comment_new.php');
 }

admin/perm.php

Indentation +21 added lines, -21 removed lines

@@ -36,24 +36,24 @@ 
 $moduleId = $GLOBALS['xoopsModule']->getVar('mid');
 
 switch ($permission) {
-    // Права на просмотр
-    case 1:
-        $formTitle             = _AM_INSTRUCTION_PERM_VIEW;
-        $permissionName        = 'instruction_view';
-        $permissionDescription = _AM_INSTRUCTION_PERM_VIEW_DSC;
-        break;
-    // Права на добавление
-    case 2:
-        $formTitle             = _AM_INSTRUCTION_PERM_SUBMIT;
-        $permissionName        = 'instruction_submit';
-        $permissionDescription = _AM_INSTRUCTION_PERM_SUBMIT_DSC;
-        break;
-    // Права на редактирование
-    case 3:
-        $formTitle             = _AM_INSTRUCTION_PERM_EDIT;
-        $permissionName        = 'instruction_edit';
-        $permissionDescription = _AM_INSTRUCTION_PERM_EDIT_DSC;
-        break;
+	// Права на просмотр
+	case 1:
+		$formTitle             = _AM_INSTRUCTION_PERM_VIEW;
+		$permissionName        = 'instruction_view';
+		$permissionDescription = _AM_INSTRUCTION_PERM_VIEW_DSC;
+		break;
+	// Права на добавление
+	case 2:
+		$formTitle             = _AM_INSTRUCTION_PERM_SUBMIT;
+		$permissionName        = 'instruction_submit';
+		$permissionDescription = _AM_INSTRUCTION_PERM_SUBMIT_DSC;
+		break;
+	// Права на редактирование
+	case 3:
+		$formTitle             = _AM_INSTRUCTION_PERM_EDIT;
+		$permissionName        = 'instruction_edit';
+		$permissionDescription = _AM_INSTRUCTION_PERM_EDIT_DSC;
+		break;
 }
 
 // Права
@@ -62,9 +62,9 @@ 
 $sql    = 'SELECT cid, pid, title FROM ' . $xoopsDB->prefix('instruction_cat') . ' ORDER BY title';
 $result = $xoopsDB->query($sql);
 if ($result) {
-    while ($row = $xoopsDB->fetchArray($result)) {
-        $permissionsForm->addItem($row['cid'], $row['title'], $row['pid']);
-    }
+	while ($row = $xoopsDB->fetchArray($result)) {
+		$permissionsForm->addItem($row['cid'], $row['title'], $row['pid']);
+	}
 }
 
 echo $permissionsForm->render();

admin/admin_header.php

Unused Use Statements -2 removed lines

@@ -1,7 +1,5 @@
 <?php
 
-use Xoopsmodules\instruction;
-
 // Автор: andrey3761
 $moduleDirName = basename(dirname(__DIR__));
 require_once __DIR__ . '/../../../include/cp_header.php';

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
 
 //if (($GLOBALS['xoopsUser'] instanceof \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();
 }

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

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

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

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

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php namespace Xoopsmodules\instruction;
 
-use Xmf\Request;
 use Xoopsmodules\instruction\common;
 
 require_once __DIR__ . '/common/VersionChecks.php';

Indentation +46 added lines, -46 removed lines

@@ -14,56 +14,56 @@
  */
 class Utility
 {
-    use common\VersionChecks; //checkVerXoops, checkVerPhp Traits
+	use common\VersionChecks; //checkVerXoops, checkVerPhp Traits
 
-    use common\ServerStats; // getServerStats Trait
+	use common\ServerStats; // getServerStats Trait
 
-    use common\FilesManagement; // Files Management Trait
+	use common\FilesManagement; // Files Management Trait
 
-    // Права
-    /**
-     * @param string $permtype
-     * @return mixed
-     */
-    public static function getItemIds($permtype = 'instruction_view')
-    {
-        //global $xoopsUser;
-        static $permissions = [];
-        // Если есть в статике
-        if (is_array($permissions) && array_key_exists($permtype, $permissions)) {
-            return $permissions[$permtype];
-        }
-        // Находим из базы
-        $moduleHandler          = xoops_getHandler('module');
-        $instrModule            = $moduleHandler->getByDirname('instruction');
-        $groups                 = ($GLOBALS['xoopsUser'] instanceof \XoopsUser) ? $GLOBALS['xoopsUser']->getGroups() : XOOPS_GROUP_ANONYMOUS;
-        $gpermHandler           = xoops_getHandler('groupperm');
-        $categories             = $gpermHandler->getItemIds($permtype, $groups, $instrModule->getVar('mid'));
-        $permissions[$permtype] = $categories;
-        return $categories;
-    }
+	// Права
+	/**
+	 * @param string $permtype
+	 * @return mixed
+	 */
+	public static function getItemIds($permtype = 'instruction_view')
+	{
+		//global $xoopsUser;
+		static $permissions = [];
+		// Если есть в статике
+		if (is_array($permissions) && array_key_exists($permtype, $permissions)) {
+			return $permissions[$permtype];
+		}
+		// Находим из базы
+		$moduleHandler          = xoops_getHandler('module');
+		$instrModule            = $moduleHandler->getByDirname('instruction');
+		$groups                 = ($GLOBALS['xoopsUser'] instanceof \XoopsUser) ? $GLOBALS['xoopsUser']->getGroups() : XOOPS_GROUP_ANONYMOUS;
+		$gpermHandler           = xoops_getHandler('groupperm');
+		$categories             = $gpermHandler->getItemIds($permtype, $groups, $instrModule->getVar('mid'));
+		$permissions[$permtype] = $categories;
+		return $categories;
+	}
 
-    // Редактор
+	// Редактор
 
-    /**
-     * @param        $caption
-     * @param        $name
-     * @param string $value
-     * @return bool|\XoopsFormEditor
-     */
-    public static function getWysiwygForm($caption, $name, $value = '')
-    {
-        $editor                   = false;
-        $editor_configs           = [];
-        $editor_configs['name']   = $name;
-        $editor_configs['value']  = $value;
-        $editor_configs['rows']   = 35;
-        $editor_configs['cols']   = 60;
-        $editor_configs['width']  = '100%';
-        $editor_configs['height'] = '350px';
-        $editor_configs['editor'] = strtolower(xoops_getModuleOption('form_options', 'instruction'));
+	/**
+	 * @param        $caption
+	 * @param        $name
+	 * @param string $value
+	 * @return bool|\XoopsFormEditor
+	 */
+	public static function getWysiwygForm($caption, $name, $value = '')
+	{
+		$editor                   = false;
+		$editor_configs           = [];
+		$editor_configs['name']   = $name;
+		$editor_configs['value']  = $value;
+		$editor_configs['rows']   = 35;
+		$editor_configs['cols']   = 60;
+		$editor_configs['width']  = '100%';
+		$editor_configs['height'] = '350px';
+		$editor_configs['editor'] = strtolower(xoops_getModuleOption('form_options', 'instruction'));
 
-        $editor = new \XoopsFormEditor($caption, $name, $editor_configs);
-        return $editor;
-    }
+		$editor = new \XoopsFormEditor($caption, $name, $editor_configs);
+		return $editor;
+	}
 }