remove ability to set custom names

Author: lieuwex

client/app/settings/pages/accountInfo/accountInfo.coffee

@@ -8,21 +8,13 @@ Template['settings_page_accountInfo'].events
 	'submit form': (event) ->
 		event.preventDefault()
 
-		firstName = Helpers.nameCap $('#firstNameInput').val()
-		lastName = Helpers.nameCap $('#lastNameInput').val()
-
 		oldPass = $('#currentPassInput').val()
 
 		mail = $('#mailInput').val().toLowerCase()
 
 		newPass = $('#newPassInput').val()
 		newPassRepeat = $('#newPassRepeatInput').val()
 
-		nameChanged = (
-			profile = getUserField Meteor.userId(), 'profile'
-			profile.firstName isnt firstName or profile.lastName isnt lastName
-		)
-
 		mailChanged = mail isnt getUserField Meteor.userId(), 'emails[0].address'
 		passChanged = newPass isnt '' or newPassRepeat isnt ''
 		needsPass = mailChanged or passChanged
@@ -61,9 +53,6 @@ Template['settings_page_accountInfo'].events
 				else
 					callback not e?
 
-		if nameChanged
-			Meteor.call 'changeName', firstName, lastName, (e) -> callback not e?
-
 		if passChanged
 			unless newPass is newPassRepeat
 				setFieldError (

client/app/settings/pages/accountInfo/accountInfo.html

@@ -1,14 +1,6 @@
 <template name="settings_page_accountInfo">
 	<div id="accountInfoSettings">
 		<form class="vertical full-width">
-			<div id="firstNameGroup" class="group">
-				<label for="firstNameInput">Voornaam</label>
-				<input type="text" id="firstNameInput" placeholder="Henk" value="{{currentUser.profile.firstName}}">
-			</div>
-			<div id="lastNameGroup" class="group">
-				<label for="lastNameInput">Achternaam</label>
-				<input type="text" id="lastNameInput" placeholder="Jansen" value="{{currentUser.profile.lastName}}">
-			</div>
 			<div id="currentPassGroup" class="group">
 				<label for="currentPassInput">Huidig wachtwoord</label>
 				<input type="password" id="currentPassInput" placeholder="●●●●●●●●●●">

client/app/setup/setup.coffee

@@ -7,6 +7,7 @@ SReactiveVar = require('meteor/simply:strict-reactive-var').default
 
 currentSelectedImage      = new SReactiveVar Number, 0
 currentSelectedCourseInfo = new SReactiveVar Number, 0
+currentSelectedName       = new SReactiveVar Number, 0
 
 weekdays = new SReactiveVar [Object]
 
@@ -54,10 +55,21 @@ courseInfos = ->
 		.value()
 
 names = ->
-	for service in externalServices.get()
-		val = service.profileData()?.nameInfo
-		return val if val?
-	undefined
+	current = currentSelectedName.get()
+
+	_(externalServices.get())
+		.map (s) -> s.profileData()?.nameInfo
+		.reject _.isEmpty
+		.uniq (info) -> "#{info.firstName} #{info.lastName}"
+		.map (c, i) ->
+			isSelected: ->
+				if current is i
+					'selected'
+				else
+					''
+			value: c
+			index: i
+		.value()
 
 addProgress = (item, cb) ->
 	ga 'send', 'event', 'setup', 'progress', item
@@ -97,11 +109,9 @@ class @Setup
 			onDone: (cb) ->
 				schoolQuery = $('#setup #schoolInput').val()
 
-				$firstNameInput = $ '#setup #firstNameInput'
-				$lastNameInput = $ '#setup #lastNameInput'
 				any = no
-				any = yes if empty($firstNameInput, '#firstNameGroup', 'Voornaam is leeg')
-				any = yes if empty($lastNameInput, '#lastNameGroup', 'Achternaam is leeg')
+
+				name = names()[currentSelectedName.get()]?.value
 
 				courseInfo = courseInfos()[currentSelectedCourseInfo.get()]?.value
 				unless courseInfo?
@@ -138,8 +148,8 @@ class @Setup
 								fetchedBy: val.fetchedBy
 						)
 						'profile.courseInfo': courseInfo
-						'profile.firstName': Helpers.nameCap $firstNameInput.val()
-						'profile.lastName': Helpers.nameCap $lastNameInput.val()
+						'profile.firstName': name.firstName
+						'profile.lastName': name.lastName
 						'profile.birthDate':
 							# Picks the date from the first externalService that has one.
 							# REVIEW: Maybe we should ask the user too?
@@ -326,12 +336,13 @@ Template.setupFooter.helpers
 Template.setupFooter.events
 	'click button': -> setup.finishStep()
 
+# === step template stuff
+
 Template['setup-extractInfo'].helpers
 	pictures: pictures
 	hasSchool: -> schoolId?
 	courseInfos: courseInfos
-	firstName: -> names()?.firstName
-	lastName: -> names()?.lastName
+	names: names
 
 Template['setup-extractInfo'].events
 	'click #pictureSelector > img': (event) ->
@@ -340,6 +351,9 @@ Template['setup-extractInfo'].events
 	'click #courseInfoSelector > div': (event) ->
 		currentSelectedCourseInfo.set @index
 
+	'click #nameSelector > div': (event) ->
+		currentSelectedName.set @index
+
 Template['setup-extractInfo'].onRendered ->
 	unless schoolId?
 		engine = new Bloodhound

client/app/setup/setup.html

@@ -122,17 +122,15 @@ <h2>Profielinfo</h2>
 		<hr>
 
 		<h2>Naam</h2>
-		<form class="vertical full-width">
-			<div id="firstNameGroup" class="group">
-				<label for="firstNameInput">Voornaam</label>
-				<input type="text" id="firstNameInput" value="{{firstName}}"/>
-			</div>
-			<div id="lastNameGroup" class="group">
-				<label for="lastNameInput">Achternaam</label>
-				<input type="text" id="lastNameInput" value="{{lastName}}"/>
+		{{#with names}}
+			<div id="nameSelector" class="selector">
+				{{#each this}}
+					<div class="{{isSelected}}">
+						{{value.firstName}} {{value.lastName}}
+					</div>
+				{{/each}}
 			</div>
-		</form>
-
+		{{/with}}
 	</div>
 </template>
 

lib/methods.coffee

@@ -117,26 +117,6 @@ Meteor.methods
 			$set: isRead: yes
 		}
 
-	###*
-	# @method changeName
-	# @param {String} firstName non empty string.
-	# @param {String} lastName non empty string.
-	###
-	changeName: (firstName, lastName) ->
-		check firstName, String
-		check lastName, String
-
-		firstName = Helpers.nameCap firstName.trim()
-		lastName = Helpers.nameCap lastName.trim()
-
-		if firstName.length is 0 or lastName.length is 0
-			throw new Meteor.Error 'name-empty'
-
-		Meteor.users.update @userId,
-			$set:
-				'profile.firstName': firstName
-				'profile.lastName': lastName
-
 	###*
 	# @method saveMessageDraft
 	# @param {Draft} draft