Код: Выделить всё
$(document).on("beforeSubmit", "#pm-form", function (event, messages) {
var formdata = $(this).serialize();
console.log(formdata);
$(this).find(':submit').attr('disabled', true);
return false;
});Pjax не используюfutbolim писал(а):Ищите в своём js коде.
Если pjax - может timeout-a не хватать.
В ином случае такого поведения не встречал.
Goog1e писал(а):Мне кажется, что у вас сперва срабатывает ajax валидация.
Покажите код формирования формы.
Код: Выделить всё
<?php
$form = ActiveForm::begin([
'id' => 'pm-form',
'action' => ['/messages/send'],
//'enableClientValidation' => true,
//'enableAjaxValidation' => true,
]);
?>
<?php if (empty($dialog)): ?>
<?= $form->field($model, 'subject',[
'template' => "<div class='field'>{label}\n{input}\n{error}</div>",
])->textInput(['maxlength' => 255]) ?>
<?php endif; ?>
<?=
$form->field($model, 'content')->widget(Widget::className(), [
'settings' => [
'buttons' => ['bold', 'italic', 'underline', 'deleted', 'link', 'image'],
'lang' => app\components\helpers\LanguageHelper::getCurrent(),
'minHeight' => 200,
'plugins' => []
]
])->label('');
?>
<?= Html::activeHiddenInput($model, 'sendfrom') ?>
<?= Html::activeHiddenInput($model, 'sendto', ['id' => 'YBoardMessage_send_to']) ?>
<?= Html::hiddenInput('dialog', $dialog) ?>
<div class="field">
<?= Html::submitButton(Yii::t('comments/send_message', 'send'))?>
</div>
<?php ActiveForm::end(); ?>Код: Выделить всё
$(document).on("beforeValidate", "form", function(event, messages, deferreds) {
$(this).find(':submit').attr('disabled', true);
}).on("afterValidate", "form", function(event, messages, errorAttributes) {
if (errorAttributes.length > 0) {
$(this).find(':submit').attr('disabled', false);
}
});
Ну так особо никто и не проявляет активность в теме, что помочь решить возникшую проблему.Onotole писал(а):Написать костыль вместо того, чтобы найти проблему и решить ее...
Код: Выделить всё
$(document).on("beforeSubmit", "#test-form", function (event, messages) {
$(this).find(':submit').attr('disabled', true);
console.log('Test new form');
return false;
});
https://github.com/yiisoft/yii2/blob/ma ... orm.js#L81jekahm писал(а):Затестил ради интереса на голом сайте. Как минимум сабмит формы происходит дважды. С помощью Enter и 3 раза удаётся отправить.То есть проблема явно не в моём коде.Код: Выделить всё
$(document).on("beforeSubmit", "#test-form", function (event, messages) { $(this).find(':submit').attr('disabled', true); console.log('Test new form'); return false; });
If the handler returns a boolean false, it will stop form submission.
Не могли бы разъяснить немного подробнее? Заранее благодаренzelenin писал(а): https://github.com/yiisoft/yii2/blob/ma ... orm.js#L81If the handler returns a boolean false, it will stop form submission.
Этот код я привёл в качестве теста, чтобы показать, что в моём случае блокирование кнопки не срабатывает моментально и в консоль лог выводится несколько раз. А Вообще возвращаю false, так как у себя в реальном проекте использую ajax для отправки формы.Loveorigami писал(а):Вы сами останавливаете submit формы, т.к. возвращаете false.
Потому как логично, что в ином случае форма будет отправлена несколько раз, если кнопка не будет заблокирована мгновенно.rugabarbo писал(а):А почему должно быть "моментально"? (:
вас спрашивают о логике в контексте работы браузера, а не о логике ваших ожиданий.jekahm писал(а):Потому как логично, что в ином случае форма будет отправлена несколько раз, если кнопка не будет заблокирована мгновенно.rugabarbo писал(а):А почему должно быть "моментально"? (:
В идеале да, но включите JS-профайлер и посмотрите, сколько там всего происходит и может происходить до вызова beforeSubmit. Не удивительно, что при зажатом Enter успевает проскочить 2-3 клика - это всё-таки асинхронная среда.jekahm писал(а):Потому как логично, что в ином случае форма будет отправлена несколько раз, если кнопка не будет заблокирована мгновенно.rugabarbo писал(а):А почему должно быть "моментально"? (:
В идеале после прохождения валидации должно быть, как здесь https://jsfiddle.net/m146a9b2/
Код: Выделить всё
$('.btn').prop('disabled', true); 
Благодарю вас, а так же участников данной темыrugabarbo писал(а):В идеале да, но включите JS-профайлер и посмотрите, сколько там всего происходит и может происходить до вызова beforeSubmit. Не удивительно, что при зажатом Enter успевает проскочить 2-3 клика - это всё-таки асинхронная среда.jekahm писал(а):Потому как логично, что в ином случае форма будет отправлена несколько раз, если кнопка не будет заблокирована мгновенно.rugabarbo писал(а):А почему должно быть "моментально"? (:
В идеале после прохождения валидации должно быть, как здесь https://jsfiddle.net/m146a9b2/
Вот здесь идёт обсуждение защиты от double-сабмитов на серверной стороне: https://github.com/yiisoft/yii2/issues/10498
Если вам нужно 100% защититься от двойных кликов, то это не решить с помощью JavaScript. Я тоже делаю для всех важных кнопок:
- и в 99.9% случаев это позволяет защититься. Но практика показывает, что в 0.1% случаев всё-таки находятся кретины, успевающие нажать сабмит очень-очень быстро два раза подрядКод: Выделить всё
$('.btn').prop('disabled', true);
Хотите гарантий - это не про JavaScript.
JavaScript - про удобство клиента.
Теперь пришло четкое осознание того, что многие вещи не могут быть реализованы и достигнуты только за счет client-side технологий. И спасибо за ссылку о server-side решении.