Skip to content

Conversation

@richard67
Copy link
Member

Pull Request for Issue # .

Summary of Changes

Follow up PR to #42955 .

This pull request (PR) updates the list of files to be deleted on update in file administrator/components/com_admin/script.php to recent changes in the 5.1-dev branch in preparation for the upcoming 5.1.0-beta2 release.

In detail deleted files and folder from following PR's are added:

  • From PR [5.1] Convert mod_login to service provider #42990 file
    /administrator/modules/mod_login/mod_login.php
  • From PR [5.1] Revert implementation of AfterInitialiseRouterEvent #43009 files
    /libraries/src/Event/Router/AfterInitialiseRouterEvent.php
    /libraries/src/Event/Router/RouterEvent.php
    folder
    /libraries/src/Event/Router
  • From PR [5.1] TUF Code Cleanup #42900 files
    /libraries/src/Http/HttpFactoryInterface.php
    /libraries/src/Service/Provider/Http.php
  • From PR [5.1] replace web-token/signature-pack with web-token/jwt-library #42954 files
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A128CCM_16_128.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A128CCM_16_64.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A128CCM_64_128.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A128CCM_64_64.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A256CCM_16_128.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A256CCM_16_64.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A256CCM_64_128.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/A256CCM_64_64.php
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption/AESCCM.php
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption/A128CTR.php
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption/A192CTR.php
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption/A256CTR.php
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption/AESCTR.php
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption/Chacha20Poly1305.php
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption/RSAOAEP384.php
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption/RSAOAEP512.php
    /libraries/vendor/web-token/jwt-experimental/LICENSE
    /libraries/vendor/web-token/jwt-experimental/Signature/Blake2b.php
    /libraries/vendor/web-token/jwt-experimental/Signature/ES256K.php
    /libraries/vendor/web-token/jwt-experimental/Signature/HS1.php
    /libraries/vendor/web-token/jwt-experimental/Signature/HS256_64.php
    /libraries/vendor/web-token/jwt-experimental/Signature/RS1.php
    folders
    /libraries/vendor/web-token/jwt-experimental/Signature
    /libraries/vendor/web-token/jwt-experimental/KeyEncryption
    /libraries/vendor/web-token/jwt-experimental/ContentEncryption
    /libraries/vendor/web-token/jwt-experimental

Testing Instructions

Code review.

Or if you want to make a real test, update a 5.1.0-beta1 or older version to the last 5.1 nightly build to get the actual result, and update a 5.1.0-beta1 or older version to to the update package built by Drone for this PR to get the expected result.

Actual result BEFORE applying this Pull Request

The files and folders mentioned above are still present after updating from a 5.1.0-beta1 or older version.

Expected result AFTER applying this Pull Request

The files and folders mentioned above have been deleted after updating from a 5.1.0-beta1 or older version.

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

@bembelimen bembelimen added this to the Joomla! 5.1.0 milestone Mar 18, 2024
@LadySolveig LadySolveig merged commit ab6cb09 into joomla:5.1-dev Mar 18, 2024
@LadySolveig
Copy link
Contributor

Thank you @richard67

@richard67 richard67 deleted the 5.1-dev-deleted-files-and-folders-2024-03-15 branch March 18, 2024 11:32
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