Checks if getenv returns proper system variable results

Updated texts and changed the variable name to match the query

Updated text + info which path for php config is used

removed the term ownCloud and squashed the commits

Updated text

diff --git a/settings/admin.php b/settings/admin.php
index 976d0a5c3f1d6620f505f7dc8015b2629576fee6..bc9224f3998fa189427908936f976b1f3dc0baf2 100644 (file)
--- a/settings/admin.php
+++ b/settings/admin.php
@@ -118,6 +118,9 @@ $databaseOverload = (strpos(\OCP\Config::getSystemValue('dbtype'), 'sqlite') !==
 $template->assign('databaseOverload', $databaseOverload);
 $template->assign('cronErrors', $appConfig->getValue('core', 'cronErrors'));
 
+// warn if php is not setup properly to get system variables with getenv
+$template->assign('getenvServerNotWorking', empty(getenv('PATH')));
+
 // warn if Windows is used
 $template->assign('WindowsWarning', OC_Util::runningOnWindows());
 

diff --git a/settings/templates/admin.php b/settings/templates/admin.php
index 4bc497df764ecf132c72bb7f2d820b2f88ca3c52..4b373891a651443679318e468c0fd82e31813473 100644 (file)
--- a/settings/templates/admin.php
+++ b/settings/templates/admin.php
@@ -67,6 +67,16 @@ if ($_['mail_smtpmode'] == 'qmail') {
        <h2><?php p($l->t('Security & setup warnings'));?></h2>
        <ul>
 <?php
+// is php setup properly to query system environment variables like getenv('PATH')
+if ($_['getenvServerNotWorking']) {
+?>
+       <li>
+               <?php p($l->t('php does not seem to be setup properly to query system environment variables. The test with getenv("PATH") only returns an empty response.')); ?><br>
+               <?php p($l->t('Please check the installation documentation for php configuration notes and the php configuration of your server, especially when using php-fpm.')); ?>
+       </li>
+<?php
+}
+
 // is read only config enabled
 if ($_['readOnlyConfigEnabled']) {
 ?>