Skip to content

Conversation

@richard67
Copy link
Member

@richard67 richard67 commented Sep 26, 2023

Pull Request for Issue # .

Summary of Changes

Add deleted files from recent changes to the list of files to be deleted on update in script.php.

I don't see any other PRs in the pipeline in 4.4-dev and 5.0-dev which could cause deleted files and folders, so I think this is ready to go.

Testing Instructions

Code review.

Or update from a current 4.4-dev branch or latest 4.4.0-beta3-dev nightly or a 4.4.0-beta2 to the latest 5.0.0-beta3-dev nightly for the actual result and to the patched update package or custom update URL created by Drone for this PR for the expected result.

Actual result BEFORE applying this Pull Request

Gzipped media files mentioned in the description above have not been deleted with the update.

File "administrator/components/com_admin/script.php" contains a typo "/media/system/css/mootree_rtl,min.css" (with comma), so the "/media/system/css/mootree_rtl.min.css" (with dot) has also not been deleted.

Expected result AFTER applying this Pull Request

Gzipped media files mentioned in the description above have been deleted with the update.

File "/media/system/css/mootree_rtl.min.css" has also been deleted.

File "administrator/components/com_admin/script.php" doesn't contain typo "/media/system/css/mootree_rtl,min.css".

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

@brianteeman
Copy link
Contributor

sorry for the errors

@richard67
Copy link
Member Author

sorry for the errors

@brianteeman No need for sorry. Can happen to anyone.

@brianteeman
Copy link
Contributor

Still good practice to acknowledge it

@richard67
Copy link
Member Author

Still good practice to acknowledge it

Yes, of course good to know you noticed.

@richard67 richard67 changed the title [5.0] [WiP] Update deleted files list in script.php for 5.0.0-beta3 [5.0] Update deleted files list in script.php for 5.0.0-beta3 Sep 26, 2023
@richard67 richard67 marked this pull request as ready for review September 26, 2023 12:53
@HLeithner HLeithner merged commit db76dfc into joomla:5.0-dev Sep 26, 2023
@HLeithner
Copy link
Member

thanks

@richard67 richard67 deleted the 5.0-dev-deleted-files-2023-09-26 branch September 26, 2023 13:36
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