Use user prop if author is empty or null

Signed-off-by: parneet-guraya <[email protected]>

Author: parneet-guraya

app/src/main/java/fr/free/nrw/commons/Media.kt

@@ -125,6 +125,19 @@ class Media constructor(
         categoriesHiddenStatus = categoriesHiddenStatus
     )
 
+    /**
+     * Returns Author if it's not null or empty, otherwise
+     * returns user
+     * @return Author or User
+     */
+    fun getAuthorOrUser(): String? {
+        return if (!author.isNullOrEmpty()) {
+            author
+        } else{
+            user
+        }
+    }
+
     /**
      * Gets media display title
      * @return Media title

app/src/main/java/fr/free/nrw/commons/category/GridViewAdapter.kt

@@ -98,14 +98,9 @@ class GridViewAdapter(
      */
     @SuppressLint("StringFormatInvalid")
     private fun setUploaderView(item: Media, uploader: TextView) {
-        if (!item.author.isNullOrEmpty()) {
-            uploader.visibility = View.VISIBLE
-            uploader.text = context.getString(
-                R.string.image_uploaded_by,
-                item.user
-            )
-        } else {
-            uploader.visibility = View.GONE
-        }
+        uploader.text = context.getString(
+            R.string.image_uploaded_by,
+            item.getAuthorOrUser()
+        )
     }
 }

app/src/main/java/fr/free/nrw/commons/contributions/ContributionViewHolder.kt

@@ -54,7 +54,7 @@ an upload might take a dozen seconds. */
         this.contribution = contribution
         this.position = position
         binding.contributionTitle.text = contribution.media.mostRelevantCaption
-        binding.authorView.text = contribution.media.author
+        binding.authorView.text = contribution.media.getAuthorOrUser()
 
         //Removes flicker of loading image.
         binding.contributionImage.hierarchy.fadeDuration = 0

app/src/main/java/fr/free/nrw/commons/delete/DeleteHelper.kt

@@ -111,7 +111,7 @@ class DeleteHelper @Inject constructor(
 
         val userPageString = "\n{{subst:idw|${media.filename}}} ~~~~"
 
-        val creator = media.author
+        val creator = media.getAuthorOrUser()
             ?: throw RuntimeException("Failed to nominate for deletion")
 
         return pageEditClient.prependEdit(

app/src/main/java/fr/free/nrw/commons/explore/media/PagedMediaAdapter.kt

@@ -52,12 +52,7 @@ class SearchImagesViewHolder(
         binding.categoryImageView.setOnClickListener { onImageClicked(item.second) }
         binding.categoryImageTitle.text = media.mostRelevantCaption
         binding.categoryImageView.setImageURI(media.thumbUrl)
-        if (media.author?.isNotEmpty() == true) {
-            binding.categoryImageAuthor.visibility = View.VISIBLE
-            binding.categoryImageAuthor.text =
-                containerView.context.getString(R.string.image_uploaded_by, media.author)
-        } else {
-            binding.categoryImageAuthor.visibility = View.GONE
-        }
+        binding.categoryImageAuthor.text =
+            containerView.context.getString(R.string.image_uploaded_by, media.getAuthorOrUser())
     }
 }

app/src/main/java/fr/free/nrw/commons/media/MediaDetailPagerFragment.java

@@ -328,7 +328,7 @@ private String getTechInfo(final Media media, final String type) {
             .append("\n\n");
 
         builder.append("User that you want to report: ")
-            .append(media.getAuthor())
+            .append(media.getUser())
             .append("\n\n");
 
         if (sessionManager.getUserName() != null) {
@@ -423,7 +423,7 @@ public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
                     // Initialize bookmark object
                     bookmark = new Bookmark(
                             m.getFilename(),
-                            m.getAuthor(),
+                            m.getAuthorOrUser(),
                             BookmarkPicturesContentProvider.uriForName(m.getFilename())
                     );
                     updateBookmarkState(menu.findItem(R.id.menu_bookmark_current_image));