search

geofot96 / StudyBuddy

2 stars 5 forks source link

[BUG] fix the app crash on group deletion #148

Closed gessfred closed 5 years ago

gessfred commented 5 years ago

Fixed the bug that made the app crash when a group was deleted (by removing the adapter before the deletion).

Also broke down the util package into another package 'tools' which should contain Adapters, Consumers, Holders, ...

codeclimate[bot] commented 5 years ago

Code Climate has analyzed commit 733f8bfe and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 84.7% (0.0% change).

View more on Code Climate.

gessfred commented 5 years ago

I agree with @geofot96, especially about the onClickListenner, but those are not issues introduced through this PR. As this was supposed to be just a quick fix, the suggested improvments will be implemented through another PR.