From 232ff57543ce72aabdc581ca9507e6deba5d4fce Mon Sep 17 00:00:00 2001
From: Raz Yalov <raz.yalov@gmail.com>
Date: Sat, 21 Apr 2018 17:02:53 -0700
Subject: [PATCH 1/2] Add a new setting to allow users to choose between `on`
 and `once` for how the query is being executed (to avoid the UI changing
 after data has been updated) Change default query to iterate over snapshot
 instead of use a single console.log, to make results more readable. Fix copy
 to clipboard to use the full result and not just the first result Also: fix
 UI to better display multiple results (multiple calls to console.log) by
 showing them in numbered tabs UI/UX: make UI show a wait animation while
 running long queries (using the 'loader' class) Firebase: upgrade to latest
 `firebase-admin` instead of `firebase` version to prevent errors with
 InitializeApp that is called with wrong parameters.

---
 create.html              |   2 +-
 index.html               |   2 +-
 js/app.js                | 122 +++++++++++++++++++++++----------------
 js/default-settings.json |   3 +-
 js/settings.js           |  10 +++-
 package.json             |   2 +-
 settings.html            |  12 ++++
 templates/main.html      |   9 ++-
 8 files changed, 105 insertions(+), 57 deletions(-)

diff --git a/create.html b/create.html
index 932b88a..1ee95e2 100644
--- a/create.html
+++ b/create.html
@@ -96,7 +96,7 @@
 <script>
 	const electron = require('electron').remote
 	const ipc = require('electron').ipcRenderer
-	const firebase = require("firebase")
+	const firebase = require("firebase-admin")
 	const writeFile = require('write-file-atomic')
 	require('electron').webFrame.setZoomLevelLimits(1, 1)
 </script>
diff --git a/index.html b/index.html
index fee5aca..bb5818c 100644
--- a/index.html
+++ b/index.html
@@ -11,7 +11,7 @@
 	<script>
 		const electron = require('electron').remote
 		const ipc = require('electron').ipcRenderer
-		const firebase = require("firebase")
+		const firebase = require("firebase-admin")
 		const writeFile = require('write-file-atomic')
 		const clipboard = electron.clipboard
 		//window.$ = window.jQuery = require('jquery')
diff --git a/js/app.js b/js/app.js
index 6a62bb2..a0941c9 100644
--- a/js/app.js
+++ b/js/app.js
@@ -74,6 +74,7 @@ var fba = angular.module('fba', ['ngRoute', 'angularResizable', 'ui.codemirror']
   $scope.codeResult = 'No Data'
   $scope.query = ''
   $scope.mode = 'explorer'
+  $scope.selectedResult = 0 // in case of multiple results, which one to show first
 
   if (process.env.mode && process.env.mode === 'dev') {
     console.oldLog = console.log
@@ -311,58 +312,76 @@ var fba = angular.module('fba', ['ngRoute', 'angularResizable', 'ui.codemirror']
 
   $scope.run = function () {
     let query = $scope.queryView.getValue()
-    $scope.results = []
-    if($scope.mode === 'explorer') {
-      $scope.collection = {}
-      try {
-        if(!query.endsWith('/')) {
-          query = query + '/'
-        }
-        query = query.substring($scope.baseURL.length, query.length - 1)
-        $scope.currentApp.database().ref(query).on('value', function (snapshot) {
-          $scope.result = snapshot.val()
-          $scope.results.push(snapshot.val())
-          if(!snapshot.key) {
-            $scope.collection.name = $scope.currentApp.name
-            $scope.collection.url = ''
-          } else {
-            $scope.collection.name = snapshot.key
-            $scope.collection.url = query
-          }
-          if (typeof $scope.result === 'object') {
-            $scope.collection.collections = $scope.updateResult($scope.result, $scope.collection.url)
-          } else {
-            $scope.collection.value = $scope.result
-            $scope.collection.leaf = true
+    $scope.results = ["fetching..."]
+    $scope.selectedResult = 0
+    $scope.queryRunning = true
+      // invoke the actual acction async so the UI can refresh for the waiting animation.
+      $timeout(() => {
+          if($scope.mode === 'explorer') {
+              $scope.collection = {}
+              try {
+                  if(!query.endsWith('/')) {
+                      query = query + '/'
+                  }
+                  query = query.substring($scope.baseURL.length, query.length - 1)
+                  let queryMethod = $rootScope.settings.queryMethod || 'on'
+                  $scope.currentApp.database().ref(query)[queryMethod]('value', function (snapshot) {
+                      $scope.result = snapshot.val()
+                      $scope.results[$scope.result]
+                      if(!snapshot.key) {
+                          $scope.collection.name = $scope.currentApp.name
+                          $scope.collection.url = ''
+                      } else {
+                          $scope.collection.name = snapshot.key
+                          $scope.collection.url = query
+                      }
+                      if (typeof $scope.result === 'object') {
+                          $scope.collection.collections = $scope.updateResult($scope.result, $scope.collection.url)
+                      } else {
+                          $scope.collection.value = $scope.result
+                          $scope.collection.leaf = true
+                      }
+                      $scope.collection.open = true
+                      $scope.codeResult = JSON.stringify($scope.result, null, 2)
+                      $scope.activeUrl = $scope.collection.url
+                      $scope.queryRunning = false
+                  }, function (err) {
+                      $scope.result = 'The read failed: ' + err.code
+                      $scope.results = [$scope.result]
+                      $scope.queryRunning = false
+                  })
+              } catch (err) {
+                  $scope.result = 'Ooops.. There is an error":\n"' + err.message
+                  $scope.results = [$scope.result]
+                  $scope.queryRunning = false
+              }
+          } else if($scope.mode === 'query') {
+              let regex = 'firebase().'
+              query = query.replace(regex, '$scope.currentApp.')
+              try {
+                  eval(query)
+                  $scope.results = []
+                  if (typeof $scope.$log !== 'undefined' && $scope.$log != null && $scope.$log.length > 0) {
+                      $scope.$log.forEach((res) => {
+                          $scope.results.push(res)
+                      })
+                  } else {
+                      // no results found
+                      $scope.results.push("No results matched your query.")
+                  }
+                  $scope.$log = null
+                  $scope.queryRunning = false
+              } catch (err) {
+                  $scope.result = errorMessage = 'Ooops.. There is an error":\n"' + err.message
+                  $scope.results = [$scope.result]
+                  $scope.queryRunning = false
+              }
           }
-          $scope.collection.open = true
-          $scope.codeResult = JSON.stringify($scope.result, null, 2)
-          $scope.activeUrl = $scope.collection.url
-        }, function (err) {
-          $scope.result = 'The read failed: ' + err.code
-        })
-      } catch (err) {
-        $scope.result = 'Ooops.. There is an error":\n"' + err.message
-      }
-    } else if($scope.mode === 'query') {
-      let regex = 'firebase().'
-      query = query.replace(regex, '$scope.currentApp.')
-       try {
-        eval(query)
-        if (typeof $scope.$log !== 'undefined' && $scope.$log.length > 0) {
-          $scope.$log.forEach((res) => {
-            $scope.results.push(res)
-          })
-        }
-        $scope.$log = null
-      } catch (err) {
-        $scope.result = 'Ooops.. There is an error":\n"' + err.message
-      }
-    }
+      })
   }
 
   $scope.copy = () => {
-    let copyText = JSON.stringify($scope.result, null, '\t')
+    let copyText = JSON.stringify($scope.results, null, '\t')
     clipboard.writeText(copyText)
     $scope.copiedNow = true
     $timeout(() => {
@@ -412,7 +431,8 @@ var fba = angular.module('fba', ['ngRoute', 'angularResizable', 'ui.codemirror']
       } else {
         $scope.query = ''
       }
-      $scope.queryView.setValue(`firebase().database().ref('/${$scope.query}').on('value', function (snapshot) {\n\tconsole.log(snapshot.val())\n})`)
+      let queryMethod = $rootScope.settings.queryMethod || 'on'
+      $scope.queryView.setValue(`firebase().database().ref('/${$scope.query}').${queryMethod}('value', function (snapshot) {\n\tsnapshot.forEach((it)=>{console.log({[it.key]:it.val()})})\n})`)
       $scope.changeQueryBoxHeight(90)
     }
     $scope.refreshQueryView()
@@ -439,6 +459,10 @@ var fba = angular.module('fba', ['ngRoute', 'angularResizable', 'ui.codemirror']
   $scope.updateChild = (collection) => {
     $scope.dbRef.child(collection.url).set(collection.value)
   }
+
+  $scope.selectResult = (resultId) => {
+    $scope.selectedResult = resultId
+  }
 })
 
 .controller('connectionController', function ($scope) {
diff --git a/js/default-settings.json b/js/default-settings.json
index d61e32e..000891d 100644
--- a/js/default-settings.json
+++ b/js/default-settings.json
@@ -1,5 +1,6 @@
 {
   "theme": "light",
   "fonts": "system",
-  "jsonTheme": "default"
+  "jsonTheme": "default",
+  "queryMethod": "on"
 }
diff --git a/js/settings.js b/js/settings.js
index 7c7aa7d..6e554bd 100644
--- a/js/settings.js
+++ b/js/settings.js
@@ -3,11 +3,13 @@ var fbaP = angular.module('fba-p', []).run(function () {
 
 .controller('settingsController', function ($scope, $timeout) {
   $scope.view = {}
-  $scope.view.theme = true
-  $scope.activeUrl = 'theme'
   $scope.saved = false
   $scope.jsonThemes = []
   $scope.items = [{
+      name: 'General',
+      url: 'general'
+  },
+  {
     name: 'Theme',
     url: 'theme'
   },{
@@ -15,6 +17,10 @@ var fbaP = angular.module('fba-p', []).run(function () {
     url: 'fonts'
   }]
 
+  let defaultView = $scope.items[0]
+  $scope.view[defaultView.url] = true
+  $scope.activeUrl = defaultView.url
+
   const userPath = electron.app.getPath('userData')
   const fs = require('graceful-fs')
   try {
diff --git a/package.json b/package.json
index 2bcf636..695d8f8 100644
--- a/package.json
+++ b/package.json
@@ -35,7 +35,7 @@
     "spectron": "^3.3.0"
   },
   "dependencies": {
-    "firebase": "^3.2.0",
+    "firebase-admin": "^5.12.0",
     "graceful-fs": "^4.1.4",
     "jquery": "^3.1.0",
     "write-file-atomic": "^1.1.4"
diff --git a/settings.html b/settings.html
index 4a0e7c6..f1cd417 100644
--- a/settings.html
+++ b/settings.html
@@ -18,6 +18,18 @@
 	</div>
 	<div class="admin-content">
 		<form class="form" ng-submit="save()" name="settings">
+			<section class="general" ng-show="view.general">
+				<div class="panel">
+					<div class="panel-heading">Query Method</div>
+					<div>Choose what method to use when retrieving data from a database reference object. Using <code>once</code> will retrieve the value once, while using <code>on</code> will continue to monitor changes to this reference and will reload the results if the real-time database detects a change.</div>
+					<div class="radio">
+						<div class="radio" ng-repeat="qType in ['on', 'once']" >
+							<input type="radio" name="queryMethod" ng-attr-id="queryMethod-{{qType}}" ng-model="form.queryMethod" ng-value="qType">
+							<label ng-attr-for="queryMethod-{{qType}}">{{qType}}</label>
+						</div>
+					</div>
+				</div>
+			</section>
 			<section class="theme" ng-show="view.theme">
 				<div class="panel">
 					<div class="panel-heading">Main Theme</div>
diff --git a/templates/main.html b/templates/main.html
index 65abbf8..dd41628 100644
--- a/templates/main.html
+++ b/templates/main.html
@@ -1,7 +1,7 @@
 <section ng-hide="rootError" class="container content">
 	<form id="query-box" class="query-box" resizable r-directions="['bottom']">
 		<ui-codemirror id="query" ui-codemirror-opts="codeQueryOptions" ng-model="query"></ui-codemirror>
-		<button type="submit" class="button" ng-click="run()">Run</button>
+		<div class="loader" ng-show="queryRunning"></div><button type="submit" class="button" ng-click="run()" ng-disable="queryRunning">Run<span ng-show="queryRunning">ing...</span></button>
 	</form>
 	<div class="result-wrap">
 		<div class="result-mode">
@@ -23,7 +23,12 @@
 			</div>
 		</div>
 		<div class="result-view raw" ng-show="view.raw">
-			<div class="result" ng-repeat="result in results">{{result}}</div>
+			<div class="result-wrap">
+				<div class="result-mode" ng-show="results.length > 1">
+					<div class="icon" ng-repeat="result in results track by $index" ng-click="selectResult($index)" ng-class="{'active': selectedResult == $index}">{{$index+1}}</div>
+				</div>
+				<div class="result" ng-repeat="result in results track by $index" ng-show="selectedResult == $index">{{result}}</div>
+			</div>
 		</div>
 		<div class="copy-wrap" ng-click="copy()">
 			<img class="copy" src="img/copy.svg">

From 3e24aff0c7fd6bf01ca30e5236c0c8386486e540 Mon Sep 17 00:00:00 2001
From: Raz Yalov <raz.yalov@gmail.com>
Date: Sat, 21 Apr 2018 17:40:38 -0700
Subject: [PATCH 2/2] Reflect the selected firebase database in the title and
 the Window menu options

---
 index.html | 2 +-
 js/app.js  | 1 +
 main.js    | 2 ++
 3 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/index.html b/index.html
index bb5818c..9c37eb7 100644
--- a/index.html
+++ b/index.html
@@ -20,7 +20,7 @@
 </head>
 <body ng-app="fba" ng-controller="mainController" class="{{os}} index">
 	<div id="mainmenu">
-		<div class="titlebar" ng-if="titleBar">Firebase Admin</div>
+		<div class="titlebar" ng-if="titleBar">Firebase Admin<span ng-show="currentApp.name"> - {{currentApp.name}}</span></div>
 		<div class="icons">
 			<div class="icon" ng-click="listShown = true;">
 				<img class="logo" src="img/list.svg">
diff --git a/js/app.js b/js/app.js
index a0941c9..c736ee3 100644
--- a/js/app.js
+++ b/js/app.js
@@ -182,6 +182,7 @@ var fba = angular.module('fba', ['ngRoute', 'angularResizable', 'ui.codemirror']
       $scope.currentApp = $scope.apps[id]
       $scope.update()
       dataBin.setData('openCon', openCon)
+      ipc.send('set-title', $scope.currentApp.name)
     } catch (err) {
       $scope.menuOverlay = false
       $scope.menuHidden = true
diff --git a/main.js b/main.js
index efbaba2..84ddb5e 100644
--- a/main.js
+++ b/main.js
@@ -92,6 +92,8 @@ ipc.on('show-context-menu', (e, args) => {
 
 ipc.on('reload-window', () => mainWindow.reload())
 
+ipc.on('set-title', function(e, newTitle) { mainWindow.setTitle(newTitle) })
+
 ipc.on('open-create-window', function (event) {
   createWindow('conWin', 'create.html', {parent: mainWindow, width: 600, height: 320})
 })