?
Referenced as Pull Request for: # 10620
avatar selfget
selfget
24 May 2016

Steps to reproduce the issue

In /administrator/components/com_menus/model/items.php, function:

protected function getListQuery()

    // Filter the items over the parent id if set.
    $parentId = $this->getState('filter.parent_id');

    if (!empty($parentId))
    {
        $query->where('p.id = ' . (int) $parentId);
    }

But p.id causes an SQL error (there is no table with p alias in the query build. Probably it should be:

    // Filter the items over the parent id if set.
    $parentId = $this->getState('filter.parent_id');

    if (!empty($parentId))
    {
        //$query->where('p.id = ' . (int) $parentId);
        $query->where('a.parent_id = ' . (int) $parentId);
    }

System information (as much as possible)

The issue was detected in Joomla 3.5.1

avatar selfget selfget - open - 24 May 2016
avatar sovainfo
sovainfo - comment - 24 May 2016

Regression due to commit on Jan 5, 2010 by Hackwar
Removing the table with p alias but not changing references!

avatar mbabker
mbabker - comment - 24 May 2016

And the award for the longest lived bug in what has to be the biggest edge case in Joomla goes to... ????

avatar alikon alikon - reference | a7ab3ff - 25 May 16
avatar brianteeman brianteeman - change - 25 May 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-05-25 09:28:17
Closed_By brianteeman
avatar brianteeman brianteeman - close - 25 May 2016
avatar brianteeman brianteeman - close - 25 May 2016
avatar brianteeman
brianteeman - comment - 25 May 2016

Closed as we have a PR #10618 thanks


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10618.

avatar alikon
alikon - comment - 25 May 2016

@selfget can you test #10620

avatar roland-d roland-d - reference | c792e6f - 25 May 16
avatar selfget
selfget - comment - 26 May 2016

@alikon tested and it works now

Add a Comment

Login with GitHub to post a comment