Do some cleanups on the data

This commit is contained in:
Evert Prants 2017-11-23 18:26:53 +02:00
parent 8cb48abb39
commit 0d792817fe
Signed by: evert
GPG Key ID: 1688DA83D222D0B5

View File

@ -62,6 +62,17 @@ const API = {
uuid: uuidV1() uuid: uuidV1()
}, data) }, data)
// Some data cleanups
// Limit display name length
udataLimited.display_name = udataLimited.display_name.substring(0, 32)
// Remove illegal characters from the username
udataLimited.username = udataLimited.username.replace(/\W+/gi, '')
// Limit user name length
udataLimited.username = udataLimited.username.substring(0, 26)
// Check if the username is already taken // Check if the username is already taken
if (await UAPI.User.get(udataLimited.username) != null) { if (await UAPI.User.get(udataLimited.username) != null) {
udataLimited.username = udataLimited.username + UAPI.Hash(4) udataLimited.username = udataLimited.username + UAPI.Hash(4)
@ -77,7 +88,7 @@ const API = {
} }
} }
// Create a new user based on the information we got from Facebook // Create a new user based on the information we got from an external service
let newUser = await models.User.query().insert(udataLimited) let newUser = await models.User.query().insert(udataLimited)
await API.Common.new(service, identifier, newUser) await API.Common.new(service, identifier, newUser)
@ -317,7 +328,7 @@ const API = {
} }
let cleanedData = Object.assign(data, { let cleanedData = Object.assign(data, {
username: data.name.replace(/\W+/gi, ''), username: data.name,
display_name: data.name, display_name: data.name,
email: data.email || '' email: data.email || ''
}) })
@ -404,7 +415,6 @@ const API = {
// Create a new user // Create a new user
let cleanedData = Object.assign(ddata, { let cleanedData = Object.assign(ddata, {
username: ddata.username.replace(/\W+/gi, '_'),
display_name: ddata.username, display_name: ddata.username,
email: ddata.email || '' email: ddata.email || ''
}) })