Jquery notification event firing twice

could you post the generated HTML instead the PHP code?

I see you put a document.ready inside a loop, so it will fire more than one time

Works fine here: http://jsbin.com/wemahi/1/edit

I would change a few things:

- You're missing semicolons at the end of your JS lines
- The SCRIPT tag should be lowercase.
- Remove this line just while you're testing:  echo "$(\"#testid\" ).show()\n";
- You have mixed use of quotes in your gritter call.  remove the double quotes and stick with the single: echo "$.gritter.add({title: 'This is a notice', text: '$messages1'});\n";

Given this is rendered on the client, can you post the rendered code please?  Load the page, right click, view source, paste here.

@Leakim971 - where is the loop? I couldn't see it...

@Rob Lurd
Thank you for your comment.

You're missing semicolons at the end of your JS lines

Where exactly?

You have mixed use of quotes in your gritter call.  remove the double quotes and stick with the single: echo "$.gritter.add({title: 'This is a notice', text: '$messages1'});\n";

That doesn't work.

Renedered code as far as I can see.

<div id="messages" style="float: left;height: 220px">
                    <div>
                        Messages
                    </div>
                    <div style="overflow: hidden;">
			<div id="message" style="font-size: 11px; padding: 10px 10px;">
							
			<script type=text/javascript>
$.gritter.add({title: 'This is a new notice', text: "You have  <span class='yourstyle'>1</span> new message(s). Please go to the message area in your control panel.", time: 3000});
</script>
						
			</div>
                        
                    </div>
                </div>

Select allOpen in new window

I see two things that might make your life easier.  First, get off MySQL ASAP.  It's a dead technology.  This article explains why and what you must do to keep your scripts running.
https://www.experts-exchange.com/Web_Development/Web_Languages-Standards/PHP/PHP_Databases/A_11177-PHP-MySQL-Deprecated-as-of-PHP-5-5-0.html

Second, when you see code like this:

echo "<SCRIPT type=text/javascript>\n";
echo "$(function() {\n";
echo "$(\"#testid\" ).show()\n";
echo "$.gritter.add({title: 'This is a notice', text: \"".$messages1."\"})\n";
echo "});\n";
echo "</script>\n";

Select allOpen in new window

get rid of it and instead use the HEREDOC notation.  It greatly simplifies the punctuation and escaping, and lets you concentrate on the contents of the data string (in this case, javaScript).

$thing = <<<EOD
<script type=text/javascript>
$(function(){
    $("#testid" ).show()
    $.gritter.add({title: 'This is a notice', text: "$messages1"});
});
</script>
EOD;
echo $thing;

Select allOpen in new window

I did not try to understand the JavaScript.  It looks a little "off" to me, and I'm not sure I understand what you want it to accomplish.

Your rendered code is missing the jQuery initializing function. E.g. $(function() {})
The missing semicolons we're not in your php file at the end if every line of JavaScript (not php) as JavaScript requires then too. However your rendered code looks fine other than the missing jQuery init.

To Rob's point about jQuery initialization, this is how I start.

<script type="text/javascript" src="http://code.jquery.com/jquery-latest.min.js"></script>
<script>
$(document).ready(function(){
});
</script>

Select allOpen in new window

Then building on that, I might come up with something like this.

<script type="text/javascript" src="http://code.jquery.com/jquery-latest.min.js"></script>
<script>
$(document).ready(function(){
    $("#testid" ).show()
    $.gritter.add({title: 'This is a notice', text: "$messages1"});
});
</script>

Select allOpen in new window

@Rob Lurd
Turned out to be quite simple. Took your advice and deleted original script code and declared new script before </body> tag and it works.

<script type="text/javascript">
       $(function () {
	   var my_php_var = <?php echo json_encode($messages1) ?>;
	   $.gritter.add({title: 'New Message', text: my_php_var, time: 4000, sticky: false});
	});
</script>

Select allOpen in new window

@Ray
Thank you for your comments. Will look into HEREDOC. Many thanks.

Thank you very much for your help.

@Rob

- You're missing semicolons at the end of your JS lines
- The SCRIPT tag should be lowercase.

Not mandatory as long as there are a newline after each line and it is not inline JS in an event handler.

Still good practice but not a source of the errors