From 2daa6161fd08e18bdf7f1ad0f30a83a58226543e Mon Sep 17 00:00:00 2001 From: gitlabhq Date: Mon, 17 Oct 2011 20:05:47 +0300 Subject: [PATCH] prevent double note submit --- app/views/notes/_form.html.haml | 2 +- app/views/notes/_notes.html.haml | 9 +++++++++ app/views/notes/create.js.haml | 3 +++ 3 files changed, 13 insertions(+), 1 deletion(-) diff --git a/app/views/notes/_form.html.haml b/app/views/notes/_form.html.haml index ccb159f12..8eaf968c8 100644 --- a/app/views/notes/_form.html.haml +++ b/app/views/notes/_form.html.haml @@ -25,4 +25,4 @@ .clear %br - = f.submit 'Add note', :class => "lbutton vm" + = f.submit 'Add note', :class => "lbutton vm", :id => "submit_note" diff --git a/app/views/notes/_notes.html.haml b/app/views/notes/_notes.html.haml index 040c2f280..c93a47239 100644 --- a/app/views/notes/_notes.html.haml +++ b/app/views/notes/_notes.html.haml @@ -9,6 +9,15 @@ $('.delete-note').live('ajax:success', function() { $(this).closest('li').fadeOut(); }); + $("#new_note").live("ajax:before", function(){ + $("#submit_note").attr("disabled", "disabled"); + }) + + $("#new_note").live("ajax:complete", function(){ + $("#submit_note").removeAttr("disabled"); + }) + + - if ["issues", "projects"].include?(controller.controller_name) :javascript $(function(){ diff --git a/app/views/notes/create.js.haml b/app/views/notes/create.js.haml index 47cff1d88..0af548ff8 100644 --- a/app/views/notes/create.js.haml +++ b/app/views/notes/create.js.haml @@ -6,3 +6,6 @@ - else :plain $("#new_note").replaceWith("#{escape_javascript(render('form'))}"); + +:plain + $("#submit_note").removeAttr("disabled"); -- 2.11.0