А что это у вас за конструкция такая странная?
Надо же так:
<script>
document.addEventListener('DOMContentLoaded', function(){
<?php if(SITE_ID=="s1"){ ?>
document.getElementsByClassName('R-B-H')[0].style.display = 'none';
<?php
} elseif(SITE_ID=="s2"){ ?>
document.getElementsByClassName('O-B-H')[0].style.display = 'none';
<?php
}
?>
});
</script>
Зачем в JavaScript какие то фигурные скобки ставить?
Ошибок вообще полно.
Исполняется JS до проверки загрузки DOM,
getElementsByClassName - возвращает не объект DOM а список объектов.
Какие то фигурные скобки.. каков их смысл?