? Success

User tests: Successful: Unsuccessful:

avatar sanderpotjer
sanderpotjer
13 Feb 2014

New date filter for the Most Read Content module (mod_articles_popular).

With the new options you can display only items:

  • published/created/modified in the past X days
  • published/created/modified between two dates

This allow you to display for example the most read content of articles created in the past 7 days.

avatar sanderpotjer sanderpotjer - open - 13 Feb 2014
avatar hans2103
hans2103 - comment - 31 May 2014

@test
test succesful

with the patch applied I notice more field in mod_articles_popular
mind that the Joomla developer test sample data has dates back to 1 January 2011
seeing it working perfectly.

avatar hans2103
hans2103 - comment - 31 May 2014

@sanderpotjer
consider hiding the configurable fields (Date Field, Start Date Range, To Date, Relative Date) when Data Filtering is Off

avatar zanardigit
zanardigit - comment - 31 May 2014

@test
Tested successfully. Applied a date range filter, and an article created outside that date range was not shown anymore.

avatar yireo
yireo - comment - 31 May 2014

Tested successfully. Checked it with articles from demo-data (ok, actually just the demo-content with 1 article) and the date set the period in which the article was created. Works as expected. The "Relative Date" also works as expected. Also inspected the underlying PHP-code. No comments from me.
You may blame the J!Tracker Application for transmitting this comment.

avatar Bakual
Bakual - comment - 31 May 2014

I'm setting this to RTC, but it has to go into 3.4-dev as new feature.

@sanderpotjer There are some minor codestyle issues which would be nice if you can fix them.

Thanks all for testing!

avatar sanderpotjer
sanderpotjer - comment - 3 Jun 2014

Thanks all for testing!

@Bakual code style issues fixed, thanks.

avatar Bakual Bakual - reference | - 9 Jul 14
avatar Bakual Bakual - close - 9 Jul 2014
avatar Bakual
Bakual - comment - 9 Jul 2014

Merged into 3.4. Thanks for code and testing!

avatar Bakual Bakual - change - 9 Jul 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-07-09 20:04:55
avatar Bakual Bakual - close - 9 Jul 2014
avatar sanderpotjer
sanderpotjer - comment - 11 Jul 2014

Thanks all for testing and merging the code!

avatar sanderpotjer sanderpotjer - head_ref_deleted - 12 Jul 2014

Add a Comment

Login with GitHub to post a comment