? ? Success
Not before # 6916

User tests: Successful: Unsuccessful:

avatar pe7er
pe7er
13 May 2015

This patch creates in "Category Manager: Banners" 4 new columns with the number of banners per category. It contains a helper file for com_banners that is used in com_categories to calculate the number of banner items (published, unpublished, archived, and trashed) in each category.

Test instructions:

The Category Manager: Banners (Components > Banners > Categories) does not display the number of banners in each category.

screen shot 2015-05-13 at 14 24 31

Create some new banners that are unpublished, trashed & archived (tip: name the unpublished banner "Unpublished Banner" to test if the calculations are correct).

screen shot 2015-05-13 at 14 24 31

Note This patch works only in combination with #6916. Install both #6916 and this patch (the order of installation does not matter, you just need both).
After installing the patch, The Category Manager: Banners (Components > Banners > Categories) should displays 4 new columns.

screen shot 2015-05-13 at 14 24 31

Please also check the other administrator template Hathor

In Extensions > Template Manager > Select Hathor
Before the patch:

screen shot 2015-05-13 at 14 24 31
After the patch:
screen shot 2015-05-13 at 14 24 31

avatar pe7er pe7er - open - 13 May 2015
avatar pe7er pe7er - change - 13 May 2015
The description was changed
avatar joomla-cms-bot joomla-cms-bot - change - 13 May 2015
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 13 May 2015
Labels Added: ?
avatar pe7er
pe7er - comment - 13 May 2015

Thanks @losedk !
I've corrected it and uploaded a new patch.

avatar hans2103
hans2103 - comment - 14 May 2015

@test succes

Tested both desktop and mobile.

avatar zero-24 zero-24 - alter_testresult - 15 May 2015 - hans2103: Tested successfully
avatar zero-24 zero-24 - change - 15 May 2015
Status New Pending
Rel_Number 0 6916
Relation Type Not before
Easy No Yes
avatar zero-24 zero-24 - change - 15 May 2015
Category Administration UI/UX
avatar zero-24 zero-24 - change - 15 May 2015
Labels Added: ?
avatar woluweb
woluweb - comment - 19 May 2015

on issue #6934 (count for BANNERS), I noticed that clicking on "zeros" would still show all banners of the category, no matter their "status".
But apparently, this little bug only occurs for com_banners (works fine for the current com_content, for com_contact and com_newsfeed)


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

avatar woluweb woluweb - test_item - 19 May 2015 - Tested unsuccessfully
avatar pe7er
pe7er - comment - 19 May 2015

Thank you both for testing. I've analysed & reproduced the problem: The status filter of the com_banners component works differently from other components.

The category manager com_categories creates number counters with hyperlinks with status filter &filter[published]=. All components listen for the "Select Status" filter to &filter[published]= (and expect a statuscode of 0,1,2 and -2).

However the banner component listens to &filter[state]=.
What's the best way to "catch" the variable filter[published] and set the filter[state]?

avatar woluweb
woluweb - comment - 19 May 2015

Hi Peter,
Your question is a bit too technical for me.
Good sense would say : if the Banner Component is the only one to use [state] instead of [published], maybe we should fix the Banner Component instead of changing "state" by "published" on the fly ?
But maybe this is a stupid suggestion, I am not a developer :-)

avatar pe7er
pe7er - comment - 20 May 2015

I've changed the 'state' field to 'published' in the filter administrator/components/com_banners/models/forms/filter_banners.xml and in the banners model.

Now this PR (in combination with PR #6916) displays a working "banner item counts" in the Category Manager: Banners that are hyperlinked correctly to ** Banner Manager: Banners** with filtered items on published & category filter.

avatar woluweb
woluweb - comment - 20 May 2015

Hi Peter,
I have just tested.
I confirm : the filters do now work perfectly also for Banner Component
Txs


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

avatar woluweb woluweb - test_item - 20 May 2015 - Tested successfully
avatar zero-24 zero-24 - change - 20 May 2015
Status Pending Ready to Commit
avatar zero-24 zero-24 - change - 20 May 2015
Labels Added: ?
avatar Kubik-Rubik Kubik-Rubik - change - 21 May 2015
Milestone Added:
avatar Bakual Bakual - reference | 28574c5 - 3 Jun 15
avatar zero-24 zero-24 - close - 3 Jun 2015
avatar Bakual Bakual - close - 3 Jun 2015
avatar Bakual
Bakual - comment - 3 Jun 2015

Merged with 28574c5 into 3.5-dev.

Thanks!

avatar Bakual Bakual - change - 3 Jun 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-06-03 14:03:32
Closed_By Bakual
avatar Bakual Bakual - close - 3 Jun 2015
avatar roland-d roland-d - reference | d85d2c8 - 6 Aug 15
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?
avatar pe7er pe7er - head_ref_deleted - 5 Nov 2015

Add a Comment

Login with GitHub to post a comment