Изменить это:
formCheck.onSubmit = doMapping()
к этому:
formCheck.onSubmit = doMapping
Когда вы добавляете круглые скобки в конец функции, вы выполняете эту функцию. Когда вы назначаете функцию (или передаете ее в качестве параметра другой функции), вам нужно опустить круглые скобки, так как это способ получить указатель на функцию в JavaScript.
Редактировать: Вам также понадобится переместить объявление функции doMapping
над назначением этой функции в событие onsubmit, как это (хороший улов tvanfosson!):
function doMapping() {
alert("form submitted");
return false;
}
formCheck.onSubmit = doMapping();
Однако, если функция doMapping
не используется в другом месте, вы можете объявить функцию doMapping
как анонимную функцию, подобную этой:
formCheck.onSubmit = function() {
alert("form submitted");
return false;
}
, который кажется мне чище.