RTC PBF bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar SniperSister
SniperSister
22 Aug 2025

Pull Request for Issue #45954 #45953

Summary of Changes

This PR fixes an edge case scenario:
So far, the code that is handling potential TUF updates was checking if the currently processed update information would offer an update that's newer than the previously processed update. However, for the very first offered updated, there was no compare target and therefore it was always considered as potential candidate.

That caused issues with the "special update" to 5.1.2 that's currently included in the 6.x branch in the TUF repo to handle a bug in the early 5.1.0 and 5.1.1 releases.

Testing Instructions

  • Install Joomla 5.4.0-beta1
  • Switch to the "Next" update channel and select beta as stability
  • Fetch update information
  • Apply patch
  • Repeat

Actual result BEFORE applying this Pull Request

  • Joomla 6.0.0 beta 1 discovered, but minimum PHP version of 8.1 shown as requirement - because that's the min version of the 5.1.2 release:
Bildschirmfoto 2025-08-23 um 13 29 11

Expected result AFTER applying this Pull Request

  • Min PHP version of 8.3 shown

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar SniperSister SniperSister - open - 22 Aug 2025
avatar SniperSister SniperSister - change - 22 Aug 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2025
Category Libraries
avatar exlemor exlemor - test_item - 23 Aug 2025 - Tested unsuccessfully
avatar exlemor
exlemor - comment - 23 Aug 2025

I have tested this item πŸ”΄ unsuccessfully on 77c1e5c

I have tested this from 2 installations and were unsuccessful in both (Rochen test site and my own hosting).

Sorry ;( @SniperSister) followed your instructions and had another PBF tester watching.


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

avatar cybersalt cybersalt - test_item - 23 Aug 2025 - Tested unsuccessfully
avatar cybersalt
cybersalt - comment - 23 Aug 2025

I have tested this item πŸ”΄ unsuccessfully on 77c1e5c

PHP 8.1.0 still showing after that patch was applied.


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

avatar crimle crimle - test_item - 23 Aug 2025 - Tested unsuccessfully
avatar crimle
crimle - comment - 23 Aug 2025

I have tested this item πŸ”΄ unsuccessfully on 77c1e5c

Still min PHP version of 8.1 shown as requirement


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

avatar richard67
richard67 - comment - 23 Aug 2025

You all know that you have to search again for updates after having applied the patch? The version will not change by only applying the patch.

avatar crimle
crimle - comment - 23 Aug 2025

You all know that you have to search again for updates after having applied the patch? The version will not change by only applying the patch.

Yes, that's what I did, but PHP 8.1 is still being displayed as minimum requirement.

avatar brianteeman brianteeman - test_item - 23 Aug 2025 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 23 Aug 2025

I have tested this item πŸ”΄ unsuccessfully on 77c1e5c


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

avatar SniperSister SniperSister - change - 23 Aug 2025
Labels Added: PBF bug PR-5.4-dev
avatar SniperSister SniperSister - change - 23 Aug 2025
The description was changed
avatar SniperSister SniperSister - edited - 23 Aug 2025
avatar SniperSister
SniperSister - comment - 23 Aug 2025

@brianteeman @exlemor @cybersalt @crimle thanks for testing, I was able to find the root cause if the issue and fix it - please have another look

avatar exlemor exlemor - test_item - 23 Aug 2025 - Tested successfully
avatar exlemor
exlemor - comment - 23 Aug 2025

I have tested this item βœ… successfully on f88b78e

Hi @SniperSister, yes indeed you found the issue ;) I have tested this successfully! :) Thanks.


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

avatar brianteeman brianteeman - test_item - 23 Aug 2025 - Tested successfully
avatar brianteeman
brianteeman - comment - 23 Aug 2025

I have tested this item βœ… successfully on f88b78e


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

avatar richard67 richard67 - change - 23 Aug 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 23 Aug 2025

RTC


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

avatar muhme muhme - change - 23 Aug 2025
Labels Added: RTC
avatar muhme
muhme - comment - 23 Aug 2025

βœ… Final test before merge, with JBT graft and the needed MariaDB 10.6 update (which was not required before this PR) πŸ‘

avatar muhme muhme - change - 23 Aug 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-08-23 13:32:01
Closed_By muhme
avatar muhme muhme - close - 23 Aug 2025
avatar muhme muhme - merge - 23 Aug 2025
avatar muhme
muhme - comment - 23 Aug 2025

Thank you @SniperSister for your contribution. Thank you @exlemor, @cybersalt, @crimle and @brianteeman for testing.

Add a Comment

Login with GitHub to post a comment