Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for issue #5764: archived articles list filter #6158

Merged
merged 1 commit into from Mar 14, 2015

Conversation

waader
Copy link
Contributor

@waader waader commented Feb 23, 2015

Please see #5764 for the description of the issue.


Hi,

On one of our customers website, we noticed a bug with the filter and pagination of archived articles.

As long as we do not use the filter except for the number of article, the pagination act normally. If we use the year or month filter, for exemple year 2013, and then click on page 2, the year / month filter does a reset and the archive is displayed the right number of article forward but loosing the year criteria.

We noticed that on a joomla 2.5.27 but as we are going to upgrade to joomla 3 soon, we decided to go for some testing on fresh installations of joomla 2.5.28 and joomla 3.3.6 with just the demo data and some articles for the purpose of the test.

Both show the same bug.

You can see exemples on the fresh installations we used for testing, filtering with year 2013 :

Joomla 3.3.6 http://test.anw-sandbox.ch/index.php/archive

Joomla 2.5.28 http://test25.anw-sandbox.ch/index.php/archive

System information (as much as possible)

Joomla 3.3.6 / 2.5.28
PHP 5.5.9

Additional comments

Thanks for your help

@n9iels
Copy link
Contributor

n9iels commented Feb 28, 2015

@test confirmed, the patch solves the bug. Thanks!

@wmgraf
Copy link

wmgraf commented Mar 14, 2015

@test success, the patch works well.

@brianteeman
Copy link
Contributor

Setting RTC - thanks for testing


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

@brianteeman brianteeman added the RTC This Pull Request is Ready To Commit label Mar 14, 2015
mbabker added a commit that referenced this pull request Mar 14, 2015
Fix for issue #5764: archived articles list filter
@mbabker mbabker merged commit 9170a31 into joomla:staging Mar 14, 2015
@mbabker mbabker added this to the Joomla! 3.4.1 milestone Mar 14, 2015
@zero-24 zero-24 removed the RTC This Pull Request is Ready To Commit label Oct 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants