Investigating snapshot diff updates (#393)

* small cleanup after looking at queriesCtrl * small cleans * one more remnant! Co-authored-by: epugh@opensourceconnections.com <>
o19s · Aug 6, 2021 · 77b272c · 77b272c
1 parent 2c5d745
commit 77b272c
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 20 deletions.
diff --git a/app/assets/javascripts/components/query_explain/query_explain_controller.js b/app/assets/javascripts/components/query_explain/query_explain_controller.js
@@ -6,11 +6,9 @@ angular.module('QuepidApp')
   .controller('QueryExplainCtrl', [
     '$uibModal',
     '$scope',
-    '$log',
     function (
       $uibModal,
-      $scope,
-      $log
+      $scope
     ) {
       var ctrl  = this;
       ctrl.query = $scope.query;
@@ -29,12 +27,8 @@ angular.module('QuepidApp')
         });
 
         modalInstance.result.then(
-          function() {
-
-          },
-          function() {
-            $log.info('INFO: Modal dismissed');
-          }
+          function() { },
+          function() { }
         );
       }
     }

diff --git a/app/assets/javascripts/controllers/queriesCtrl.js b/app/assets/javascripts/controllers/queriesCtrl.js
@@ -2,7 +2,7 @@
 /*jslint latedef:false*/
 
 angular.module('QuepidApp')
-  .controller('queriesCtrl', [
+  .controller('QueriesCtrl', [
     '$scope',
     '$rootScope',
     '$q',
@@ -206,27 +206,30 @@ angular.module('QuepidApp')
       };
 
       $scope.queries.selectedDiffName = function() {
+        var diffName = '';
         if (queryViewSvc.diffSetting === null) {
-          return 'disabled';
+          diffName = 'disabled';
         }
         else if (queryViewSvc.diffSetting === 'best') {
-          return 'target';
+          diffName = 'target';
         } else {
-          return 'snapshot';
+          diffName = 'snapshot';
         }
-        return 'meow';
+        return diffName;
       };
 
       $scope.queries.fullDiffName = function() {
+        var fullDiffName = '';
         if (queryViewSvc.diffSetting === null) {
-          return 'disabled';
+          fullDiffName = 'disabled';
         }
         else if (queryViewSvc.diffSetting === 'best') {
-          return 'Highest ratest results for each query';
+          fullDiffName = 'Highest ratest results for each query';
         } else {
           var snapshot = querySnapshotSvc.snapshots[queryViewSvc.diffSetting];
-          return snapshot.name();
+          fullDiffName = snapshot.name();
         }
+        return fullDiffName;
       };
 
       function saveScoring() {

diff --git a/app/assets/javascripts/directives/queries.js b/app/assets/javascripts/directives/queries.js
@@ -6,7 +6,7 @@ angular.module('QuepidApp')
       return {
         restrict: 'E',
         transclude: true,
-        controller: 'queriesCtrl',
+        controller: 'QueriesCtrl',
         templateUrl: 'views/queries.html',
         replace: true
       };

diff --git a/app/assets/templates/views/queriesLayout.html b/app/assets/templates/views/queriesLayout.html
@@ -3,7 +3,7 @@
     <div class="pane_main results-control">
       <div id="case-header" class="row">
         <div class="col-sm-12">
-          <div ng-controller="queriesCtrl" style="display: inline-block">
+          <div ng-controller="QueriesCtrl" style="display: inline-block">
             <qscore
               class="case-score"
               diff-label="queries.selectedDiffName()"

diff --git a/app/assets/templates/views/searchResults.html b/app/assets/templates/views/searchResults.html
@@ -8,7 +8,6 @@
       diff-label="''"
       full-diff-name="''"
       max-score="maxScore || 100";
-      ng-click="pickUnitTestScorer()"
       scorable="query"
       score-label="''"
       score-type="'query'"