Wiki itemname displaying in toast

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

@@ -68,6 +68,7 @@ public Contribution[] newArray(int i) {
     public String decimalCoords;
     public boolean isMultiple;
     public String wikiDataEntityId;
+    public String wikiItemName;
     private String p18Value;
     public Uri contentProviderUri;
     public String dateCreatedSource;
@@ -112,6 +113,7 @@ public Contribution(Parcel in) {
         state = in.readInt();
         transferred = in.readLong();
         isMultiple = in.readInt() == 1;
+        wikiItemName = in.readString();
     }
 
     @Override
@@ -122,6 +124,7 @@ public void writeToParcel(Parcel parcel, int flags) {
         parcel.writeInt(state);
         parcel.writeLong(transferred);
         parcel.writeInt(isMultiple ? 1 : 0);
+        parcel.writeString(wikiItemName);
     }
 
     public void setDateCreatedSource(String dateCreatedSource) {
@@ -263,6 +266,10 @@ public String getWikiDataEntityId() {
         return wikiDataEntityId;
     }
 
+    public String getWikiItemName() {
+        return wikiItemName;
+    }
+
     /**
      * When the corresponding wikidata entity is known as in case of nearby uploads, it can be set
      * using the setter method
@@ -272,6 +279,10 @@ public void setWikiDataEntityId(String wikiDataEntityId) {
         this.wikiDataEntityId = wikiDataEntityId;
     }
 
+    public void setWikiItemName(String wikiItemName) {
+        this.wikiItemName = wikiItemName;
+    }
+
     public String getP18Value() {
         return p18Value;
     }

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

@@ -160,6 +160,7 @@ public Observable<Contribution> buildContributions() {
                     CommonsApplication.DEFAULT_EDIT_SUMMARY, item.gpsCoords.getDecimalCoords());
             if (item.place != null) {
                 contribution.setWikiDataEntityId(item.place.getWikiDataEntityId());
+                contribution.setWikiItemName(item.place.getName());
                 // If item already has an image, we need to know it. We don't want to override existing image later
                 contribution.setP18Value(item.place.pic);
             }

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

@@ -281,7 +281,7 @@ private void uploadContribution(Contribution contribution) {
                         Timber.d("Contribution upload success. Initiating Wikidata edit for"
                                 + " entity id %s if necessary (if P18 is null). P18 value is %s",
                                 contribution.getWikiDataEntityId(), contribution.getP18Value());
-                        wikidataEditService.createClaimWithLogging(contribution.getWikiDataEntityId(), canonicalFilename, contribution.getP18Value());
+                        wikidataEditService.createClaimWithLogging(contribution.getWikiDataEntityId(), contribution.getWikiItemName(), canonicalFilename, contribution.getP18Value());
                         contribution.setFilename(canonicalFilename);
                         contribution.setImageUrl(uploadResult.getImageinfo().getOriginalUrl());
                         contribution.setState(Contribution.STATE_COMPLETED);

app/src/main/java/fr/free/nrw/commons/wikidata/WikidataEditService.java

@@ -50,7 +50,7 @@ public class WikidataEditService {
      * @param fileName name of the file we will upload
      * @param p18Value pic attribute of Wikidata item
      */
-    public void createClaimWithLogging(String wikidataEntityId, String fileName, @NonNull String p18Value) {
+    public void createClaimWithLogging(String wikidataEntityId, String wikiItemName, String fileName, @NonNull String p18Value) {
         if (wikidataEntityId == null) {
             Timber.d("Skipping creation of claim as Wikidata entity ID is null");
             return;
@@ -71,7 +71,7 @@ public void createClaimWithLogging(String wikidataEntityId, String fileName, @No
             return;
         }
 
-        editWikidataProperty(wikidataEntityId, fileName);
+        editWikidataProperty(wikidataEntityId, wikiItemName, fileName);
     }
 
     /**
@@ -82,7 +82,7 @@ public void createClaimWithLogging(String wikidataEntityId, String fileName, @No
      * @param fileName
      */
     @SuppressLint("CheckResult")
-    private void editWikidataProperty(String wikidataEntityId, String fileName) {
+    private void editWikidataProperty(String wikidataEntityId, String wikiItemName, String fileName) {
         Timber.d("Upload successful with wiki data entity id as %s", wikidataEntityId);
         Timber.d("Attempting to edit Wikidata property %s", wikidataEntityId);
 
@@ -98,18 +98,18 @@ private void editWikidataProperty(String wikidataEntityId, String fileName) {
                 })
                 .subscribeOn(Schedulers.io())
                 .observeOn(AndroidSchedulers.mainThread())
-                .subscribe(revisionId -> handleClaimResult(wikidataEntityId, String.valueOf(revisionId)), throwable -> {
+                .subscribe(revisionId -> handleClaimResult(wikidataEntityId, wikiItemName, String.valueOf(revisionId)), throwable -> {
                     Timber.e(throwable, "Error occurred while making claim");
                     ViewUtil.showLongToast(context, context.getString(R.string.wikidata_edit_failure));
                 });
     }
 
-    private void handleClaimResult(String wikidataEntityId, String revisionId) {
+    private void handleClaimResult(String wikidataEntityId, String wikiItemName, String revisionId) {
         if (revisionId != null) {
             if (wikidataEditListener != null) {
                 wikidataEditListener.onSuccessfulWikidataEdit();
             }
-            showSuccessToast();
+            showSuccessToast(wikiItemName);
         } else {
             Timber.d("Unable to make wiki data edit for entity %s", wikidataEntityId);
             ViewUtil.showLongToast(context, context.getString(R.string.wikidata_edit_failure));
@@ -119,10 +119,9 @@ private void handleClaimResult(String wikidataEntityId, String revisionId) {
     /**
      * Show a success toast when the edit is made successfully
      */
-    private void showSuccessToast() {
-        String title = directKvStore.getString("Title", "");
+    private void showSuccessToast(String wikiItemName) {
         String successStringTemplate = context.getString(R.string.successful_wikidata_edit);
-        String successMessage = String.format(Locale.getDefault(), successStringTemplate, title);
+        String successMessage = String.format(Locale.getDefault(), successStringTemplate, wikiItemName);
         ViewUtil.showLongToast(context, successMessage);
     }
 

app/src/test/kotlin/fr/free/nrw/commons/wikidata/WikidataEditServiceTest.kt

@@ -34,27 +34,27 @@ class WikidataEditServiceTest {
 
     @Test
     fun noClaimsWhenEntityIdIsNull() {
-        wikidataEditService!!.createClaimWithLogging(null, "Test.jpg","")
+        wikidataEditService!!.createClaimWithLogging(null, null,"Test.jpg","")
         verifyZeroInteractions(wikidataClient!!)
     }
 
     @Test
     fun noClaimsWhenFileNameIsNull() {
-        wikidataEditService!!.createClaimWithLogging("Q1", null,"")
+        wikidataEditService!!.createClaimWithLogging("Q1", "Test", null,"")
         verifyZeroInteractions(wikidataClient!!)
     }
 
     @Test
     fun noClaimsWhenP18IsNotEmpty() {
-        wikidataEditService!!.createClaimWithLogging("Q1", "Test.jpg","Previous.jpg")
+        wikidataEditService!!.createClaimWithLogging("Q1", "Test","Test.jpg","Previous.jpg")
         verifyZeroInteractions(wikidataClient!!)
     }
 
     @Test
     fun noClaimsWhenLocationIsNotCorrect() {
         `when`(directKvStore!!.getBoolean("Picture_Has_Correct_Location", true))
                 .thenReturn(false)
-        wikidataEditService!!.createClaimWithLogging("Q1", "Test.jpg","")
+        wikidataEditService!!.createClaimWithLogging("Q1", "","Test.jpg","")
         verifyZeroInteractions(wikidataClient!!)
     }
 
@@ -66,7 +66,7 @@ class WikidataEditServiceTest {
                 .thenReturn(Observable.just(1L))
         `when`(wikidataClient!!.addEditTag(anyLong(), ArgumentMatchers.anyString(), ArgumentMatchers.anyString()))
                 .thenReturn(Observable.just(mock(AddEditTagResponse::class.java)))
-        wikidataEditService!!.createClaimWithLogging("Q1", "Test.jpg","")
+        wikidataEditService!!.createClaimWithLogging("Q1", "Test","Test.jpg","")
         verify(wikidataClient!!, times(1))
                 .createClaim(ArgumentMatchers.anyString(), ArgumentMatchers.anyString())
     }