Tienes un código bastante "sucio", no es necesario tener cada link con un id, simplemente puedes hacer esto:
Código HTML:
Ver original<script type="text/javascript" src="//ajax.googleapis.com/ajax/libs/jquery/1.10.2/jquery.min.js"></script> $(document).ready(function(){
$('#menu a').click(function(){
$('#menu a').removeClass('active');
$(this).addClass('active');
});
});
<a href="#"><img src="<?=$url?>assets/images/icons/credits.png" style="margin-top:5px;">Inicio
</a> <a href="#"><img src="<?=$url?>assets/images/icons/news.png" style="margin-top:5px;">Noticias
</a> <a href="#"><img src="<?=$url?>assets/images/icons/community2.png">Comunidad
</a> <a href="#"><img src="<?=$url?>assets/images/icons/duckets.png" style="margin-top:5px;">Duckets
</a>
Con eso bastará.
Saludos