? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
6 Apr 2016

Similar patches to #9739

When the field categoryedit is used, the list of categories will now display the language code (xx-XX) when it is not set to "All" languages.

To test, for example, edit an article and click on the Parent field.

You should get:

screen shot 2016-04-06 at 08 41 19

Although the field categoryparent is not used in core (should it be tagged as deprecated?), it may be used by third party extensions. I therefore also modified it the same way.
To test, change in /administrator/components/com_categories/models/forms/category.xml the field name "parent_id" from

    <field
        name="parent_id"
        type="categoryedit"
        label="COM_CATEGORIES_FIELD_PARENT_LABEL"
        description="COM_CATEGORIES_FIELD_PARENT_DESC"/>

to

    <field
        name="parent_id"
        type="categoryparent"
        label="COM_CATEGORIES_FIELD_PARENT_LABEL"
        description="COM_CATEGORIES_FIELD_PARENT_DESC"/>

This patch, imho, is more important than #9741, although I see the usefulness of #9741.

In an ideal world, both would be useful, but the dropdown width is so narrow that I am not sure we can add both the id and the language in the list.

In any case, we can merged this series with the language and decide later if displaying also the id in the 3 fields concerned if OK.

Testers and comments welcome:
@brianteeman @MATsxm @SharkyKZ @andrepereiradasilva
@joomla/cms-maintainers

avatar infograf768 infograf768 - open - 6 Apr 2016
avatar infograf768 infograf768 - change - 6 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2016
Labels Added: ?
avatar dfrassi
dfrassi - comment - 6 Apr 2016

With respect to general government ( in Italy 23% of the sites is made in joomla ) it is more rare to have a site in multi - language. While it is very common to have multiple categories with the same name. I think it is good to keep them both.

avatar infograf768
infograf768 - comment - 6 Apr 2016

Let's maintainers decide. :smiley:
It would give something like this:
screen shot 2016-04-06 at 09 42 56

busy busy as you can see

avatar brianteeman brianteeman - change - 6 Apr 2016
Category Multilanguage Templates (admin) UI/UX
avatar brianteeman brianteeman - test_item - 6 Apr 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 6 Apr 2016

I have tested this item :white_check_mark: successfully on 44954f5


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

avatar SharkyKZ SharkyKZ - test_item - 6 Apr 2016 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 6 Apr 2016

I have tested this item :white_check_mark: successfully on 44954f5


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

avatar brianteeman brianteeman - change - 6 Apr 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 6 Apr 2016

RTC - thanks


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

avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 6 Apr 2016
Milestone Added:
avatar dfrassi
dfrassi - comment - 7 Apr 2016

Thanks

avatar rdeutz rdeutz - reference | 1693f7a - 12 Apr 16
avatar rdeutz rdeutz - merge - 12 Apr 2016
avatar rdeutz rdeutz - close - 12 Apr 2016
avatar rdeutz rdeutz - change - 12 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-12 20:29:06
Closed_By rdeutz
avatar rdeutz rdeutz - close - 12 Apr 2016
avatar rdeutz rdeutz - merge - 12 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - close - 12 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - change - 12 Apr 2016
Labels Removed: ?
avatar infograf768 infograf768 - head_ref_deleted - 17 Apr 2016
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:

Add a Comment

Login with GitHub to post a comment