From 43a783e10f0b89820fb8d1e5677b0fdbcf7e9b6e Mon Sep 17 00:00:00 2001
From: Alexandre Pretto Nunes <anunes@thoughtworks.com>
Date: Thu, 15 Jan 2015 15:04:48 -0200
Subject: #153 fix mail checking and unchecking on mail list

---
 web-ui/app/js/mail_list/ui/mail_items/mail_item.js |  1 -
 web-ui/app/js/mail_list/ui/mail_list.js            | 10 +++++-----
 2 files changed, 5 insertions(+), 6 deletions(-)

(limited to 'web-ui/app/js/mail_list/ui')

diff --git a/web-ui/app/js/mail_list/ui/mail_items/mail_item.js b/web-ui/app/js/mail_list/ui/mail_items/mail_item.js
index 94f08a9f..f7ae2f0c 100644
--- a/web-ui/app/js/mail_list/ui/mail_items/mail_item.js
+++ b/web-ui/app/js/mail_list/ui/mail_items/mail_item.js
@@ -36,7 +36,6 @@ define(
 
     this.doSelect = function () {
       this.$node.addClass('selected');
-      this.checkCheckbox();
     };
 
     this.doUnselect = function () {
diff --git a/web-ui/app/js/mail_list/ui/mail_list.js b/web-ui/app/js/mail_list/ui/mail_list.js
index c2746c88..69327a57 100644
--- a/web-ui/app/js/mail_list/ui/mail_list.js
+++ b/web-ui/app/js/mail_list/ui/mail_list.js
@@ -121,15 +121,15 @@ define(
       };
 
       this.updateCheckAllCheckbox = function () {
-        this.trigger(document, events.ui.mails.hasMailsChecked, {hasMailsChecked: _.keys(this.attr.checkedMails).length > 0});
+        this.trigger(document, events.ui.mails.hasMailsChecked, _.keys(this.attr.checkedMails).length > 0);
       };
 
-      this.addToSelectedMails = function (ev, data) {
+      this.addToCheckedMails = function (ev, data) {
         this.attr.checkedMails[data.mail.ident] = data.mail;
         this.updateCheckAllCheckbox();
       };
 
-      this.removeFromSelectedMails = function (ev, data) {
+      this.removeFromCheckedMails = function (ev, data) {
         if (data.mails) {
           _.each(data.mails, function (mail) {
             delete this.attr.checkedMails[mail.ident];
@@ -169,8 +169,8 @@ define(
 
         this.on(document, events.ui.mail.updateSelected, this.updateSelected);
         this.on(document, events.ui.mail.wantChecked, this.respondWithCheckedMails);
-        this.on(document, events.ui.mail.checked, this.addToSelectedMails);
-        this.on(document, events.ui.mail.unchecked, this.removeFromSelectedMails);
+        this.on(document, events.ui.mail.checked, this.addToCheckedMails);
+        this.on(document, events.ui.mail.unchecked, this.removeFromCheckedMails);
 
         this.openMailFromUrl = utils.once(function () {
           if (this.shouldSelectEmailFromUrlMailIdent()) {
-- 
cgit v1.2.3