jenkins-bot has submitted this change and it was merged. Change subject: Hygiene: extremely light clean up ......................................................................
Hygiene: extremely light clean up • Replace deprecated method call. • Add missing @Override. • Remove unused namespace. No functional changes intended. Change-Id: Ieb8d172269f5f721f45823d53d9f1f4a13669e4c --- M app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java M app/src/main/java/org/wikipedia/page/PageActivity.java M app/src/main/res/layout/item_gallery_thumbnail.xml 3 files changed, 54 insertions(+), 54 deletions(-) Approvals: Mholloway: Looks good to me, approved jenkins-bot: Verified diff --git a/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java b/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java index 8a01bdd..a4f1d3c 100644 --- a/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java +++ b/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java @@ -21,7 +21,7 @@ this.password = password; } - public RequestBuilder buildRequest(Api api) { + @Override public RequestBuilder buildRequest(Api api) { // HACK: T124384 WikipediaApp.getInstance().getEditTokenStorage().clearAllTokens(); diff --git a/app/src/main/java/org/wikipedia/page/PageActivity.java b/app/src/main/java/org/wikipedia/page/PageActivity.java index 1fa1bc8..aa17648 100644 --- a/app/src/main/java/org/wikipedia/page/PageActivity.java +++ b/app/src/main/java/org/wikipedia/page/PageActivity.java @@ -1,12 +1,58 @@ package org.wikipedia.page; +import android.annotation.TargetApi; +import android.app.SearchManager; +import android.appwidget.AppWidgetManager; +import android.content.ComponentName; +import android.content.DialogInterface; +import android.content.Intent; +import android.content.res.Configuration; +import android.location.Location; +import android.net.Uri; +import android.os.Build; +import android.os.Bundle; +import android.os.Handler; +import android.os.Looper; +import android.support.annotation.NonNull; +import android.support.annotation.Nullable; +import android.support.design.widget.BottomSheetDialog; +import android.support.design.widget.BottomSheetDialogFragment; +import android.support.design.widget.NavigationView; +import android.support.design.widget.Snackbar; +import android.support.v4.app.Fragment; +import android.support.v4.app.FragmentManager; +import android.support.v4.app.FragmentTransaction; +import android.support.v4.content.ContextCompat; +import android.support.v4.view.GravityCompat; +import android.support.v4.widget.DrawerLayout; +import android.support.v7.app.ActionBarDrawerToggle; +import android.support.v7.app.AlertDialog; +import android.support.v7.preference.PreferenceManager; +import android.support.v7.view.ActionMode; +import android.support.v7.widget.Toolbar; +import android.text.Html; +import android.text.TextUtils; +import android.text.format.DateUtils; +import android.view.KeyEvent; +import android.view.Menu; +import android.view.MenuItem; +import android.view.View; +import android.view.WindowManager; +import android.widget.ProgressBar; +import android.widget.TextView; + +import com.squareup.otto.Bus; +import com.squareup.otto.Subscribe; + +import net.hockeyapp.android.metrics.MetricsManager; + import org.wikipedia.BackPressedHandler; import org.wikipedia.R; import org.wikipedia.Site; -import org.wikipedia.activity.ActivityUtil; -import org.wikipedia.activity.ThemedActionBarActivity; import org.wikipedia.ViewAnimations; import org.wikipedia.WikipediaApp; +import org.wikipedia.activity.ActivityUtil; +import org.wikipedia.activity.ThemedActionBarActivity; import org.wikipedia.analytics.IntentFunnel; import org.wikipedia.analytics.ReadingListsFunnel; import org.wikipedia.analytics.WikipediaZeroUsageFunnel; @@ -38,53 +84,8 @@ import org.wikipedia.widgets.WidgetProviderFeaturedPage; import org.wikipedia.zero.ZeroConfig; -import com.squareup.otto.Bus; -import com.squareup.otto.Subscribe; - -import android.annotation.TargetApi; -import android.app.SearchManager; -import android.appwidget.AppWidgetManager; -import android.content.ComponentName; -import android.content.DialogInterface; -import android.content.Intent; -import android.content.res.Configuration; -import android.location.Location; -import android.net.Uri; -import android.os.Build; -import android.os.Bundle; -import android.os.Handler; -import android.os.Looper; -import android.support.annotation.NonNull; -import android.support.annotation.Nullable; -import android.support.design.widget.BottomSheetDialog; -import android.support.design.widget.BottomSheetDialogFragment; -import android.support.design.widget.NavigationView; -import android.support.design.widget.Snackbar; -import android.support.v4.app.Fragment; -import android.support.v4.app.FragmentManager; -import android.support.v4.app.FragmentTransaction; -import android.support.v4.view.GravityCompat; -import android.support.v4.widget.DrawerLayout; -import android.support.v7.app.ActionBarDrawerToggle; -import android.support.v7.app.AlertDialog; -import android.support.v7.preference.PreferenceManager; -import android.support.v7.widget.Toolbar; -import android.support.v7.view.ActionMode; -import android.text.Html; -import android.text.TextUtils; -import android.text.format.DateUtils; -import android.view.KeyEvent; -import android.view.Menu; -import android.view.MenuItem; -import android.view.View; -import android.view.WindowManager; -import android.widget.ProgressBar; -import android.widget.TextView; - -import net.hockeyapp.android.metrics.MetricsManager; - -import static org.wikipedia.util.DeviceUtil.isBackKeyUp; import static org.wikipedia.util.DeviceUtil.hideSoftKeyboard; +import static org.wikipedia.util.DeviceUtil.isBackKeyUp; import static org.wikipedia.util.UriUtil.visitInExternalBrowser; public class PageActivity extends ThemedActionBarActivity { @@ -526,7 +527,7 @@ } /** - * Add a new fragment to the top of the activity's backstack, and optionally allow state loss. + * Add a new fragment to the top of the activity's backstack, and optionally allow state loss. * Useful for cases where we might push a fragment from an AsyncTask result. * @param f New fragment to place on top. * @param allowStateLoss Whether to allow state loss. @@ -791,8 +792,8 @@ if (leftZeroRatedNetwork(latestZeroEnabledState)) { app.getWikipediaZeroHandler().showZeroOffBanner(PageActivity.this, getString(R.string.zero_charged_verbiage), - getResources().getColor(R.color.holo_red_dark), - getResources().getColor(android.R.color.white)); + ContextCompat.getColor(PageActivity.this, R.color.holo_red_dark), + ContextCompat.getColor(PageActivity.this, android.R.color.white)); navDrawerHelper.setupDynamicNavDrawerItems(); } @@ -1046,4 +1047,4 @@ widgetIntent.putExtra(AppWidgetManager.EXTRA_APPWIDGET_IDS, ids); sendBroadcast(widgetIntent); } -} +} \ No newline at end of file diff --git a/app/src/main/res/layout/item_gallery_thumbnail.xml b/app/src/main/res/layout/item_gallery_thumbnail.xml index 834edb2..c214fa6 100644 --- a/app/src/main/res/layout/item_gallery_thumbnail.xml +++ b/app/src/main/res/layout/item_gallery_thumbnail.xml @@ -2,7 +2,6 @@ <com.facebook.drawee.view.SimpleDraweeView style="@style/SimpleDraweeViewPlaceholder" xmlns:android="http://schemas.android.com/apk/res/android" - xmlns:app="http://schemas.android.com/apk/res-auto" android:id="@+id/gallery_thumbnail_image" android:layout_width="@dimen/linkPreviewImageSize" android:layout_height="@dimen/linkPreviewImageSize" -- To view, visit https://gerrit.wikimedia.org/r/293214 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb8d172269f5f721f45823d53d9f1f4a13669e4c Gerrit-PatchSet: 1 Gerrit-Project: apps/android/wikipedia Gerrit-Branch: master Gerrit-Owner: Niedzielski <sniedziel...@wikimedia.org> Gerrit-Reviewer: BearND <bsitzm...@wikimedia.org> Gerrit-Reviewer: Brion VIBBER <br...@wikimedia.org> Gerrit-Reviewer: Dbrant <dbr...@wikimedia.org> Gerrit-Reviewer: Mholloway <mhollo...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits