Resolve or Create new Issues for all TODO/FIXME Comments #6508
Description
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.
Metadata
Metadata
Assignees
Labels
No labels