run script if user logged in not working?

Question

<?php if(isset($_SESSION['logged_in'])) {
    $somevar= "something";
?>

<script type="text/javascript">
    var somevar2 = <?php echo $value; ?>;
</script>

<?php } ?>

I put the code inside <head>, the console shows somevar2 is undefined, I was expecting the script don't run because it put in a scope where it execute only when user is logged in. What is the cause of this problem?

Most likely the error is coming from further down your code, where you try to access somevar2, but it doesn’t exist because the shown <script> block has not been written into the HTML output for the not logged-in user. — C3roe
– C3roe, Commented Dec 3, 2013 at 11:52
This is because $value is not declared anywhere in your code. — mutiemule
– mutiemule, Commented Dec 3, 2013 at 11:54
@WayneC in another file.php which when execute when the ajax query sent. Now I think it's because the ajax is slower than the script? — user3057928
– user3057928, Commented Dec 3, 2013 at 11:56
is value defined somewhere?. does $value contain something that will be evaluated as valid javascript? have you turned on all errors and warnings in php? — Javier Neyra
– Javier Neyra, Commented Dec 3, 2013 at 12:08

D. Schreier · Accepted Answer · 2018-07-06 10:59:22Z

2

<?php if(isset($_SESSION['logged_in'])) {
    $somevar= "something";
?>

    <script type="text/javascript">
        var somevar2 = <?php echo $somevar; ?>;
    </script>

<?php } ?>

That should do the job. My guess is that you just forgot to change the variable echoed.

edited Jul 6, 2018 at 10:59

D. Schreier

1,7961 gold badge24 silver badges35 bronze badges

answered Dec 3, 2013 at 11:50

DannyThunder

1,0031 gold badge13 silver badges29 bronze badges

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

3 Comments

Shankar Narayana Damodaran Over a year ago

That is Right. Danny. +1

Colin M Over a year ago

This still isn't right, the result wouldn't be as expected. This would result in: var somevar2 = something; and something won't be defined in JS. This should be changed to: var somevar2 = <?php echo json_encode($somevar); ?>

user3057928 Over a year ago

no.. the somevar2 is defined at another file (process.php) where I used ajax.. what I want is, if the user is not logged in, don't run the script. possible?

Collectives™ on Stack Overflow

run script if user logged in not working?

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related