fix: use context instead of requireContext() for backward compatibility (#6403)

rohit9625 · RitikaPahwa4444 · web-flow · commit 4f3f7b97fd83 · 2025-08-23T12:33:30.000+05:30
It fixes crash when opening certain screens like Contribution Details, Bookmark, etc. on lower Android versions

Co-authored-by: Ritika Pahwa &lt;83745993+RitikaPahwa4444@users.noreply.github.com&gt;
diff --git a/app/src/main/java/fr/free/nrw/commons/bookmarks/items/BookmarkItemsContentProvider.kt b/app/src/main/java/fr/free/nrw/commons/bookmarks/items/BookmarkItemsContentProvider.kt
@@ -36,7 +36,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {
             requireDb(), projection, selection,
             selectionArgs, null, null, sortOrder
         ).apply {
-            setNotificationUri(requireContext().contentResolver, uri)
+            setNotificationUri(context?.contentResolver, uri)
         }
     }
 
@@ -66,7 +66,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {
             )
         }
 
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return rowsUpdated
     }
 
@@ -75,7 +75,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {
      */
     override fun insert(uri: Uri, contentValues: ContentValues?): Uri? {
         val id = requireDb().insert(TABLE_NAME, null, contentValues)
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return "$BASE_URI/$id".toUri()
     }
 
@@ -89,7 +89,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {
             "$COLUMN_ID = ?",
             arrayOf(uri.lastPathSegment)
         )
-        requireContext().contentResolver.notifyChange(uri, null)
+       context?.contentResolver?.notifyChange(uri, null)
         return rows
     }
 
diff --git a/app/src/main/java/fr/free/nrw/commons/bookmarks/pictures/BookmarkPicturesContentProvider.kt b/app/src/main/java/fr/free/nrw/commons/bookmarks/pictures/BookmarkPicturesContentProvider.kt
@@ -36,7 +36,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {
             requireDb(), projection, selection,
             selectionArgs, null, null, sortOrder
         )
-        cursor.setNotificationUri(requireContext().contentResolver, uri)
+        cursor.setNotificationUri(context?.contentResolver, uri)
 
         return cursor
     }
@@ -66,7 +66,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {
                 "Parameter `selection` should be empty when updating an ID"
             )
         }
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return rowsUpdated
     }
 
@@ -75,7 +75,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {
      */
     override fun insert(uri: Uri, contentValues: ContentValues?): Uri {
         val id = requireDb().insert(TABLE_NAME, null, contentValues)
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return "$BASE_URI/$id".toUri()
     }
 
@@ -85,7 +85,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {
             "media_name = ?",
             arrayOf(uri.lastPathSegment)
         )
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return rows
     }
 
diff --git a/app/src/main/java/fr/free/nrw/commons/explore/recentsearches/RecentSearchesContentProvider.kt b/app/src/main/java/fr/free/nrw/commons/explore/recentsearches/RecentSearchesContentProvider.kt
@@ -50,7 +50,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {
             else -> throw IllegalArgumentException("Unknown URI$uri")
         }
 
-        cursor.setNotificationUri(requireContext().contentResolver, uri)
+        cursor.setNotificationUri(context?.contentResolver, uri)
 
         return cursor
     }
@@ -67,7 +67,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {
 
             else -> throw IllegalArgumentException("Unknown URI: $uri")
         }
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return "$BASE_URI/$id".toUri()
     }
 
@@ -88,7 +88,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {
 
             else -> throw IllegalArgumentException("Unknown URI - $uri")
         }
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return rows
     }
 
@@ -108,7 +108,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {
         }
         sqlDB.setTransactionSuccessful()
         sqlDB.endTransaction()
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return values.size
     }
 
@@ -147,7 +147,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {
 
             else -> throw IllegalArgumentException("Unknown URI: $uri with type $uriType")
         }
-        requireContext().contentResolver.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return rowsUpdated
     }
 
diff --git a/app/src/main/java/fr/free/nrw/commons/recentlanguages/RecentLanguagesContentProvider.kt b/app/src/main/java/fr/free/nrw/commons/recentlanguages/RecentLanguagesContentProvider.kt
@@ -59,7 +59,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {
             null,
             sortOrder
         )
-        cursor.setNotificationUri(requireContext().contentResolver, uri)
+        cursor.setNotificationUri(context?.contentResolver, uri)
         return cursor
     }
 
@@ -90,7 +90,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {
             throw IllegalArgumentException("Parameter `selection` should be empty when updating an ID")
         }
 
-        requireContext().contentResolver?.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return rowsUpdated
     }
 
@@ -105,7 +105,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {
             null,
             contentValues
         )
-        requireContext().contentResolver?.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return "$BASE_URI/$id".toUri()
     }
 
@@ -119,7 +119,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {
             "language_code = ?",
             arrayOf(uri.lastPathSegment)
         )
-        requireContext().contentResolver?.notifyChange(uri, null)
+        context?.contentResolver?.notifyChange(uri, null)
         return rows
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {`
`36`	`36`	`requireDb(), projection, selection,`
`37`	`37`	`selectionArgs, null, null, sortOrder`
`38`	`38`	`).apply {`
`39`		`- setNotificationUri(requireContext().contentResolver, uri)`
	`39`	`+ setNotificationUri(context?.contentResolver, uri)`
`40`	`40`	`}`
`41`	`41`	`}`
`42`	`42`
`@@ -66,7 +66,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {`
`66`	`66`	`)`
`67`	`67`	`}`
`68`	`68`
`69`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`69`	`+ context?.contentResolver?.notifyChange(uri, null)`
`70`	`70`	`return rowsUpdated`
`71`	`71`	`}`
`72`	`72`
`@@ -75,7 +75,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {`
`75`	`75`	`*/`
`76`	`76`	`override fun insert(uri: Uri, contentValues: ContentValues?): Uri? {`
`77`	`77`	`val id = requireDb().insert(TABLE_NAME, null, contentValues)`
`78`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`78`	`+ context?.contentResolver?.notifyChange(uri, null)`
`79`	`79`	`return "$BASE_URI/$id".toUri()`
`80`	`80`	`}`
`81`	`81`
`@@ -89,7 +89,7 @@ class BookmarkItemsContentProvider : CommonsDaggerContentProvider() {`
`89`	`89`	`"$COLUMN_ID = ?",`
`90`	`90`	`arrayOf(uri.lastPathSegment)`
`91`	`91`	`)`
`92`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`92`	`+ context?.contentResolver?.notifyChange(uri, null)`
`93`	`93`	`return rows`
`94`	`94`	`}`
`95`	`95`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {`
`36`	`36`	`requireDb(), projection, selection,`
`37`	`37`	`selectionArgs, null, null, sortOrder`
`38`	`38`	`)`
`39`		`- cursor.setNotificationUri(requireContext().contentResolver, uri)`
	`39`	`+ cursor.setNotificationUri(context?.contentResolver, uri)`
`40`	`40`
`41`	`41`	`return cursor`
`42`	`42`	`}`
`@@ -66,7 +66,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {`
`66`	`66`	"Parameter `selection` should be empty when updating an ID"
`67`	`67`	`)`
`68`	`68`	`}`
`69`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`69`	`+ context?.contentResolver?.notifyChange(uri, null)`
`70`	`70`	`return rowsUpdated`
`71`	`71`	`}`
`72`	`72`
`@@ -75,7 +75,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {`
`75`	`75`	`*/`
`76`	`76`	`override fun insert(uri: Uri, contentValues: ContentValues?): Uri {`
`77`	`77`	`val id = requireDb().insert(TABLE_NAME, null, contentValues)`
`78`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`78`	`+ context?.contentResolver?.notifyChange(uri, null)`
`79`	`79`	`return "$BASE_URI/$id".toUri()`
`80`	`80`	`}`
`81`	`81`
`@@ -85,7 +85,7 @@ class BookmarkPicturesContentProvider : CommonsDaggerContentProvider() {`
`85`	`85`	`"media_name = ?",`
`86`	`86`	`arrayOf(uri.lastPathSegment)`
`87`	`87`	`)`
`88`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`88`	`+ context?.contentResolver?.notifyChange(uri, null)`
`89`	`89`	`return rows`
`90`	`90`	`}`
`91`	`91`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {`
`50`	`50`	`else -> throw IllegalArgumentException("Unknown URI$uri")`
`51`	`51`	`}`
`52`	`52`
`53`		`- cursor.setNotificationUri(requireContext().contentResolver, uri)`
	`53`	`+ cursor.setNotificationUri(context?.contentResolver, uri)`
`54`	`54`
`55`	`55`	`return cursor`
`56`	`56`	`}`
`@@ -67,7 +67,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {`
`67`	`67`
`68`	`68`	`else -> throw IllegalArgumentException("Unknown URI: $uri")`
`69`	`69`	`}`
`70`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`70`	`+ context?.contentResolver?.notifyChange(uri, null)`
`71`	`71`	`return "$BASE_URI/$id".toUri()`
`72`	`72`	`}`
`73`	`73`
`@@ -88,7 +88,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {`
`88`	`88`
`89`	`89`	`else -> throw IllegalArgumentException("Unknown URI - $uri")`
`90`	`90`	`}`
`91`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`91`	`+ context?.contentResolver?.notifyChange(uri, null)`
`92`	`92`	`return rows`
`93`	`93`	`}`
`94`	`94`
`@@ -108,7 +108,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {`
`108`	`108`	`}`
`109`	`109`	`sqlDB.setTransactionSuccessful()`
`110`	`110`	`sqlDB.endTransaction()`
`111`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`111`	`+ context?.contentResolver?.notifyChange(uri, null)`
`112`	`112`	`return values.size`
`113`	`113`	`}`
`114`	`114`
`@@ -147,7 +147,7 @@ class RecentSearchesContentProvider : CommonsDaggerContentProvider() {`
`147`	`147`
`148`	`148`	`else -> throw IllegalArgumentException("Unknown URI: $uri with type $uriType")`
`149`	`149`	`}`
`150`		`- requireContext().contentResolver.notifyChange(uri, null)`
	`150`	`+ context?.contentResolver?.notifyChange(uri, null)`
`151`	`151`	`return rowsUpdated`
`152`	`152`	`}`
`153`	`153`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {`
`59`	`59`	`null,`
`60`	`60`	`sortOrder`
`61`	`61`	`)`
`62`		`- cursor.setNotificationUri(requireContext().contentResolver, uri)`
	`62`	`+ cursor.setNotificationUri(context?.contentResolver, uri)`
`63`	`63`	`return cursor`
`64`	`64`	`}`
`65`	`65`
`@@ -90,7 +90,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {`
`90`	`90`	throw IllegalArgumentException("Parameter `selection` should be empty when updating an ID")
`91`	`91`	`}`
`92`	`92`
`93`		`- requireContext().contentResolver?.notifyChange(uri, null)`
	`93`	`+ context?.contentResolver?.notifyChange(uri, null)`
`94`	`94`	`return rowsUpdated`
`95`	`95`	`}`
`96`	`96`
`@@ -105,7 +105,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {`
`105`	`105`	`null,`
`106`	`106`	`contentValues`
`107`	`107`	`)`
`108`		`- requireContext().contentResolver?.notifyChange(uri, null)`
	`108`	`+ context?.contentResolver?.notifyChange(uri, null)`
`109`	`109`	`return "$BASE_URI/$id".toUri()`
`110`	`110`	`}`
`111`	`111`
`@@ -119,7 +119,7 @@ class RecentLanguagesContentProvider : CommonsDaggerContentProvider() {`
`119`	`119`	`"language_code = ?",`
`120`	`120`	`arrayOf(uri.lastPathSegment)`
`121`	`121`	`)`
`122`		`- requireContext().contentResolver?.notifyChange(uri, null)`
	`122`	`+ context?.contentResolver?.notifyChange(uri, null)`
`123`	`123`	`return rows`
`124`	`124`	`}`
`125`	`125`	`}`