Skip to content

[4.4] Fix filter for option, format, view and layout in cache and pagination#43983

Merged
MacJoom merged 2 commits intojoomla:4.4-devfrom
SniperSister:fix/pagination-parameter-filter
Aug 25, 2024
Merged

[4.4] Fix filter for option, format, view and layout in cache and pagination#43983
MacJoom merged 2 commits intojoomla:4.4-devfrom
SniperSister:fix/pagination-parameter-filter

Conversation

@SniperSister
Copy link
Contributor

Summary of Changes

Improved version of the PR #43976 that covers all relevant parameters and reflects the changes in the cache key generation method.

Testing Instructions

See original PR

Actual result BEFORE applying this Pull Request

See original PR

Expected result AFTER applying this Pull Request

See original PR

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • [x ] No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • [x ] No documentation changes for manual.joomla.org needed

@Fedik
Copy link
Member

Fedik commented Aug 25, 2024

I have tested this item ✅ successfully on 76e607e


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

@richard67
Copy link
Member

I have tested this item ✅ successfully on 76e607e


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 25, 2024
@MacJoom MacJoom added this to the Joomla! 4.4.8 milestone Aug 25, 2024
@MacJoom MacJoom merged commit b08efee into joomla:4.4-dev Aug 25, 2024
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 25, 2024
@MacJoom
Copy link
Contributor

MacJoom commented Aug 25, 2024

Thanks for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants