Hjælp til at lave en funktion

Hej... Nu skal I se: Jeg har et layer der bliver vist. Følgende virker fint:

<a href = "java script:void(0)" onclick = "document.getElementById('test').style.display='block';document.getElementById('fade').style.display='block';frame2.location.href='test.htm'">Test</a>

Jeg vil gerne lave ovenstående om til en funktion, så mine a href's ikke bliver så lange. Jeg har forsøgt mig frem med:

<script>
function open_layer( id, url)
{
document.getElementById( + id + ).style.display='block';
document.getElementById('fade').style.display='block';frame1.location.href='' + url + '';
}
</script>

- som kaldes med:

<a href = "java script:void(open_layer('test','test.htm'));">Test</a>


Er der en, der kan se, hvad jeg gør galt?