From: <be...@us...> - 2015-01-23 13:15:33
|
Revision: 12945 http://sourceforge.net/p/xoops/svn/12945 Author: beckmi Date: 2015-01-23 13:15:29 +0000 (Fri, 23 Jan 2015) Log Message: ----------- Code cleanup, adding XoopsRequest in couple of places Modified Paths: -------------- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_links.php XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_menus.php XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/blocks/mymenus_block.php XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/class/registry.php XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/include/xoops.php XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/dynamic/dynamic.php XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/mymenus/mymenus.php XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/bootstrap/templates/template.tpl XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/clean/templates/template.tpl XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/default/templates/template.tpl XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/mainmenu/templates/template.tpl XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/multiblack/stuHover.js XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish/templates/template.tpl XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish_navbar/templates/template.tpl XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/tw_blue/templates/template.tpl XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/xoops_version.php Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_links.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_links.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_links.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -34,8 +34,8 @@ } $valid_menu_ids = array_keys($menus_list); -if (isset($_REQUEST['mid']) && in_array($_REQUEST['mid'], $valid_menu_ids)) { - $menu_id = (int) $_REQUEST['mid']; +if (XoopsRequest::getInt('mid', 0) && in_array(XoopsRequest::getInt('mid', 0), $valid_menu_ids)) { + $menu_id = XoopsRequest::getInt('mid', 0); $menu_title = $menus_list[$menu_id]; } else { $keys = array_keys($menus_list); @@ -340,7 +340,7 @@ $link_obj = $links_handler->get((int) $id); //if this was moved then parent could be in different menu, if so then set parent to top level - if (!empty($_POST['pid'])) { + if (XoopsRequest::getInt('pid', '', 'POST')) { $parent_obj = $links_handler->get($link_obj->getVar('pid')); //get the parent oject if(($parent_obj instanceof MylinksLinks) && ($link_obj->getVar('mid') != $parent_obj->getVar('mid'))) { $link_obj->setVar('pid', 0); Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_menus.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_menus.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/admin/admin_menus.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -45,7 +45,7 @@ echo $indexAdmin->addNavigation('admin_menus.php'); //mymenus_adminMenu(0, _MI_MYMENUS_MENUSMANAGER); echo mymenus_admin_form($id); - include 'admin_footer.php'; + include_once __DIR__ . '/admin_footer.php'; break; case 'editok': mymenus_admin_edit($id); @@ -67,7 +67,7 @@ xoops_cp_header(); echo $indexAdmin->addNavigation('admin_menus.php'); echo mymenus_admin_list($start, $limit); - include 'admin_footer.php'; + include_once __DIR__ . '/admin_footer.php'; break; } Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/blocks/mymenus_block.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/blocks/mymenus_block.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/blocks/mymenus_block.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -176,8 +176,9 @@ */ function mymenus_block_edit($options) { + xoops_load('XoopsRequest'); //Unique ID - if (!$options[3] || (isset($_GET['op']) && 'clone' == $_GET['op'])) { + if (!$options[3] || ('clone' == XoopsRequest::getCmd('op', '', 'GET'))) { $options[3] = time(); } $i = 0; Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/class/registry.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/class/registry.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/class/registry.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -40,7 +40,7 @@ /** * @return object MymenusRegistry */ - static public function getInstance() + public static function getInstance() { static $instance = false; if (!$instance) { Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/include/xoops.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/include/xoops.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/include/xoops.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -134,7 +134,7 @@ $cP = $id; $sql="SELECT storyid FROM ".$xoopsDB->prefix($modulname)." WHERE parent_id=".intval($cP); $result = $xoopsDB->query($sql); - while ($row = $xoopsDB->fetchArray($result)) { + while ($row == $xoopsDB->fetchArray($result)) { if (intval($row['storyid']) > 0) { $cP = intval($row['storyid']); $currentParent[]=$cP; Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/dynamic/dynamic.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/dynamic/dynamic.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/dynamic/dynamic.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -72,7 +72,7 @@ xoops_loadLanguage('modinfo', $module); $overwrite = false; - if ($force = true) { //can set to false for debug + if ($force == true) { //can set to false for debug if (!($xoopsModule instanceof XoopsModule) || ($xoopsModule->getVar('dirname') != $module)) { // @todo: check the following 2 statements, they're basically just assigns - is this intended? $_xoopsModule = ($xoopsModule instanceof XoopsModule) ? $xoopsModule : $xoopsModule; Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/mymenus/mymenus.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/mymenus/mymenus.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/plugins/mymenus/mymenus.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -30,6 +30,7 @@ { $registry =& MymenusRegistry::getInstance(); $member_handler =& xoops_getHandler('member'); + xoops_load('XoopsRequest'); $user = ($GLOBALS['xoopsUser'] instanceof XoopsUser) ? $GLOBALS['xoopsUser'] : null; if (!$user) { @@ -38,7 +39,7 @@ $user->setVar('uname', $GLOBALS['xoopsConfig']['anonymous']); } - $ownerid = isset($_GET['uid']) ? intval($_GET['uid']) : null; + $ownerid = XoopsRequest::getInt('uid', null, 'GET'); $owner = $member_handler->getUser($ownerid); //if uid > 0 but user does not exists if (!($owner instanceof XoopsUser)) { @@ -53,7 +54,7 @@ $registry->setEntry('owner', $owner->getValues()); $registry->setEntry('user_groups', ($GLOBALS['xoopsUser'] instanceof XoopsUser) ? $GLOBALS['xoopsUser']->getGroups() : array(XOOPS_GROUP_ANONYMOUS)); $registry->setEntry('user_uid', ($GLOBALS['xoopsUser'] instanceof XoopsUser) ? $GLOBALS['xoopsUser']->getVar('uid') : 0); - $registry->setEntry('get_uid', isset($_GET['uid']) ? intval($_GET['uid']) : 0); + $registry->setEntry('get_uid', XoopsRequest::getInt('uid', 0, 'GET')); } @@ -127,7 +128,7 @@ } if ($validator == 'uri') { - $value = isset($_GET[$value]) ? $_GET[$value] : 0; + $value = XoopsRequest::getString($value, 0, 'GET'); $string = str_replace($expression, $value, $string); } Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/bootstrap/templates/template.tpl =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/bootstrap/templates/template.tpl 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/bootstrap/templates/template.tpl 2015-01-23 13:15:29 UTC (rev 12945) @@ -1,30 +1,43 @@ <{foreach item=menu from = $block}> <{if $menu.oul}> -<{ if $menu.level == 0}> - <ul class="nav navbar-nav<{if $menu.css}> <{$menucss}><{/if}>"> -<{ elseif $menu.level == 1}> +<{if $menu.level == 0}> +<ul class="nav navbar-nav<{if $menu.css}> <{$menucss}><{/if}>"> + <{elseif $menu.level == 1}> <ul class="dropdown-menu<{if $menu.css}> <{$menucss}><{/if}>"> -<{ else}> - <ul class="dropdown-menu sub-menu<{if $menu.css}> <{$menucss}><{/if}>"> -<{ /if}> -<{/if}> -<{if $menu.oli}> -<{ assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <{else}> + <ul class="dropdown-menu sub-menu<{if $menu.css}> <{$menucss}><{/if}>"> + <{/if}> + <{/if}> + <{if $menu.oli}> + <{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> -<{ if $menu.hassub && $menu.level == 0}> - <li class="dropdown<{if $menu.selected}> active<{/if}><{if $menu.css}> <{$menu.css}><{/if}>"> - <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> -<{ if $menu.image}><img class="menu-image" src="<{$menu.image}>" /> <{/if}><{$menu.title}> <b class="caret"></b></a> -<{ elseif $menu.hassub && $menu.level == 1}> - <li<{if $menu.selected | $menu.css}> class="<{/if}><{if $menu.selected}>active<{/if}><{if $menu.css}> <{$menu.css}><{/if}><{if $menu.selected | $menu.css}>"<{/if}>> - <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> -<{ if $menu.image}><img class="menu-image" src="<{$menu.image}>" /> <{/if}><{$menu.title}> <i class="glyphicon glyphicon-arrow-right"></i></a> -<{ else}> - <li<{if $menu.selected | $menu.css}> class="<{/if}><{if $menu.selected}>active<{/if}><{if $menu.css}> <{$menu.css}><{/if}><{if $menu.selected | $menu.css}>"<{/if}>> - <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> -<{ if $menu.image}><img class="menu-image" src="<{$menu.image}>" /> <{/if}><{$menu.title}></a><{/if}> -<{ /if}> -<{/if}> -<{if $menu.close != ''}><{$menu.close}><{/if}> -<{/foreach}> + <{if $menu.hassub && $menu.level == 0}> + <li class="dropdown<{if $menu.selected}> active<{/if}><{if $menu.css}> <{$menu.css}><{/if}>"> + <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> + <{if $menu.image}><img class="menu-image" src="<{$menu.image}>"/> <{/if}> + <{$menu.title}> <b class="caret"></b> + </a> + <{elseif $menu.hassub && $menu.level == 1}> + <li<{if $menu.selected || $menu.css}> class="<{/if}> + <{if $menu.selected}>active<{/if}> + <{if $menu.css}> <{$menu.css}><{/if}> + <{if $menu.selected || $menu.css}>"<{/if}>> + <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> + <{if $menu.image}><img class="menu-image" src="<{$menu.image}>"/> <{/if}> + <{$menu.title}> <i class="glyphicon glyphicon-arrow-right"></i> + </a> + <{else}> + <li<{if $menu.selected || $menu.css}> class="<{/if}> + <{if $menu.selected}>active<{/if}> + <{if $menu.css}> <{$menu.css}><{/if}> + <{if $menu.selected || $menu.css}>"<{/if}>> + <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> + <{if $menu.image}><img class="menu-image" src="<{$menu.image}>"/> <{/if}> + <{$menu.title}> + </a> + <{/if}> + <{/if}> + <{/if}> + <{if $menu.close != ''}><{$menu.close}><{/if}> + <{/foreach}> Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/clean/templates/template.tpl =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/clean/templates/template.tpl 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/clean/templates/template.tpl 2015-01-23 13:15:29 UTC (rev 12945) @@ -1,40 +1,43 @@ <{foreach item=menu from=$block}> -<{ if $menu.oul && $menu.level == 0}> -<{ if $menucss != ''}> +<{if $menu.oul && $menu.level == 0}> +<{if $menucss != ''}> <ul class="<{$menucss}>"> -<{ else}> -<ul> -<{ /if}> -<{ /if}> + <{else}> + <ul> + <{/if}> + <{/if}> -<{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> -<{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> -<{ if $menu.level == 0}> - <li<{if $menu.css}> class="<{$menu.css}>"<{/if}>> - <a<{if $menu.selected}> class="active"<{/if}> href="<{$menu.link}>"><{$menu.title}></a> -<{ if $menu.hassub}> -<{ foreach item=sub from=$block name=sublp}> -<{ if $smarty.foreach.sublp.first}> <ul class="dropdown-menu"><!-- sub menu --><{/if}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> -<{ if $menu.id == $sub.pid}> - <li<{if $sub.css}> class="<{$sub.css}>"<{/if}>> - <a<{if $sub.selected}> class="active"<{/if}> href="<{$sub.link}>" target="<{$sub.target}>" title="<{$sub.alt_title}>"><{$sub.title}></a> -<{ if !empty($block.subsub)}> -<{ foreach item=subsub from=$block name=subsublp}> -<{ if $smarty.foreach.subsublp.first}> <ul class="dropdown-menu"><!-- subsub menu --><{/if}> -<{ if $sub.id == $subsub.pid}> - <li<{if $subsub.css}> class="<{$subsub.css}>"<{/if}>> - <a<{if $subsub.selected}> class="active"<{/if}> href="<{$subsub.link}>" target="<{$subsub.target}>" title="<{$subsub.alt_title}>"><{$subsub.title}></a> -<{ /if}> -<{ if $smarty.foreach.subsublp.last}></ul><{/if}> -<{ /foreach}> -<{ /if}> -<{ /if}> -<{ /if}> -<{ /foreach}> -<{ /if}> -<{ /if}> -<{/if}> + <{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <{if $menu.level == 0}> + <li<{if $menu.css}> class="<{$menu.css}>"<{/if}>> + <a<{if $menu.selected}> class="active"<{/if}> href="<{$menu.link}>"><{$menu.title}></a> + <{if $menu.hassub}> + <{foreach item=sub from=$block name=sublp}> + <{if $smarty.foreach.sublp.first}> + <ul class="dropdown-menu"><!-- sub menu --><{/if}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <{if $menu.id == $sub.pid}> + <li<{if $sub.css}> class="<{$sub.css}>"<{/if}>> + <a<{if $sub.selected}> class="active"<{/if}> href="<{$sub.link}>" target="<{$sub.target}>" title="<{$sub.alt_title}>"><{$sub.title}></a> + <{if !empty($block.subsub)}> + <{foreach item=subsub from=$block name=subsublp}> + <{if $smarty.foreach.subsublp.first}> + <ul class="dropdown-menu"><!-- subsub menu --><{/if}> + <{if $sub.id == $subsub.pid}> + <li<{if $subsub.css}> class="<{$subsub.css}>"<{/if}>> + <a<{if $subsub.selected}> class="active"<{/if}> href="<{$subsub.link}>" target="<{$subsub.target}>" title="<{$subsub.alt_title}>"><{$subsub.title}></a> + <{/if}> + <{if $smarty.foreach.subsublp.last}> + </ul><{/if}> + <{/foreach}> + <{/if}> + <{/if}> + <{/if}> + <{/foreach}> + <{/if}> + <{/if}> + <{/if}> -<{if '' != $menu.close}><{$menu.close}><{/if}> -<{/foreach}> + <{if '' != $menu.close}><{$menu.close}><{/if}> + <{/foreach}> Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/default/templates/template.tpl =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/default/templates/template.tpl 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/default/templates/template.tpl 2015-01-23 13:15:29 UTC (rev 12945) @@ -1,30 +1,30 @@ <span class="default_preload1"></span> <span class="default_preload2"></span> <{foreach item=menu from = $block}> -<{ if $menu.oul}> -<{ if 0 == $menu.level}> +<{if $menu.oul}> +<{if 0 == $menu.level}> <ul id="default_nav"> -<{ elseif 1 == $menu.level}> -<ul class="default_sub"> -<{ else}> -<ul> -<{ /if}> -<{ /if}> -<{ if $menu.oli}> -<{ if 0 == $menu.level}> - <li class="default_top<{if $menu.css}> <{$menu.css}><{/if}>"> -<{ else}> - <li<{if $menu.css}> class="<{$menu.css}>"<{/if}>> -<{ /if}> -<{ /if}> -<{ assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> -<{ if 0 == $menu.level}> - <a href="<{$menu.link}>" class="default_top_link<{if $menu.selected}> default_selected<{/if}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> - <span<{if $menu.hassub}> class="default_down"<{/if}>><{if $menu.image}><img src="<{$menu.image}>" /><{/if}><{$menu.title}></span></a> -<{ else}> - <a href="<{$menu.link}>"<{if $menu.hassub}> class="default_fly"<{/if}> target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"><{if $menu.image}><img src="<{$menu.image}>" /><{/if}><{$menu.title}></a> -<{ /if}> -<{ /if}> -<{ if '' != $menu.close}><{$menu.close}><{/if}> -<{/foreach}> + <{elseif 1 == $menu.level}> + <ul class="default_sub"> + <{else}> + <ul> + <{/if}> + <{/if}> + <{if $menu.oli}> + <{if 0 == $menu.level}> + <li class="default_top<{if $menu.css}> <{$menu.css}><{/if}>"> + <{else}> + <li<{if $menu.css}> class="<{$menu.css}>"<{/if}>> + <{/if}> + <{/if}> + <{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) || ($menu.image|strstr:$myStr)) || !$xlanguage)}> + <{if 0 == $menu.level}> + <a href="<{$menu.link}>" class="default_top_link<{if $menu.selected}> default_selected<{/if}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> + <span<{if $menu.hassub}> class="default_down"<{/if}>><{if $menu.image}><img src="<{$menu.image}>" /><{/if}><{$menu.title}></span></a> + <{else}> + <a href="<{$menu.link}>"<{if $menu.hassub}> class="default_fly"<{/if}> target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"><{if $menu.image}><img src="<{$menu.image}>" /><{/if}><{$menu.title}></a> + <{/if}> + <{/if}> + <{if '' != $menu.close}><{$menu.close}><{/if}> + <{/foreach}> Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/mainmenu/templates/template.tpl =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/mainmenu/templates/template.tpl 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/mainmenu/templates/template.tpl 2015-01-23 13:15:29 UTC (rev 12945) @@ -1,23 +1,27 @@ <div id="mainmenu"> -<{foreach item=menu from=$block}> -<{ if $menu.level == 0}> -<{ assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> - <a class="menuMain<{if $menu.css}> <{$menu.css}><{/if}><{if $menu.topselected}> maincurrent<{/if}>" href="<{$menu.link}>" target="<{$menu.target}>" title="<{$menu.alt_title}>"> - <{if $menu.image}><img src="<{$menu.image}>"/><{/if}><{$menu.title}></a> - <br /> -<{ if $menu.selected}> -<{ foreach item=sub from=$block}> -<{ if $menu.id == $sub.pid}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> - <a class="menuSub marg5<{if $sub.css}> <{$sub.css}><{/if}><{if $sub.selected}> maincurrent<{/if}>" href="<{$sub.link}>" target="<{$sub.target}>" title="<{$sub.alt_title}>"> - <{if $sub.image}><img src="<{$sub.image}>" alt="<{$sub.alt_title}>" /><{/if}><{$sub.title}></a> - <br /> -<{ /if}> -<{ /if}> -<{ /foreach}> -<{ /if}> -<{ /if}> -<{ /if}> -<{/foreach}> + <{foreach item=menu from=$block}> + <{if $menu.level == 0}> + <{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) || ($menu.image|strstr:$myStr)) || !$xlanguage)}> + <a class="menuMain<{if $menu.css}> <{$menu.css}><{/if}><{if $menu.topselected}> maincurrent<{/if}>" href="<{$menu.link}>" target="<{$menu.target}>" title="<{$menu.alt_title}>"> + <{if $menu.image}><img src="<{$menu.image}>"/><{/if}> + <{$menu.title}> + </a> + <br/> + <{if $menu.selected}> + <{foreach item=sub from=$block}> + <{if $menu.id == $sub.pid}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <a class="menuSub marg5<{if $sub.css}> <{$sub.css}><{/if}><{if $sub.selected}> maincurrent<{/if}>" href="<{$sub.link}>" target="<{$sub.target}>" title="<{$sub.alt_title}>"> + <{if $sub.image}><img src="<{$sub.image}>" alt="<{$sub.alt_title}>" /><{/if}> + <{$sub.title}> + </a> + <br/> + <{/if}> + <{/if}> + <{/foreach}> + <{/if}> + <{/if}> + <{/if}> + <{/foreach}> </div> Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/multiblack/stuHover.js =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/multiblack/stuHover.js 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/multiblack/stuHover.js 2015-01-23 13:15:29 UTC (rev 12945) @@ -30,6 +30,6 @@ this.className=this.className.replace(new RegExp(" iehover\\b"), ""); } } -} +}; if (window.attachEvent) window.attachEvent("onload", stuHover); Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish/templates/template.tpl =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish/templates/template.tpl 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish/templates/template.tpl 2015-01-23 13:15:29 UTC (rev 12945) @@ -1,19 +1,19 @@ <{foreach item=menu from = $block}> -<{ if $menu.oul}> -<{ if 0 == $menu.level}> +<{if $menu.oul}> +<{if 0 == $menu.level}> <ul class="sf-menu"> -<{ else}> + <{else}> <ul> -<{ /if}> -<{ /if}> -<{ if $menu.oli}> - <li<{if $menu.css | $menu.selected}> class="<{/if}><{if $menu.css}><{$menu.css}> <{/if}><{if $menu.selected}>current<{/if}><{if $menu.css | $menu.selected}>"<{/if}>> -<{ /if}> -<{ assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> - <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> -<{if $main.image}> <img src="<{$menu.image}>" alt="<{$menu.alt_title}>" /><{/if}><{$menu.title}></a> -<{ if $menu.close != ''}><{$menu.close}><{/if}> -<{ /if}> -<{/foreach}> -<div style="clear: both"></div> + <{/if}> + <{/if}> + <{if $menu.oli}> + <li<{if $menu.css || $menu.selected}> class="<{/if}><{if $menu.css}><{$menu.css}> <{/if}><{if $menu.selected}>current<{/if}><{if $menu.css || $menu.selected}>"<{/if}>> + <{/if}> + <{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> + <{if $main.image}> <img src="<{$menu.image}>" alt="<{$menu.alt_title}>" /><{/if}><{$menu.title}></a> + <{if $menu.close != ''}><{$menu.close}><{/if}> + <{/if}> + <{/foreach}> + <div style="clear: both"></div> Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish_navbar/templates/template.tpl =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish_navbar/templates/template.tpl 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/superfish_navbar/templates/template.tpl 2015-01-23 13:15:29 UTC (rev 12945) @@ -1,19 +1,19 @@ <{foreach item=menu from = $block}> -<{ if $menu.oul}> -<{ if 0 == $menu.level}> +<{if $menu.oul}> +<{if 0 == $menu.level}> <ul class="sf-menu sf-navbar"> -<{ else}> + <{else}> <ul> -<{ /if}> -<{ /if}> -<{ if $menu.oli}> - <li<{if $menu.css | $menu.selected}> class="<{/if}><{if $menu.css}>{<$menu.css}> <{/if}><{if $menu.selected}>current<{/if}><{if $menu.css | $menu.selected}>"<{/if}>> -<{ /if}> -<{ assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> - <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> -<{if $main.image}><img src="<{$menu.image}>" alt="<{$menu.alt_title}>" /><{/if}><{$menu.title}></a> -<{ /if}> -<{ if '' != $menu.close}><{$menu.close}><{/if}> -<{/foreach}> -<div style="clear:both"></div> + <{/if}> + <{/if}> + <{if $menu.oli}> + <li<{if $menu.css || $menu.selected}> class="<{/if}><{if $menu.css}>{<$menu.css}> <{/if}><{if $menu.selected}>current<{/if}><{if $menu.css || $menu.selected}>"<{/if}>> + <{/if}> + <{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> + <{if $main.image}><img src="<{$menu.image}>" alt="<{$menu.alt_title}>" /><{/if}><{$menu.title}></a> + <{/if}> + <{if '' != $menu.close}><{$menu.close}><{/if}> + <{/foreach}> + <div style="clear:both"></div> Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/tw_blue/templates/template.tpl =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/tw_blue/templates/template.tpl 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/skins/tw_blue/templates/template.tpl 2015-01-23 13:15:29 UTC (rev 12945) @@ -4,37 +4,47 @@ <div class="nav-left"></div> <div class="nav"> <ul id="navigation"> -<{ foreach item=menu from=$block}> -<{ assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> -<{ if 0 == $menu.level}> - <li<{if $menu.css | $menu.selected}> class="<{/if}><{if $menu.selected}>active<{/if}><{if $menu.css}> <{$menu.css}><{/if}><{if $menu.css | $menu.selected}>"<{/if}>> - <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> - <span class="menu-left"></span> - <span class="menu-mid"><{if $menu.image}><img src="<{$menu.image}>" /><{/if}><{$menu.title}></span> - <span class="menu-right"></span> - </a> -<{ if $menu.hassub}> - <div class="sub"> - <ul> -<{ foreach item=sub from=$block}> -<{ if $sub.pid == $menu.id}> -<{ if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> - <li<{if $sub.class}> class="<{$sub.css}>"<{/if}>> - <a href="<{$sub.link}>" target="<{$sub.target}>" alt="<{$sub.alt_title}>" title="<{$sub.alt_title}>"> - <{if $sub.image}><img src="<{$sub.image}>" alt="<{$sub.alt_title}>" /><{/if}><{$sub.title}></a> + <{foreach item=menu from=$block}> + <{assign var=myStr value="["|cat:$xoops_langcode|cat:"]"}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <{if 0 == $menu.level}> + <li<{if $menu.css || $menu.selected}> class="<{/if}> + <{if $menu.selected}>active<{/if}> + <{if $menu.css}> <{$menu.css}><{/if}> + <{if $menu.css || $menu.selected}>"<{/if}>> + <a href="<{$menu.link}>" target="<{$menu.target}>" alt="<{$menu.alt_title}>" title="<{$menu.alt_title}>"> + <span class="menu-left"></span> + <span class="menu-mid"> + <{if $menu.image}> + <img src="<{$menu.image}>"/> + <{/if}> + <{$menu.title}> + </span> + <span class="menu-right"></span> + </a> + <{if $menu.hassub}> + <div class="sub"> + <ul> + <{foreach item=sub from=$block}> + <{if $sub.pid == $menu.id}> + <{if ($xlanguage && (($menu.title|strstr:$myStr) OR ($menu.image|strstr:$myStr)) OR !$xlanguage)}> + <li<{if $sub.class}> class="<{$sub.css}>"<{/if}>> + <a href="<{$sub.link}>" target="<{$sub.target}>" alt="<{$sub.alt_title}>" title="<{$sub.alt_title}>"> + <{if $sub.image}><img src="<{$sub.image}>" alt="<{$sub.alt_title}>" /><{/if}> + <{$sub.title}> + </a> + </li> + <{/if}> + <{/if}> + <{/foreach}> + </ul> + <div class="btm-bg"></div> + </div> + <{/if}> </li> -<{ /if}> -<{ /if}> -<{ /foreach}> - </ul> - <div class="btm-bg"></div> - </div> -<{ /if}> - </li> -<{ /if}> -<{ /if}> -<{ /foreach}> + <{/if}> + <{/if}> + <{/foreach}> </ul> </div> <div class="nav-right"></div> Modified: XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/xoops_version.php =================================================================== --- XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/xoops_version.php 2015-01-23 13:05:09 UTC (rev 12944) +++ XoopsModules/modulepacks/x257basicmodulepack/trunk/modules/mymenus/xoops_version.php 2015-01-23 13:15:29 UTC (rev 12945) @@ -28,7 +28,7 @@ $modversion['help'] = 'page=help'; $modversion['license'] = 'GNU GPL 2.0'; $modversion['license_url'] = "www.gnu.org/licenses/gpl-2.0.html"; -$modversion['official'] = 0; +$modversion['official'] = 1; //1 indicates supported by XOOPS Dev Team, 0 means 3rd party supported $modversion['image'] = "assets/images/mymenus.png"; $modversion['dirname'] = basename(__DIR__); $modversion['dirmoduleadmin'] = '/Frameworks/moduleclasses/moduleadmin'; |