Task #49357

Task #49162: Rewrite install tool

Remove Environment Check checkMailCapabilities

Added by Anja Leichsenring about 2 years ago. Updated about 2 years ago.

Status:Resolved Start date:2013-06-23
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:Code Cleanup Spent time: -
Target version:-
TYPO3 Version:6.2 Complexity:
PHP Version: Sprint Focus:

Description

This check takes the php.ini settings into account, no matter what settings exist in LocalConfiguration.php.
During the installation the Environment Check is executed, before LocalConfiguration.php even is created.
So this test can provide misleading information, reporting broken mail functionality, even when it works just fine with the settings given in LocalConfiguration.php.
So we decided to remove this test in Environment Check entirely and enhance the documentation on the real mail test in Test Setup with some hints, what can be wrong when no mail is received.
In short order the autoconfiguration tool will be introduced. This will reintegrate the test with some sane default configuration, not relying on the php.ini settings.


Related issues

duplicated by Core - Bug #47774: Installtool System environment -> false negative on mail Resolved 2013-05-01

Associated revisions

Revision 681add8e
Added by Anja Leichsenring about 2 years ago

[TASK] Remove Environment Check checkMailCapabilities

The check is removed due to providing sometimes inaccurate error status
based on php.ini settings.
For now the real mail test in Test Setup got some hints in case of
missing test email. Later on the auto configuration tool (that not yet
exists) will take over from here.

Resolves: #49357
Releases: 6.2, 6.1
Change-Id: I290cca3fa4653b3a5310b9f27c2b125d9bfeb460
Reviewed-on: https://review.typo3.org/21607
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe

History

#1 Updated by Gerrit Code Review about 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21607

#2 Updated by Gerrit Code Review about 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21607

#3 Updated by Anja Leichsenring about 2 years ago

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

Also available in: Atom PDF