java script stop working on my forum

[purp1eone]

Last night java script stop working on my forum.

Been using the code for months.

Still works on my test forum.

What happened with that?

link

[Chris]

Feb 23, 2018 6:18:18 GMT -8 purp1eone said:

Last night java script stop working on my forum.



Been using the code for months.



Still works on my test forum.



What happened with that?



link

Please fix the link and perhaps go into a bit more detail about "the code" that you are referring to to help anyone trying to help you so they are not wasting time trying to play where is Waldo on a forum you may know but they have never seen before.

[purp1eone]

Feb 23, 2018 6:58:15 GMT -8 Chris said:

Feb 23, 2018 6:18:18 GMT -8 purp1eone said:

Last night java script stop working on my forum.



Been using the code for months.



Still works on my test forum.



What happened with that?



link

Please fix the link and perhaps go into a bit more detail about "the code" that you are referring to to help anyone trying to help you so they are not wasting time trying to play where is Waldo on a forum you may know but they have never seen before.

Link is updated.

I was combining the the user online info from staff, members and guests to one number.

Basically like this example code

<p id="demo"></p>

<script>
var a = 5;
var b = 6;
var c = 7;
var total = a + b + c;
document.getElementById("demo").innerHTML =
"total: " + total;
</script>

Like I said it works on my test forum and even another version of it works on another page on my main forum.

Last night it stopped working on the Home page.

[Brian]

The script is erroring because you're treating $[total_staff_online_24], $[total_members_online_24], and $[total_guests_online_24] as integers in your code.

The problem with this is once any of these variables outputs 1,000 or higher it includes a comma which isn't accepted by Javascript as part of an integer. You need to output each of these numbers as strings, remove the comma, then parse them as integers.

var a = parseInt("$[total_staff_online_24]".replace(/,/g,''));
var b = parseInt("$[total_members_online_24]".replace(/,/g,''));
var c = parseInt("$[total_guests_online_24]".replace(/,/g,''));

[purp1eone]

Thanks Brian I will give that a try.