Una llamada a la función multiplicar iba sin parámetros --> multiplicar() entonces no es capaz de cazar los id.
Aún así te paso tu código un pelín mejorado para que funcione la función sumar. Dentro de la función multiplicar llama a sumar() porque solo la tenías puesto en los textbox con id = importeX. Si no pasa por dicho textbox nunca se ejecutará la función:
(codigo js)
function multiplicar(id){
m1 = document.getElementById('cantidad' + id).value;
m2 = document.getElementById('ml' + id).value;
r = m1*m2;
document.getElementById('importe' + id).value = r;
suma();
}
function suma(){
numero1 = new Number(document.getElementById("importe1").value);
numero2 = new Number(document.getElementById("importe2").value);
numero3 = new Number(document.getElementById("importe3").value);
numero4 = new Number(document.getElementById("importe4").value);
numero5 = new Number(document.getElementById("importe5").value);
numero6 = new Number(document.getElementById("importe6").value);
resultado = numero1 + numero2 + numero3 + numero4 + numero5 + numero6;
if(isNaN(resultado))
resultado = '';
document.getElementById("subtotal").value = resultado;
}
(codigo HTML)
<div>
<table>
<tr>
<td align="left" valign="top">
<table width="650" height="519" border="0" class="tabla">
<tr bgcolor="#CCCCCC">
<td height="20" align="left" valign="top" class="texto_forms1">
<div align="center">
cantidad</div>
</td>
<td align="left" valign="top" class="texto_forms1">
<div align="center">
concepto</div>
</td>
<td align="left" valign="top" class="texto_forms1">
<div align="center">
m2/ml</div>
</td>
<td align="left" valign="top" class="texto_forms1">
<div align="center">
importe</div>
</td>
</tr>
<tr>
<td width="60" height="70" align="left" valign="top" class="texto_forms1">
<input name="cantidad1" type="text" class="estilocampofacturacantidad" id="cantidad1"
onchange="multiplicar(1);" /></td>
<td width="422" align="left" valign="top" class="texto_forms1">
<textarea name="concepto1" class="estilotextareafactura" id="concepto1"></textarea></td>
<td width="60" align="left" valign="top" class="texto_forms1">
<input name="ml1" type="text" class="estilocampofacturacantidad" id="ml1" onchange="multiplicar(1);" /></td>
<td width="90" align="left" valign="top" class="texto_forms1">
<input name="importe1" type="text" class="estilocampofacturaimporte" id="importe1"
onchange="suma();" /></td>
</tr>
<tr>
<td height="70" align="left" valign="top" class="texto_forms1">
<input name="cantidad2" type="text" class="estilocampofacturacantidad" id="cantidad2"
onchange="multiplicar(2);" /></td>
<td align="left" valign="top" class="texto_forms1">
<textarea name="concepto2" class="estilotextareafactura" id="concepto2"></textarea></td>
<td align="left" valign="top" class="texto_forms1">
<input name="ml2" type="text" class="estilocampofacturacantidad" id="ml2" onchange="multiplicar(2);" /></td>
<td align="left" valign="top" class="texto_forms1">
<input name="importe2" type="text" class="estilocampofacturaimporte" id="importe2"
onchange="suma();" /></td>
</tr>
<tr>
<td height="70" align="left" valign="top" class="texto_forms1">
<input name="cantidad3" type="text" class="estilocampofacturacantidad" id="cantidad3"
onchange="multiplicar(3);" /></td>
<td align="left" valign="top" class="texto_forms1">
<textarea name="concepto3" class="estilotextareafactura" id="concepto3"></textarea></td>
<td align="left" valign="top" class="texto_forms1">
<input name="ml3" type="text" class="estilocampofacturacantidad" id="ml3" onchange="multiplicar(3);" /></td>
<td align="left" valign="top" class="texto_forms1">
<input name="importe3" type="text" class="estilocampofacturaimporte" id="importe3"
onchange="suma();" /></td>
</tr>
<tr>
<td height="70" align="left" valign="top" class="texto_forms1">
<input name="cantidad4" type="text" class="estilocampofacturacantidad" id="cantidad4"
onchange="multiplicar(4);" /></td>
<td align="left" valign="top" class="texto_forms1">
<textarea name="concepto4" class="estilotextareafactura" id="concepto4"></textarea></td>
<td align="left" valign="top" class="texto_forms1">
<input name="ml4" type="text" class="estilocampofacturacantidad" id="ml4" onchange="multiplicar(4);" /></td>
<td align="left" valign="top" class="texto_forms1">
<input name="importe4" type="text" class="estilocampofacturaimporte" id="importe4"
onchange="suma();" /></td>
</tr>
<tr>
<td height="70" align="left" valign="top" class="texto_forms1">
<input name="cantidad5" type="text" class="estilocampofacturacantidad" id="cantidad5"
onchange="multiplicar(5);" /></td>
<td align="left" valign="top" class="texto_forms1">
<textarea name="concepto5" class="estilotextareafactura" id="concepto5"></textarea></td>
<td align="left" valign="top" class="texto_forms1">
<input name="ml5" type="text" class="estilocampofacturacantidad" id="ml5" onchange="multiplicar(5);" /></td>
<td align="left" valign="top" class="texto_forms1">
<input name="importe5" type="text" class="estilocampofacturaimporte" id="importe5"
onchange="suma();" /></td>
</tr>
<tr>
<td height="70" align="left" valign="top" class="texto_forms1">
<input name="cantidad6" type="text" class="estilocampofacturacantidad" id="cantidad6"
onchange="multiplicar(6);" /></td>
<td align="left" valign="top" class="texto_forms1">
<textarea name="concepto6" class="estilotextareafactura" id="concepto6"></textarea></td>
<td align="left" valign="top" class="texto_forms1">
<input name="ml6" type="text" class="estilocampofacturacantidad" id="ml6" onchange="multiplicar(6);" /></td>
<td align="left" valign="top" class="texto_forms1">
<input name="importe6" type="text" class="estilocampofacturaimporte" id="importe6"
onchange="suma();" /></td>
</tr>
<tr>
<td height="20" align="left" valign="top" class="texto_forms1">
</td>
<td align="left" valign="top" class="texto_forms1">
</td>
<td align="left" valign="top" class="texto_forms1">
subtotal</td>
<td align="left" valign="top" class="texto_forms1">
<input name="subtotal" type="text" class="estilocampofacturatotal" id="subtotal" /></td>
</tr>
</table>
</div>
salu2