add users.avatar_url constraint and default (#1141)

maproulette · Aug 13, 2024 · 2a2e18c · 2a2e18c
1 parent 208f358
commit 2a2e18c
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 6 deletions.
diff --git a/app/org/maproulette/framework/repository/LeaderboardRepository.scala b/app/org/maproulette/framework/repository/LeaderboardRepository.scala
@@ -44,7 +44,7 @@ class LeaderboardRepository @Inject() (override val db: Database) extends Reposi
   ): RowParser[LeaderboardUser] = {
     get[Long]("user_id") ~
       get[String]("user_name") ~
-      get[String]("user_avatar_url").? ~
+      get[String]("user_avatar_url") ~
       get[Int]("user_score") ~
       get[Int]("user_ranking") ~
       get[DateTime]("created").? ~
@@ -61,7 +61,7 @@ class LeaderboardRepository @Inject() (override val db: Database) extends Reposi
         new LeaderboardUser(
           userId,
           name,
-          avatarURL.getOrElse(""),
+          avatarURL,
           score,
           rank,
           completedTasks,

diff --git a/app/org/maproulette/framework/repository/UserRepository.scala b/app/org/maproulette/framework/repository/UserRepository.scala
@@ -466,10 +466,10 @@ object UserRepository {
       get[Long]("users.id") ~
       get[Long]("users.osm_id") ~
       get[String]("users.name") ~
-      get[Option[String]]("users.avatar_url") ~
+      get[String]("users.avatar_url") ~
       get[List[Int]]("roles") map {
       case projectId ~ userId ~ osmId ~ displayName ~ avatarURL ~ roles =>
-        ProjectManager(projectId, userId, osmId, displayName, avatarURL.getOrElse(""), roles)
+        ProjectManager(projectId, userId, osmId, displayName, avatarURL, roles)
     }
   }
 
@@ -493,7 +493,7 @@ object UserRepository {
       get[DateTime]("users.osm_created") ~
       get[String]("users.name") ~
       get[Option[String]]("users.description") ~
-      get[Option[String]]("users.avatar_url") ~
+      get[String]("users.avatar_url") ~
       get[Option[String]]("home") ~
       get[Option[String]]("users.api_key") ~
       get[String]("users.oauth_token") ~
@@ -544,7 +544,7 @@ object UserRepository {
             osmId,
             displayName,
             description.getOrElse(""),
-            avatarURL.getOrElse(""),
+            avatarURL,
             Location(locationWKT.getX, locationWKT.getY),
             osmCreated,
             oauthToken

diff --git a/conf/evolutions/default/95.sql b/conf/evolutions/default/95.sql
@@ -0,0 +1,16 @@
+-- --- !Ups
+-- Update any existing null avatar_url values to the default
+UPDATE users
+SET avatar_url = '/assets/images/user_no_image.png'
+WHERE avatar_url IS NULL;
+
+-- Alter the table to set avatar_url as NOT NULL with a default value
+ALTER TABLE IF EXISTS users
+ALTER COLUMN avatar_url SET DEFAULT '/assets/images/user_no_image.png',
+ALTER COLUMN avatar_url SET NOT NULL;
+
+-- --- !Downs
+-- Revert the avatar_url column to allow null values and remove the default value
+ALTER TABLE IF EXISTS users
+ALTER COLUMN avatar_url DROP NOT NULL,
+ALTER COLUMN avatar_url DROP DEFAULT;