Niedzielski has uploaded a new change for review. https://gerrit.wikimedia.org/r/315457
Change subject: Hygiene: rename Site to WikiSite ...................................................................... Hygiene: rename Site to WikiSite Move org.wikipedia.Site to org.wikipedia.dataclient.WikiSite. No user visible change intended. Site appears to have originally been intended as a generic wiki URL. If not, that's what is expected now as class assumptions around URL structure are wiki specific, especially language. This patch renames Site to WikiSite and moves it under the dataclient package to help alleviate confusion over its purpose. Site is misused to accept unsupported URLs in some cases. A future refactor should find and fix these instances. There are also some wiki corner cases that aren't fully handled around language including: meta, upload, deployment, and test. Maybe a more generic Site super class would help. Additional changes: • Use "wiki" terminology instead of "site". Retaining site nomenclature would be keeping some confusion. Where models are thought to be kept as JSON, @SerializedName is used. For volatile storage, @SerializedName is not used • Use WikiSite.forLanguageCode instead of hardcoding *.wikipedia.org • Removed always null Site member from GalleryCollection • In tests, replace a couple instances of: (WikipediaApp) getInstrumentation().getTargetContext().getApplicationContext()` With: `WikipediaApp.getInstance()` My apologies for the unhappily large diff. I recommend viewing through a few programs including icdiff and grep. Change-Id: Ic967146a77b1301c87a0e6fa552c2f70663ca2fe --- A app/src/androidTest/java/org/wikipedia/dataclient/WikiSiteTest.java M app/src/androidTest/java/org/wikipedia/editing/EditPreviewTaskTest.java M app/src/androidTest/java/org/wikipedia/page/BasePageLoadTest.java M app/src/androidTest/java/org/wikipedia/page/PageLoadTest.java M app/src/androidTest/java/org/wikipedia/test/CreateAccountTaskTest.java M app/src/androidTest/java/org/wikipedia/test/EditTaskTest.java M app/src/androidTest/java/org/wikipedia/test/FetchSectionWikitextTaskTests.java M app/src/androidTest/java/org/wikipedia/test/FullSearchTaskTests.java M app/src/androidTest/java/org/wikipedia/test/GetDescriptionsTaskTests.java M app/src/androidTest/java/org/wikipedia/test/LanguageVariantTests.java M app/src/androidTest/java/org/wikipedia/test/LoginClientTest.java M app/src/androidTest/java/org/wikipedia/test/PageTests.java M app/src/androidTest/java/org/wikipedia/test/ParcelableTest.java D app/src/androidTest/java/org/wikipedia/test/SiteTests.java M app/src/androidTest/java/org/wikipedia/test/SuggestionsTaskTests.java M app/src/main/java/org/wikipedia/LongPressHandler.java M app/src/main/java/org/wikipedia/WikipediaApp.java M app/src/main/java/org/wikipedia/analytics/AppearanceChangeFunnel.java M app/src/main/java/org/wikipedia/analytics/EditFunnel.java M app/src/main/java/org/wikipedia/analytics/FindInPageFunnel.java M app/src/main/java/org/wikipedia/analytics/Funnel.java M app/src/main/java/org/wikipedia/analytics/GalleryFunnel.java M app/src/main/java/org/wikipedia/analytics/ProtectedEditAttemptFunnel.java M app/src/main/java/org/wikipedia/analytics/ReadingListsFunnel.java M app/src/main/java/org/wikipedia/analytics/ShareAFactFunnel.java M app/src/main/java/org/wikipedia/analytics/SuggestedPagesFunnel.java M app/src/main/java/org/wikipedia/analytics/TimedFunnel.java M app/src/main/java/org/wikipedia/analytics/ToCInteractionFunnel.java M app/src/main/java/org/wikipedia/createaccount/CreateAccountActivity.java R app/src/main/java/org/wikipedia/dataclient/WikiSite.java M app/src/main/java/org/wikipedia/dataclient/retrofit/MwCachedService.java M app/src/main/java/org/wikipedia/dataclient/retrofit/RbCachedService.java M app/src/main/java/org/wikipedia/dataclient/retrofit/RetrofitFactory.java D app/src/main/java/org/wikipedia/dataclient/retrofit/SiteCachedService.java A app/src/main/java/org/wikipedia/dataclient/retrofit/WikiCachedService.java M app/src/main/java/org/wikipedia/editing/CaptchaHandler.java M app/src/main/java/org/wikipedia/editing/CaptchaResult.java M app/src/main/java/org/wikipedia/editing/EditHandler.java M app/src/main/java/org/wikipedia/editing/EditPreviewFragment.java M app/src/main/java/org/wikipedia/editing/EditPreviewTask.java M app/src/main/java/org/wikipedia/editing/EditSectionActivity.java M app/src/main/java/org/wikipedia/editing/EditTask.java M app/src/main/java/org/wikipedia/editing/EditTokenStorage.java M app/src/main/java/org/wikipedia/editing/FetchEditTokenTask.java M app/src/main/java/org/wikipedia/editing/FetchSectionWikitextTask.java M app/src/main/java/org/wikipedia/editing/RefreshCaptchaTask.java M app/src/main/java/org/wikipedia/editing/summaries/EditSummaryHandler.java M app/src/main/java/org/wikipedia/feed/FeedCoordinatorBase.java M app/src/main/java/org/wikipedia/feed/FeedFragment.java M app/src/main/java/org/wikipedia/feed/aggregated/AggregatedFeedContentClient.java M app/src/main/java/org/wikipedia/feed/becauseyouread/BecauseYouReadClient.java M app/src/main/java/org/wikipedia/feed/continuereading/ContinueReadingClient.java M app/src/main/java/org/wikipedia/feed/dataclient/DummyClient.java M app/src/main/java/org/wikipedia/feed/dataclient/FeedClient.java M app/src/main/java/org/wikipedia/feed/featured/FeaturedArticleCard.java M app/src/main/java/org/wikipedia/feed/image/FeaturedImage.java M app/src/main/java/org/wikipedia/feed/image/FeaturedImageCard.java M app/src/main/java/org/wikipedia/feed/mainpage/MainPageCard.java M app/src/main/java/org/wikipedia/feed/mainpage/MainPageCardView.java M app/src/main/java/org/wikipedia/feed/mainpage/MainPageClient.java M app/src/main/java/org/wikipedia/feed/mostread/MostReadItemCard.java M app/src/main/java/org/wikipedia/feed/mostread/MostReadListCard.java M app/src/main/java/org/wikipedia/feed/news/NewsItem.java M app/src/main/java/org/wikipedia/feed/news/NewsItemCard.java M app/src/main/java/org/wikipedia/feed/news/NewsListCard.java M app/src/main/java/org/wikipedia/feed/random/RandomCard.java M app/src/main/java/org/wikipedia/feed/random/RandomCardView.java M app/src/main/java/org/wikipedia/feed/random/RandomClient.java M app/src/main/java/org/wikipedia/feed/searchbar/SearchClient.java M app/src/main/java/org/wikipedia/history/HistoryEntryDatabaseTable.java M app/src/main/java/org/wikipedia/json/UriTypeAdapter.java M app/src/main/java/org/wikipedia/language/AcceptLanguageUtil.java M app/src/main/java/org/wikipedia/language/LangLinksActivity.java M app/src/main/java/org/wikipedia/language/LangLinksFetchTask.java M app/src/main/java/org/wikipedia/login/LoginActivity.java M app/src/main/java/org/wikipedia/login/LoginClient.java M app/src/main/java/org/wikipedia/main/MainFragment.java M app/src/main/java/org/wikipedia/nearby/NearbyFetchTask.java M app/src/main/java/org/wikipedia/nearby/NearbyFragment.java M app/src/main/java/org/wikipedia/news/NewsActivity.java M app/src/main/java/org/wikipedia/news/NewsFragment.java M app/src/main/java/org/wikipedia/news/NewsLinkCard.java M app/src/main/java/org/wikipedia/page/DisambigListAdapter.java M app/src/main/java/org/wikipedia/page/ImageLicenseFetchTask.java M app/src/main/java/org/wikipedia/page/LinkHandler.java M app/src/main/java/org/wikipedia/page/MwApiResultPage.java M app/src/main/java/org/wikipedia/page/Namespace.java M app/src/main/java/org/wikipedia/page/Page.java M app/src/main/java/org/wikipedia/page/PageActivity.java M app/src/main/java/org/wikipedia/page/PageDataClient.java M app/src/main/java/org/wikipedia/page/PageFragment.java M app/src/main/java/org/wikipedia/page/PageInfoUnmarshaller.java M app/src/main/java/org/wikipedia/page/PageProperties.java M app/src/main/java/org/wikipedia/page/PageQueryTask.java M app/src/main/java/org/wikipedia/page/PageTitle.java M app/src/main/java/org/wikipedia/page/SuggestionsTask.java M app/src/main/java/org/wikipedia/page/ToCHandler.java M app/src/main/java/org/wikipedia/page/bottomcontent/BottomContentHandler.java M app/src/main/java/org/wikipedia/page/gallery/GalleryActivity.java M app/src/main/java/org/wikipedia/page/gallery/GalleryCollection.java M app/src/main/java/org/wikipedia/page/gallery/GalleryCollectionFetchTask.java M app/src/main/java/org/wikipedia/page/gallery/GalleryItemFetchTask.java M app/src/main/java/org/wikipedia/page/gallery/GalleryItemFragment.java M app/src/main/java/org/wikipedia/page/leadimages/LeadImagesHandler.java M app/src/main/java/org/wikipedia/page/linkpreview/LinkPreviewContents.java M app/src/main/java/org/wikipedia/page/linkpreview/LinkPreviewDialog.java M app/src/main/java/org/wikipedia/page/snippet/ShareHandler.java M app/src/main/java/org/wikipedia/pageimages/PageImageDatabaseTable.java M app/src/main/java/org/wikipedia/pageimages/PageImagesTask.java M app/src/main/java/org/wikipedia/random/RandomSummaryService.java M app/src/main/java/org/wikipedia/readinglist/AddToReadingListDialog.java M app/src/main/java/org/wikipedia/readinglist/ReadingListImageFetcher.java M app/src/main/java/org/wikipedia/readinglist/ReadingListPageInfoTask.java M app/src/main/java/org/wikipedia/readinglist/database/ReadingListTable.java M app/src/main/java/org/wikipedia/readinglist/page/ReadingListPageRow.java M app/src/main/java/org/wikipedia/readinglist/page/database/ReadingListDaoProxy.java M app/src/main/java/org/wikipedia/readinglist/page/database/ReadingListPageTable.java M app/src/main/java/org/wikipedia/savedpages/SavedPageDatabaseTable.java M app/src/main/java/org/wikipedia/savedpages/SavedPageSyncService.java M app/src/main/java/org/wikipedia/search/FullSearchArticlesTask.java M app/src/main/java/org/wikipedia/search/SearchResults.java M app/src/main/java/org/wikipedia/search/SearchResultsFragment.java M app/src/main/java/org/wikipedia/search/TitleSearchTask.java M app/src/main/java/org/wikipedia/server/PageLeadProperties.java M app/src/main/java/org/wikipedia/server/PageServiceFactory.java M app/src/main/java/org/wikipedia/server/mwapi/MwPageEndpointsCache.java M app/src/main/java/org/wikipedia/server/mwapi/MwPageLead.java M app/src/main/java/org/wikipedia/server/mwapi/MwPageService.java M app/src/main/java/org/wikipedia/server/restbase/RbPageCombo.java M app/src/main/java/org/wikipedia/server/restbase/RbPageEndpointsCache.java M app/src/main/java/org/wikipedia/server/restbase/RbPageLead.java M app/src/main/java/org/wikipedia/server/restbase/RbPageService.java M app/src/main/java/org/wikipedia/settings/RbSwitch.java M app/src/main/java/org/wikipedia/theme/ThemeChooserDialog.java M app/src/main/java/org/wikipedia/useroption/dataclient/DefaultUserOptionDataClient.java M app/src/main/java/org/wikipedia/useroption/dataclient/UserOptionDataClientSingleton.java M app/src/main/java/org/wikipedia/util/FeedbackUtil.java M app/src/main/java/org/wikipedia/util/L10nUtil.java M app/src/main/java/org/wikipedia/util/UriUtil.java M app/src/main/java/org/wikipedia/widgets/WidgetProviderFeaturedPage.java M app/src/main/java/org/wikipedia/wikidata/GetDescriptionsTask.java M app/src/main/java/org/wikipedia/wiktionary/WiktionaryDialog.java M app/src/main/java/org/wikipedia/zero/WikipediaZeroHandler.java M app/src/test/java/org/wikipedia/page/PageTest.java M app/src/test/java/org/wikipedia/page/PageTitleTest.java 145 files changed, 876 insertions(+), 873 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia refs/changes/57/315457/1 -- To view, visit https://gerrit.wikimedia.org/r/315457 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic967146a77b1301c87a0e6fa552c2f70663ca2fe Gerrit-PatchSet: 1 Gerrit-Project: apps/android/wikipedia Gerrit-Branch: master Gerrit-Owner: Niedzielski <sniedziel...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits