Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

# [#30422] Tags drill down incorrectly finds match to multi-tag tagged

items menu item. Thanks Mark
  • Loading branch information...
commit 49f99c6c94486d2e99d886c5e168daaedb528fbe 1 parent 97a6475
@dextercowley dextercowley authored infograf768 committed
Showing with 5 additions and 2 deletions.
  1. +4 −2 components/com_tags/helpers/route.php
  2. +1 −0  installation/CHANGELOG
View
6 components/com_tags/helpers/route.php
@@ -117,7 +117,8 @@ protected static function _findItem($needles = null)
if ($items) {
foreach ($items as $item)
{
- if (isset($item->query) && isset($item->query['view'])) {
+ if (isset($item->query) && isset($item->query['view']))
+ {
$view = $item->query['view'];
if (!isset(self::$lookup[$view]))
@@ -125,7 +126,8 @@ protected static function _findItem($needles = null)
self::$lookup[$view] = array();
}
- if (isset($item->query['id'][0]))
+ // Only match menu items that list one tag
+ if (isset($item->query['id'][0]) && count($item->query['id']) == 1)
{
// Here it will become a bit tricky
// language != * can override existing entries
View
1  installation/CHANGELOG
@@ -30,6 +30,7 @@ $ -> Language fix or change
# [#30463] Missing closing div in compact tag layout. Thanks Thomas Hunziker and Constantin Romankiewicz
# [#30464] ACL Undefined variable error in contentadministrator.php on line 112. Thanks Constantin Romankiewicz
# [#30062] Base Item option in mod_menu has no effect [regression]. Thanks René Alain Erichsen
+ # [#30422] Tags drill down incorrectly finds match to multi-tag tagged items menu item. Thanks Mark
03-Apr-2013 Jean-Marie Simonet
# [#30349] Cleaned up use of database & query objects. Thanks Peter van Westen and Michael Babker
Please sign in to comment.
Something went wrong with that request. Please try again.