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