[Feature] Enhance the places deep linking support in the Wikipedia app to support coordinates #2
@ -1,3 +1,4 @@
|
||||
@import CoreLocation;
|
||||
@import Foundation;
|
||||
NS_ASSUME_NONNULL_BEGIN
|
||||
|
||||
@ -46,6 +47,8 @@ extern NSString *const WMFNavigateToActivityNotification;
|
||||
|
||||
- (nullable NSURL *)wmf_linkURL;
|
||||
|
||||
- (nullable CLLocation *)wmf_locationFromURL;
|
||||
|
||||
- (NSURL *)wmf_contentURL;
|
||||
|
||||
+ (NSURL *)wmf_baseURLForActivityOfType:(WMFUserActivityType)type;
|
||||
|
@ -62,15 +62,42 @@ __attribute__((annotate("returns_localized_nsstring"))) static inline NSString *
|
||||
+ (instancetype)wmf_placesActivityWithURL:(NSURL *)activityURL {
|
||||
NSURLComponents *components = [NSURLComponents componentsWithURL:activityURL resolvingAgainstBaseURL:NO];
|
||||
NSURL *articleURL = nil;
|
||||
NSNumber *latitude = nil;
|
||||
NSNumber *longitude = nil;
|
||||
|
||||
for (NSURLQueryItem *item in components.queryItems) {
|
||||
if ([item.name isEqualToString:@"WMFArticleURL"]) {
|
||||
NSString *articleURLString = item.value;
|
||||
articleURL = [NSURL URLWithString:articleURLString];
|
||||
break;
|
||||
}
|
||||
|
||||
if ([item.name isEqualToString:@"coordinates"]) {
|
||||
NSArray *numbers = [item.value componentsSeparatedByString:@","];
|
||||
|
||||
if (numbers.count == 2) {
|
||||
NSNumberFormatter *formatter = [[NSNumberFormatter alloc] init];
|
||||
|
||||
latitude = [formatter numberFromString:numbers.firstObject];
|
||||
longitude = [formatter numberFromString:numbers.lastObject];
|
||||
}
|
||||
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
NSUserActivity *activity = [self wmf_pageActivityWithName:@"Places"];
|
||||
activity.webpageURL = articleURL;
|
||||
|
||||
if (latitude != nil && longitude != nil) {
|
||||
NSMutableDictionary *userInfo = [activity.userInfo mutableCopy];
|
||||
|
||||
userInfo[@"WMFCoordinatesLatitude"] = latitude;
|
||||
userInfo[@"WMFCoordinatesLongitude"] = longitude;
|
||||
|
||||
activity.userInfo = [userInfo copy];
|
||||
}
|
||||
|
||||
return activity;
|
||||
}
|
||||
|
||||
@ -264,6 +291,18 @@ __attribute__((annotate("returns_localized_nsstring"))) static inline NSString *
|
||||
}
|
||||
}
|
||||
|
||||
- (CLLocation *)wmf_locationFromURL {
|
||||
NSNumber *latitude = self.userInfo[@"WMFCoordinatesLatitude"];
|
||||
NSNumber *longitude = self.userInfo[@"WMFCoordinatesLongitude"];
|
||||
|
||||
if (latitude != nil && longitude != nil) {
|
||||
return [[CLLocation alloc] initWithLatitude:(CLLocationDegrees)[latitude floatValue]
|
||||
longitude:(CLLocationDegrees)[longitude floatValue]];
|
||||
} else {
|
||||
return nil;
|
||||
}
|
||||
}
|
||||
|
||||
- (NSURL *)wmf_contentURL {
|
||||
return self.userInfo[@"WMFURL"];
|
||||
}
|
||||
|
@ -1126,6 +1126,10 @@ class PlacesViewController: ViewController, UISearchBarDelegate, ArticlePopoverV
|
||||
viewMode = .map
|
||||
}
|
||||
|
||||
@objc func centerMap(onLocation location: CLLocation) {
|
||||
zoomAndPanMapView(toLocation: location)
|
||||
}
|
||||
|
||||
func selectArticlePlace(_ articlePlace: ArticlePlace) {
|
||||
mapView.selectAnnotation(articlePlace, animated: articlePlace.identifier != previouslySelectedArticlePlaceIdentifier)
|
||||
previouslySelectedArticlePlaceIdentifier = articlePlace.identifier
|
||||
|
@ -1187,10 +1187,16 @@ NSString *const WMFLanguageVariantAlertsLibraryVersion = @"WMFLanguageVariantAle
|
||||
[self setSelectedIndex:WMFAppTabTypePlaces];
|
||||
[self.navigationController popToRootViewControllerAnimated:animated];
|
||||
NSURL *articleURL = activity.wmf_linkURL;
|
||||
if (articleURL) {
|
||||
CLLocation *locationFromURL = activity.wmf_locationFromURL;
|
||||
if (articleURL || locationFromURL) {
|
||||
// For "View on a map" action to succeed, view mode has to be set to map.
|
||||
[[self placesViewController] updateViewModeToMap];
|
||||
[[self placesViewController] showArticleURL:articleURL];
|
||||
if (locationFromURL) {
|
||||
[[self placesViewController] centerMapOnLocation:locationFromURL];
|
||||
}
|
||||
else if (articleURL) {
|
||||
[[self placesViewController] showArticleURL:articleURL];
|
||||
}
|
||||
}
|
||||
} break;
|
||||
case WMFUserActivityTypeContent: {
|
||||
|
@ -113,8 +113,6 @@ public enum WikimediaProject: Hashable {
|
||||
self = .wikibooks(languageCode, localizedLanguageName)
|
||||
} else if siteURLString.contains(Configuration.Domain.wiktionary) {
|
||||
self = .wiktionary(languageCode, localizedLanguageName)
|
||||
} else if siteURLString.contains(Configuration.Domain.wiktionary) {
|
||||
self = .wiktionary(languageCode, localizedLanguageName)
|
||||
} else if siteURLString.contains(Configuration.Domain.wikisource) {
|
||||
self = .wikisource(languageCode, localizedLanguageName)
|
||||
} else if siteURLString.contains(Configuration.Domain.wikinews) {
|
||||
|
@ -44,6 +44,76 @@
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypeSavedPages);
|
||||
}
|
||||
|
||||
- (void)testPlacesURL {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://places"];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypePlaces);
|
||||
XCTAssertNil(activity.webpageURL);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLatitude"]);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLongitude"]);
|
||||
}
|
||||
|
||||
- (void)testPlacesURLWithCoordinates {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://places?coordinates=52.370216,4.895168"];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypePlaces);
|
||||
XCTAssertNil(activity.webpageURL);
|
||||
XCTAssert([activity.userInfo[@"WMFCoordinatesLatitude"] isEqualToNumber:@52.370216]);
|
||||
XCTAssert([activity.userInfo[@"WMFCoordinatesLongitude"] isEqualToNumber:@4.895168]);
|
||||
}
|
||||
|
||||
- (void)testPlacesURLWithCoordinatesButNegatives {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://places?coordinates=-34.603722,-58.381592"];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypePlaces);
|
||||
XCTAssertNil(activity.webpageURL);
|
||||
XCTAssert([activity.userInfo[@"WMFCoordinatesLatitude"] isEqualToNumber:@-34.603722]);
|
||||
XCTAssert([activity.userInfo[@"WMFCoordinatesLongitude"] isEqualToNumber:@-58.381592]);
|
||||
}
|
||||
|
||||
- (void)testPlacesURLWithCoordinatesButNoLongitude {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://places?coordinates=52.370216,"];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypePlaces);
|
||||
XCTAssertNil(activity.webpageURL);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLatitude"]);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLongitude"]);
|
||||
}
|
||||
|
||||
- (void)testPlacesURLWithCoordinatesButNoLatitude {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://places?coordinates=,4.895168"];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypePlaces);
|
||||
XCTAssertNil(activity.webpageURL);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLatitude"]);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLongitude"]);
|
||||
}
|
||||
|
||||
- (void)testPlacesURLWithCoordinatesButNoCoordinates {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://places?coordinates=,"];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypePlaces);
|
||||
XCTAssertNil(activity.webpageURL);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLatitude"]);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLongitude"]);
|
||||
}
|
||||
|
||||
- (void)testPlacesURLWithCoordinatesButEmpty {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://places?coordinates="];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
||||
XCTAssertEqual(activity.wmf_type, WMFUserActivityTypePlaces);
|
||||
XCTAssertNil(activity.webpageURL);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLatitude"]);
|
||||
XCTAssertNil(activity.userInfo[@"WMFCoordinatesLongitude"]);
|
||||
}
|
||||
|
||||
- (void)testSearchURL {
|
||||
NSURL *url = [NSURL URLWithString:@"wikipedia://en.wikipedia.org/w/index.php?search=dog"];
|
||||
NSUserActivity *activity = [NSUserActivity wmf_activityForWikipediaScheme:url];
|
||||
|
Loading…
x
Reference in New Issue
Block a user