Set contact colours on uids (#152)

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

controller/smscontroller.php

@@ -92,6 +92,7 @@ class SmsController extends Controller {
 		$contactsSrc = $this->contactCache->getContacts();
 		$contacts = array();
 		$photos = $this->contactCache->getContactPhotos();
+		$uids = $this->contactCache->getContactUids();
 
 		// Cache country because of loops
 		$configuredCountry = $this->configMapper->getCountry();
@@ -115,7 +116,7 @@ class SmsController extends Controller {
 			$photoversion = 2;
 		}
 
-		return new JSONResponse(array("phonelist" => $phoneList, "contacts" => $contacts, "lastRead" => $lastRead, "photos" => $photos, "photo_version" => $photoversion));
+		return new JSONResponse(array("phonelist" => $phoneList, "contacts" => $contacts, "lastRead" => $lastRead, "photos" => $photos, "uids" => $uids, "photo_version" => $photoversion));
 	}
 
 	/**
@@ -216,13 +217,16 @@ class SmsController extends Controller {
 		$phoneList = $this->smsMapper->getNewMessagesCountForAllPhonesNumbers($this->userId, $lastDate);
 		$contactsSrc = $this->contactCache->getContacts();
 		$photosSrc = $this->contactCache->getContactPhotos();
+		$uidsSrc = $this->contactCache->getContactUids();
 		$contacts = array();
 		$photos = array();
+		$uids = array();
 
 		foreach ($phoneList as $number => $ts) {
 			$fmtPN = preg_replace("#[ ]#","", $number);
 			if (isset($contactsSrc[$fmtPN])) {
 				$contacts[$fmtPN] = $contactsSrc[$fmtPN];
+				$uids[$fmtPN] = $uidsSrc[$fmtPN];
 
 				if (isset($photosSrc[$contacts[$fmtPN]])) {
 					$photos[$contacts[$fmtPN]] = $photosSrc[$contacts[$fmtPN]];
@@ -230,7 +234,7 @@ class SmsController extends Controller {
 			}
 		}
 
-		return new JSONResponse(array("phonelist" => $phoneList, "contacts" => $contacts, "photos" => $photos));
+		return new JSONResponse(array("phonelist" => $phoneList, "contacts" => $contacts, "photos" => $photos, "uids" => $uids));
 	}
 
 	/**

js/public/app.js

@@ -236,6 +236,12 @@ app.controller('OcSmsController', ['$scope', '$interval', '$timeout', '$compile'
 								contactObj.avatar = jsondata['photos'][peerLabel];
 							}
 
+							if (typeof jsondata['uids'][peerLabel] != 'undefined') {
+								contactObj.uid = jsondata['uids'][peerLabel];
+							} else {
+								contactObj.uid = peerLabel;
+							}
+
 							$scope.modifyContact(contactObj);
 							bufferedContacts.push(peerLabel);
 
@@ -403,6 +409,13 @@ app.controller('OcSmsController', ['$scope', '$interval', '$timeout', '$compile'
 					if (typeof(jsondata['photos'][peerLabel]) != 'undefined') {
 						contactObj['avatar'] = jsondata['photos'][peerLabel];
 					}
+
+					if (typeof jsondata['uids'][peerLabel] != 'undefined') {
+						contactObj.uid = jsondata['uids'][peerLabel];
+					} else {
+						contactObj.uid = peerLabel;
+					}
+
 					$scope.addContact(contactObj);
 					bufferedContacts.push(peerLabel);
 				}

lib/contactcache.php

@@ -23,6 +23,7 @@ class ContactCache {
 	private $contacts;
 	private $contactsInverted;
 	private $contactPhotos;
+	private $contactUids;
 
 	private $cfgMapper;
 	private $contactsManager;
@@ -30,6 +31,7 @@ class ContactCache {
 	public function __construct (ConfigMapper $cfgMapper, IContactsManager $contactsManager) {
 		$this->contacts = array();
 		$this->contactPhotos = array();
+		$this->contactUids = array();
 		$this->contactsInverted = array();
 
 		$this->cfgMapper = $cfgMapper;
@@ -60,6 +62,14 @@ class ContactCache {
 		return $this->contactPhotos;
 	}
 
+	public function getContactUids() {
+		// Only load contacts if they aren't in the buffer
+		if(count($this->contactUids) == 0) {
+			$this->loadContacts();
+		}
+		return $this->contactUids;
+	}
+
 	/**
 	 * Partially importe this function from owncloud Chat app
 	 * https://github.com/owncloud/chat/blob/master/app/chat.php
@@ -99,6 +109,10 @@ class ContactCache {
 					$photoURL = preg_replace("#^VALUE=uri:#","",$r["PHOTO"], 1);
 					$this->contactPhotos[$r["FN"]] = $photoURL;
 				}
+				
+				if (isset ($r["UID"])) {
+					$this->contactUids[$r["FN"]] = $r["UID"];
+				}
 			}
 		}
 	}

templates/main.php

@@ -13,7 +13,7 @@ use \OCA\OcSms\Lib\CountryCodes;
 		<ul class="contact-list" ng-show="!isContactsLoading">
 			<li ng-repeat="contact in contacts | orderBy:setting_contactOrder:setting_contactOrderReverse" peer-label="{{ contact.label }}" ng-click="loadConversation(contact);" href="#">
 				<img class="ocsms-plavatar" ng-src="{{ contact.avatar }}" ng-show="contact.avatar !== undefined" />
-				<div class="ocsms-plavatar" ng-show="contact.avatar === undefined" ng-style="{'background-color': (contact.label | peerColor)}">{{ contact.label | firstCharacter }}</div>
+				<div class="ocsms-plavatar" ng-show="contact.avatar === undefined" ng-style="{'background-color': (contact.uid | peerColor)}">{{ contact.label | firstCharacter }}</div>
 				<a class="ocsms-plname" style="{{ contact.unread > 0 ? 'font-weight:bold;' : ''}}" mailbox-label="{{ contact.label }}" mailbox-navigation="{{ contact.nav }}">{{ contact.label }}{{ contact.unread > 0 ? ' (' + contact.unread + ') ' : '' }}</a>
 			</li>
 		</ul>
@@ -61,7 +61,7 @@ use \OCA\OcSms\Lib\CountryCodes;
 		<div id="app-content-loader" class="icon-loading" ng-show="isConvLoading">
 		</div>
 		<div id="app-content-header" ng-show="!isConvLoading && selectedContact.label !== undefined && selectedContact.label !== ''"
-			 ng-style="{'background-color': (selectedContact.label | peerColor)}">
+			 ng-style="{'background-color': (selectedContact.uid | peerColor)}">
 			<div id="ocsms-contact-avatar">
 				<img class="ocsms-plavatar-big" ng-src="{{ selectedContact.avatar }}"
 					 ng-show="selectedContact.avatar !== undefined" />