From 8099fd94c5f59f2e0ec94d24783ec841123d710e Mon Sep 17 00:00:00 2001
From: Olivier Lamy <olamy@apache.org>
Date: Fri, 10 Feb 2012 17:23:36 +0000
Subject: [PATCH] [MRM-1576] rewrite proxies connector page add edition too.

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1242861 13f79535-47bb-0310-9956-ffa450edef68
---
 .../apache/archiva/i18n/default.properties    |  1 +
 .../webapp/js/archiva/proxy-connectors.js     | 81 +++++++++++++------
 .../archiva/templates/proxy-connectors.html   |  6 +-
 3 files changed, 57 insertions(+), 31 deletions(-)

diff --git a/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties b/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties
index 3dc72f980..943441cf7 100644
--- a/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties
+++ b/archiva-modules/archiva-web/archiva-web-common/src/main/resources/org/apache/archiva/i18n/default.properties
@@ -138,6 +138,7 @@ proxy-connector.blacklist=Black List
 proxy-connector.whitelist=White List
 proxyconnector.removed=ProxyConnector removed.
 proxyconnector.added=ProxyConnector added.
+proxyconnector.updated=ProxyConnector updated.
 proxy-connectors.grid.remoterepo.settings.popover.title=Proxy Connector Settings
 proxy-connectors.remoterepo.settings.networkproxy=Network Proxy
 
diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/proxy-connectors.js b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/proxy-connectors.js
index 6b0f89ffe..523e9fd96 100644
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/proxy-connectors.js
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/proxy-connectors.js
@@ -138,6 +138,7 @@ $(function() {
     this.proxyConnectorsViewModel=proxyConnectorsViewModel;
     this.update=update;
     this.modified=ko.observable(false);
+
     getSelectedPolicyOption=function(id){
       $.log("getSelectedPolicyOption:"+id);
 
@@ -145,8 +146,9 @@ $(function() {
       $.log("getSelectedPolicyOption policies.length:"+policiesEntries.length);
       if (policiesEntries!=null){
         for (i=0;i<policiesEntries.length;i++){
-          if (id==policiesEntries[i].key()){
-            return policiesEntries[i].value();
+          var curKey = $.isFunction(policiesEntries[i].key)? policiesEntries[i].key():policiesEntries[i].key;
+          if (id==curKey){
+            return $.isFunction(policiesEntries[i].value)? policiesEntries[i].value():policiesEntries[i].value;
           }
         }
       }
@@ -190,11 +192,26 @@ $(function() {
     save=function(){
       //FIXME data controls !!!
       clearUserMessages();
+      // update is delete then add
       if (this.update){
-
+        $.ajax("restServices/archivaServices/proxyConnectorService/updateProxyConnector",
+          {
+            type: "POST",
+            data: "{\"proxyConnector\": " + ko.toJSON(self.proxyConnector)+"}",
+            contentType: 'application/json',
+            dataType: 'json',
+            success: function(data) {
+              displaySuccessMessage($.i18n.prop('proxyconnector.updated'));
+              activateProxyConnectorsGridTab();
+              self.proxyConnector.modified(false);
+            },
+            error: function(data) {
+              var res = $.parseJSON(data.responseText);
+              displayRestError(res);
+            }
+          }
+        );
       } else {
-        var json = $.toJSON(ko.toJS(self.proxyConnector));
-        $.log("toJSON:"+json);
 
         $.ajax("restServices/archivaServices/proxyConnectorService/addProxyConnector",
           {
@@ -214,7 +231,6 @@ $(function() {
             }
           }
         );
-
       }
     }
 
@@ -231,29 +247,21 @@ $(function() {
     this.remoteRepositories=ko.observableArray([]);
     this.networkProxies=ko.observableArray([]);
 
-    editProxyConnector=function(managedRepositoryConnectorView){
-      $.log("editProxyConnector");
+    editProxyConnector=function(proxyConnector){
+      var proxyConnectorViewModel=new ProxyConnectorViewModel(proxyConnector,true,self);
+      var mainContent = $("#main-content");
+      mainContent.find("#proxy-connectors-edit").html($("#proxy-connector-edit-form-tmpl").tmpl());
+      ko.applyBindings(proxyConnectorViewModel,mainContent.find("#proxy-connectors-edit").get(0));
+      activateProxyConnectorsEditTab();
     }
     deleteProxyConnector=function(proxyConnector){
       clearUserMessages();
-      var url="restServices/archivaServices/proxyConnectorService/removeProxyConnector?";
-      url += "sourceRepoId="+encodeURIComponent(proxyConnector.sourceRepoId());
-      url += "&targetRepoId="+encodeURIComponent(proxyConnector.targetRepoId());
-      $.ajax(url,
-        {
-          type: "GET",
-          contentType: 'application/json',
-          success: function(data) {
-            displaySuccessMessage($.i18n.prop('proxyconnector.removed'));
-            self.proxyConnectors.remove(proxyConnector);
-            self.displayGrid();
-          },
-          error: function(data) {
-            var res = $.parseJSON(data.responseText);
-            displayRestError(res);
-          }
-        }
-      );
+      removeProxyConnector(proxyConnector,function(){
+        displaySuccessMessage($.i18n.prop('proxyconnector.removed'));
+        self.proxyConnectors.remove(proxyConnector);
+        //self.displayGrid();
+      });
+
 
     }
 
@@ -463,4 +471,25 @@ $(function() {
            });
   }
 
+  removeProxyConnector=function(proxyConnector,fnSuccessCallback){
+    clearUserMessages();
+    var url="restServices/archivaServices/proxyConnectorService/removeProxyConnector?";
+    url += "sourceRepoId="+encodeURIComponent(proxyConnector.sourceRepoId());
+    url += "&targetRepoId="+encodeURIComponent(proxyConnector.targetRepoId());
+    $.ajax(url,
+      {
+        type: "GET",
+        contentType: 'application/json',
+        success: function(data) {
+          fnSuccessCallback();
+        },
+        error: function(data) {
+          var res = $.parseJSON(data.responseText);
+          displayRestError(res);
+        }
+      }
+    );
+
+  }
+
 });
diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/proxy-connectors.html b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/proxy-connectors.html
index 5412ac869..712816d69 100644
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/proxy-connectors.html
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/proxy-connectors.html
@@ -98,11 +98,7 @@
         {{each(i, entry) policiesEntries}}
         <tr>
           <td>${entry.key}</td>
-          {{if $.isFunction( entry.value ) }}
-            <td>function=====${entry.value()}</td>
-          {{else}}
-            <td>${entry.value}</td>
-          {{/if}}
+          <td>${entry.value}</td>
         </tr>
         {{/each}}
       </tbody>
-- 
2.39.5