Make sure user ids are integers

developmentseed · Dec 23, 2022 · b54cbea · b54cbea
1 parent 2dde70b
commit b54cbea
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/models/team.js b/src/models/team.js
@@ -66,10 +66,13 @@ const teamAttributes = [
  *
  */
 async function resolveMemberNames(ids) {
+  // TODO Quick fix, we need to do proper type validation
+  const userIds = ids.map((i) => parseInt(i))
+
   // get the display names from the database table first
-  const foundUsers = await db('osm_users').whereIn('id', ids)
+  const foundUsers = await db('osm_users').whereIn('id', userIds)
   const foundUserIds = foundUsers.map(prop('id'))
-  const notFound = difference(ids, foundUserIds)
+  const notFound = difference(userIds, foundUserIds)
 
   let usersFromOSM = []
   if (notFound.length > 0) {