? ? Failure

User tests: Successful: Unsuccessful:

avatar nvyush
nvyush
28 Jul 2017

Pull Request for Issue # .

Summary of Changes

For Contacts component:
/administrator/components/com_contact/config.xml - add missing show_category_heading_title_text parameter.
/components/com_contact/views/categories/tmpl/default.xml - add missing show_category_heading_title_text parameter, code style.
/components/com_contact/views/category/tmpl/default.xml - add missing show_category_heading_title_text parameter, code style.

Testing Instructions

Use a fresh Joomla, apply the patch.
Create a tree of some contact categories.
Create contacts with default options and place they to some contact categories.
Create menu items of the types Contacts » List All Contact Categories and Contacts » List Contacts in a Category for the root contact category.
Check the contact categories in the frontend. All should be fine.
Play with the option 'Show Subcategories Text' in the Global Configuration » Contacts, in the Category tab of the menu items you created.

NB!: When testing the menu item of the type Contacts » List All Contact Categories, the menu item of the type Contacts » List Contacts in a Category must be unpublished.

Expected result

When the option 'Show Subcategories Text' is Off in backend, the label "Subcategories" must not be shown in frontend.

Actual result

The option 'Show Subcategories Text' is missing in backend, the label "Subcategories" is shown always in frontend.

Documentation Changes Required

avatar joomla-cms-bot joomla-cms-bot - change - 28 Jul 2017
Category Administration com_contact Front End
avatar nvyush nvyush - open - 28 Jul 2017
avatar nvyush nvyush - change - 28 Jul 2017
Status New Pending
avatar nvyush nvyush - change - 28 Jul 2017
The description was changed
avatar nvyush nvyush - edited - 28 Jul 2017
avatar nvyush nvyush - change - 28 Jul 2017
Title
Patch 3
Add missing 'Show Subcategories Text' param to Contacts
avatar nvyush nvyush - edited - 28 Jul 2017
avatar Heggi93 Heggi93 - test_item - 21 Aug 2017 - Tested unsuccessfully
avatar Heggi93
Heggi93 - comment - 21 Aug 2017

I have tested this item 🔴 unsuccessfully on 5f42d0f

I create a tree of categories with a depth of 3 and added some contacts to the categories. For the type Contacts >> List Contacts in a Category the button worked fine. The button is shown in the administration view and the title 'Subcategories' in the front-end worked as described.

For the type Contacts >> List All Contact Categories the button is shown in backend but doesen't work. Also if I unpublish the Contacts >> List Contacts in a Category item the selection has no effect on the view. After the selection of the root category the title 'Subcategories' is always shown.


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

avatar nvyush
nvyush - comment - 29 Aug 2017

@Heggi93 I rechecked, all work. When you test the menu item of the type 'Contacts >> List All Contact Categories' you need to open some subcategory of the root category and test that page. It is important that the menu item of the type 'Contacts » List Contacts in a Category' must be unpublished, otherwise in this case that menu item will be opened.

avatar nvyush nvyush - change - 26 Apr 2019
Labels Removed: J3 Issue
avatar MeghaBiranje MeghaBiranje - test_item - 5 Dec 2020 - Tested successfully
avatar MeghaBiranje
MeghaBiranje - comment - 5 Dec 2020

I have tested this item successfully on 5116eca

I have tested this successfully


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

avatar gostn gostn - test_item - 6 Dec 2020 - Tested successfully
avatar gostn
gostn - comment - 6 Dec 2020

I have tested this item successfully on 5116eca


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

avatar Quy Quy - change - 5 Feb 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 5 Feb 2022

RTC


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

avatar Quy
Quy - comment - 5 Feb 2022

RTC


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

avatar zero-24 zero-24 - change - 20 Feb 2022
Labels Added: ? ?
Removed: ?
avatar laoneo
laoneo - comment - 11 Mar 2022

Since Joomla 3 is not accepting new features, we would like to ask you to rebase your PR to the 4.2 branch to be tested in Joomla 4. In the meantime we close it. When ready for testing please reopen the pr.

avatar laoneo laoneo - close - 11 Mar 2022
avatar laoneo laoneo - change - 11 Mar 2022
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2022-03-11 18:55:48
Closed_By laoneo
avatar brianteeman
brianteeman - comment - 11 Mar 2022

it is NOT a new feature it is fixing a bug!!!

avatar laoneo
laoneo - comment - 11 Mar 2022

When you add new parameters, then I doubt that it is fixing a bug. What is broken without the pr?

avatar brianteeman
brianteeman - comment - 11 Mar 2022

If you read the post you will see - I really wonder sometimes if people prefer to just click close instead of actually reading and trying to understand

avatar laoneo
laoneo - comment - 12 Mar 2022

@brianteeman does this pr the same as #32736?

avatar brianteeman
brianteeman - comment - 12 Mar 2022

That PR is for j4

Add a Comment

Login with GitHub to post a comment