Skip to content

Commit fd9f0cc

Browse files
committed
Merge pull request #6 from longbai/up_params
upload params check
2 parents f99eba4 + 8948b68 commit fd9f0cc

File tree

5 files changed

+22
-13
lines changed

5 files changed

+22
-13
lines changed

QiniuSDK/Storage/QNResumeUpload.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ - (void)putChunk:(NSString *)uphost
279279
}
280280

281281
- (BOOL)isCancelled {
282-
return self.option && [self.option isCancelled];
282+
return self.option && self.option.priv_isCancelled;
283283
}
284284

285285
- (void)makeFile:(NSString *)uphost

QiniuSDK/Storage/QNUploadManager.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ - (void)putData:(NSData *)data
7272
parameters[@"token"] = token;
7373

7474
if (option && option.params) {
75-
[parameters addEntriesFromDictionary:[option p_convertToPostParams]];
75+
[parameters addEntriesFromDictionary:option.params];
7676
}
7777

7878
NSString *mimeType = option.mimeType;

QiniuSDK/Storage/QNUploadOption+Private.h

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,5 @@
1010

1111
@interface QNUploadOption (Private)
1212

13-
@property (nonatomic, readonly, copy) NSDictionary *p_convertToPostParams;
14-
15-
@property (nonatomic, getter = isCancelled, readonly) BOOL cancelled;
13+
@property (nonatomic, getter = priv_isCancelled, readonly) BOOL cancelled;
1614
@end

QiniuSDK/Storage/QNUploadOption.m

Lines changed: 18 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,22 @@ - (instancetype)initWithProgessHandler:(QNUpProgressHandler)progress {
1818
return self;
1919
}
2020

21+
+ (NSDictionary *)filteParam:(NSDictionary *)params {
22+
if (params == nil) {
23+
return nil;
24+
}
25+
NSMutableDictionary *ret = [NSMutableDictionary dictionary];
26+
@autoreleasepool {
27+
NSEnumerator *e = [params keyEnumerator];
28+
for (NSString *key = [e nextObject]; key != nil; key = [e nextObject]) {
29+
if ([key hasPrefix:@"x:"]) {
30+
ret[key] = params[key];
31+
}
32+
}
33+
}
34+
return ret;
35+
}
36+
2137
- (instancetype)initWithMime:(NSString *)mimeType
2238
progressHandler:(QNUpProgressHandler)progress
2339
params:(NSDictionary *)params
@@ -26,20 +42,15 @@ - (instancetype)initWithMime:(NSString *)mimeType
2642
if (self = [super init]) {
2743
_mimeType = mimeType;
2844
_progressHandler = progress;
29-
_params = params;
45+
_params = [QNUploadOption filteParam:params];
3046
_checkCrc = check;
3147
_cancellationSignal = cancel;
3248
}
3349

3450
return self;
3551
}
3652

37-
- (NSDictionary *)p_convertToPostParams {
38-
NSMutableDictionary *params = [NSMutableDictionary dictionaryWithDictionary:self.params];
39-
return params;
40-
}
41-
42-
- (BOOL)isCancelled {
53+
- (BOOL)priv_isCancelled {
4354
return _cancellationSignal && _cancellationSignal();
4455
}
4556

QiniuSDKTests/QNResumeUploadTest.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ - (void)testCancel {
4646
__block BOOL flag = NO;
4747
QNUploadOption *opt = [[QNUploadOption alloc] initWithMime:nil progressHandler: ^(NSString *key, float percent) {
4848
flag = YES;
49-
} params:nil checkCrc:NO cancellationSignal: ^BOOL () {
49+
} params:@{ @"x:七牛":@"objc" } checkCrc:NO cancellationSignal: ^BOOL () {
5050
return flag;
5151
}];
5252
[_upManager putFile:tempFile.path key:keyUp token:g_token complete: ^(QNResponseInfo *i, NSString *k, NSDictionary *resp) {

0 commit comments

Comments
 (0)