? Success

User tests: Successful: Unsuccessful:

avatar davdebcom
davdebcom
21 Oct 2014

In one of the previous Joomla! 3 major versions, 3.0, 3.1 or 3.2, I can't remember which, the default ordering for the Components menu was changed to be case sensitive. From that moment, extensions that started with a lowercase letter in their name would be listed at the bottom of the Components menu. This pull request reverts that change back, so the Components menu is truly in alphabetical order again.

The situation currently in Joomla! 3.3.x:
joomla - administration - control panel 2014-10-21 19-17-44

Corrected:
joomla - administration - control panel 2014-10-21 19-18-17

avatar chillcreations chillcreations - open - 21 Oct 2014
avatar jissues-bot jissues-bot - change - 21 Oct 2014
Labels Added: ?
avatar Bakual
Bakual - comment - 21 Oct 2014

In one of the previous Joomla! 3 major versions, 3.0, 3.1 or 3.2, I can't remember which, the default ordering for the Components menu was changed to be case sensitive.

That argument had been changed with http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=23340 before release of 1.7.3, commited on 2010-11-29.

So it's like this for two major releases already. Are you sure this was different in earlier Joomla 3.x versions?

avatar brianteeman
brianteeman - comment - 21 Oct 2014

Even if it was never this way before it is a simple and logical change to
make.

On 21 October 2014 18:39, Thomas Hunziker notifications@github.com wrote:

In one of the previous Joomla! 3 major versions, 3.0, 3.1 or 3.2, I can't
remember which, the default ordering for the Components menu was changed to
be case sensitive.

That argument had been changed with
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=23340
before release of 1.7.3, commited on 2010-11-29.

So it's like this for two major releases already. Are you sure this was
different in earlier Joomla 3.x versions?


Reply to this email directly or view it on GitHub
#4889 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar davdebcom
davdebcom - comment - 21 Oct 2014

@Bakual I have been selling these extensions with the lowercase double c since 2008, I'm pretty sure it wasn't like this in the 1.x or 2.x versions. But here is a screenshot on J 2.5.27

screen shot 2014-10-21 at 19 44 26 png 2014-10-21 19-44-44

I'm not sure, but the link you mention seems to be about the Extensions menu, I'm talking about the Components menu.

It's also a minor usability issue: sometimes people file a support request thinking the extension was not installed correctly. But they fail to look at the bottom of the Components menu, which is not a logical location to look for an extension starting with a "c" in it's name. I can imagine it's more user friendly for the Components menu to be truly alphabetical.

avatar davdebcom
davdebcom - comment - 21 Oct 2014

I tried to find earlier references to that file and code.

Seems it was still "false" in October 2010:
546b8fe

Was changed to "true" in November 2010 (that's the same issue you mentioned @Bakual, you are right about that):
66667fb

And then $lang->getLocale() gets changed to "true" by Elin Waring even later.

Why it was changed from false to true in the past or if the change from true to false now will cause the issue described in the earlier link, is not something I can answer. But in 1.7.3 then, and 2.5.27 now, the ordering is not case sensitive as my screenshot shows?

I don't know what this means, maybe a (framework) change somewhere else? But off course my opinion is: let's change it unless we see real issues that might occur because of the change.

avatar davdebcom
davdebcom - comment - 21 Oct 2014

By the way, free extension that can be used for testing:
http://extensions.joomla.org/extensions/languages/multi-lingual-content/21014

avatar hvdmeer hvdmeer - test_item - 21 Oct 2014 - Tested successfully
avatar hvdmeer
hvdmeer - comment - 21 Oct 2014

@test Downloaded jDiction which can be found at bottom of Components menu. After applying patch it was sorted alphabetically correctly.

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

avatar Bakual
Bakual - comment - 21 Oct 2014

Even if it was never this way before it is a simple and logical change to
make.

I don't disagree with that.

I just wondered why it was changed and found it was long ago. There is likely a change somewhere else as well which impacted that since it apparently is different in 2.5. It may be related to the last argument as the ordering to my knowledge is different in the various languages.

avatar davdebcom
davdebcom - comment - 21 Oct 2014

I have emailed Martijn and asked him if he remembers the previous issue. I
hope he'll reply here when he finds the time.

avatar MartijnMaandag
MartijnMaandag - comment - 22 Oct 2014

The item I reported in 2010 is about the names in the filters (content of filter).
If I look now the order in the filters is still not in the used language but of the original English names (2.5 and 3.x). (picture).
I agree that it's better that the Components menu is not case sensitive.
The patch fixes this in the components menu.
I must say that the display of Extension manager - Manage is also case sensitive and your extensions (and templates like beez, hathor and isis) are at the end. It's better to change this also (like it is in 2.5.x)?
filter

avatar gwsdesk gwsdesk - test_item - 23 Oct 2014 - Tested successfully
avatar gwsdesk gwsdesk - alter_testresult - 23 Oct 2014 - gwsdesk: Tested successfully
avatar gwsdesk
gwsdesk - comment - 23 Oct 2014

@test Patch works4me 'breezingforms' shows now alphabetically correct

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

avatar brianteeman brianteeman - change - 23 Oct 2014
Category Components
avatar davdebcom
davdebcom - comment - 23 Oct 2014

@MartijnMaandag Thank you for replying! I'll look into the other changes required on other locations when this change is merged successfully.

@Bakual So it seems the previous change in the module helper was not related to the issue that Martijn was discussing in 2010.

It seems to be this change has no remaining reasons not to be merged?

avatar Bakual Bakual - change - 23 Oct 2014
Labels Added: ?
avatar Bakual
Bakual - comment - 23 Oct 2014

Setting RTC.

avatar phproberto phproberto - close - 23 Oct 2014
avatar phproberto phproberto - change - 23 Oct 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-10-23 15:32:46

Add a Comment

Login with GitHub to post a comment