Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#44490] - [5.3] Replace xml_set_object with proper callable parameters

Feature PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
20 Nov 2024

Pull Request for Issue # .

Summary of Changes

Replace xml_set_object call by calling set_xml_*_handler with callable object.

Testing Instructions

Test extension installation
Test joomla update

Actual result BEFORE applying this Pull Request

Works (php 8.4 with deprecation warinings)

Expected result AFTER applying this Pull Request

Works (php 8.4 without deprecation warnings)

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 HLeithner HLeithner - open - 20 Nov 2024
avatar HLeithner HLeithner - change - 20 Nov 2024
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Nov 2024
Category ⇒ Libraries
avatar richard67 richard67 - change - 20 Nov 2024
Labels Added: PR-5.3-dev
avatar richard67
richard67 - comment - 20 Nov 2024

It sems that the system tests need to be adapted to this PR, or there is something wrong because they are currently failing: https://ci.joomla.org/joomla/joomla-cms/80456/1/22

Running:  administrator/components/com_joomlaupdate/Update.cy.js                       (19 of 129)

  Test the update retrieval logic
    1) "after each" hook for "Can fetch available updates with valid metadata"

  0 passing (2s)
  1 failing

  1) Test the update retrieval logic
       "after each" hook for "Can fetch available updates with valid metadata":
     Error: Unwanted PHP Warning: "  xml_parse(): Unable to call handler Joomla\\CMS\\Updater\\Adapter\\ExtensionAdapter::startElement() in <b>/tests/www/cmysql/libraries/src/Updater/Adapter/ExtensionAdapter.php</b> on line <b>286</b>"

Because this error occurred during a `after each` hook we are skipping all of the remaining tests.
  at supportCommands/ (webpack://joomla/./node_modules/joomla-cypress/src/support.js:110:0)
...
avatar HLeithner
HLeithner - comment - 20 Nov 2024

Copy paste issue...

avatar laoneo
laoneo - comment - 22 Dec 2024

Why still in draft? Looks ok for me.

avatar HLeithner
HLeithner - comment - 22 Dec 2024

because of the failing system test

avatar HLeithner
HLeithner - comment - 4 Mar 2025

This pull request has been automatically rebased to 6.0-dev.

avatar HLeithner HLeithner - change - 4 Mar 2025
Title
…
[5.3] Replace xml_set_object with proper callable parameters
[6.0] Replace xml_set_object with proper callable parameters
avatar HLeithner HLeithner - edited - 4 Mar 2025
avatar HLeithner HLeithner - change - 5 Mar 2025
Title
…
[6.0] Replace xml_set_object with proper callable parameters
[5.3] Replace xml_set_object with proper callable parameters
avatar HLeithner HLeithner - edited - 5 Mar 2025
avatar HLeithner HLeithner - change - 5 Mar 2025
Labels Added: Feature PR-6.0-dev
avatar laoneo laoneo - change - 7 Mar 2025
Labels Removed: PR-6.0-dev
avatar laoneo laoneo - test_item - 13 Mar 2025 - Tested successfully
avatar laoneo
laoneo - comment - 13 Mar 2025

I have tested this item ✅ successfully on 5d1db42

Installed the DPCalendar package and a module. All worked as expected. Additionally the system tests are still running and now also on PHP 8.4 as you can see in #45125.

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

avatar bembelimen bembelimen - close - 14 Mar 2025
avatar bembelimen bembelimen - merge - 14 Mar 2025
avatar bembelimen bembelimen - change - 14 Mar 2025
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2025-03-14 10:50:18
Closed_By ⇒ bembelimen
avatar bembelimen
bembelimen - comment - 14 Mar 2025

Thx

Add a Comment

Login with GitHub to post a comment