From 92f98d766309bf34e269b816eccf7eecd30380b4 Mon Sep 17 00:00:00 2001
From: mattab <matthieu.aubry@gmail.com>
Date: Fri, 3 Oct 2014 16:39:53 +1300
Subject: Fixes #6374 Make sure top menu URLs are generated properly based on
 user preferences , refs #3426

---
 core/Menu/MenuAbstract.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'core/Menu')

diff --git a/core/Menu/MenuAbstract.php b/core/Menu/MenuAbstract.php
index 8fcc4786cc..0d77ff21d0 100644
--- a/core/Menu/MenuAbstract.php
+++ b/core/Menu/MenuAbstract.php
@@ -103,7 +103,7 @@ abstract class MenuAbstract extends Singleton
     public function addItem($menuName, $subMenuName, $url, $order = 50, $tooltip = false)
     {
         // make sure the idSite value used is numeric (hack-y fix for #3426)
-        if (!is_numeric(Common::getRequestVar('idSite', false))) {
+        if (isset($url['idSite']) && !is_numeric($url['idSite'])) {
             $idSites = API::getInstance()->getSitesIdWithAtLeastViewAccess();
             $url['idSite'] = reset($idSites);
         }
-- 
cgit v1.2.3