Extract common code

groonga · Nov 4, 2014 · 1c44be6 · 1c44be6
1 parent dda8fb2
commit 1c44be6
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/app/scripts/controllers/table-search-controller.js b/app/scripts/controllers/table-search-controller.js
@@ -56,11 +56,20 @@ angular.module('groongaAdminApp')
       $location.search({});
     }
 
+    function addOutputColumn(name) {
+      var outputColumns = $scope.parameters.output_columns;
+      var inUse = true;
+      if (outputColumns) {
+        inUse = outputColumns.indexOf(name) !== -1;
+      }
+      $scope.outputColumns.push({name: name, inUse: inUse});
+    }
+
     function extractTableInfo(table) {
       if (table.name === $scope.table) {
-        $scope.outputColumns.push({name: '_id', inUse: true});
+        addOutputColumn('_id');
         if (table.hasKey) {
-          $scope.outputColumns.push({name: '_key', inUse: true});
+          addOutputColumn('_key');
         }
       }
 
@@ -72,16 +81,11 @@ angular.module('groongaAdminApp')
 
     function extractColumnsInfo(table, columns) {
       if (table.name === $scope.table) {
-        var outputColumns = $scope.parameters.output_columns;
         columns.forEach(function(column) {
           if (column.isIndex) {
             return;
           }
-          var inUse = true;
-          if (outputColumns) {
-            inUse = outputColumns.indexOf(column.name) !== -1;
-          }
-          $scope.outputColumns.push({name: column.name, inUse: inUse});
+          addOutputColumn(column.name);
         });
       }