Cambiar clase de un objecto clickando en otro objeto

[alexg88]

Tienes algunos fallos que he solucionado y ya se ejecuta, en una de las sentencias:

Código Javascript:

var myIdClass = document.getElementById(myId).className;
 

document.getElementById(myId) da null (no hay ningún elemento con ese id) y, por tanto, no hay propiedad className. Ya verás como lo arreglas.

Código Javascript:

window.onload = initAll;
 
function initAll() {
    var allLinks = document.getElementsByTagName("a");
    for ( var i = 0; i<allLinks.length; i++) {
        if (allLinks[i].className == "menulink") {
            allLinks[i].onclick = toggleMenu;
        }
    }
}
 
function toggleMenu() {
                var startId = this.href.lastIndexOf("/")+1;
                var stopId = this.href.lastIndexOf(".");
                
                var myId = this.href.substring(startId,stopId);
                var myIdClass = document.getElementById(myId).className;
                
                if (myIdClass == "contentvisible") {
            myIdClass = "contentnovisible";
            } else {
                myIdClass = "contentvisible";
                }
                return false;
            }
 

Fíjate en que no hay que poner los paréntesis cuando asignas una función a un evento (por que sino le estás asignando el resultado de llamarla) y el evento es onclick en minúsculas