Javascript function in php doesn't work

Question

I have a php function which ends with a refresh of the page and then throws a javascript function declared:

<script>
    var u = null;
    function tab(u) {
        $('#element1').css('display', 'block');
        $('#element2').css('display', 'none');
    }
</script>
<?php
    if (isset($_POST['submit1'])) {
        $tab = 1;
        header("Refresh:0");            
        echo "<script>window.scrollTo(0, 1000);</script>";
        echo "<script>tab(".$tab.");</script>";
    }
?>

The function of window.scrollto works, but the tab function not. I wrote ob_start and ob_end_flush to avoid headers problems.

there might be a problem because you are using header function after generating an output... — ewcz
– ewcz, Commented Jan 7, 2017 at 18:46

Ionut Necula · Accepted Answer · 2017-01-07 19:14:37Z

2

You are missing a closing ) on the if statement. Try this:

  <script>
    var u = null;
    function tab(u) {
        $('#element1').css('display', 'block');
        $('#element2').css('display', 'none');
    }
</script>
<?php
    if (isset($_POST['submit1'])) {
        $tab = 1;
        header("Refresh:0");            
        echo "<script>window.scrollTo(0, 1000);</script>";
        echo "<script>tab(".$tab.");</script>";
    }
?>

The header("Refresh:0"); shoudn't be a problem as you said. In the worst case you will get an warning saying Warning: Cannot modify header information - headers already sent by....

Also, in development you should always have error_reporting turned on to be able to see the possible errors or warnings. You can turn on error reporting like this:

error_reporting(E_ALL);
ini_set('display_errors', 1);

Don't forget to remove/comment that on production/live.

edited Jan 7, 2017 at 19:14

answered Jan 7, 2017 at 18:51

Ionut Necula

11.6k4 gold badges49 silver badges72 bronze badges

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

5 Comments

Sandeep Nayak Over a year ago

Good catch!. Should it not throw error on server side?

gerardet46 Over a year ago

I have it like this but writing the question I made a mistake.

Ionut Necula Over a year ago

@SandeepNayak, it should have if he had the error reporting turned on.

Ionut Necula Over a year ago

@GerryStudios, let me check it again.

gerardet46 Over a year ago

I've erased the header("Refresh:0") because I saw I don't need it and now it works, thanks.

Collectives™ on Stack Overflow

Javascript function in php doesn't work

1 Answer 1

5 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related