javascript send form with get method

Question

I try this dropdown form to send wrong data on page (movie information). This html code is dropdown form to send when user click on submit, javascript send GET data to another php file and there process data, but on place where is form data put Text: "Thank you"

Problem is: when i click on submit don't work, javascript is not doing its job.

html:

<div id='premijera' class='neispravniPodaci'>Report wrong data
    <div id='neispravnaForma' class='arrow_box' >
            <form name='frm1' action='' onSubmit='neispravniPod()' method='get'>
            <input class='textbox'  type='text' name='forma' placeholder='input for wrong data'>
            <input type='submit' name='sendForm' value='Send'  class='button'>
            </form>
        </div>
</div>

And javascript code is:

function neispravniPod()
{
var xmlhttp;
var forma=document.forms['frm1']['forma].value;
if (window.XMLHttpRequest)
  {// code for IE7+, Firefox, Chrome, Opera, Safari
  xmlhttp=new XMLHttpRequest();
  }
else
  {// code for IE6, IE5
  xmlhttp=new ActiveXObject('Microsoft.XMLHTTP');
  }
xmlhttp.onreadystatechange=function()
  {
  if (xmlhttp.readyState==4 && xmlhttp.status==200)
    {
    document.getElementById('neispravnaForma').innerHTML='<t style=\"font-size:15px;font-weight:bold;color:black;\">Hvala!</t>';
    }
  }

xmlhttp.open('GET','glasanje.php?id=".$idfilm."&neispravno='+forma+'&ip=".$_SERVER["REMOTE_ADDR"]."&korisnik=".$userdata['user_id']."',true);
xmlhttp.send();
}

dcasadevall · Accepted Answer · 2013-03-03 18:20:14Z

1

Like Rene points out, this is most likely a javascript syntax error, that is preventing the onSubmit function to kick in before the actual submit.

Suggest trying your code in jsfiddle.

I see PHP code in the xmlhttp.open request. How are you inserting that code? this might be a fail point if you are just trying to use that code in a js file and expecting the client to execute that (it will just spit out a syntax error, because a javascript file will never and should never interpret server side code).

Also , have you considered using jQuery or another js library that would help you in writing cleaner ajax requests?

Edit With Answer (according comment below):

Not sure what your PHP is for the rest of the file, but here is what I have used and tested locally to make sure it works. Corrected some syntax errors, also on the PHP side, the way you output variables. Paste this code in your index.php file or whatever you are using:

<div id='premijera' class='neispravniPodaci'>Report wrong data
    <div id='neispravnaForma' class='arrow_box'>
        <form name='frm1' action='javascript:neispravniPod();' method='get'>
            <input class='textbox' type='text' name='forma' placeholder='input for wrong data'>
            <input type='submit' name='sendForm' value='Send' class='button'>
        </form>
    </div>
</div>

<script type="text/javascript">
    function neispravniPod() {
        var xmlhttp;
        var forma = document.forms.frm1.forma.value;
        if (window.XMLHttpRequest) { // code for IE7+, Firefox, Chrome, Opera, Safari
            xmlhttp = new XMLHttpRequest();
        } else { // code for IE6, IE5
            xmlhttp = new ActiveXObject('Microsoft.XMLHTTP');
        }
        xmlhttp.onreadystatechange = function () {
            if (xmlhttp.readyState == 4 && xmlhttp.status == 200) {
                document.getElementById('neispravnaForma').innerHTML = '<t style=\"font-size:15px;font-weight:bold;color:black;\">Hvala!</t>';
            }
        };

        xmlhttp.open('GET', 'glasanje.php?id=<?php echo $idfilm ?>&neispravno='+forma+'&ip=<?php echo $_SERVER["REMOTE_ADDR"] ?>&korisnik=<?php echo $userdata['user_id'] ?>', true);
        xmlhttp.send();
    }       
</script>

edited Mar 3, 2013 at 18:20

answered Mar 3, 2013 at 13:51

dcasadevall

3161 silver badge5 bronze badges

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

2 Comments

user1721620 Over a year ago

no, i had one example for vote and i try to make and for this form but some is wrong... yes this is PHP page... I do not know how to implement ajax becouse i try this.... ok, how now i can send this form? jsfiddle.net/D9fpd

dcasadevall Over a year ago

Edited my answer with the code that should work for you. Let me know if that does it. (Not sure how to paste code on comments :/)

ReneOllino · Accepted Answer · 2013-03-03 13:38:28Z

0

First thing i would do is check the javascript, seems like you have a lot of errors in the script ... i like to check my script in jsbin.com ... just open the javascript panel and paste the script there ... i found in your script 33 warnings ... click: http://jsbin.com/oqiliq/1/edit

answered Mar 3, 2013 at 13:38

ReneOllino

262 bronze badges

Collectives™ on Stack Overflow

javascript send form with get method

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related