From fe3016a3623efe3430fbb87d43c7ab317cc7ed16 Mon Sep 17 00:00:00 2001 From: Andres Jaan Tack Date: Thu, 30 Nov 2017 15:47:16 +0200 Subject: [PATCH] Remove an unnecessary global. --- static/sync/index.js | 34 ++++++++++++---------------------- 1 file changed, 12 insertions(+), 22 deletions(-) diff --git a/static/sync/index.js b/static/sync/index.js index d585191..1cc3728 100755 --- a/static/sync/index.js +++ b/static/sync/index.js @@ -8,10 +8,6 @@ $(function () { //Our interface to the Sync service var syncClient; - //We're going to use a single Sync document, our simplest - //synchronisation primitive, for this demo - var syncDoc; - //Get an access token for the current user, passing a device ID //In browser-based apps, every tab is like its own unique device //synchronizing state -- so we'll use a random UUID to identify @@ -28,10 +24,7 @@ $(function () { //This code will create and/or open a Sync document //Note the use of promises - syncClient.document('SyncGame').then(function(doc) { - //Lets store it in our global variable - syncDoc = doc; - + syncClient.document('SyncGame').then(function(syncDoc) { //Initialize game board UI to current state (if it exists) var data = syncDoc.get(); if (data.board) { @@ -42,24 +35,21 @@ $(function () { //changes on this document, we can trigger our UI to update syncDoc.on('updated', updateUserInterface); + //Whenever a board button is clicked, update that document. + $buttons.on('click', function (e) { + //Toggle the value: X, O, or empty + toggleCellValue($(e.target)); + + //Send updated document to Sync + //This should trigger "updated" events on other clients + var data = readGameBoardFromUserInterface(); + syncDoc.set(data); + + }); }); }); - //Whenever a board button is clicked: - $buttons.on('click', function (e) { - //Toggle the value: X, O, or empty - toggleCellValue($(e.target)); - - //Update the document - var data = readGameBoardFromUserInterface(); - - //Send updated document to Sync - //This should trigger "updated" events on other clients - syncDoc.set(data); - - }); - //Toggle the value: X, O, or empty (  for UI) function toggleCellValue($cell) { var cellValue = $cell.html();