From f7e09c993d1060b9ba431e134b10ec5ff235b8b0 Mon Sep 17 00:00:00 2001 From: "konstantin.grabar" Date: Tue, 10 Apr 2012 17:15:48 +0400 Subject: [PATCH] [refs #223] Optimise code by jQuery chains --- app/assets/javascripts/extra/build_list.js | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/app/assets/javascripts/extra/build_list.js b/app/assets/javascripts/extra/build_list.js index 6368c3c00..682cd9c4a 100644 --- a/app/assets/javascripts/extra/build_list.js +++ b/app/assets/javascripts/extra/build_list.js @@ -6,8 +6,7 @@ $(document).ready(function() { base_platforms.each(function(){ if ($.inArray(platform_id, base_platforms.map(function(){ return $(this).val() }).get()) >= 0) { if ($(this).val() == platform_id) { - $(this).attr('checked', 'checked'); - $(this).removeAttr('disabled'); + $(this).attr('checked', 'checked').removeAttr('disabled'); $(this).parent().find('.offset25 input[type="checkbox"]').removeAttr('disabled'); var rep_name = $('#build_list_pl_id option[value="' + $(this).val() + '"]').text().match(/[\w-]+\/([\w-]+)/)[1]; @@ -16,16 +15,12 @@ $(document).ready(function() { } $(this).parent().find('.offset25 input[type="checkbox"][rep_name="main"]').attr('checked', 'checked'); } else { - $(this).removeAttr('checked'); - $(this).attr('disabled', 'disabled'); - $(this).parent().find('.offset25 input[type="checkbox"]').attr('disabled', 'disabled'); - $(this).parent().find('.offset25 input[type="checkbox"]').removeAttr('checked'); + $(this).removeAttr('checked').attr('disabled', 'disabled'); + $(this).parent().find('.offset25 input[type="checkbox"]').attr('disabled', 'disabled').removeAttr('checked'); } } else { - $(this).removeAttr('disabled'); - $(this).removeAttr('checked'); - $(this).parent().find('.offset25 input[type="checkbox"]').removeAttr('disabled'); - $(this).parent().find('.offset25 input[type="checkbox"]').removeAttr('checked'); + $(this).removeAttr('disabled').removeAttr('checked'); + $(this).parent().find('.offset25 input[type="checkbox"]').removeAttr('disabled').removeAttr('checked'); } }); });