From 7f8b9b07415d986da2620c818b63a6d85e1fa2f6 Mon Sep 17 00:00:00 2001
From: Luis Fernando Molina <Luis-Fernando-Molina@users.noreply.github.com>
Date: Thu, 11 Apr 2019 11:43:17 +0200
Subject: [PATCH] TimeScale persisted to local storage

---
 Scales/TimeScale.js | 12 ++++++++-
 TimeMachine.js      | 66 +++++++++++++++++++++++----------------------
 2 files changed, 45 insertions(+), 33 deletions(-)

diff --git a/Scales/TimeScale.js b/Scales/TimeScale.js
index b52ab8c10..0e2e62417 100644
--- a/Scales/TimeScale.js
+++ b/Scales/TimeScale.js
@@ -11,9 +11,10 @@ function newTimeScale () {
     draw: draw,
     getContainer: getContainer,
     initialize: initialize,
-    lenghtPercentage: 100
+    lenghtPercentage: undefined
   }
 
+  const LENGHT_PERCENTAGE_DEFAULT_VALUE = 50
   const RIGHT_MARGIN = 50
   const STEP_SIZE = 5
 
@@ -34,6 +35,12 @@ function newTimeScale () {
 
   function initialize () {
     thisObject.container.eventHandler.listenToEvent('Mouse Wheel', onMouseWheel)
+
+    thisObject.lenghtPercentage = window.localStorage.getItem(MODULE_NAME)
+    if (!thisObject.lenghtPercentage) { thisObject.lenghtPercentage = LENGHT_PERCENTAGE_DEFAULT_VALUE }
+
+    event.lenghtPercentage = thisObject.lenghtPercentage
+    thisObject.container.eventHandler.raiseEvent('Lenght Percentage Changed', event)
   }
 
   function onMouseWheel (event) {
@@ -45,8 +52,11 @@ function newTimeScale () {
       thisObject.lenghtPercentage = thisObject.lenghtPercentage + STEP_SIZE
       if (thisObject.lenghtPercentage > 100) { thisObject.lenghtPercentage = 100 }
     }
+
     event.lenghtPercentage = thisObject.lenghtPercentage
     thisObject.container.eventHandler.raiseEvent('Lenght Percentage Changed', event)
+
+    window.localStorage.setItem(MODULE_NAME, thisObject.lenghtPercentage)
   }
 
   function getContainer (point) {
diff --git a/TimeMachine.js b/TimeMachine.js
index 1d6a040ec..1dcbaf300 100644
--- a/TimeMachine.js
+++ b/TimeMachine.js
@@ -67,24 +67,6 @@ function newTimeMachine () {
     controlPanelHandle = canvas.panelsSpace.createNewPanel('Time Control Panel', undefined, panelOwner)
     let controlPanel = canvas.panelsSpace.getPanel(controlPanelHandle, panelOwner)
 
-       /* Each Time Machine has a Time Scale. */
-
-    timeScale = newTimeScale()
-    timeScale.initialize()
-
-    timeScale.container.eventHandler.listenToEvent('Lenght Percentage Changed', function (event) {
-      thisObject.container.frame.width = TIME_MACHINE_WIDTH * event.lenghtPercentage / 100
-      thisObject.container.eventHandler.raiseEvent('Dimmensions Changed', event)
-    })
-
-    rigthScale = newRigthScale()
-    rigthScale.initialize()
-
-    rigthScale.container.eventHandler.listenToEvent('Height Percentage Changed', function (event) {
-      thisObject.container.frame.height = TIME_MACHINE_HEIGHT * event.heightPercentage / 100
-      thisObject.container.eventHandler.raiseEvent('Dimmensions Changed', event)
-    })
-
       /* First, we initialize the market that we are going to show first on screen. Later all the other markets will be initialized on the background. */
 
     let position = 0 // This defines the position of each chart respect to each other.
@@ -103,11 +85,7 @@ function newTimeMachine () {
     timelineChart.initialize(DEFAULT_EXCHANGE, DEFAULT_MARKET, onDefaultInitialized)
 
     function onDefaultInitialized (err) {
-      if (INFO_LOG === true) { logger.write('[INFO] initialize -> onDefaultInitialized -> Entering function.') }
-
       if (err.result !== GLOBAL.DEFAULT_OK_RESPONSE.result) {
-        if (INFO_LOG === true) { logger.write('[INFO] initialize -> onDefaultInitialized -> Initialization of the only market failed.') }
-
         callBackFunction(err)
         return
       }
@@ -117,6 +95,26 @@ function newTimeMachine () {
       controlPanel.container.eventHandler.listenToEvent('Datetime Changed', timelineChart.setDatetime, undefined)
       timelineChart.container.eventHandler.listenToEvent('Datetime Changed', controlPanel.setDatetime)
 
+      /* Each Time Machine has a Time Scale and a Right Scale. */
+
+      timeScale = newTimeScale()
+
+      timeScale.container.eventHandler.listenToEvent('Lenght Percentage Changed', function (event) {
+        thisObject.container.frame.width = TIME_MACHINE_WIDTH * event.lenghtPercentage / 100
+        thisObject.container.eventHandler.raiseEvent('Dimmensions Changed', event)
+      })
+
+      timeScale.initialize()
+
+      rigthScale = newRigthScale()
+
+      rigthScale.container.eventHandler.listenToEvent('Height Percentage Changed', function (event) {
+        thisObject.container.frame.height = TIME_MACHINE_HEIGHT * event.heightPercentage / 100
+        thisObject.container.eventHandler.raiseEvent('Dimmensions Changed', event)
+      })
+
+      rigthScale.initialize()
+
       initializeTheRest()
     }
 
@@ -195,24 +193,28 @@ function newTimeMachine () {
       chart.draw()
     }
 
-    timeScale.draw()
-    rigthScale.draw()
+    if (timeScale !== undefined) { timeScale.draw() }
+    if (rigthScale !== undefined) { rigthScale.draw() }
   }
 
   function getContainer (point, purpose) {
     let container
 
-    container = timeScale.getContainer(point)
-    if (container !== undefined) {
-      if (container.isForThisPurpose(purpose)) {
-        return container
+    if (timeScale !== undefined) {
+      container = timeScale.getContainer(point)
+      if (container !== undefined) {
+        if (container.isForThisPurpose(purpose)) {
+          return container
+        }
       }
     }
 
-    container = rigthScale.getContainer(point)
-    if (container !== undefined) {
-      if (container.isForThisPurpose(purpose)) {
-        return container
+    if (rigthScale !== undefined) {
+      container = rigthScale.getContainer(point)
+      if (container !== undefined) {
+        if (container.isForThisPurpose(purpose)) {
+          return container
+        }
       }
     }