From 43c2bd183c479b500eaec98cff2a3d2d34f4bf73 Mon Sep 17 00:00:00 2001
From: Jessica Forrester <jforrest@redhat.com>
Date: Wed, 15 Apr 2015 13:17:52 -0400
Subject: [PATCH] Move pod template describing into its own directive

---
 bower.json                        |  2 +-
 dist/object-describer.js          | 36 +++++++++++++++++++++----------
 object-describer.js               |  9 ++++++++
 views/pod-template.html           | 11 ++++++++++
 views/replication-controller.html | 12 +----------
 5 files changed, 47 insertions(+), 23 deletions(-)
 create mode 100644 views/pod-template.html

diff --git a/bower.json b/bower.json
index a3bb72d..482b8b0 100644
--- a/bower.json
+++ b/bower.json
@@ -1,6 +1,6 @@
 {
   "name": "kubernetes-object-describer",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "Provides an extendable describer for kubernetes resources. Includes an angular directive for rendering an instance of a describer.",
   "moduleType": [
     "globals"
diff --git a/dist/object-describer.js b/dist/object-describer.js
index f7ce40c..3b7cd2a 100644
--- a/dist/object-describer.js
+++ b/dist/object-describer.js
@@ -96,6 +96,15 @@ angular.module('kubernetesUI')
     templateUrl: 'views/footer.html'
   }
 })
+.directive("kubernetesObjectDescribePodTemplate", function() {
+  return {
+    restrict: 'E',
+    scope: {
+      template: '='
+    },
+    templateUrl: 'views/pod-template.html'
+  }
+})
 .directive("kubernetesObjectDescribeVolumes", function() {
   return {
     restrict: 'E',
@@ -192,6 +201,21 @@ angular.module('kubernetesUI').run(['$templateCache', function($templateCache) {
   );
 
 
+  $templateCache.put('views/pod-template.html',
+    "<h3>Pod Template</h3>\n" +
+    "<dl class=\"dl-horizontal\">\n" +
+    "  <dt>Restart policy</dt>\n" +
+    "  <dd>{{template.restartPolicy}}</dd>\n" +
+    "  <dt>DNS policy</dt>\n" +
+    "  <dd>{{template.dnsPolicy}}</dd>\n" +
+    "</dl>  \n" +
+    "<h4>Containers</h4>\n" +
+    "<kubernetes-object-describe-containers containers=\"template.containers\"></kubernetes-object-describe-containers>\n" +
+    "<h4>Volumes</h4>\n" +
+    "<kubernetes-object-describe-volumes volumes=\"template.volumes\"></kubernetes-object-describe-volumes> "
+  );
+
+
   $templateCache.put('views/pod.html',
     "<div>\n" +
     "  <kubernetes-object-describe-header resource=\"resource\"></kubernetes-object-describe-header>\n" +
@@ -243,17 +267,7 @@ angular.module('kubernetesUI').run(['$templateCache', function($templateCache) {
     "    <dt ng-repeat-start=\"(selectorKey, selectorValue) in resource.spec.selector\">{{selectorKey}}</dt>\n" +
     "    <dd ng-repeat-end>{{selectorValue}}</dd>\n" +
     "  </dl>\n" +
-    "  <h3>Pod Template</h3>\n" +
-    "  <dl class=\"dl-horizontal\">\n" +
-    "    <dt>Restart policy</dt>\n" +
-    "    <dd>{{resource.spec.template.spec.restartPolicy}}</dd>\n" +
-    "    <dt>DNS policy</dt>\n" +
-    "    <dd>{{resource.spec.template.spec.dnsPolicy}}</dd>\n" +
-    "  </dl>  \n" +
-    "  <h4>Containers</h4>\n" +
-    "  <kubernetes-object-describe-containers containers=\"resource.spec.template.spec.containers\"></kubernetes-object-describe-containers>\n" +
-    "  <h4>Volumes</h4>\n" +
-    "  <kubernetes-object-describe-volumes volumes=\"resource.spec.template.spec.volumes\"></kubernetes-object-describe-volumes>\n" +
+    "  <kubernetes-object-describe-pod-template template=\"resource.spec.template.spec\"></kubernetes-object-describe-pod-template>\n" +
     "  <kubernetes-object-describe-labels resource=\"resource\"></kubernetes-object-describe-labels>\n" +
     "  <kubernetes-object-describe-annotations resource=\"resource\"></kubernetes-object-describe-annotations>\n" +
     "  <kubernetes-object-describe-footer resource=\"resource\"></kubernetes-object-describe-footer>\n" +
diff --git a/object-describer.js b/object-describer.js
index 64b9c78..db083b2 100644
--- a/object-describer.js
+++ b/object-describer.js
@@ -96,6 +96,15 @@ angular.module('kubernetesUI')
     templateUrl: 'views/footer.html'
   }
 })
+.directive("kubernetesObjectDescribePodTemplate", function() {
+  return {
+    restrict: 'E',
+    scope: {
+      template: '='
+    },
+    templateUrl: 'views/pod-template.html'
+  }
+})
 .directive("kubernetesObjectDescribeVolumes", function() {
   return {
     restrict: 'E',
diff --git a/views/pod-template.html b/views/pod-template.html
new file mode 100644
index 0000000..94aeada
--- /dev/null
+++ b/views/pod-template.html
@@ -0,0 +1,11 @@
+<h3>Pod Template</h3>
+<dl class="dl-horizontal">
+  <dt>Restart policy</dt>
+  <dd>{{template.restartPolicy}}</dd>
+  <dt>DNS policy</dt>
+  <dd>{{template.dnsPolicy}}</dd>
+</dl>  
+<h4>Containers</h4>
+<kubernetes-object-describe-containers containers="template.containers"></kubernetes-object-describe-containers>
+<h4>Volumes</h4>
+<kubernetes-object-describe-volumes volumes="template.volumes"></kubernetes-object-describe-volumes> 
\ No newline at end of file
diff --git a/views/replication-controller.html b/views/replication-controller.html
index 0d89127..6016f45 100644
--- a/views/replication-controller.html
+++ b/views/replication-controller.html
@@ -15,17 +15,7 @@ <h3>Selector</h3>
     <dt ng-repeat-start="(selectorKey, selectorValue) in resource.spec.selector">{{selectorKey}}</dt>
     <dd ng-repeat-end>{{selectorValue}}</dd>
   </dl>
-  <h3>Pod Template</h3>
-  <dl class="dl-horizontal">
-    <dt>Restart policy</dt>
-    <dd>{{resource.spec.template.spec.restartPolicy}}</dd>
-    <dt>DNS policy</dt>
-    <dd>{{resource.spec.template.spec.dnsPolicy}}</dd>
-  </dl>  
-  <h4>Containers</h4>
-  <kubernetes-object-describe-containers containers="resource.spec.template.spec.containers"></kubernetes-object-describe-containers>
-  <h4>Volumes</h4>
-  <kubernetes-object-describe-volumes volumes="resource.spec.template.spec.volumes"></kubernetes-object-describe-volumes>
+  <kubernetes-object-describe-pod-template template="resource.spec.template.spec"></kubernetes-object-describe-pod-template>
   <kubernetes-object-describe-labels resource="resource"></kubernetes-object-describe-labels>
   <kubernetes-object-describe-annotations resource="resource"></kubernetes-object-describe-annotations>
   <kubernetes-object-describe-footer resource="resource"></kubernetes-object-describe-footer>