From beac9142c2766ed8daf7dd13b6ba22b7895402a5 Mon Sep 17 00:00:00 2001 From: Leandro Gualter Date: Wed, 8 Apr 2015 19:37:46 -0300 Subject: Adds trash actions template and respective javascript --- .../js/mail_list_actions/ui/mail_list_actions.js | 15 +++++-- .../mail_list_actions/ui/recover_many_trigger.js | 47 ++++++++++++++++++++++ web-ui/app/js/views/templates.js | 1 + .../templates/mail_actions/trash_actions_box.hbs | 5 +++ 4 files changed, 65 insertions(+), 3 deletions(-) create mode 100644 web-ui/app/js/mail_list_actions/ui/recover_many_trigger.js create mode 100644 web-ui/app/templates/mail_actions/trash_actions_box.hbs (limited to 'web-ui') diff --git a/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js b/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js index 8987a7d4..2f59d367 100644 --- a/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js +++ b/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js @@ -51,14 +51,13 @@ define( function mailsActions() { this.render = function() { - this.$node.html(templates.mailActions.actionsBox({ - txtDeleteButton: this.getTxtDeleteButton() - })); + this.$node.html(this.getActionsBoxTemplate()); refreshTrigger.attachTo('#refresh-trigger'); composeTrigger.attachTo('#compose-trigger'); toggleCheckAllMailTrigger.attachTo('#toggle-check-all-emails'); paginationTrigger.attachTo('#pagination-trigger'); deleteManyTrigger.attachTo('#delete-selected'); + recoverManyTrigger.attachTo('#recover-selected'); markManyAsReadTrigger.attachTo('#mark-selected-as-read'); markAsUnreadTrigger.attachTo('#mark-selected-as-unread'); refresher.attachTo(document); @@ -81,6 +80,16 @@ define( this.render(); }; + this.getActionsBoxTemplate = function () { + if(this.getCurrentTag() === 'trash') { + return templates.mailActions.trashActionsBox(); + } else { + return templates.mailActions.actionsBox({ + txtDeleteButton: this.getTxtDeleteButton(); + }); + } + }; + this.after('initialize', function () { this.on(document, events.ui.tag.select, this.updateCurrentTag); this.render(); diff --git a/web-ui/app/js/mail_list_actions/ui/recover_many_trigger.js b/web-ui/app/js/mail_list_actions/ui/recover_many_trigger.js new file mode 100644 index 00000000..e0a32094 --- /dev/null +++ b/web-ui/app/js/mail_list_actions/ui/recover_many_trigger.js @@ -0,0 +1,47 @@ +/* + * Copyright (c) 2014 ThoughtWorks, Inc. + * + * Pixelated is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * Pixelated is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with Pixelated. If not, see . + */ +define( + [ + 'flight/lib/component', + 'views/templates', + 'mixins/with_enable_disable_on_event', + 'page/events' + ], + + function(defineComponent, templates, withEnableDisableOnEvent, events) { + 'use strict'; + + return defineComponent(recoverManyTrigger, withEnableDisableOnEvent(events.ui.mails.hasMailsChecked)); + + function recoverManyTrigger() { + this.defaultAttrs({}); + + this.getMailsToRecover = function(event) { + this.trigger(document, events.ui.mail.wantChecked, this.$node); + }; + + this.recoverManyEmails = function (event, data) { + this.trigger(document, events.ui.mail.recoverMany, data); + }; + + this.after('initialize', function () { + this.on('click', this.getMailsToRecover); + this.on(events.ui.mail.hereChecked, this.recoverManyEmails); + }); + } + } +); diff --git a/web-ui/app/js/views/templates.js b/web-ui/app/js/views/templates.js index 64e269ca..5e991a20 100644 --- a/web-ui/app/js/views/templates.js +++ b/web-ui/app/js/views/templates.js @@ -46,6 +46,7 @@ define(['hbs/templates'], function (templates) { }, mailActions: { actionsBox: window.Pixelated['app/templates/mail_actions/actions_box.hbs'], + trashActionsBox: window.Pixelated['app/templates/mail_actions/trash_actions_box.hbs'], composeTrigger: window.Pixelated['app/templates/mail_actions/compose_trigger.hbs'], refreshTrigger: window.Pixelated['app/templates/mail_actions/refresh_trigger.hbs'], paginationTrigger: window.Pixelated['app/templates/mail_actions/pagination_trigger.hbs'] diff --git a/web-ui/app/templates/mail_actions/trash_actions_box.hbs b/web-ui/app/templates/mail_actions/trash_actions_box.hbs new file mode 100644 index 00000000..ad849838 --- /dev/null +++ b/web-ui/app/templates/mail_actions/trash_actions_box.hbs @@ -0,0 +1,5 @@ +
    • +
    • +
    • +
  • +
  • -- cgit v1.2.3 From e5b4ca353863a600a8d6151090b83f3210720a47 Mon Sep 17 00:00:00 2001 From: Roberto Soares Date: Thu, 9 Apr 2015 15:58:22 -0300 Subject: listening event 'recoverMany' --- .../js/mail_list_actions/ui/mail_list_actions.js | 14 ++------ web-ui/app/js/page/default.js | 3 ++ web-ui/app/js/page/events.js | 2 ++ web-ui/app/js/services/mail_service.js | 26 +++++++++++++++ web-ui/app/js/services/recover_service.js | 39 ++++++++++++++++++++++ web-ui/app/templates/mail_actions/actions_box.hbs | 2 +- .../templates/mail_actions/trash_actions_box.hbs | 2 +- .../mail_list_actions/ui/mail_list_actions.spec.js | 18 ++++++++++ web-ui/test/spec/services/mail_service.spec.js | 36 ++++++++++++++++++++ web-ui/test/spec/services/recover_service.spec.js | 32 ++++++++++++++++++ 10 files changed, 161 insertions(+), 13 deletions(-) create mode 100644 web-ui/app/js/services/recover_service.js create mode 100644 web-ui/test/spec/services/recover_service.spec.js (limited to 'web-ui') diff --git a/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js b/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js index 2f59d367..2c9c699f 100644 --- a/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js +++ b/web-ui/app/js/mail_list_actions/ui/mail_list_actions.js @@ -28,6 +28,7 @@ define( 'mail_list_actions/ui/toggle_check_all_trigger', 'mail_list_actions/ui/pagination_trigger', 'mail_list_actions/ui/delete_many_trigger', + 'mail_list_actions/ui/recover_many_trigger', 'mail_list_actions/ui/mark_many_as_read_trigger', 'mail_list_actions/ui/mark_as_unread_trigger' ], @@ -43,6 +44,7 @@ define( toggleCheckAllMailTrigger, paginationTrigger, deleteManyTrigger, + recoverManyTrigger, markManyAsReadTrigger, markAsUnreadTrigger ) { @@ -67,14 +69,6 @@ define( return this.attr.currentTag || urlParams.getTag(); }; - this.getTxtDeleteButton = function() { - if(this.getCurrentTag() === 'trash') { - return 'Delete permanently'; - } else { - return 'Delete'; - } - }; - this.updateCurrentTag = function (ev, data) { this.attr.currentTag = data.tag; this.render(); @@ -84,9 +78,7 @@ define( if(this.getCurrentTag() === 'trash') { return templates.mailActions.trashActionsBox(); } else { - return templates.mailActions.actionsBox({ - txtDeleteButton: this.getTxtDeleteButton(); - }); + return templates.mailActions.actionsBox(); } }; diff --git a/web-ui/app/js/page/default.js b/web-ui/app/js/page/default.js index 57c7fd00..16918d90 100644 --- a/web-ui/app/js/page/default.js +++ b/web-ui/app/js/page/default.js @@ -28,6 +28,7 @@ define( 'mail_view/data/mail_sender', 'services/mail_service', 'services/delete_service', + 'services/recover_service', 'tags/ui/tag_list', 'tags/data/tags', 'page/router', @@ -58,6 +59,7 @@ define( mailSender, mailService, deleteService, + recoverService, tagList, tags, router, @@ -92,6 +94,7 @@ define( mailService.attachTo(document); deleteService.attachTo(document); + recoverService.attachTo(document); tags.attachTo(document); tagList.attachTo('#tag-list'); diff --git a/web-ui/app/js/page/events.js b/web-ui/app/js/page/events.js index f1f426f7..cf1b29ad 100644 --- a/web-ui/app/js/page/events.js +++ b/web-ui/app/js/page/events.js @@ -55,6 +55,7 @@ define(function () { updateSelected: 'ui:mail:updateSelected', delete: 'ui:mail:delete', deleteMany: 'ui:mail:deleteMany', + recoverMany: 'ui:mail:recoverMany', wantChecked: 'ui:mail:wantChecked', hereChecked: 'ui:mail:hereChecked', checked: 'ui:mail:checked', @@ -108,6 +109,7 @@ define(function () { unread: 'mail:unread', delete: 'mail:delete', deleteMany: 'mail:deleteMany', + recoverMany: 'mail:recoverMany', deleted: 'mail:deleted', saveDraft: 'draft:save', draftSaved: 'draft:saved', diff --git a/web-ui/app/js/services/mail_service.js b/web-ui/app/js/services/mail_service.js index 1fa41619..04194964 100644 --- a/web-ui/app/js/services/mail_service.js +++ b/web-ui/app/js/services/mail_service.js @@ -123,6 +123,16 @@ define( }, this); }; + this.triggerRecovered = function (dataToRecover) { + return _.bind(function () { + var mails = dataToRecover.mails || [dataToRecover.mail]; + + this.refreshMails(); + this.trigger(document, events.ui.userAlerts.displayMessage, { message: dataToRecover.successMessage}); + this.trigger(document, events.ui.mails.uncheckAll); + }, this); + }; + this.deleteMail = function (ev, data) { monitoredAjax(this, '/mail/' + data.mail.ident, {type: 'DELETE'}) @@ -145,6 +155,21 @@ define( .fail(this.errorMessage(i18n('Could not delete emails'))); }; + this.recoverManyMails = function (ev, data) { + var dataToRecover = data; + var mailIdents = _.map(data.mails, function (mail) { + return mail.ident; + }); + + monitoredAjax(this, '/mails/recover', { + type: 'POST', + dataType: 'json', + contentType: 'application/json; charset=utf-8', + data: JSON.stringify({idents: mailIdents}) + }).done(this.triggerRecovered(dataToRecover)) + .fail(this.errorMessage(i18n('Could not move emails to inbox'))); + }; + function compileQuery(data) { var query = 'tag:"' + that.attr.currentTag + '"'; @@ -273,6 +298,7 @@ define( this.on(document, events.mail.unread, this.unreadMail); this.on(document, events.mail.delete, this.deleteMail); this.on(document, events.mail.deleteMany, this.deleteManyMails); + this.on(document, events.mail.recoverMany, this.recoverManyMails); this.on(document, events.search.perform, this.newSearch); this.on(document, events.ui.tag.selected, this.fetchByTag); this.on(document, events.ui.tag.select, this.fetchByTag); diff --git a/web-ui/app/js/services/recover_service.js b/web-ui/app/js/services/recover_service.js new file mode 100644 index 00000000..c3fbc98c --- /dev/null +++ b/web-ui/app/js/services/recover_service.js @@ -0,0 +1,39 @@ +/* + * Copyright (c) 2014 ThoughtWorks, Inc. + * + * Pixelated is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * Pixelated is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with Pixelated. If not, see . + */ + +define(['flight/lib/component', 'page/events', 'views/i18n'], function (defineComponent, events, i18n) { + 'use strict'; + + return defineComponent(function() { + + this.recoverManyEmails = function (event, data) { + debugger; + var emails = _.values(data.checkedMails); + + this.trigger(document, events.mail.recoverMany, { + mails: emails, + successMessage: i18n('Your messages were moved to inbox!') + }); + + }; + + this.after('initialize', function () { + this.on(document, events.ui.mail.recoverMany, this.recoverManyEmails); + }); + + }); +}); diff --git a/web-ui/app/templates/mail_actions/actions_box.hbs b/web-ui/app/templates/mail_actions/actions_box.hbs index 628a6240..b6dc2f53 100644 --- a/web-ui/app/templates/mail_actions/actions_box.hbs +++ b/web-ui/app/templates/mail_actions/actions_box.hbs @@ -1,6 +1,6 @@
    • -
    • +
  • diff --git a/web-ui/app/templates/mail_actions/trash_actions_box.hbs b/web-ui/app/templates/mail_actions/trash_actions_box.hbs index ad849838..7852bd6f 100644 --- a/web-ui/app/templates/mail_actions/trash_actions_box.hbs +++ b/web-ui/app/templates/mail_actions/trash_actions_box.hbs @@ -1,5 +1,5 @@
    • -
    • +
  • diff --git a/web-ui/test/spec/mail_list_actions/ui/mail_list_actions.spec.js b/web-ui/test/spec/mail_list_actions/ui/mail_list_actions.spec.js index 7f7ba64a..d8917ed9 100644 --- a/web-ui/test/spec/mail_list_actions/ui/mail_list_actions.spec.js +++ b/web-ui/test/spec/mail_list_actions/ui/mail_list_actions.spec.js @@ -28,6 +28,24 @@ describeComponent('mail_list_actions/ui/mail_list_actions', function () { expect(this.component.$node.html()).toMatch('
    • '); }); + + it('should render move to inbox if on trash', function () { + var urlParams = require('page/router/url_params'); + spyOn(urlParams, 'getTag').and.returnValue('trash'); + + this.setupComponent(); + + expect(this.component.$node.html()).toMatch('
    • '); + }); + + it('should not render move to inbox if on trash', function () { + var urlParams = require('page/router/url_params'); + spyOn(urlParams, 'getTag').and.returnValue('inbox'); + + this.setupComponent(); + + expect(this.component.$node.html()).not.toMatch('
    • '); + }); }); }); diff --git a/web-ui/test/spec/services/mail_service.spec.js b/web-ui/test/spec/services/mail_service.spec.js index 1a5cb9f7..7fb2bfda 100644 --- a/web-ui/test/spec/services/mail_service.spec.js +++ b/web-ui/test/spec/services/mail_service.spec.js @@ -174,6 +174,42 @@ describeComponent('services/mail_service', function () { expect(spyEvent).toHaveBeenTriggeredOnAndWith(document, {message: i18n('Could not delete email')} ); }); + it('will try to recover a message when requested to', function() { + var spyAjax = spyOn($, 'ajax').and.returnValue($.Deferred()); + this.component.trigger(Pixelated.events.mail.recoverMany, {mails: [{ident: '43'}, {ident: '44'}]}); + expect(spyAjax).toHaveBeenCalled(); + expect(spyAjax.calls.mostRecent().args[0]).toEqual('/mails/recover'); + expect(spyAjax.calls.mostRecent().args[1].type).toEqual('POST'); + expect(spyAjax.calls.all()[0].args[1].data).toEqual(JSON.stringify({ idents: ['43', '44'] } )); + }); + + describe('when successfuly recovers emails', function () { + var displayMessageEvent, uncheckAllEvent, mailsRecoveredEvent; + + beforeEach(function () { + displayMessageEvent = spyOnEvent(document, Pixelated.events.ui.userAlerts.displayMessage); + uncheckAllEvent = spyOnEvent(document, Pixelated.events.ui.mails.uncheckAll); + spyOn(this.component, 'refreshMails'); + + this.component.triggerRecovered({ + successMessage: 'A success message', + mails: {1: 'email 1', 2: 'email 2'} + })(); + }); + + it('will trigger that a message has been recovered when it is done recovering', function() { + expect(this.component.refreshMails).toHaveBeenCalled(); + }); + + it('displays a success message', function () { + expect(displayMessageEvent).toHaveBeenTriggeredOnAndWith(document, {message: 'A success message'}); + }); + + it('unchecks all checked mails', function () { + expect(uncheckAllEvent).toHaveBeenTriggeredOn(document); + }); + }); + it('triggers mails:available with received mails and keeps that tag as the current tag', function() { var eventSpy = spyOnEvent(document, Pixelated.events.mails.available); diff --git a/web-ui/test/spec/services/recover_service.spec.js b/web-ui/test/spec/services/recover_service.spec.js new file mode 100644 index 00000000..86fe9f87 --- /dev/null +++ b/web-ui/test/spec/services/recover_service.spec.js @@ -0,0 +1,32 @@ +describeComponent('services/recover_service', function () { + 'use strict'; + + var i18n; + + beforeEach( function () { + this.setupComponent(); + i18n = require('views/i18n'); + }); + + var mail1 = { + ident: 42, + isInTrash: function() { return false; } + }; + + var mail2 = { + ident: 34, + isInTrash: function() { return true; } + }; + + it('moves selected emails from trash back to inbox', function () { + var mailRecoverManyEvent = spyOnEvent(document, Pixelated.events.mail.recoverMany); + this.component.trigger(document, Pixelated.events.ui.mail.recoverMany, {checkedMails: {mail1: mail1, mail2: mail2}}); + + var expectedRecoverManyEventData = { + mails: [mail1, mail2], + successMessage: i18n('Your messages were moved to inbox!') + }; + + expect(mailRecoverManyEvent).toHaveBeenTriggeredOnAndWith(document, expectedRecoverManyEventData); + }); +}); -- cgit v1.2.3 From e6931bed853a798acaced4ae92985866b2a32691 Mon Sep 17 00:00:00 2001 From: Roberto Soares Date: Thu, 9 Apr 2015 16:42:08 -0300 Subject: removes debugger (whoops) --- web-ui/app/js/services/recover_service.js | 1 - 1 file changed, 1 deletion(-) (limited to 'web-ui') diff --git a/web-ui/app/js/services/recover_service.js b/web-ui/app/js/services/recover_service.js index c3fbc98c..62fcf1f9 100644 --- a/web-ui/app/js/services/recover_service.js +++ b/web-ui/app/js/services/recover_service.js @@ -21,7 +21,6 @@ define(['flight/lib/component', 'page/events', 'views/i18n'], function (defineCo return defineComponent(function() { this.recoverManyEmails = function (event, data) { - debugger; var emails = _.values(data.checkedMails); this.trigger(document, events.mail.recoverMany, { -- cgit v1.2.3