From: jordan <jordan@0cfe37f9-358a-4d5e-be75-b63607b5c754>
Date: Thu, 26 Apr 2012 00:51:30 +0000 (+0000)
Subject: Users should be warned before getting their karma points decreased to certain values... 
X-Git-Tag: live~62
X-Git-Url: https://git.openstreetmap.org./osqa.git/commitdiff_plain/cc041eef8cd45e535ba702e46822d10f6939bfc5

Users should be warned before getting their karma points decreased to certain values on reputation award

git-svn-id: http://svn.osqa.net/svnroot/osqa/trunk@1255 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 89a59ef..c8b8783 100644
--- a/forum/skins/default/media/js/osqa.main.js
+++ b/forum/skins/default/media/js/osqa.main.js
@@ -173,7 +173,7 @@ function show_dialog (extern) {
         $diag.fadeOut('fast', function() {
             $diag.remove();
         });
-    }
+    };
 
     var options = {
         extra_class: '',
@@ -201,7 +201,7 @@ function show_dialog (extern) {
         options.pos = {x: options.event.pageX, y: options.event.pageY};
     }
 
-    var html = '<div class="dialog ' + options.extra_class + '" style="display: none;">'
+    var html = '<div class="dialog ' + options.extra_class + '" style="display: none; z-index: 999;">'
              + '<div class="dialog-content">' + options.html + '</div><div class="dialog-buttons">';
 
     if (options.show_no) {
@@ -263,14 +263,20 @@ function show_dialog (extern) {
         message.style.visibility = "visible";
     });
 
-    $dialog.find('.dialog-no').click(function() {
-        default_close_function($dialog);
-    });
-
     $dialog.find('.dialog-yes').click(function() {
         options.yes_callback($dialog);
     });
 
+    if (options.hasOwnProperty("no_callback")) {
+        $dialog.find('.dialog-no:first-child').click(function() {
+            options.no_callback($dialog);
+        });
+    } else {
+        $dialog.find('.dialog-no:first-child').click(function() {
+            default_close_function($dialog);
+        });
+    }
+
     if (options.close_on_clickoutside) {
         $dialog.one('clickoutside', function() {
             default_close_function($dialog);
diff --git a/forum/skins/default/templates/node/award_points.html b/forum/skins/default/templates/node/award_points.html
index a031fa0..d3e4936 100644
--- a/forum/skins/default/templates/node/award_points.html
+++ b/forum/skins/default/templates/node/award_points.html
@@ -1,15 +1,15 @@
 {% load i18n %}
 
 <p>
-{% blocktrans %}
-If you like this answer you can award {{ awarded_user }} with reputation points.
-{% endblocktrans %}
+    {% blocktrans %}
+    If you like this answer you can award {{ awarded_user }} with reputation points.
+    {% endblocktrans %}
 </p>
 
 <p>
-{% blocktrans %}
-Use the slider below or the input field to choose the amount of points you would like to give.
-{% endblocktrans %}
+    {% blocktrans %}
+    Use the slider below or the input field to choose the amount of points you would like to give.
+    {% endblocktrans %}
 </p>
 
 <table>
@@ -22,18 +22,51 @@ Use the slider below or the input field to choose the amount of points you would
 <div class="award_points_slider"></div>
 
 <script>
-$('.award_points_slider').slider({
-    min: 1, max: {{ user.reputation }},
+    var awardPointsNotified = false;
+    var reputation_to_comment = {{ reputation_to_comment }};
+    var user_reputation = {{ user.reputation }};
+
+    function points_award_confirm(reputation, event) {
+        var message = '<div style="max-width: 300px !important; text-align: left;">';
+        message += "{% blocktrans %}Your karma points will be below the minimum reputation to comment. Are you sure you want to award points?{% endblocktrans %}";
+        message += "</div>";
+
+        if ((user_reputation - reputation) < reputation_to_comment && !awardPointsNotified) {
+            awardPointsNotified = true;
+
+            var doptions = {
+                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
+            }
+
+            doptions.event = event;
+            show_dialog(doptions);
+        }
+    }
+
+    $('.award_points_slider').slider({
+        min: 1, max: user_reputation,
     slide: function(evt, ui) {
-        $('#points_input').val(ui.value)
+        $('#points_input').val(ui.value);
+        points_award_confirm(Number(ui.value), evt)
     }
-});
+    });
 
-$('#points_input').bind('change keydown', function() {
-    if ($(this).val() > {{ user.reputation }}) {
-        $(this).attr('value', {{ user.reputation }})
+    $('#points_input').bind('change keydown', function(evt) {
+        if ($(this).val() > user_reputation) {
+        $(this).attr('value', user_reputation);
     }
     $('.award_points_slider').slider('option', 'value', $(this).val())
-})
+
+    points_award_confirm(Number($(this).val()), evt);
+    });
 
 </script>
\ No newline at end of file
diff --git a/forum/templatetags/node_tags.py b/forum/templatetags/node_tags.py
index 2bc3c4b..9e0482f 100644
--- a/forum/templatetags/node_tags.py
+++ b/forum/templatetags/node_tags.py
@@ -105,7 +105,7 @@ def post_controls(post, user):
                                      title=_("answer permanent link"), command=True, withprompt=True, copy=True))
 
         # Users should be able to award points for an answer. Users cannot award their own answers
-        if user != post.author and user.is_authenticated():
+        if user != post.author and user.is_authenticated() and user.reputation > 1:
             controls.append(post_control(_("award points"), reverse('award_points', kwargs={'user_id' : post.author.id,
                                          'answer_id' : post.id}), title=_("award points to %s") % smart_unicode(post.author.username),
                                          command=True, withprompt=True))
diff --git a/forum/views/commands.py b/forum/views/commands.py
index 2f35c1e..83a6211 100644
--- a/forum/views/commands.py
+++ b/forum/views/commands.py
@@ -613,7 +613,11 @@ def award_points(request, user_id, answer_id):
         raise AnonymousNotAllowedException(_('award'))
 
     if not request.POST:
-        return render_to_response("node/award_points.html", { 'user' : user, 'awarded_user' : awarded_user, })
+        return render_to_response("node/award_points.html", {
+            'user' : user,
+            'awarded_user' : awarded_user,
+            'reputation_to_comment' : str(settings.REP_TO_COMMENT)
+        })
     else:
         points = int(request.POST['points'])