Skip to content

Commit

Permalink
bug #750 Fix event listener registered multiple times (ogizanagi)
Browse files Browse the repository at this point in the history
This PR was merged into the master branch.

Discussion
----------

Fix event listener registered multiple times

Fixes #746

Commits
-------

b808add Fix event listener registered multiple times
  • Loading branch information
javiereguiluz committed Jan 4, 2016
2 parents 590fc47 + b808add commit 6525cf5
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion Resources/views/default/edit.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,8 @@
e.preventDefault();
$('#modal-delete').modal({ backdrop: true, keyboard: true })
.one('click', '#modal-delete-button', function (e) {
.off('click', '#modal-delete-button')
.on('click', '#modal-delete-button', function (e) {
$('#delete-form').trigger('submit');
});
});
Expand Down
3 changes: 2 additions & 1 deletion Resources/views/default/list.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -235,7 +235,8 @@
var id = $(this).parents('tr').first().data('id');
$('#modal-delete').modal({ backdrop: true, keyboard: true })
.one('click', '#modal-delete-button', function (e) {
.off('click', '#modal-delete-button')
.on('click', '#modal-delete-button', function (e) {
var deleteForm = $('#delete-form');
deleteForm.attr('action', deleteForm.attr('action').replace('__id__', id));
deleteForm.trigger('submit');
Expand Down
3 changes: 2 additions & 1 deletion Resources/views/default/show.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,8 @@
e.preventDefault();
$('#modal-delete').modal({ backdrop: true, keyboard: true })
.one('click', '#modal-delete-button', function (e) {
.off('click', '#modal-delete-button')
.on('click', '#modal-delete-button', function (e) {
$('#delete-form').trigger('submit');
});
});
Expand Down

0 comments on commit 6525cf5

Please sign in to comment.