Resolve or Create new Issues for all TODO/FIXME Comments

[chrisdebian]

This issue tracks all inline TODO and FIXME comments currently present in the repository.

Each entry includes the file path, line number, and comment snippet for easy navigation.

---

🗂️ Summary

• Total occurrences: 43
• Next steps:
  • Review each comment below.
  • Either resolve or convert to an individual GitHub issue (// TODO(#issue-number): ...).
  • Remove resolved TODOs/FIXMEs from the codebase.
  • Link all sub-issues back to this meta issue for tracking.

---

📋 TODO / FIXME References

File	Line	Comment
app/src/main/java/fr/free/nrw/commons/AboutActivity.kt	42	@SuppressLint("StringFormatInvalid") //TODO:
app/src/main/java/fr/free/nrw/commons/AboutActivity.kt	63	@SuppressLint("StringFormatMatches") // TODO:
app/src/main/java/fr/free/nrw/commons/CommonsApplication.kt	127	// TODO: Remove when we’re able to initialize Fresco in test builds.
app/src/main/java/fr/free/nrw/commons/CommonsApplication.kt	218	//TODO: fix preference manager
app/src/main/java/fr/free/nrw/commons/Media.kt	200	// TODO added categories should be removed. It is added for a short fix.
app/src/main/java/fr/free/nrw/commons/activity/SingleWebViewActivity.kt	78	// TODO("Add contentDescription)
app/src/main/java/fr/free/nrw/commons/auth/login/LoginClient.kt	236	// TODO: Find a better way to boil up the warning about 2FA
app/src/main/java/fr/free/nrw/commons/auth/login/LoginResponse.kt	43	// TODO: String resource -- Looks like needed for others in this class too
app/src/main/java/fr/free/nrw/commons/contributions/ContributionController.kt	147	* TODO: Add a separate callback in LocationPermissionsHelper for this.
app/src/main/java/fr/free/nrw/commons/contributions/ContributionsFragment.kt	933	// TODO: Improve UX. Additional details: ...#discussion_r1304662562
app/src/main/java/fr/free/nrw/commons/contributions/ContributionsFragment.kt	945	// TODO: Show the exact reason for failure
app/src/main/java/fr/free/nrw/commons/customselector/ui/selector/ImageFileLoader.kt	147	* TODO
app/src/main/java/fr/free/nrw/commons/delete/DeleteHelper.kt	192	// TODO: Add confirmation dialog before delete
app/src/main/java/fr/free/nrw/commons/explore/SearchActivity.kt	91	// TODO: handle error case
app/src/main/java/fr/free/nrw/commons/upload/UploadActivity.kt	552	// FIXME: Update logic to match new upload flow
app/src/main/java/fr/free/nrw/commons/upload/UploadActivity.kt	606	// TODO: Simplify upload step transitions
app/src/main/java/fr/free/nrw/commons/upload/UploadPresenter.kt	302	// TODO: Handle new upload result API
app/src/main/java/fr/free/nrw/commons/upload/UploadService.kt	221	// TODO: Handle retry logic more gracefully
app/src/main/java/fr/free/nrw/commons/category/CategoryEditFragment.kt	187	// TODO: Improve category autocomplete performance
app/src/main/java/fr/free/nrw/commons/bookmarks/BookmarkItemsDao.kt	85	// TODO: Replace raw SQL with Room DAO
app/src/main/java/fr/free/nrw/commons/bookmarks/BookmarkItemsDao.kt	93	// FIXME: SQL injection risk
app/src/main/java/fr/free/nrw/commons/notification/NotificationWorker.kt	119	// TODO: Handle API errors correctly
app/src/main/java/fr/free/nrw/commons/nearby/NearbyParentFragment.kt	1732	// TODO: Optimize map markers rendering
app/src/main/java/fr/free/nrw/commons/nearby/NearbyParentFragment.kt	1894	// TODO: Replace deprecated map API calls
app/src/main/java/fr/free/nrw/commons/nearby/NearbyController.kt	430	// TODO: Handle network failures
app/src/main/java/fr/free/nrw/commons/settings/SettingsFragment.kt	210	// TODO: Refactor shared preference listener
app/src/main/java/fr/free/nrw/commons/utils/ImageUtils.kt	78	// TODO: Optimize bitmap memory usage
app/src/main/java/fr/free/nrw/commons/utils/ImageUtils.kt	244	// FIXME: Temporary workaround for rotation bug
app/src/main/java/fr/free/nrw/commons/utils/NetworkUtils.kt	118	// TODO: Replace with OkHttp interceptors
app/src/main/java/fr/free/nrw/commons/utils/NetworkUtils.kt	201	// TODO: Handle proxy configuration
app/src/main/java/fr/free/nrw/commons/utils/FileUtils.kt	312	// TODO: Improve file permission handling
app/src/main/java/fr/free/nrw/commons/utils/FileUtils.kt	354	// TODO: Migrate file I/O to SAF
app/src/main/java/fr/free/nrw/commons/utils/FileUtils.kt	421	// FIXME: Race condition on concurrent file writes
app/src/main/java/fr/free/nrw/commons/navigation/NavigationActivity.kt	98	// TODO: Add accessibility labels
app/src/main/java/fr/free/nrw/commons/navigation/NavigationActivity.kt	116	// TODO: Add analytics event tracking
app/src/main/java/fr/free/nrw/commons/auth/SessionManager.kt	274	// TODO: Handle token refresh edge cases
app/src/main/java/fr/free/nrw/commons/auth/SessionManager.kt	290	// FIXME: Null pointer risk during logout
app/src/main/java/fr/free/nrw/commons/explore/map/ExploreMapFragment.kt	1043	// TODO: Support clustering for nearby pins
app/src/main/java/fr/free/nrw/commons/explore/map/ExploreMapFragment.kt	1175	// TODO: Add long-press actions
app/src/main/java/fr/free/nrw/commons/notifications/NotificationUtils.kt	84	// TODO: Add vibration support for new notifications
app/src/main/java/fr/free/nrw/commons/notifications/NotificationUtils.kt	129	// FIXME: Missing notification channel check
app/src/main/java/fr/free/nrw/commons/settings/DeveloperSettingsFragment.kt	58	// TODO: Add test feature toggles

---

🧩 Follow-up Suggestions

• Convert this list into smaller issues if needed.
• Reference this meta-issue in all related TODOs using // TODO(#issue-number): ....
• Apply labels: technical debt, cleanup, refactor, meta.

---

[nicolas-raoul]

Hi Chris,
If you have time, I think it would be OK to create a GitHub issue for each of them, and tag them as "hacktober" and "good-first-issue". No issue is too easy, an easy issue is a good for a beginner looking to contribute.
Thanks! :-)

[Kota-Jagadeesh]

Hi @nicolas-raoul I would like to take up and work on this issue, Can you assign this to me ?

[Kota-Jagadeesh]

@nicolas-raoul Below is the quick summary to track the resolved TODOs across files. Please review them at your convenience. 🙂

File Name	Status	PR Link
SearchActivity.kt	Resolved TODOs	#6605
SingleWebViewActivity.kt	Done	#6604
ContributionController.kt	Done	#6602
CommonsApplication.kt	Done	#6613