if statement not working jquery

Question

This should be simple, but doesn't work

Essentially I want to get a return result and reload my page if the answer is 'yes'

Here is the timer.php

<?php
    $filename = 'timer.tmr';

    if (file_exists($filename)) {
        unlink($filename);
        echo 'yes';
    } else {
        echo 'no';
    }
?>

and here is the script in my main file

var myVar = setInterval(function () {myTimer()}, 5000);
function myTimer() {
     sendto =  "timer.php";
     $.get(sendto, function(data, status) {
         if (data === 'yes') {
            location.reload();
         }
     });
}

Basically in the timer.php

If the file exists I delete it and return a 'yes'

But the jquery doesn't do the reload function, if I check the data with an alert(data), it returns a 'yes' So I just don't understand the problem.

How do you know it's not reloading? Try location.reload(true) to force a hard reload. — FJT, May 22 '17 at 12:49
Can you try with `if (data == 'yes')`? because there might be an issue with `'yes'`. — Shaunak Shukla, May 22 '17 at 13:01
If you are getting into the `if` condition, then you should try `window.location.href = window.location.href;` instead of `location.reload();` — Shaunak Shukla, May 22 '17 at 13:08
and If you're not getting into the `if` condition, change condition with `if ($.trim(data) === "yes") ` — Shaunak Shukla, May 22 '17 at 13:17
Ahhh finally success, thank you that did it, thank you so much — Karter, May 22 '17 at 14:28

score 2 · Answer 1 · edited May 24 '17 at 12:47

If you are getting into the if condition, then you should try window.location.href = window.location.href; instead of location.reload();

and If you're not getting into the if condition, change condition with if ($.trim(data) === "yes")

thanks to Shaunak Shukla (again thank you)

here is the final solution

     var myVar = setInterval(function () {myTimer()}, 10000);
     function myTimer() {
     sendto =  "timer.php";
     $.get(sendto, function(data, status) {
     if ($.trim(data) === "yes"){
       location.reload();
       }
      });

       }

it needed the trim(data) must have had something in it..

capcj · Answer 2 · 2017-05-22T12:52:35.847

0

Use json_encode in the echos and improve your checking:

<?php
$filename = 'timer.tmr';

if (file_exists($filename)) {

    unlink($filename);
    if (!file_exists($filename))
       echo json_encode('yes');
    else
       echo json_encode('no');
} else {

    echo json_encode('no');
}

?>

And change the reload to a more direct scope:

 var myVar = setInterval(function () {myTimer()}, 5000);
 function myTimer() {

 sendto =  "timer.php";
 $.get(sendto, function(data, status) {
   if (data === 'yes'){
    window.location.reload();
  }
  });

  }

edited May 22 '17 at 12:52

answered May 22 '17 at 12:49

capcj

1,535
1
16
23

He mentioned that `alert(data)` does return 'yes'. Which means, its working fine and doesn't need json. Hmm? – Milan Chheda May 22 '17 at 12:52
Yeah, but is better always send a json content when working with ajax. – capcj May 22 '17 at 12:53
I made the changes and it still didn't work... driving me bonkers – Karter May 22 '17 at 13:01
Put data == 'yes' just to test. And put console.log to us, please – capcj May 22 '17 at 13:04
I know the reload isn't working because my page isn't getting refreshed.. – Karter May 22 '17 at 13:05
@Karter what happens when you use `=='yes'` instead of `==='yes` ? – Masivuye Cokile May 22 '17 at 13:06
I had already tried that, but just to make sure, I changed it and still no change – Karter May 22 '17 at 13:07
Try .done(function() { window.location.reload(); }) – capcj May 22 '17 at 13:08
it appears that it is not getting past the if statement – Karter May 22 '17 at 13:36
@ShaunakShukla I know, i'm just trying other approach to see if the callbacks are working. Karte, As I thought. Put typeof data here plz – capcj May 22 '17 at 14:56
@ShaunakShukla It's true... thanks man, sry, i didn't pay attention – capcj May 22 '17 at 15:00

score 0 · Answer 3 · answered May 22 '17 at 12:56

0

Let complete ajax request then try to reload :

setTimeout(function(){// wait for 5 secs(2)
        location.reload(); // then reload the page.(3)
}, 5000);

answered May 22 '17 at 12:56

Ahmed Ginani

6,522
2
15
33

don't quite understand what you want me to do here – Karter May 22 '17 at 13:16
@Karter https://stackoverflow.com/questions/7881089/reload-the-page-after-ajax-success – Ahmed Ginani May 22 '17 at 13:21

score 0 · Answer 4 · answered May 22 '17 at 13:21

0

I would Suggest, please check the Type casting (data type). I find, you have used === , it will check string to string or int to int.

answered May 22 '17 at 13:21

Deepak Patil

23
5

it is returning a string – Karter May 22 '17 at 13:31
Then please use trim function on data that return from ajax – Deepak Patil May 23 '17 at 03:13

if statement not working jquery

4 Answers4