Skip to content

Remove the autoload_psr4 before running finalisation.php#38525

Merged
roland-d merged 1 commit intojoomla:4.2-devfrom
nikosdion:fix/38486-part1-autoload-psr4
Aug 18, 2022
Merged

Remove the autoload_psr4 before running finalisation.php#38525
roland-d merged 1 commit intojoomla:4.2-devfrom
nikosdion:fix/38486-part1-autoload-psr4

Conversation

@nikosdion
Copy link
Copy Markdown
Contributor

Replaces #38491

Pull Request for Issue #38486 .

Summary of Changes

Add code in extract.php to remove the administrator/cache/autoload_psr4.php file before loading the finalisation.php as the latter might try to use core code which depends on the former.

Testing Instructions

From Joomla 4.0.0

  • Install a Joomla 4.0.0 site
  • Replace administrator/components/com_joomlaupdate, components/com_joomlaupdate, media/com_joomlaupdate, and administrator/language/en-GB/com_joomlaupdate.* with the files from this PR.
  • Log into the backend
  • Do an md5sum of the administrator/cache/autoload_psr4.php file
  • Upgrade the site to Joomla 4.2 using the package built from this PR
  • Try to access your site's frontend
  • Do an md5sum of the administrator/cache/autoload_psr4.php file

From Joomla 4.1.5

  • Install a Joomla 4.1.5 site
  • Log into the backend
  • Do an md5sum of the administrator/cache/autoload_psr4.php file
  • Upgrade the site to Joomla 4.2 using the package built from this PR
  • Try to access your site's frontend
  • Do an md5sum of the administrator/cache/autoload_psr4.php file

Actual result BEFORE applying this Pull Request

The MD5 sums are identical (the autoload_psr4.php file was not removed on update).

Expected result AFTER applying this Pull Request

The MD5 sums are different (the autoload_psr4.php file was removed on update and replaced when accessing your site again).

Documentation Changes Required

None.

@richard67 richard67 added this to the Joomla! 4.2.1 milestone Aug 18, 2022
@roland-d roland-d merged commit 30ab12a into joomla:4.2-dev Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants