Skip to content

Commit 76b84c6

Browse files
authored
Merge pull request commons-app#635 from dbrant/deleted
Don't show uploads that have been deleted.
2 parents e34ef2b + 269cef5 commit 76b84c6

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

app/src/main/java/fr/free/nrw/commons/contributions/ContributionsSyncAdapter.java

+6-1
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ public void onPerformSync(Account account, Bundle bundle, String s, ContentProvi
7474
MWApi.RequestBuilder builder = api.action("query")
7575
.param("list", "logevents")
7676
.param("letype", "upload")
77-
.param("leprop", "title|timestamp")
77+
.param("leprop", "title|timestamp|ids")
7878
.param("leuser", user)
7979
.param("lelimit", getLimit());
8080
if(!TextUtils.isEmpty(lastModified)) {
@@ -97,6 +97,11 @@ public void onPerformSync(Account account, Bundle bundle, String s, ContentProvi
9797
Timber.d("%d results!", uploads.size());
9898
ArrayList<ContentValues> imageValues = new ArrayList<>();
9999
for(ApiResult image: uploads) {
100+
String pageId = image.getString("@pageid");
101+
if (pageId.equals("0")) {
102+
// means that this upload was deleted.
103+
continue;
104+
}
100105
String filename = image.getString("@title");
101106
if(fileExists(contentProviderClient, filename)) {
102107
Timber.d("Skipping %s", filename);

0 commit comments

Comments
 (0)