Accept suggested improvement

Co-authored-by: Andrew Nowak <[email protected]>
guardian · Dec 2, 2024 · 851926c · 851926c
1 parent d294f9e
commit 851926c
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/cropper/app/lib/CropSpecMetadata.scala b/cropper/app/lib/CropSpecMetadata.scala
@@ -19,15 +19,12 @@ trait CropSpecMetadata {
       "height" -> dimensions.height,
       "aspect-ratio" -> r)
 
-    val nonEmptyMetadata = metadata.filter {
-      case (_, None) => false
-      case _ => true
+    val flattenedMetadata = metadata.collect {
+      case (key, Some(value))            => key -> value.toString
+      case (key, value) if value != None => key -> value.toString
     }
 
-    val flattenedMetadata = nonEmptyMetadata.collect {
-      case (key, Some(value)) => key -> value
-      case (key, value) => key -> value
-    }.view.mapValues(_.toString).toMap
+    flattenedMetadata
 
     flattenedMetadata
   }