Allow to directly set the expireDate on a new (link)share#18651
Allow to directly set the expireDate on a new (link)share#18651
Conversation
Since this extends the API we now properly parse the date. We only accept valid ISO 8601 Dates (YYYY-MM-DD). Currently this only works for link shares (it is just ignored for other shares). Since we do not have user/group/federated expiring shares yet. * Tests added
|
🚀 Test PASSed.🚀 |
|
sweeet 👍 |
apps/files_sharing/api/local.php
Outdated
There was a problem hiding this comment.
Add exception message "Invalid date" ?
|
Works 👍, tested with this pyocclient branch: owncloud/pyocclient#120 |
|
@rullzer can you fix the comments/strings, then feel free to merge |
|
👍 It is working fine. |
* Added more intergration tests
rullzer
force-pushed
the
ocs_share_create_with_expire
branch
from
21b996e to
2aff11c
Compare
|
A new inspection was created. |
|
🚀 Test PASSed.🚀 |
|
Raised doc ticket here: owncloud-archive/documentation#1615 |
Allow to directly set the expireDate on a new (link)share
|
Now I wonder whether we should backport this. Anyway, API consumers (like pyocclient) will have to work with older OC servers that don't have this API implemented. |
|
@PVince81 yeah not trivial. But clients will need to handle it properly anyway. Since if sopmebody is still on 8.0.3 is should also work. So I do not think this has to be backported. Clients can check on version number. So that is probably simpler. |
5 participants
As requested by @PVince81 in #14808 and again in #18631 (comment) ;)
Now it is possible to directly set the expiration date on shares.
TODO: