diff --git a/app/assets/javascripts/discourse/controllers/preferences_badge_title_controller.js b/app/assets/javascripts/discourse/controllers/preferences_badge_title_controller.js new file mode 100644 index 00000000000..85352626d87 --- /dev/null +++ b/app/assets/javascripts/discourse/controllers/preferences_badge_title_controller.js @@ -0,0 +1,64 @@ +/** + Controller for selecting a badge to use as your title. + + @class PreferencesBadgeTitleController + @extends Ember.ArrayController + @namespace Discourse + @module Discourse +**/ +Discourse.PreferencesBadgeTitleController = Ember.ArrayController.extend({ + saving: false, + saved: false, + + savingStatus: function() { + if (this.get('saving')) { + return I18n.t('saving'); + } else { + return I18n.t('save'); + } + }.property('saving'), + + selectableUserBadges: Em.computed.filter('model', function(userBadge) { + var badgeType = userBadge.get('badge.badge_type.name'); + return (badgeType === "Gold" || badgeType === "Silver"); + }), + + selectedUserBadge: function() { + var selectedUserBadgeId = parseInt(this.get('selectedUserBadgeId')); + var selectedUserBadge = null; + this.get('selectableUserBadges').forEach(function(userBadge) { + if (userBadge.get('id') === selectedUserBadgeId) { + selectedUserBadge = userBadge; + } + }); + return selectedUserBadge; + }.property('selectedUserBadgeId'), + + titleNotChanged: function() { + return this.get('user.title') === this.get('selectedUserBadge.badge.name'); + }.property('selectedUserBadge', 'user.title'), + + disableSave: Em.computed.or('saving', 'titleNotChanged'), + + actions: { + save: function() { + var self = this; + + self.set('saved', false); + self.set('saving', true); + + Discourse.ajax("/users/" + self.get('user.username_lower') + "/preferences/badge_title", { + type: "PUT", + data: { + user_badge_id: self.get('selectedUserBadgeId') + } + }).then(function() { + self.set('saved', true); + self.set('saving', false); + self.set('user.title', self.get('selectedUserBadge.badge.name')); + }, function() { + bootbox.alert(I18n.t('generic_error')); + }); + } + } +}); diff --git a/app/assets/javascripts/discourse/controllers/preferences_controller.js b/app/assets/javascripts/discourse/controllers/preferences_controller.js index c5f6c143054..1467b87ba97 100644 --- a/app/assets/javascripts/discourse/controllers/preferences_controller.js +++ b/app/assets/javascripts/discourse/controllers/preferences_controller.js @@ -38,6 +38,17 @@ Discourse.PreferencesController = Discourse.ObjectController.extend({ return Discourse.SiteSettings.enable_names; }.property(), + canSelectTitle: function() { + if (!Discourse.SiteSettings.enable_badges || this.get('model.badge_count') === 0) { + return false; + } + + // If the first featured badge isn't gold or silver we know the user won't have + // _any_ gold or silver badges. + var badgeType = this.get('model.featured_user_badges')[0].get('badge.badge_type.name'); + return (badgeType === "Gold" || badgeType === "Silver"); + }.property('model.badge_count', 'model.featured_user_badges.@each.badge.badge_type.name'), + availableLocales: function() { return Discourse.SiteSettings.available_locales.split('|').map( function(s) { return {name: s, value: s}; diff --git a/app/assets/javascripts/discourse/routes/application_routes.js b/app/assets/javascripts/discourse/routes/application_routes.js index 9e46bfb9b75..e14386703af 100644 --- a/app/assets/javascripts/discourse/routes/application_routes.js +++ b/app/assets/javascripts/discourse/routes/application_routes.js @@ -81,6 +81,7 @@ Discourse.Route.buildRoutes(function() { this.route('username'); this.route('email'); this.route('about', { path: '/about-me' }); + this.route('badgeTitle', { path: '/badge_title' }); }); this.route('invited'); diff --git a/app/assets/javascripts/discourse/routes/preferences_routes.js b/app/assets/javascripts/discourse/routes/preferences_routes.js index fbf0d63b2a7..ef59b99d418 100644 --- a/app/assets/javascripts/discourse/routes/preferences_routes.js +++ b/app/assets/javascripts/discourse/routes/preferences_routes.js @@ -44,7 +44,7 @@ Discourse.PreferencesRoute = Discourse.RestrictedUserRoute.extend({ user.set('avatar_template', avatarSelector.get('avatarTemplate')); avatarSelector.send('closeModal'); }, - + showProfileBackgroundFileSelector: function() { $("#profile-background-input").click(); }, @@ -161,3 +161,43 @@ Discourse.PreferencesUsernameRoute = Discourse.RestrictedUserRoute.extend({ controller.setProperties({ model: user, newUsername: user.get('username') }); } }); + +/** + The route for updating a user's title to one of their badges + + @class PreferencesBadgeTitleRoute + @extends Discourse.RestrictedUserRoute + @namespace Discourse + @module Discourse +**/ +Discourse.PreferencesBadgeTitleRoute = Discourse.RestrictedUserRoute.extend({ + model: function() { + return Discourse.UserBadge.findByUsername(this.modelFor('user').get('username')); + }, + + renderTemplate: function() { + return this.render('user/badge-title', { into: 'user', outlet: 'userOutlet' }); + }, + + // A bit odd, but if we leave to /preferences we need to re-render that outlet + deactivate: function() { + this._super(); + this.render('preferences', { into: 'user', outlet: 'userOutlet', controller: 'preferences' }); + }, + + setupController: function(controller, model) { + controller.set('model', model); + controller.set('user', this.modelFor('user')); + + model.forEach(function(userBadge) { + if (userBadge.get('badge.name') === controller.get('user.title')) { + controller.set('selectedUserBadgeId', userBadge.get('id')); + } + }); + if (!controller.get('selectedUserBadgeId')) { + controller.set('selectedUserBadgeId', controller.get('selectableUserBadges')[0].get('id')); + } + } +}); + + diff --git a/app/assets/javascripts/discourse/templates/user/badge-title.js.handlebars b/app/assets/javascripts/discourse/templates/user/badge-title.js.handlebars new file mode 100644 index 00000000000..6bf807b8de1 --- /dev/null +++ b/app/assets/javascripts/discourse/templates/user/badge-title.js.handlebars @@ -0,0 +1,25 @@ +
+
+ +
+
+

{{i18n badges.select_badge_for_title}}

+
+
+ +
+ +
+ {{combobox valueAttribute="id" value=selectedUserBadgeId nameProperty="badge.name" content=selectableUserBadges}} +
+
+ +
+
+ + {{#if saved}}{{i18n saved}}{{/if}} +
+
+ +
+
diff --git a/app/assets/javascripts/discourse/templates/user/preferences.js.handlebars b/app/assets/javascripts/discourse/templates/user/preferences.js.handlebars index be9d804b6cd..846b486473a 100644 --- a/app/assets/javascripts/discourse/templates/user/preferences.js.handlebars +++ b/app/assets/javascripts/discourse/templates/user/preferences.js.handlebars @@ -37,6 +37,16 @@ {{/if}} + {{#if canSelectTitle}} +
+ +
+ {{title}} + {{#link-to "preferences.badgeTitle" class="btn pad-left"}}{{/link-to}} +
+
+ {{/if}} +
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb index 24a3d9443ae..9aca2800fc9 100644 --- a/app/controllers/users_controller.rb +++ b/app/controllers/users_controller.rb @@ -61,6 +61,21 @@ class UsersController < ApplicationController render nothing: true end + def badge_title + params.require(:user_badge_id) + + user = fetch_user_from_params + guardian.ensure_can_edit!(user) + + user_badge = UserBadge.find(params[:user_badge_id]) + if user_badge.user == user && ["Gold", "Silver"].include?(user_badge.badge.badge_type.name) + user.title = user_badge.badge.name + user.save! + end + + render nothing: true + end + def preferences render nothing: true end diff --git a/app/services/badge_granter.rb b/app/services/badge_granter.rb index 53b3d1fb965..8b4de1a54f7 100644 --- a/app/services/badge_granter.rb +++ b/app/services/badge_granter.rb @@ -40,8 +40,15 @@ class BadgeGranter if options[:revoked_by] StaffActionLogger.new(options[:revoked_by]).log_badge_revoke(user_badge) end - # Revoke badge -- This is inefficient, but not very easy to optimize unless - # the data hash is converted into a hstore. + + # If the user's title is the same as the badge name, remove their title. + if user_badge.user.title == user_badge.badge.name + user_badge.user.title = nil + user_badge.user.save! + end + + # Delete notification -- This is inefficient, but not very easy to optimize + # unless the data hash is converted into a hstore. notification = user_badge.user.notifications.where(notification_type: Notification.types[:granted_badge]).where("data LIKE ?", "%" + user_badge.badge_id.to_s + "%").select {|n| n.data_hash["badge_id"] == user_badge.badge_id }.first notification && notification.destroy end diff --git a/config/locales/client.en.yml b/config/locales/client.en.yml index 02dd9a36932..b4f8745c881 100644 --- a/config/locales/client.en.yml +++ b/config/locales/client.en.yml @@ -1779,6 +1779,7 @@ en: awarded: one: "1 awarded" other: "%{count} awarded" + select_badge_for_title: Select a badge to use as your title example_badge: name: Example Badge description: This is a generic example badge. diff --git a/config/routes.rb b/config/routes.rb index 137e2434795..4d6a44ff475 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -184,6 +184,8 @@ Discourse::Application.routes.draw do get "users/:username/preferences/email" => "users#preferences", constraints: {username: USERNAME_ROUTE_FORMAT} put "users/:username/preferences/email" => "users#change_email", constraints: {username: USERNAME_ROUTE_FORMAT} get "users/:username/preferences/about-me" => "users#preferences", constraints: {username: USERNAME_ROUTE_FORMAT} + get "users/:username/preferences/badge_title" => "users#preferences", constraints: {username: USERNAME_ROUTE_FORMAT} + put "users/:username/preferences/badge_title" => "users#badge_title", constraints: {username: USERNAME_ROUTE_FORMAT} get "users/:username/preferences/username" => "users#preferences", constraints: {username: USERNAME_ROUTE_FORMAT} put "users/:username/preferences/username" => "users#username", constraints: {username: USERNAME_ROUTE_FORMAT} get "users/:username/avatar(/:size)" => "users#avatar", constraints: {username: USERNAME_ROUTE_FORMAT} # LEGACY ROUTE diff --git a/spec/services/badge_granter_spec.rb b/spec/services/badge_granter_spec.rb index 476bc6a4b4a..dcdbd7008c3 100644 --- a/spec/services/badge_granter_spec.rb +++ b/spec/services/badge_granter_spec.rb @@ -53,13 +53,15 @@ describe BadgeGranter do let(:admin) { Fabricate(:admin) } let!(:user_badge) { BadgeGranter.grant(badge, user) } - it 'revokes the badge, deletes the notification and decrements grant_count' do + it 'revokes the badge and does necessary cleanup' do + user.title = badge.name; user.save! badge.reload.grant_count.should eq(1) StaffActionLogger.any_instance.expects(:log_badge_revoke).with(user_badge) BadgeGranter.revoke(user_badge, revoked_by: admin) UserBadge.where(user: user, badge: badge).first.should_not be_present badge.reload.grant_count.should eq(0) user.notifications.where(notification_type: Notification.types[:granted_badge]).should be_empty + user.reload.title.should == nil end end