diff options
| author | Lukas Reschke <lukas@owncloud.com> | 2016-02-12 13:11:39 +0300 |
|---|---|---|
| committer | Lukas Reschke <lukas@owncloud.com> | 2016-02-12 13:11:39 +0300 |
| commit | 4b90429178f8a72e330356ffb743b02c1c1904a1 (patch) | |
| tree | 812b129f4b6305bd9189c769cb94451c0503017e /settings | |
| parent | f8607ac1327f3254f6f4fe86c8418f1b3d447730 (diff) | |
Add note if integrity check is disabled
Our issue template states that users should post the output of `/index.php/settings/integrity/failed`, at the moment it displays that all passes have been passed if the integrity checker has been disabled.
This is however a wrong approach considering that some distributions are gonna package Frankenstein releases and makes it harder for us to detect such issues. Thus if the integrity code checker is disabled (using the config switch) it displays now: `Appcode checker has been disabled. Integrity cannot be verified.`
This is not displayed anywhere else in the UI except these URL used for us for debugging purposes.
Diffstat (limited to 'settings')
| -rw-r--r-- | settings/controller/checksetupcontroller.php | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/settings/controller/checksetupcontroller.php b/settings/controller/checksetupcontroller.php index 26194bb1180..cfdfa5021bc 100644 --- a/settings/controller/checksetupcontroller.php +++ b/settings/controller/checksetupcontroller.php @@ -271,6 +271,10 @@ class CheckSetupController extends Controller { * @return DataResponse */ public function getFailedIntegrityCheckFiles() { + if(!$this->checker->isCodeCheckEnforced()) { + return new DataDisplayResponse('Integrity checker has been disabled. Integrity cannot be verified.'); + } + $completeResults = $this->checker->getResults(); if(!empty($completeResults)) { |