From 3a4a827f1fb5b7a9188e53ce8fae99924111c458 Mon Sep 17 00:00:00 2001 From: denise Date: Wed, 8 Aug 2012 15:24:47 -0400 Subject: [PATCH] CC-4180: Smart Playlist Builder: modifier select boxes do not change when criteria changes -fixed --- .../airtime/playlist/smart_playlistbuilder.js | 37 ++++++++++--------- 1 file changed, 19 insertions(+), 18 deletions(-) diff --git a/airtime_mvc/public/js/airtime/playlist/smart_playlistbuilder.js b/airtime_mvc/public/js/airtime/playlist/smart_playlistbuilder.js index 81ba7acaa..8123e99c4 100644 --- a/airtime_mvc/public/js/airtime/playlist/smart_playlistbuilder.js +++ b/airtime_mvc/public/js/airtime/playlist/smart_playlistbuilder.js @@ -429,26 +429,27 @@ function sizeTextBoxes(ele, classToRemove, classToAdd) { } function populateModifierSelect(e) { - /*var criteria = $(e).val(), - criteria_type = criteriaTypes[criteria],*/ var criteria_type = getCriteriaOptionType(e), - div = $(e).siblings('select[id^="sp_criteria_modifier"]'); + index = getRowIndex($(e).parent()), + divs = $(e).parents().find('select[id^="sp_criteria_modifier_'+index+'"]'); - div.children().remove(); - - if (criteria_type == 's') { - $.each(stringCriteriaOptions, function(key, value){ - div.append($('') - .attr('value', key) - .text(value)); - }); - } else { - $.each(numericCriteriaOptions, function(key, value){ - div.append($('') - .attr('value', key) - .text(value)); - }); - } + $.each(divs, function(i, div){ + $(div).children().remove(); + + if (criteria_type == 's') { + $.each(stringCriteriaOptions, function(key, value){ + $(div).append($('') + .attr('value', key) + .text(value)); + }); + } else { + $.each(numericCriteriaOptions, function(key, value){ + $(div).append($('') + .attr('value', key) + .text(value)); + }); + } + }); } function getCriteriaOptionType(e) {