ajax not working on button click

Question

I have been looking at this since morning and I can't figure out where the problem is from..

this is the javascript/ajax

$('#subscribe').live('click', function(){
        rel = $(this).attr("rel");
        datas = "topic_id="+rel;
        $.ajax({
            type: 'POST',
            url: 'subscribe.php',
            data: datas,
            success: function(result){
                alert("k");
                $(this).val(result);
            }
        });
    });

and this is the php handling the ajax is been posted to "subscribe.php"

<?php
session_start();
require_once(functions/functions.php);

if(isset($_POST["topic_id"])){
        $uid = user_id($_SESSION['username']);
        $tid = $_POST["topic_id"];
        $qry = "SELECT user_id FROM subscribe WHERE topic_id = $tid";
        $rst = mysql_query($qry);
        if(!$rst){
            $query = "INSERT INTO subscribe (user_id, topic_id) VALUES ($uid, $tid)";
            $qry = mysql_query($query);
            echo "Subscribed";
        }else{
            echo "hmmm";
        }
}?>

and this is the button

<input type="button" value="Subscribe" name="buton" id="subscribe" rel="'.$output["id"].'" />

alert seems to work but the changes don't take effect and no changes in the database

@RajeevRanjan i have another code that is using same structure and it's working..maybe when i get this one to work i will switch to the accepted one..but in the mean time can u tell me where i made the mistake — 1baga
– 1baga, Commented Jun 4, 2013 at 8:43
can you see any response using firefox console or Chrome Net. this seems relatively simple. — shanavascet
– shanavascet, Commented Jun 4, 2013 at 8:44

A. Wolff · Accepted Answer · 2013-06-04 08:53:15Z

You should use object for request data:

datas = {topic_id:rel};

You need to keep reference on 'this' inside ajax callback success:

$('#subscribe').live('click', function(){
        var $self = $(this);
        rel = $self.attr("rel");
        datas = "topic_id="+rel;
        $.ajax({
            type: 'POST',
            url: 'subscribe.php',
            data: datas,
            success: function(result){
                alert("k");
                $self.val(result);
            }
        });
    });

You could use a closure too:

$('#subscribe').live('click', function () {
    rel = $(this).attr("rel");
    datas = "topic_id=" + rel;
    (function ($self) {
        $.ajax({
            type: 'POST',
            url: 'subscribe.php',
            data: datas,
            success: function (result) {
                alert("k");
                $self.val(result);
            }
        });
    })(this);
});

Or see for $.proxy() method.

BTW, here 'rel' is a global variable, not sure it is what you really expect. To make it local, use 'var' for its declaration. And 'live' is depreacated, you should use delegation with .on() instead.

i just changed and the text of the button changed but no to the specified result...its like the ajax is not posting to the php page...
Check in console network tab result returned by you php script (errors, etc...)
i just checked the console but there is no error related to the script
Fanx man i got it to work...you re really good..fanx for the help

harish · Accepted Answer · 2013-06-04 08:58:31Z

0

Check your file locations. where is subscribe.php ? and where is javascript file. try to use firebug Firefox addon and go to console and check ajax request details. whether it is able to find subscribe.php

other wise give relative path in 'url' parameter in ajax request

Other reason may be failure of require_once("functions.php"); so try writing die("hii"); above that line in subscribe.php

answered Jun 4, 2013 at 8:58

harish

6442 gold badges10 silver badges21 bronze badges

2 Comments

1baga Over a year ago

i wrote die and the button text changed to it...so wat could be the problem....i just checked the require_once and its correct

1baga Over a year ago

just fixed it..it was a problem from the sql per say....fanx you actually put me on the right path

Surya Peddada · Accepted Answer · 2013-06-04 09:20:24Z

0

In place of live function you on() like below.Its works fine.

$('#subscribe').on('click', function(){
    rel = $(this).attr("rel");
    datas = "topic_id="+rel;
    $.ajax({
        type: 'POST',
        url: 'subscribe.php',
        data: datas,
        success: function(result){
            alert("k");
            $(this).val(result);
        }
    });
});

answered Jun 4, 2013 at 9:20

Surya Peddada

2651 gold badge3 silver badges11 bronze badges

Comments

Mahesh · Accepted Answer · 2013-06-04 09:26:49Z

0

Try this

$('#subscribe').live('click', function(){
        var t = $(this);
        var rel = t.attr("rel");
        datas = {'topic_id' : rel};
        $.ajax({
            type: 'POST',
            url: 'subscribe.php',
            data: datas,
            dataType : 'json',
            success: function(result){
               t.val(result);
            }
        });
    });

answered Jun 4, 2013 at 9:26

Mahesh

3201 silver badge11 bronze badges

Comments

Brad Larson · Accepted Answer · 2015-01-06 18:49:19Z

0

$.ajax({ type: "POST", url: "JQueryServlets", data: { message : mge}, success : function(data){ alert('success'+data); } });

try this code

and at servlet

protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
    System.out.println(" inside do post "+request.getParameter("message"));
}

edited Jan 6, 2015 at 18:49

Brad Larson

170k45 gold badges401 silver badges574 bronze badges

answered Nov 12, 2014 at 17:44

Konzern

12810 bronze badges

Comments

John Smith · Accepted Answer · 2013-06-04 08:41:20Z

-1

you post if(!$rst){ ... wouldn't that make the insert query NOT run if the select query is successful? regards

answered Jun 4, 2013 at 8:41

John Smith

551 silver badge10 bronze badges

Collectives™ on Stack Overflow

ajax not working on button click

6 Answers 6

4 Comments

2 Comments

Comments

Comments

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

4 Comments

2 Comments

Comments

Comments

Comments

Comments

Related