[PATCH conductor] Add delete confirmation to Pools filter view pages

[Tomas Sedovic]

From: Tomas Sedovic

This is somewhat related to the bug #717538

https://bugzilla.redhat.com/show_bug.cgi?id=717538

When a table of objects was loaded via ajax (for example by switching to the
Filter View), the javascript click handlers set up in `multiDestroyValidation`
under application.js were lost.

As a result, some Delete buttons no longer asked for confirmation on delete.
---
 src/public/javascripts/application.js |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/public/javascripts/application.js b/src/public/javascripts/application.js
index 975cf92..e6b5c42 100644
--- a/src/public/javascripts/application.js
+++ b/src/public/javascripts/application.js
@@ -147,7 +147,7 @@ var Conductor = {
   },

multiDestroyValidation: function() {
-    $('#delete_button').click(function(e) {
+    $('#delete_button').live('click', function(e) {
       if ($(".checkbox_table input[@type=radio]:checked").length == 0) {
         alert('Please make a selection before clicking Delete button.');
         e.preventDefault();

[Tomas Hrcka]

On Thu, 30 Jun 2011 16:44:44 +0200
[e-mail] wrote:
> From: Tomas Sedovic
>
> This is somewhat related to the bug #717538
>
> https://bugzilla.redhat.com/show_bug.cgi?id=717538
>
> When a table of objects was loaded via ajax (for example by switching
> to the Filter View), the javascript click handlers set up in
> `multiDestroyValidation` under application.js were lost.
>
> As a result, some Delete buttons no longer asked for confirmation on
> delete. ---
>  src/public/javascripts/application.js |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/public/javascripts/application.js
> b/src/public/javascripts/application.js index 975cf92..e6b5c42 100644
> --- a/src/public/javascripts/application.js
> +++ b/src/public/javascripts/application.js
> @@ -147,7 +147,7 @@ var Conductor = {
>    },
>
> multiDestroyValidation: function() {
> $('#delete_button').click(function(e) {
> $('#delete_button').live('click', function(e) {
>   if ($(".checkbox_table input[@type=radio]:checked").length ==
> { alert('Please make a selection before clicking Delete button.');
>       e.preventDefault();

ACK.