?
avatar brianteeman
brianteeman
13 Dec 2015

Since the change to combine the two ordering selects into a single select (field plus direction) the ordering is completely broken

To test - install latest staging with sample data
Go to newsfeeds and try to sort by ID Ascending or ID Descending. The order should change but it doesnt
Go to tags and try to sort by ID or Title. The order should change but it doesnt

avatar brianteeman brianteeman - open - 13 Dec 2015
avatar brianteeman brianteeman - change - 13 Dec 2015
The description was changed
Title
Regression - ordering broken in Newsfeed UI
Regression - ordering broken in Newsfeed and Tags
avatar brianteeman
brianteeman - comment - 13 Dec 2015

Updated to include tags component


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

avatar brianteeman brianteeman - change - 13 Dec 2015
Title
Regression - ordering broken in Newsfeed UI
Regression - ordering broken in Newsfeed and Tags
avatar Bakual
Bakual - comment - 13 Dec 2015

Do you know which PR it was that changed this select? Was it part of the Smart Search implementation?

avatar brianteeman
brianteeman - comment - 13 Dec 2015

Not had a chance to check but I would suspect so.
The problem probably came when the ordering select was changed from two boxes (field and direction) to the combined box

I discovered it while trying to create a single select for the two remaining places where there is a double select (banner-tracks and content-languages) and couldnt work out why it wasnt working

avatar bertmert
bertmert - comment - 13 Dec 2015

Just had a short look (newsfeeds default.php)
1.)

<input type="hidden" name="filter_order" value="<?php echo $listOrder; ?>" />
<input type="hidden" name="filter_order_Dir" value="<?php echo $listDirn; ?>" />

must be removed from default.php.

$sortFields = $this->getSortFields();

should be removed

JavaScript block

Joomla.orderTable = function()

should be removed

All

JHtml::_('grid.sort', 

must be

JHtml::_('searchtools.sort'

We had a similiar issue in com_contact some days ago.

avatar brianteeman
brianteeman - comment - 13 Dec 2015

Can you do a PR for Newsfeeds and Tags please - might be needed in other places but didnt spot any

avatar bertmert
bertmert - comment - 13 Dec 2015

OK.

avatar brianteeman
brianteeman - comment - 13 Dec 2015

Thanks

avatar Bakual
Bakual - comment - 13 Dec 2015

Personally I would prefer the two box select as I think the single box is getting confused with such many options. But that's me.
It's for sure better to be consistent and working obviously :smile:

avatar brianteeman
brianteeman - comment - 13 Dec 2015

See #8670 for a fix for Newsfeeds

avatar brianteeman brianteeman - change - 13 Dec 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-12-13 18:28:03
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 13 Dec 2015

Closed as we have PR


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

avatar brianteeman brianteeman - close - 13 Dec 2015

Add a Comment

Login with GitHub to post a comment