diff --git a/CHANGELOG.md b/CHANGELOG.md index 60562d751..87f4a9cfe 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,13 @@ # Wikimedia Commons for Android +## v2.8.3 +- Fixed issues with session tokens not being cleared in 2FA, which should reduce p18 edit failures as well +- Fixed crash caused by bug in fetching revert count +- Fixed crash potentially caused by Traceur library + +## v2.8.2 +- Fixed bug with uploads sent via Share being given .jpeg extensions and overwriting files of the same name + ## v2.8.1 - Fixed bug with category edits not being sent to server diff --git a/app/build.gradle b/app/build.gradle index 4a395787c..994710fda 100644 --- a/app/build.gradle +++ b/app/build.gradle @@ -59,6 +59,8 @@ dependencies { testImplementation 'com.squareup.okhttp3:mockwebserver:3.8.1' implementation 'com.dinuscxj:circleprogressbar:1.1.1' + implementation 'com.tspoon.traceur:traceur:1.0.1' + androidTestImplementation "org.jetbrains.kotlin:kotlin-stdlib-jre7:$kotlin_version" androidTestImplementation 'com.squareup.okhttp3:mockwebserver:3.8.1' androidTestImplementation "com.android.support:support-annotations:$SUPPORT_LIB_VERSION" @@ -83,8 +85,8 @@ android { defaultConfig { applicationId 'fr.free.nrw.commons' - versionCode 88 - versionName '2.8.1' + versionCode 90 + versionName '2.8.3' setProperty("archivesBaseName", "app-commons-v$versionName-" + getBranchName()) minSdkVersion project.minSdkVersion diff --git a/app/src/main/java/fr/free/nrw/commons/CommonsApplication.java b/app/src/main/java/fr/free/nrw/commons/CommonsApplication.java index 2ee5f305f..e2c7f367b 100644 --- a/app/src/main/java/fr/free/nrw/commons/CommonsApplication.java +++ b/app/src/main/java/fr/free/nrw/commons/CommonsApplication.java @@ -1,5 +1,6 @@ package fr.free.nrw.commons; +import android.annotation.SuppressLint; import android.app.Application; import android.app.NotificationChannel; import android.app.NotificationManager; @@ -14,6 +15,8 @@ import com.facebook.imagepipeline.core.ImagePipelineConfig; import com.facebook.stetho.Stetho; import com.squareup.leakcanary.LeakCanary; import com.squareup.leakcanary.RefWatcher; +import com.tspoon.traceur.Traceur; +import com.tspoon.traceur.TraceurConfig; import org.acra.ACRA; import org.acra.ReportingInteractionMode; @@ -84,6 +87,12 @@ public class CommonsApplication extends Application { @Override public void onCreate() { super.onCreate(); + if (BuildConfig.DEBUG) { + //FIXME: Traceur should be disabled for release builds until error fixed + //See https://github.com/commons-app/apps-android-commons/issues/1877 + Traceur.enableLogging(); + } + ApplicationlessInjection .getInstance(this) .getCommonsApplicationComponent() @@ -152,6 +161,7 @@ public class CommonsApplication extends Application { * @param context Application context * @param logoutListener Implementation of interface LogoutListener */ + @SuppressLint("CheckResult") public void clearApplicationData(Context context, LogoutListener logoutListener) { File cacheDirectory = context.getCacheDir(); File applicationDirectory = new File(cacheDirectory.getParent()); @@ -164,7 +174,7 @@ public class CommonsApplication extends Application { } } - sessionManager.clearAllAccounts() + sessionManager.logout() .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) .subscribe(() -> { @@ -175,7 +185,6 @@ public class CommonsApplication extends Application { applicationPrefs.edit().putBoolean("firstrun", false).apply(); otherPrefs.edit().clear().apply(); updateAllDatabases(); - logoutListener.onLogoutComplete(); }); } diff --git a/app/src/main/java/fr/free/nrw/commons/Utils.java b/app/src/main/java/fr/free/nrw/commons/Utils.java index 4e4b46b01..5f829abfc 100644 --- a/app/src/main/java/fr/free/nrw/commons/Utils.java +++ b/app/src/main/java/fr/free/nrw/commons/Utils.java @@ -111,7 +111,7 @@ public class Utils { } /** - * Fixing incorrect extension + * Adds extension to filename. Converts to .jpg if system provides .jpeg, adds .jpg if no extension detected * @param title File name * @param extension Correct extension * @return File with correct extension @@ -128,6 +128,15 @@ public class Utils { .endsWith("." + extension.toLowerCase(Locale.ENGLISH))) { title += "." + extension; } + + // If extension is still null, make it jpg. (Hotfix for https://github.com/commons-app/apps-android-commons/issues/228) + // If title has an extension in it, if won't be true + // FIXME: .png uploads fail when uploaded via Share + if (extension == null && title.lastIndexOf(".")<=0) { + extension = "jpg"; + title += "." + extension; + } + return title; } diff --git a/app/src/main/java/fr/free/nrw/commons/achievements/Achievements.java b/app/src/main/java/fr/free/nrw/commons/achievements/Achievements.java index 5fac8da9a..98fe099da 100644 --- a/app/src/main/java/fr/free/nrw/commons/achievements/Achievements.java +++ b/app/src/main/java/fr/free/nrw/commons/achievements/Achievements.java @@ -1,7 +1,5 @@ package fr.free.nrw.commons.achievements; -import android.util.Log; - /** * represnts Achievements class ans stores all the parameters */ @@ -45,62 +43,20 @@ public class Achievements { } /** - * Builder class for Achievements class + * Get Achievements object from FeedbackResponse + * + * @param response + * @return */ - public class AchievementsBuilder { - private int nestedUniqueUsedImages; - private int nestedArticlesUsingImages; - private int nestedThanksReceived; - private int nestedImagesEditedBySomeoneElse; - private int nestedFeaturedImages; - private int nestedImagesUploaded; - private int nestedRevertCount; - - public AchievementsBuilder setUniqueUsedImages(int uniqueUsedImages) { - this.nestedUniqueUsedImages = uniqueUsedImages; - return this; - } - - public AchievementsBuilder setArticlesUsingImages(int articlesUsingImages) { - this.nestedArticlesUsingImages = articlesUsingImages; - return this; - } - - public AchievementsBuilder setThanksReceived(int thanksReceived) { - this.nestedThanksReceived = thanksReceived; - return this; - } - - public AchievementsBuilder setImagesEditedBySomeoneElse(int imagesEditedBySomeoneElse) { - this.nestedImagesEditedBySomeoneElse = imagesEditedBySomeoneElse; - return this; - } - - public AchievementsBuilder setFeaturedImages(int featuredImages) { - this.nestedFeaturedImages = featuredImages; - return this; - } - - public AchievementsBuilder setImagesUploaded(int imagesUploaded) { - this.nestedImagesUploaded = imagesUploaded; - return this; - } - - public AchievementsBuilder setRevertCount( int revertCount){ - this.nestedRevertCount = revertCount; - return this; - } - - public Achievements createAchievements(){ - return new Achievements(nestedUniqueUsedImages, - nestedArticlesUsingImages, - nestedThanksReceived, - nestedImagesEditedBySomeoneElse, - nestedFeaturedImages, - nestedImagesUploaded, - nestedRevertCount); - } - + public static Achievements from(FeedbackResponse response) { + return new Achievements(response.getUniqueUsedImages(), + response.getArticlesUsingImages(), + response.getThanksReceived(), + response.getImagesEditedBySomeoneElse(), + response.getFeaturedImages().getQualityImages() + + response.getFeaturedImages().getFeaturedPicturesOnWikimediaCommons(), + 0, + response.getDeletedUploads()); } /** diff --git a/app/src/main/java/fr/free/nrw/commons/achievements/AchievementsActivity.java b/app/src/main/java/fr/free/nrw/commons/achievements/AchievementsActivity.java index b894bac29..73143fb80 100644 --- a/app/src/main/java/fr/free/nrw/commons/achievements/AchievementsActivity.java +++ b/app/src/main/java/fr/free/nrw/commons/achievements/AchievementsActivity.java @@ -2,16 +2,9 @@ package fr.free.nrw.commons.achievements; import android.accounts.Account; import android.annotation.SuppressLint; -import android.annotation.TargetApi; import android.content.Context; -import android.content.DialogInterface; import android.content.Intent; import android.graphics.Bitmap; -import android.graphics.BitmapFactory; -import android.graphics.Canvas; -import android.graphics.Color; -import android.graphics.Paint; -import android.graphics.Rect; import android.graphics.drawable.BitmapDrawable; import android.graphics.drawable.Drawable; import android.net.Uri; @@ -20,7 +13,6 @@ import android.support.v4.content.res.ResourcesCompat; import android.support.v7.app.AlertDialog; import android.support.v7.widget.Toolbar; import android.util.DisplayMetrics; -import android.util.Log; import android.view.ContextThemeWrapper; import android.view.LayoutInflater; import android.view.Menu; @@ -34,13 +26,10 @@ import android.widget.TextView; import com.dinuscxj.progressbar.CircleProgressBar; -import org.json.JSONException; -import org.json.JSONObject; - import java.io.File; import java.io.FileOutputStream; import java.io.IOException; -import java.util.Optional; +import java.util.Objects; import javax.inject.Inject; @@ -65,10 +54,7 @@ public class AchievementsActivity extends NavigationBaseActivity { private static final double BADGE_IMAGE_WIDTH_RATIO = 0.4; private static final double BADGE_IMAGE_HEIGHT_RATIO = 0.3; - private Boolean isUploadFetched = false; - private Boolean isStatisticsFetched = false; - private Boolean isRevertFetched = false; - private Achievements achievements = new Achievements(); + private LevelController.LevelInfo levelInfo; @BindView(R.id.achievement_badge) @@ -140,8 +126,6 @@ public class AchievementsActivity extends NavigationBaseActivity { progressBar.setVisibility(View.VISIBLE); hideLayouts(); setAchievements(); - setUploadCount(); - setRevertCount(); initDrawer(); } @@ -203,60 +187,48 @@ public class AchievementsActivity extends NavigationBaseActivity { private void setAchievements() { if(checkAccount()) { compositeDisposable.add(mediaWikiApi - .getAchievements(sessionManager.getCurrentAccount().name) + .getAchievements(Objects.requireNonNull(sessionManager.getCurrentAccount()).name) .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) .subscribe( - jsonObject -> parseJson(jsonObject), - t -> Timber.e(t, "Fetching achievements statisticss failed") + response -> { + if (response != null) { + setUploadCount(Achievements.from(response)); + } else { + onError(); + } + }, + t -> { + Timber.e(t, "Fetching achievements statistics failed"); + onError(); + } )); } } /** - * To call the API to get reverts count in form of JSONObject - * + * Shows a generic error toast when error occurs while loading achievements or uploads */ - - private void setRevertCount(){ - if(checkAccount()) { - compositeDisposable.add(mediaWikiApi - .getRevertRespObjectSingle(sessionManager.getCurrentAccount().name) - .subscribeOn(Schedulers.io()) - .observeOn(AndroidSchedulers.mainThread()) - .subscribe( - object -> parseJsonRevertCount(object), - t -> Timber.e(t, "Fetching revert count failed") - )); - } - } - - /** - * used to set number of deleted images - * @param object - */ - private void parseJsonRevertCount(JSONObject object){ - try { - achievements.setRevertCount(object.getInt("deletedUploads")); - } catch (JSONException e) { - Timber.d( e, e.getMessage()); - } - isRevertFetched = true; - hideProgressBar(); + private void onError() { + ViewUtil.showLongToast(this, getResources().getString(R.string.error_occurred)); + progressBar.setVisibility(View.GONE); } /** * used to the count of images uploaded by user */ - private void setUploadCount() { + private void setUploadCount(Achievements achievements) { if(checkAccount()) { compositeDisposable.add(mediaWikiApi - .getUploadCount(sessionManager.getCurrentAccount().name) + .getUploadCount(Objects.requireNonNull(sessionManager.getCurrentAccount()).name) .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) .subscribe( - uploadCount -> setAchievementsUploadCount(uploadCount), - t -> Timber.e(t, "Fetching upload count failed") + uploadCount -> setAchievementsUploadCount(achievements, uploadCount), + t -> { + Timber.e(t, "Fetching upload count failed"); + onError(); + } )); } } @@ -265,10 +237,9 @@ public class AchievementsActivity extends NavigationBaseActivity { * used to set achievements upload count and call hideProgressbar * @param uploadCount */ - private void setAchievementsUploadCount(int uploadCount){ + private void setAchievementsUploadCount(Achievements achievements, int uploadCount) { achievements.setImagesUploaded(uploadCount); - isUploadFetched = true; - hideProgressBar(); + hideProgressBar(achievements); } /** @@ -293,33 +264,12 @@ public class AchievementsActivity extends NavigationBaseActivity { imagesRevertLimitText.setText(getResources().getString(R.string.achievements_revert_limit_message)+ levelInfo.getMinNonRevertPercentage() + "%"); } - /** - * used to parse the JSONObject containing results - * @param object - */ - private void parseJson(JSONObject object) { - try { - achievements.setUniqueUsedImages(object.getInt("uniqueUsedImages")); - achievements.setArticlesUsingImages(object.getInt("articlesUsingImages")); - achievements.setThanksReceived(object.getInt("thanksReceived")); - achievements.setImagesEditedBySomeoneElse(object.getInt("imagesEditedBySomeoneElse")); - JSONObject featuredImages = object.getJSONObject("featuredImages"); - achievements.setFeaturedImages - (featuredImages.getInt("Quality_images") + - featuredImages.getInt("Featured_pictures_on_Wikimedia_Commons")); - } catch (JSONException e) { - e.printStackTrace(); - } - isStatisticsFetched = true; - hideProgressBar(); - } - /** * Used the inflate the fetched statistics of the images uploaded by user * and assign badge and level * @param achievements */ - private void inflateAchievements(Achievements achievements ){ + private void inflateAchievements(Achievements achievements) { thanksReceived.setText(Integer.toString(achievements.getThanksReceived())); imagesUsedByWikiProgessbar.setProgress (100*achievements.getUniqueUsedImages()/levelInfo.getMaxUniqueImages() ); @@ -350,8 +300,8 @@ public class AchievementsActivity extends NavigationBaseActivity { /** * to hide progressbar */ - private void hideProgressBar() { - if (progressBar != null && isUploadFetched && isStatisticsFetched && isRevertFetched) { + private void hideProgressBar(Achievements achievements) { + if (progressBar != null) { levelInfo = LevelController.LevelInfo.from(achievements.getImagesUploaded(), achievements.getUniqueUsedImages(), achievements.getNotRevertPercentage()); @@ -393,17 +343,8 @@ public class AchievementsActivity extends NavigationBaseActivity { TextView shareMessage = (TextView) view.findViewById(R.id.alert_text); shareMessage.setText(R.string.achievements_share_message); alertadd.setView(view); - alertadd.setPositiveButton(R.string.about_translate_proceed, new DialogInterface.OnClickListener() { - public void onClick(DialogInterface dialog, int which) { - shareScreen(screenshot); - } - }); - alertadd.setNegativeButton(android.R.string.cancel, new DialogInterface.OnClickListener() { - @Override - public void onClick(DialogInterface dialog, int which) { - dialog.cancel(); - } - }); + alertadd.setPositiveButton(R.string.about_translate_proceed, (dialog, which) -> shareScreen(screenshot)); + alertadd.setNegativeButton(android.R.string.cancel, (dialog, which) -> dialog.cancel()); alertadd.show(); } diff --git a/app/src/main/java/fr/free/nrw/commons/achievements/FeaturedImages.java b/app/src/main/java/fr/free/nrw/commons/achievements/FeaturedImages.java new file mode 100644 index 000000000..8b9992e28 --- /dev/null +++ b/app/src/main/java/fr/free/nrw/commons/achievements/FeaturedImages.java @@ -0,0 +1,25 @@ +package fr.free.nrw.commons.achievements; + +import com.google.gson.annotations.SerializedName; + +public class FeaturedImages { + + @SerializedName("Quality_images") + private final int qualityImages; + + @SerializedName("Featured_pictures_on_Wikimedia_Commons") + private final int featuredPicturesOnWikimediaCommons; + + public FeaturedImages(int qualityImages, int featuredPicturesOnWikimediaCommons) { + this.qualityImages = qualityImages; + this.featuredPicturesOnWikimediaCommons = featuredPicturesOnWikimediaCommons; + } + + public int getQualityImages() { + return qualityImages; + } + + public int getFeaturedPicturesOnWikimediaCommons() { + return featuredPicturesOnWikimediaCommons; + } +} \ No newline at end of file diff --git a/app/src/main/java/fr/free/nrw/commons/achievements/FeedbackResponse.java b/app/src/main/java/fr/free/nrw/commons/achievements/FeedbackResponse.java new file mode 100644 index 000000000..37fea3e2d --- /dev/null +++ b/app/src/main/java/fr/free/nrw/commons/achievements/FeedbackResponse.java @@ -0,0 +1,64 @@ +package fr.free.nrw.commons.achievements; + +public class FeedbackResponse { + + private final String status; + private final int uniqueUsedImages; + private final int articlesUsingImages; + private final int deletedUploads; + private final FeaturedImages featuredImages; + private final int thanksReceived; + private final String user; + private final int imagesEditedBySomeoneElse; + + + public FeedbackResponse(String status, + int uniqueUsedImages, + int articlesUsingImages, + int deletedUploads, + FeaturedImages featuredImages, + int thanksReceived, + String user, + int imagesEditedBySomeoneElse) { + this.status = status; + this.uniqueUsedImages = uniqueUsedImages; + this.articlesUsingImages = articlesUsingImages; + this.deletedUploads = deletedUploads; + this.featuredImages = featuredImages; + this.thanksReceived = thanksReceived; + this.user = user; + this.imagesEditedBySomeoneElse = imagesEditedBySomeoneElse; + } + + public String getStatus() { + return status; + } + + public int getUniqueUsedImages() { + return uniqueUsedImages; + } + + public int getArticlesUsingImages() { + return articlesUsingImages; + } + + public int getDeletedUploads() { + return deletedUploads; + } + + public FeaturedImages getFeaturedImages() { + return featuredImages; + } + + public int getThanksReceived() { + return thanksReceived; + } + + public String getUser() { + return user; + } + + public int getImagesEditedBySomeoneElse() { + return imagesEditedBySomeoneElse; + } +} \ No newline at end of file diff --git a/app/src/main/java/fr/free/nrw/commons/auth/SessionManager.java b/app/src/main/java/fr/free/nrw/commons/auth/SessionManager.java index cd23d1282..e8745e25b 100644 --- a/app/src/main/java/fr/free/nrw/commons/auth/SessionManager.java +++ b/app/src/main/java/fr/free/nrw/commons/auth/SessionManager.java @@ -153,11 +153,19 @@ public class SessionManager { } } - public Completable clearAllAccounts() { + /** + * 1. Clears existing accounts from account manager + * 2. Calls MediaWikiApi's logout function to clear cookies + * @return + */ + public Completable logout() { AccountManager accountManager = AccountManager.get(context); Account[] allAccounts = accountManager.getAccountsByType(BuildConfig.ACCOUNT_TYPE); return Completable.fromObservable(Observable.fromArray(allAccounts) .map(a -> accountManager.removeAccount(a, null, null).getResult())) - .doOnComplete(() -> currentAccount = null); + .doOnComplete(() -> { + mediaWikiApi.logout(); + currentAccount = null; + }); } } diff --git a/app/src/main/java/fr/free/nrw/commons/di/CommonsApplicationModule.java b/app/src/main/java/fr/free/nrw/commons/di/CommonsApplicationModule.java index 99db50d0a..d59cf5f64 100644 --- a/app/src/main/java/fr/free/nrw/commons/di/CommonsApplicationModule.java +++ b/app/src/main/java/fr/free/nrw/commons/di/CommonsApplicationModule.java @@ -21,6 +21,7 @@ import fr.free.nrw.commons.nearby.NearbyPlaces; import fr.free.nrw.commons.upload.UploadController; import fr.free.nrw.commons.wikidata.WikidataEditListener; import fr.free.nrw.commons.wikidata.WikidataEditListenerImpl; +import okhttp3.OkHttpClient; import static android.content.Context.MODE_PRIVATE; diff --git a/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.java b/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.java index cd043e950..953b7ecee 100644 --- a/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.java +++ b/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.java @@ -7,6 +7,8 @@ import android.support.annotation.NonNull; import com.google.gson.Gson; import com.google.gson.GsonBuilder; +import java.io.File; + import javax.inject.Named; import javax.inject.Singleton; @@ -15,6 +17,7 @@ import dagger.Provides; import fr.free.nrw.commons.BuildConfig; import fr.free.nrw.commons.mwapi.ApacheHttpClientMediaWikiApi; import fr.free.nrw.commons.mwapi.MediaWikiApi; +import okhttp3.Cache; import okhttp3.HttpUrl; import okhttp3.OkHttpClient; @@ -25,8 +28,11 @@ public class NetworkingModule { @Provides @Singleton - public OkHttpClient provideOkHttpClient() { - return new OkHttpClient.Builder().build(); + public OkHttpClient provideOkHttpClient(Context context) { + File dir = new File(context.getCacheDir(), "okHttpCache"); + return new OkHttpClient.Builder() + .cache(new Cache(dir, OK_HTTP_CACHE_SIZE)) + .build(); } @Provides @@ -34,8 +40,9 @@ public class NetworkingModule { public MediaWikiApi provideMediaWikiApi(Context context, @Named("default_preferences") SharedPreferences defaultPreferences, @Named("category_prefs") SharedPreferences categoryPrefs, - Gson gson) { - return new ApacheHttpClientMediaWikiApi(context, BuildConfig.WIKIMEDIA_API_HOST, BuildConfig.WIKIDATA_API_HOST, defaultPreferences, categoryPrefs, gson); + Gson gson, + OkHttpClient okHttpClient) { + return new ApacheHttpClientMediaWikiApi(context, BuildConfig.WIKIMEDIA_API_HOST, BuildConfig.WIKIDATA_API_HOST, defaultPreferences, categoryPrefs, gson, okHttpClient); } @Provides diff --git a/app/src/main/java/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApi.java b/app/src/main/java/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApi.java index 12b8972ff..3218ccf18 100644 --- a/app/src/main/java/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApi.java +++ b/app/src/main/java/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApi.java @@ -24,7 +24,6 @@ import org.apache.http.impl.conn.tsccm.ThreadSafeClientConnManager; import org.apache.http.params.BasicHttpParams; import org.apache.http.params.CoreProtocolPNames; import org.apache.http.util.EntityUtils; -import org.json.JSONObject; import org.w3c.dom.Element; import org.w3c.dom.Node; import org.w3c.dom.NodeList; @@ -45,6 +44,7 @@ import java.util.concurrent.Callable; import fr.free.nrw.commons.BuildConfig; import fr.free.nrw.commons.Media; import fr.free.nrw.commons.PageTitle; +import fr.free.nrw.commons.achievements.FeedbackResponse; import fr.free.nrw.commons.auth.AccountUtil; import fr.free.nrw.commons.category.CategoryImageUtils; import fr.free.nrw.commons.category.QueryContinue; @@ -76,14 +76,17 @@ public class ApacheHttpClientMediaWikiApi implements MediaWikiApi { private SharedPreferences defaultPreferences; private SharedPreferences categoryPreferences; private Gson gson; + private final OkHttpClient okHttpClient; public ApacheHttpClientMediaWikiApi(Context context, String apiURL, String wikidatApiURL, SharedPreferences defaultPreferences, SharedPreferences categoryPreferences, - Gson gson) { + Gson gson, + OkHttpClient okHttpClient) { this.context = context; + this.okHttpClient = okHttpClient; BasicHttpParams params = new BasicHttpParams(); SchemeRegistry schemeRegistry = new SchemeRegistry(); schemeRegistry.register(new Scheme("http", PlainSocketFactory.getSocketFactory(), 80)); @@ -233,17 +236,27 @@ public class ApacheHttpClientMediaWikiApi implements MediaWikiApi { @Override public String getCentralAuthToken() throws IOException { - String centralAuthToken = api.action("centralauthtoken") - .get() - .getString("/api/centralauthtoken/@centralauthtoken"); + CustomApiResult result = api.action("centralauthtoken").get(); + String centralAuthToken = result.getString("/api/centralauthtoken/@centralauthtoken"); + Timber.d("MediaWiki Central auth token is %s", centralAuthToken); - if(centralAuthToken == null || centralAuthToken.isEmpty()) { + if ((centralAuthToken == null || centralAuthToken.isEmpty()) + && "notLoggedIn".equals(result.getString("api/error/@code"))) { + Timber.d("Central auth token isn't valid. Trying to fetch a fresh token"); api.removeAllCookies(); - String login = login(AccountUtil.getUserName(context), AccountUtil.getPassword(context)); - if(login.equals("PASS")) { + String loginResultCode = login(AccountUtil.getUserName(context), AccountUtil.getPassword(context)); + if(loginResultCode.equals("PASS")) { return getCentralAuthToken(); + } else if(loginResultCode.equals("2FA")) { + Timber.e("Cannot refresh session for 2FA enabled user. Login required"); + } else { + Timber.e("Error occurred in refreshing session. Error code is %s", loginResultCode); } + } else { + Timber.e("Error occurred while fetching auth token. Error code is %s and message is %s", + result.getString("api/error/@code"), + result.getString("api/error/@info")); } return centralAuthToken; } @@ -467,13 +480,12 @@ public class ApacheHttpClientMediaWikiApi implements MediaWikiApi { return false; } - Node node = result.getNode("api").getDocument(); - Element element = (Element) node; - - if (element != null && element.getAttribute("status").equals("success")) { + if ("success".equals(result.getString("api/tag/result/@status"))) { return true; } else { - Timber.e(result.getString("api/error/@code") + " " + result.getString("api/error/@info")); + Timber.e("Error occurred in creating claim. Error code is: %s and message is %s", + result.getString("api/error/@code"), + result.getString("api/error/@info")); } return false; } @@ -954,12 +966,11 @@ public class ApacheHttpClientMediaWikiApi implements MediaWikiApi { /** * This takes userName as input, which is then used to fetch the feedback/achievements * statistics using OkHttp and JavaRx. This function return JSONObject - * @param userName + * @param userName MediaWiki user name * @return */ - @NonNull @Override - public Single getAchievements(String userName) { + public Single getAchievements(String userName) { final String fetchAchievementUrlTemplate = wikiMediaToolforgeUrl + "urbanecmbot/commonsmisc/feedback.py"; return Single.fromCallable(() -> { @@ -973,46 +984,19 @@ public class ApacheHttpClientMediaWikiApi implements MediaWikiApi { Request request = new Request.Builder() .url(urlBuilder.toString()) .build(); - OkHttpClient client = new OkHttpClient(); - Response response = client.newCall(request).execute(); - String jsonData = response.body().string(); - JSONObject jsonObject = new JSONObject(jsonData); - return jsonObject; + Response response = okHttpClient.newCall(request).execute(); + if (response != null && response.body() != null && response.isSuccessful()) { + String json = response.body().string(); + if (json == null) { + return null; + } + return gson.fromJson(json, FeedbackResponse.class); + } + return null; }); } - /** - * This takes userName as input, which is then used to fetch the no of images deleted - * using OkHttp and JavaRx. This function return JSONObject - * @param userName - * @return - */ - @NonNull - @Override - public Single getRevertRespObjectSingle(String userName){ - final String fetchRevertCountUrlTemplate = - wikiMediaToolforgeUrl + "urbanecmbot/commonsmisc/feedback.py"; - return Single.fromCallable(() -> { - String url = String.format( - Locale.ENGLISH, - fetchRevertCountUrlTemplate, - new PageTitle(userName).getText()); - HttpUrl.Builder urlBuilder = HttpUrl.parse(url).newBuilder(); - urlBuilder.addQueryParameter("user", userName); - urlBuilder.addQueryParameter("fetch","deletedUploads"); - Log.i("url", urlBuilder.toString()); - Request request = new Request.Builder() - .url(urlBuilder.toString()) - .build(); - OkHttpClient client = new OkHttpClient(); - Response response = client.newCall(request).execute(); - String jsonData = response.body().string(); - JSONObject jsonRevertObject = new JSONObject(jsonData); - return jsonRevertObject; - }); - } - private Date parseMWDate(String mwDate) { SimpleDateFormat isoFormat = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss'Z'", Locale.ENGLISH); // Assuming MW always gives me UTC isoFormat.setTimeZone(TimeZone.getTimeZone("UTC")); @@ -1023,4 +1007,15 @@ public class ApacheHttpClientMediaWikiApi implements MediaWikiApi { } } + /** + * Calls media wiki's logout API + */ + public void logout() { + try { + api.logout(); + } catch (IOException e) { + Timber.e(e, "Error occurred while logging out"); + } + } + } diff --git a/app/src/main/java/fr/free/nrw/commons/mwapi/CustomMwApi.java b/app/src/main/java/fr/free/nrw/commons/mwapi/CustomMwApi.java index 1b9e93fa9..21810d865 100644 --- a/app/src/main/java/fr/free/nrw/commons/mwapi/CustomMwApi.java +++ b/app/src/main/java/fr/free/nrw/commons/mwapi/CustomMwApi.java @@ -167,6 +167,8 @@ public class CustomMwApi { // I should be doing more validation here, but meh isLoggedIn = false; this.action("logout").post(); + removeAllCookies(); + authCookie = null; } private CustomApiResult makeRequest(String method, HashMap params) throws IOException { diff --git a/app/src/main/java/fr/free/nrw/commons/mwapi/MediaWikiApi.java b/app/src/main/java/fr/free/nrw/commons/mwapi/MediaWikiApi.java index d96e9b177..805aab056 100644 --- a/app/src/main/java/fr/free/nrw/commons/mwapi/MediaWikiApi.java +++ b/app/src/main/java/fr/free/nrw/commons/mwapi/MediaWikiApi.java @@ -4,13 +4,12 @@ import android.net.Uri; import android.support.annotation.NonNull; import android.support.annotation.Nullable; -import org.json.JSONObject; - import java.io.IOException; import java.io.InputStream; import java.util.List; import fr.free.nrw.commons.Media; +import fr.free.nrw.commons.achievements.FeedbackResponse; import fr.free.nrw.commons.notification.Notification; import io.reactivex.Observable; import io.reactivex.Single; @@ -98,13 +97,11 @@ public interface MediaWikiApi { @NonNull Single getUploadCount(String userName); - @NonNull - Single getRevertRespObjectSingle(String userName); - boolean isUserBlockedFromCommons(); - @NonNull - Single getAchievements(String userName); + Single getAchievements(String userName); + + void logout(); interface ProgressListener { void onProgress(long transferred, long total); diff --git a/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.java b/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.java index abc6bf48f..654c08fa9 100644 --- a/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.java +++ b/app/src/main/java/fr/free/nrw/commons/quiz/QuizChecker.java @@ -1,11 +1,9 @@ package fr.free.nrw.commons.quiz; import android.content.Context; -import android.content.DialogInterface; import android.content.Intent; import android.content.SharedPreferences; import android.support.v7.app.AlertDialog.Builder; -import android.util.Log; import fr.free.nrw.commons.R; import fr.free.nrw.commons.WelcomeActivity; @@ -34,16 +32,16 @@ public class QuizChecker { private SharedPreferences revertPref; private SharedPreferences countPref; - private final int UPLOAD_COUNT_THRESHOLD = 5; - private final String REVERT_PERCENTAGE_FOR_MESSAGE = "50%"; + private static final int UPLOAD_COUNT_THRESHOLD = 5; + private static final String REVERT_PERCENTAGE_FOR_MESSAGE = "50%"; private final String REVERT_SHARED_PREFERENCE = "revertCount"; private final String UPLOAD_SHARED_PREFERENCE = "uploadCount"; /** * constructor to set the parameters for quiz - * @param context - * @param userName - * @param mediaWikiApi + * @param context context + * @param userName Commons user name + * @param mediaWikiApi instance of MediaWikiApi */ public QuizChecker(Context context, String userName, MediaWikiApi mediaWikiApi) { this.context = context; @@ -63,8 +61,7 @@ public class QuizChecker { .getUploadCount(userName) .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) - .subscribe( - uploadCount -> setTotalUploadCount(uploadCount), + .subscribe(this::setTotalUploadCount, t -> Timber.e(t, "Fetching upload count failed") )); } @@ -72,7 +69,7 @@ public class QuizChecker { /** * set the sub Title of Contibutions Activity and * call function to check for quiz - * @param uploadCount + * @param uploadCount user's upload count */ private void setTotalUploadCount(int uploadCount) { totalUploadCount = uploadCount - countPref.getInt(UPLOAD_SHARED_PREFERENCE,0); @@ -89,17 +86,21 @@ public class QuizChecker { */ private void setRevertCount() { compositeDisposable.add(mediaWikiApi - .getRevertRespObjectSingle(userName) + .getAchievements(userName) .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) .subscribe( - object -> setRevertParameter(object.getInt("deletedUploads")) - )); + response -> { + if (response != null) { + setRevertParameter(response.getDeletedUploads()); + } + }, throwable -> Timber.e(throwable, "Fetching feedback failed")) + ); } /** * to calculate the number of images reverted after previous quiz - * @param revertCountFetched + * @param revertCountFetched count of deleted uploads */ private void setRevertParameter(int revertCountFetched) { revertCount = revertCountFetched - revertPref.getInt(REVERT_SHARED_PREFERENCE,0); @@ -130,30 +131,22 @@ public class QuizChecker { /** * Alert which prompts to quiz */ - public void callQuiz() { + private void callQuiz() { Builder alert = new Builder(context); alert.setTitle(context.getResources().getString(R.string.quiz)); alert.setMessage(context.getResources().getString(R.string.quiz_alert_message, REVERT_PERCENTAGE_FOR_MESSAGE)); - alert.setPositiveButton(R.string.about_translate_proceed, new DialogInterface.OnClickListener() { - @Override - public void onClick(DialogInterface dialog, int which) { - int newRevetSharedPrefs = revertCount+ revertPref.getInt(REVERT_SHARED_PREFERENCE,0); - revertPref.edit().putInt(REVERT_SHARED_PREFERENCE, newRevetSharedPrefs).apply(); - int newUploadCount = totalUploadCount + countPref.getInt(UPLOAD_SHARED_PREFERENCE,0); - countPref.edit().putInt(UPLOAD_SHARED_PREFERENCE,newUploadCount).apply(); - Intent i = new Intent(context, WelcomeActivity.class); - i.putExtra("isQuiz", true); - dialog.dismiss(); - context.startActivity(i); - } - }); - alert.setNegativeButton("Cancel", new DialogInterface.OnClickListener() { - @Override - public void onClick(DialogInterface dialogInterface, int i) { - dialogInterface.cancel(); - } + alert.setPositiveButton(R.string.about_translate_proceed, (dialog, which) -> { + int newRevetSharedPrefs = revertCount + revertPref.getInt(REVERT_SHARED_PREFERENCE, 0); + revertPref.edit().putInt(REVERT_SHARED_PREFERENCE, newRevetSharedPrefs).apply(); + int newUploadCount = totalUploadCount + countPref.getInt(UPLOAD_SHARED_PREFERENCE, 0); + countPref.edit().putInt(UPLOAD_SHARED_PREFERENCE, newUploadCount).apply(); + Intent i = new Intent(context, WelcomeActivity.class); + i.putExtra("isQuiz", true); + dialog.dismiss(); + context.startActivity(i); }); + alert.setNegativeButton(android.R.string.cancel, (dialogInterface, i) -> dialogInterface.cancel()); android.support.v7.app.AlertDialog dialog = alert.create(); dialog.show(); } diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 7f5f8f4a2..061f75368 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -351,6 +351,8 @@ The number of images you have uploaded to Commons, via any upload software The percentage of images you have uploaded to Commons that were not deleted The number of images you have uploaded to Commons that were used in Wikimedia articles + + Error occurred! Commons Notification Storage Permission We need your permission to access the external storage of your device in order to upload images. diff --git a/app/src/test/kotlin/fr/free/nrw/commons/UtilsFixExtensionTest.kt b/app/src/test/kotlin/fr/free/nrw/commons/UtilsFixExtensionTest.kt index 0082b9d28..1f3b127fd 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/UtilsFixExtensionTest.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/UtilsFixExtensionTest.kt @@ -65,4 +65,14 @@ class UtilsFixExtensionTest { fun inWordJpegToJpgResultsInJpg() { assertEquals("X.jpeg.SAMPLE.jpg", fixExtension("X.jpeg.SAMPLE", "jpg")) } + + @Test + fun noExtensionShouldResultInJpg() { + assertEquals("Sample.jpg", fixExtension("Sample", null)) + } + + @Test + fun extensionAlreadyInTitleShouldRemain() { + assertEquals("Sample.jpg", fixExtension("Sample.jpg", null)) + } } diff --git a/app/src/test/kotlin/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApiTest.kt b/app/src/test/kotlin/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApiTest.kt index b070aed2a..7b6ae96d1 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApiTest.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApiTest.kt @@ -6,6 +6,7 @@ import android.preference.PreferenceManager import com.google.gson.Gson import fr.free.nrw.commons.BuildConfig import fr.free.nrw.commons.TestCommonsApplication +import okhttp3.OkHttpClient import okhttp3.mockwebserver.MockResponse import okhttp3.mockwebserver.MockWebServer import okhttp3.mockwebserver.RecordedRequest @@ -30,14 +31,16 @@ class ApacheHttpClientMediaWikiApiTest { private lateinit var wikidataServer: MockWebServer private lateinit var sharedPreferences: SharedPreferences private lateinit var categoryPreferences: SharedPreferences + private lateinit var okHttpClient: OkHttpClient @Before fun setUp() { server = MockWebServer() wikidataServer = MockWebServer() + okHttpClient = OkHttpClient() sharedPreferences = PreferenceManager.getDefaultSharedPreferences(RuntimeEnvironment.application) categoryPreferences = PreferenceManager.getDefaultSharedPreferences(RuntimeEnvironment.application) - testObject = ApacheHttpClientMediaWikiApi(RuntimeEnvironment.application, "http://" + server.hostName + ":" + server.port + "/", "http://" + wikidataServer.hostName + ":" + wikidataServer.port + "/", sharedPreferences, categoryPreferences, Gson()) + testObject = ApacheHttpClientMediaWikiApi(RuntimeEnvironment.application, "http://" + server.hostName + ":" + server.port + "/", "http://" + wikidataServer.hostName + ":" + wikidataServer.port + "/", sharedPreferences, categoryPreferences, Gson(), okHttpClient) testObject.setWikiMediaToolforgeUrl("http://" + server.hostName + ":" + server.port + "/") }