fixed errors and warnings

app/src/main/java/fr/free/nrw/commons/upload/FailedUploadsFragment.kt

@@ -17,19 +17,16 @@ import fr.free.nrw.commons.profile.ProfileActivity
 import fr.free.nrw.commons.utils.DialogUtil
 import fr.free.nrw.commons.utils.ViewUtil
 import org.apache.commons.lang3.StringUtils
-import java.util.Locale
 import javax.inject.Inject
 
 /**
- * Fragment for displaying a list of failed uploads in Upload Progress Activity. This fragment provides
+ * Fragment for displaying a list of failed uploads in Upload Progress Activity. It provides
  * functionality for the user to retry or cancel failed uploads.
  */
 class FailedUploadsFragment :
     CommonsDaggerSupportFragment(),
     PendingUploadsContract.View,
     FailedUploadsAdapter.Callback {
-    @Inject
-    lateinit var pendingUploadsPresenter: PendingUploadsPresenter
 
     @Inject
     lateinit var mediaClient: MediaClient
@@ -38,14 +35,11 @@ class FailedUploadsFragment :
     lateinit var sessionManager: SessionManager
 
     private var userName: String? = null
-
+    private lateinit var binding: FragmentFailedUploadsBinding
-    lateinit var binding: FragmentFailedUploadsBinding
-
     private lateinit var adapter: FailedUploadsAdapter
-
+    private val contributionsList = ArrayList<Contribution>()
-    var contributionsList = ArrayList<Contribution>()
-
     private lateinit var uploadProgressActivity: UploadProgressActivity
+    lateinit var pendingUploadsPresenter: PendingUploadsPresenter
 
     override fun onAttach(context: Context) {
         super.onAttach(context)
@@ -56,12 +50,8 @@ class FailedUploadsFragment :
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
-        // Now that we are allowing this fragment to be started for
+        // Expecting userName as an argument; fallback to sessionManager's userName if not provided
-        // any userName- we expect it to be passed as an argument
+        userName = arguments?.getString(ProfileActivity.KEY_USERNAME)
-        if (arguments != null) {
-            userName = requireArguments().getString(ProfileActivity.KEY_USERNAME)
-        }
-
         if (StringUtils.isEmpty(userName)) {
             userName = sessionManager.userName
         }
@@ -71,14 +61,14 @@ class FailedUploadsFragment :
         inflater: LayoutInflater,
         container: ViewGroup?,
         savedInstanceState: Bundle?,
-    ): View? {
+    ): View {
-        binding = FragmentFailedUploadsBinding.inflate(layoutInflater)
+        binding = FragmentFailedUploadsBinding.inflate(inflater, container, false)
         pendingUploadsPresenter.onAttachView(this)
         initAdapter()
         return binding.root
     }
 
-    fun initAdapter() {
+    private fun initAdapter() {
         adapter = FailedUploadsAdapter(this)
     }
 
@@ -93,21 +83,19 @@ class FailedUploadsFragment :
     /**
      * Initializes the recycler view.
      */
-    fun initRecyclerView() {
+    private fun initRecyclerView() {
-        binding.failedUploadsRecyclerView.setLayoutManager(LinearLayoutManager(this.context))
+        binding.failedUploadsRecyclerView.layoutManager = LinearLayoutManager(context)
         binding.failedUploadsRecyclerView.adapter = adapter
         pendingUploadsPresenter.getFailedContributions()
         pendingUploadsPresenter.failedContributionList.observe(
             viewLifecycleOwner,
         ) { list: PagedList<Contribution?> ->
             adapter.submitList(list)
-            contributionsList = ArrayList()
+            contributionsList.clear()
-            list.forEach {
+            list.forEach { contribution ->
-                if (it != null) {
+                contribution?.let { contributionsList.add(it) }
-                    contributionsList.add(it)
             }
-            }
+            if (list.isEmpty()) {
-            if (list.size == 0) {
                 uploadProgressActivity.setErrorIconsVisibility(false)
                 binding.nofailedTextView.visibility = View.VISIBLE
                 binding.failedUplaodsLl.visibility = View.GONE
@@ -115,7 +103,7 @@ class FailedUploadsFragment :
                 uploadProgressActivity.setErrorIconsVisibility(true)
                 binding.nofailedTextView.visibility = View.GONE
                 binding.failedUplaodsLl.visibility = View.VISIBLE
-                binding.failedUploadsRecyclerView.setAdapter(adapter)
+                // Setting the adapter again is redundant; it's already set above
             }
         }
     }
@@ -124,27 +112,23 @@ class FailedUploadsFragment :
      * Restarts all the failed uploads.
      */
     fun restartUploads() {
-        if (contributionsList != null) {
         pendingUploadsPresenter.restartUploads(
             contributionsList,
             0,
-                this.requireContext().applicationContext,
+            requireContext().applicationContext,
         )
     }
-    }
 
     /**
      * Restarts a specific upload.
      */
     override fun restartUpload(index: Int) {
-        if (contributionsList != null) {
         pendingUploadsPresenter.restartUpload(
             contributionsList,
             index,
-                this.requireContext().applicationContext,
+            requireContext().applicationContext,
         )
     }
-    }
 
     /**
      * Deletes a specific upload after getting a confirmation from the user using Dialog.
@@ -152,24 +136,18 @@ class FailedUploadsFragment :
     override fun deleteUpload(contribution: Contribution?) {
         DialogUtil.showAlertDialog(
             requireActivity(),
-            String.format(
+            getString(R.string.cancelling_upload),
-                Locale.getDefault(),
+            getString(R.string.cancel_upload_dialog),
-                requireActivity().getString(R.string.cancelling_upload),
+            getString(R.string.yes),
-            ),
+            getString(R.string.no),
-            String.format(
-                Locale.getDefault(),
-                requireActivity().getString(R.string.cancel_upload_dialog),
-            ),
-            String.format(Locale.getDefault(), requireActivity().getString(R.string.yes)),
-            String.format(Locale.getDefault(), requireActivity().getString(R.string.no)),
             {
                 ViewUtil.showShortToast(context, R.string.cancelling_upload)
                 pendingUploadsPresenter.deleteUpload(
                     contribution,
-                    this.requireContext().applicationContext,
+                    requireContext().applicationContext,
                 )
             },
-            {},
+            {}
         )
     }
 
@@ -177,19 +155,12 @@ class FailedUploadsFragment :
      * Deletes all the uploads after getting a confirmation from the user using Dialog.
      */
     fun deleteUploads() {
-        if (contributionsList != null) {
         DialogUtil.showAlertDialog(
             requireActivity(),
-                String.format(
+            getString(R.string.cancelling_all_the_uploads),
-                    Locale.getDefault(),
+            getString(R.string.are_you_sure_that_you_want_cancel_all_the_uploads),
-                    requireActivity().getString(R.string.cancelling_all_the_uploads),
+            getString(R.string.yes),
-                ),
+            getString(R.string.no),
-                String.format(
-                    Locale.getDefault(),
-                    requireActivity().getString(R.string.are_you_sure_that_you_want_cancel_all_the_uploads),
-                ),
-                String.format(Locale.getDefault(), requireActivity().getString(R.string.yes)),
-                String.format(Locale.getDefault(), requireActivity().getString(R.string.no)),
             {
                 ViewUtil.showShortToast(context, R.string.cancelling_upload)
                 uploadProgressActivity.hidePendingIcons()
@@ -197,8 +168,7 @@ class FailedUploadsFragment :
                     listOf(Contribution.STATE_FAILED),
                 )
             },
-                {},
+            {}
         )
     }
 }
-}