Skip to content

[4.2] Inject the database into the installer#37625

Merged
roland-d merged 12 commits intojoomla:4.2-devfrom
Digital-Peak:j4/installer/api-db-inject
Jun 20, 2022
Merged

[4.2] Inject the database into the installer#37625
roland-d merged 12 commits intojoomla:4.2-devfrom
Digital-Peak:j4/installer/api-db-inject

Conversation

@laoneo
Copy link
Member

@laoneo laoneo commented Apr 23, 2022

Summary of Changes

Standardizes the database access in the installer library. This is a preparation to load the installer through a service provider.

Testing Instructions

  • Install Joomla
  • Discover extension
  • Install any kind of extension, for example the free version of DPCalendar which contains a package, component, modules and plugins
  • Uninstall these extensions

Actual result BEFORE applying this Pull Request

All is working as before.

Expected result AFTER applying this Pull Request

All is working as before.

@laoneo laoneo requested review from rdeutz and zero-24 as code owners April 23, 2022 12:49
@laoneo laoneo changed the title Inject the database into the installer [4.2] Inject the database into the installer Apr 23, 2022
@roland-d roland-d merged commit a36f76c into joomla:4.2-dev Jun 20, 2022
@roland-d roland-d deleted the j4/installer/api-db-inject branch June 20, 2022 14:25
@roland-d
Copy link
Contributor

Thank you

@roland-d roland-d added this to the Joomla 4.2.0 milestone Jun 20, 2022
heelc29 added a commit to heelc29/joomla-cms that referenced this pull request Jun 27, 2022
joomla#37625 (Inject the database into the installer)
laoneo pushed a commit that referenced this pull request Jun 28, 2023
* Improve Container DatabaseDriver

* Improve Container DatabaseDriver

* update to DatabaseInterface

* Update SysinfoModel.php

* add more

* add more

* add more

#37625 (Inject the database into the installer)

* psr12 part 1

* Revert "psr12 part 1"

This reverts commit 1f42948.

* Phase 1 convert BRANCH to PSR-12

* Phase 2 convert BRANCH to PSR-12

* no longer needed
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.

3 participants