minor cleanup on selector usage

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1298566 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js
index 985f0323b9b3bff334132705e8f87776bd647471..f060100f0424c36242bbe6ec2b6bc8bc6970926a 100644 (file)
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/main.js
@@ -82,6 +82,10 @@ $(function() {
     var screen = $.urlParam('screen');
 
     if(screen){
+      if(screen=='network-proxies'&& hasKarma('archiva-manage-configuration')){
+        displayNetworkProxies();
+        return;
+      }
       if(screen=='proxy-connectors'&& hasKarma('archiva-manage-configuration')){
         displayProxyConnectors();
         return;

diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/network-proxies.js b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/network-proxies.js
index 1127d1e9a32d3040e0d768d5cbd6539fc34482e4..eea73dddbb01e9886545c29feae1a0c71523ba32 100644 (file)
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/network-proxies.js
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/network-proxies.js
@@ -270,7 +270,8 @@ $(function() {
   }
 
   activateNetworkProxyFormValidation=function(){
-    var validator = $("#main-content #network-proxy-edit-form").validate({
+    var editForm=$("#main-content #network-proxy-edit-form");
+    var validator = editForm.validate({
       rules: {id: {
        required: true,
        remote: {
@@ -279,7 +280,7 @@ $(function() {
        }
       }},
       showErrors: function(validator, errorMap, errorList) {
-       customShowError("#main-content #network-proxy-edit-form",validator,errorMap,errorMap);
+       customShowError(editForm,validator,errorMap,errorMap);
       }
     });
     validator.settings.messages["id"]=$.i18n.prop("id.required.or.alreadyexists");

diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/network-proxies.html b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/network-proxies.html
index cd1ba54cdf25a73b2a8976cb3e54c01d00b87aff..701a5ac4d913b1f29856d422b7c80e66167143df 100644 (file)
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/network-proxies.html
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/network-proxies.html
@@ -98,7 +98,7 @@
     <form id="network-proxy-edit-form" class="well form-horizontal">
       <fieldset id="network-proxy-edit-fieldset">
         <div class="control-group">
-          <label class="control-label"  for="id">${$.i18n.prop('id')}</label>
+          <label class="control-label" for="id">${$.i18n.prop('id')}</label>
           <div class="controls">
             {{if update}}
               <span class="uneditable-input">${$data.networkProxy.id}</span>

diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/utils.js b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/utils.js
index eee85a1323a79f4f4df268af4b95a12667d1f618..815605f927da892ec653c471ba2366df6f89b7cb 100644 (file)
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/utils.js
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/utils.js
@@ -285,11 +285,11 @@ displayRestError=function(data,idToAppend){
  * @param errorList
  */
 customShowError=function(selector, validator, errorMap, errorList) {
-  $.isFunction(selector)? selector.find("div.control-group" ).removeClass( "error" ):$(selector).find("div.control-group" ).removeClass( "error" );
-  $.isFunction(selector)? selector.find("span.help-inline").remove():$(selector).find("span.help-inline").remove();
+  $(selector).find("div.control-group" ).removeClass( "error" );
+  $(selector).find("span.help-inline").remove();
   for ( var i = 0; errorList[i]; i++ ) {
     var error = errorList[i];
-    var field = $("#"+error.element.id);
+    var field = $(selector).find("#"+error.element.id);// $.isFunction(selector)? selector.find(fieldSelector ).get(0):$(selector).find(selector);
     field.parents( "div.control-group" ).addClass( "error" );
     field.parent().append( "<span class=\"help-inline\">" + error.message + "</span>" );
   }