Проблемы с функцией create_row ()

Question

Все,

Этот код JavaScript динамически добавляет поля ввода на основе выбора пользователем раскрывающегося списка.У меня это почти там, где должно быть, но сейчас, в случаях «msds_copy» и «cofa_copy» отображаются одни и те же данные для «html2», когда я тестирую.При выборе «msds_copy» во втором поле формы должно быть написано «Регион / Язык», но вместо этого указано «Номер партии».

Есть идеи, почему это может происходить?

inquiry_type_onchange: function(e) {
        var place_order = 1,
            order_status = 2,
            telalert_signup = 3,
            invoice_questions = 4,
            msds_copy = 5,
            cofa_copy = 6,
            html = null,
            html2 = null,
            inquiry = e.target,
            id = "inquiry_type_addendum",
            form_row = dojo.query("."+id);

        //Clear any possible previous additions. 
        if (form_row != null)
            form_row.forEach(dojo.destroy);

        //Add the correct new field to the form.
        switch (inquiry.selectedIndex) {
            case place_order:
                html = this.create_form_row(id, "Account Number:");
                break;
            case order_status:
                html = this.create_form_row(id, "Order Number:");
                break;
            case telalert_signup:
                html = this.create_form_row(id, "Account Number:");
                break;
            case invoice_questions:
                html = this.create_form_row(id, "Invoice Number");
                break;
            case msds_copy:
                html = this.create_form_row(id, "Product Name:");
                html2 = this.create_form_row(id + "_2", "Region / Language:");  
            case cofa_copy:
                html = this.create_form_row(id, "Product Name:");
                html2 = this.create_form_row(id + "_2", "Batch Number:");
            default:
        }

        if (html == null) return;
        //Place the new element below the inquiry_type field. 
        var placeat = dojo.byId('buttons');
        dojo.place(html, placeat, "before");
        if(html2!=null)
            dojo.place(html2, placeat, "before");
    },

    create_form_row: function(id, label) {
        //Container
        var a = dojo.create("div", { id: id, className: "question inquiry_type_addendum", style: "padding-top:4px;" });
        //Label
        var b = dojo.create("div", { className: "label", innerHTML: label, style: "margin-top:8px;" }, a);
        //Field
        var c = dojo.create("div", { className: "field" });
        var d = dojo.create("span", { className: "full_number_span span" });
        var e = dojo.create("input", { type: "text", className: "textbox full_number", name: label }, d);
        dojo.place(d, c);
        dojo.place(c, a);
        return a;
    }
});

Answer 1

Вам не хватает break s, и выполняется переключение регистра msds_copy, затем переход к следующему случаю cofa_copy.

        case msds_copy:
            html = this.create_form_row(id, "Product Name:");
            html2 = this.create_form_row(id + "_2", "Region / Language:");  
            break; // <----
        case cofa_copy:
            html = this.create_form_row(id, "Product Name:");
            html2 = this.create_form_row(id + "_2", "Batch Number:");
            break; // <----