Is there something wrong with my code?

Question

I don't think there is but this wont load inside facebox:

 $("#email").submit(function(event) {
    event.preventDefault();
    if ($("#tinput").val() == "") {
        $('#error').append('You need to select a template.<br />');
    } else if ($("#tsubject").val() == ""){
        $('#error').append('Subject cannot be blank.<br />');
    } else if ($("#tmessage").val() == "") {
        $('#error').append('Message cannot be blank.<br />');
    } else {
        $('#ecanvas').html('<center><img src="/templates/v2/images/loading.gif" /></center>');
        var formdata = $(this).serialize();
        $.post("/wizard/email-add", formdata, function (data) {
            $('#ecanvas').html(data);
        });
    return false;
})

This works without any problems:

$("#email").submit(function(event) {
    event.preventDefault();
        $('#ecanvas').html('<center><img src="/templates/v2/images/loading.gif" /></center>');
        var formdata = $(this).serialize();
        $.post("/wizard/email-add", formdata, function (data) {
            $('#ecanvas').html(data);
        });
    return false;
})

I should mention #ecanvas, #error, #tinput, #tsubject and #tmessage all exist.

What does the JavaScript error console say? – JJJ Sep 12 '11 at 08:03 — JJJ, Sep 12 '11 at 08:03

score 5 · Accepted Answer · answered Sep 12 '11 at 08:02

5

You failed to close the opening brace of the else

answered Sep 12 '11 at 08:02

Rolando Cruz

2,834
1
16
24

Would you mind accepting the answer if it solves the problem? – Rolando Cruz Sep 12 '11 at 08:04
I will in about 7 minutes (it won't let me before that). – Meisam Mulla Sep 12 '11 at 08:06

Is there something wrong with my code?

1 Answers1