Hola fran_ba, estás cometiendo algunos horrores, digo errores, básicos; primero olvidaste poner comillas en el parámetro de la función getElementById()
Código Javascript
:
Ver originaldocument.getElementById('ID').href = "es/"}
Por otra parte, las sentencias que has puesto en el evento onclick de los enlaces (idioma = 'es' / idioma = 'en') aunque sí modifican el valor de la variable idioma, no ejecutan el bloque de los if ya que estos sólo se interpretan al cargarse la página y nada más, para ejecutarlos puedes ponerlos en los onclick separandolos de la otra sentencia con punto y coma (;)
Código Javascript
:
Ver original<a href="es/" target="marco" onclick="idioma = 'es'; if (idioma == 'es'){document.getElementById('ID').href = "es/"} if (idioma == 'en'){document.getElementById('ID').href = "en/"}">ESPAÑOL</a>
<a href="en/" target="marco" onclick="idioma = 'en'; if (idioma == 'es'){document.getElementById('ID').href = "es/"} if (idioma == 'en'){document.getElementById('ID').href = "en/"}">INGLES</a>
Pero, como esto es evidentemente muy inconveniente, lo que más adecudo es crear una función que será llamada con el evento onclick de los enlaces, además con esto ya no será necesaría la variable idioma ya que el valor lo recibirá directamente la función como parámetro.
Código Javascript
:
Ver original<script>
function cambiarIdioma(idioma)
{
if (idioma == 'es')
{
document.getElementById('ID').href = "es/";
}
if (idioma == 'en')
{
document.getElementById('ID').href = "en/";
}
}
</script>
<a href="es/" target="marco" onclick="cambiarIdioma('es')">ESPAÑOL</a>
<a href="en/" target="marco" onclick="cambiarIdioma('en')">INGLES</a>
De esta manera manera el código se ejecutará al momento de ocurir el evento click de los enlaces, que es lo que se desea, un saludo.