Skip to content

Commit e80da55

Browse files
author
刘欢
committed
feat: change info.defaultRequest
1 parent 983bcb7 commit e80da55

File tree

3 files changed

+12
-15
lines changed

3 files changed

+12
-15
lines changed

src/AjaxUploader.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,7 @@ class AjaxUploader extends Component<UploadProps> {
255255

256256
const request =
257257
typeof propsCustomRequest === 'function'
258-
? args => propsCustomRequest({ ...args, info: { defaultRequest } })
258+
? args => propsCustomRequest(args, { defaultRequest })
259259
: propsCustomRequest || defaultRequest;
260260

261261
const requestOption = {

src/interface.tsx

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export interface UploadProps
3131
file: RcFile,
3232
FileList: RcFile[],
3333
) => BeforeUploadFileType | Promise<void | BeforeUploadFileType> | void;
34-
customRequest?: (option: CustomUploadRequestOption) => void | { abort: () => void };
34+
customRequest?: CustomUploadRequestOption;
3535
withCredentials?: boolean;
3636
openFileDialogOnClick?: boolean;
3737
prefixCls?: string;
@@ -78,11 +78,10 @@ export interface UploadRequestOption<T = any> {
7878
method: UploadRequestMethod;
7979
}
8080

81-
export interface CustomUploadRequestOption extends UploadRequestOption {
82-
info: {
83-
defaultRequest: (option: UploadRequestOption) => { abort: () => void } | void;
84-
};
85-
}
81+
export type CustomUploadRequestOption = (
82+
option: UploadRequestOption,
83+
info: { defaultRequest: (option: UploadRequestOption) => { abort: () => void } | void },
84+
) => void | { abort: () => void };
8685
export interface RcFile extends File {
8786
uid: string;
8887
}

tests/uploader.spec.tsx

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1267,10 +1267,9 @@ describe('uploader', () => {
12671267
expect(container.querySelector('span')!).not.toHaveAttribute('role', 'button');
12681268
});
12691269
it('should support defaultRequest in customRequest', done => {
1270-
const customRequest = jest.fn(({ file, onSuccess, onError, info }) => {
1271-
// 模拟条件判断后使用默认上传
1270+
const customRequest = jest.fn(({ file, onSuccess, onError }, { defaultRequest }) => {
12721271
if (file.name === 'success.png') {
1273-
info.defaultRequest({ file, onSuccess, onError });
1272+
defaultRequest({ file, onSuccess, onError });
12741273
} else {
12751274
onError(new Error('custom error'));
12761275
}
@@ -1286,13 +1285,12 @@ describe('uploader', () => {
12861285
value: i => files[i],
12871286
});
12881287
fireEvent.change(input, { target: { files } });
1288+
12891289
setTimeout(() => {
12901290
requests[0].respond(200, {}, `["","${files[0].name}"]`);
1291-
setTimeout(() => {
1292-
expect(customRequest).toHaveBeenCalled();
1293-
expect(onSuccess).toHaveBeenCalled();
1294-
done();
1295-
}, 100);
1291+
expect(customRequest).toHaveBeenCalled();
1292+
expect(onSuccess).toHaveBeenCalled();
1293+
done();
12961294
}, 100);
12971295
});
12981296
});

0 commit comments

Comments
 (0)