From eee6476d6adbeca235fdeb90737c3b3004e9209d Mon Sep 17 00:00:00 2001
From: Mike Primm <mike@primmhome.com>
Date: Sun, 4 Sep 2011 01:35:02 -0500
Subject: [PATCH] Fix player faces in sidebar, while still preserving load
 cache

---
 web/js/minecraft.js | 38 +++++++++++++++++++++++++++++++++++---
 1 file changed, 35 insertions(+), 3 deletions(-)

diff --git a/web/js/minecraft.js b/web/js/minecraft.js
index fabe2867..b1ced802 100644
--- a/web/js/minecraft.js
+++ b/web/js/minecraft.js
@@ -9,12 +9,44 @@ function createMinecraftHead(player,size,completed,failed) {
 	faceImage.src = dynmap.options.tileUrl + 'faces/' + size + 'x' + size + '/' + player + '.png';
 }
 
+var playerHeads = {};
+
 function getMinecraftHead(player,size,completed) {
-	createMinecraftHead(player, size, completed, function() {
-		console.error('Failed to retrieve face of "', player, '" with size "', size, '"!')
-	});
+    var key = player + '.' + size;
+	var head = playerHeads[key];
+	// Synchronous
+	if (!completed) {
+		return (!head || head.working) ? null : head;
+	}
+
+	// Asynchronous
+	if (!head) {
+		playerHeads[key] = { working: true, hooks: [{f:completed}] };
+		createMinecraftHead(player, size, function(head) {
+			hooks = playerHeads[key].hooks;
+			playerHeads[key] = head;
+			var i;
+			for(i=0;i<hooks.length;i++) {
+				hooks[i].f(head);
+			}
+			head.working = false;
+		}, function() {
+
+		});
+	} else if (head.working) {
+		//console.log('Other process working on head of ',player,', will add myself to hooks...');
+		head.hooks[head.hooks.length] = {f:completed};
+	} else {
+		completed(head);
+	}
 }
 
+//function getMinecraftHead(player,size,completed) {
+//	createMinecraftHead(player, size, completed, function() {
+//		console.error('Failed to retrieve face of "', player, '" with size "', size, '"!')
+//	});
+//}
+
 function getMinecraftTime(servertime) {
 	servertime = parseInt(servertime);
 	var day = servertime >= 0 && servertime < 13700;