From 85d6b5d1ed1d49412d813b75d60bb076653675bf Mon Sep 17 00:00:00 2001 From: Tim Graham Date: Mon, 20 Jul 2015 16:29:10 -0400 Subject: [PATCH] Fixed JavaScript "no-shadow" violations. --- .eslintrc | 2 +- .../contrib/admin/static/admin/js/inlines.js | 6 ++---- .../admin/static/admin/js/inlines.min.js | 18 +++++++++--------- 3 files changed, 12 insertions(+), 14 deletions(-) diff --git a/.eslintrc b/.eslintrc index 9c313a42a1..701d7f9fbc 100644 --- a/.eslintrc +++ b/.eslintrc @@ -21,7 +21,7 @@ "no-underscore-dangle": [1], "no-unused-vars": [2, {"vars": "local", "args": "none"}], "no-script-url": [1], - "no-shadow": [1, {"hoist": "functions"}], + "no-shadow": [2, {"hoist": "functions"}], "quotes": [ 1, "single" diff --git a/django/contrib/admin/static/admin/js/inlines.js b/django/contrib/admin/static/admin/js/inlines.js index dd8a446349..0c6da51986 100644 --- a/django/contrib/admin/static/admin/js/inlines.js +++ b/django/contrib/admin/static/admin/js/inlines.js @@ -57,7 +57,6 @@ } addButton.click(function(e) { e.preventDefault(); - var totalForms = $("#id_" + options.prefix + "-TOTAL_FORMS"); var template = $("#" + options.prefix + "-empty"); var row = template.clone(true); row.removeClass(options.emptyCssClass) @@ -89,10 +88,9 @@ addButton.parent().hide(); } // The delete button of each row triggers a bunch of other things - row.find("a." + options.deleteCssClass).click(function(e) { - e.preventDefault(); + row.find("a." + options.deleteCssClass).click(function(e1) { + e1.preventDefault(); // Remove the parent form containing this button: - var row = $(this).parents("." + options.formCssClass); row.remove(); nextIndex -= 1; // If a post-delete callback was provided, call it with the deleted form: diff --git a/django/contrib/admin/static/admin/js/inlines.min.js b/django/contrib/admin/static/admin/js/inlines.min.js index 705a3c4ec9..d968b58f4d 100644 --- a/django/contrib/admin/static/admin/js/inlines.min.js +++ b/django/contrib/admin/static/admin/js/inlines.min.js @@ -1,9 +1,9 @@ -(function(c){c.fn.formset=function(b){var a=c.extend({},c.fn.formset.defaults,b),d=c(this);b=d.parent();var k=function(a,g,l){var h=new RegExp("("+g+"-(\\d+|__prefix__))");g=g+"-"+l;c(a).prop("for")&&c(a).prop("for",c(a).prop("for").replace(h,g));a.id&&(a.id=a.id.replace(h,g));a.name&&(a.name=a.name.replace(h,g))},e=c("#id_"+a.prefix+"-TOTAL_FORMS").prop("autocomplete","off"),l=parseInt(e.val(),10),g=c("#id_"+a.prefix+"-MAX_NUM_FORMS").prop("autocomplete","off"),e=""===g.val()||0'+a.addText+""),h=b.find("tr:last a")):(d.filter(":last").after('"),h=d.filter(":last").next().find("a"));h.click(function(b){b.preventDefault();var d=c("#id_"+a.prefix+ -"-TOTAL_FORMS");b=c("#"+a.prefix+"-empty");var f=b.clone(!0);f.removeClass(a.emptyCssClass).addClass(a.formCssClass).attr("id",a.prefix+"-"+l);f.is("tr")?f.children(":last").append('
'+a.deleteText+"
"):f.is("ul")||f.is("ol")?f.append('
  • '+a.deleteText+"
  • "):f.children(":first").append(''+a.deleteText+ -"");f.find("*").each(function(){k(this,a.prefix,d.val())});f.insertBefore(c(b));c(d).val(parseInt(d.val(),10)+1);l+=1;""!==g.val()&&0>=g.val()-d.val()&&h.parent().hide();f.find("a."+a.deleteCssClass).click(function(b){b.preventDefault();b=c(this).parents("."+a.formCssClass);b.remove();--l;a.removed&&a.removed(b);b=c("."+a.formCssClass);c("#id_"+a.prefix+"-TOTAL_FORMS").val(b.length);(""===g.val()||0'+a.addText+""),m=c.find("tr:last a")):(f.filter(":last").after('"),m=f.filter(":last").next().find("a"));m.click(function(d){d.preventDefault();d=b("#"+a.prefix+ +"-empty");var g=d.clone(!0);g.removeClass(a.emptyCssClass).addClass(a.formCssClass).attr("id",a.prefix+"-"+l);g.is("tr")?g.children(":last").append('
    '+a.deleteText+"
    "):g.is("ul")||g.is("ol")?g.append('
  • '+a.deleteText+"
  • "):g.children(":first").append(''+a.deleteText+"");g.find("*").each(function(){k(this, +a.prefix,h.val())});g.insertBefore(b(d));b(h).val(parseInt(h.val(),10)+1);l+=1;""!==e.val()&&0>=e.val()-h.val()&&m.parent().hide();g.find("a."+a.deleteCssClass).click(function(d){d.preventDefault();g.remove();--l;a.removed&&a.removed(g);d=b("."+a.formCssClass);b("#id_"+a.prefix+"-TOTAL_FORMS").val(d.length);(""===e.val()||0