From 616f252b4f00ddd0da4ed6bd130c21579f46a6f9 Mon Sep 17 00:00:00 2001
From: jordan <jordan@0cfe37f9-358a-4d5e-be75-b63607b5c754>
Date: Thu, 26 Apr 2012 13:18:41 +0000
Subject: [PATCH] the  variable needs to be defined with var in front of it,
 because otherwise it is taken from the previous scope and messes the dialog
 when two or more of them are used simultaneously

git-svn-id: http://svn.osqa.net/svnroot/osqa/trunk@1256 0cfe37f9-358a-4d5e-be75-b63607b5c754
---
 forum/skins/default/media/js/osqa.main.js            | 3 ++-
 forum/skins/default/templates/node/award_points.html | 5 -----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/forum/skins/default/media/js/osqa.main.js b/forum/skins/default/media/js/osqa.main.js
index c8b8783..0122c25 100644
--- a/forum/skins/default/media/js/osqa.main.js
+++ b/forum/skins/default/media/js/osqa.main.js
@@ -210,7 +210,8 @@ function show_dialog (extern) {
 
     html += '<button class="dialog-yes"' + copy_id + '>' + options.yes_text + '</button>' + '</div></div>';
 
-    $dialog = $(html);
+    var $dialog = $(html);
+
     $('body').append($dialog);
     var message = $('.dialog-content')[0];
     message.style.visibility = "hidden";
diff --git a/forum/skins/default/templates/node/award_points.html b/forum/skins/default/templates/node/award_points.html
index d3e4936..a23ac54 100644
--- a/forum/skins/default/templates/node/award_points.html
+++ b/forum/skins/default/templates/node/award_points.html
@@ -38,11 +38,6 @@
                 html: message,
                 extra_class: 'confirm',
                 yes_text: messages.yes,
-                no_callback: function() {
-                    $(".dialog").fadeOut("fast", function() {
-                        $('.dialog').remove();
-                    })
-                },
                 show_no: true,
                 no_text: messages.no
             }
-- 
2.39.5