-
-
Notifications
You must be signed in to change notification settings - Fork 457
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use MutableObjectAdapter<Row> for rows #2075
Use MutableObjectAdapter<Row> for rows #2075
Conversation
|
||
//React to deletion | ||
val dataRefreshService = get<DataRefreshService>(DataRefreshService::class.java) | ||
if (activity != null && !requireActivity().isFinishing && currentRow != null && currentItem != null && currentItem!!.itemId != null && currentItem!!.itemId.equals(dataRefreshService.lastDeletedItemId)) { |
Check warning
Code scanning / detekt
Complex conditions should be simplified and extracted into well-named methods if necessary.
|
||
//React to deletion | ||
val dataRefreshService = get<DataRefreshService>(DataRefreshService::class.java) | ||
if (activity != null && !requireActivity().isFinishing && currentRow != null && currentItem != null && currentItem!!.itemId != null && currentItem!!.itemId.equals(dataRefreshService.lastDeletedItemId)) { |
Check warning
Code scanning / detekt
Line detected, which is longer than the defined maximum line length in the code style.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found 96 potential problems in the proposed changes. Check the Files changed tab for more details.
b9ed468
to
dbf75fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found 96 potential problems in the proposed changes. Check the Files changed tab for more details.
Depends on #2074
Changes
Use our custom ObjectAdapter that is more flexible for all item rows, right now this shouldn't change the behavior.
Issues