android / android/architecture-samples
Why not the same as deleteAllTasks()?
- Dominant language
- Kotlin
- Stars
- 45.8k
- Forks
- 11.9k
- PR merge metrics
- No merged PRs in 30d
Description
https://github.com/googlesamples/android-architecture/blob/6419d4c523b67d020120fc400ed5a7372e5615f2/todoapp/app/src/main/java/com/example/android/architecture/blueprints/todoapp/data/source/TasksRepository.java#L299
In line 288, it judges the Map is null or not. Then new a map? If line 299 not like this, NullPointException will be trigged.
Contributor guide
Research direction
Start with TasksRepository.java at the referenced lines 288-299 and compare that path with deleteAllTasks(). Check whether the map can be null at line 299 and confirm the expected behavior by reviewing the surrounding repository logic; done means the null-handling difference and any resulting NullPointerException are addressed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, java
- Domain
- mobile
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100