[Bug Report] The JS timer should be only started when top header date enabled

[mosser]

In my site, I disabled the whole top header option. The JS timer throw error every 100ms in all pages, because the element with ID “newsexo-time” does not render.

Error message:
custom.js?ver=6.7:45
Uncaught TypeError: Cannot set properties of null (setting ‘innerHTML’)

File:
\newsexo-pro\assets\js\custom.js (#line 45)

Code:

var myVar = setInterval(function() {
myTimer();
}, 100);

function myTimer() {
var d = new Date();
document.getElementById(“newsexo-time”).innerHTML = d.toLocaleTimeString();
}

[themearile]

Hi Mosser

Please enable the site header on your site so we check the error and provide you better solution.

Thanks

[themearile]

Hi Mosser

Please enable the site header on your site so we check the error and provide you better solution.

Thanks

[mosser]

Un commented the setInterval code snippet, you can now check the error log in my site.
https://www.langsong.site/104448.html

[themearile]

Hi Mosser

Ok please provide your site admin login details on our below email ID so we can fix your issue.

themearile@gmail.com

Thanks

[mosser]

Hi,

thanks for your update.
I think no need to further debug. In the next theme version, you just need to add a null check for html element newsexo-time in file assets/js/custom.js as below:

		if(jQuery('#newsexo-time').length >0 ){
		 var myVar = setInterval(function() {
			myTimer();
		  }, 100);
		}  

[themearile]

Great, Thanks

[Author]

Posts