RTC PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar Reda-Muhamed
Reda-Muhamed
20 Sep 2026

Pull Request resolves #48294.

This is the 5.4 version of PR #48296, which targets 6.2-dev.

I am opening this against 5.4 because the bug also exists in the current 5.4 branch and the fix is small, self-contained, and backward-compatible. Getting it merged in 5.4 first allows the fix to be available sooner in the next 5.4 patch release and then forward-merged through the usual Joomla release branches.

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

Fixes banners being displayed and opened when their category is unpublished or inaccessible to the current user.

The Banners module now filters banners by the published state and access level of their category. The direct banner click endpoint applies the same checks, so unavailable banners cannot redirect users or increment click counts.

The banner click cache key now includes the current user's authorised view levels, so cached banner results are not shared across users with different access.

Testing Instructions

  1. Create and publish a banner with a valid click URL.
  2. Assign it to a published banner category and display it with the Banners module.
  3. Confirm the banner is displayed and its direct click URL works.
  4. Set the category access to Registered.
  5. Visit the page and direct click URL as a guest.
    • The banner must not be displayed.
    • The direct click URL must return 404 and not increment clicks.
  6. Repeat steps 4-5 after unpublishing the category.

Actual result BEFORE applying this Pull Request

Banners are displayed by the Banners module even when their category is unpublished or inaccessible to the current user.

The direct banner click URL also redirects to the destination and increments the click count in both cases.

Expected result AFTER applying this Pull Request

Banners are not displayed when their primary category is unpublished or inaccessible to the current user.

The direct banner click URL returns a 404 response and does not increment the click count.

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar Reda-Muhamed Reda-Muhamed - open - 20 Sep 2026
avatar Reda-Muhamed Reda-Muhamed - change - 20 Sep 2026
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Sep 2026
Category ⇒ Front End com_banners Modules
avatar Reda-Muhamed Reda-Muhamed - change - 20 Sep 2026
The description was changed
avatar Reda-Muhamed Reda-Muhamed - edited - 20 Sep 2026
avatar chmst chmst - test_item - 20 Sep 2026 - Tested successfully
avatar chmst
chmst - comment - 20 Sep 2026

I have tested this item ✅ successfully on 8602c7e


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

avatar chmst
chmst - comment - 20 Sep 2026

I have tested this item ✅ successfully on 8602c7e


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

avatar brianteeman brianteeman - test_item - 25 Sep 2026 - Tested successfully
avatar brianteeman
brianteeman - comment - 25 Sep 2026

I have tested this item ✅ successfully on 8602c7e

tested category usergroup restrictions and state


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

avatar brianteeman
brianteeman - comment - 25 Sep 2026

I have tested this item ✅ successfully on 8602c7e

tested category usergroup restrictions and state


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

avatar richard67 richard67 - change - 25 Sep 2026
Status Pending ⇒ Ready to Commit
Labels Added: PR-5.4-dev
avatar richard67
richard67 - comment - 25 Sep 2026

RTC


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

avatar richard67
richard67 - comment - 25 Sep 2026

RTC


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

avatar muhme muhme - change - 1 Oct 2026
Labels Added: RTC
avatar muhme
muhme - comment - 1 Oct 2026

✅ Final test before merge with JBT using PostgreSQL

  • Created banner-category, banner and banner site module
  • The banner is visible and counting impressions and clicks
  • Before PR seen
    • Setting banner-category access Registered: Banner is still visible and counting impressions and clicks
    • Unpublishing banner-category: Banner is still visible and counting impressions and clicks
  • Applied PR with Patch Tester
    • Published banner-category, access Public: Banner is visible and counting impressions and clicks
      • Same for access Guest
    • Published banner-category, access Registered: Banner is not visible, not counting impressions and click page is 404 and not counting
      • Same for Special and Super Users
    • Unpublished banner-category, access Public: Banner is not visible, not counting impressions and click page is 404 and not counting
      • Same vor Archived And Trash
    • No unusual PHP or Joomla log messages
  • This PR will also resolve #46351
  • Code review is recommended before merge
avatar richard67 richard67 - change - 2 Oct 2026
Status Ready to Commit ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2026-10-02 16:26:26
Closed_By ⇒ richard67
avatar richard67 richard67 - close - 2 Oct 2026
avatar richard67 richard67 - merge - 2 Oct 2026
avatar richard67
richard67 - comment - 2 Oct 2026

Thanks @Reda-Muhamed for this PR, and @chmst , @brianteeman and @muhme for testing.

Add a Comment

Login with GitHub to post a comment