Почему не работает js код в файле со всеми скиптами?

Question

[Николай Романов]

Есть скрипт который добавляет css к ссылке, он не работает если я вставляю это в код со всеми другими скриптами.
Файл со скриптами подгружается все скрипты работают кроме этого

(function($) {
try{
var el=document.getElementById('msgText').getElementsByTagName('a');
var url=document.location.href;
for(var i=0;i<el.length; i++){
if (url==el[i].href){
el[i].className += ' act';
};
};
}catch(e){}
})(jQuery);

Если я вставляю код сразу в html, скрипт работает.

<script type="text/javascript">
try{
var el=document.getElementById('msgText').getElementsByTagName('a');
var url=document.location.href;
for(var i=0;i<el.length; i++){
if (url==el[i].href){
el[i].className += ' act';
};
};
}catch(e){}
</script>

Comments

[LJ322]

Попробуйте в скрипте оставить только эту запись, не беря её в функцию

try{
var el=document.getElementById('msgText').getElementsByTagName('a');
var url=document.location.href;
for(var i=0;i<el.length; i++){
if (url==el[i].href){
el[i].className += ' act';
};
};
}catch(e){}
}

Если заработает, то надо смотреть что в коде с $ и jQuery происходит

[Николай Романов]

LJ322, Пробывал, даже удалил все кроме этого скрипта.
Вордпресс подключает jquery.min.js?ver=3.6.0 и jquery-migrate.min.js?ver=3.3.2

Подключаются 2 jquery и один мой файл со скриптами

Честно я только в php разобрался, js вообще пока темный лес)

Answer 1

[Максим Иванущик]

Если ваш код не использует jQuery, то не сто́ит его вставлять в анонимную самовызываемую функцию, предназначенную для выполнения jQuery-кода.

Попробуйте сделать так:

window.addEventListener('DOMContentLoaded', function () {
	try {
		const url = document.location.href;
		const $elements = document.getElementById('msgText').getElementsByTagName('a');
		$elements.forEach($el => {
			if (url === $el.getAttribute('href')) {
				$el.classList.add('act');
			}
		});
	} catch(e) {}
});

Comments

[Николай Романов]

Не помогло, пока оставил как есть. Буду позже разбираться

[Сергей delphinpro]

Если ваш код не использует jQuery, то не сто́ит его вставлять в анонимную самовызываемую функцию,

Мне очень интересно – почему же?

[Максим Иванущик]

Сергей delphinpro, просто незачем. Bad practices. Если код ванильный, то джквери нет смысла смешивать с ним.