From d53b6373de504fe4391f332ab5e73554989151c5 Mon Sep 17 00:00:00 2001
From: Brian Federle <brian.federle@citrix.com>
Date: Thu, 23 Aug 2012 11:06:00 -0700
Subject: [PATCH] CS-16144: Show warning before enabling local storage in zone
 wizard

reviewed-by: jessica
---
 .../classes/resources/messages.properties     |  1 +
 ui/dictionary.jsp                             |  1 +
 ui/scripts/zoneWizard.js                      | 21 ++++++++++++++++++-
 3 files changed, 22 insertions(+), 1 deletion(-)

diff --git a/client/WEB-INF/classes/resources/messages.properties b/client/WEB-INF/classes/resources/messages.properties
index 786569a3205..ce82c4aa97b 100644
--- a/client/WEB-INF/classes/resources/messages.properties
+++ b/client/WEB-INF/classes/resources/messages.properties
@@ -1,4 +1,5 @@
 #new labels (begin) **********************************************************************************************
+message.zoneWizard.enable.local.storage=WARNING: If you enable local storage for this zone, you must do the following, depending on where you would like your system VMs to launch:<br/><br/>1. If system VMs need to be launched in primary storage, primary storage needs to be added to the zone after creation. You must also start the zone in a disabled state.<br/><br/>2. If system VMs need to be launched in local storage, system.vm.use.local.storage needs to be set to true before you enable the zone.<br/><br/><br/>Would you like to continue?
 label.local.storage.enabled=Local storage enabled
 label.tier.details=Tier details
 label.edit.tags=Edit tags
diff --git a/ui/dictionary.jsp b/ui/dictionary.jsp
index 32028f7263c..899a2846077 100644
--- a/ui/dictionary.jsp
+++ b/ui/dictionary.jsp
@@ -7,6 +7,7 @@
 <% long now = System.currentTimeMillis(); %>
 <script language="javascript">
 dictionary = {
+'message.zoneWizard.enable.local.storage': '<fmt:message key="message.zoneWizard.enable.local.storage"/>',
 'label.local.storage.enabled': '<fmt:message key="label.local.storage.enabled"/>',
 'label.tier.details': '<fmt:message key="label.tier.details"/>',
 'label.edit.tags': '<fmt:message key="label.edit.tags"/>',
diff --git a/ui/scripts/zoneWizard.js b/ui/scripts/zoneWizard.js
index 3170e0d0719..0aa46d5555e 100644
--- a/ui/scripts/zoneWizard.js
+++ b/ui/scripts/zoneWizard.js
@@ -492,7 +492,26 @@
           },
           localstorageenabled: {
             label: 'label.local.storage.enabled',
-            isBoolean: true
+            isBoolean: true,
+            onChange: function(args) {
+              var $checkbox = args.$checkbox;
+
+              if ($checkbox.is(':checked')) {
+                cloudStack.dialog.confirm({
+                  message: 'message.zoneWizard.enable.local.storage',
+                  action: function() {
+                    $checkbox.attr('checked', true);
+                  },
+                  cancelAction: function() {
+                    $checkbox.attr('checked', false);
+                  }
+                });
+
+                return false;
+              }
+
+              return true;
+            }
           }
         }
       },