From dd08ca723ce986673ca08532edfb257698cf4ca5 Mon Sep 17 00:00:00 2001
From: Kelly Domico <kelly.domico@hp.com>
Date: Mon, 23 Nov 2015 17:16:35 -0800
Subject: [PATCH] Fix flavor quota charts in NG Launch Instance

The flavor quota charts aren't rendering correctly
when the instance count is changed in the Angular
Launch Instance modal. The <svg> element was not
available to add slices as 'chartData.showChart'
is originally undefined.

This patches fixes the the issue by using ng-show
instead of ng-if.

Change-Id: I7b24da7942c3d02a55adc732d70a26aeb5b623ec
Closes-Bug: #1518030
---
 horizon/static/framework/widgets/charts/pie-chart.html    | 2 +-
 horizon/static/framework/widgets/charts/pie-chart.spec.js | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/horizon/static/framework/widgets/charts/pie-chart.html b/horizon/static/framework/widgets/charts/pie-chart.html
index 1a120b99b2..b47be287cc 100644
--- a/horizon/static/framework/widgets/charts/pie-chart.html
+++ b/horizon/static/framework/widgets/charts/pie-chart.html
@@ -7,7 +7,7 @@
   </div>
 
   <svg class="svg-pie-chart"
-    ng-if="chartData.showChart"
+    ng-show="chartData.showChart"
     ng-attr-height="{$ ::model.settings.diameter $}"
     ng-attr-width="{$ ::model.settings.diameter $}">
     <g class="chart"
diff --git a/horizon/static/framework/widgets/charts/pie-chart.spec.js b/horizon/static/framework/widgets/charts/pie-chart.spec.js
index 6861000b4c..298a6a3f00 100644
--- a/horizon/static/framework/widgets/charts/pie-chart.spec.js
+++ b/horizon/static/framework/widgets/charts/pie-chart.spec.js
@@ -138,8 +138,8 @@
       expect($elementTotal.find('svg').length).toBe(1);
     });
 
-    it('Unlimited quota chart should have no svg element', function () {
-      expect($elementNoQuota.find('svg').length).toBe(0);
+    it('Unlimited quota chart should have hidden svg element', function () {
+      expect($elementNoQuota.find('svg').is(':hidden')).toBe(true);
     });
 
     it('Max chart should have 3 path elements', function () {