Commit c647065b authored by Travis Swicegood's avatar Travis Swicegood
Browse files

Make sure these functions don't bleed into the global scope 

This makes sure that all of these functions are assigned to variables
assigned to the current scope, rather than the global scope.  It also
adds a trailing semi-colon to make sure various linters are happy.
parent aa757ac2

django/contrib/admin/static/admin/js/actions.js

+7 −7
Original line number Diff line number Diff line
@@ -3,7 +3,7 @@ 
		var options = $.extend({}, $.fn.actions.defaults, opts);

		var actionCheckboxes = $(this);

		var list_editable_changed = false;

		checker = function(checked) {

		var checker = function(checked) {

			if (checked) {

				showQuestion();

			} else {
@@ -11,7 +11,7 @@ 
			}

			$(actionCheckboxes).attr("checked", checked)

				.parent().parent().toggleClass(options.selectedClass, checked);

		}

		},

		updateCounter = function() {

			var sel = $(actionCheckboxes).filter(":checked").length;

			$(options.counterContainer).html(interpolate(
@@ -29,30 +29,30 @@ 
				}

				return value;

			});

		}

		},

		showQuestion = function() {

			$(options.acrossClears).hide();

			$(options.acrossQuestions).show();

			$(options.allContainer).hide();

		}

		},

		showClear = function() {

			$(options.acrossClears).show();

			$(options.acrossQuestions).hide();

			$(options.actionContainer).toggleClass(options.selectedClass);

			$(options.allContainer).show();

			$(options.counterContainer).hide();

		}

		},

		reset = function() {

			$(options.acrossClears).hide();

			$(options.acrossQuestions).hide();

			$(options.allContainer).hide();

			$(options.counterContainer).show();

		}

		},

		clearAcross = function() {

			reset();

			$(options.acrossInput).val(0);

			$(options.actionContainer).removeClass(options.selectedClass);

		}

		};

		// Show counter by default

		$(options.counterContainer).show();

		// Check state of checkboxes and reinit state if needed