Merge pull request #18395 from owncloud/hide-empty-security-warning

[admin settings] Show success message if security warnings are empty

settings/css/settings.css

@@ -361,6 +361,11 @@ table.grid td.date{
 	list-style: initial;
 	margin: 10px 0;
 }
+#security-warning-state span {
+	padding-left: 25px;
+	background-position: 5px center;
+	margin-left: -5px;
+}
 
 #shareAPI p { padding-bottom: 0.8em; }
 #shareAPI input#shareapiExpireAfterNDays {width: 25px;}

settings/js/admin.js

@@ -179,6 +179,10 @@ $(document).ready(function(){
 		var $el = $('#postsetupchecks');
 		$el.find('.loading').addClass('hidden');
 		if (messages.length === 0) {
+			var securityWarning = $('#security-warning');
+			if (securityWarning.children('ul').children().length === 0) {
+				$('#security-warning-state').find('span').removeClass('hidden');
+			}
 		} else {
 			var $errorsEl = $el.find('.errors');
 			var $warningsEl = $el.find('.warnings');

settings/templates/admin.php

@@ -177,6 +177,9 @@ if ($_['cronErrors']) {
 		<?php print_unescaped($l->t('Please double check the <a target="_blank" href="%s">installation guides ↗</a>, and check for any errors or warnings in the <a href="#log-section">log</a>.', link_to_docs('admin-install'))); ?>
 	</p>
 </div>
+<div id="security-warning-state">
+	<span class="hidden icon-checkmark"><?php p($l->t('All checks passed.'));?></span>
+</div>
 </div>
 
 	<div class="section" id="shareAPI">