From f327ab8bbb1091dd5fe3eeb7d1de9eb9fa69b342 Mon Sep 17 00:00:00 2001
From: Jannik Schönartz
Date: Fri, 17 Mar 2017 16:32:38 +0100
Subject: SetCredentials error message is no longer ignored.

---
 modules-available/locationinfo/page.inc.php | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'modules-available/locationinfo/page.inc.php')

diff --git a/modules-available/locationinfo/page.inc.php b/modules-available/locationinfo/page.inc.php
index ccf37d0e..dec022b2 100644
--- a/modules-available/locationinfo/page.inc.php
+++ b/modules-available/locationinfo/page.inc.php
@@ -259,10 +259,13 @@ class Page_LocationInfo extends Page
 			$dbresult = Database::queryFirst("SELECT * FROM `setting_location_info` WHERE serverid = :serverid", array('serverid' => $serverid));
 
 			$serverInstance = CourseBackend::getInstance($dbresult['servertype']);
-			$serverInstance->setCredentials(json_decode($dbresult['credentials'], true), $dbresult['serverurl'], $serverid);
-			$setCred = $serverInstance->checkConnection();
+			$setCredentials = $serverInstance->setCredentials(json_decode($dbresult['credentials'], true), $dbresult['serverurl'], $serverid);
 
-			if (!$setCred) {
+			if ($setCredentials) {
+				$setCred = $serverInstance->checkConnection();
+			}
+
+			if (!$setCredentials || !$setCred) {
 				$error['timestamp'] = time();
 				$error['error'] = $serverInstance->getError();
 				Database::exec("UPDATE `setting_location_info` Set error=:error WHERE serverid=:id", array('id' => $serverid, 'error' => json_encode($error, true)));
@@ -491,7 +494,7 @@ class Page_LocationInfo extends Page
 					$credential['type'] = "array";
 					$credential['array'] = $selection;
 				}
-				
+
 				$backend['credentials'][] = $credential;
 			}
 			$serverBackends[] = $backend;
-- 
cgit v1.2.3-55-g7522