Exiting from javascript function not working

Question

I have this javascript function:

function displayMessage() {
    var message = $("#msg").val();

    if (message == "") { 
        alert("You need to enter a message");//alert the user
        return false;
    }

    postData = {
        "message": message,
    };    

    ...

}

What am hoping this achieves is, if the input field is empty, display the alert and remain in the function.If it isn't then continue. My submit button is linked to another page but this page is displayed anyways regardless of what happens in the if statement.

This is the form code:

<form id="post" action="http://localhost:8080/uploadNewMessage" method="post">
    <fieldset>
        <div data-role="fieldcontain">
            <label for="msg" class="input">Message:</label>
            <input type="text" name="msg" id="msg" size="10"/>
        </div>
        <a href="profile-page.html" id="submit" data-role="button" data- theme="b">Submit</a>
    </fieldset>
</form>

and the full javascript code just incase:

$(document).ready(function() {
    // 1. The Registration button
    $("#submit").bind('click', function(event) {

        displayMessage();

    });

});

function doPostRequest(postData) {

    $.ajax({

        type: "POST",
        url: URL,
        dataType: "json",
        data: postData

    });
}

function displayMessage() {
    var message = $("#msg").val();

    if (message == "") { 
        alert("You need to enter a message");//alert the user
        return false;
    }

    postData = {
        "message": message,
    };    

    ...
    doPostRequest(postData);
}

It's my native language. :D Pardon me, let me edit that,i was just trying to leave that part out. — Ojonugwa Jude Ochalifu
– Ojonugwa Jude Ochalifu, Commented Jun 2, 2014 at 1:50

The Alpha · Accepted Answer · 2014-06-02 02:00:48Z

2

You may try something like this:

$("#submit").bind('click', function(event) {
    var message = $.trim($("#msg").val());
    if(!message.length) {
        alert("You need to enter a message");
        return false;
    }
    else {
        event.preventDefault();
        doPostRequest({"message":message});
    }
});

edited Jun 2, 2014 at 2:00

answered Jun 2, 2014 at 1:54

The Alpha

147k30 gold badges294 silver badges313 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Roko C. Buljan Over a year ago

why is the return false; needed?

Ojonugwa Jude Ochalifu Over a year ago

I have tried this, but somehow i am still being redirected to the profile page

The Alpha Over a year ago

Then you have other JS errors, check for the errors on browser's console and fix them.

Roko C. Buljan Over a year ago

@OjonugwaOchalifu have you tried all the suggested solutions? Have you set the url: URL, -> URL ?

Ojonugwa Jude Ochalifu Over a year ago

Am trying them all out now.

Roko C. Buljan · Accepted Answer · 2014-06-02 01:59:18Z

1

demo

$(function() {
    $("#submit").on('click', function(event) {
        event.preventDefault(); // prevent default anchor behavior
        displayMessage();
    });
});

and also:

function displayMessage() { 
    var message = $.trim( $("#msg").val() ); // trim whitespaces
    if (message === "") { 
        alert("You need to enter a message");
    }else{                              // Use the else statement
      doPostRequest({
        "message" : message
      });   
    }
}

answered Jun 2, 2014 at 1:59

Roko C. Buljan

209k41 gold badges335 silver badges346 bronze badges

1 Comment

Ojonugwa Jude Ochalifu Over a year ago

When i try this with event.preventDefault(), the button changes to another color and the message is posted to GAE but the page doesn't change.

user3697679 · Accepted Answer · 2014-06-02 01:50:12Z

0

The event variable that is passed via your click event handler contains a function named preventDefault. If you don't want the form to submit, call this (event.preventDefault()). This will prevent the submit button from submitting the form.

answered Jun 2, 2014 at 1:50

user3697679

2 Comments

user3697679 Over a year ago

Objects are variables too.

user3697679 Over a year ago

If I create a new object and assign it to a variable, is that variable no longer a variable?

Collectives™ on Stack Overflow

Exiting from javascript function not working

3 Answers 3

5 Comments

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

5 Comments

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related