From 09f2836dd91d15297ef80de4a0effcbe76a5ef83 Mon Sep 17 00:00:00 2001
From: Jessica Wang <jessica@cloud.com>
Date: Tue, 19 Oct 2010 17:37:55 -0700
Subject: [PATCH] new UI - ISO page - remove edit button, add Edit ISO link
 into action menu

---
 ui/new/jsp/iso.jsp                     |  5 +---
 ui/new/scripts/cloud.core2.iso.js      | 39 ++++++++++++++++++++------
 ui/new/scripts/cloud.core2.template.js |  1 -
 3 files changed, 32 insertions(+), 13 deletions(-)

diff --git a/ui/new/jsp/iso.jsp b/ui/new/jsp/iso.jsp
index b1133d781e9..fdc2b8362b2 100644
--- a/ui/new/jsp/iso.jsp
+++ b/ui/new/jsp/iso.jsp
@@ -24,10 +24,7 @@
             <%=t.t("Details")%></div>
     </div>
     <div id="tab_content_details">
-        <div class="grid_actionpanel">
-            
-            <div class="grid_editbox" id="edit_button">
-            </div>
+        <div class="grid_actionpanel">            
             <div class="gridheader_loaderbox" id="spinning_wheel" style="border: 1px solid #999;
                 display: none;">
                 <div class="gridheader_loader" id="Div1">
diff --git a/ui/new/scripts/cloud.core2.iso.js b/ui/new/scripts/cloud.core2.iso.js
index ede54a28d72..e7ffe19082a 100644
--- a/ui/new/scripts/cloud.core2.iso.js
+++ b/ui/new/scripts/cloud.core2.iso.js
@@ -84,11 +84,6 @@ function afterLoadIsoJSP() {
         return false;
     });
     
-    //edit button ***
-    var $readonlyFields  = $detailsTab.find("#name, #displaytext");
-    var $editFields = $detailsTab.find("#name_edit, #displaytext_edit"); 
-    initializeEditFunction($readonlyFields, $editFields, doUpdateIso); 
-       
     //populate dropdown ***
     var addIsoZoneField = $("#dialog_add_iso #add_iso_zone");    	
 	if (isAdmin())  
@@ -219,10 +214,11 @@ function isoJsonToDetailsTab(jsonObj) {
     
     // "Edit", "Copy", "Create VM" 
 	if ((isUser() && jsonObj.ispublic == "true" && !(jsonObj.domainid == g_domainid && jsonObj.account == g_account)) || jsonObj.isready == "false") {		
-		$("#edit_button").hide();
+		//$("#edit_button").hide();
     }
     else {        
-        $("#edit_button").show();
+        buildActionLinkForDetailsTab("Edit ISO", isoActionMap, $actionMenu, midmenuId);		
+        //$("#edit_button").show();
         buildActionLinkForDetailsTab("Copy ISO", isoActionMap, $actionMenu, midmenuId);		
         noAvailableActions = false;
     }
@@ -274,6 +270,9 @@ function isoClearDetailsTab() {
 }
 
 var isoActionMap = {  
+    "Edit ISO": {
+        dialogBeforeActionFn : doEditISO  
+    },
     "Delete ISO": {                  
         isAsyncJob: true,
         asyncJobResponse: "deleteisosresponse",
@@ -303,7 +302,31 @@ var isoActionMap = {
     }  
 }   
 
-function doUpdateIso() { 
+function doEditISO() {   
+    var $detailsTab = $("#right_panel_content #tab_content_details");  
+    var $readonlyFields  = $detailsTab.find("#name, #displaytext");
+    var $editFields = $detailsTab.find("#name_edit, #displaytext_edit"); 
+           
+    $readonlyFields.hide();
+    $editFields.show();  
+    $detailsTab.find("#cancel_button, #save_button").show();
+    
+    $detailsTab.find("#cancel_button").unbind("click").bind("click", function(event){    
+        $editFields.hide();
+        $readonlyFields.show();   
+        $("#save_button, #cancel_button").hide();       
+        return false;
+    });
+    $detailsTab.find("#save_button").unbind("click").bind("click", function(event){        
+        doEditISO2($detailsTab);     
+        $editFields.hide();      
+        $readonlyFields.show();       
+        $("#save_button, #cancel_button").hide();       
+        return false;
+    });   
+}
+
+function doEditISO2($detailsTab) { 
     var $detailsTab = $("#right_panel_content #tab_content_details");      
     
     // validate values
diff --git a/ui/new/scripts/cloud.core2.template.js b/ui/new/scripts/cloud.core2.template.js
index b5b1f881e0a..60156687361 100644
--- a/ui/new/scripts/cloud.core2.template.js
+++ b/ui/new/scripts/cloud.core2.template.js
@@ -280,7 +280,6 @@ function templateJsonToDetailsTab(jsonObj) {
     
     // action Edit, Copy, Create VM 			
 	if ((isUser() && jsonObj.ispublic == "true" && !(jsonObj.domainid == g_domainid && jsonObj.account == g_account)) || jsonObj.id==DomRTemplateId || jsonObj.isready == "false") {
-		//template.find("#template_edit_container, #template_copy_container, #template_create_vm_container").hide(); 
 		//$("#edit_button").hide();		
     }
     else {