Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.0] Remove child template conversion of core templates from script.php #40253

Conversation

richard67
Copy link
Member

@richard67 richard67 commented Mar 30, 2023

Pull Request for Issue # .

Summary of Changes

This pull request (PR) removes the conversion of the 4.x core templates "Atum" and "Cassiopeia" from old style templates to child templates from script.php which once had been added with PR's #35874 and #36585 .

The update SQL scripts have already been removed with PR #40083 . This PR here removes the rest.

The PR is one in a sequence of PR's for removing obsolete actions from script.php which are not relevant anymore because updates to 5.x will have to be done from a 4.4 version, like it is now with 4.x from 3.10.

Testing Instructions

Code review: Make sure that the removed methods are not called anywhere else and that they do something which is definitely not needed when updating from 4.4 to 5.

Optional real test: Make sure that updating the latest 4.4-dev nightly build to the update package created by drone works when using the upload & update method.

Actual result BEFORE applying this Pull Request

Obsolete code for migrating the core templates "Atum" and "Cassiopeia" to child templates when updating from 4.0.x to 4.1.0 or a later 4.x is present in script.php.

Expected result AFTER applying this Pull Request

Obsolete code for migrating the core templates "Atum" and "Cassiopeia" to child templates when updating from 4.0.x to 4.1.0 or a later 4.x is not present anymore in script.php.

Updating from a 4.4-dev nightly build works.

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

@HLeithner HLeithner merged commit bb91526 into joomla:5.0-dev Mar 30, 2023
@HLeithner
Copy link
Member

thanks

@richard67 richard67 deleted the 5.0-dev-script-php-clean-up-child-templates branch March 30, 2023 19:06
@richard67 richard67 added this to the Joomla! 5.0 milestone Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants