From 7bc5a75ab268fa9143fabc20487554ba11d9abf4 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 14:03:01 -0700
Subject: [PATCH 01/11] Use order<0 to set an app hidden

---
 src/kibana/plugins/kibana/_apps.js    | 6 ++++++
 src/kibana/plugins/kibana/kibana.html | 2 +-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/kibana/plugins/kibana/_apps.js b/src/kibana/plugins/kibana/_apps.js
index c0e948f58450a..b2ab9c425bbe3 100644
--- a/src/kibana/plugins/kibana/_apps.js
+++ b/src/kibana/plugins/kibana/_apps.js
@@ -11,6 +11,10 @@ define(function (require) {
       return app.lastPath;
     }
 
+    function getShow(app) {
+      app.show = app.order >= 0 ? true : false;
+    }
+
     function setLastPath(app, path) {
       app.lastPath = path;
       return sessionStorage.set(appKey(app), path);
@@ -19,6 +23,8 @@ define(function (require) {
     $scope.apps = Private(require('registry/apps'));
     // initialize each apps lastPath (fetch it from storage)
     $scope.apps.forEach(getLastPath);
+    $scope.apps.forEach(getShow);
+
 
     function onRouteChange() {
       var route = $location.path().split(/\//);
diff --git a/src/kibana/plugins/kibana/kibana.html b/src/kibana/plugins/kibana/kibana.html
index 728f8f131fbfd..295b22ef8e68c 100644
--- a/src/kibana/plugins/kibana/kibana.html
+++ b/src/kibana/plugins/kibana/kibana.html
@@ -18,7 +18,7 @@
       <!-- Full navbar -->
       <div collapse="!showCollapsed" class="navbar-collapse" id="kibana-primary-navbar">
         <ul class="nav navbar-nav">
-          <li ng-repeat="app in apps.inOrder" ng-class="{active: activeApp === app}">
+          <li ng-repeat="app in apps.inOrder | filter:{show: true}" ng-class="{active: activeApp === app}">
             <a ng-href="#{{app.lastPath}}" bo-text="app.name"></a>
           </li>
         </ul>

From 4f0a6e6fcae5c2ca6a62fc8bc48c30a14b77731a Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 14:03:46 -0700
Subject: [PATCH 02/11] Add doc view app, abstract out computed fields
 discovery

---
 .../index_patterns/_get_computed_fields.js    | 22 ++++++
 src/kibana/plugins/doc/controllers/doc.js     | 68 +++++++++++++++++++
 src/kibana/plugins/doc/index.html             | 47 +++++++++++++
 src/kibana/plugins/doc/index.js               | 13 ++++
 src/kibana/plugins/doc/styles/main.less       |  7 ++
 5 files changed, 157 insertions(+)
 create mode 100644 src/kibana/components/index_patterns/_get_computed_fields.js
 create mode 100644 src/kibana/plugins/doc/controllers/doc.js
 create mode 100644 src/kibana/plugins/doc/index.html
 create mode 100644 src/kibana/plugins/doc/index.js
 create mode 100644 src/kibana/plugins/doc/styles/main.less

diff --git a/src/kibana/components/index_patterns/_get_computed_fields.js b/src/kibana/components/index_patterns/_get_computed_fields.js
new file mode 100644
index 0000000000000..3cfd8ea5ad6bc
--- /dev/null
+++ b/src/kibana/components/index_patterns/_get_computed_fields.js
@@ -0,0 +1,22 @@
+// Takes a hit, merges it with any stored/scripted fields, and with the metaFields
+// returns a flattened version
+define(function (require) {
+  var _ = require('lodash');
+  return function () {
+    var self = this;
+    var scriptFields = {};
+
+    _.each(self.fields.byType['date'], function (field) {
+      if (field.indexed) {
+        scriptFields[field.name] = {
+          script: 'doc["' + field.name + '"].value'
+        };
+      }
+    });
+    return {
+      fields: ['*', '_source'],
+      scriptFields: scriptFields
+    };
+
+  };
+});
diff --git a/src/kibana/plugins/doc/controllers/doc.js b/src/kibana/plugins/doc/controllers/doc.js
new file mode 100644
index 0000000000000..33d0714e2ba14
--- /dev/null
+++ b/src/kibana/plugins/doc/controllers/doc.js
@@ -0,0 +1,68 @@
+define(function (require) {
+  var _ = require('lodash');
+  var angular = require('angular');
+
+  require('components/notify/notify');
+  require('components/courier/courier');
+  require('components/doc_viewer/doc_viewer');
+  require('components/index_patterns/index_patterns');
+
+  var app = require('modules').get('apps/doc', [
+    'kibana/notify',
+    'kibana/courier',
+    'kibana/index_patterns'
+  ]);
+
+  require('routes')
+  .when('/doc/:indexPattern/:index/:type/:id', {
+    template: require('text!plugins/doc/index.html'),
+    resolve: {
+      indexPattern: function (courier, savedSearches, $route) {
+        return courier.indexPatterns.get($route.current.params.indexPattern);
+      }
+    }
+  });
+
+  app.controller('doc', function ($scope, courier, $route, Notifier, es) {
+
+    // Pretty much only need this for formatting, not actually using it for fetching anything.
+    $scope.indexPattern = $route.current.locals.indexPattern;
+
+    var notify = new Notifier({
+      location: 'Doc'
+    });
+
+    var computedFields = $scope.indexPattern.getComputedFields();
+
+    // Q: Hey jerk, why are you using this instead of just getting a document?
+    // A: No need for name calling. This is the only way we can get the scripted & stored fields.
+    // Q: Then why not use a searchSource?
+    // A: I only need the one index, constructing a temporary index pattern seems wasteful.
+    es.search({
+      index: $route.current.params.index,
+      type: $route.current.params.type,
+      body: {
+        query: {
+          ids: {
+            values: [$route.current.params.id]
+          }
+        },
+        fields: computedFields.fields,
+        script_fields: computedFields.scriptFields
+      }
+    }).then(function (resp) {
+      if (resp.hits) {
+        if (resp.hits.total < 1) {
+          $scope.status = 'notFound';
+        } else {
+          $scope.status = 'found';
+          $scope.hit = resp.hits.hits[0];
+        }
+      }
+    }).catch(function (err) {
+      $scope.status = 'error';
+      $scope.resp = err;
+    });
+
+  });
+});
diff --git a/src/kibana/plugins/doc/index.html b/src/kibana/plugins/doc/index.html
new file mode 100644
index 0000000000000..78af147e8690a
--- /dev/null
+++ b/src/kibana/plugins/doc/index.html
@@ -0,0 +1,47 @@
+<div ng-controller="doc" class="app-container">
+
+  <div class="container-fluid">
+    <div class="row">
+
+      <!-- no results -->
+      <div class="col-md-12" ng-if="status === 'notFound'">
+          <div class="col-md-12">
+            <h1>Failed to locate document. <i class="fa fa-meh-o"></i></h1>
+
+            <p>
+              Unfortunately I could not find any documents matching that id, of that type, in that index. I tried really hard. I wanted it to be there. Sometimes I swear documents grow legs and just walk out of the index. Sneaky. I wish I could offer some advice here, something to make you feel better
+            </p>
+
+          </div>
+      </div>
+
+            <!-- no results -->
+      <div class="col-md-12" ng-if="status === 'error'">
+          <div class="col-md-12">
+            <h1>This is bad. <i class="fa fa-meh-o"></i></h1>
+
+            <p>
+              Oh no. Something went very wrong. Its not just that I couldn't find your document, I couldn't even try. The index was missing, or the type. Go check out Elasticsearch, something isn't quite right here.
+            </p>
+
+          </div>
+      </div>
+
+      <!-- loading -->
+      <div class="col-md-12" ng-if="status === undefined">
+        <div class="discover-overlay">
+          <h2>Searching</h2>
+          <div class="spinner large"></div>
+          <div ng-show="fetchStatus">{{fetchStatus.complete}}/{{fetchStatus.total}}</div>
+        </div>
+      </div>
+
+      <!-- result -->
+      <div class="col-md-12" ng-if="status === 'found'">
+        <h2><b>Doc:</b> {{hit._index}}/{{hit._type}}/{{hit._id}}</h2>
+
+        <doc-viewer hit="hit" index-pattern="indexPattern"></doc-viewer>
+      </div>
+    </div>
+  </div>
+</div>
diff --git a/src/kibana/plugins/doc/index.js b/src/kibana/plugins/doc/index.js
new file mode 100644
index 0000000000000..daa8473a12e73
--- /dev/null
+++ b/src/kibana/plugins/doc/index.js
@@ -0,0 +1,13 @@
+define(function (require, module, exports) {
+  require('plugins/doc/controllers/doc');
+  require('css!plugins/doc/styles/main.css');
+
+  var apps = require('registry/apps');
+  apps.register(function DocAppModule() {
+    return {
+      id: 'doc',
+      name: 'Doc Viewer',
+      order: -1
+    };
+  });
+});
\ No newline at end of file
diff --git a/src/kibana/plugins/doc/styles/main.less b/src/kibana/plugins/doc/styles/main.less
new file mode 100644
index 0000000000000..ae2a89ba62f84
--- /dev/null
+++ b/src/kibana/plugins/doc/styles/main.less
@@ -0,0 +1,7 @@
+@import (reference) "../../../styles/_bootstrap.less";
+@import (reference) "../../../styles/theme/_theme.less";
+@import (reference) "../../../styles/_variables.less";
+
+.application-doc {
+  // Styles here
+}
\ No newline at end of file

From de5d29d92fd0b10414b39e76f611ad71d00b4244 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 14:04:28 -0700
Subject: [PATCH 03/11] Move computed fields discovery from courier to index
 pattern

---
 .../components/courier/data_source/_abstract.js    | 14 +++++---------
 .../components/index_patterns/_index_pattern.js    |  4 ++++
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/kibana/components/courier/data_source/_abstract.js b/src/kibana/components/courier/data_source/_abstract.js
index 148d714e7bee6..1af63f18e9cb8 100644
--- a/src/kibana/components/courier/data_source/_abstract.js
+++ b/src/kibana/components/courier/data_source/_abstract.js
@@ -264,16 +264,12 @@ define(function (require) {
               'match_all': {}
             };
           }
-          flatState.body.fields = ['*', '_source'];
 
-          _.each(flatState.index.fields.byType['date'], function (field) {
-            if (field.indexed) {
-              flatState.body.script_fields = flatState.body.script_fields || {};
-              flatState.body.script_fields[field.name] = {
-                script: 'doc["' + field.name + '"].value'
-              };
-            }
-          });
+          var computedFields = flatState.index.getComputedFields();
+          flatState.body.fields = computedFields.fields;
+          flatState.body.script_fields = flatState.body.script_fields || {};
+          _.extend(flatState.body.script_fields, computedFields.scriptFields);
+
 
           /**
            * Create a filter that can be reversed for filters with negate set
diff --git a/src/kibana/components/index_patterns/_index_pattern.js b/src/kibana/components/index_patterns/_index_pattern.js
index dc42a019cd4e5..a601fe891e8d6 100644
--- a/src/kibana/components/index_patterns/_index_pattern.js
+++ b/src/kibana/components/index_patterns/_index_pattern.js
@@ -12,6 +12,8 @@ define(function (require) {
     var DocSource = Private(require('components/courier/data_source/doc_source'));
     var flattenSearchResponse = require('components/index_patterns/_flatten_search_response');
     var flattenHit = require('components/index_patterns/_flatten_hit');
+    var getComputedFields = require('components/index_patterns/_get_computed_fields');
+
 
     var IndexedArray = require('utils/indexed_array/index');
 
@@ -199,6 +201,8 @@ define(function (require) {
       self.metaFields = config.get('metaFields');
       self.flattenSearchResponse = flattenSearchResponse.bind(self);
       self.flattenHit = flattenHit.bind(self);
+      self.getComputedFields = getComputedFields.bind(self);
+
 
     }
     return IndexPattern;

From 703d817c042a7a2a4db71d9091cbbe8ed856f835 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 14:04:38 -0700
Subject: [PATCH 04/11] Remove border

---
 src/kibana/components/doc_viewer/doc_viewer.less | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/kibana/components/doc_viewer/doc_viewer.less b/src/kibana/components/doc_viewer/doc_viewer.less
index b3458d3d5e151..a7eefa7b627b9 100644
--- a/src/kibana/components/doc_viewer/doc_viewer.less
+++ b/src/kibana/components/doc_viewer/doc_viewer.less
@@ -14,4 +14,12 @@ doc-viewer .doc-viewer {
     white-space: pre-wrap;
   }
 
+  td, pre {
+    font-family: "Lucida Console", Monaco, monospace;
+  }
+
+  .nav-tabs {
+    border-bottom: 0px !important;
+  }
+
 }
\ No newline at end of file

From 4cda920b8a49c139392052be64b165699119d900 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 14:14:35 -0700
Subject: [PATCH 05/11] Add computed fields tests

---
 .../index_pattern/_get_computed_fields.js     | 34 +++++++++++++++++++
 .../specs/components/index_pattern/index.js   |  2 ++
 2 files changed, 36 insertions(+)
 create mode 100644 test/unit/specs/components/index_pattern/_get_computed_fields.js

diff --git a/test/unit/specs/components/index_pattern/_get_computed_fields.js b/test/unit/specs/components/index_pattern/_get_computed_fields.js
new file mode 100644
index 0000000000000..8a41da9eced70
--- /dev/null
+++ b/test/unit/specs/components/index_pattern/_get_computed_fields.js
@@ -0,0 +1,34 @@
+define(function (require) {
+  return ['get computed fields', function () {
+    var _ = require('lodash');
+
+    var indexPattern, getComputedFields, fn;
+    beforeEach(module('kibana'));
+    beforeEach(inject(function (Private, $injector) {
+      indexPattern = Private(require('fixtures/stubbed_logstash_index_pattern'));
+      getComputedFields = require('components/index_patterns/_get_computed_fields');
+      indexPattern.getComputedFields = getComputedFields.bind(indexPattern);
+      fn = indexPattern.getComputedFields;
+
+    }));
+
+    it('should be a function', function () {
+      expect(fn).to.be.a(Function);
+    });
+
+    it('should request all stored fields', function () {
+      expect(fn().fields).to.contain('*');
+    });
+
+    it('should request _source seperately', function () {
+      expect(fn().fields).to.contain('_source');
+    });
+
+    it('should request date fields as scripts', function () {
+      expect(fn().scriptFields).to.include.keys('@timestamp');
+      expect(fn().scriptFields).to.not.include.keys('bytes');
+    });
+
+
+  }];
+});
\ No newline at end of file
diff --git a/test/unit/specs/components/index_pattern/index.js b/test/unit/specs/components/index_pattern/index.js
index 59fcc306cabe1..f6b55e457ef6b 100644
--- a/test/unit/specs/components/index_pattern/index.js
+++ b/test/unit/specs/components/index_pattern/index.js
@@ -4,6 +4,8 @@ define(function (require) {
     run(require('specs/components/index_pattern/_map_field'));
     run(require('specs/components/index_pattern/_pattern_to_wildcard'));
     run(require('specs/components/index_pattern/_field_formats'));
+    run(require('specs/components/index_pattern/_get_computed_fields'));
+
     function run(mod) { describe(mod[0], mod[1]); }
   });
 });
\ No newline at end of file

From 3b04e446a732d64dcc1e09c9b35d8303c8d185a0 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 14:19:08 -0700
Subject: [PATCH 06/11] Removing unused services

---
 src/kibana/plugins/doc/controllers/doc.js | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/src/kibana/plugins/doc/controllers/doc.js b/src/kibana/plugins/doc/controllers/doc.js
index 33d0714e2ba14..2e125c85a5430 100644
--- a/src/kibana/plugins/doc/controllers/doc.js
+++ b/src/kibana/plugins/doc/controllers/doc.js
@@ -23,21 +23,13 @@ define(function (require) {
     }
   });
 
-  app.controller('doc', function ($scope, courier, $route, Notifier, es) {
+  app.controller('doc', function ($scope, $route, es) {
 
     // Pretty much only need this for formatting, not actually using it for fetching anything.
     $scope.indexPattern = $route.current.locals.indexPattern;
 
-    var notify = new Notifier({
-      location: 'Doc'
-    });
-
     var computedFields = $scope.indexPattern.getComputedFields();
 
-    // Q: Hey jerk, why are you using this instead of just getting a document?
-    // A: No need for name calling. This is the only way we can get the scripted & stored fields.
-    // Q: Then why not use a searchSource?
-    // A: I only need the one index, constructing a temporary index pattern seems wasteful.
     es.search({
       index: $route.current.params.index,
       type: $route.current.params.type,

From 652ff5c8524b20880cc1488f41e22d16989dd63c Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 16:03:50 -0700
Subject: [PATCH 07/11] Add test harness for doc viewer

---
 .../stubbed_logstash_index_pattern.js         |  2 +
 test/unit/specs/apps/doc/doc.js               | 59 +++++++++++++++++++
 2 files changed, 61 insertions(+)
 create mode 100644 test/unit/specs/apps/doc/doc.js

diff --git a/test/unit/fixtures/stubbed_logstash_index_pattern.js b/test/unit/fixtures/stubbed_logstash_index_pattern.js
index d828518f2e3ab..2574f7234e71c 100644
--- a/test/unit/fixtures/stubbed_logstash_index_pattern.js
+++ b/test/unit/fixtures/stubbed_logstash_index_pattern.js
@@ -3,6 +3,7 @@ define(function (require) {
     var StubIndexPattern = Private(require('test_utils/stub_index_pattern'));
     var flattenSearchResponse = require('components/index_patterns/_flatten_search_response');
     var flattenHit = require('components/index_patterns/_flatten_hit');
+    var getComputedFields = require('components/index_patterns/_get_computed_fields');
 
     var _ = require('lodash');
 
@@ -23,6 +24,7 @@ define(function (require) {
       { name: 'custom_user_field',  displayName: 'custom_user_field', type: 'conflict',   indexed: false, analyzed: false,  count: 0 }
     ]);
 
+    indexPattern.getComputedFields = _.bind(getComputedFields, indexPattern);
     indexPattern.flattenSearchResponse = _.bind(flattenSearchResponse, indexPattern);
     indexPattern.flattenHit = _.bind(flattenHit, indexPattern);
     indexPattern.metaFields = ['_id', '_type', '_source'];
diff --git a/test/unit/specs/apps/doc/doc.js b/test/unit/specs/apps/doc/doc.js
new file mode 100644
index 0000000000000..fb321d9d0627c
--- /dev/null
+++ b/test/unit/specs/apps/doc/doc.js
@@ -0,0 +1,59 @@
+define(function (require) {
+  var angular = require('angular');
+  var $ = require('jquery');
+
+  // Load the kibana app dependencies.
+  require('angular-route');
+
+  require('plugins/doc/index');
+
+  var $scope, createController, $route;
+
+  var init = function (index, type, id) {
+
+    module('kibana');
+
+    // Stub services
+    module(function ($provide) {
+      $provide.service('$route', function (Private) {
+        this.current = {
+          locals: {
+            indexPattern: Private(require('fixtures/stubbed_logstash_index_pattern'))
+          },
+          params: {
+            index: index || 'myIndex',
+            type: type || 'myType',
+            id: id || 'myId'
+          }
+        };
+      });
+    });
+
+    // Create the scope
+    inject(function ($rootScope, $controller, _$route_) {
+
+      $route = _$route_;
+      $scope = $rootScope.$new();
+
+      createController = function () {
+        return $controller('doc', {
+          '$scope': $scope
+        });
+      };
+    });
+  };
+
+
+  describe('Doc app controller', function () {
+
+    beforeEach(function () {
+      init();
+    });
+
+    it('should have $scope.foo', function () {
+      createController();
+    });
+
+  });
+
+});

From 6aab30038edf1e8d1fdd0511a65247fd6dae6419 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Thu, 4 Dec 2014 16:24:44 -0700
Subject: [PATCH 08/11] Disable time filter in doc app and add link form
 discover to doc app

---
 src/kibana/plugins/discover/partials/table_row/details.html | 3 +++
 src/kibana/plugins/doc/controllers/doc.js                   | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/kibana/plugins/discover/partials/table_row/details.html b/src/kibana/plugins/discover/partials/table_row/details.html
index bef929434bbab..c3ab47ee219b8 100644
--- a/src/kibana/plugins/discover/partials/table_row/details.html
+++ b/src/kibana/plugins/discover/partials/table_row/details.html
@@ -1,3 +1,6 @@
 <td colspan="{{ columns.length + 2 }}">
+  <a class="pull-right" ng-href="#/doc/{{indexPattern.id}}/{{row._index}}/{{row._type}}/{{row._id}}">
+    <small>Link to /{{row._index}}/{{row._type}}/{{row._id}}</small></i>
+  </a>
   <doc-viewer hit="row" filter="filter" index-pattern="indexPattern"></doc-viewer>
 </td>
\ No newline at end of file
diff --git a/src/kibana/plugins/doc/controllers/doc.js b/src/kibana/plugins/doc/controllers/doc.js
index 2e125c85a5430..1649458d09720 100644
--- a/src/kibana/plugins/doc/controllers/doc.js
+++ b/src/kibana/plugins/doc/controllers/doc.js
@@ -23,7 +23,9 @@ define(function (require) {
     }
   });
 
-  app.controller('doc', function ($scope, $route, es) {
+  app.controller('doc', function ($scope, $route, es, timefilter) {
+
+    timefilter.enabled = false;
 
     // Pretty much only need this for formatting, not actually using it for fetching anything.
     $scope.indexPattern = $route.current.locals.indexPattern;

From faf0a5b622a8ad030d302638d90aad8a5e891461 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Fri, 5 Dec 2014 10:41:13 -0700
Subject: [PATCH 09/11] Add tests for the doc_viewer controller

---
 src/kibana/plugins/doc/controllers/doc.js |  8 ++-
 test/unit/specs/apps/doc/doc.js           | 88 +++++++++++++++++++++--
 2 files changed, 88 insertions(+), 8 deletions(-)

diff --git a/src/kibana/plugins/doc/controllers/doc.js b/src/kibana/plugins/doc/controllers/doc.js
index 1649458d09720..d9c483099b181 100644
--- a/src/kibana/plugins/doc/controllers/doc.js
+++ b/src/kibana/plugins/doc/controllers/doc.js
@@ -54,8 +54,12 @@ define(function (require) {
         }
       }
     }).catch(function (err) {
-      $scope.status = 'error';
-      $scope.resp = err;
+      if (err.status === 404) {
+        $scope.status = 'notFound';
+      } else {
+        $scope.status = 'error';
+        $scope.resp = err;
+      }
     });
 
   });
diff --git a/test/unit/specs/apps/doc/doc.js b/test/unit/specs/apps/doc/doc.js
index fb321d9d0627c..375995fc237df 100644
--- a/test/unit/specs/apps/doc/doc.js
+++ b/test/unit/specs/apps/doc/doc.js
@@ -7,7 +7,7 @@ define(function (require) {
 
   require('plugins/doc/index');
 
-  var $scope, createController, $route;
+  var $scope, createController, $route, timefilter;
 
   var init = function (index, type, id) {
 
@@ -27,33 +27,109 @@ define(function (require) {
           }
         };
       });
+
+      $provide.service('es', function (Private, $q) {
+        this.search = function (config) {
+          var deferred = $q.defer();
+
+          switch (config.index) {
+            case 'goodSearch':
+              deferred.resolve({
+                hits: {
+                  total: 1,
+                  hits: [{
+                    _source: {
+                      foo: true
+                    }
+                  }]
+                }
+              });
+              break;
+            case 'badSearch':
+              deferred.resolve({
+                hits: {
+                  total: 0,
+                  hits: []
+                }
+              });
+              break;
+            case 'missingIndex':
+              deferred.reject({status: 404});
+              break;
+            case 'badRequest':
+              deferred.reject({status: 500});
+              break;
+          }
+
+          return deferred.promise;
+        };
+      });
     });
 
     // Create the scope
-    inject(function ($rootScope, $controller, _$route_) {
+    inject(function ($rootScope, $controller, _$route_, _timefilter_) {
 
       $route = _$route_;
       $scope = $rootScope.$new();
 
+      timefilter = _timefilter_;
+
       createController = function () {
         return $controller('doc', {
           '$scope': $scope
         });
       };
     });
+
+    createController();
   };
 
 
   describe('Doc app controller', function () {
 
-    beforeEach(function () {
-      init();
+    it('should set status=found if the document was found', function (done) {
+      init('goodSearch');
+      $scope.$digest();
+      expect($scope.status).to.be('found');
+      done();
     });
 
-    it('should have $scope.foo', function () {
-      createController();
+    it('should attach the hit to scope', function (done) {
+      init('goodSearch');
+      $scope.$digest();
+      expect($scope.hit).to.be.an(Object);
+      expect($scope.hit._source).to.be.an(Object);
+      done();
     });
 
+    it('should set status=notFound if the document was missing', function (done) {
+      init('badSearch');
+      $scope.$digest();
+      expect($scope.status).to.be('notFound');
+      done();
+    });
+
+    it('should set status=notFound if the request returns a 404', function (done) {
+      init('missingIndex');
+      $scope.$digest();
+      expect($scope.status).to.be('notFound');
+      done();
+    });
+
+    it('should set status=error if the request fails with any other code', function (done) {
+      init('badRequest');
+      $scope.$digest();
+      expect($scope.status).to.be('error');
+      done();
+    });
+
+    it('should disable the time filter', function (done) {
+      init();
+      expect(timefilter.enabled).to.be(false);
+      done();
+    });
+
+
   });
 
 });

From 2bddc7bf4fb616c885ab5321723a310dd46df0c3 Mon Sep 17 00:00:00 2001
From: Joe Fleming <joe.fleming@gmail.com>
Date: Fri, 5 Dec 2014 11:17:49 -0700
Subject: [PATCH 10/11] fix dov viewer tab style

---
 .../components/doc_viewer/doc_viewer.html     | 68 ++++++++++---------
 .../components/doc_viewer/doc_viewer.less     |  5 +-
 .../index_patterns/_get_computed_fields.js    |  1 +
 3 files changed, 38 insertions(+), 36 deletions(-)

diff --git a/src/kibana/components/doc_viewer/doc_viewer.html b/src/kibana/components/doc_viewer/doc_viewer.html
index 7c91f0079ca66..c7b7e56a37c17 100644
--- a/src/kibana/components/doc_viewer/doc_viewer.html
+++ b/src/kibana/components/doc_viewer/doc_viewer.html
@@ -4,39 +4,41 @@
     <li ng-class="{active: mode == 'json'}"><a ng-click="mode='json'">JSON</a></li>
   </ul>
 
-  <table class="table table-condensed" ng-show="mode == 'table'" bindonce>
-    <tbody>
-      <tr ng-repeat="field in fields" bindonce>
-        <td field-name="field"
-            field-type="mapping[field].type"
-            width="1%"
-            class="doc-viewer-field">
-        </td>
-        <td width="1%" class="doc-viewer-buttons" ng-if="filter">
-          <span bo-if="showFilters(mapping[field])">
-            <i ng-click="filter(field, flattened[field], '+')" class="fa fa-search-plus"></i>
-            <i ng-click="filter(field, flattened[field],'-')" class="fa fa-search-minus"></i>
-          </span>
-          <span bo-if="!showFilters(mapping[field])" tooltip="Unindexed fields can not be searched">
-            <i class="fa fa-search-plus text-muted"></i>
-            <i class="fa fa-search-minus text-muted"></i>
-          </span>
-        </td>
+  <div class="content">
+    <table class="table table-condensed" ng-show="mode == 'table'" bindonce>
+      <tbody>
+        <tr ng-repeat="field in fields" bindonce>
+          <td field-name="field"
+              field-type="mapping[field].type"
+              width="1%"
+              class="doc-viewer-field">
+          </td>
+          <td width="1%" class="doc-viewer-buttons" ng-if="filter">
+            <span bo-if="showFilters(mapping[field])">
+              <i ng-click="filter(field, flattened[field], '+')" class="fa fa-search-plus"></i>
+              <i ng-click="filter(field, flattened[field],'-')" class="fa fa-search-minus"></i>
+            </span>
+            <span bo-if="!showFilters(mapping[field])" tooltip="Unindexed fields can not be searched">
+              <i class="fa fa-search-plus text-muted"></i>
+              <i class="fa fa-search-minus text-muted"></i>
+            </span>
+          </td>
 
-        <td>
-          <i bo-if="!mapping[field] && !showArrayInObjectsWarning(doc, field)"
-            tooltip-placement="top"
-            tooltip="No cached mapping for this field. Refresh your mapping from the Settings > Indices page"
-            class="fa fa-warning text-color-warning ng-scope doc-viewer-no-mapping"></i>
-          <i bo-if="showArrayInObjectsWarning(doc, field)"
-            tooltip-placement="top"
-            tooltip="Objects in arrays are not well supported."
-            class="fa fa-warning text-color-warning ng-scope doc-viewer-object-array"></i>
-          <span class="doc-viewer-value" ng-bind-html="(formatted[field] || hit[field]) | highlight : hit.highlight[field] | trustAsHtml"></span>
-        </td>
-      </tr>
-    </tbody>
-  </table>
+          <td>
+            <i bo-if="!mapping[field] && !showArrayInObjectsWarning(doc, field)"
+              tooltip-placement="top"
+              tooltip="No cached mapping for this field. Refresh your mapping from the Settings > Indices page"
+              class="fa fa-warning text-color-warning ng-scope doc-viewer-no-mapping"></i>
+            <i bo-if="showArrayInObjectsWarning(doc, field)"
+              tooltip-placement="top"
+              tooltip="Objects in arrays are not well supported."
+              class="fa fa-warning text-color-warning ng-scope doc-viewer-object-array"></i>
+            <span class="doc-viewer-value" ng-bind-html="(formatted[field] || hit[field]) | highlight : hit.highlight[field] | trustAsHtml"></span>
+          </td>
+        </tr>
+      </tbody>
+    </table>
 
-  <pre ng-show="mode == 'json'">{{hit | json}}</pre>
+    <pre ng-show="mode == 'json'">{{hit | json}}</pre>
+  </div>
 </div>
\ No newline at end of file
diff --git a/src/kibana/components/doc_viewer/doc_viewer.less b/src/kibana/components/doc_viewer/doc_viewer.less
index a7eefa7b627b9..321d297bf31d5 100644
--- a/src/kibana/components/doc_viewer/doc_viewer.less
+++ b/src/kibana/components/doc_viewer/doc_viewer.less
@@ -18,8 +18,7 @@ doc-viewer .doc-viewer {
     font-family: "Lucida Console", Monaco, monospace;
   }
 
-  .nav-tabs {
-    border-bottom: 0px !important;
+  .content {
+    margin-top: @padding-base-vertical;
   }
-
 }
\ No newline at end of file
diff --git a/src/kibana/components/index_patterns/_get_computed_fields.js b/src/kibana/components/index_patterns/_get_computed_fields.js
index 3cfd8ea5ad6bc..dda200fdbbb63 100644
--- a/src/kibana/components/index_patterns/_get_computed_fields.js
+++ b/src/kibana/components/index_patterns/_get_computed_fields.js
@@ -13,6 +13,7 @@ define(function (require) {
         };
       }
     });
+
     return {
       fields: ['*', '_source'],
       scriptFields: scriptFields

From a755b738aadac24ba4ba10dd959eddfc03425bd8 Mon Sep 17 00:00:00 2001
From: Rashid Khan <rashidkpc@gmail.com>
Date: Fri, 5 Dec 2014 11:28:26 -0700
Subject: [PATCH 11/11] Remove unused less

---
 src/kibana/plugins/doc/index.js         | 1 -
 src/kibana/plugins/doc/styles/main.less | 7 -------
 2 files changed, 8 deletions(-)
 delete mode 100644 src/kibana/plugins/doc/styles/main.less

diff --git a/src/kibana/plugins/doc/index.js b/src/kibana/plugins/doc/index.js
index daa8473a12e73..70b8ab3c75a41 100644
--- a/src/kibana/plugins/doc/index.js
+++ b/src/kibana/plugins/doc/index.js
@@ -1,6 +1,5 @@
 define(function (require, module, exports) {
   require('plugins/doc/controllers/doc');
-  require('css!plugins/doc/styles/main.css');
 
   var apps = require('registry/apps');
   apps.register(function DocAppModule() {
diff --git a/src/kibana/plugins/doc/styles/main.less b/src/kibana/plugins/doc/styles/main.less
deleted file mode 100644
index ae2a89ba62f84..0000000000000
--- a/src/kibana/plugins/doc/styles/main.less
+++ /dev/null
@@ -1,7 +0,0 @@
-@import (reference) "../../../styles/_bootstrap.less";
-@import (reference) "../../../styles/theme/_theme.less";
-@import (reference) "../../../styles/_variables.less";
-
-.application-doc {
-  // Styles here
-}
\ No newline at end of file