? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
4 Dec 2021

Pull Request for Issue #36102 .

Summary of Changes

Progressive caching caches the modules per position, however this breaks custom menu assignment for module.
I have add Itemid to cache key for Progressive caching, so it works correctly now.

The same issue is for Joomla 3, can be back-ported if this PR is okay.

Testing Instructions

Apply patch.
Create a custom module, and assign to one specific menu.
Enable Progressive caching, and visit that menu, then visit other pages.

Actual result BEFORE applying this Pull Request

The module visible on all pages.

Expected result AFTER applying this Pull Request

The module visible only on selected page.

Documentation Changes Required

None.

avatar Fedik Fedik - open - 4 Dec 2021
avatar Fedik Fedik - change - 4 Dec 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Dec 2021
Category Libraries
avatar Fedik Fedik - change - 4 Dec 2021
Labels Added: ?
avatar ceford ceford - test_item - 7 Dec 2021 - Tested successfully
avatar ceford
ceford - comment - 7 Dec 2021

I have tested this item successfully on 6100906


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

avatar zero-24
zero-24 - comment - 8 Jan 2022

FYI i did the changes somiliar to the 3.10 PR here too. cc @wilsonge

avatar wilsonge wilsonge - change - 8 Jan 2022
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-01-08 20:40:47
Closed_By wilsonge
avatar wilsonge wilsonge - close - 8 Jan 2022
avatar wilsonge wilsonge - merge - 8 Jan 2022
avatar wilsonge
wilsonge - comment - 8 Jan 2022

Thanks!

Add a Comment

Login with GitHub to post a comment