RTC bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar SniperSister
SniperSister
25 Aug 2025

Pull Request for Issue #45977

Summary of Changes

The legacy update XML format used a XML attribute per infourl, wheras the JSON-based TUF is using two separate keys. That leads to a different data structure, that had to be converted - this PR adds the conversion logic.

Testing Instructions

  • Install Joomla_5.4.0-beta2-dev+pr.45696-Development-Full_Package.zip
  • Disable Automated Updates and delete Installation folder
  • Installed additional plg_system_autoupdatepreventer and enabled it
  • 'Log Almost Everything' and 'Debug System' enabled
  • Installed 'System - Alpha Update Server' 0.6.1 plugin
  • Check with manual update 'Joomla Update to Joomla! 5.4.113 is available (twice needed)
    • Fixed Database structure
  • Enable Automated Update

Actual result BEFORE applying this Pull Request

PHP Warning

Expected result AFTER applying this Pull Request

No warning

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 - 25 Aug 2025
avatar SniperSister SniperSister - change - 25 Aug 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Aug 2025
Category Libraries
avatar richard67 richard67 - change - 25 Aug 2025
Labels Added: PR-5.4-dev
avatar exlemor
exlemor - comment - 26 Aug 2025

@SniperSister I am trying to test this PR but your 1st step says:

Download /install: Joomla_5.4.0-beta2-dev+pr.45696-Development-Full_Package.zip, except that PR:
#45696 is Merged so it doesn't seem to allow/have the download page ;(

I found this https://artifacts.joomla.org/drone/joomla/joomla-cms/5.4-dev/45696/downloads/87417/
(which seems to be the most recent (but update version) but I would prefer a confirmation from you)

For others looking for the 2 plugins:
Plugin: plg_system_autoupdatepreventer
Plugin: 'System - Alpha Update Server' 0.6.1 plugin

What PHP warning is expected?
I see some BEFORE and AFTER...

avatar muhme
muhme - comment - 26 Aug 2025

What PHP warning is expected?

[25-Aug-2025 08:50:17 UTC] PHP Warning: Array to string conversion in /var/www/joomla-test.heikol.de/public_html/libraries/src/HTML/HTMLHelper.php on line 327

See the fixed issue and maybe from there the tested PR.

avatar muhme muhme - test_item - 26 Aug 2025 - Tested unsuccessfully
avatar muhme
muhme - comment - 26 Aug 2025

I have tested this item πŸ”΄ unsuccessfully on 4fdaf73

* Veryfied with new test the PHP error is still logged on enabling and disabling Automated Updates

  • New installation with Joomla_5.4.0-beta2-dev+pr.45980-Development-Full_Package.zip
    • Tested enabling and disabling Automated Updates
    • βœ… The PHP warning 'Array to string conversion' no longer occurs
    • ❌ Two new PHP warnings are logged for each enabling / disabling Automated Updates:
[26-Aug-2025 04:20:07 UTC] PHP Warning:  Attempt to read property "url" on array in /var/www/joomla-test.heikol.de/public_html/libraries/src/Updater/Update.php on line 623
[26-Aug-2025 04:20:07 UTC] PHP Warning:  Attempt to read property "title" on array in /var/www/joomla-test.heikol.de/public_html/libraries/src/Updater/Update.php on line 6241G1G


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45980.
avatar SniperSister SniperSister - change - 26 Aug 2025
Labels Added: bug
avatar SniperSister
SniperSister - comment - 26 Aug 2025

Fixed @muhme - sorry, my bad, did not commit the final version

avatar muhme muhme - test_item - 26 Aug 2025 - Tested successfully
avatar muhme
muhme - comment - 26 Aug 2025

I have tested this item βœ… successfully on 58742e1

* New installation with fresh Joomla_5.4.0-beta2-dev+pr.45980-Development-Full_Package.zip

  • Tested enabling and disabling Automated Updates
    • βœ… No PHP warnings
    • βœ… Email still sent 'Your site could not be updated from 5.4.0-beta2-dev+pr.45980 to 5.4.113.'
  • Disabled plg_system_autoupdatepreventer and enabled Automated Updates
    • βœ… Automated Update to 5.4.113, including email
    • βœ… No PHP warnings
    • βœ… Checked administrator/logs/* useful update log messages, no new erros (only known email ERRORS, API Exception "Update stopped by plugin.")

Thank you @SniperSister


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

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

I have tested this item βœ… successfully on 58742e1

I have successfully test this PR. Thanks @muhme, got the same results as you - I feel better knowing that I had the same and thought I was doing something incorrect but wasn't for once ;) lol.

Thanks @SniperSister for the PR and fixes :)


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

avatar ChristineWk
ChristineWk - comment - 26 Aug 2025

Unfortunately, I'm too late, as I just noticed.
I've been on part of the instructions earlier...
@SniperSister informed you via email. Thank you.


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

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

RTC


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

avatar muhme muhme - change - 26 Aug 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-08-26 12:07:04
Closed_By muhme
Labels Added: RTC
avatar muhme muhme - close - 26 Aug 2025
avatar muhme muhme - merge - 26 Aug 2025
avatar muhme
muhme - comment - 26 Aug 2025

Thank you @SniperSister for your contribution. Thank you @richard67 for your support. Thank you @exlemor for testing.

Add a Comment

Login with GitHub to post a comment