Bug #55392
closed
Missing opcode cache throws warning (instead of notice) and triggers mail
100%
Description
If you run TYPO3 on a system without opcode cache (as in many shared hosting environments, e.g. Mittwald), the install tool registers this and shows a warning.
I'd say while APC is certainly a very good addon and everyone should try to use it with TYPO3 because of performance, sadly not everyone can use it. The system runs fine without. Nothing to really "warn" about.
Also, if you have a cronjob running each night, triggering the scheduler running a "report" task, you will get a mail with (by the way 2) failed tests/warnings each night.
It is only a tiny change neccesary:
File typo3/sysext/install/Classes/SystemEnvironment/Check.php
Line 747:
change from
$status = new WarningStatus();
to
$status = new NoticeStatus();
Updated by Markus Klein almost 11 years ago
- Status changed from New to Accepted
- Target version set to next-patchlevel
- Estimated time deleted (
0.50 h)
Hi!
We intended to change that with #55252.
Currently it is being discussed whether this should be Warning or Notice, but your point is very valid.
Updated by Alexander Opitz almost 11 years ago
- Complexity deleted (
no-brainer)
I would like to discuss this issue here, as #55252 addresses a very different problem.
At first I'd like to know, what is the difference between error and warning, beside the fact that they have different colors in the install and report tool.
So I have 3 solutions in my mind:
1) Only report Error in the mail (to get a difference in handling for error/warning).
2) Add functionality to acknowledge Errors/Warnings, so they don't get mailed every time (and maybe show them in another style in install/report).
3) Change opcode cache check from Warning to Notice, but get more attention on notices in installer.
1) and 3) would be easy but 2) would be perfect also for other errors/warnings.
Any more solutions around?
Updated by Alexander Opitz almost 11 years ago
For example the "PHP Maximum upload filesize too small" Error is something I know on some systems and I do not need any report about that.
Updated by
Updated by Gerrit Code Review almost 11 years ago
- Status changed from Accepted 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/28216
Updated by Gerrit Code Review almost 11 years 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/28216
Updated by Sebastian Müller almost 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 9c35421bff3d0055bcca5b38e8d9cfdfcc13bb25.
Updated by