Skip to content

[5.0] Update composer dependency maximebf/debugbar to v1.19.0#41932

Merged
HLeithner merged 1 commit intojoomla:5.0-devfrom
richard67:5.0-dev-update-maximebf-debugbar
Sep 26, 2023
Merged

[5.0] Update composer dependency maximebf/debugbar to v1.19.0#41932
HLeithner merged 1 commit intojoomla:5.0-devfrom
richard67:5.0-dev-update-maximebf-debugbar

Conversation

@richard67
Copy link
Copy Markdown
Member

@richard67 richard67 commented Sep 26, 2023

Pull Request for Issue # .

Summary of Changes

In the 4.4-dev branch we currently have in composer.json "maximebf/debugbar": "dev-master". This resolves in commit "30f65f18f7ac086255a77a079f8e0dcdd35e828e" in the lock file. This commit belongs to their version "1.19.0".

In the 5.0-dev branch we have in composer.json "maximebf/debugbar": "^1.18.2", which results in version "1.18.2" being used, which is the version before the "1.19.0", see php-debugbar/php-debugbar@v1.18.2...v1.19.0 .

That means that when we currently update the CMS from 4.4 to 5, we downgrade the "maximebf/debugbar" dependency.

This PR here fixes that by updating the maximebf/debugbar dependency to version "1.19.0" here in the 5.0-dev branch, together with a PR for 4.4-dev to change there from "dev-master" to "^1.19.0", too.

The PR here might become obsolete when the other one has been merged into 4.40-dev and that has been upmerged to 5.0-dev then. But it can be used to check the result of that.

Testing Instructions

Check that the debug bar works.

Actual result BEFORE applying this Pull Request

Works.

Expected result AFTER applying this Pull Request

Still works.

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

@HLeithner
Copy link
Copy Markdown
Member

thanks

@richard67 richard67 deleted the 5.0-dev-update-maximebf-debugbar branch September 26, 2023 09:59
HLeithner pushed a commit to HLeithner/joomla-cms that referenced this pull request Sep 26, 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.

3 participants