Parse Error

**marain** · 01-23-2022, 09:10 PM

G'day Folks,

This

Code:

<?php
if ($here !== 'quotations') {
	echo '<body class = "' . $here . '">';
}
else {
	echo '<body class = "quotations" onbeforeunload = "alert('Thanks for visiting. Sorry you couldn't stay longer. We hope you return soon. And tell your friends!')">';
}
?>

Results in this:

Parse error: syntax error, unexpected 'Thanks' (T_STRING), expecting ';' or ',' in /home/marainla/public_html/includes/top.inc.php on line 594

Seeking fresh eyes.

A.

**james438** · 01-23-2022, 11:19 PM

Let me know if this works.

I notice you have a bit of what appears to be javascript mixed in there of which I have virtually zero knowledge of, so I may have escaped too many quotes.

Code:

<?php
$here='quotations';
if ($here !== 'quotations') {
	echo "<body class = \"' . $here . '\">";
}
else {
	echo "<body class = \"quotations\" onbeforeunload = \"alert(\'Thanks for visiting. Sorry you couldn\'t stay longer. We hope you return soon. And tell your friends!\')\">";
}
?>

**coothead** · 01-24-2022, 12:37 AM

Hi there marain,

As James has pointed out, it appears that you have an escaping
apostrophes problem.

Try it like this...

PHP Code:


<?php

 $here = 'codetest';

if ( $here !== 'quotations' ) {
  echo '<body class="'. $here .'">';
 }
else {
echo '<body class="quotations" onbeforeunload="alert(\'Thanks for visiting. Sorry you couldn\'t stay longer. We hope you return soon. And tell your friends!\')">';
 }
?>

From the little I know about javascript, I think that you are going to
have the biggest problem with onbeforeunload implementation.

And it certainly should not be in the body tag. If it has any possible
chance of working it will need to be coded in the script element.

coothead

**marain** · 01-25-2022, 02:08 AM

James and Coothead,

Thank you both.

My apologies for my delay in responding: I've been tremendously busy at work, and unable to test your proposed solutions.

On the subject of whether the onbeforeunload should be in the body, I was using as a model https://www.w3schools.com/jsref/even...foreunload.asp.

**coothead** · 01-25-2022, 10:22 AM

Hi there marain,

you should not consider w3schools to be a reliable source for good coding advise.

As for...

Code:

<body onbeforeunload="alert('Thanks for visiting...')">

Further reading:-

Is inline javascript a bad practise

Try this site which has far more reliable information concerning onbeforeunload...

MDN - WindowEventHandlers.onbeforeunload

Personally though, I have not been able to implement it in any of my browsers.

coothead

**coothead** · 01-25-2022, 04:32 PM

Hi there marain,

I have now unraveled the mystery that is beforeunload.

Here is an example page...

Code:

<!DOCTYPE HTML>
<html lang="en">
<head>

<meta charset="utf-8">
<meta name="viewport" content="width=device-width,height=device-height,initial-scale=1">

<title>Beforeunload Test</title>

</head>
<body>

 <h1>Beforeunload Test</h1> 

 <p>
  Clicking this link will activate the <i>beforeunload</i> event<br>
  in both Firefox and Chrome.
</p>
        <a href="https://www.example.com/">example.com </a>
 <p>
    Clicking tab close (  X ) or the page refresh tab(  C )  will <br>
     only work if the link has  been <b>previously clicked</b>.
</p><p>
    Neither browser will accept user text.<br>
    Firefox uses this...
</p>
<blockquote>
     <i>"This page is asking you to confirm that you want to<br>
      leave   — information you’ve entered may not be saved."</i>
</blockquote>
<p>
    ...while Chrome  uses this...
</p>
<blockquote>
     <i>"Changes that you made may not be saved."</i>
</blockquote>

<script>
   window.addEventListener( 'beforeunload',  function(event)  {
      event.preventDefault();
 /* Google Chrome requires returnValue to be set. */
      event.returnValue = '';
  });
</script>

</body>
</html>

I trust that this will help.

coothead

**coothead** · 01-26-2022, 11:05 AM

...of course, as you can no longer use this...

"Thanks for visiting. Sorry you couldn't stay longer.
We hope you return soon. And tell your friends!"

...I can see no good reason to aggravate your visitors by
using the intrusive beforeunload.

coothead

**marain** · 02-18-2022, 08:21 PM

Well, I'm having a design problem that I find perplexing. Many people visit the Poker Portal page on www.MarainLaw.com, but nobody, absolutely nobody, exhausts their allotted number of poker hands. I don't know why people leave so early, and would love to find some way to entice visitors to tell me.

I also see that I never properly thanked you for your response of 1-25-2022. Thank you!

**coothead** · 02-19-2022, 01:42 AM

Originally Posted by marain

I don't know why people leave so early, and would love to find some way to entice visitors to tell me.

I believe, as it appears to be some kind of game, that it should be presented in a
much less cluttered enviroment.

Something more on these lines...

poker portal page image

Perhaps you should also consider adding a comments option to the page.

Another possible reason for prople leaving so early may be because, like me,
they do not fully understand your instructions and/or intentions.

coothead

Thread: Parse Error

Thread Tools

Search Thread

Parse Error

The Following User Says Thank You to james438 For This Useful Post:

The Following 2 Users Say Thank You to coothead For This Useful Post:

The Following User Says Thank You to coothead For This Useful Post:

The Following User Says Thank You to coothead For This Useful Post:

The Following User Says Thank You to coothead For This Useful Post:

Similar Threads

Resolved Parse error: syntax error unexpected 'echo'

I get this error-Parse error: syntax error, unexpected T_STRING, expecting ')' in /ho

Parse error: syntax error, unexpected $end

Parse error: syntax error, unexpected T_VARIABLE, expecting ',' or ';'

Parse error: syntax error, unexpected '=', expecting '&' or T_VARIABLE

Bookmarks

Bookmarks

Posting Permissions