mnalis/apps-android-commons
1
0
Fork 0
mirror of https://github.com/commons-app/apps-android-commons.git synced 2025-10-27 04:43:54 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Fix bugs in peer review flow (#3039) (#3042)

Browse source 
* Fix bugs in peer review flow

* Fix tests

* Bug fixes

* Fix remaining issues with peer review
This commit is contained in:
Vivek Maskara 2019-06-26 21:31:06 +05:30 • committed by GitHub
parent 4ec92c2e7b
commit d7628959a7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
8 changed files with 140 additions and 125 deletions
Download patch file Download diff file Expand all files Collapse all files

36
app/src/main/java/fr/free/nrw/commons/delete/DeleteHelper.java
View file

@ -1,9 +1,12 @@
package fr.free.nrw.commons.delete; package fr.free.nrw.commons.delete;
import android.annotation.SuppressLint;
import android.content.Context; import android.content.Context;
import android.content.Intent; import android.content.Intent;
import android.net.Uri; import android.net.Uri;
import androidx.appcompat.app.AlertDialog;
import java.text.SimpleDateFormat; import java.text.SimpleDateFormat;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Calendar; import java.util.Calendar;
@ -12,17 +15,16 @@ import java.util.Locale;
import javax.inject.Inject; import javax.inject.Inject;
import javax.inject.Singleton; import javax.inject.Singleton;
import androidx.appcompat.app.AlertDialog;
import fr.free.nrw.commons.BuildConfig; import fr.free.nrw.commons.BuildConfig;
import fr.free.nrw.commons.Media; import fr.free.nrw.commons.Media;
import fr.free.nrw.commons.auth.SessionManager; import fr.free.nrw.commons.auth.SessionManager;
import fr.free.nrw.commons.mwapi.MediaWikiApi; import fr.free.nrw.commons.mwapi.MediaWikiApi;
import fr.free.nrw.commons.notification.NotificationHelper; import fr.free.nrw.commons.notification.NotificationHelper;
import fr.free.nrw.commons.review.ReviewActivity;
import fr.free.nrw.commons.utils.ViewUtil;
import fr.free.nrw.commons.review.ReviewController; import fr.free.nrw.commons.review.ReviewController;
import fr.free.nrw.commons.utils.ViewUtilWrapper; import fr.free.nrw.commons.utils.ViewUtilWrapper;
import io.reactivex.Single; import io.reactivex.Single;
import io.reactivex.android.schedulers.AndroidSchedulers;
import io.reactivex.schedulers.Schedulers;
import timber.log.Timber; import timber.log.Timber;
import static fr.free.nrw.commons.notification.NotificationHelper.NOTIFICATION_DELETE; import static fr.free.nrw.commons.notification.NotificationHelper.NOTIFICATION_DELETE;
@ -57,7 +59,6 @@ public class DeleteHelper {
*/ */
public Single<Boolean> makeDeletion(Context context, Media media, String reason) { public Single<Boolean> makeDeletion(Context context, Media media, String reason) {
viewUtil.showShortToast(context, "Trying to nominate " + media.getDisplayTitle() + " for deletion"); viewUtil.showShortToast(context, "Trying to nominate " + media.getDisplayTitle() + " for deletion");
return Single.fromCallable(() -> delete(media, reason)) return Single.fromCallable(() -> delete(media, reason))
.flatMap(result -> Single.fromCallable(() -> .flatMap(result -> Single.fromCallable(() ->
showDeletionNotification(context, media, result))); showDeletionNotification(context, media, result)));
@ -99,7 +100,8 @@ public class DeleteHelper {
try { try {
editToken = mwApi.getEditToken(); editToken = mwApi.getEditToken();
if (editToken.equals("+\\")) {
if(editToken == null) {
return false; return false;
} }
@ -143,7 +145,12 @@ public class DeleteHelper {
* @param question * @param question
* @param problem * @param problem
*/ */
public void askReasonAndExecute(Media media, Context context, String question, ReviewController.DeleteReason problem) { @SuppressLint("CheckResult")
public void askReasonAndExecute(Media media,
Context context,
String question,
ReviewController.DeleteReason problem,
ReviewController.ReviewCallback reviewCallback) {
AlertDialog.Builder alert = new AlertDialog.Builder(context); AlertDialog.Builder alert = new AlertDialog.Builder(context);
alert.setTitle(question); alert.setTitle(question);
@ -183,12 +190,19 @@ public class DeleteHelper {
} }
} }
((ReviewActivity) context).reviewController.swipeToNext(); makeDeletion(context, media, reason)
((ReviewActivity) context).runRandomizer(); .subscribeOn(Schedulers.io())
.observeOn(AndroidSchedulers.mainThread())
makeDeletion(context, media, reason); .subscribe(aBoolean -> {
if (aBoolean) {
reviewCallback.onSuccess();
} else {
reviewCallback.onFailure();
}
}); });
alert.setNegativeButton("Cancel", null);
});
alert.setNegativeButton("Cancel", (dialog, which) -> reviewCallback.onFailure());
AlertDialog d = alert.create(); AlertDialog d = alert.create();
d.show(); d.show();
} }

18
app/src/main/java/fr/free/nrw/commons/review/ReviewActivity.java
View file

@ -20,8 +20,6 @@ import com.facebook.drawee.view.SimpleDraweeView;
import com.google.android.material.navigation.NavigationView; import com.google.android.material.navigation.NavigationView;
import com.viewpagerindicator.CirclePageIndicator; import com.viewpagerindicator.CirclePageIndicator;
import java.util.ArrayList;
import javax.inject.Inject; import javax.inject.Inject;
import butterknife.BindView; import butterknife.BindView;
@ -92,6 +90,9 @@ public class ReviewActivity extends AuthenticatedActivity {
private CompositeDisposable compositeDisposable = new CompositeDisposable(); private CompositeDisposable compositeDisposable = new CompositeDisposable();
public Media getMedia() {
return media;
}
@Override @Override
protected void onAuthCookieAcquired(String authCookie) { protected void onAuthCookieAcquired(String authCookie) {
@ -163,23 +164,18 @@ public class ReviewActivity extends AuthenticatedActivity {
simpleDraweeView.setImageURI(media.getImageUrl()); simpleDraweeView.setImageURI(media.getImageUrl());
reviewController.onImageRefreshed(fileName); //file name is updated reviewController.onImageRefreshed(media); //file name is updated
compositeDisposable.add(reviewHelper.getFirstRevisionOfFile(fileName) compositeDisposable.add(reviewHelper.getFirstRevisionOfFile(fileName)
.subscribeOn(Schedulers.io()) .subscribeOn(Schedulers.io())
.observeOn(AndroidSchedulers.mainThread()) .observeOn(AndroidSchedulers.mainThread())
.subscribe(revision -> { .subscribe(revision -> {
reviewController.firstRevision = revision; reviewController.firstRevision = revision;
reviewPagerAdapter.updateFileInformation(fileName); reviewPagerAdapter.updateFileInformation();
imageCaption.setText(fileName + " is uploaded by: " + revision.getUser()); String caption = String.format(getString(R.string.review_is_uploaded_by), fileName, revision.getUser());
imageCaption.setText(caption);
progressBar.setVisibility(View.GONE); progressBar.setVisibility(View.GONE);
})); }));
reviewPager.setCurrentItem(0); reviewPager.setCurrentItem(0);
updateCategories(media.getCategories());
}
private void updateCategories(ArrayList<String> categories) {
reviewController.onCategoriesRefreshed(categories);
reviewPagerAdapter.updateCategories();
} }
public void swipeToNext() { public void swipeToNext() {

89
app/src/main/java/fr/free/nrw/commons/review/ReviewController.java
View file

@ -7,17 +7,16 @@ import android.content.Context;
import android.view.Gravity; import android.view.Gravity;
import android.widget.Toast; import android.widget.Toast;
import org.wikipedia.dataclient.mwapi.MwQueryPage; import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.core.app.NotificationCompat;
import java.util.ArrayList; import org.wikipedia.dataclient.mwapi.MwQueryPage;
import javax.inject.Inject; import javax.inject.Inject;
import javax.inject.Singleton; import javax.inject.Singleton;
import androidx.annotation.NonNull; import fr.free.nrw.commons.CommonsApplication;
import androidx.annotation.Nullable;
import androidx.core.app.NotificationCompat;
import androidx.viewpager.widget.ViewPager;
import fr.free.nrw.commons.Media; import fr.free.nrw.commons.Media;
import fr.free.nrw.commons.R; import fr.free.nrw.commons.R;
import fr.free.nrw.commons.auth.SessionManager; import fr.free.nrw.commons.auth.SessionManager;
@ -31,13 +30,11 @@ import timber.log.Timber;
@Singleton @Singleton
public class ReviewController { public class ReviewController {
private String fileName; private static final int NOTIFICATION_SEND_THANK = 0x102;
public static final int NOTIFICATION_SEND_THANK = 0x102; private static final int NOTIFICATION_CHECK_CATEGORY = 0x101;
protected static ArrayList<String> categories;
public static final int NOTIFICATION_CHECK_CATEGORY = 0x101;
private final DeleteHelper deleteHelper; private final DeleteHelper deleteHelper;
@Nullable @Nullable
public MwQueryPage.Revision firstRevision; // TODO: maybe we can expand this class to include fileName MwQueryPage.Revision firstRevision; // TODO: maybe we can expand this class to include fileName
@Inject @Inject
MediaWikiApi mwApi; MediaWikiApi mwApi;
@Inject @Inject
@ -46,32 +43,19 @@ public class ReviewController {
private NotificationCompat.Builder notificationBuilder; private NotificationCompat.Builder notificationBuilder;
private Media media; private Media media;
private ViewPager viewPager;
private ReviewActivity reviewActivity;
ReviewController(DeleteHelper deleteHelper, Context context) { ReviewController(DeleteHelper deleteHelper, Context context) {
this.deleteHelper = deleteHelper; this.deleteHelper = deleteHelper;
reviewActivity = (ReviewActivity) context; CommonsApplication.createNotificationChannel(context.getApplicationContext());
viewPager = ((ReviewActivity) context).reviewPager; notificationManager = (NotificationManager) context.getSystemService(Context.NOTIFICATION_SERVICE);
notificationBuilder = new NotificationCompat.Builder(context, CommonsApplication.NOTIFICATION_CHANNEL_ID_ALL);
} }
public void onImageRefreshed(String fileName) { void onImageRefreshed(Media media) {
this.fileName = fileName; this.media = media;
media = new Media("File:" + fileName);
ReviewController.categories = new ArrayList<>();
} }
public void onCategoriesRefreshed(ArrayList<String> categories) { public Media getMedia() {
ReviewController.categories = categories; return media;
}
public void swipeToNext() {
int nextPos = viewPager.getCurrentItem() + 1;
if (nextPos <= 3) {
viewPager.setCurrentItem(nextPos);
} else {
reviewActivity.runRandomizer();
}
} }
public enum DeleteReason { public enum DeleteReason {
@ -79,30 +63,32 @@ public class ReviewController {
COPYRIGHT_VIOLATION COPYRIGHT_VIOLATION
} }
public void reportSpam(@NonNull Activity activity) { void reportSpam(@NonNull Activity activity, ReviewCallback reviewCallback) {
deleteHelper.askReasonAndExecute(new Media("File:" + fileName), Timber.d("Report spam for %s", media.getFilename());
deleteHelper.askReasonAndExecute(media,
activity, activity,
activity.getResources().getString(R.string.review_spam_report_question), activity.getResources().getString(R.string.review_spam_report_question),
DeleteReason.SPAM); DeleteReason.SPAM,
reviewCallback);
} }
public void reportPossibleCopyRightViolation(@NonNull Activity activity) { void reportPossibleCopyRightViolation(@NonNull Activity activity, ReviewCallback reviewCallback) {
deleteHelper.askReasonAndExecute(new Media("File:" + fileName), Timber.d("Report spam for %s", media.getFilename());
deleteHelper.askReasonAndExecute(media,
activity, activity,
activity.getResources().getString(R.string.review_c_violation_report_question), activity.getResources().getString(R.string.review_c_violation_report_question),
DeleteReason.COPYRIGHT_VIOLATION); DeleteReason.COPYRIGHT_VIOLATION,
reviewCallback);
} }
@SuppressLint("CheckResult") @SuppressLint("CheckResult")
public void reportWrongCategory(@NonNull Activity activity) { void reportWrongCategory(@NonNull Activity activity, ReviewCallback reviewCallback) {
Context context = activity.getApplicationContext(); Context context = activity.getApplicationContext();
ApplicationlessInjection ApplicationlessInjection
.getInstance(context) .getInstance(context)
.getCommonsApplicationComponent() .getCommonsApplicationComponent()
.inject(this); .inject(this);
notificationManager = (NotificationManager) context.getSystemService(Context.NOTIFICATION_SERVICE);
notificationBuilder = new NotificationCompat.Builder(context);
Toast toast = new Toast(context); Toast toast = new Toast(context);
toast.setGravity(Gravity.CENTER, 0, 0); toast.setGravity(Gravity.CENTER, 0, 0);
toast = Toast.makeText(context, context.getString(R.string.check_category_toast, media.getDisplayTitle()), Toast.LENGTH_SHORT); toast = Toast.makeText(context, context.getString(R.string.check_category_toast, media.getDisplayTitle()), Toast.LENGTH_SHORT);
@ -120,7 +106,8 @@ public class ReviewController {
try { try {
editToken = mwApi.getEditToken(); editToken = mwApi.getEditToken();
if (editToken.equals("+\\")) {
if (editToken == null) {
return false; return false;
} }
publishProgress(context, 1); publishProgress(context, 1);
@ -136,15 +123,17 @@ public class ReviewController {
.subscribeOn(Schedulers.io()) .subscribeOn(Schedulers.io())
.observeOn(AndroidSchedulers.mainThread()) .observeOn(AndroidSchedulers.mainThread())
.subscribe((result) -> { .subscribe((result) -> {
String message = ""; String message;
String title = ""; String title;
if (result) { if (result) {
title = context.getString(R.string.check_category_success_title); title = context.getString(R.string.check_category_success_title);
message = context.getString(R.string.check_category_success_message, media.getDisplayTitle()); message = context.getString(R.string.check_category_success_message, media.getDisplayTitle());
reviewCallback.onSuccess();
} else { } else {
title = context.getString(R.string.check_category_failure_title); title = context.getString(R.string.check_category_failure_title);
message = context.getString(R.string.check_category_failure_message, media.getDisplayTitle()); message = context.getString(R.string.check_category_failure_message, media.getDisplayTitle());
reviewCallback.onFailure();
} }
notificationBuilder.setDefaults(NotificationCompat.DEFAULT_ALL) notificationBuilder.setDefaults(NotificationCompat.DEFAULT_ALL)
@ -176,15 +165,13 @@ public class ReviewController {
notificationManager.notify(NOTIFICATION_CHECK_CATEGORY, notificationBuilder.build()); notificationManager.notify(NOTIFICATION_CHECK_CATEGORY, notificationBuilder.build());
} }
@SuppressLint("CheckResult") @SuppressLint({"CheckResult", "StringFormatInvalid"})
public void sendThanks(@NonNull Activity activity) { void sendThanks(@NonNull Activity activity) {
Context context = activity.getApplicationContext(); Context context = activity.getApplicationContext();
ApplicationlessInjection ApplicationlessInjection
.getInstance(context) .getInstance(context)
.getCommonsApplicationComponent() .getCommonsApplicationComponent()
.inject(this); .inject(this);
notificationManager = (NotificationManager) context.getSystemService(Context.NOTIFICATION_SERVICE);
notificationBuilder = new NotificationCompat.Builder(context);
Toast toast = new Toast(context); Toast toast = new Toast(context);
toast.setGravity(Gravity.CENTER, 0, 0); toast.setGravity(Gravity.CENTER, 0, 0);
toast = Toast.makeText(context, context.getString(R.string.send_thank_toast, media.getDisplayTitle()), Toast.LENGTH_SHORT); toast = Toast.makeText(context, context.getString(R.string.send_thank_toast, media.getDisplayTitle()), Toast.LENGTH_SHORT);
@ -200,7 +187,7 @@ public class ReviewController {
try { try {
editToken = mwApi.getEditToken(); editToken = mwApi.getEditToken();
if (editToken.equals("+\\")) { if (editToken == null) {
return false; return false;
} }
publishProgress(context, 1); publishProgress(context, 1);
@ -238,4 +225,10 @@ public class ReviewController {
}, Timber::e); }, Timber::e);
} }
public interface ReviewCallback {
void onSuccess();
void onFailure();
}
} }

6
app/src/main/java/fr/free/nrw/commons/review/ReviewHelper.java
View file

@ -6,6 +6,7 @@ import org.wikipedia.dataclient.mwapi.MwQueryPage;
import org.wikipedia.dataclient.mwapi.RecentChange; import org.wikipedia.dataclient.mwapi.RecentChange;
import org.wikipedia.util.DateUtil; import org.wikipedia.util.DateUtil;
import java.util.Collections;
import java.util.Date; import java.util.Date;
import java.util.Random; import java.util.Random;
@ -53,7 +54,7 @@ public class ReviewHelper {
return reviewInterface.getRecentChanges(rcStart) return reviewInterface.getRecentChanges(rcStart)
.map(mwQueryResponse -> mwQueryResponse.query().getRecentChanges()) .map(mwQueryResponse -> mwQueryResponse.query().getRecentChanges())
.map(recentChanges -> { .map(recentChanges -> {
//Collections.shuffle(recentChanges); Collections.shuffle(recentChanges);
return recentChanges; return recentChanges;
}) })
.flatMapIterable(changes -> changes) .flatMapIterable(changes -> changes)
@ -113,7 +114,8 @@ public class ReviewHelper {
* @return * @return
*/ */
private boolean isChangeReviewable(RecentChange recentChange) { private boolean isChangeReviewable(RecentChange recentChange) {
if (recentChange.getType().equals("log") && !(recentChange.getOldRevisionId() == 0)) { if ((recentChange.getType().equals("log") && !(recentChange.getOldRevisionId() == 0))
|| !recentChange.getType().equals("log")) {
return false; return false;
} }

96
app/src/main/java/fr/free/nrw/commons/review/ReviewImageFragment.java
View file

@ -10,26 +10,26 @@ import android.view.ViewGroup;
import android.widget.Button; import android.widget.Button;
import android.widget.ProgressBar; import android.widget.ProgressBar;
import android.widget.TextView; import android.widget.TextView;
import java.util.List;
import butterknife.BindView; import butterknife.BindView;
import butterknife.ButterKnife; import butterknife.ButterKnife;
import butterknife.OnClick; import butterknife.OnClick;
import fr.free.nrw.commons.Media;
import fr.free.nrw.commons.R; import fr.free.nrw.commons.R;
import fr.free.nrw.commons.di.CommonsDaggerSupportFragment; import fr.free.nrw.commons.di.CommonsDaggerSupportFragment;
import org.wikipedia.dataclient.mwapi.MwQueryPage;
public class ReviewImageFragment extends CommonsDaggerSupportFragment { public class ReviewImageFragment extends CommonsDaggerSupportFragment {
public static final int SPAM = 0; static final int CATEGORY = 2;
public static final int COPYRIGHT = 1; private static final int SPAM = 0;
public static final int CATEGORY = 2; private static final int COPYRIGHT = 1;
public static final int THANKS = 3; private static final int THANKS = 3;
private int position; private int position;
private String fileName;
private String catString;
public ProgressBar progressBar; public ProgressBar progressBar;
private MwQueryPage.Revision revision;
@BindView(R.id.tv_review_question) @BindView(R.id.tv_review_question)
TextView textViewQuestion; TextView textViewQuestion;
@ -40,24 +40,21 @@ public class ReviewImageFragment extends CommonsDaggerSupportFragment {
@BindView(R.id.button_no) @BindView(R.id.button_no)
Button noButton; Button noButton;
public void update(int position) {
public void update(int position, String fileName) {
this.position = position; this.position = position;
this.fileName = fileName;
} }
public void updateCategories(Iterable<String> categories) { private String updateCategoriesQuestion() {
if (categories != null && isAdded()) { Media media = getReviewActivity().getMedia();
catString = TextUtils.join(", ", categories); if (media != null && media.getCategories() != null && isAdded()) {
String catString = TextUtils.join(", ", media.getCategories());
if (catString != null && !catString.equals("") && textViewQuestionContext != null) { if (catString != null && !catString.equals("") && textViewQuestionContext != null) {
catString = "<b>" + catString + "</b>"; catString = "<b>" + catString + "</b>";
String stringToConvertHtml = String.format(getResources().getString(R.string.review_category_explanation), catString); String stringToConvertHtml = String.format(getResources().getString(R.string.review_category_explanation), catString);
textViewQuestionContext.setText(Html.fromHtml(stringToConvertHtml)); return Html.fromHtml(stringToConvertHtml).toString();
} else if (textViewQuestionContext != null) {
textViewQuestionContext.setText(getResources().getString(R.string.review_no_category));
} }
} }
return getResources().getString(R.string.review_no_category);
} }
@Override @Override
@ -75,29 +72,34 @@ public class ReviewImageFragment extends CommonsDaggerSupportFragment {
String question, explanation, yesButtonText, noButtonText; String question, explanation, yesButtonText, noButtonText;
switch (position) { switch (position) {
case COPYRIGHT:
question = getString(R.string.review_copyright);
explanation = getString(R.string.review_copyright_explanation);
yesButtonText = getString(R.string.review_copyright_yes_button_text);
noButtonText = getString(R.string.review_copyright_no_button_text);
yesButton.setOnClickListener(view -> getReviewActivity().reviewController.reportPossibleCopyRightViolation(requireActivity()));
break;
case CATEGORY:
question = getString(R.string.review_category);
explanation = getString(R.string.review_no_category);
yesButtonText = getString(R.string.review_category_yes_button_text);
noButtonText = getString(R.string.review_category_no_button_text);
yesButton.setOnClickListener(view -> {
getReviewActivity().reviewController.reportWrongCategory(requireActivity());
getReviewActivity().swipeToNext();
});
break;
case SPAM: case SPAM:
question = getString(R.string.review_spam); question = getString(R.string.review_spam);
explanation = getString(R.string.review_spam_explanation); explanation = getString(R.string.review_spam_explanation);
yesButtonText = getString(R.string.review_spam_yes_button_text); yesButtonText = getString(R.string.review_spam_yes_button_text);
noButtonText = getString(R.string.review_spam_no_button_text); noButtonText = getString(R.string.review_spam_no_button_text);
yesButton.setOnClickListener(view -> getReviewActivity().reviewController.reportSpam(requireActivity())); yesButton.setOnClickListener(view -> getReviewActivity()
.reviewController.reportSpam(requireActivity(), getReviewCallback()));
break;
case COPYRIGHT:
question = getString(R.string.review_copyright);
explanation = getString(R.string.review_copyright_explanation);
yesButtonText = getString(R.string.review_copyright_yes_button_text);
noButtonText = getString(R.string.review_copyright_no_button_text);
yesButton.setOnClickListener(view -> getReviewActivity()
.reviewController
.reportPossibleCopyRightViolation(requireActivity(), getReviewCallback()));
break;
case CATEGORY:
question = getString(R.string.review_category);
explanation = updateCategoriesQuestion();
yesButtonText = getString(R.string.review_category_yes_button_text);
noButtonText = getString(R.string.review_category_no_button_text);
yesButton.setOnClickListener(view -> {
getReviewActivity()
.reviewController
.reportWrongCategory(requireActivity(), getReviewCallback());
getReviewActivity().swipeToNext();
});
break; break;
case THANKS: case THANKS:
question = getString(R.string.review_thanks); question = getString(R.string.review_thanks);
@ -122,16 +124,26 @@ public class ReviewImageFragment extends CommonsDaggerSupportFragment {
textViewQuestionContext.setText(explanation); textViewQuestionContext.setText(explanation);
yesButton.setText(yesButtonText); yesButton.setText(yesButtonText);
noButton.setText(noButtonText); noButton.setText(noButtonText);
if (position == CATEGORY) {
updateCategories(ReviewController.categories);
}
return layoutView; return layoutView;
} }
private ReviewController.ReviewCallback getReviewCallback() {
return new ReviewController
.ReviewCallback() {
@Override
public void onSuccess() {
getReviewActivity().runRandomizer();
}
@Override
public void onFailure() {
//do nothing
}
};
}
@OnClick(R.id.button_no) @OnClick(R.id.button_no)
public void onNoButtonClicked() { void onNoButtonClicked() {
getReviewActivity().swipeToNext(); getReviewActivity().swipeToNext();
} }

15
app/src/main/java/fr/free/nrw/commons/review/ReviewPagerAdapter.java
View file

@ -6,11 +6,13 @@ import androidx.fragment.app.Fragment;
import androidx.fragment.app.FragmentManager; import androidx.fragment.app.FragmentManager;
import androidx.fragment.app.FragmentStatePagerAdapter; import androidx.fragment.app.FragmentStatePagerAdapter;
import java.util.List;
public class ReviewPagerAdapter extends FragmentStatePagerAdapter { public class ReviewPagerAdapter extends FragmentStatePagerAdapter {
ReviewImageFragment[] reviewImageFragments; private ReviewImageFragment[] reviewImageFragments;
public ReviewPagerAdapter(FragmentManager fm) { ReviewPagerAdapter(FragmentManager fm) {
super(fm); super(fm);
reviewImageFragments = new ReviewImageFragment[]{ reviewImageFragments = new ReviewImageFragment[]{
new ReviewImageFragment(), new ReviewImageFragment(),
@ -25,18 +27,13 @@ public class ReviewPagerAdapter extends FragmentStatePagerAdapter {
return reviewImageFragments.length; return reviewImageFragments.length;
} }
public void updateFileInformation(String fileName) { void updateFileInformation() {
for (int i = 0; i < getCount(); i++) { for (int i = 0; i < getCount(); i++) {
ReviewImageFragment fragment = reviewImageFragments[i]; ReviewImageFragment fragment = reviewImageFragments[i];
fragment.update(i, fileName); fragment.update(i);
} }
} }
public void updateCategories() {
ReviewImageFragment categoryFragment = reviewImageFragments[ReviewImageFragment.CATEGORY];
categoryFragment.updateCategories(ReviewController.categories);
}
@Override @Override
public Fragment getItem(int position) { public Fragment getItem(int position) {
Bundle bundle = new Bundle(); Bundle bundle = new Bundle();

1
app/src/main/res/values/strings.xml
View file

@ -558,5 +558,6 @@ Upload your first media by tapping on the add button.</string>
<string name="upload_cancelled">Cancelled Upload</string> <string name="upload_cancelled">Cancelled Upload</string>
<string name="previous_image_title_description_not_found">There is no data for previous image\'s title or description</string> <string name="previous_image_title_description_not_found">There is no data for previous image\'s title or description</string>
<string name="dialog_box_text_nomination">Why should %1$s be deleted?</string> <string name="dialog_box_text_nomination">Why should %1$s be deleted?</string>
<string name="review_is_uploaded_by">%1$s is uploaded by: %2$s</string>
<string name="default_description_language">Default description language</string> <string name="default_description_language">Default description language</string>
</resources> </resources>

6
app/src/test/kotlin/fr/free/nrw/commons/review/ReviewHelperTest.kt
View file

@ -48,9 +48,9 @@ class ReviewHelperTest {
`when`(mockRevision.user).thenReturn("TestUser") `when`(mockRevision.user).thenReturn("TestUser")
`when`(mwQueryPage.revisions()).thenReturn(listOf(mockRevision)) `when`(mwQueryPage.revisions()).thenReturn(listOf(mockRevision))
val recentChange = getMockRecentChange("test", "File:Test1.jpeg", 0) val recentChange = getMockRecentChange("log", "File:Test1.jpeg", 0)
val recentChange1 = getMockRecentChange("test", "File:Test2.png", 0) val recentChange1 = getMockRecentChange("log", "File:Test2.png", 0)
val recentChange2 = getMockRecentChange("test", "File:Test3.jpg", 0) val recentChange2 = getMockRecentChange("log", "File:Test3.jpg", 0)
val mwQueryResult = mock(MwQueryResult::class.java) val mwQueryResult = mock(MwQueryResult::class.java)
`when`(mwQueryResult.recentChanges).thenReturn(listOf(recentChange, recentChange1, recentChange2)) `when`(mwQueryResult.recentChanges).thenReturn(listOf(recentChange, recentChange1, recentChange2))
`when`(mwQueryResult.firstPage()).thenReturn(mwQueryPage) `when`(mwQueryResult.firstPage()).thenReturn(mwQueryPage)