Skip to content

Commit c7f7b39

Browse files
authored
Fixes commons-app#3380 - "Nearby Place found" despite already Nearby upload (commons-app#4408)
* fix issue with "nearby place found" pop up on nearby upload * added javadoc Co-authored-by: Pratham2305 <Pratham2305@users.noreply.github.com>
1 parent 2b62d84 commit c7f7b39

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

app/src/main/java/fr/free/nrw/commons/upload/UploadItem.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ public class UploadItem {
1818
private final String mimeType;
1919
private ImageCoordinates gpsCoords;
2020
private List<UploadMediaDetail> uploadMediaDetails;
21-
private final Place place;
21+
private Place place;
2222
private final long createdTimestamp;
2323
private final String createdTimestampSource;
2424
private final BehaviorSubject<Integer> imageQuality;
@@ -70,6 +70,14 @@ public void setImageQuality(final int imageQuality) {
7070
this.imageQuality.onNext(imageQuality);
7171
}
7272

73+
/**
74+
* Sets the corresponding place to the uploadItem
75+
* @param place geolocated Wikidata item
76+
*/
77+
public void setPlace(Place place) {
78+
this.place = place;
79+
}
80+
7381
public Place getPlace() {
7482
return place;
7583
}

app/src/main/java/fr/free/nrw/commons/upload/mediaDetails/UploadMediaPresenter.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ public void receiveImage(UploadableFile uploadableFile, Place place) {
9292
gpsCoords != null && gpsCoords.getImageCoordsExists();
9393
view.showMapWithImageCoordinates(hasImageCoordinates);
9494
view.showProgress(false);
95-
if (hasImageCoordinates) {
95+
if (hasImageCoordinates && place == null) {
9696
checkNearbyPlaces(uploadItem);
9797
}
9898
},
@@ -194,6 +194,9 @@ public void onUserConfirmedUploadIsOfPlace(Place place, int uploadItemPosition)
194194
final List<UploadMediaDetail> uploadMediaDetails = repository.getUploads()
195195
.get(uploadItemPosition)
196196
.getUploadMediaDetails();
197+
UploadItem uploadItem = repository.getUploads()
198+
.get(uploadItemPosition);
199+
uploadItem.setPlace(place);
197200
uploadMediaDetails.set(0, new UploadMediaDetail(place));
198201
view.updateMediaDetails(uploadMediaDetails);
199202
}

0 commit comments

Comments
 (0)