nextcloud / nextcloud/talk-android
Refactor: Replace deprecated CurrentUserProviderOld with CurrentUserProvider
Nobody has claimed this yet.
- Dominant language
- Kotlin
- Stars
- 739
- Forks
- 321
- Avg merge
- 14h 59m
- Merged PRs (30d)
- 151
Description
Summary
Multiple files across the codebase still use the deprecated CurrentUserProviderOld interface. The compiler warns on every build: 'interface CurrentUserProviderOld : Any' is deprecated.
Use CurrentUserProvider instead.
Affected files
See build warnings.
Proposed solution
Replace all usages of CurrentUserProviderOld with CurrentUserProvider across the affected files and remove CurrentUserProviderOldImpl once no longer referenced.
Notes
Happy to work on this if the maintainers are open to it. Could be done file by file in separate PRs to keep changes
reviewable, or in one larger refactor PR; whatever the team prefers.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by running a build and reviewing the deprecation warnings to identify the affected files using CurrentUserProviderOld. Replace those usages with CurrentUserProvider, then check references to CurrentUserProviderOldImpl. Done means no remaining references to the deprecated interface or implementation and no related compiler warnings.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, kotlin
- Domain
- mobile
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100