Always log cron execution

There was a setting to disable the last execution of cron. There is no known
problem with this write access and it was also questioned when this feature
was build in https://github.com/owncloud/core/pull/7689#issuecomment-38264707

Recently there was also a bug report about a non-visible last cron execution
(#6088) - let's better remove this.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

5 files changed, 22 insertions, 40 deletions

diff --git a/config/config.sample.php b/config/config.sample.php
index 71d1aa3a02c..9f6ef668c5e 100644
--- a/config/config.sample.php
+++ b/config/config.sample.php
@@ -739,13 +739,6 @@ $CONFIG = array(
 'log_query' => false,
 
 /**
- * Log successful cron runs.
- *
- * Defaults to ``true``
- */
-'cron_log' => true,
-
-/**
  * Enables log rotation and limits the total size of logfiles. The default is 0,
  * or no rotation. Specify a size in bytes, for example 104857600 (100 megabytes
  * = 100 * 1024 * 1024 bytes). A new logfile is created with a new name when the
diff --git a/cron.php b/cron.php
index 74935db0da9..b7443ef3ce2 100644
--- a/cron.php
+++ b/cron.php
@@ -152,9 +152,7 @@ try {
 	}
 
 	// Log the successful cron execution
-	if (\OC::$server->getConfig()->getSystemValue('cron_log', true)) {
-		\OC::$server->getConfig()->setAppValue('core', 'lastcron', time());
-	}
+	\OC::$server->getConfig()->setAppValue('core', 'lastcron', time());
 	exit();
 
 } catch (Exception $ex) {
diff --git a/lib/private/Settings/Admin/Server.php b/lib/private/Settings/Admin/Server.php
index 994d927aff0..f9512928e5e 100644
--- a/lib/private/Settings/Admin/Server.php
+++ b/lib/private/Settings/Admin/Server.php
@@ -130,7 +130,6 @@ class Server implements ISettings {
 
 			// Background jobs
 			'backgroundjobs_mode' => $this->config->getAppValue('core', 'backgroundjobs_mode', 'ajax'),
-			'cron_log'            => $this->config->getSystemValue('cron_log', true),
 			'lastcron'            => $this->config->getAppValue('core', 'lastcron', false),
 			'cronErrors'		  => $this->config->getAppValue('core', 'cronErrors'),
 			'cli_based_cron_possible' => function_exists('posix_getpwuid'),
diff --git a/settings/templates/settings/admin/server.php b/settings/templates/settings/admin/server.php
index 407badcff41..ab30f8e7700 100644
--- a/settings/templates/settings/admin/server.php
+++ b/settings/templates/settings/admin/server.php
@@ -156,28 +156,26 @@
 
 <div class="section" id="backgroundjobs">
 	<h2 class="inlineblock"><?php p($l->t('Background jobs'));?></h2>
-	<?php if ($_['cron_log']): ?>
-		<p class="cronlog inlineblock">
-			<?php if ($_['lastcron'] !== false):
-				$relative_time = relative_modified_date($_['lastcron']);
-				$absolute_time = OC_Util::formatDate($_['lastcron']);
-				if (time() - $_['lastcron'] <= 3600): ?>
-					<span class="status success"></span>
-					<span class="crondate" title="<?php p($absolute_time);?>">
-					<?php p($l->t("Last job ran %s.", [$relative_time]));?>
-				</span>
-				<?php else: ?>
-					<span class="status error"></span>
-					<span class="crondate" title="<?php p($absolute_time);?>">
-					<?php p($l->t("Last job execution ran %s. Something seems wrong.", [$relative_time]));?>
-				</span>
-				<?php endif;
-			else: ?>
+	<p class="cronlog inlineblock">
+		<?php if ($_['lastcron'] !== false):
+			$relative_time = relative_modified_date($_['lastcron']);
+			$absolute_time = OC_Util::formatDate($_['lastcron']);
+			if (time() - $_['lastcron'] <= 3600): ?>
+				<span class="status success"></span>
+				<span class="crondate" title="<?php p($absolute_time);?>">
+				<?php p($l->t("Last job ran %s.", [$relative_time]));?>
+			</span>
+			<?php else: ?>
 				<span class="status error"></span>
-				<?php p($l->t("Background job didn’t run yet!"));
-			endif; ?>
-		</p>
-	<?php endif; ?>
+				<span class="crondate" title="<?php p($absolute_time);?>">
+				<?php p($l->t("Last job execution ran %s. Something seems wrong.", [$relative_time]));?>
+			</span>
+			<?php endif;
+		else: ?>
+			<span class="status error"></span>
+			<?php p($l->t("Background job didn’t run yet!"));
+		endif; ?>
+	</p>
 	<a target="_blank" rel="noreferrer" class="icon-info"
 	   title="<?php p($l->t('Open documentation'));?>"
 	   href="<?php p(link_to_docs('admin-background-jobs')); ?>"></a>
diff --git a/tests/lib/Settings/Admin/ServerTest.php b/tests/lib/Settings/Admin/ServerTest.php
index a71aef0178e..106390f25a5 100644
--- a/tests/lib/Settings/Admin/ServerTest.php
+++ b/tests/lib/Settings/Admin/ServerTest.php
@@ -80,12 +80,12 @@ class ServerTest extends TestCase {
 			->with('core', 'backgroundjobs_mode', 'ajax')
 			->willReturn('ajax');
 		$this->config
-			->expects($this->at(4))
+			->expects($this->at(3))
 			->method('getAppValue')
 			->with('core', 'lastcron', false)
 			->willReturn(false);
 		$this->config
-			->expects($this->at(5))
+			->expects($this->at(4))
 			->method('getAppValue')
 			->with('core', 'cronErrors')
 			->willReturn('');
@@ -94,11 +94,6 @@ class ServerTest extends TestCase {
 			->method('getSystemValue')
 			->with('check_for_working_wellknown_setup', true)
 			->willReturn(true);
-		$this->config
-			->expects($this->at(3))
-			->method('getSystemValue')
-			->with('cron_log', true)
-			->willReturn(true);
 		$this->l10n
 			->expects($this->at(0))
 			->method('t')
@@ -139,7 +134,6 @@ class ServerTest extends TestCase {
 
 				// Background jobs
 				'backgroundjobs_mode' => 'ajax',
-				'cron_log'            => true,
 				'lastcron'            => false,
 				'cronErrors'		  => '',
 				'cli_based_cron_possible' => true,