mirror of
https://code.briarproject.org/briar/briar.git
synced 2026-02-19 14:19:53 +01:00
Merge branch '1210-fix-list-duplicates' into 'master'
[android] Fix duplicate items in lists Closes #1210 See merge request briar/briar!1174
This commit is contained in:
@@ -267,7 +267,7 @@ public class ContactListFragment extends BaseFragment implements EventListener,
|
|||||||
if (revision == adapter.getRevision()) {
|
if (revision == adapter.getRevision()) {
|
||||||
adapter.incrementRevision();
|
adapter.incrementRevision();
|
||||||
if (contacts.isEmpty()) list.showData();
|
if (contacts.isEmpty()) list.showData();
|
||||||
else adapter.addAll(contacts);
|
else adapter.replaceAll(contacts);
|
||||||
} else {
|
} else {
|
||||||
LOG.info("Concurrent update, reloading");
|
LOG.info("Concurrent update, reloading");
|
||||||
loadContacts();
|
loadContacts();
|
||||||
|
|||||||
@@ -182,7 +182,7 @@ public class ForumListFragment extends BaseEventFragment implements
|
|||||||
if (revision == adapter.getRevision()) {
|
if (revision == adapter.getRevision()) {
|
||||||
adapter.incrementRevision();
|
adapter.incrementRevision();
|
||||||
if (forums.isEmpty()) list.showData();
|
if (forums.isEmpty()) list.showData();
|
||||||
else adapter.addAll(forums);
|
else adapter.replaceAll(forums);
|
||||||
} else {
|
} else {
|
||||||
LOG.info("Concurrent update, reloading");
|
LOG.info("Concurrent update, reloading");
|
||||||
loadForums();
|
loadForums();
|
||||||
|
|||||||
@@ -194,7 +194,7 @@ public class GroupListFragment extends BaseFragment implements
|
|||||||
if (revision == adapter.getRevision()) {
|
if (revision == adapter.getRevision()) {
|
||||||
adapter.incrementRevision();
|
adapter.incrementRevision();
|
||||||
if (groups.isEmpty()) list.showData();
|
if (groups.isEmpty()) list.showData();
|
||||||
else adapter.addAll(groups);
|
else adapter.replaceAll(groups);
|
||||||
} else {
|
} else {
|
||||||
LOG.info("Concurrent update, reloading");
|
LOG.info("Concurrent update, reloading");
|
||||||
loadGroups();
|
loadGroups();
|
||||||
|
|||||||
@@ -79,6 +79,10 @@ public abstract class BriarAdapter<T, V extends ViewHolder>
|
|||||||
this.items.addAll(items);
|
this.items.addAll(items);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public void replaceAll(Collection<T> items) {
|
||||||
|
this.items.replaceAll(items);
|
||||||
|
}
|
||||||
|
|
||||||
public void setItems(Collection<T> items) {
|
public void setItems(Collection<T> items) {
|
||||||
this.items.beginBatchedUpdates();
|
this.items.beginBatchedUpdates();
|
||||||
this.items.clear();
|
this.items.clear();
|
||||||
|
|||||||
Reference in New Issue
Block a user