Simplify failed plugin list creation
ClosedPublic

Authored by nicolasfella on Apr 15 2018, 8:25 PM.

Details

Summary

Use some lambdas to simplify the code

Test Plan

Denie some permissions, open permission explanation dialogs

Diff Detail

Repository
R225 KDE Connect - Android application
Branch
java8
Lint
No Linters Available
Unit
No Unit Test Coverage
nicolasfella created this revision.Apr 15 2018, 8:25 PM
Restricted Application added a project: KDE Connect. · View Herald TranscriptApr 15 2018, 8:25 PM
nicolasfella requested review of this revision.Apr 15 2018, 8:25 PM
  • Add license header
mtijink added inline comments.
src/org/kde/kdeconnect/UserInterface/DeviceFragment.java
330 ↗(On Diff #32876)

You removed the retry code. Does that make sense? (ie never fired, or retries happen anyway?)

src/org/kde/kdeconnect/UserInterface/PluginSmallEntryItem.java
25 ↗(On Diff #32223)

What does this name mean? I'd pick something clearer, although that seems hard in this case.

  • Revert lambda, nename PluginSmallEntryItem
nicolasfella added inline comments.Apr 17 2018, 4:11 PM
src/org/kde/kdeconnect/UserInterface/DeviceFragment.java
330 ↗(On Diff #32876)

It was blocking making it a lambda, but it was probably a bad idea to blindly remove it

mtijink added inline comments.Apr 17 2018, 4:45 PM
src/org/kde/kdeconnect/UserInterface/PluginListItem.java
26 ↗(On Diff #32875)

Better, but these are only for the required permissions, failure, etc., right? Maybe FailedPluginListItem?

Restricted Application added a subscriber: kdeconnect. · View Herald TranscriptSep 12 2018, 11:59 PM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 4 2018, 5:03 PM
This revision was automatically updated to reflect the committed changes.