Adding one more conditional to make sure we deal with a core Login Menu

item in case some 3rd party component uses similar field names
joomla · May 6, 2016 · 5265043 · 5265043
1 parent f65ea4b
commit 5265043
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/administrator/components/com_menus/models/item.php b/administrator/components/com_menus/models/item.php
@@ -688,14 +688,17 @@ public function getItem($pk = null)
 
 			// Special case for the Login menu item.
 			// Display the login or logout redirect URL fields if not empty
-			if (!empty($result->params['login_redirect_url']))
+			if ($table->link == 'index.php?option=com_users&view=login')
 			{
-				$result->params['loginredirectchoice'] = '0';
-			}
+				if (!empty($result->params['login_redirect_url']))
+				{
+					$result->params['loginredirectchoice'] = '0';
+				}
 
-			if (!empty($result->params['logout_redirect_url']))
-			{
-				$result->params['logoutredirectchoice'] = '0';
+				if (!empty($result->params['logout_redirect_url']))
+				{
+					$result->params['logoutredirectchoice'] = '0';
+				}
 			}
 		}