Skip to content

some little improvements#7

Open
traximus wants to merge 2 commits intojkichline:masterfrom
traximus:master
Open

some little improvements#7
traximus wants to merge 2 commits intojkichline:masterfrom
traximus:master

Conversation

@traximus
Copy link

1.fix a problem on ACWebDAVItem's createDateStr and modifyDateStr,
it will not return nil now(reference from DAVKit)


2.fix a bug on transfering the parameter overwrite in ACWebDAVCopyRequest;
It didn't transfer overwrite successfully before.


3 fix the delegate call of ACWebDAVCopyRequest and ACWebDAVMoveRequest;


4.ACWebDAVDownloadRequest.h&.m

  • add a new var to hold the reference of currentConnection, used to cancel a
    request if needed;
  • also add a new method cancel to cancel this request

5.ACWebDAVDeleteRequest.m
- connection: didReceiveResponse:

  • response.statusCode=201 OR response.statusCode=204 all shows deleted successfully.

6.ACWebDAVClient.h

  • - add new optional delegate methods

- (BOOL)needToCancelDownload

- (void)resetCancelDownload:(BOOL)cancel

- (BOOL)needToCancelUpload

- (void)resetCancelUpload:(BOOL)cancel

used to check if need to cancel a request

1. fix a problem on ACWebDAVItem's createDateStr and modifyDateStr, it
will not return nil now(reference from DAVKit)

2.fix a bug on transfering the parameter overwrite on
ACWebDAVCopyRequest;
It didn't  transfer  overwrite successfully  before.

3. fix the delegate call of ACWebDAVCopyRequest and ACWebDAVMoveRequest
1.ACWebDAVDownloadRequest.h&.m
  add a new var to hold the reference of currentConnection, used to
cancel a request if needed;
  also add a new method(cancel) to cancel this request

2.ACWebDAVDeleteRequest.m
   - connection: didReceiveResponse: response.statusCode=201
      OR response.statusCode=204 all shows deleted successfully.

3.ACWebDAVClient.h  - add new optional delegate methods
- (BOOL)needToCancelDownload;
- (void)resetCancelDownload:(BOOL)cancel;
- (BOOL)needToCancelUpload;
- (void)resetCancelUpload:(BOOL)cancel;
used to check if need to cancel a request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant