From 95619b42fb7081352d84ddbcf80202363dc94264 Mon Sep 17 00:00:00 2001
From: Piotr Sarnacki <drogus@gmail.com>
Date: Fri, 6 Feb 2015 13:20:56 +0100
Subject: [PATCH] Don't filter env vars in controller

Depending on isNew can result in rerendering templates in some cases, which will
also create a new object controller, which in turn will reset isEditing value.
---
 app/controllers/env-vars.coffee |  2 +-
 app/templates/env-vars.hbs      | 26 ++++++++++++++------------
 2 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/app/controllers/env-vars.coffee b/app/controllers/env-vars.coffee
index fbd940a4..2580ccad 100644
--- a/app/controllers/env-vars.coffee
+++ b/app/controllers/env-vars.coffee
@@ -1,6 +1,6 @@
 `import Ember from 'ember'`
 
 Controller = Ember.ArrayController.extend
-  vars: Ember.computed.filterBy('model', 'isNew', false)
+  vars: Ember.computed.alias('model')
 
 `export default Controller`
diff --git a/app/templates/env-vars.hbs b/app/templates/env-vars.hbs
index 7e5f06d8..152ebe68 100644
--- a/app/templates/env-vars.hbs
+++ b/app/templates/env-vars.hbs
@@ -2,17 +2,19 @@
 
 <ul class="env-vars">
   {{#each vars itemController="envVar"}}
-    <li class="env-var">
-      {{#if isEditing}}
-        {{partial 'env_vars/form'}}
-      {{else}}
-        <a href="#" class="edit-var" {{action "edit"}}>Edit</a>
-        <a href="#" {{action "delete"}} {{bind-attr class=":delete-var isDeleting:deleting"}}>
-          Delete
-        </a>
-        <span class="name">{{name}}</span>
-        <span {{bind-attr class=":value :value-display public::secure"}}>{{value}}</span>
-      {{/if}}
-    </li>
+    {{#unless isNew}}
+      <li class="env-var">
+        {{#if isEditing}}
+          {{partial 'env_vars/form'}}
+        {{else}}
+          <a href="#" class="edit-var" {{action "edit"}}>Edit</a>
+          <a href="#" {{action "delete"}} {{bind-attr class=":delete-var isDeleting:deleting"}}>
+            Delete
+          </a>
+          <span class="name">{{name}}</span>
+          <span {{bind-attr class=":value :value-display public::secure"}}>{{value}}</span>
+        {{/if}}
+      </li>
+    {{/unless}}
   {{/each}}
 </ul>