From a5aedf850fe93a8acfa40f9d6713924944bf0e30 Mon Sep 17 00:00:00 2001 From: Phillip Thelen Date: Tue, 5 Aug 2014 00:30:53 +0200 Subject: [PATCH 01/50] Add sorting by username and date joined for partymembers --- public/js/controllers/groupsCtrl.js | 2 ++ public/js/controllers/headerCtrl.js | 6 ++++++ 2 files changed, 8 insertions(+) diff --git a/public/js/controllers/groupsCtrl.js b/public/js/controllers/groupsCtrl.js index ca062e95e8..0910bcd176 100644 --- a/public/js/controllers/groupsCtrl.js +++ b/public/js/controllers/groupsCtrl.js @@ -201,6 +201,8 @@ habitrpg.controller("GroupsCtrl", ['$scope', '$rootScope', 'Shared', 'Groups', ' 'level': window.env.t('sortLevel'), 'random': window.env.t('sortRandom'), 'pets': window.env.t('sortPets'), + 'username': window.env.t('sortUsername'), + 'habitrpg_date_joined' : window.env.t('sortHabitrpgJoined'), 'party_date_joined': window.env.t('sortJoined'), }; diff --git a/public/js/controllers/headerCtrl.js b/public/js/controllers/headerCtrl.js index d039bca7bb..3ac7bc90d3 100644 --- a/public/js/controllers/headerCtrl.js +++ b/public/js/controllers/headerCtrl.js @@ -22,6 +22,12 @@ habitrpg.controller("HeaderCtrl", ['$scope', 'Groups', 'User', case 'pets': return member.items.pets.length; break; + case 'username': + return member.profile.name; + break; + case 'habitrpg_date_joined': + return member.auth.timestamps.created; + break default: // party date joined return true; From 4af860ea24062c36fbc5e6bc5941008e4c397d1e Mon Sep 17 00:00:00 2001 From: Phillip Thelen Date: Tue, 5 Aug 2014 00:31:22 +0200 Subject: [PATCH 02/50] allow partymember sorting to be ascending or descending --- public/js/controllers/groupsCtrl.js | 5 +++++ public/js/controllers/headerCtrl.js | 5 ++++- src/models/user.js | 1 + views/options/social/group.jade | 6 ++++++ 4 files changed, 16 insertions(+), 1 deletion(-) diff --git a/public/js/controllers/groupsCtrl.js b/public/js/controllers/groupsCtrl.js index 0910bcd176..0f932f61b0 100644 --- a/public/js/controllers/groupsCtrl.js +++ b/public/js/controllers/groupsCtrl.js @@ -206,6 +206,11 @@ habitrpg.controller("GroupsCtrl", ['$scope', '$rootScope', 'Shared', 'Groups', ' 'party_date_joined': window.env.t('sortJoined'), }; + $scope.partyOrderAscendingChoices = { + 'ascending': window.env.t('ascendingSort'), + 'descending': window.env.t('descendingSort') + } + }]) .controller("GuildsCtrl", ['$scope', 'Groups', 'User', 'Challenges', '$rootScope', '$state', '$location', '$compile', diff --git a/public/js/controllers/headerCtrl.js b/public/js/controllers/headerCtrl.js index 3ac7bc90d3..b826f18141 100644 --- a/public/js/controllers/headerCtrl.js +++ b/public/js/controllers/headerCtrl.js @@ -33,7 +33,10 @@ habitrpg.controller("HeaderCtrl", ['$scope', 'Groups', 'User', return true; } } - ).reverse() + ) + if (User.user.party.orderAscending == "descending") { + $scope.partyMinusSelf = $scope.partyMinusSelf.reverse() + } }); } ]); diff --git a/src/models/user.js b/src/models/user.js index 2877256b14..98a18decc2 100644 --- a/src/models/user.js +++ b/src/models/user.js @@ -229,6 +229,7 @@ var UserSchema = new Schema({ party: { // id // FIXME can we use a populated doc instead of fetching party separate from user? order: {type:String, 'default':'level'}, + orderAscending: {type:String, 'default':'ascending'}, quest: { key: String, progress: { diff --git a/views/options/social/group.jade b/views/options/social/group.jade index 78092b630a..b64c54c78d 100644 --- a/views/options/social/group.jade +++ b/views/options/social/group.jade @@ -59,6 +59,12 @@ a.pull-right.gem-wallet(popover-trigger='mouseenter', popover-title=env.t('guild ng-options='k as v for (k , v) in partyOrderChoices', ng-change='set({"party.order": user.party.order})' ) + select#partyOrderAscending( + ng-model='user.party.orderAscending', + ng-controller='ChatCtrl', + ng-options='k as v for (k , v) in partyOrderAscendingChoices', + ng-change='set({"party.orderAscending": user.party.orderAscending})' + ) table.table.table-striped(bindonce='group') tr(ng-repeat='member in group.members') td From a8888a0a53d267c8161f0c05bafe185c267b7157 Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Tue, 30 Dec 2014 13:05:55 -0600 Subject: [PATCH 03/50] Added last logged in choice for sorting --- public/js/controllers/groupsCtrl.js | 1 + public/js/controllers/headerCtrl.js | 3 +++ 2 files changed, 4 insertions(+) diff --git a/public/js/controllers/groupsCtrl.js b/public/js/controllers/groupsCtrl.js index 9c4a5fd3bc..7549867abe 100644 --- a/public/js/controllers/groupsCtrl.js +++ b/public/js/controllers/groupsCtrl.js @@ -305,6 +305,7 @@ habitrpg.controller("GroupsCtrl", ['$scope', '$rootScope', 'Shared', 'Groups', ' 'pets': window.env.t('sortPets'), 'habitrpg_date_joined' : window.env.t('sortHabitrpgJoined'), 'party_date_joined': window.env.t('sortJoined'), + 'habitrpg_last_logged_in': window.env.t('sortHabitrpgLastLoggedIn'), 'name': window.env.t('sortName'), 'backgrounds': window.env.t('sortBackgrounds'), }; diff --git a/public/js/controllers/headerCtrl.js b/public/js/controllers/headerCtrl.js index 196b785463..ada83f24c2 100644 --- a/public/js/controllers/headerCtrl.js +++ b/public/js/controllers/headerCtrl.js @@ -31,6 +31,9 @@ habitrpg.controller("HeaderCtrl", ['$scope', 'Groups', 'User', case 'habitrpg_date_joined': return member.auth.timestamps.created; break + case 'habitrpg_last_logged_in': + return member.auth.timestamps.loggedin; + break default: // party date joined return true; From 38f7c1e634c62dcfbd8bf4940ed5a5222d3f645f Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Fri, 9 Jan 2015 19:10:10 -0600 Subject: [PATCH 04/50] Closes #4490. Day start saves on blur instead of change --- views/options/settings.jade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 3fb026d635..c5e97bc987 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -75,7 +75,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') h5.hint(popover=env.t('clockInfo'), popover-trigger='mouseenter')=env.t('customDayStart') .form-group .input-group - input.form-control(type='number', min='0', max='23', ng-model='user.preferences.dayStart', ng-change='saveDayStart()') + input.form-control(type='number', min='0', max='23', ng-model='user.preferences.dayStart', ng-blur='saveDayStart()') span.input-group-addon= ':00 (' + env.t('24HrClock') + ')' small =env.t('subWarning1') From c3fce5fcbaf29efa0b64df3a326f357404060044 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 01:18:37 -0500 Subject: [PATCH 05/50] changed Username to Login Name --- views/options/settings.jade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 3fb026d635..4678e16034 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -90,7 +90,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-body p(ng-if='user.auth.facebook.id') Registered with Facebook div(ng-if='user.auth.local.username') - p Username: {{user.auth.local.username}} + p Login Name: {{user.auth.local.username}} p Email: {{user.auth.local.email}} hr h5=env.t('changeUsername') From 998fe684e41b330da89b0fdf939348143706f18b Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 03:20:22 -0500 Subject: [PATCH 06/50] added new translatable string emailChange --- views/options/settings.jade | 2 ++ 1 file changed, 2 insertions(+) diff --git a/views/options/settings.jade b/views/options/settings.jade index 4678e16034..c4b5e6c5a8 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -92,6 +92,8 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') div(ng-if='user.auth.local.username') p Login Name: {{user.auth.local.username}} p Email: {{user.auth.local.email}} + small + =env.t('emailChange') hr h5=env.t('changeUsername') form(ng-submit='changeUsername(changeUser)', ng-show='user.auth.local') From 9907379a60ebf378e3bceb16594d7a4664b96c36 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 10:22:24 -0500 Subject: [PATCH 07/50] plain text to translatable strings --- views/options/settings.jade | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index c4b5e6c5a8..f2c23ac371 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -88,10 +88,10 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-heading span Registration .panel-body - p(ng-if='user.auth.facebook.id') Registered with Facebook + p(ng-if='user.auth.facebook.id') 'registeredWithFb' div(ng-if='user.auth.local.username') - p Login Name: {{user.auth.local.username}} - p Email: {{user.auth.local.email}} + p 'username': {{user.auth.local.username}} + p 'email': {{user.auth.local.email}} small =env.t('emailChange') hr From d560bacd32f0f0cf28d1808e4c8970c4d6889253 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 10:36:14 -0500 Subject: [PATCH 08/50] plain text to translatable strings --- views/options/settings.jade | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index f2c23ac371..40b555a584 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -88,10 +88,13 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-heading span Registration .panel-body - p(ng-if='user.auth.facebook.id') 'registeredWithFb' + p(ng-if='user.auth.facebook.id') + =env.t('registeredWithFb') div(ng-if='user.auth.local.username') - p 'username': {{user.auth.local.username}} - p 'email': {{user.auth.local.email}} + =env.t('username'): {{user.auth.local.username}} + |  + =env.t('email)': {{user.auth.local.email}} + |  small =env.t('emailChange') hr From 684c6a997d482162db0738a01bce99e65b73dec8 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 13:18:00 -0500 Subject: [PATCH 09/50] fixed emailChange message, displays properly on localhost --- views/options/settings.jade | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 40b555a584..bf7d06b1c6 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -91,12 +91,19 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') p(ng-if='user.auth.facebook.id') =env.t('registeredWithFb') div(ng-if='user.auth.local.username') - =env.t('username'): {{user.auth.local.username}} - |  - =env.t('email)': {{user.auth.local.email}} - |  + p + =env.t('username') + |: {{user.auth.local.username}} + p + =env.t('email') + |: {{user.auth.local.email}} + p small - =env.t('emailChange') + =env.t('emailChange1') + |  + a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + |  + =env.t('emailChange3') hr h5=env.t('changeUsername') form(ng-submit='changeUsername(changeUser)', ng-show='user.auth.local') From 393ef6b1f44b4936f219135bf285feae08f680a4 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 20:17:05 -0500 Subject: [PATCH 10/50] added description of display name and link to page to edit login name --- views/options/profile.jade | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/views/options/profile.jade b/views/options/profile.jade index 59c48b0d6e..b348d25ede 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -213,6 +213,15 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') button.btn.btn-default(ng-click='_editing.profile = true', ng-show='!_editing.profile')= env.t('edit') h4=env.t('displayName') span(ng-show='profile.profile.name') {{profile.profile.name}} + p + small + =env.t('displayNameDescription1') + |  + a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') + |  + =env.t('displayNameDescription3') + br + //span(ng-show='profile.profile.name') {{profile.profile.name}} span.muted(ng-hide='profile.profile.name') -  =env.t('none') |  - From 8e3e66dc08d435e0488bf85e8b862ad82ee5b185 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 20:30:25 -0500 Subject: [PATCH 11/50] added description of login name and link to page to edit display name --- views/options/settings.jade | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index bf7d06b1c6..809fbf3372 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -93,7 +93,15 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') div(ng-if='user.auth.local.username') p =env.t('username') - |: {{user.auth.local.username}} + |: {{user.auth.local.username}} + p + small + =env.t('loginNameDescription1') + |  + a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') + |  + =env.t('loginNameDescription3') + br p =env.t('email') |: {{user.auth.local.email}} From ca2d45113c7f0d9ad5b59ac919355bb0d93737ef Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Mon, 12 Jan 2015 11:50:46 -0700 Subject: [PATCH 12/50] fix(billing): script to fix current_period_end, see #4420 --- migrations/current_period_end.js | 5 +++++ 1 file changed, 5 insertions(+) create mode 100644 migrations/current_period_end.js diff --git a/migrations/current_period_end.js b/migrations/current_period_end.js new file mode 100644 index 0000000000..31e33e4a79 --- /dev/null +++ b/migrations/current_period_end.js @@ -0,0 +1,5 @@ +// mongo habitrpg ./node_modules/moment/moment.js ./migrations/current_period_end.js +db.users.update( + {_id:''}, + {$set:{'purchased.plan.dateTerminated':moment().add({days:7}).toDate()}} +) \ No newline at end of file From 0bd22e7c77dad436030428fb867e92e19854175e Mon Sep 17 00:00:00 2001 From: Alice Harris Date: Tue, 13 Jan 2015 06:10:46 +1000 Subject: [PATCH 13/50] disallow fetching of chat messages --- src/controllers/groups.js | 19 +++++++++++++++++-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/src/controllers/groups.js b/src/controllers/groups.js index 3891c70618..9ef34b9c67 100644 --- a/src/controllers/groups.js +++ b/src/controllers/groups.js @@ -211,8 +211,23 @@ api.attachGroup = function(req, res, next) { } api.getChat = function(req, res, next) { - return res.json(res.locals.group.chat); -} + // TODO: This code is duplicated from api.get - pull it out into a function to remove duplication. + var user = res.locals.user; + var gid = req.params.gid; + var q = (gid == 'party') + ? Group.findOne({type: 'party', members: {$in:[user._id]}}) + : Group.findOne({$or:[ + {_id:gid, privacy:'public'}, + {_id:gid, privacy:'private', members: {$in:[user._id]}} + ]}); + populateQuery(gid, q); + q.exec(function(err, group){ + if (err) return next(err); + if (!group && gid!=='party') return res.json(404,{err: "Group not found or you don't have access."}); + res.json(res.locals.group.chat); + gid = null; + }); +}; /** * TODO make this it's own ngResource so we don't have to send down group data with each chat post From ad7c02d50d5a62570b7ecf356c93930a9047fb15 Mon Sep 17 00:00:00 2001 From: Sabe Jones Date: Tue, 13 Jan 2015 11:49:16 -0600 Subject: [PATCH 14/50] feat(world-boss): Healing on rage --- src/models/group.js | 1 + 1 file changed, 1 insertion(+) diff --git a/src/models/group.js b/src/models/group.js index 5b5243b71f..cb936bb1a6 100644 --- a/src/models/group.js +++ b/src/models/group.js @@ -280,6 +280,7 @@ GroupSchema.statics.tavernBoss = function(user,progress) { tavern.quest.extra.worldDmg.recent = scene; tavern.markModified('quest.extra.worldDmg'); tavern.quest.progress.rage = 0; + tavern.quest.progress.hp += (quest.boss.rage.healing * quest.boss.hp); } } tavern.save(cb); From 04ff888819e54a43ae1c19780238df489b0d3e4c Mon Sep 17 00:00:00 2001 From: Sabe Jones Date: Tue, 13 Jan 2015 13:19:04 -0600 Subject: [PATCH 15/50] fix(world-boss): Scale based on current HP --- src/models/group.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/models/group.js b/src/models/group.js index cb936bb1a6..dd0da040e3 100644 --- a/src/models/group.js +++ b/src/models/group.js @@ -280,7 +280,7 @@ GroupSchema.statics.tavernBoss = function(user,progress) { tavern.quest.extra.worldDmg.recent = scene; tavern.markModified('quest.extra.worldDmg'); tavern.quest.progress.rage = 0; - tavern.quest.progress.hp += (quest.boss.rage.healing * quest.boss.hp); + tavern.quest.progress.hp += (quest.boss.rage.healing * tavern.quest.progress.hp); } } tavern.save(cb); From 17ab1d660fe23dce2c70f60109bb03b7c88b1aad Mon Sep 17 00:00:00 2001 From: Sabe Jones Date: Tue, 13 Jan 2015 13:29:10 -0600 Subject: [PATCH 16/50] chore(news): Stress Strike 1 Bailey --- views/shared/new-stuff.jade | 36 ++++++++++++++++++++++-------------- 1 file changed, 22 insertions(+), 14 deletions(-) diff --git a/views/shared/new-stuff.jade b/views/shared/new-stuff.jade index 4ab302be4d..ba223564bb 100644 --- a/views/shared/new-stuff.jade +++ b/views/shared/new-stuff.jade @@ -1,25 +1,33 @@ -h5 WORLD BOSS BATTLE BEGINS! +h5 STRESS HAS STRUCK! tr td - h5 World Boss: The Abominable Stressbeast! - .quest_stressbeast.pull-right - p A new World Boss has appeared in the Tavern! All of the completed Dailies and To-Dos of Habiticans damage the World Boss. Incomplete Dailies fill the Stress Strike Bar. When the Stress Strike bar is full, the World Boss will attack an NPC. - p A World Boss will never damage individual players or accounts in any way. Only active accounts who are not resting in the inn will have their incomplete Dailies tallied. Read on for the details! - tr - td - h5 Winter Plot-Line: The Abominable Stressbeast Attacks! - p The first thing we hear are the footsteps, slower and more thundering than the stampede. One by one, Habiticans look outside their doors, and words fail us. - p We've all seen Stressbeasts before, of course - tiny vicious creatures that attack during difficult times. But this? This towers taller than the buildings, with paws that could crush a dragon with ease. Frost swings from its stinking fur, and as it roars, the icy blast rips the roofs off our houses. A monster of this magnitude has never been mentioned outside of distant legend. - p "Beware, Habiticans!" SabreCat cries. "Barricade yourselves indoors - this is the Abominable Stressbeast itself!" - p "That thing must be made of centuries of stress!" Kiwibot says, locking the Tavern door tightly and shuttering the windows. - p "The Stoïkalm Steppes," Lemoness says, face grim. "All this time, we thought they were placid and untroubled, but they must have been secretly hiding their stress somewhere. Over generations, it grew into this, and now it's broken free and attacked them - and us!" - p There's only one way to drive away a Stressbeast, Abominable or otherwise, and that's to attack it with completed Dailies and To-Dos! Let's all band together and fight off this fearsome foe - but be sure not to slack on your tasks, or our undone Dailies may enrage it so much that it lashes out... + h5 World Boss: First Stress Strike! + p The World Boss in the Tavern has used its first Stress Strike! + p Despite our best efforts, we've let some Dailies get away from us, and their dark-red color has infuriated the Abominable Stressbeast and caused it to regain some of its health! The horrible creature lunges for the Stables, but Matt the Beast Master heroically leaps into the fray to protect the pets and mounts. The Stressbeast has seized Matt in its vicious grip, but at least it's distracted for the moment. + p Complete Dailies and To-Dos to damage the World Boss! Incomplete Dailies fill the Stress Strike Bar. When the Stress Strike bar is full, the World Boss will attack an NPC. A World Boss will never damage individual players or accounts in any way. Only active accounts who are not resting in the inn will have their incomplete Dailies tallied. p.small.muted by Lemoness, Kiwibot, and SabreCat hr a(href='/static/old-news', target='_blank') Read older news mixin oldNews + h5 1/8/2015 + tr + td + h5 World Boss: The Abominable Stressbeast! + .quest_stressbeast.pull-right + p A new World Boss has appeared in the Tavern! All of the completed Dailies and To-Dos of Habiticans damage the World Boss. Incomplete Dailies fill the Stress Strike Bar. When the Stress Strike bar is full, the World Boss will attack an NPC. + p A World Boss will never damage individual players or accounts in any way. Only active accounts who are not resting in the inn will have their incomplete Dailies tallied. Read on for the details! + tr + td + h5 Winter Plot-Line: The Abominable Stressbeast Attacks! + p The first thing we hear are the footsteps, slower and more thundering than the stampede. One by one, Habiticans look outside their doors, and words fail us. + p We've all seen Stressbeasts before, of course - tiny vicious creatures that attack during difficult times. But this? This towers taller than the buildings, with paws that could crush a dragon with ease. Frost swings from its stinking fur, and as it roars, the icy blast rips the roofs off our houses. A monster of this magnitude has never been mentioned outside of distant legend. + p "Beware, Habiticans!" SabreCat cries. "Barricade yourselves indoors - this is the Abominable Stressbeast itself!" + p "That thing must be made of centuries of stress!" Kiwibot says, locking the Tavern door tightly and shuttering the windows. + p "The Stoïkalm Steppes," Lemoness says, face grim. "All this time, we thought they were placid and untroubled, but they must have been secretly hiding their stress somewhere. Over generations, it grew into this, and now it's broken free and attacked them - and us!" + p There's only one way to drive away a Stressbeast, Abominable or otherwise, and that's to attack it with completed Dailies and To-Dos! Let's all band together and fight off this fearsome foe - but be sure not to slack on your tasks, or our undone Dailies may enrage it so much that it lashes out... + p.small.muted by Lemoness, Kiwibot, and SabreCat h5 1/5/2015 tr td From 1c771fa4db1a87d9a5df269f75c3618ccce23823 Mon Sep 17 00:00:00 2001 From: etaiklein Date: Wed, 14 Jan 2015 02:06:41 -0500 Subject: [PATCH 17/50] fixed bug where +habits would behave strangely when targeted by spells --- views/shared/tasks/task.jade | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/views/shared/tasks/task.jade b/views/shared/tasks/task.jade index 20a48ed63c..3362e1249b 100644 --- a/views/shared/tasks/task.jade +++ b/views/shared/tasks/task.jade @@ -63,8 +63,8 @@ li(bindonce='list', id='task-{{::task.id}}', ng-repeat='task in obj[list.type+"s // Habits span(ng-if='::task.type=="habit"') // score() is overridden in challengesCtrl to do nothing - a.task-action-btn(ng-if='task.up', ng-click='score(task,"up")') + - a.task-action-btn(ng-if='task.down', ng-click='score(task,"down")') - + a.task-action-btn(ng-if='task.up', ng-click='$root.applyingAction || score(task,"up")') + + a.task-action-btn(ng-if='task.down', ng-click='$root.applyingAction || score(task,"down")') - // Rewards span(ng-show='task.type=="reward"') From 9fe93351ae1bc7d4b32c036935d1412e8ca1b53b Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Wed, 14 Jan 2015 22:38:02 -0500 Subject: [PATCH 18/50] applied .muted to text so less cluttered looking --- views/options/profile.jade | 14 ++++++-------- views/options/settings.jade | 25 ++++++++++++------------- 2 files changed, 18 insertions(+), 21 deletions(-) diff --git a/views/options/profile.jade b/views/options/profile.jade index b348d25ede..8f91880cbe 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -214,14 +214,12 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') h4=env.t('displayName') span(ng-show='profile.profile.name') {{profile.profile.name}} p - small - =env.t('displayNameDescription1') - |  - a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') - |  - =env.t('displayNameDescription3') - br - //span(ng-show='profile.profile.name') {{profile.profile.name}} + small.muted + =env.t('displayNameDescription1') + |  + a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') + |  + =env.t('displayNameDescription3') span.muted(ng-hide='profile.profile.name') -  =env.t('none') |  - diff --git a/views/options/settings.jade b/views/options/settings.jade index 809fbf3372..4457c68ab7 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -95,23 +95,22 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') =env.t('username') |: {{user.auth.local.username}} p - small - =env.t('loginNameDescription1') - |  - a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') - |  - =env.t('loginNameDescription3') - br + small.muted + =env.t('loginNameDescription1') + |  + a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') + |  + =env.t('loginNameDescription3') p =env.t('email') |: {{user.auth.local.email}} p - small - =env.t('emailChange1') - |  - a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') - |  - =env.t('emailChange3') + small.muted + =env.t('emailChange1') + |  + a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + |  + =env.t('emailChange3') hr h5=env.t('changeUsername') form(ng-submit='changeUsername(changeUser)', ng-show='user.auth.local') From f9b507e782c4f9867fb91785496b82fd13663d81 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Thu, 15 Jan 2015 08:23:23 -0500 Subject: [PATCH 19/50] changed links to relative, corrected mailto link --- views/options/profile.jade | 2 +- views/options/settings.jade | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/views/options/profile.jade b/views/options/profile.jade index 8f91880cbe..5f6ed31b04 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -217,7 +217,7 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') small.muted =env.t('displayNameDescription1') |  - a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') + a(href='/#/options/settings/settings')=env.t('displayNameDescription2') |  =env.t('displayNameDescription3') span.muted(ng-hide='profile.profile.name') -  diff --git a/views/options/settings.jade b/views/options/settings.jade index 4457c68ab7..2a5fee087c 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -98,7 +98,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') small.muted =env.t('loginNameDescription1') |  - a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') + a(href='/#/options/profile/profile')=env.t('loginNameDescription2') |  =env.t('loginNameDescription3') p @@ -108,7 +108,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') small.muted =env.t('emailChange1') |  - a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + a(href='mailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') |  =env.t('emailChange3') hr From 0b63e48c6e10478e1717f0ce132ceb0ade4ccb4d Mon Sep 17 00:00:00 2001 From: Sabe Jones Date: Thu, 15 Jan 2015 15:38:51 -0600 Subject: [PATCH 20/50] feat(quest): T-Rex pet Includes news announcements for 1/15/2014, Market implementation for T-Rex dual quest, and the ability for "healing" type Rage meters to work in party-scale boss fights. --- src/controllers/groups.js | 1 + src/models/group.js | 11 ++++++++++- views/options/inventory/inventory.jade | 5 +++++ views/shared/new-stuff.jade | 12 +++++++++++- 4 files changed, 27 insertions(+), 2 deletions(-) diff --git a/src/controllers/groups.js b/src/controllers/groups.js index 9ef34b9c67..a3076ed09c 100644 --- a/src/controllers/groups.js +++ b/src/controllers/groups.js @@ -662,6 +662,7 @@ questStart = function(req, res, next) { group.quest.active = true; if (quest.boss) group.quest.progress.hp = quest.boss.hp; + if (quest.boss.rage) group.quest.progress.rage = 0; else group.quest.progress.collect = collected; group.quest.members = questMembers; diff --git a/src/models/group.js b/src/models/group.js index dd0da040e3..516267e469 100644 --- a/src/models/group.js +++ b/src/models/group.js @@ -301,8 +301,17 @@ GroupSchema.statics.bossQuest = function(user, progress, cb) { var down = progress.down * quest.boss.str; // multiply by boss strength group.quest.progress.hp -= progress.up; - group.sendChat("`" + user.profile.name + " attacks " + quest.boss.name('en') + " for " + (progress.up.toFixed(1)) + " damage, " + quest.boss.name('en') + " attacks party for " + Math.abs(down).toFixed(1) + " damage.`"); + group.sendChat("`" + user.profile.name + " attacks " + quest.boss.name('en') + " for " + (progress.up.toFixed(1)) + " damage, " + quest.boss.name('en') + " attacks party for " + Math.abs(down).toFixed(1) + " damage.`"); //TODO Create a party preferred language option so emits like this can be localized + // If boss has Rage, increment Rage as well + if (quest.boss.rage) { + group.quest.progress.rage += Math.abs(down); + if (group.quest.progress.rage >= quest.boss.rage.value) { + group.sendChat(quest.boss.rage.effect('en')); + group.quest.progress.rage = 0; + if (quest.boss.rage.healing) group.quest.progress.hp += (group.quest.progress.hp * quest.boss.rage.healing); //TODO To make Rage effects more expandable, let's turn these into functions in quest.boss.rage + } + } // Everyone takes damage var series = [ function(cb2){ diff --git a/views/options/inventory/inventory.jade b/views/options/inventory/inventory.jade index cfdebc7513..f5e760f30e 100644 --- a/views/options/inventory/inventory.jade +++ b/views/options/inventory/inventory.jade @@ -183,6 +183,11 @@ script(type='text/ng-template', id='partials/options.inventory.drops.html') p | {{::Content.eggs.#{egg}.value}}  span.Pet_Currency_Gem1x.inline-gems + div(ng-show='(user.achievements.quests.trex + user.achievements.quests.trex_undead) > 1') + button.customize-option(popover='{{::Content.eggs.TRex.notes()}}', popover-title!=env.t("egg", {eggType: "{{Content.eggs.TRex.text()}}"}), popover-trigger='mouseenter', popover-placement='top', ng-click='purchase("eggs", Content.eggs.TRex)', class='Pet_Egg_TRex') + p + | {{::Content.eggs.TRex.value}}  + span.Pet_Currency_Gem1x.inline-gems li.customize-menu menu.pets-menu(label=env.t('hatchingPotions')) diff --git a/views/shared/new-stuff.jade b/views/shared/new-stuff.jade index ba223564bb..81c719a57b 100644 --- a/views/shared/new-stuff.jade +++ b/views/shared/new-stuff.jade @@ -1,4 +1,14 @@ -h5 STRESS HAS STRUCK! +h5 TYRANNOSAUR PET QUEST, SPREAD THE WORD CHALLENGE, AND FIRST STRESS STRIKE! + tr + td + h5 Tyrannosaur Pet Quest + p In the Market there are now two new pet quests: King of the Dinosaurs and The Dinosaur Unearthed! They both give out the same rewards, including pet Tyrannosaur eggs. The difference is that "King of the Dinosaurs" is a normal pet quest, like all the others, whereas "The Dinosaur Unearthed" has less HP - but also a Rage bar (a la World Bosses) that allows it to heal if you skip too many of your Dailies. Both bosses still attack your party based on how many Dailies are incomplete. Users will be able to buy Tyrannosaur eggs after defeating either boss twice or both bosses once. + p Have fun! + p.small.muted by Baconsaur, Urse, Lemoness, and SabreCat + tr + td + h5 Spread the Word Challenge Reminder + p In case you missed it, we're running our second Spread the Word Challenge! The rules are simple: make a post some time between December 31st 2014 and January 31st 2015 on some form of blog or social media that tells people about HabitRPG. The top post will be awarded 100 GEMS, and the next nineteen top posts will be awarded 80 GEMS each. Learn more and join in here! tr td h5 World Boss: First Stress Strike! From 35e8097b0c93ae1c89647c421362d18c9ab8d46d Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Thu, 15 Jan 2015 15:24:02 -0700 Subject: [PATCH 21/50] refactor(challenges): wrap group.challenges as $resources, so they're convenience functions $join/$leave/etc can be called directly --- public/js/services/groupServices.js | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/public/js/services/groupServices.js b/public/js/services/groupServices.js index e07f21b15c..8707424a08 100644 --- a/public/js/services/groupServices.js +++ b/public/js/services/groupServices.js @@ -4,13 +4,26 @@ * Services that persists and retrieves user from localStorage. */ -angular.module('groupServices', ['ngResource']). - factory('Groups', ['ApiUrlService', '$resource', '$q', '$http', 'User', - function(ApiUrlService, $resource, $q, $http, User) { +angular.module('groupServices', ['ngResource', 'challengeServices']). + factory('Groups', ['ApiUrlService', '$resource', '$q', '$http', 'User', 'Challenges', + function(ApiUrlService, $resource, $q, $http, User, Challenges) { + var Group = $resource(ApiUrlService.get() + '/api/v2/groups/:gid', {gid:'@_id', messageId: '@_messageId'}, { - //query: {method: "GET", isArray:false}, + get: { + method: "GET", + isArray:false, + // Wrap challenges as ngResource so they have functions like $leave or $join + transformResponse: function(data, headers) { + data = angular.fromJson(data); + _.each(data && data.challenges, function(c) { + angular.extend(c, Challenges.Challenge.prototype); + }); + return data; + } + }, + postChat: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/chat'}, deleteChatMessage: {method: "DELETE", url: ApiUrlService.get() + '/api/v2/groups/:gid/chat/:messageId'}, flagChatMessage: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/chat/:messageId/flag'}, From 290e602f213357b60cc1217eb3534307bf3ef99c Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Fri, 16 Jan 2015 10:22:47 -0600 Subject: [PATCH 22/50] Added brackets to quest.boss check. Closes #4526 --- src/controllers/groups.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/controllers/groups.js b/src/controllers/groups.js index a3076ed09c..4c7b37c919 100644 --- a/src/controllers/groups.js +++ b/src/controllers/groups.js @@ -660,11 +660,12 @@ questStart = function(req, res, next) { }) group.quest.active = true; - if (quest.boss) + if (quest.boss) { group.quest.progress.hp = quest.boss.hp; if (quest.boss.rage) group.quest.progress.rage = 0; - else + } else { group.quest.progress.collect = collected; + } group.quest.members = questMembers; group.markModified('quest'); // members & progress.collect are both Mixed types parallel.push(function(cb2){group.save(cb2)}); From 9a087d2d9079618477c5e0cb2d2995f17c94cc6c Mon Sep 17 00:00:00 2001 From: Matteo Pagliazzi Date: Fri, 16 Jan 2015 17:39:47 +0100 Subject: [PATCH 23/50] update angular and do not send api key to loggly --- bower.json | 6 +++--- src/middleware.js | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/bower.json b/bower.json index b469574b1f..3f4eee4a3f 100644 --- a/bower.json +++ b/bower.json @@ -16,10 +16,10 @@ "dependencies": { "jquery": "~2.1.0", "jquery.cookie": "~1.4.0", - "angular": "1.3.3", + "angular": "1.3.9", "angular-ui": "~0.4.0", - "angular-sanitize": "1.3.3", - "angular-resource": "1.3.3", + "angular-sanitize": "1.3.9", + "angular-resource": "1.3.9", "angular-ui-utils": "~0.1.0", "angular-ui-select2": "git://github.com/angular-ui/ui-select2.git", "angular-bootstrap": "~0.12.0", diff --git a/src/middleware.js b/src/middleware.js index 56e72bda7e..189275443b 100644 --- a/src/middleware.js +++ b/src/middleware.js @@ -79,7 +79,7 @@ module.exports.errorHandler = function(err, req, res, next) { error: "Uncaught error", stack: (err.stack || err.message || err), body: req.body, headers: req.header, - auth: (req.headers['x-api-user'] + ' | ' + req.headers['x-api-key']), + auth: req.headers['x-api-user'], originalUrl: req.originalUrl }); var message = err.message ? err.message : err; From 90c7142da5c5feb6b6dacfbb005c51493c787d10 Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Fri, 16 Jan 2015 16:48:53 -0700 Subject: [PATCH 24/50] refactor(config): move httpInterceptor from website/app.js to habitrpg-shared/config.js. --- public/js/app.js | 52 ++------------------------------------------ public/manifest.json | 1 + 2 files changed, 3 insertions(+), 50 deletions(-) diff --git a/public/js/app.js b/public/js/app.js index d97c3d3980..f8410f1fd1 100644 --- a/public/js/app.js +++ b/public/js/app.js @@ -14,8 +14,8 @@ window.habitrpg = angular.module('habitrpg', .constant("MOBILE_APP", false) //.constant("STORAGE_GROUPS_ID", "") // if we decide to take groups offline - .config(['$stateProvider', '$urlRouterProvider', '$httpProvider', '$provide', 'STORAGE_SETTINGS_ID', - function($stateProvider, $urlRouterProvider, $httpProvider, $provide, STORAGE_SETTINGS_ID) { + .config(['$stateProvider', '$urlRouterProvider', '$httpProvider', 'STORAGE_SETTINGS_ID', + function($stateProvider, $urlRouterProvider, $httpProvider, STORAGE_SETTINGS_ID) { $urlRouterProvider // Setup default selected tabs @@ -222,52 +222,4 @@ window.habitrpg = angular.module('habitrpg', $httpProvider.defaults.headers.common['x-api-user'] = settings.auth.apiId; $httpProvider.defaults.headers.common['x-api-key'] = settings.auth.apiToken; } - - // Handle errors - $provide.factory('myHttpInterceptor', ['$rootScope','$q',function($rootScope,$q) { - return { - response: function(response) { - return response; - }, - responseError: function(response) { - // Offline - if (response.status == 0 || - // don't know why we're getting 404 here, should be 0 - (response.status == 404 && _.isEmpty(response.data))) { - $rootScope.$broadcast('responseText', window.env.t('serverUnreach')); - - // Needs refresh - } else if (response.needRefresh) { - $rootScope.$broadcast('responseError', "The site has been updated and the page needs to refresh. The last action has not been recorded, please refresh and try again."); - - } else if (response.data.code && response.data.code === 'ACCOUNT_SUSPENDED') { - confirm(response.data.err); - localStorage.clear(); - window.location.href = '/logout'; - - // 400 range? - } else if (response.status < 500) { - $rootScope.$broadcast('responseText', response.data.err || response.data); - // Need to reject the prompse so the error is handled correctly - if (response.status === 401) { - return $q.reject(response); - } - - // Error - } else { - var error = 'Please reload, ' + - '"'+window.env.t('error')+' '+(response.data.err || response.data || 'something went wrong')+'" ' + - window.env.t('seeConsole'); - $rootScope.$broadcast('responseError', error); - console.error(response); - } - - return response; - // this completely halts the chain, meaning we can't queue offline actions - //if (canRecover(response)) return responseOrNewPromise - //return $q.reject(response); - } - }; - }]); - $httpProvider.interceptors.push('myHttpInterceptor'); }]) diff --git a/public/manifest.json b/public/manifest.json index dd4e9abe2a..03df4c4535 100644 --- a/public/manifest.json +++ b/public/manifest.json @@ -37,6 +37,7 @@ "js/env.js", "js/app.js", + "bower_components/habitrpg-shared/script/config.js", "js/services/sharedServices.js", "js/services/authServices.js", "js/services/notificationServices.js", From 6a5210d3ad728e5d06d8d61cdce372c15f2889ff Mon Sep 17 00:00:00 2001 From: Alice Harris Date: Sat, 17 Jan 2015 16:45:55 +1000 Subject: [PATCH 25/50] fix(plans): when gifting subscription, set plan.dateUpdated so it can roll gemCapExtra next month. Fixes #4480. Removes datedUpdated from previous version of fix. --- migrations/20150107_plan_dateUpdated_null.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/migrations/20150107_plan_dateUpdated_null.js b/migrations/20150107_plan_dateUpdated_null.js index ae231a0414..6e03b41a55 100644 --- a/migrations/20150107_plan_dateUpdated_null.js +++ b/migrations/20150107_plan_dateUpdated_null.js @@ -1,5 +1,8 @@ db.users.update( {'purchased.plan.customerId':{$ne:null}, 'purchased.plan.dateUpdated':null}, - {$set: {'purchased.plan.datedUpdated': new Date('12/01/2014')}}, + { + $set: {'purchased.plan.dateUpdated': new Date('12/01/2014')}, + $unset: {'purchased.plan.datedUpdated':''} + }, {multi:true} -); \ No newline at end of file +); From 2306b7c57289d2db9e053b528ccf3c5a473ee24e Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Sat, 17 Jan 2015 18:43:37 -0700 Subject: [PATCH 26/50] refactor(services): move to a single "habitrpg" module for services / directives / filters --- public/js/app.js | 4 +- public/js/controllers/authCtrl.js | 12 +- public/js/controllers/footerCtrl.js | 194 +++++----- public/js/controllers/groupsCtrl.js | 4 +- public/js/controllers/hallCtrl.js | 12 +- public/js/controllers/rootCtrl.js | 6 +- public/js/controllers/settingsCtrl.js | 16 +- public/js/controllers/tasksCtrl.js | 6 +- public/js/services/authServices.js | 158 ++++---- public/js/services/challengeServices.js | 37 +- public/js/services/groupServices.js | 134 ++++--- public/js/services/guideServices.js | 402 ++++++++++----------- public/js/services/memberServices.js | 143 ++++---- public/js/services/notificationServices.js | 155 ++++---- public/js/services/paymentServices.js | 2 +- public/js/services/sharedServices.js | 16 +- public/js/static.js | 2 +- views/static/layout.jade | 2 +- views/static/new-stuff.jade | 2 +- 19 files changed, 646 insertions(+), 661 deletions(-) diff --git a/public/js/app.js b/public/js/app.js index f8410f1fd1..a5bcb6c66a 100644 --- a/public/js/app.js +++ b/public/js/app.js @@ -1,9 +1,7 @@ "use strict"; window.habitrpg = angular.module('habitrpg', - ['ngResource', 'ngSanitize', 'userServices', 'groupServices', 'memberServices', 'challengeServices', - 'authServices', 'notificationServices', 'guideServices', 'authCtrl', 'paymentServices', - 'ui.bootstrap', 'ui.keypress', 'ui.router', 'chieffancypants.loadingBar', 'At', 'infinite-scroll', 'ui.select2', 'angular.filter']) + ['authCtrl', 'ui.bootstrap', 'ui.keypress', 'ui.router', 'chieffancypants.loadingBar', 'At', 'infinite-scroll', 'ui.select2', 'angular.filter', 'ngResource']) // @see https://github.com/angular-ui/ui-router/issues/110 and https://github.com/HabitRPG/habitrpg/issues/1705 // temporary hack until they have a better solution diff --git a/public/js/controllers/authCtrl.js b/public/js/controllers/authCtrl.js index bacc41c28f..321058ae6f 100644 --- a/public/js/controllers/authCtrl.js +++ b/public/js/controllers/authCtrl.js @@ -5,8 +5,8 @@ */ angular.module('authCtrl', []) - .controller("AuthCtrl", ['$scope', '$rootScope', 'User', '$http', '$location', '$window','ApiUrlService', '$modal', - function($scope, $rootScope, User, $http, $location, $window, ApiUrlService, $modal) { + .controller("AuthCtrl", ['$scope', '$rootScope', 'User', '$http', '$location', '$window','ApiUrl', '$modal', + function($scope, $rootScope, User, $http, $location, $window, ApiUrl, $modal) { $scope.logout = function() { localStorage.clear(); @@ -36,7 +36,7 @@ angular.module('authCtrl', []) if ($scope.registrationForm.$invalid) { return; } - var url = ApiUrlService.get() + "/api/v2/register"; + var url = ApiUrl.get() + "/api/v2/register"; if($rootScope.selectedLanguage) url = url + '?lang=' + $rootScope.selectedLanguage.code; $http.post(url, $scope.registerVals).success(function(data, status, headers, config) { runAuth(data.id, data.apiToken); @@ -48,7 +48,7 @@ angular.module('authCtrl', []) username: $scope.loginUsername || $('#login-tab input[name="username"]').val(), password: $scope.loginPassword || $('#login-tab input[name="password"]').val() }; - $http.post(ApiUrlService.get() + "/api/v2/user/auth/local", data) + $http.post(ApiUrl.get() + "/api/v2/user/auth/local", data) .success(function(data, status, headers, config) { runAuth(data.id, data.token); }).error(errorAlert); @@ -70,7 +70,7 @@ angular.module('authCtrl', []) if(email == null || email.length == 0) { alert(window.env.t('invalidEmail')); } else { - $http.post(ApiUrlService.get() + '/api/v2/user/reset-password', {email:email}) + $http.post(ApiUrl.get() + '/api/v2/user/reset-password', {email:email}) .success(function(){ alert(window.env.t('newPassSent')); }) @@ -120,7 +120,7 @@ angular.module('authCtrl', []) $scope.socialLogin = function(network){ hello(network).login({scope:'email'}).then(function(auth){ - $http.post(ApiUrlService.get() + "/api/v2/user/auth/social", auth) + $http.post(ApiUrl.get() + "/api/v2/user/auth/social", auth) .success(function(data, status, headers, config) { runAuth(data.id, data.token); }).error(errorAlert); diff --git a/public/js/controllers/footerCtrl.js b/public/js/controllers/footerCtrl.js index ccb4084860..e704785f86 100644 --- a/public/js/controllers/footerCtrl.js +++ b/public/js/controllers/footerCtrl.js @@ -1,108 +1,108 @@ "use strict"; -(typeof habitrpg !== 'undefined' ? habitrpg : habitrpgStatic) - .controller("FooterCtrl", ['$scope', '$rootScope', 'User', '$http', 'Notification', 'ApiUrlService', - function($scope, $rootScope, User, $http, Notification, ApiUrlService) { +angular.module('habitrpg').controller("FooterCtrl", +['$scope', '$rootScope', 'User', '$http', 'Notification', 'ApiUrl', +function($scope, $rootScope, User, $http, Notification, ApiUrl) { - if(typeof habitrpg === "undefined"){ - $scope.languages = env.avalaibleLanguages; - $scope.selectedLanguage = _.find(env.avalaibleLanguages, {code: env.language.code}); + if(typeof habitrpg === "undefined"){ + $scope.languages = env.avalaibleLanguages; + $scope.selectedLanguage = _.find(env.avalaibleLanguages, {code: env.language.code}); - $rootScope.selectedLanguage = $scope.selectedLanguage; - - $scope.changeLang = function(){ - window.location = '?lang='+$scope.selectedLanguage.code; - } + $rootScope.selectedLanguage = $scope.selectedLanguage; + + $scope.changeLang = function(){ + window.location = '?lang='+$scope.selectedLanguage.code; + } + } + + /** + External Scripts + JS files not needed right away (google charts) or entirely optional (analytics) + Each file gets loaded async via $.getScript, so it doesn't bog page-load + */ + $scope.deferredScripts = function(){ + + // Stripe + $.getScript('//checkout.stripe.com/v2/checkout.js'); + + // Google Analytics, only in production + if (window.env.NODE_ENV === 'production') { + (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){ + (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o), + m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m) + })(window,document,'script','//www.google-analytics.com/analytics.js','ga'); + ga('create', window.env.GA_ID, {userId:User.user._id}); + ga('require', 'displayfeatures'); + ga('send', 'pageview'); } - /** - External Scripts - JS files not needed right away (google charts) or entirely optional (analytics) - Each file gets loaded async via $.getScript, so it doesn't bog page-load - */ - $scope.deferredScripts = function(){ + // Scripts only for desktop + if (!window.env.IS_MOBILE) { + // Add This + //$.getScript("//s7.addthis.com/js/300/addthis_widget.js#pubid=ra-5016f6cc44ad68a4"); //FIXME why isn't this working when here? instead it's now in + var addthisServices = 'facebook,twitter,googleplus,tumblr,'+window.env.BASE_URL.replace('https://','').replace('http://',''); + window.addthis_config = { + ui_click: true, + services_custom:{ + name: "Download", + url: window.env.BASE_URL+"/export/avatar-"+User.user._id+".png", + icon: window.env.BASE_URL+"/favicon.ico" + }, + services_expanded:addthisServices, + services_compact:addthisServices + }; - // Stripe - $.getScript('//checkout.stripe.com/v2/checkout.js'); - - // Google Analytics, only in production - if (window.env.NODE_ENV === 'production') { - (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){ - (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o), - m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m) - })(window,document,'script','//www.google-analytics.com/analytics.js','ga'); - ga('create', window.env.GA_ID, {userId:User.user._id}); - ga('require', 'displayfeatures'); - ga('send', 'pageview'); - } - - // Scripts only for desktop - if (!window.env.IS_MOBILE) { - // Add This - //$.getScript("//s7.addthis.com/js/300/addthis_widget.js#pubid=ra-5016f6cc44ad68a4"); //FIXME why isn't this working when here? instead it's now in - var addthisServices = 'facebook,twitter,googleplus,tumblr,'+window.env.BASE_URL.replace('https://','').replace('http://',''); - window.addthis_config = { - ui_click: true, - services_custom:{ - name: "Download", - url: window.env.BASE_URL+"/export/avatar-"+User.user._id+".png", - icon: window.env.BASE_URL+"/favicon.ico" - }, - services_expanded:addthisServices, - services_compact:addthisServices - }; - - // Google Charts - $.getScript("//www.google.com/jsapi", function() { - google.load("visualization", "1", { - packages: ["corechart"], - callback: function() {} - }); + // Google Charts + $.getScript("//www.google.com/jsapi", function() { + google.load("visualization", "1", { + packages: ["corechart"], + callback: function() {} }); - } + }); } + } - /** - * Debug functions. Note that the server route for gems is only available if process.env.DEBUG=true - */ - if (_.contains(['development','test'],window.env.NODE_ENV)) { - $scope.setHealthLow = function(){ - User.set({ - 'stats.hp': 1 - }); - } - $scope.addMissedDay = function(){ - if (!confirm("Are you sure you want to reset the day?")) return; - var dayBefore = moment(User.user.lastCron).subtract(1, 'days').toDate(); - User.set({'lastCron': dayBefore}); - Notification.text('-1 day, remember to refresh'); - } - $scope.addTenGems = function(){ - $http.post(ApiUrlService.get() + '/api/v2/user/addTenGems').success(function(){ - User.log({}); - }) - } - $scope.addGold = function(){ - User.set({ - 'stats.gp': User.user.stats.gp + 500, - }); - } - $scope.addLevelsAndGold = function(){ - User.set({ - 'stats.exp': User.user.stats.exp + 10000, - 'stats.gp': User.user.stats.gp + 10000, - 'stats.mp': User.user.stats.mp + 10000 - }); - } - $scope.addOneLevel = function(){ - User.set({ - 'stats.exp': User.user.stats.exp + (Math.round(((Math.pow(User.user.stats.lvl, 2) * 0.25) + (10 * User.user.stats.lvl) + 139.75) / 10) * 10) - }); - } - $scope.addBossQuestProgressUp = function(){ - User.set({ - 'party.quest.progress.up': User.user.party.quest.progress.up + 1000 - }); - } + /** + * Debug functions. Note that the server route for gems is only available if process.env.DEBUG=true + */ + if (_.contains(['development','test'],window.env.NODE_ENV)) { + $scope.setHealthLow = function(){ + User.set({ + 'stats.hp': 1 + }); } - }]) + $scope.addMissedDay = function(){ + if (!confirm("Are you sure you want to reset the day?")) return; + var dayBefore = moment(User.user.lastCron).subtract(1, 'days').toDate(); + User.set({'lastCron': dayBefore}); + Notification.text('-1 day, remember to refresh'); + } + $scope.addTenGems = function(){ + $http.post(ApiUrl.get() + '/api/v2/user/addTenGems').success(function(){ + User.log({}); + }) + } + $scope.addGold = function(){ + User.set({ + 'stats.gp': User.user.stats.gp + 500, + }); + } + $scope.addLevelsAndGold = function(){ + User.set({ + 'stats.exp': User.user.stats.exp + 10000, + 'stats.gp': User.user.stats.gp + 10000, + 'stats.mp': User.user.stats.mp + 10000 + }); + } + $scope.addOneLevel = function(){ + User.set({ + 'stats.exp': User.user.stats.exp + (Math.round(((Math.pow(User.user.stats.lvl, 2) * 0.25) + (10 * User.user.stats.lvl) + 139.75) / 10) * 10) + }); + } + $scope.addBossQuestProgressUp = function(){ + User.set({ + 'party.quest.progress.up': User.user.party.quest.progress.up + 1000 + }); + } + } +}]) diff --git a/public/js/controllers/groupsCtrl.js b/public/js/controllers/groupsCtrl.js index fe30af9239..700e58bf3f 100644 --- a/public/js/controllers/groupsCtrl.js +++ b/public/js/controllers/groupsCtrl.js @@ -211,7 +211,7 @@ habitrpg.controller("GroupsCtrl", ['$scope', '$rootScope', 'Shared', 'Groups', ' }); }]) - .controller('ChatCtrl', ['$scope', 'Groups', 'User', '$http', 'ApiUrlService', 'Notification', 'Members', '$rootScope', function($scope, Groups, User, $http, ApiUrlService, Notification, Members, $rootScope){ + .controller('ChatCtrl', ['$scope', 'Groups', 'User', '$http', 'ApiUrl', 'Notification', 'Members', '$rootScope', function($scope, Groups, User, $http, ApiUrl, Notification, Members, $rootScope){ $scope.message = {content:''}; $scope._sending = false; @@ -275,7 +275,7 @@ habitrpg.controller("GroupsCtrl", ['$scope', '$rootScope', 'Shared', 'Groups', ' } //Chat.Chat.like({gid:group._id,mid:message.id}); - $http.post(ApiUrlService.get() + '/api/v2/groups/' + group._id + '/chat/' + message.id + '/like'); + $http.post(ApiUrl.get() + '/api/v2/groups/' + group._id + '/chat/' + message.id + '/like'); } $scope.flagChatMessage = function(groupId,message) { diff --git a/public/js/controllers/hallCtrl.js b/public/js/controllers/hallCtrl.js index c135f5e8df..6956bfedd5 100644 --- a/public/js/controllers/hallCtrl.js +++ b/public/js/controllers/hallCtrl.js @@ -1,8 +1,8 @@ "use strict"; -habitrpg.controller("HallHeroesCtrl", ['$scope', '$rootScope', 'User', 'Notification', 'ApiUrlService', '$resource', - function($scope, $rootScope, User, Notification, ApiUrlService, $resource) { - var Hero = $resource(ApiUrlService.get() + '/api/v2/hall/heroes/:uid', {uid:'@_id'}); +habitrpg.controller("HallHeroesCtrl", ['$scope', '$rootScope', 'User', 'Notification', 'ApiUrl', '$resource', + function($scope, $rootScope, User, Notification, ApiUrl, $resource) { + var Hero = $resource(ApiUrl.get() + '/api/v2/hall/heroes/:uid', {uid:'@_id'}); $scope.hero = undefined; $scope.loadHero = function(uuid){ $scope.hero = Hero.get({uid:uuid}); @@ -19,9 +19,9 @@ habitrpg.controller("HallHeroesCtrl", ['$scope', '$rootScope', 'User', 'Notifica $scope.heroes = Hero.query(); }]); -habitrpg.controller("HallPatronsCtrl", ['$scope', '$rootScope', 'User', 'Notification', 'ApiUrlService', '$resource', - function($scope, $rootScope, User, Notification, ApiUrlService, $resource) { - var Patron = $resource(ApiUrlService.get() + '/api/v2/hall/patrons/:uid', {uid:'@_id'}); +habitrpg.controller("HallPatronsCtrl", ['$scope', '$rootScope', 'User', 'Notification', 'ApiUrl', '$resource', + function($scope, $rootScope, User, Notification, ApiUrl, $resource) { + var Patron = $resource(ApiUrl.get() + '/api/v2/hall/patrons/:uid', {uid:'@_id'}); var page = 0; $scope.patrons = []; diff --git a/public/js/controllers/rootCtrl.js b/public/js/controllers/rootCtrl.js index 24c7fbb290..3533d28a6b 100644 --- a/public/js/controllers/rootCtrl.js +++ b/public/js/controllers/rootCtrl.js @@ -3,8 +3,8 @@ /* Make user and settings available for everyone through root scope. */ -habitrpg.controller("RootCtrl", ['$scope', '$rootScope', '$location', 'User', '$http', '$state', '$stateParams', 'Notification', 'Groups', 'Shared', 'Content', '$modal', '$timeout', 'ApiUrlService', 'Payments', - function($scope, $rootScope, $location, User, $http, $state, $stateParams, Notification, Groups, Shared, Content, $modal, $timeout, ApiUrlService, Payments) { +habitrpg.controller("RootCtrl", ['$scope', '$rootScope', '$location', 'User', '$http', '$state', '$stateParams', 'Notification', 'Groups', 'Shared', 'Content', '$modal', '$timeout', 'ApiUrl', 'Payments', + function($scope, $rootScope, $location, User, $http, $state, $stateParams, Notification, Groups, Shared, Content, $modal, $timeout, ApiUrl, Payments) { var user = User.user; var initSticky = _.once(function(){ @@ -221,7 +221,7 @@ habitrpg.controller("RootCtrl", ['$scope', '$rootScope', '$location', 'User', '$ $scope.spell = null; $rootScope.applyingAction = false; - $http.post(ApiUrlService.get() + '/api/v2/user/class/cast/'+spell.key+'?targetType='+type+'&targetId='+targetId) + $http.post(ApiUrl.get() + '/api/v2/user/class/cast/'+spell.key+'?targetType='+type+'&targetId='+targetId) .success(function(){ var msg = window.env.t('youCast', {spell: spell.text()}); switch (type) { diff --git a/public/js/controllers/settingsCtrl.js b/public/js/controllers/settingsCtrl.js index a7927d5d0c..7220791520 100644 --- a/public/js/controllers/settingsCtrl.js +++ b/public/js/controllers/settingsCtrl.js @@ -2,8 +2,8 @@ // Make user and settings available for everyone through root scope. habitrpg.controller('SettingsCtrl', - ['$scope', 'User', '$rootScope', '$http', 'ApiUrlService', 'Guide', '$location', '$timeout', 'Notification', 'Shared', - function($scope, User, $rootScope, $http, ApiUrlService, Guide, $location, $timeout, Notification, Shared) { + ['$scope', 'User', '$rootScope', '$http', 'ApiUrl', 'Guide', '$location', '$timeout', 'Notification', 'Shared', + function($scope, User, $rootScope, $http, ApiUrl, Guide, $location, $timeout, Notification, Shared) { // FIXME we have this re-declared everywhere, figure which is the canonical version and delete the rest // $scope.auth = function (id, token) { @@ -81,7 +81,7 @@ habitrpg.controller('SettingsCtrl', if (!changeUser.newUsername || !changeUser.password) { return alert(window.env.t('fillAll')); } - $http.post(ApiUrlService.get() + '/api/v2/user/change-username', changeUser) + $http.post(ApiUrl.get() + '/api/v2/user/change-username', changeUser) .success(function(){ alert(window.env.t('usernameSuccess')); $scope.changeUser = {}; @@ -95,7 +95,7 @@ habitrpg.controller('SettingsCtrl', if (!changePass.oldPassword || !changePass.newPassword || !changePass.confirmNewPassword) { return alert(window.env.t('fillAll')); } - $http.post(ApiUrlService.get() + '/api/v2/user/change-password', changePass) + $http.post(ApiUrl.get() + '/api/v2/user/change-password', changePass) .success(function(data, status, headers, config){ if (data.err) return alert(data.err); alert(window.env.t('passSuccess')); @@ -132,7 +132,7 @@ habitrpg.controller('SettingsCtrl', } $scope['delete'] = function(){ - $http['delete'](ApiUrlService.get() + '/api/v2/user') + $http['delete'](ApiUrl.get() + '/api/v2/user') .success(function(res, code){ if (res.err) return alert(res.err); localStorage.clear(); @@ -141,14 +141,14 @@ habitrpg.controller('SettingsCtrl', } $scope.enterCoupon = function(code) { - $http.post(ApiUrlService.get() + '/api/v2/user/coupon/' + code).success(function(res,code){ + $http.post(ApiUrl.get() + '/api/v2/user/coupon/' + code).success(function(res,code){ if (code!==200) return; User.sync(); Notification.text('Coupon applied! Check your inventory'); }); } $scope.generateCodes = function(codes){ - $http.post(ApiUrlService.get() + '/api/v2/coupons/generate/'+codes.event+'?count='+(codes.count || 1)) + $http.post(ApiUrl.get() + '/api/v2/coupons/generate/'+codes.event+'?count='+(codes.count || 1)) .success(function(res,code){ $scope._codes = {}; if (code!==200) return; @@ -183,7 +183,7 @@ habitrpg.controller('SettingsCtrl', } $scope.applyCoupon = function(coupon){ - $http.get(ApiUrlService.get() + '/api/v2/coupons/valid-discount/'+coupon) + $http.get(ApiUrl.get() + '/api/v2/coupons/valid-discount/'+coupon) .success(function(){ Notification.text("Coupon applied!"); var subs = $scope.Content.subscriptionBlocks; diff --git a/public/js/controllers/tasksCtrl.js b/public/js/controllers/tasksCtrl.js index fd8bcfe0a9..17cfab26dc 100644 --- a/public/js/controllers/tasksCtrl.js +++ b/public/js/controllers/tasksCtrl.js @@ -1,7 +1,7 @@ "use strict"; -habitrpg.controller("TasksCtrl", ['$scope', '$rootScope', '$location', 'User','Notification', '$http', 'ApiUrlService', '$timeout', 'Shared', - function($scope, $rootScope, $location, User, Notification, $http, ApiUrlService, $timeout, Shared) { +habitrpg.controller("TasksCtrl", ['$scope', '$rootScope', '$location', 'User','Notification', '$http', 'ApiUrl', '$timeout', 'Shared', + function($scope, $rootScope, $location, User, Notification, $http, ApiUrl, $timeout, Shared) { $scope.obj = User.user; // used for task-lists $scope.user = User.user; @@ -88,7 +88,7 @@ habitrpg.controller("TasksCtrl", ['$scope', '$rootScope', '$location', 'User','N $scope.unlink = function(task, keep) { // TODO move this to userServices, turn userSerivces.user into ng-resource - $http.post(ApiUrlService.get() + '/api/v2/user/tasks/' + task.id + '/unlink?keep=' + keep) + $http.post(ApiUrl.get() + '/api/v2/user/tasks/' + task.id + '/unlink?keep=' + keep) .success(function(){ User.log({}); }); diff --git a/public/js/services/authServices.js b/public/js/services/authServices.js index 46bfaddb0d..5c7dc90be9 100644 --- a/public/js/services/authServices.js +++ b/public/js/services/authServices.js @@ -7,98 +7,94 @@ var facebook = {} -angular.module('authServices', ['userServices']). -factory('Facebook', - ['$http', '$location', 'User', 'ApiUrlService', - function($http, $location, User, ApiUrlService) { - //TODO FB.init({appId: '${section.parameters['facebook.app.id']}', status: true, cookie: true, xfbml: true}); - var auth, user = User.user; +angular.module('habitrpg').factory('Facebook', +['$http', '$location', 'User', 'ApiUrl', +function($http, $location, User, ApiUrl) { + //TODO FB.init({appId: '${section.parameters['facebook.app.id']}', status: true, cookie: true, xfbml: true}); + var auth, user = User.user; - facebook.handleStatusChange = function(session) { - if (session.authResponse) { + facebook.handleStatusChange = function(session) { + if (session.authResponse) { - FB.api('/me', { - fields: 'name, picture, email' - }, function(response) { - console.log(response.error) - if (!response.error) { + FB.api('/me', { + fields: 'name, picture, email' + }, function(response) { + console.log(response.error) + if (!response.error) { - var data = { - name: response.name, - facebook_id: response.id, - email: response.email - } - - $http.post(ApiUrlService.get() + '/api/v2/user/auth/facebook', data).success(function(data, status, headers, config) { - User.authenticate(data.id, data.token, function(err) { - if (!err) { - alert(window.env.t('loginSuccess')); - $location.path("/habit"); - } - }); - }).error(function(response) { - console.log('error') - }) - - } else { - alert('napaka') + var data = { + name: response.name, + facebook_id: response.id, + email: response.email } - //clearAction(); - }); - } else { - document.body.className = 'not_connected'; + + $http.post(ApiUrl.get() + '/api/v2/user/auth/facebook', data).success(function(data, status, headers, config) { + User.authenticate(data.id, data.token, function(err) { + if (!err) { + alert(window.env.t('loginSuccess')); + $location.path("/habit"); + } + }); + }).error(function(response) { + console.log('error') + }) + + } else { + alert('napaka') + } //clearAction(); - } + }); + } else { + document.body.className = 'not_connected'; + //clearAction(); } - - return { - - authUser: function() { - FB.Event.subscribe('auth.statusChange', facebook.handleStatusChange); - }, - - getAuth: function() { - return auth; - }, - - login: function() { - - FB.login(null, { - scope: 'email' - }); - }, - - logout: function() { - FB.logout(function(response) { - window.location.reload(); - }); - } - } - } -]) + + return { + + authUser: function() { + FB.Event.subscribe('auth.statusChange', facebook.handleStatusChange); + }, + + getAuth: function() { + return auth; + }, + + login: function() { + + FB.login(null, { + scope: 'email' + }); + }, + + logout: function() { + FB.logout(function(response) { + window.location.reload(); + }); + } + } + +}]) .factory('LocalAuth', - ['$http', 'User', - function($http, User) { - var auth, - user = User.user; +['$http', 'User', +function($http, User) { + var auth, + user = User.user; - return { - getAuth: function() { - return auth; - }, + return { + getAuth: function() { + return auth; + }, - login: function() { - user.id = ''; - user.apiToken = ''; - User.authenticate(); - return; + login: function() { + user.id = ''; + user.apiToken = ''; + User.authenticate(); + return; - }, - - logout: function() {} - } + }, + logout: function() {} } -]); +}]); diff --git a/public/js/services/challengeServices.js b/public/js/services/challengeServices.js index 0b1d39eaf8..51b91de8c2 100644 --- a/public/js/services/challengeServices.js +++ b/public/js/services/challengeServices.js @@ -4,24 +4,23 @@ * Services that persists and retrieves user from localStorage. */ -angular.module('challengeServices', ['ngResource']). - factory('Challenges', ['ApiUrlService', '$resource', 'User', '$q', 'Members', - function(ApiUrlService, $resource, User, $q, Members) { - var Challenge = $resource(ApiUrlService.get() + '/api/v2/challenges/:cid', - {cid:'@_id'}, - { - //'query': {method: "GET", isArray:false} - join: {method: "POST", url: ApiUrlService.get() + '/api/v2/challenges/:cid/join'}, - leave: {method: "POST", url: ApiUrlService.get() + '/api/v2/challenges/:cid/leave'}, - close: {method: "POST", params: {uid:''}, url: ApiUrlService.get() + '/api/v2/challenges/:cid/close'}, - getMember: {method: "GET", url: ApiUrlService.get() + '/api/v2/challenges/:cid/member/:uid'} - }); +angular.module('habitrpg').factory('Challenges', +['ApiUrl', '$resource', +function(ApiUrl, $resource) { + var Challenge = $resource(ApiUrl.get() + '/api/v2/challenges/:cid', + {cid:'@_id'}, + { + //'query': {method: "GET", isArray:false} + join: {method: "POST", url: ApiUrl.get() + '/api/v2/challenges/:cid/join'}, + leave: {method: "POST", url: ApiUrl.get() + '/api/v2/challenges/:cid/leave'}, + close: {method: "POST", params: {uid:''}, url: ApiUrl.get() + '/api/v2/challenges/:cid/close'}, + getMember: {method: "GET", url: ApiUrl.get() + '/api/v2/challenges/:cid/member/:uid'} + }); - //var challenges = []; + //var challenges = []; - return { - Challenge: Challenge - //challenges: challenges - } - } -]); + return { + Challenge: Challenge + //challenges: challenges + } +}]); diff --git a/public/js/services/groupServices.js b/public/js/services/groupServices.js index 8707424a08..03fe1a708e 100644 --- a/public/js/services/groupServices.js +++ b/public/js/services/groupServices.js @@ -4,76 +4,74 @@ * Services that persists and retrieves user from localStorage. */ -angular.module('groupServices', ['ngResource', 'challengeServices']). - factory('Groups', ['ApiUrlService', '$resource', '$q', '$http', 'User', 'Challenges', - function(ApiUrlService, $resource, $q, $http, User, Challenges) { - - var Group = $resource(ApiUrlService.get() + '/api/v2/groups/:gid', - {gid:'@_id', messageId: '@_messageId'}, - { - get: { - method: "GET", - isArray:false, - // Wrap challenges as ngResource so they have functions like $leave or $join - transformResponse: function(data, headers) { - data = angular.fromJson(data); - _.each(data && data.challenges, function(c) { - angular.extend(c, Challenges.Challenge.prototype); - }); - return data; - } - }, - - postChat: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/chat'}, - deleteChatMessage: {method: "DELETE", url: ApiUrlService.get() + '/api/v2/groups/:gid/chat/:messageId'}, - flagChatMessage: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/chat/:messageId/flag'}, - clearFlagCount: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/chat/:messageId/clearflags'}, - join: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/join'}, - leave: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/leave'}, - invite: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/invite'}, - removeMember: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/removeMember'}, - questAccept: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/questAccept'}, - questReject: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/questReject'}, - questCancel: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/questCancel'}, - questAbort: {method: "POST", url: ApiUrlService.get() + '/api/v2/groups/:gid/questAbort'} +angular.module('habitrpg').factory('Groups', +['ApiUrl', '$resource', '$q', '$http', 'User', 'Challenges', +function(ApiUrl, $resource, $q, $http, User, Challenges) { + var Group = $resource(ApiUrl.get() + '/api/v2/groups/:gid', + {gid:'@_id', messageId: '@_messageId'}, + { + get: { + method: "GET", + isArray:false, + // Wrap challenges as ngResource so they have functions like $leave or $join + transformResponse: function(data, headers) { + data = angular.fromJson(data); + _.each(data && data.challenges, function(c) { + angular.extend(c, Challenges.Challenge.prototype); }); - - // Defer loading everything until they're requested - var data = {party: undefined, myGuilds: undefined, publicGuilds: undefined, tavern: undefined}; - - return { - party: function(cb){ - if (!data.party) return (data.party = Group.get({gid: 'party'}, cb)); - return (cb) ? cb(party) : data.party; - }, - publicGuilds: function(){ - //TODO combine these as {type:'guilds,public'} and create a $filter() to separate them - if (!data.publicGuilds) data.publicGuilds = Group.query({type:'public'}); - return data.publicGuilds; - }, - myGuilds: function(){ - if (!data.myGuilds) data.myGuilds = Group.query({type:'guilds'}); - return data.myGuilds; - }, - tavern: function(){ - if (!data.tavern) data.tavern = Group.get({gid:'habitrpg'}); - return data.tavern; - }, - - // On enter, set chat message to "seen" - seenMessage: function(gid){ - $http.post(ApiUrlService.get() + '/api/v2/groups/'+gid+'/chat/seen'); - if (User.user.newMessages) delete User.user.newMessages[gid]; - }, - - // Pass reference to party, myGuilds, publicGuilds, tavern; inside data in order to - // be able to modify them directly (otherwise will be stick with cached version) - data: data, - - Group: Group + return data; } - } - ]) + }, + + postChat: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/chat'}, + deleteChatMessage: {method: "DELETE", url: ApiUrl.get() + '/api/v2/groups/:gid/chat/:messageId'}, + flagChatMessage: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/chat/:messageId/flag'}, + clearFlagCount: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/chat/:messageId/clearflags'}, + join: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/join'}, + leave: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/leave'}, + invite: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/invite'}, + removeMember: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/removeMember'}, + questAccept: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/questAccept'}, + questReject: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/questReject'}, + questCancel: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/questCancel'}, + questAbort: {method: "POST", url: ApiUrl.get() + '/api/v2/groups/:gid/questAbort'} + }); + + // Defer loading everything until they're requested + var data = {party: undefined, myGuilds: undefined, publicGuilds: undefined, tavern: undefined}; + + return { + party: function(cb){ + if (!data.party) return (data.party = Group.get({gid: 'party'}, cb)); + return (cb) ? cb(party) : data.party; + }, + publicGuilds: function(){ + //TODO combine these as {type:'guilds,public'} and create a $filter() to separate them + if (!data.publicGuilds) data.publicGuilds = Group.query({type:'public'}); + return data.publicGuilds; + }, + myGuilds: function(){ + if (!data.myGuilds) data.myGuilds = Group.query({type:'guilds'}); + return data.myGuilds; + }, + tavern: function(){ + if (!data.tavern) data.tavern = Group.get({gid:'habitrpg'}); + return data.tavern; + }, + + // On enter, set chat message to "seen" + seenMessage: function(gid){ + $http.post(ApiUrl.get() + '/api/v2/groups/'+gid+'/chat/seen'); + if (User.user.newMessages) delete User.user.newMessages[gid]; + }, + + // Pass reference to party, myGuilds, publicGuilds, tavern; inside data in order to + // be able to modify them directly (otherwise will be stick with cached version) + data: data, + + Group: Group + } +}]) /** * TODO Get this working. Make ChatService it's own ngResource, so we can update chat without having to sync the whole * group object (expensive). Also so we can add chat-specific routes diff --git a/public/js/services/guideServices.js b/public/js/services/guideServices.js index 3b808f0ba1..8e4eb1bc60 100644 --- a/public/js/services/guideServices.js +++ b/public/js/services/guideServices.js @@ -4,219 +4,217 @@ * Services for each tour step when you unlock features */ -angular.module('guideServices', []). - factory('Guide', ['$rootScope', 'User', '$timeout', function($rootScope, User, $timeout) { - - /** - * Init and show the welcome tour. Note we do it listening to a $rootScope broadcasted 'userLoaded' message, - * this because we need to determine whether to show the tour *after* the user has been pulled from the server, - * otherwise it's always start off as true, and then get set to false later - */ - $rootScope.$on('userUpdated', initTour); - function initTour(){ - if (User.user.flags.showTour === false) return; - var tourSteps = [ - { - element: ".main-herobox", - title: window.env.t('welcomeHabit'), - content: window.env.t('welcomeHabitT1') + " Justin, " + window.env.t('welcomeHabitT2'), - }, { - element: ".main-herobox", - title: window.env.t('yourAvatar'), - content: window.env.t('yourAvatarText'), - }, { - element: ".main-herobox", - title: window.env.t('avatarCustom'), - content: window.env.t('avatarCustomText'), - }, { - element: "#bars", - title: window.env.t('hitPoints'), - content: window.env.t('hitPointsText'), - }, { - element: "#bars", - title: window.env.t('expPoints'), - content: window.env.t('expPointsText'), - }, { - element: "ul.habits", - title: window.env.t('typeGoals'), - content: window.env.t('typeGoalsText'), - placement: "top" - }, { - element: "ul.habits", - title: window.env.t('habits'), - content: window.env.t('tourHabits'), - placement: "top" - }, { - element: "ul.dailys", - title: window.env.t('dailies'), - content: window.env.t('tourDailies'), - placement: "top" - }, { - element: "ul.todos", - title: window.env.t('todos'), - content: window.env.t('tourTodos'), - placement: "top" - }, { - element: "ul.main-list.rewards", - title: window.env.t('rewards'), - content: window.env.t('tourRewards'), - placement: "top" - }, { - element: "ul.habits li:first-child", - title: window.env.t('hoverOver'), - content: window.env.t('hoverOverText'), - placement: "right" - }, { - element: "ul.habits li:first-child", - title: window.env.t('unlockFeatures'), - content: window.env.t('unlockFeaturesT1') + " " + window.env.t('habitWiki') + " " + window.env.t('unlockFeaturesT2'), - placement: "right" - } - ]; - _.each(tourSteps, function(step){ - if (env.worldDmg.guide) { - step.content = "
" + step.content + "
"; - } else { - step.content = "
" + step.content + "
"; - } - }); - $('.main-herobox').popover('destroy'); - var tour = new Tour({ - template: "
" + - "

" + - "
" + - "" + - "" + - "
" + - "
", - onEnd: function(){ - User.set({'flags.showTour': false}); - } - }); - tourSteps.forEach(function(step) { - tour.addStep(_.defaults(step, {html: true})); - }); - tour.restart(); // Tour doesn't quite mesh with our handling of flags.showTour, just restart it on page load - //tour.start(true); - }; - - var alreadyShown = function(before, after) { - return !(!before && after === true); - }; - - var showPopover = function(selector, title, html, placement) { - if (!placement) placement = 'bottom'; - $(selector).popover('destroy'); - var button = ""; +angular.module('habitrpg').factory('Guide', +['$rootScope', 'User', '$timeout', +function($rootScope, User, $timeout) { + /** + * Init and show the welcome tour. Note we do it listening to a $rootScope broadcasted 'userLoaded' message, + * this because we need to determine whether to show the tour *after* the user has been pulled from the server, + * otherwise it's always start off as true, and then get set to false later + */ + $rootScope.$on('userUpdated', initTour); + function initTour(){ + if (User.user.flags.showTour === false) return; + var tourSteps = [ + { + element: ".main-herobox", + title: window.env.t('welcomeHabit'), + content: window.env.t('welcomeHabitT1') + " Justin, " + window.env.t('welcomeHabitT2'), + }, { + element: ".main-herobox", + title: window.env.t('yourAvatar'), + content: window.env.t('yourAvatarText'), + }, { + element: ".main-herobox", + title: window.env.t('avatarCustom'), + content: window.env.t('avatarCustomText'), + }, { + element: "#bars", + title: window.env.t('hitPoints'), + content: window.env.t('hitPointsText'), + }, { + element: "#bars", + title: window.env.t('expPoints'), + content: window.env.t('expPointsText'), + }, { + element: "ul.habits", + title: window.env.t('typeGoals'), + content: window.env.t('typeGoalsText'), + placement: "top" + }, { + element: "ul.habits", + title: window.env.t('habits'), + content: window.env.t('tourHabits'), + placement: "top" + }, { + element: "ul.dailys", + title: window.env.t('dailies'), + content: window.env.t('tourDailies'), + placement: "top" + }, { + element: "ul.todos", + title: window.env.t('todos'), + content: window.env.t('tourTodos'), + placement: "top" + }, { + element: "ul.main-list.rewards", + title: window.env.t('rewards'), + content: window.env.t('tourRewards'), + placement: "top" + }, { + element: "ul.habits li:first-child", + title: window.env.t('hoverOver'), + content: window.env.t('hoverOverText'), + placement: "right" + }, { + element: "ul.habits li:first-child", + title: window.env.t('unlockFeatures'), + content: window.env.t('unlockFeaturesT1') + " " + window.env.t('habitWiki') + " " + window.env.t('unlockFeaturesT2'), + placement: "right" + } + ]; + _.each(tourSteps, function(step){ if (env.worldDmg.guide) { - html = "
" + html + '
' + button + '
'; + step.content = "
" + step.content + "
"; } else { - html = "
" + html + '
' + button + '
'; + step.content = "
" + step.content + "
"; } - $(selector).popover({ - title: title, - placement: placement, - trigger: 'manual', - html: true, - content: html - }).popover('show'); - }; - - $rootScope.$watch('user.flags.customizationsNotification', function(after, before) { - if (alreadyShown(before, after)) return; - showPopover('.main-herobox', window.env.t('customAvatar'), window.env.t('customAvatarText'), 'bottom'); }); - - $rootScope.$watch('user.flags.itemsEnabled', function(after, before) { - if (alreadyShown(before, after)) return; - var html = window.env.t('storeUnlockedText'); - showPopover('div.rewards', window.env.t('storeUnlocked'), html, 'left'); - }); - - $rootScope.$watch('user.flags.partyEnabled', function(after, before) { - if (alreadyShown(before, after)) return; - var html = window.env.t('partySysText'); - showPopover('.user-menu', window.env.t('partySys'), html, 'bottom'); - }); - - $rootScope.$watch('user.flags.dropsEnabled', function(after, before) { - if (alreadyShown(before, after)) return; - var eggs = User.user.items.eggs || {}; - if (!eggs) { - eggs['Wolf'] = 1; // This is also set on the server + $('.main-herobox').popover('destroy'); + var tour = new Tour({ + template: "
" + + "

" + + "
" + + "" + + "" + + "
" + + "
", + onEnd: function(){ + User.set({'flags.showTour': false}); } - $rootScope.openModal('dropsEnabled'); }); - - $rootScope.$watch('user.flags.rebirthEnabled', function(after, before) { - if (alreadyShown(before, after)) return; - $rootScope.openModal('rebirthEnabled'); + tourSteps.forEach(function(step) { + tour.addStep(_.defaults(step, {html: true})); }); + tour.restart(); // Tour doesn't quite mesh with our handling of flags.showTour, just restart it on page load + //tour.start(true); + }; + + var alreadyShown = function(before, after) { + return !(!before && after === true); + }; + + var showPopover = function(selector, title, html, placement) { + if (!placement) placement = 'bottom'; + $(selector).popover('destroy'); + var button = ""; + if (env.worldDmg.guide) { + html = "
" + html + '
' + button + '
'; + } else { + html = "
" + html + '
' + button + '
'; + } + $(selector).popover({ + title: title, + placement: placement, + trigger: 'manual', + html: true, + content: html + }).popover('show'); + }; + + $rootScope.$watch('user.flags.customizationsNotification', function(after, before) { + if (alreadyShown(before, after)) return; + showPopover('.main-herobox', window.env.t('customAvatar'), window.env.t('customAvatarText'), 'bottom'); + }); + + $rootScope.$watch('user.flags.itemsEnabled', function(after, before) { + if (alreadyShown(before, after)) return; + var html = window.env.t('storeUnlockedText'); + showPopover('div.rewards', window.env.t('storeUnlocked'), html, 'left'); + }); + + $rootScope.$watch('user.flags.partyEnabled', function(after, before) { + if (alreadyShown(before, after)) return; + var html = window.env.t('partySysText'); + showPopover('.user-menu', window.env.t('partySys'), html, 'bottom'); + }); + + $rootScope.$watch('user.flags.dropsEnabled', function(after, before) { + if (alreadyShown(before, after)) return; + var eggs = User.user.items.eggs || {}; + if (!eggs) { + eggs['Wolf'] = 1; // This is also set on the server + } + $rootScope.openModal('dropsEnabled'); + }); + + $rootScope.$watch('user.flags.rebirthEnabled', function(after, before) { + if (alreadyShown(before, after)) return; + $rootScope.openModal('rebirthEnabled'); + }); - /** - * Classes Tour - */ - function classesTour(){ + /** + * Classes Tour + */ + function classesTour(){ - // TODO notice my hack-job `onShow: _.once()` functions. Without these, the syncronous path redirects won't properly handle showing tour - var tourSteps = [ - { - path: '/#/options/inventory/equipment', - onShow: _.once(function(tour){ - $timeout(function(){tour.goTo(0)}); - }), - element: '.equipment-tab', - title: window.env.t('classGear'), - content: window.env.t('classGearText', {klass: User.user.stats.class}) - }, - { - path: '/#/options/profile/stats', - onShow: _.once(function(tour){ - $timeout(function(){tour.goTo(1)}); - }), - element: ".allocate-stats", - title: window.env.t('stats'), - content: window.env.t('classStats'), - }, { - element: ".auto-allocate", - title: window.env.t('autoAllocate'), - placement: 'left', - content: window.env.t('autoAllocateText'), - }, { - element: ".meter.mana", - title: window.env.t('spells'), - content: window.env.t('spellsText') + " " + window.env.t('toDo') + "." - }, { - orphan: true, - title: window.env.t('readMore'), - content: window.env.t('moreClass') + " Wikia." - } - ]; - _.each(tourSteps, function(step){ - if (env.worldDmg.guide) { - step.content = "
" + step.content + "
"; - } else { - step.content = "
" + step.content + "
"; - } - }); - $('.allocate-stats').popover('destroy'); - var tour = new Tour({ + // TODO notice my hack-job `onShow: _.once()` functions. Without these, the syncronous path redirects won't properly handle showing tour + var tourSteps = [ + { + path: '/#/options/inventory/equipment', + onShow: _.once(function(tour){ + $timeout(function(){tour.goTo(0)}); + }), + element: '.equipment-tab', + title: window.env.t('classGear'), + content: window.env.t('classGearText', {klass: User.user.stats.class}) + }, + { + path: '/#/options/profile/stats', + onShow: _.once(function(tour){ + $timeout(function(){tour.goTo(1)}); + }), + element: ".allocate-stats", + title: window.env.t('stats'), + content: window.env.t('classStats'), + }, { + element: ".auto-allocate", + title: window.env.t('autoAllocate'), + placement: 'left', + content: window.env.t('autoAllocateText'), + }, { + element: ".meter.mana", + title: window.env.t('spells'), + content: window.env.t('spellsText') + " " + window.env.t('toDo') + "." + }, { + orphan: true, + title: window.env.t('readMore'), + content: window.env.t('moreClass') + " Wikia." + } + ]; + _.each(tourSteps, function(step){ + if (env.worldDmg.guide) { + step.content = "
" + step.content + "
"; + } else { + step.content = "
" + step.content + "
"; + } + }); + $('.allocate-stats').popover('destroy'); + var tour = new Tour({ // onEnd: function(){ // User.set({'flags.showTour': false}); // } - }); - tourSteps.forEach(function(step) { - tour.addStep(_.defaults(step, {html: true})); - }); - tour.restart(); // Tour doesn't quite mesh with our handling of flags.showTour, just restart it on page load - //tour.start(true); - }; + }); + tourSteps.forEach(function(step) { + tour.addStep(_.defaults(step, {html: true})); + }); + tour.restart(); // Tour doesn't quite mesh with our handling of flags.showTour, just restart it on page load + //tour.start(true); + }; - return { - initTour: initTour, - classesTour: classesTour - }; - - } -]); + return { + initTour: initTour, + classesTour: classesTour + }; +}]); diff --git a/public/js/services/memberServices.js b/public/js/services/memberServices.js index bb3a4c5ac5..53a1c1e998 100644 --- a/public/js/services/memberServices.js +++ b/public/js/services/memberServices.js @@ -4,85 +4,84 @@ * Services that persists and retrieves user from localStorage. */ -angular.module('memberServices', ['ngResource', 'sharedServices']). - factory('Members', ['$rootScope', 'Shared', 'ApiUrlService', '$resource', - function($rootScope, Shared, ApiUrlService, $resource) { - var members = {}; - var Member = $resource(ApiUrlService.get() + '/api/v2/members/:uid', {uid:'@_id'}); - var memberServices = { +angular.module('habitrpg').factory('Members', +['$rootScope', 'Shared', 'ApiUrl', '$resource', +function($rootScope, Shared, ApiUrl, $resource) { + var members = {}; + var Member = $resource(ApiUrl.get() + '/api/v2/members/:uid', {uid:'@_id'}); + var memberServices = { - Member: Member, + Member: Member, - members: members, + members: members, - /** - * Allows us to lazy-load party / group / public members throughout the application. - * @param obj - either a group or an individual member. If it's a group, we lazy-load all of its members. - */ - populate: function(obj){ + /** + * Allows us to lazy-load party / group / public members throughout the application. + * @param obj - either a group or an individual member. If it's a group, we lazy-load all of its members. + */ + populate: function(obj){ - function populateGroup(group){ - _.each(group.members, function(member){ - // meaning `populate('members')` wasn't run on the server, so we're getting the "in-database" form of - // the members array, which is just a list of IDs - not the populated objects - if (_.isString(member)) return; + function populateGroup(group){ + _.each(group.members, function(member){ + // meaning `populate('members')` wasn't run on the server, so we're getting the "in-database" form of + // the members array, which is just a list of IDs - not the populated objects + if (_.isString(member)) return; - // lazy-load - members[member._id] = member; - }) - } + // lazy-load + members[member._id] = member; + }) + } - // Array of groups - if (_.isArray(obj)) { - if (obj[0] && obj[0].members) { - _.each(obj, function(group){ - populateGroup(group); - }) - } - - // Individual Group - } else if (obj.members) - populateGroup(obj); - - // individual Member - if (obj._id) { - members[obj._id] = obj; - } - }, - - selectedMember: undefined, - - /** - * Once users are populated, we fetch them throughout the application (eg, modals). This - * either gets them or fetches if not available - * @param uid - */ - selectMember: function(uid, cb) { - var self = this; - // Fetch from cache if we can. For guild members, only their uname will have been fetched on initial load, - // check if they have full fields (eg, check profile.items and an item inside - // because sometimes profile.items exists but it's empty like when user is fetched for party - // and then for guild) - // and if not, fetch them - if (members[uid] && members[uid].items && members[uid].items.weapon) { - Shared.wrap(members[uid],false); - self.selectedMember = members[uid]; - cb(); - } else { - Member.get({uid: uid}, function(member){ - self.populate(member); // lazy load for later - Shared.wrap(member,false); - self.selectedMember = members[member._id]; - cb(); - }); - } - } + // Array of groups + if (_.isArray(obj)) { + if (obj[0] && obj[0].members) { + _.each(obj, function(group){ + populateGroup(group); + }) } - $rootScope.$on('userUpdated', function(event, user){ - memberServices.populate(user); - }) + // Individual Group + } else if (obj.members) + populateGroup(obj); - return memberServices; + // individual Member + if (obj._id) { + members[obj._id] = obj; } -]); + }, + + selectedMember: undefined, + + /** + * Once users are populated, we fetch them throughout the application (eg, modals). This + * either gets them or fetches if not available + * @param uid + */ + selectMember: function(uid, cb) { + var self = this; + // Fetch from cache if we can. For guild members, only their uname will have been fetched on initial load, + // check if they have full fields (eg, check profile.items and an item inside + // because sometimes profile.items exists but it's empty like when user is fetched for party + // and then for guild) + // and if not, fetch them + if (members[uid] && members[uid].items && members[uid].items.weapon) { + Shared.wrap(members[uid],false); + self.selectedMember = members[uid]; + cb(); + } else { + Member.get({uid: uid}, function(member){ + self.populate(member); // lazy load for later + Shared.wrap(member,false); + self.selectedMember = members[member._id]; + cb(); + }); + } + } + } + + $rootScope.$on('userUpdated', function(event, user){ + memberServices.populate(user); + }) + + return memberServices; +}]); diff --git a/public/js/services/notificationServices.js b/public/js/services/notificationServices.js index da2cd51afa..a0dde36fbb 100644 --- a/public/js/services/notificationServices.js +++ b/public/js/services/notificationServices.js @@ -1,85 +1,84 @@ /** Set up "+1 Exp", "Level Up", etc notifications */ -angular.module("notificationServices", []) - .factory("Notification", [function() { - var stack_topright = {"dir1": "down", "dir2": "left", "spacing1": 15, "spacing2": 15, "firstpos1": 60}; - function notify(html, type, icon) { - var notice = $.pnotify({ - type: type || 'warning', //('info', 'text', 'warning', 'success', 'gp', 'xp', 'hp', 'lvl', 'death', 'mp', 'crit') - text: html, - opacity: 1, - addclass: 'alert-' + type, - delay: 7000, - hide: (type == 'error') ? false : true, - mouse_reset: false, - width: "250px", - stack: stack_topright, - icon: icon || false - }).click(function() { notice.pnotify_remove() }); - }; +angular.module("habitrpg").factory("Notification", +[function() { + var stack_topright = {"dir1": "down", "dir2": "left", "spacing1": 15, "spacing2": 15, "firstpos1": 60}; + function notify(html, type, icon) { + var notice = $.pnotify({ + type: type || 'warning', //('info', 'text', 'warning', 'success', 'gp', 'xp', 'hp', 'lvl', 'death', 'mp', 'crit') + text: html, + opacity: 1, + addclass: 'alert-' + type, + delay: 7000, + hide: (type == 'error') ? false : true, + mouse_reset: false, + width: "250px", + stack: stack_topright, + icon: icon || false + }).click(function() { notice.pnotify_remove() }); + }; - /** - Show "+ 5 {gold_coin} 3 {silver_coin}" - */ - function coins(money) { - var absolute, gold, silver; - absolute = Math.abs(money); - gold = Math.floor(absolute); - silver = Math.floor((absolute - gold) * 100); - if (gold && silver > 0) { - return "" + gold + " " + silver + " "; - } else if (gold > 0) { - return "" + gold + " "; - } else if (silver > 0) { - return "" + silver + " "; - } - }; - - var sign = function(number){ - return number?number<0?'-':'+':'+'; + /** + Show "+ 5 {gold_coin} 3 {silver_coin}" + */ + function coins(money) { + var absolute, gold, silver; + absolute = Math.abs(money); + gold = Math.floor(absolute); + silver = Math.floor((absolute - gold) * 100); + if (gold && silver > 0) { + return "" + gold + " " + silver + " "; + } else if (gold > 0) { + return "" + gold + " "; + } else if (silver > 0) { + return "" + silver + " "; } + }; - var round = function(number){ - return Math.abs(number.toFixed(1)); - } - - return { - coins: coins, - hp: function(val) { - // don't show notifications if user dead - notify(sign(val) + " " + round(val) + " " + window.env.t('hp'), 'hp', 'glyphicon glyphicon-heart'); - }, - exp: function(val) { - if (val < -50) return; // don't show when they level up (resetting their exp) - notify(sign(val) + " " + round(val) + " " + window.env.t('xp'), 'xp', 'glyphicon glyphicon-star'); - }, - gp: function(val, bonus) { - notify(sign(val) + " " + coins(val - bonus), 'gp'); - }, - text: function(val){ - if (val) { - notify(val, 'info'); - } - }, - lvl: function(){ - notify(window.env.t('levelUp'), 'lvl', 'glyphicon glyphicon-chevron-up'); - }, - error: function(error){ - notify(error, "danger", 'glyphicon glyphicon-exclamation-sign'); - }, - mp: function(val) { - notify(sign(val) + " " + round(val) + " " + window.env.t('mp'), 'mp', 'glyphicon glyphicon-fire'); - }, - crit: function(val) { - notify(window.env.t('critBonus') + Math.round(val) + "%", 'crit', 'glyphicon glyphicon-certificate'); - }, - streak: function(val) { - notify(window.env.t('streakName') + ': ' + val, 'streak', 'glyphicon glyphicon-repeat'); - }, - drop: function(val) { - notify(val, 'drop', 'glyphicon glyphicon-gift'); - } - }; + var sign = function(number){ + return number?number<0?'-':'+':'+'; } -]); + + var round = function(number){ + return Math.abs(number.toFixed(1)); + } + + return { + coins: coins, + hp: function(val) { + // don't show notifications if user dead + notify(sign(val) + " " + round(val) + " " + window.env.t('hp'), 'hp', 'glyphicon glyphicon-heart'); + }, + exp: function(val) { + if (val < -50) return; // don't show when they level up (resetting their exp) + notify(sign(val) + " " + round(val) + " " + window.env.t('xp'), 'xp', 'glyphicon glyphicon-star'); + }, + gp: function(val, bonus) { + notify(sign(val) + " " + coins(val - bonus), 'gp'); + }, + text: function(val){ + if (val) { + notify(val, 'info'); + } + }, + lvl: function(){ + notify(window.env.t('levelUp'), 'lvl', 'glyphicon glyphicon-chevron-up'); + }, + error: function(error){ + notify(error, "danger", 'glyphicon glyphicon-exclamation-sign'); + }, + mp: function(val) { + notify(sign(val) + " " + round(val) + " " + window.env.t('mp'), 'mp', 'glyphicon glyphicon-fire'); + }, + crit: function(val) { + notify(window.env.t('critBonus') + Math.round(val) + "%", 'crit', 'glyphicon glyphicon-certificate'); + }, + streak: function(val) { + notify(window.env.t('streakName') + ': ' + val, 'streak', 'glyphicon glyphicon-repeat'); + }, + drop: function(val) { + notify(val, 'drop', 'glyphicon glyphicon-gift'); + } + }; +}]); diff --git a/public/js/services/paymentServices.js b/public/js/services/paymentServices.js index e0a059c07c..bc677b9155 100644 --- a/public/js/services/paymentServices.js +++ b/public/js/services/paymentServices.js @@ -1,6 +1,6 @@ 'use strict'; -angular.module('paymentServices',[]).factory('Payments', +angular.module('habitrpg').factory('Payments', ['$rootScope', 'User', '$http', 'Content', function($rootScope, User, $http, Content) { var Payments = {}; diff --git a/public/js/services/sharedServices.js b/public/js/services/sharedServices.js index 31ef0fc120..5694d56fb3 100644 --- a/public/js/services/sharedServices.js +++ b/public/js/services/sharedServices.js @@ -4,12 +4,10 @@ * Services that expose habitrpg-shared */ -angular.module('sharedServices', []). - factory('Shared', [function () { - return window.habitrpgShared; - } -]). - factory('Content', ['Shared', function (Shared) { - return Shared.content; - } -]); +angular.module('habitrpg') +.factory('Shared', [function () { + return window.habitrpgShared; +}]) +.factory('Content', ['Shared', function (Shared) { + return Shared.content; +}]); diff --git a/public/js/static.js b/public/js/static.js index 860dbf5587..a192a301db 100644 --- a/public/js/static.js +++ b/public/js/static.js @@ -1,6 +1,6 @@ "use strict"; -window.habitrpgStatic = angular.module('habitrpgStatic', ['notificationServices', 'userServices', 'chieffancypants.loadingBar', 'authCtrl', 'ui.bootstrap']) +window.habitrpg = angular.module('habitrpg', ['chieffancypants.loadingBar', 'authCtrl', 'ui.bootstrap']) .constant("API_URL", "") .constant("STORAGE_USER_ID", 'habitrpg-user') .constant("STORAGE_SETTINGS_ID", 'habit-mobile-settings') diff --git a/views/static/layout.jade b/views/static/layout.jade index 3c8bd38bd7..8744b06518 100644 --- a/views/static/layout.jade +++ b/views/static/layout.jade @@ -1,7 +1,7 @@ //-Trick needed to pass 'env' to ./layout block vars doctype html -html(ng-app='habitrpgStatic') +html(ng-app='habitrpg') head block extraHead diff --git a/views/static/new-stuff.jade b/views/static/new-stuff.jade index 9cbf77b87f..6757b48f68 100644 --- a/views/static/new-stuff.jade +++ b/views/static/new-stuff.jade @@ -1,6 +1,6 @@ block vars doctype html -html(ng-app='habitrpgStatic') +html(ng-app='habitrpg') head block title From 503fd9ff51649bbf26a77c7593505e81291adb2b Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Sat, 17 Jan 2015 18:54:03 -0700 Subject: [PATCH 27/50] refactor(services): remove un-used authService, cleanup authCtrl --- public/js/app.js | 2 +- public/js/controllers/authCtrl.js | 2 +- public/js/services/authServices.js | 100 ----------------------------- public/js/static.js | 2 +- public/manifest.json | 1 - 5 files changed, 3 insertions(+), 104 deletions(-) delete mode 100644 public/js/services/authServices.js diff --git a/public/js/app.js b/public/js/app.js index a5bcb6c66a..4b5a38b9fe 100644 --- a/public/js/app.js +++ b/public/js/app.js @@ -1,7 +1,7 @@ "use strict"; window.habitrpg = angular.module('habitrpg', - ['authCtrl', 'ui.bootstrap', 'ui.keypress', 'ui.router', 'chieffancypants.loadingBar', 'At', 'infinite-scroll', 'ui.select2', 'angular.filter', 'ngResource']) + ['ui.bootstrap', 'ui.keypress', 'ui.router', 'chieffancypants.loadingBar', 'At', 'infinite-scroll', 'ui.select2', 'angular.filter', 'ngResource']) // @see https://github.com/angular-ui/ui-router/issues/110 and https://github.com/HabitRPG/habitrpg/issues/1705 // temporary hack until they have a better solution diff --git a/public/js/controllers/authCtrl.js b/public/js/controllers/authCtrl.js index 321058ae6f..a167ecb309 100644 --- a/public/js/controllers/authCtrl.js +++ b/public/js/controllers/authCtrl.js @@ -4,7 +4,7 @@ The authentication controller (login & facebook) */ -angular.module('authCtrl', []) +angular.module('habitrpg') .controller("AuthCtrl", ['$scope', '$rootScope', 'User', '$http', '$location', '$window','ApiUrl', '$modal', function($scope, $rootScope, User, $http, $location, $window, ApiUrl, $modal) { diff --git a/public/js/services/authServices.js b/public/js/services/authServices.js deleted file mode 100644 index 5c7dc90be9..0000000000 --- a/public/js/services/authServices.js +++ /dev/null @@ -1,100 +0,0 @@ -'use strict'; - -/** - * Services that persists and retrieves user from localStorage. - * FIXME is this file ever used? - */ - -var facebook = {} - -angular.module('habitrpg').factory('Facebook', -['$http', '$location', 'User', 'ApiUrl', -function($http, $location, User, ApiUrl) { - //TODO FB.init({appId: '${section.parameters['facebook.app.id']}', status: true, cookie: true, xfbml: true}); - var auth, user = User.user; - - facebook.handleStatusChange = function(session) { - if (session.authResponse) { - - FB.api('/me', { - fields: 'name, picture, email' - }, function(response) { - console.log(response.error) - if (!response.error) { - - var data = { - name: response.name, - facebook_id: response.id, - email: response.email - } - - $http.post(ApiUrl.get() + '/api/v2/user/auth/facebook', data).success(function(data, status, headers, config) { - User.authenticate(data.id, data.token, function(err) { - if (!err) { - alert(window.env.t('loginSuccess')); - $location.path("/habit"); - } - }); - }).error(function(response) { - console.log('error') - }) - - } else { - alert('napaka') - } - //clearAction(); - }); - } else { - document.body.className = 'not_connected'; - //clearAction(); - } - } - - return { - - authUser: function() { - FB.Event.subscribe('auth.statusChange', facebook.handleStatusChange); - }, - - getAuth: function() { - return auth; - }, - - login: function() { - - FB.login(null, { - scope: 'email' - }); - }, - - logout: function() { - FB.logout(function(response) { - window.location.reload(); - }); - } - } - -}]) - -.factory('LocalAuth', -['$http', 'User', -function($http, User) { - var auth, - user = User.user; - - return { - getAuth: function() { - return auth; - }, - - login: function() { - user.id = ''; - user.apiToken = ''; - User.authenticate(); - return; - - }, - - logout: function() {} - } -}]); diff --git a/public/js/static.js b/public/js/static.js index a192a301db..8f88fb3551 100644 --- a/public/js/static.js +++ b/public/js/static.js @@ -1,6 +1,6 @@ "use strict"; -window.habitrpg = angular.module('habitrpg', ['chieffancypants.loadingBar', 'authCtrl', 'ui.bootstrap']) +window.habitrpg = angular.module('habitrpg', ['chieffancypants.loadingBar', 'ui.bootstrap']) .constant("API_URL", "") .constant("STORAGE_USER_ID", 'habitrpg-user') .constant("STORAGE_SETTINGS_ID", 'habit-mobile-settings') diff --git a/public/manifest.json b/public/manifest.json index 03df4c4535..15deca809c 100644 --- a/public/manifest.json +++ b/public/manifest.json @@ -39,7 +39,6 @@ "js/app.js", "bower_components/habitrpg-shared/script/config.js", "js/services/sharedServices.js", - "js/services/authServices.js", "js/services/notificationServices.js", "bower_components/habitrpg-shared/script/userServices.js", "bower_components/habitrpg-shared/script/directives.js", From 801a5a957a6f3b9a9a41d8b8889538e2e3a87113 Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:33:58 -0600 Subject: [PATCH 28/50] Removed target="_blank" attribute from mailto link --- views/options/settings.jade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 2a5fee087c..fd9ae4cc0c 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -108,7 +108,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') small.muted =env.t('emailChange1') |  - a(href='mailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + a(href='mailto:admin@habitrpg.com')=env.t('emailChange2') |  =env.t('emailChange3') hr From ad612a9c3691ebdca4b011a4bc43a59c70efee0d Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:38:16 -0600 Subject: [PATCH 29/50] Adjusted some spacing and line breaks --- views/options/settings.jade | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index fd9ae4cc0c..7f07c2d01e 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -88,12 +88,10 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-heading span Registration .panel-body - p(ng-if='user.auth.facebook.id') - =env.t('registeredWithFb') + p(ng-if='user.auth.facebook.id')=env.t('registeredWithFb') div(ng-if='user.auth.local.username') - p - =env.t('username') - |: {{user.auth.local.username}} + p=env.t('username') + |: {{user.auth.local.username}} p small.muted =env.t('loginNameDescription1') @@ -101,9 +99,8 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') a(href='/#/options/profile/profile')=env.t('loginNameDescription2') |  =env.t('loginNameDescription3') - p - =env.t('email') - |: {{user.auth.local.email}} + p=env.t('email') + |: {{user.auth.local.email}} p small.muted =env.t('emailChange1') From c4c2c6a1dc59b0301d42b14b20a110a739fefc78 Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:39:45 -0600 Subject: [PATCH 30/50] Adjusted spacing and line breaks --- views/options/profile.jade | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/views/options/profile.jade b/views/options/profile.jade index 5f6ed31b04..19a2ba48d1 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -214,12 +214,11 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') h4=env.t('displayName') span(ng-show='profile.profile.name') {{profile.profile.name}} p - small.muted - =env.t('displayNameDescription1') - |  - a(href='/#/options/settings/settings')=env.t('displayNameDescription2') - |  - =env.t('displayNameDescription3') + small.muted=env.t('displayNameDescription1') + |  + a(href='/#/options/settings/settings')=env.t('displayNameDescription2') + |  + =env.t('displayNameDescription3') span.muted(ng-hide='profile.profile.name') -  =env.t('none') |  - From 62c9aafb7d2fff40f261a5f0dff77833487749e2 Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:50:43 -0600 Subject: [PATCH 31/50] Added sync after changing username so username would update --- public/js/controllers/settingsCtrl.js | 1 + 1 file changed, 1 insertion(+) diff --git a/public/js/controllers/settingsCtrl.js b/public/js/controllers/settingsCtrl.js index 7220791520..599e29664c 100644 --- a/public/js/controllers/settingsCtrl.js +++ b/public/js/controllers/settingsCtrl.js @@ -85,6 +85,7 @@ habitrpg.controller('SettingsCtrl', .success(function(){ alert(window.env.t('usernameSuccess')); $scope.changeUser = {}; + User.sync(); }) .error(function(data){ alert(data.err); From 5026e67ead219f9d5a1f228351cf6efc0298f08f Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:51:00 -0600 Subject: [PATCH 32/50] Added one way binding for email --- views/options/settings.jade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 7f07c2d01e..b2437f81cf 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -100,7 +100,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') |  =env.t('loginNameDescription3') p=env.t('email') - |: {{user.auth.local.email}} + |: {{::user.auth.local.email}} p small.muted =env.t('emailChange1') From a053486fcde200fb212e9863ffefe60f3a562600 Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Sun, 18 Jan 2015 14:51:40 -0700 Subject: [PATCH 33/50] fix(avatar-sharing): angular@1.3.9 upgrade fixed the /#?memberId=x bug. Also, anon wasn't able to view avatar shares! --- bower.json | 4 ++-- public/js/static.js | 9 ++++++++- src/controllers/groups.js | 4 ++-- src/routes/apiv2.coffee | 2 +- views/shared/header/avatar.jade | 2 +- 5 files changed, 14 insertions(+), 7 deletions(-) diff --git a/bower.json b/bower.json index 3f4eee4a3f..2d62dbefef 100644 --- a/bower.json +++ b/bower.json @@ -45,10 +45,10 @@ "angular-ui-router": "~0.2.13" }, "devDependencies": { - "angular-mocks": "1.3.3" + "angular-mocks": "1.3.9" }, "resolutions": { - "angular": "1.3.3", + "angular": "1.3.9", "jquery": ">=1.9.0" } } diff --git a/public/js/static.js b/public/js/static.js index 8f88fb3551..0078b62243 100644 --- a/public/js/static.js +++ b/public/js/static.js @@ -7,8 +7,15 @@ window.habitrpg = angular.module('habitrpg', ['chieffancypants.loadingBar', 'ui. .constant("MOBILE_APP", false) .controller("RootCtrl", ['$scope', '$location', '$modal', '$http', function($scope, $location, $modal, $http){ - // must be #?memberId=xx, see https://github.com/angular/angular.js/issues/7239 var memberId = $location.search()['memberId']; + + // TODO Adding legacy support, remove this after 2/15/2015 or so + // angular@1.3.3 has bug, requires /#?memberId=x instead of ?memberId=x (https://github.com/angular/angular.js/issues/7239) + if (!memberId) { + memberId = /#\?memberId\=(.+)/.exec($location.$$absUrl); + if (memberId) memberId = memberId[memberId.length-1]; + } + if (memberId) { $http.get('/api/v2/members/'+memberId).success(function(data, status, headers, config){ $scope.profile = data; diff --git a/src/controllers/groups.js b/src/controllers/groups.js index 4c7b37c919..c3a93e047e 100644 --- a/src/controllers/groups.js +++ b/src/controllers/groups.js @@ -309,12 +309,12 @@ api.flagChatMessage = function(req, res, next){ {name: "REPORTER_USERNAME", content: user.profile.name}, {name: "REPORTER_UUID", content: user._id}, {name: "REPORTER_EMAIL", content: user.auth.local ? user.auth.local.email : ((user.auth.facebook && user.auth.facebook.emails && user.auth.facebook.emails[0]) ? user.auth.facebook.emails[0].value : null)}, - {name: "REPORTER_MODAL_URL", content: "https://habitrpg.com/static/front/#?memberId=" + user._id}, + {name: "REPORTER_MODAL_URL", content: "https://habitrpg.com/static/front?memberId=" + user._id}, {name: "AUTHOR_USERNAME", content: message.user}, {name: "AUTHOR_UUID", content: message.uuid}, {name: "AUTHOR_EMAIL", content: author.auth.local ? author.auth.local.email : ((author.auth.facebook && author.auth.facebook.emails && author.auth.facebook.emails[0]) ? author.auth.facebook.emails[0].value : null)}, - {name: "AUTHOR_MODAL_URL", content: "https://habitrpg.com/static/front/#?memberId=" + message.uuid}, + {name: "AUTHOR_MODAL_URL", content: "https://habitrpg.com/static/front?memberId=" + message.uuid}, {name: "GROUP_NAME", content: group.name}, {name: "GROUP_TYPE", content: group.type}, diff --git a/src/routes/apiv2.coffee b/src/routes/apiv2.coffee index 6c93e3ba42..39d4523a75 100644 --- a/src/routes/apiv2.coffee +++ b/src/routes/apiv2.coffee @@ -607,7 +607,7 @@ module.exports = (swagger, v2) -> path: '/members/{uuid}' description: "Get a member." parameters: [path('uuid','Member ID','string')] - middleware: [auth.auth, i18n.getUserLanguage] + middleware: [i18n.getUserLanguage] # removed auth.auth, so anon users can view shared avatars action: members.getMember "/members/{uuid}/message": spec: diff --git a/views/shared/header/avatar.jade b/views/shared/header/avatar.jade index 75692866ff..e338b19fcf 100644 --- a/views/shared/header/avatar.jade +++ b/views/shared/header/avatar.jade @@ -7,7 +7,7 @@ mixin avatar(opts) .character-sprites - .addthis_native_toolbox(ng-if='profile._id==user._id', data-url="#{env.BASE_URL}/static/front/#?memberId={{profile._id}}", data-title="Check out my HabitRPG progress!") + .addthis_native_toolbox(ng-if='profile._id==user._id', data-url="#{env.BASE_URL}/static/front?memberId={{profile._id}}", data-title="Check out my HabitRPG progress!") // Mount Body if !opts.minimal From a44e0f7c31dbb7f70c0a024a961ae10b7d5554e6 Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Sun, 18 Jan 2015 14:53:18 -0700 Subject: [PATCH 34/50] fix(avatar-sharing): screw it, 1.3.3 backwards compat isn't working at the router level anyway. Besides, avatar sharing hasn't worked via addthis for a while, so we don't need this --- public/js/static.js | 8 -------- 1 file changed, 8 deletions(-) diff --git a/public/js/static.js b/public/js/static.js index 0078b62243..3fdd1e6f2d 100644 --- a/public/js/static.js +++ b/public/js/static.js @@ -8,14 +8,6 @@ window.habitrpg = angular.module('habitrpg', ['chieffancypants.loadingBar', 'ui. .controller("RootCtrl", ['$scope', '$location', '$modal', '$http', function($scope, $location, $modal, $http){ var memberId = $location.search()['memberId']; - - // TODO Adding legacy support, remove this after 2/15/2015 or so - // angular@1.3.3 has bug, requires /#?memberId=x instead of ?memberId=x (https://github.com/angular/angular.js/issues/7239) - if (!memberId) { - memberId = /#\?memberId\=(.+)/.exec($location.$$absUrl); - if (memberId) memberId = memberId[memberId.length-1]; - } - if (memberId) { $http.get('/api/v2/members/'+memberId).success(function(data, status, headers, config){ $scope.profile = data; From 7c990ba10972af193888be0d1ba040c4a64faf82 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 01:18:37 -0500 Subject: [PATCH 35/50] changed Username to Login Name --- views/options/settings.jade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 3fb026d635..4678e16034 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -90,7 +90,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-body p(ng-if='user.auth.facebook.id') Registered with Facebook div(ng-if='user.auth.local.username') - p Username: {{user.auth.local.username}} + p Login Name: {{user.auth.local.username}} p Email: {{user.auth.local.email}} hr h5=env.t('changeUsername') From 330a9191a51c0e92ab06ffdacd5e236c5030a5ad Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 03:20:22 -0500 Subject: [PATCH 36/50] added new translatable string emailChange --- views/options/settings.jade | 2 ++ 1 file changed, 2 insertions(+) diff --git a/views/options/settings.jade b/views/options/settings.jade index 4678e16034..c4b5e6c5a8 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -92,6 +92,8 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') div(ng-if='user.auth.local.username') p Login Name: {{user.auth.local.username}} p Email: {{user.auth.local.email}} + small + =env.t('emailChange') hr h5=env.t('changeUsername') form(ng-submit='changeUsername(changeUser)', ng-show='user.auth.local') From c4113994e6582d15c351d8c91c9572da4a308b99 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 10:22:24 -0500 Subject: [PATCH 37/50] plain text to translatable strings --- views/options/settings.jade | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index c4b5e6c5a8..f2c23ac371 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -88,10 +88,10 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-heading span Registration .panel-body - p(ng-if='user.auth.facebook.id') Registered with Facebook + p(ng-if='user.auth.facebook.id') 'registeredWithFb' div(ng-if='user.auth.local.username') - p Login Name: {{user.auth.local.username}} - p Email: {{user.auth.local.email}} + p 'username': {{user.auth.local.username}} + p 'email': {{user.auth.local.email}} small =env.t('emailChange') hr From 5a4ebda18743b7f5ddba98a8d8013339cd2a5bd6 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 10:36:14 -0500 Subject: [PATCH 38/50] plain text to translatable strings --- views/options/settings.jade | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index f2c23ac371..40b555a584 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -88,10 +88,13 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-heading span Registration .panel-body - p(ng-if='user.auth.facebook.id') 'registeredWithFb' + p(ng-if='user.auth.facebook.id') + =env.t('registeredWithFb') div(ng-if='user.auth.local.username') - p 'username': {{user.auth.local.username}} - p 'email': {{user.auth.local.email}} + =env.t('username'): {{user.auth.local.username}} + |  + =env.t('email)': {{user.auth.local.email}} + |  small =env.t('emailChange') hr From 18317f334677a9a672777b8ac9771c1829dda638 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 13:18:00 -0500 Subject: [PATCH 39/50] fixed emailChange message, displays properly on localhost --- views/options/settings.jade | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 40b555a584..bf7d06b1c6 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -91,12 +91,19 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') p(ng-if='user.auth.facebook.id') =env.t('registeredWithFb') div(ng-if='user.auth.local.username') - =env.t('username'): {{user.auth.local.username}} - |  - =env.t('email)': {{user.auth.local.email}} - |  + p + =env.t('username') + |: {{user.auth.local.username}} + p + =env.t('email') + |: {{user.auth.local.email}} + p small - =env.t('emailChange') + =env.t('emailChange1') + |  + a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + |  + =env.t('emailChange3') hr h5=env.t('changeUsername') form(ng-submit='changeUsername(changeUser)', ng-show='user.auth.local') From ae90889decc199fdb4cde89ebe36c3685cd7d0b1 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 20:17:05 -0500 Subject: [PATCH 40/50] added description of display name and link to page to edit login name --- views/options/profile.jade | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/views/options/profile.jade b/views/options/profile.jade index 59c48b0d6e..b348d25ede 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -213,6 +213,15 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') button.btn.btn-default(ng-click='_editing.profile = true', ng-show='!_editing.profile')= env.t('edit') h4=env.t('displayName') span(ng-show='profile.profile.name') {{profile.profile.name}} + p + small + =env.t('displayNameDescription1') + |  + a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') + |  + =env.t('displayNameDescription3') + br + //span(ng-show='profile.profile.name') {{profile.profile.name}} span.muted(ng-hide='profile.profile.name') -  =env.t('none') |  - From b978852e1160c592c1961e76e7247513d98ad6f7 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Sun, 11 Jan 2015 20:30:25 -0500 Subject: [PATCH 41/50] added description of login name and link to page to edit display name --- views/options/settings.jade | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index bf7d06b1c6..809fbf3372 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -93,7 +93,15 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') div(ng-if='user.auth.local.username') p =env.t('username') - |: {{user.auth.local.username}} + |: {{user.auth.local.username}} + p + small + =env.t('loginNameDescription1') + |  + a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') + |  + =env.t('loginNameDescription3') + br p =env.t('email') |: {{user.auth.local.email}} From 1c151f57c2d5d6ddad6761b90ef8665f5a94cea3 Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Wed, 14 Jan 2015 22:38:02 -0500 Subject: [PATCH 42/50] applied .muted to text so less cluttered looking --- views/options/profile.jade | 14 ++++++-------- views/options/settings.jade | 25 ++++++++++++------------- 2 files changed, 18 insertions(+), 21 deletions(-) diff --git a/views/options/profile.jade b/views/options/profile.jade index b348d25ede..8f91880cbe 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -214,14 +214,12 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') h4=env.t('displayName') span(ng-show='profile.profile.name') {{profile.profile.name}} p - small - =env.t('displayNameDescription1') - |  - a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') - |  - =env.t('displayNameDescription3') - br - //span(ng-show='profile.profile.name') {{profile.profile.name}} + small.muted + =env.t('displayNameDescription1') + |  + a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') + |  + =env.t('displayNameDescription3') span.muted(ng-hide='profile.profile.name') -  =env.t('none') |  - diff --git a/views/options/settings.jade b/views/options/settings.jade index 809fbf3372..4457c68ab7 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -95,23 +95,22 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') =env.t('username') |: {{user.auth.local.username}} p - small - =env.t('loginNameDescription1') - |  - a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') - |  - =env.t('loginNameDescription3') - br + small.muted + =env.t('loginNameDescription1') + |  + a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') + |  + =env.t('loginNameDescription3') p =env.t('email') |: {{user.auth.local.email}} p - small - =env.t('emailChange1') - |  - a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') - |  - =env.t('emailChange3') + small.muted + =env.t('emailChange1') + |  + a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + |  + =env.t('emailChange3') hr h5=env.t('changeUsername') form(ng-submit='changeUsername(changeUser)', ng-show='user.auth.local') From b3c9371d9f0de9e8fad6543c39f86106c8f7fe0d Mon Sep 17 00:00:00 2001 From: scribblypanda Date: Thu, 15 Jan 2015 08:23:23 -0500 Subject: [PATCH 43/50] changed links to relative, corrected mailto link --- views/options/profile.jade | 2 +- views/options/settings.jade | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/views/options/profile.jade b/views/options/profile.jade index 8f91880cbe..5f6ed31b04 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -217,7 +217,7 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') small.muted =env.t('displayNameDescription1') |  - a(href='https://habitrpg.com/#/options/settings/settings')=env.t('displayNameDescription2') + a(href='/#/options/settings/settings')=env.t('displayNameDescription2') |  =env.t('displayNameDescription3') span.muted(ng-hide='profile.profile.name') -  diff --git a/views/options/settings.jade b/views/options/settings.jade index 4457c68ab7..2a5fee087c 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -98,7 +98,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') small.muted =env.t('loginNameDescription1') |  - a(href='https://habitrpg.com/#/options/profile/profile')=env.t('loginNameDescription2') + a(href='/#/options/profile/profile')=env.t('loginNameDescription2') |  =env.t('loginNameDescription3') p @@ -108,7 +108,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') small.muted =env.t('emailChange1') |  - a(href='emailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + a(href='mailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') |  =env.t('emailChange3') hr From 4a5fe1867d49dd60a5b8fef6bcfe18c9e285a6eb Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:33:58 -0600 Subject: [PATCH 44/50] Removed target="_blank" attribute from mailto link --- views/options/settings.jade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 2a5fee087c..fd9ae4cc0c 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -108,7 +108,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') small.muted =env.t('emailChange1') |  - a(href='mailto:admin@habitrpg.com' target='_blank')=env.t('emailChange2') + a(href='mailto:admin@habitrpg.com')=env.t('emailChange2') |  =env.t('emailChange3') hr From cdad6e36da1b6263c6ad837ba4929ae21c4661ef Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:38:16 -0600 Subject: [PATCH 45/50] Adjusted some spacing and line breaks --- views/options/settings.jade | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index fd9ae4cc0c..7f07c2d01e 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -88,12 +88,10 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') .panel-heading span Registration .panel-body - p(ng-if='user.auth.facebook.id') - =env.t('registeredWithFb') + p(ng-if='user.auth.facebook.id')=env.t('registeredWithFb') div(ng-if='user.auth.local.username') - p - =env.t('username') - |: {{user.auth.local.username}} + p=env.t('username') + |: {{user.auth.local.username}} p small.muted =env.t('loginNameDescription1') @@ -101,9 +99,8 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') a(href='/#/options/profile/profile')=env.t('loginNameDescription2') |  =env.t('loginNameDescription3') - p - =env.t('email') - |: {{user.auth.local.email}} + p=env.t('email') + |: {{user.auth.local.email}} p small.muted =env.t('emailChange1') From e0a081b0ff26a277e5cb7c7d17fcaf124cbf517d Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:39:45 -0600 Subject: [PATCH 46/50] Adjusted spacing and line breaks --- views/options/profile.jade | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/views/options/profile.jade b/views/options/profile.jade index 5f6ed31b04..19a2ba48d1 100644 --- a/views/options/profile.jade +++ b/views/options/profile.jade @@ -214,12 +214,11 @@ script(id='partials/options.profile.profile.html', type='text/ng-template') h4=env.t('displayName') span(ng-show='profile.profile.name') {{profile.profile.name}} p - small.muted - =env.t('displayNameDescription1') - |  - a(href='/#/options/settings/settings')=env.t('displayNameDescription2') - |  - =env.t('displayNameDescription3') + small.muted=env.t('displayNameDescription1') + |  + a(href='/#/options/settings/settings')=env.t('displayNameDescription2') + |  + =env.t('displayNameDescription3') span.muted(ng-hide='profile.profile.name') -  =env.t('none') |  - From 13e89f19475e21e4f518bdb2b1f5e25b5c4cffec Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:50:43 -0600 Subject: [PATCH 47/50] Added sync after changing username so username would update --- public/js/controllers/settingsCtrl.js | 1 + 1 file changed, 1 insertion(+) diff --git a/public/js/controllers/settingsCtrl.js b/public/js/controllers/settingsCtrl.js index 7220791520..599e29664c 100644 --- a/public/js/controllers/settingsCtrl.js +++ b/public/js/controllers/settingsCtrl.js @@ -85,6 +85,7 @@ habitrpg.controller('SettingsCtrl', .success(function(){ alert(window.env.t('usernameSuccess')); $scope.changeUser = {}; + User.sync(); }) .error(function(data){ alert(data.err); From 137ff71bc837879e9810d617e02ba071d528a0b6 Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 15:51:00 -0600 Subject: [PATCH 48/50] Added one way binding for email --- views/options/settings.jade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/views/options/settings.jade b/views/options/settings.jade index 7f07c2d01e..b2437f81cf 100644 --- a/views/options/settings.jade +++ b/views/options/settings.jade @@ -100,7 +100,7 @@ script(type='text/ng-template', id='partials/options.settings.settings.html') |  =env.t('loginNameDescription3') p=env.t('email') - |: {{user.auth.local.email}} + |: {{::user.auth.local.email}} p small.muted =env.t('emailChange1') From bb78b121da585365e86337d4376e0e0d8faed481 Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Sun, 18 Jan 2015 16:38:31 -0700 Subject: [PATCH 49/50] revert part of a053486, angular is madeningly inconsistent with this bug --- src/controllers/groups.js | 4 ++-- views/shared/header/avatar.jade | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/controllers/groups.js b/src/controllers/groups.js index c3a93e047e..4c7b37c919 100644 --- a/src/controllers/groups.js +++ b/src/controllers/groups.js @@ -309,12 +309,12 @@ api.flagChatMessage = function(req, res, next){ {name: "REPORTER_USERNAME", content: user.profile.name}, {name: "REPORTER_UUID", content: user._id}, {name: "REPORTER_EMAIL", content: user.auth.local ? user.auth.local.email : ((user.auth.facebook && user.auth.facebook.emails && user.auth.facebook.emails[0]) ? user.auth.facebook.emails[0].value : null)}, - {name: "REPORTER_MODAL_URL", content: "https://habitrpg.com/static/front?memberId=" + user._id}, + {name: "REPORTER_MODAL_URL", content: "https://habitrpg.com/static/front/#?memberId=" + user._id}, {name: "AUTHOR_USERNAME", content: message.user}, {name: "AUTHOR_UUID", content: message.uuid}, {name: "AUTHOR_EMAIL", content: author.auth.local ? author.auth.local.email : ((author.auth.facebook && author.auth.facebook.emails && author.auth.facebook.emails[0]) ? author.auth.facebook.emails[0].value : null)}, - {name: "AUTHOR_MODAL_URL", content: "https://habitrpg.com/static/front?memberId=" + message.uuid}, + {name: "AUTHOR_MODAL_URL", content: "https://habitrpg.com/static/front/#?memberId=" + message.uuid}, {name: "GROUP_NAME", content: group.name}, {name: "GROUP_TYPE", content: group.type}, diff --git a/views/shared/header/avatar.jade b/views/shared/header/avatar.jade index e338b19fcf..75692866ff 100644 --- a/views/shared/header/avatar.jade +++ b/views/shared/header/avatar.jade @@ -7,7 +7,7 @@ mixin avatar(opts) .character-sprites - .addthis_native_toolbox(ng-if='profile._id==user._id', data-url="#{env.BASE_URL}/static/front?memberId={{profile._id}}", data-title="Check out my HabitRPG progress!") + .addthis_native_toolbox(ng-if='profile._id==user._id', data-url="#{env.BASE_URL}/static/front/#?memberId={{profile._id}}", data-title="Check out my HabitRPG progress!") // Mount Body if !opts.minimal From b73518af5fea9ca95dd18e2fc4fac30555c57669 Mon Sep 17 00:00:00 2001 From: Blade Barringer Date: Sun, 18 Jan 2015 22:54:07 -0600 Subject: [PATCH 50/50] Removed $root from view --- views/shared/tasks/task.jade | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/views/shared/tasks/task.jade b/views/shared/tasks/task.jade index 3362e1249b..95fed6b8f1 100644 --- a/views/shared/tasks/task.jade +++ b/views/shared/tasks/task.jade @@ -63,8 +63,8 @@ li(bindonce='list', id='task-{{::task.id}}', ng-repeat='task in obj[list.type+"s // Habits span(ng-if='::task.type=="habit"') // score() is overridden in challengesCtrl to do nothing - a.task-action-btn(ng-if='task.up', ng-click='$root.applyingAction || score(task,"up")') + - a.task-action-btn(ng-if='task.down', ng-click='$root.applyingAction || score(task,"down")') - + a.task-action-btn(ng-if='task.up', ng-click='applyingAction || score(task,"up")') + + a.task-action-btn(ng-if='task.down', ng-click='applyingAction || score(task,"down")') - // Rewards span(ng-show='task.type=="reward"')