krikkert/habitica
1
0
Fork 0
mirror of https://github.com/HabitRPG/habitica.git synced 2025-12-18 15:17:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

[#1532] add better validation on preferences.dayStart, and a migration to

Browse Source 
cleanup corrupt ones
This commit is contained in:
Tyler Renelle
2013-11-26 15:18:20 -07:00
parent a4fb0b42ad
commit 2b128f71f4
3 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
migrations/20131126_clean_dayStart.js Normal file
View File

@@ -0,0 +1,5 @@
db.users.find({'preferences.dayStart':{$exists:1}},{'preferences.dayStart':1}).forEach(function(user){
var dayStart = +user.preferences.dayStart;
dayStart = (_.isNaN(dayStart) || dayStart < 0 || dayStart > 24) ? 0 : dayStart;
db.users.update({_id:user._id}, {$set:{'preferences.dayStart':dayStart}});
});

5
public/js/controllers/settingsCtrl.js
View File

@@ -27,10 +27,11 @@ habitrpg.controller('SettingsCtrl',
$scope.saveDayStart = function(){
var dayStart = +User.user.preferences.dayStart;
if (dayStart < 0 || dayStart > 24) {
if (_.isNaN(dayStart) || dayStart < 0 || dayStart > 24) {
dayStart = 0;
return alert('Please enter a number between 0 and 24');
}
User.log({'op':'set', data:{'preferences.dayStart': dayStart}});
User.set('preferences.dayStart', dayStart);
}
$scope.language = window.env.language;

4
src/models/user.js
View File

@@ -259,6 +259,10 @@ UserSchema.virtual('tasks').get(function () {
UserSchema.pre('save', function(next) {
//this.markModified('tasks');
if (_.isNaN(this.preferences.dayStart) || this.preferences.dayStart < 0 || this.preferences.dayStart > 0) {
this.preferences.dayStart = 0;
}
if (!this.profile.name) {
var fb = this.auth.facebook;
this.profile.name =