Hola:
 
Se puede simplificar un poco, además los eventos son cosa javascript por lo que poner "javascript
:" es redundante... 
código simplificado, suponiendo que es parte de un formulario:  
Código:
 <input name="valor" value="Si" onclick="texto.disabled=false;" type="radio" />Si<br />
<input name="valor" value="No" onclick="texto.disabled=true;" type="radio" />No<br /><br />
<input name="texto" type="text" disabled="disabled" />
  Otra cosa, que el true puede ser también this.checked, y el false -> !this.checked 
Saludos 
