? Pending

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
22 Jun 2019

Summary of Changes

In backend when sorting the articles by "Ordering", it will now first order by the category title and only after that it sorts by ordering.
This is consistent with other core components (eg contacts, banners, newsfeeds) and makes drag-and-drop ordering more reliable.

Testing Instructions

  • Create multiple articles in multiple categories.
  • Order by "Ordering"
  • Check if items from the same category are grouped.
  • Move the items around - reload the page and verify that they "stick" to your order.

Expected result

Items aren't grouped by category.
Items keep in place after reloading.

Actual result

Items of the same category are right beside them.
The items jump around seemingly randomingly

Documentation Changes Required

None

avatar Bakual Bakual - open - 22 Jun 2019
avatar Bakual Bakual - change - 22 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jun 2019
Category Administration com_content
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Jun 2019
avatar rogercreagh
rogercreagh - comment - 23 Jun 2019

@rogercreagh please test (how to: https://docs.joomla.org/Testing_Joomla!_patches)

Happy to do so, but it would take me several days or weeks of work to grapple with that as it looks very complex to someone who doesn't use git, has never used Joomla unit tests (or any other type of unit testing) and is a !professional software developer with better things to do than fill my brain up with a load of important stuff that I'll probably never use again.

I got in a right twizzle a few months ago when someone suggested I generate a pull request for a different problem and I then found myself lost in a flood of confusion (for a start it seems to me that a pull is actually a push and vice versa). As a result the fix that I had for some bugs in a JHtml string function have never been implemented as the case was closed. I'm reluctant to mess up this fix likewise...

On the face of it by inspecting the code it looks like it should work. Pretty much the same as my hack solution which works fine on several sites.

PS the bugs in JHtml::_('string.truncate',...) still exist but I have developed a plugin that overrides the duff core function and works fine on my sites so I'm not going to revisit that here - once bitten, twice shy!

I wish there was a way of reporting bugs and fixes in Joomla for people who find them and have identified the problems and have a fix that works but are not able to do the whole git core development thing.
And VERY many thanks to those who are able and do do all that stuff - you are heroes and I wish I could help you in some way.

avatar Bakual
Bakual - comment - 23 Jun 2019

Try with https://docs.joomla.org/Component_Patchtester_for_Testers. That's much simpler and should work for this PR just fine.

avatar Bakual
Bakual - comment - 23 Jun 2019

Also, for creating a simple PR like this one (and maybe your truncate one), it may be sufficient to use the GitHub WebUI. See https://docs.joomla.org/Using_the_Github_UI_to_Make_Pull_Requests

I agree, Git can be confusing. Especially at the beginning the learning curve is very steep.
The GitHub WebGUI takes care of that stuff for creating simple PRs. And the Joomla Patchtester takes care of that stuff for testing Fixes.

avatar Quy Quy - test_item - 24 Jun 2019 - Tested successfully
avatar Quy
Quy - comment - 24 Jun 2019

I have tested this item successfully on a2fcdda


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

avatar Quy
Quy - comment - 24 Jun 2019

I have tested this item successfully on a2fcdda


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

avatar alikon alikon - test_item - 24 Jun 2019 - Tested successfully
avatar alikon
alikon - comment - 24 Jun 2019

I have tested this item successfully on a2fcdda


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 24 Jun 2019
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 24 Jun 2019

Status "Ready To Commit".

avatar HLeithner HLeithner - change - 25 Jun 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-06-25 12:02:08
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 25 Jun 2019
avatar HLeithner HLeithner - merge - 25 Jun 2019
avatar HLeithner
HLeithner - comment - 25 Jun 2019

thx

avatar oorzaak
oorzaak - comment - 12 Dec 2019

Hi,

I experienced a similar issue when transfering an article from one category to another. Depending on the content, it may be likely that this article will have the same ordering number as an article that is already present in the new category, giving unpredicable results. This is especially the case with articles that are newly created; typically these will have number 0. If there are more articles with numer 0 (or even with negative numbers) this gets very confusing.

Are situations like this covered by this issue # ?

avatar Bakual
Bakual - comment - 12 Dec 2019

Commenting on merged or closed PRs will not get you the attention you need.
You should instead open a new issue.

Add a Comment

Login with GitHub to post a comment