refactor: accept List in metadata's constructors

There's no need to force API consumers to use `ArrayList` - any `List` will be just fine.
Parsely · Dec 12, 2023 · fa379a9 · fa379a9
1 parent baa1400
commit fa379a9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/parsely/src/main/java/com/parsely/parselyandroid/ParselyMetadata.kt b/parsely/src/main/java/com/parsely/parselyandroid/ParselyMetadata.kt
@@ -21,10 +21,10 @@ open class ParselyMetadata
  * @param title    The title of the content.
  * @param publicationDateMilliseconds  The date this piece of content was published.
  */(
-    private val authors: ArrayList<String>?,
+    private val authors: List<String>?,
     @JvmField internal val link: String?,
     private val section: String?,
-    private val tags: ArrayList<String>?,
+    private val tags: List<String>?,
     private val thumbUrl: String?,
     private val title: String?,
     private val publicationDateMilliseconds: Long

diff --git a/parsely/src/main/java/com/parsely/parselyandroid/ParselyVideoMetadata.kt b/parsely/src/main/java/com/parsely/parselyandroid/ParselyVideoMetadata.kt
@@ -16,10 +16,10 @@ class ParselyVideoMetadata
  * @param publicationDateMilliseconds         The timestamp in milliseconds this video was published.
  * @param durationSeconds Duration of the video in seconds. Required.
  */(
-    authors: ArrayList<String>?,
+    authors: List<String>?,
     videoId: String,
     section: String?,
-    tags: ArrayList<String>?,
+    tags: List<String>?,
     thumbUrl: String?,
     title: String?,
     publicationDateMilliseconds: Long,