tei/MapLibreParticleLab
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Replaced user-facing assertions with exceptions

Browse Source 
Errors caused by client code should be communicated through NSExceptions. NSAsserts are intended for internal usage; because they include helpful information like the absolute path to the file containing the assertion, stripping the build (#1650) also strips assertion messages. Also replaced hardcoded references to methods in these messages with __PRETTY_FUNCTION__, which avoids future copy-pasta and handles cases in which the MGL class has been subclassed.
This commit is contained in:
Minh Nguyễn
2015-06-17 15:00:06 -07:00
parent ac4179e7de
commit 0e3270cb15
3 changed files with 34 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
platform/ios/MGLMapView.mm
View File

@@ -145,13 +145,20 @@ std::chrono::steady_clock::duration secondsAsDuration(float duration)
- (nullable NSString *)accessToken
{
NSAssert(NO, @"-[MGLMapView accessToken] has been removed. Use +[MGLAccountManager accessToken] or get MGLMapboxAccessToken from the Info.plist.");
[NSException raise:@"Method unavailable" format:
@"%s has been removed. "
@"Use +[MGLAccountManager accessToken] or get MGLMapboxAccessToken from the Info.plist.",
__PRETTY_FUNCTION__];
return nil;
}
- (void)setAccessToken:(nullable NSString *)accessToken
{
NSAssert(NO, @"-[MGLMapView setAccessToken:] has been replaced by +[MGLAccountManager setAccessToken:].\n\nIf you previously set this access token in a storyboard inspectable, select the MGLMapView in Interface Builder and delete the “accessToken” entry from the User Defined Runtime Attributes section of the Identity inspector. Then go to the Info.plist file and set MGLMapboxAccessToken to “%@”.", accessToken);
[NSException raise:@"Method unavailable" format:
@"%s has been replaced by +[MGLAccountManager setAccessToken:].\n\n"
@"If you previously set this access token in a storyboard inspectable, select the MGLMapView in Interface Builder and delete the “accessToken” entry from the User Defined Runtime Attributes section of the Identity inspector. "
@"Then go to the Info.plist file and set MGLMapboxAccessToken to “%@”.",
__PRETTY_FUNCTION__, accessToken];
}
+ (NS_SET_OF(NSString *) *)keyPathsForValuesAffectingStyleURL
@@ -1573,13 +1580,19 @@ CLLocationCoordinate2D MGLLocationCoordinate2DFromLatLng(mbgl::LatLng latLng)
- (nullable NSString *)mapID
{
NSAssert(NO, @"-[MGLMapView mapID] has been renamed -[MGLMapView styleID].");
[NSException raise:@"Method unavailable" format:
@"%s has been renamed -[MGLMapView styleID].",
__PRETTY_FUNCTION__];
return nil;
}
- (void)setMapID:(nullable NSString *)mapID
{
NSAssert(NO, @"-[MGLMapView setMapID:] has been renamed -[MGLMapView setStyleID:].\n\nIf you previously set this map ID in a storyboard inspectable, select the MGLMapView in Interface Builder and delete the “mapID” entry from the User Defined Runtime Attributes section of the Identity inspector. Then go to the Attributes inspector and enter “%@” into the “Style ID” field.", mapID);
[NSException raise:@"Method unavailable" format:
@"%s has been renamed -[MGLMapView setStyleID:].\n\n"
@"If you previously set this map ID in a storyboard inspectable, select the MGLMapView in Interface Builder and delete the “mapID” entry from the User Defined Runtime Attributes section of the Identity inspector. "
@"Then go to the Attributes inspector and enter “%@” into the “Style ID” field.",
__PRETTY_FUNCTION__, mapID];
}
- (NS_ARRAY_OF(NSString *) *)styleClasses
@@ -2003,8 +2016,11 @@ CLLocationCoordinate2D MGLLocationCoordinate2DFromLatLng(mbgl::LatLng latLng)
{
BOOL hasLocationDescription = [[NSBundle mainBundle] objectForInfoDictionaryKey:@"NSLocationWhenInUseUsageDescription"] ||
[[NSBundle mainBundle] objectForInfoDictionaryKey:@"NSLocationAlwaysUsageDescription"];
NSAssert(hasLocationDescription,
@"For iOS 8 and above, your app must have a value for NSLocationWhenInUseUsageDescription or NSLocationAlwaysUsageDescription in its Info.plist");
if (!hasLocationDescription)
{
[NSException raise:@"Missing Location Services usage description" format:
@"In iOS 8 and above, this app must have a value for NSLocationWhenInUseUsageDescription or NSLocationAlwaysUsageDescription in its Info.plist."];
}
[self.locationManager requestWhenInUseAuthorization];
}
#endif
@@ -2553,7 +2569,11 @@ CLLocationCoordinate2D MGLLocationCoordinate2DFromLatLng(mbgl::LatLng latLng)
{
NSString *path = [[NSBundle bundleWithPath:[NSBundle mgl_resourceBundlePath]] pathForResource:name ofType:extension inDirectory:directory];
NSAssert(path, @"Resource %@ not found in application.", name);
if (!path)
{
[NSException raise:@"Resource not found" format:
@"The resource named “%@” could not be found in the Mapbox GL resource bundle.", name];
}
return path;
}

7
platform/ios/MGLMapboxEvents.m
View File

@@ -219,7 +219,12 @@ const NSTimeInterval MGLFlushInterval = 60;
}
}
NSAssert(defaultEnabledValue, @"End users must be able to opt out of Metrics in your app, either inside Settings (via Settings.bundle) or inside this app. If you implement the opt-out control inside this app, disable this assertion by setting MGLMapboxMetricsEnabledSettingShownInApp to YES in Info.plist.");
if (!defaultEnabledValue)
{
[NSException raise:@"MGLMapboxMetricsEnabled setting missing" format:
@"End users must be able to opt out of Metrics in your app, either inside Settings (via Settings.bundle) or inside this app. "
@"If you implement the opt-out control inside this app, disable this assertion by setting MGLMapboxMetricsEnabledSettingShownInApp to YES in Info.plist."];
}
[[NSUserDefaults standardUserDefaults] registerDefaults:@{
@"MGLMapboxMetricsEnabled": defaultEnabledValue,
}];

2
platform/ios/NSException+MGLAdditions.h
View File

@@ -1,3 +1,3 @@
#import <Foundation/Foundation.h>
#define MGLAssertIsMainThread() NSAssert([[NSThread currentThread] isMainThread], @"-[%@ %@] must be accessed on the main thread, not %@", [self class], NSStringFromSelector(_cmd), [NSThread currentThread])
#define MGLAssertIsMainThread() NSAssert([[NSThread currentThread] isMainThread], @"%s must be accessed on the main thread, not %@", __PRETTY_FUNCTION__, [NSThread currentThread])