From ad8d078c871c80d3f2f8a3664a75ce98008bf9e2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9gis=20Hanol?= <regis@hanol.fr>
Date: Thu, 21 Mar 2013 01:25:41 +0100
Subject: [PATCH] fix the issue regarding the unban button

---
 .../javascripts/admin/models/admin_user.js    | 23 ++++----
 .../admin/templates/user.js.handlebars        | 54 +++++++++----------
 2 files changed, 36 insertions(+), 41 deletions(-)

diff --git a/app/assets/javascripts/admin/models/admin_user.js b/app/assets/javascripts/admin/models/admin_user.js
index 1bc09ddb9..0afcbb130 100644
--- a/app/assets/javascripts/admin/models/admin_user.js
+++ b/app/assets/javascripts/admin/models/admin_user.js
@@ -71,15 +71,18 @@ Discourse.AdminUser = Discourse.Model.extend({
     return Discourse.get('site.trust_levels').findProperty('id', this.get('trust_level'));
   }).property('trust_level'),
 
+  isBanned: (function() {
+    return this.get('is_banned') === true;
+  }).property('is_banned'),
+
   canBan: (function() {
-    return !this.admin && !this.moderator;
+    return !this.get('admin') && !this.get('moderator');
   }).property('admin', 'moderator'),
 
   banDuration: (function() {
-    var banned_at, banned_till;
-    banned_at = Date.create(this.banned_at);
-    banned_till = Date.create(this.banned_till);
-    return "" + (banned_at.short()) + " - " + (banned_till.short());
+    var banned_at = Date.create(this.banned_at);
+    var banned_till = Date.create(this.banned_till);
+    return banned_at.short() + " - " + banned_till.short();
   }).property('banned_till', 'banned_at'),
 
   ban: function() {
@@ -94,10 +97,7 @@ Discourse.AdminUser = Discourse.Model.extend({
             window.location.reload();
           },
           error: function(e) {
-            var error;
-            error = Em.String.i18n('admin.user.ban_failed', {
-              error: "http: " + e.status + " - " + e.body
-            });
+            var error = Em.String.i18n('admin.user.ban_failed', { error: "http: " + e.status + " - " + e.body });
             bootbox.alert(error);
           }
         });
@@ -113,10 +113,7 @@ Discourse.AdminUser = Discourse.Model.extend({
         window.location.reload();
       },
       error: function(e) {
-        var error;
-        error = Em.String.i18n('admin.user.unban_failed', {
-          error: "http: " + e.status + " - " + e.body
-        });
+        var error = Em.String.i18n('admin.user.unban_failed', { error: "http: " + e.status + " - " + e.body });
         bootbox.alert(error);
       }
     });
diff --git a/app/assets/javascripts/admin/templates/user.js.handlebars b/app/assets/javascripts/admin/templates/user.js.handlebars
index ca6832ca9..e38d63089 100644
--- a/app/assets/javascripts/admin/templates/user.js.handlebars
+++ b/app/assets/javascripts/admin/templates/user.js.handlebars
@@ -8,13 +8,13 @@
       <a href="{{unbound content.path}}" class='btn'>
         <i class='icon icon-user'></i>
         {{i18n admin.user.show_public_profile}}
-      </a> 
+      </a>
       {{#if content.can_impersonate}}
           <button class='btn' {{action impersonate target="content"}}>
             <i class='icon icon-screenshot'></i>
             {{i18n admin.user.impersonate}}
           </button>
-      {{/if}}      
+      {{/if}}
     </div>
   </div>
   <div class='display-row'>
@@ -44,21 +44,21 @@
     <div class='field'>{{i18n admin.users.approved}}</div>
     <div class='value'>
       {{#if content.approved}}
-        {{i18n admin.user.approved_by}}        
+        {{i18n admin.user.approved_by}}
         <a href="{{unbound content.approved_by.adminPath}}">{{avatar approved_by imageSize="small"}}</a>
         <a href="{{unbound adminPath}}">{{content.approved_by.username}}</a>
       {{else}}
         {{i18n no_value}}
       {{/if}}
-      
+
     </div>
     <div class='controls'>
-      {{#if content.can_approve}}        
+      {{#if content.can_approve}}
         <button class='btn' {{action approve target="content"}}>
           <i class='icon icon-ok'></i>
           {{i18n admin.user.approve}}
         </button>
-      {{/if}}    
+      {{/if}}
     </div>
   </div>
 
@@ -66,7 +66,7 @@
     <div class='field'>{{i18n admin.user.admin}}</div>
     <div class='value'>{{content.admin}}</div>
     <div class='controls'>
-      {{#if content.can_revoke_admin}}        
+      {{#if content.can_revoke_admin}}
         <button class='btn' {{action revokeAdmin target="content"}}>
           <i class='icon icon-trophy'></i>
           {{i18n admin.user.revoke_admin}}
@@ -77,7 +77,7 @@
           <i class='icon icon-trophy'></i>
           {{i18n admin.user.grant_admin}}
         </button>
-      {{/if}}      
+      {{/if}}
     </div>
 
   </div>
@@ -85,7 +85,7 @@
     <div class='field'>{{i18n admin.user.moderator}}</div>
     <div class='value'>{{content.moderator}}</div>
     <div class='controls'>
-      {{#if content.can_revoke_moderation}}        
+      {{#if content.can_revoke_moderation}}
         <button class='btn' {{action revokeModeration target="content"}}>
           <i class='icon icon-eye-close'></i>
           {{i18n admin.user.revoke_moderation}}
@@ -96,26 +96,24 @@
           <i class='icon icon-eye-open'></i>
           {{i18n admin.user.grant_moderation}}
         </button>
-      {{/if}}      
+      {{/if}}
     </div>
-    
+
   </div>
   <div class='display-row'>
     <div class='field'>{{i18n trust_level}}</div>
     <div class='value'>{{content.trustLevel.name}}</div>
-  </div>   
+  </div>
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.banned}}</div>
-    <div class='value'>{{content.is_banned}}</div>
+    <div class='value'>{{content.isBanned}}</div>
     <div class='controls'>
-    {{#if content.is_banned}}
-      {{#if content.canBan}}
-        <button class='btn' {{action unban target="content"}}>
-          <i class='icon icon-screenshot'></i>
-          {{i18n admin.user.unban}}
-        </button>
-        {{content.banDuration}}
-      {{/if}}
+    {{#if content.isBanned}}
+      <button class='btn' {{action unban target="content"}}>
+        <i class='icon icon-screenshot'></i>
+        {{i18n admin.user.unban}}
+      </button>
+      {{content.banDuration}}
     {{else}}
       {{#if content.canBan}}
         <button class='btn' {{action ban target="content"}}>
@@ -125,7 +123,7 @@
       {{/if}}
     {{/if}}
     </div>
-  </div>   
+  </div>
 </section>
 
 <section class='details'>
@@ -150,7 +148,7 @@
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.topics_entered}}</div>
     <div class='value'>{{content.topics_entered}}</div>
-  </div>  
+  </div>
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.post_count}}</div>
     <div class='value'>{{content.post_count}}</div>
@@ -166,15 +164,15 @@
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.posts_read_count}}</div>
     <div class='value'>{{content.posts_read_count}}</div>
-  </div>  
+  </div>
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.flags_given_count}}</div>
     <div class='value'>{{content.flags_given_count}}</div>
-  </div>  
+  </div>
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.flags_received_count}}</div>
     <div class='value'>{{content.flags_received_count}}</div>
-  </div>    
+  </div>
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.private_topics_count}}</div>
     <div class='value'>{{content.private_topics_count}}</div>
@@ -182,10 +180,10 @@
   <div class='display-row'>
     <div class='field'>{{i18n admin.user.time_read}}</div>
     <div class='value'>{{{content.time_read}}}</div>
-  </div>  
+  </div>
   <div class='display-row'>
     <div class='field'>{{i18n user.invited.days_visited}}</div>
     <div class='value'>{{{content.days_visited}}}</div>
-  </div>  
+  </div>
 </section>