? ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
9 Sep 2017

Pull Request for Issue #17901.

Summary of Changes

fix category list/blog/featured sql error on postgresql

you cannot ORDER BY on a field not selected

Testing Instructions

Go to frontpage and click on Article Category Blog or Article Category List
/index.php/article-category-list
/index.php/article-category-blog
/index.php/featured-articles
/index.php/content-modules

Expected result

no error

Actual result

sql error

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
4.00

avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2017
Category Front End com_content
avatar alikon alikon - open - 9 Sep 2017
avatar alikon alikon - change - 9 Sep 2017
Status New Pending
avatar alikon alikon - change - 9 Sep 2017
Labels Added: ?
avatar alikon alikon - change - 9 Sep 2017
The description was changed
avatar alikon alikon - edited - 9 Sep 2017
avatar alikon alikon - change - 9 Sep 2017
Title
[com_content] - category list/blog sql error on postgresql
[com_content] - category list/blog/featured sql error on postgresql
avatar alikon alikon - edited - 9 Sep 2017
avatar csthomas csthomas - test_item - 12 Sep 2017 - Tested successfully
avatar csthomas
csthomas - comment - 12 Sep 2017

I have tested this item successfully on e34d35f

Tested on postgresql and mysql.


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

avatar csthomas csthomas - test_item - 14 Sep 2017 - Tested successfully
avatar csthomas csthomas - test_item - 14 Sep 2017 - Tested successfully
avatar csthomas
csthomas - comment - 14 Sep 2017

I have tested this item successfully on 678e67c


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

avatar csthomas
csthomas - comment - 22 Sep 2017

@tormes You may be interested in testing this PR and you can mark it as success at https://issues.joomla.org/tracker/joomla-cms/17902
After two success tests it should be mark as ready to commit (RTC).

avatar tormes
tormes - comment - 22 Sep 2017

When I overwrite the file components/com_content/models/articles.php with the same file from @alikon (last commit 678e67c of Pull Request #17902), all errors reported are fixed. Note that this file has another difference in line 270 from the original in 3.8.0.


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

avatar tormes tormes - test_item - 22 Sep 2017 - Tested successfully
avatar tormes
tormes - comment - 22 Sep 2017

I have tested this item successfully on 678e67c

Tested overwriting the modified file over 3.8.0 stable.


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Sep 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Sep 2017

RTC after two successful tests.

avatar mbabker mbabker - close - 25 Sep 2017
avatar mbabker mbabker - merge - 25 Sep 2017
avatar mbabker mbabker - change - 25 Sep 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-09-25 11:43:59
Closed_By mbabker
Labels Added: ?

Add a Comment

Login with GitHub to post a comment