Skip to content
Snippets Groups Projects
Commit 009f307b authored by Christian Kuhn's avatar Christian Kuhn
Browse files

[FEATURE] Central system environment check

This patch adds a new structure to the install tool to centralize
all system and especially PHP related system checks cluttered in
various modules and parts of the core at the moment.

The idea is to have a basic, standalone script to test for all
requirements TYPO3 CMS has in order to run smoothly. The script does
not use any TYPO3 core code and has no further dependencies. With
a later patch, the check will be integrated as the very first
step during install, to give a good indicator if the final
installed system will work well or if any system constraints are
not met.

Currently, the script is included as a new section in the install
tool, called "System environment", where all checks are performed
and each gives a status back. This will already help to support
questions on mailing lists and should be the first step every user
is guided to: "Do you have warnings or errors in the system
environment?". The install tool implementation should also help
already if moving an instance to a different server.

Furthermore, there is a simple integration in the reports module
that points to the install tool if issues are found.

The class structure is as simple as possible, it is very easy to
adapt single checks later on, the API is a single getStatus() method
that returns all check object of the specific tests. Due to the
nature of the script and the later usage in the installation
process, things like localization and so on are not done and
should also not be implemented later on.

Most current check code is fetched from existing install tool,
reports module and bootstrap code. This also make the main
Installer.php class of the install tool already a bit better
maintainable since 600 lines of code are removed from it.

Change-Id: I64387ca9aaa4b429ee4a909cdab4386180af0d45
Resolves: #46219
Releases: 6.1
Reviewed-on: https://review.typo3.org/18870
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
parent 338cf5b6
Branches
Tags
No related merge requests found
Showing
with 1713 additions and 765 deletions
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment