From: jordan <jordan@0cfe37f9-358a-4d5e-be75-b63607b5c754>
Date: Thu, 26 Apr 2012 13:18:41 +0000 (+0000)
Subject: the  variable needs to be defined with var in front of it, because otherwise it is... 
X-Git-Tag: live~61
X-Git-Url: https://git.openstreetmap.org/osqa.git/commitdiff_plain/616f252b4f00ddd0da4ed6bd130c21579f46a6f9

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
---

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
             }