chinwag/chinwag-android
chinwag/chinwag-android
2
0
Fork 0
Code Issues Pull requests Releases Activity

remove duplicate request when timeline is empty (#1491)

Browse source
This commit is contained in:
Konrad Pozniak 2019-09-23 17:09:13 +02:00 committed by GitHub
parent b79e86cacd
commit 0d465d3bee
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
app/src/main/java/com/keylesspalace/tusky/fragment/TimelineFragment.java
View file

@ -68,7 +68,6 @@ import com.keylesspalace.tusky.view.EndlessOnScrollListener;
import com.keylesspalace.tusky.viewdata.StatusViewData; import com.keylesspalace.tusky.viewdata.StatusViewData;
import java.io.IOException; import java.io.IOException;
import java.util.ArrayList;
import java.util.Collections; import java.util.Collections;
import java.util.Iterator; import java.util.Iterator;
import java.util.List; import java.util.List;
@ -286,12 +285,12 @@ public class TimelineFragment extends SFragment implements
} }
private void updateCurrent() { private void updateCurrent() {
String topId;
if (this.statuses.isEmpty()) { if (this.statuses.isEmpty()) {
topId = null; return;
} else {
topId = CollectionsKt.first(this.statuses, Either::isRight).asRight().getId();
} }
String topId = CollectionsKt.first(this.statuses, Either::isRight).asRight().getId();
this.timelineRepo.getStatuses(topId, null, null, LOAD_AT_ONCE, this.timelineRepo.getStatuses(topId, null, null, LOAD_AT_ONCE,
TimelineRequestMode.NETWORK) TimelineRequestMode.NETWORK)
.observeOn(AndroidSchedulers.mainThread()) .observeOn(AndroidSchedulers.mainThread())
@ -303,7 +302,7 @@ public class TimelineFragment extends SFragment implements
if (!statuses.isEmpty()) { if (!statuses.isEmpty()) {
filterStatuses(statuses); filterStatuses(statuses);
if (!this.statuses.isEmpty() && topId != null) { if (!this.statuses.isEmpty()) {
// clear old cached statuses // clear old cached statuses
Iterator<Either<Placeholder, Status>> iterator = this.statuses.iterator(); Iterator<Either<Placeholder, Status>> iterator = this.statuses.iterator();
while (iterator.hasNext()) { while (iterator.hasNext()) {