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

[6.0] Remove deprecated BaseApplication and CLI classes #42884

Merged
merged 2 commits into from
Apr 4, 2024

Conversation

Hackwar
Copy link
Member

@Hackwar Hackwar commented Feb 25, 2024

Summary of Changes

This removes the deprecated classes from the /libraries/src/Application folder. This code is not used anywhere in the CMS anymore.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org: CMS BaseApplication removed Manual#247

  • No documentation changes for manual.joomla.org needed

@laoneo
Copy link
Member

laoneo commented Mar 1, 2024

Can you make a manual entry and fix the conflict?

@laoneo laoneo added the b/c break This item changes the behavior in an incompatible why. HEADS UP label Mar 5, 2024
@laoneo laoneo added this to the Joomla! 6.0.0 milestone Mar 5, 2024
….0-application

# Conflicts:
#	libraries/src/Application/BaseApplication.php
#	libraries/src/Application/CliApplication.php
@Hackwar
Copy link
Member Author

Hackwar commented Apr 3, 2024

done.

@laoneo laoneo merged commit 3b2e07b into joomla:6.0-dev Apr 4, 2024
3 checks passed
@laoneo
Copy link
Member

laoneo commented Apr 4, 2024

Thanks!

richard67 added a commit to richard67/joomla-cms that referenced this pull request Jun 1, 2024
- Add 5.x update SQL scripts
- Add files abd folder from PR joomla#42884
- Add files from PR joomla#42890
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
b/c break This item changes the behavior in an incompatible why. HEADS UP PR-6.0-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants