Task #89794

Refactoring the database check in install tool

Added by Manuel Selbach 2 months ago. Updated 3 days ago.

Status:
Resolved
Priority:
Should have
Assignee:
Manuel Selbach
Category:
Install Tool
Target version:
-
Start date:
2019-11-28
Due date:
% Done:

100%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The current state of the check regarding database requirements is not separated properly.
Thus I suggest to refactor that to add platform and driver specific specification to have this maintainable.

Related issues

Related to TYPO3 Core - Bug #81531: Warning "PHP extension mysqli not loaded" when using PostgreSQL New 2017-06-09
Related to TYPO3 Core - Bug #90223: Wrong type hint in install tool typo3/sysext/install/Classes/SystemEnvironment/DatabaseCheck Under Review 2020-01-27

Associated revisions

Revision ad43bc30 (diff)
Added by Manuel Selbach 3 days ago

[TASK] Separate database requirements checks in Install tool

This change separates database requirement checks regarding the DBMS
platform (e.g. MySQL, PostgreSQL, ...) from drivers as checks for
specific drivers may differ (e.g. pdo_mysql requires other PHP
extensions being loaded than mysqli).

Additionally the encoding for the database is defined in
\TYPO3\CMS\Core\Database\Platform\PlatformInformation
which will be enforced during the install process of the Install tool.

Resolves: #89794
Releases: master
Change-Id: I45ff0c57c3af30af0aff349520a0dbea3937eccd
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472
Tested-by: TYPO3com <>
Tested-by: Susanne Moog <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Susanne Moog <>
Reviewed-by: Andreas Fernandez <>

History

#1 Updated by Manuel Selbach 2 months ago

  • Related to Bug #81531: Warning "PHP extension mysqli not loaded" when using PostgreSQL added

#2 Updated by Gerrit Code Review 2 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#3 Updated by Gerrit Code Review 2 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#4 Updated by Gerrit Code Review about 2 months ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#5 Updated by Gerrit Code Review about 2 months ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#6 Updated by Gerrit Code Review about 2 months ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#7 Updated by Gerrit Code Review 25 days ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#8 Updated by Gerrit Code Review 14 days ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#9 Updated by Gerrit Code Review 13 days ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#10 Updated by Gerrit Code Review 5 days ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#11 Updated by Gerrit Code Review 5 days ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#12 Updated by Gerrit Code Review 5 days ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#13 Updated by Gerrit Code Review 5 days ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#14 Updated by Gerrit Code Review 5 days ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#15 Updated by Gerrit Code Review 5 days ago

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#16 Updated by Gerrit Code Review 5 days ago

Patch set 15 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#17 Updated by Gerrit Code Review 5 days ago

Patch set 16 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#18 Updated by Gerrit Code Review 5 days ago

Patch set 17 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62472

#19 Updated by Manuel Selbach 3 days ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#20 Updated by Manuel Selbach about 8 hours ago

  • Related to Bug #90223: Wrong type hint in install tool typo3/sysext/install/Classes/SystemEnvironment/DatabaseCheck added

Also available in: Atom PDF