This is an automated email from the ASF dual-hosted git repository. luckychen pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-weex.git
The following commit(s) were added to refs/heads/master by this push: new 74ea092 Add universal error code. (#2533) 74ea092 is described below commit 74ea09232edfd7db4b276871f11ef4ce6eb4ed77 Author: wqyfavor <qianyuan....@taobao.com> AuthorDate: Wed Jun 12 13:41:09 2019 +0800 Add universal error code. (#2533) --- ios/sdk/WeexSDK/Sources/Engine/WXSDKError.h | 20 ++++++++++++++++++++ ios/sdk/WeexSDK/Sources/Engine/WXSDKError.m | 28 +++++++++++++++++++--------- 2 files changed, 39 insertions(+), 9 deletions(-) diff --git a/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.h b/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.h index 0909ab6..ecb7f0c 100644 --- a/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.h +++ b/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.h @@ -30,9 +30,26 @@ #define WX_ERROR_TYPE_RENDER @"RENDER_ERROR" #define WX_ERROR_TYPE_DOWNLOAD @"DOWN_LOAD_ERROR" +typedef NS_ENUM(int, WXSDKUniversalErrCode) +{ + WX_UNI_KEY_EXCEPTION_DEGRADE = -1000, + WX_UNI_KEY_EXCEPTION_DEGRADE_CHECK_CONTENT_LENGTH_FAILED = -1003, + WX_UNI_KEY_EXCEPTION_DEGRADE_BUNDLE_CONTENTTYPE_ERROR = -1004, + WX_UNI_KEY_EXCEPTION_DEGRADE_OTHER_CAUSE = -1005, + WX_UNI_KEY_EXCEPTION_DEGRADE_EAGLE_RENDER_ERROR = -1007, + WX_UNI_KEY_EXCEPTION_DEGRADE_NET_CODE_CAUSE = -1008, +}; +/* For legacy usage. + Defines all error codes only for iOS platform which may be different from Android error codes. + For monitor uploads, we will remap the error code to WXSDKUniversalErrCode above if an + error code is declared having a remapped value. + + You could either use WXSDKUniversalErrCode or WXSDKErrCode when committing an error. + */ typedef NS_ENUM(int, WXSDKErrCode) { + // No use WX_ERR_JSFRAMEWORK_START = -1001, WX_ERR_JSFRAMEWORK_LOAD = -1002, WX_ERR_JSFRAMEWORK_EXECUTE = -1003, @@ -70,6 +87,7 @@ typedef NS_ENUM(int, WXSDKErrCode) WX_KEY_EXCEPTION_DOM = -9300, WX_KEY_EXCEPTION_WXBRIDGE=-9400, + // The following error codes have a remapped value defined in WXSDKUniversalErrCode WX_KEY_EXCEPTION_DEGRADE = -9500, WX_KEY_EXCEPTION_DEGRADE_CHECK_CONTENT_LENGTH_FAILED = -9501, WX_KEY_EXCEPTION_DEGRADE_BUNDLE_CONTENTTYPE_ERROR = -9502, @@ -77,6 +95,7 @@ typedef NS_ENUM(int, WXSDKErrCode) WX_KEY_EXCEPTION_DEGRADE_NET_CODE_CAUSE = -9504, WX_KEY_EXCEPTION_DEGRADE_EAGLE_RENDER_ERROR = -9505, + // No use WX_KEY_EXCEPTION_ABILITY_DOWN = -9600, WX_KEY_EXCEPTION_ABILITY_DOWN_IMAGE = -9601, WX_KEY_EXCEPTION_ABILITY_DOWN_TOH5 = -9602, @@ -110,6 +129,7 @@ typedef NS_ENUM (NSInteger,WXSDKErrorGroup){ + (WXSDKErrorGroup) getErrorGroupByCode:(WXSDKErrCode) code; + (NSString *) convertGroupToStringName:(WXSDKErrorGroup) group; + (NSString *) convertTypeToStringName:(WXSDKErrorType)type; ++ (NSString *) convertToUniversalCode:(WXSDKErrCode)code; @end diff --git a/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.m b/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.m index 9db1895..0eb0066 100644 --- a/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.m +++ b/ios/sdk/WeexSDK/Sources/Engine/WXSDKError.m @@ -21,7 +21,8 @@ #import "WXSDKError.h" #define ERROR_TYPE @"type" -#define ERROR_GROUP @"groupe" +#define ERROR_GROUP @"group" +#define ERROR_ALIAS @"alias" @implementation WXSDKErrCodeUtil @@ -64,7 +65,6 @@ @(WX_ERR_JSBRIDGE_END):@{ERROR_TYPE:@(WX_JS_ERROR),ERROR_GROUP:@(WX_JS)}, @(WX_ERR_RENDER_START):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, - @(WX_ERR_JSFRAMEWORK_START):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_ERR_RENDER_UPDATTR):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_ERR_RENDER_UPDSTYLE):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_ERR_RENDER_ADDELEMENT):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @@ -89,17 +89,16 @@ @(WX_KEY_EXCEPTION_DOM):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_KEY_EXCEPTION_WXBRIDGE):@{ERROR_TYPE:@(WX_JS_ERROR),ERROR_GROUP:@(WX_JS)}, - @(WX_KEY_EXCEPTION_DEGRADE):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, - @(WX_KEY_EXCEPTION_DEGRADE_CHECK_CONTENT_LENGTH_FAILED):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NET)}, - @(WX_KEY_EXCEPTION_DEGRADE_BUNDLE_CONTENTTYPE_ERROR):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NET)}, - @(WX_KEY_EXCEPTION_DEGRADE_EAGLE_RENDER_ERROR):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_JS)}, - @(WX_KEY_EXCEPTION_DEGRADE_OTHER_CAUSE):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, - @(WX_KEY_EXCEPTION_DEGRADE_NET_CODE_CAUSE):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NET)}, + @(WX_KEY_EXCEPTION_DEGRADE):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NATIVE),ERROR_ALIAS:@(WX_UNI_KEY_EXCEPTION_DEGRADE)}, + @(WX_KEY_EXCEPTION_DEGRADE_CHECK_CONTENT_LENGTH_FAILED):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NET),ERROR_ALIAS:@(WX_UNI_KEY_EXCEPTION_DEGRADE_CHECK_CONTENT_LENGTH_FAILED)}, + @(WX_KEY_EXCEPTION_DEGRADE_BUNDLE_CONTENTTYPE_ERROR):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NET), ERROR_ALIAS:@(WX_UNI_KEY_EXCEPTION_DEGRADE_BUNDLE_CONTENTTYPE_ERROR)}, + @(WX_KEY_EXCEPTION_DEGRADE_EAGLE_RENDER_ERROR):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_JS),ERROR_ALIAS:@(WX_UNI_KEY_EXCEPTION_DEGRADE_EAGLE_RENDER_ERROR)}, + @(WX_KEY_EXCEPTION_DEGRADE_OTHER_CAUSE):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NATIVE),ERROR_ALIAS:@(WX_UNI_KEY_EXCEPTION_DEGRADE_OTHER_CAUSE)}, + @(WX_KEY_EXCEPTION_DEGRADE_NET_CODE_CAUSE):@{ERROR_TYPE:@(WX_DEGRADE_ERROR),ERROR_GROUP:@(WX_NET),ERROR_ALIAS:@(WX_UNI_KEY_EXCEPTION_DEGRADE_NET_CODE_CAUSE)}, @(WX_KEY_EXCEPTION_ABILITY_DOWN):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_KEY_EXCEPTION_ABILITY_DOWN_IMAGE):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_KEY_EXCEPTION_ABILITY_DOWN_TOH5):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, - @(WX_ERR_JSFRAMEWORK_START):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_KEY_EXCEPTION_ABILITY_DOWN_):@{ERROR_TYPE:@(WX_NATIVE_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @(WX_KEY_EXCEPTION_EMPTY_SCREEN_JS):@{ERROR_TYPE:@(WX_RENDER_ERROR),ERROR_GROUP:@(WX_JS)}, @(WX_KEY_EXCEPTION_EMPTY_SCREEN_NATIVE):@{ERROR_TYPE:@(WX_RENDER_ERROR),ERROR_GROUP:@(WX_NATIVE)}, @@ -141,4 +140,15 @@ } } ++ (NSString *) convertToUniversalCode:(WXSDKErrCode)code +{ + id aliasCode = [[[self getMap] objectForKey:@(code)] objectForKey:ERROR_ALIAS]; + if (aliasCode) { + return [aliasCode description]; + } + else { + return [@(code) description]; + } +} + @end