[Pkg-owncloud-commits] [owncloud] 43/69: Properly detect setup check messages set in the HTML template

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch stable8
in repository owncloud.

commit db3209a4d953d6b4a645975a29e67229fc4c94fe
Author: Robin McCorkell <rmccorkell at owncloud.com>
Date:   Sun Nov 8 00:02:59 2015 +0000

    Properly detect setup check messages set in the HTML template
---
 settings/js/admin.js | 64 +++++++++++++++++++++++++++++-----------------------
 1 file changed, 36 insertions(+), 28 deletions(-)

diff --git a/settings/js/admin.js b/settings/js/admin.js
index 9f7133c..eb3b756 100644
--- a/settings/js/admin.js
+++ b/settings/js/admin.js
@@ -174,38 +174,46 @@ $(document).ready(function(){
 		var messages = [].concat(check1, check2, check3);
 		var $el = $('#postsetupchecks');
 		$el.find('.loading').addClass('hidden');
-		if (messages.length === 0) {
+
+		var hasMessages = false;
+		var $errorsEl = $el.find('.errors');
+		var $warningsEl = $el.find('.warnings');
+		var $infoEl = $el.find('.info');
+
+		for (var i = 0; i < messages.length; i++ ) {
+			switch(messages[i].type) {
+				case OC.SetupChecks.MESSAGE_TYPE_INFO:
+					$infoEl.append('<li>' + messages[i].msg + '</li>');
+					break;
+				case OC.SetupChecks.MESSAGE_TYPE_WARNING:
+					$warningsEl.append('<li>' + messages[i].msg + '</li>');
+					break;
+				case OC.SetupChecks.MESSAGE_TYPE_ERROR:
+				default:
+					$errorsEl.append('<li>' + messages[i].msg + '</li>');
+			}
+		}
+
+		if ($errorsEl.find('li').length > 0) {
+			$errorsEl.removeClass('hidden');
+			hasMessages = true;
+		}
+		if ($warningsEl.find('li').length > 0) {
+			$warningsEl.removeClass('hidden');
+			hasMessages = true;
+		}
+		if ($infoEl.find('li').length > 0) {
+			$infoEl.removeClass('hidden');
+			hasMessages = true;
+		}
+
+		if (hasMessages) {
+			$el.find('.hint').removeClass('hidden');
+		} else {
 			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');
-			var $infoEl = $el.find('.info');
-			for (var i = 0; i < messages.length; i++ ) {
-				switch(messages[i].type) {
-					case OC.SetupChecks.MESSAGE_TYPE_INFO:
-						$infoEl.append('<li>' + messages[i].msg + '</li>');
-						break;
-					case OC.SetupChecks.MESSAGE_TYPE_WARNING:
-						$warningsEl.append('<li>' + messages[i].msg + '</li>');
-						break;
-					case OC.SetupChecks.MESSAGE_TYPE_ERROR:
-					default:
-						$errorsEl.append('<li>' + messages[i].msg + '</li>');
-				}
-			}
-			if ($errorsEl.find('li').length > 0) {
-				$errorsEl.removeClass('hidden');
-			}
-			if ($warningsEl.find('li').length > 0) {
-				$warningsEl.removeClass('hidden');
-			}
-			if ($infoEl.find('li').length > 0) {
-				$infoEl.removeClass('hidden');
-			}
-			$el.find('.hint').removeClass('hidden');
 		}
 	});
 });

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/owncloud.git