? Success
Pull Request for # 9186

User tests: Successful: Unsuccessful:

avatar bembelimen
bembelimen
25 Mar 2016

Pull Request for Issue #9186

With 3.5 a new function was implemented in Joomla!: the items of a category can be counted via a helper method. The problem is, that this only works for basic category calls without sections because the extension string is not handled properly!

Summary of Changes

I moved the countItems stuff to its own method in the category and implemented a cleaner way for this function.

Testing Instructions

  1. Open the contact categories
  2. Change the link paameter to extension=com_contact.foobar
  3. create a new category
  4. no Counting visible, because Joomla! tries to load Contact.FoobarHelper class (pay attention to the ".")
  5. Apply path => Try again

Now extensions could use the second parameter "$section" to check for the correct items

Votes

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

avatar bembelimen bembelimen - open - 25 Mar 2016
avatar bembelimen bembelimen - change - 25 Mar 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Mar 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 25 Mar 2016
Category Components
avatar matrikular matrikular - test_item - 25 Mar 2016 - Tested successfully
avatar matrikular
matrikular - comment - 25 Mar 2016

I have tested this item :white_check_mark: successfully on 6ed19dd


Since without a component that supports sections one cannot really check if the count is still correct. I've tested this by assinging test contacts to that created category via phpmyadmin.

contact_category
contacts

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

avatar chmst
chmst - comment - 25 Mar 2016

I tested this successfully - simulating sections via phpmyadmin as matrikular did.

avatar chmst chmst - test_item - 25 Mar 2016 - Tested successfully
avatar chmst
chmst - comment - 25 Mar 2016

I have tested this item :white_check_mark: successfully on 6ed19dd


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

avatar brianteeman brianteeman - change - 25 Mar 2016
Rel_Number 0 9186
Relation Type Pull Request for
avatar zero-24
zero-24 - comment - 26 Mar 2016

@brianteeman can we get a RTC label here?

avatar brianteeman brianteeman - change - 26 Mar 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 26 Mar 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 26 Mar 2016
Labels Added: ?
avatar Kubik-Rubik Kubik-Rubik - change - 29 Mar 2016
Milestone Added:
avatar Kubik-Rubik Kubik-Rubik - change - 29 Mar 2016
Milestone Added:
avatar Kubik-Rubik Kubik-Rubik - change - 29 Mar 2016
Milestone Removed:
avatar Kubik-Rubik
Kubik-Rubik - comment - 29 Mar 2016

Thank you @bembelimen! This can be merged already for the next 3.5.1.

avatar Kubik-Rubik Kubik-Rubik - change - 29 Mar 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-03-29 11:01:02
Closed_By Kubik-Rubik
avatar Kubik-Rubik Kubik-Rubik - close - 29 Mar 2016
avatar Kubik-Rubik Kubik-Rubik - merge - 29 Mar 2016
avatar joomla-cms-bot joomla-cms-bot - close - 29 Mar 2016
avatar Kubik-Rubik Kubik-Rubik - reference | eeba92e - 29 Mar 16
avatar Kubik-Rubik Kubik-Rubik - merge - 29 Mar 2016
avatar Kubik-Rubik Kubik-Rubik - close - 29 Mar 2016
avatar joomla-cms-bot joomla-cms-bot - change - 29 Mar 2016
Labels Removed: ?
avatar Kubik-Rubik
Kubik-Rubik - comment - 29 Mar 2016

Sorry, I had to revert the merge because we are already in RC state and I should not have merged it.

@bembelimen Could you please do the same PR once again, then we can merge it into 3.5.2. Sorry for the inconvenience!

avatar wilsonge wilsonge - reference | 9e75468 - 29 Mar 16
avatar Kubik-Rubik
Kubik-Rubik - comment - 29 Mar 2016

@wilsonge Thank you for merging! :-)

avatar bembelimen bembelimen - head_ref_deleted - 29 Mar 2016

Add a Comment

Login with GitHub to post a comment