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


Все,

Этот код javascript динамически добавляет поля ввода, основанные на выборе пользователем раскрывающегося списка. У меня он почти там, где он должен быть, но прямо сейчас, оба случая "msds_copy" и "cofa_copy" отображают одни и те же данные для "html2", когда я тестирую. Выбор " msds_copy "должен заставить 2-е поле формы сказать" регион/язык", но вместо этого он говорит"номер партии".

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

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;
    }
});
1 2

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; // <----