aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib/Controller
Commit message (Collapse)AuthorAgeFilesLines
...
* | Fix testsRoeland Jago Douma2018-04-241-1/+3
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Make the ShareAPIController strictRoeland Jago Douma2018-04-231-40/+39
|/ | | | | | | | Fixes #9279 With types we can force php to not cast a full nummeric user to an int. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Move common menu templates to public APIJulius Härtl2018-04-051-2/+2
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* When formatting a share node an Empty target is invalidRoeland Jago Douma2018-04-041-1/+1
| | | | | | | | | | | Fixes #9028 For federated shares the share table holds no target information (since it is on the other server). So when a node is actually invalid and not found we should not display it anymore in the shared with sections etc and thus throw the proper exceptions. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Set "share_with" field to the ID of the circleDaniel Calviño Sánchez2018-03-161-2/+10
| | | | | | | | | | | When a share is shared with a circle the "share_with" field returned by the API endpoint was always set to the name of the circle. However, the name is not enough to identify a circle. The Circles app now provides the ID of the circle in the "shared with" field of a Share, so this commit modifies the API endpoint to set the "share_with" field to the ID of the circle when provided by the Circles app. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* fixup! Sharing: redirect to download after authentification if requestedJohn Molakvoæ (skjnldsv)2018-03-011-1/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fixed testsJohn Molakvoæ (skjnldsv)2018-02-281-1/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* fixup! Sharing: redirect to download after authentification if requestedJohn Molakvoæ (skjnldsv)2018-02-281-2/+9
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Sharing: redirect to download after authentification if requestedJohn Molakvoæ (skjnldsv)2018-02-281-4/+7
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Merge pull request #7363 from nextcloud/default-share-permsMorris Jobke2018-02-271-28/+35
|\ | | | | Let the admin configure the default share permissions
| * share api: use default permission of no permission is givenBjoern Schiessle2018-02-271-28/+35
| | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | Fix files_sharing testsJulius Härtl2018-02-271-1/+1
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Use PublicTemplateResponse for files_sharingJulius Härtl2018-02-271-1/+15
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Remove unused import statementsMorris Jobke2018-02-142-2/+0
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Show open graph preview in WhatsAppRoeland Jago Douma2018-02-121-0/+11
| | | | | | | Whatsapp is picky about the size of the open graph images. So we do some special handling. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Use ::class statement instead of stringMorris Jobke2018-01-291-1/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Simplify ternary operator statementsMorris Jobke2018-01-261-1/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Return value immediately instead of assigning to a one-time variableMorris Jobke2018-01-261-3/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Properly log the full exception instead of only the messageMorris Jobke2018-01-231-4/+4
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Change @georgehrke's emailMorris Jobke2017-11-062-1/+2
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Update license headersMorris Jobke2017-11-066-5/+36
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Use the new direct endpoint for the og:imageRoeland Jago Douma2017-10-311-1/+6
| | | | | | | | | | | | Some providers had issues when using the preview link (since it was double encoded). Now we actually serve the max size preview so it looks better The image isn't cropped anymore so supporting platforms can just embed the whole image. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Do not error when downloading from a numeric public subfolderRoeland Jago Douma2017-10-301-0/+4
| | | | | | Fixes #6820 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Updated OpenGraph title to display folder name instead of a nextcloud ↵Sascha Sambale2017-10-251-2/+2
| | | | | | | | | | standard text When sharing a folder via Facebook, WhatsApp or any other platform that makes use of OpenGraph, the display title was "Nextcloud - a safe home for all your data" and the description was the actual name of the folder. This should not be the case, so the display title has been changed to display the folder name and the description now displays the former OpenGraph title. Signed-off-by: Sascha Sambale <mastixmc@gmail.com> Fixing whitespaces and empty line.
* Merge pull request #6328 from nextcloud/split-sharees-api-logicblizzz2017-10-041-578/+36
|\ | | | | Splits off the logic from sharees endpoint thus making it available from within Nc/via PHP.
| * shareeAPIController unit test clean up and last fixesArthur Schiwon2017-09-261-0/+1
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * remove unused methodArthur Schiwon2017-09-261-22/+0
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * adjust ShareeAPIController search testArthur Schiwon2017-09-261-1/+4
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * allow more than one plugin per share typeArthur Schiwon2017-09-261-0/+2
| | | | | | | | | | | | | | however it does not dedupe (appears too complex/expensive while we don't havve the issue currently) Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * don't keep result types hard codedArthur Schiwon2017-09-261-1/+2
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * Splits off the logic from sharees endpoint thus making it available fromArthur Schiwon2017-09-261-558/+31
| | | | | | | | | | | | within Nc/via PHP. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | Only have direct preview urls for image filesRoeland Jago Douma2017-09-251-2/+5
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Add annotationRoeland Jago Douma2017-09-251-0/+1
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | For images we have a preview for use the preview linkRoeland Jago Douma2017-09-251-0/+2
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Add direct preview link to single shared image filesRoeland Jago Douma2017-09-251-0/+46
|/ | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #6120 from nextcloud/public_upload_share_api_fixblizzz2017-09-041-0/+1
|\ | | | | Public upload share permission handling
| * Public upload share permission handlingRoeland Jago Douma2017-08-161-0/+1
| | | | | | | | | | | | | | | | If you set the permissions on a public share the SHARE permission makes no sense. So instead of throwing a warning. Just filter out the share permission. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Correctly format OCS response with favoritesRoeland Jago Douma2017-08-291-2/+2
|/ | | | | | | | | | The helper funtion did not handle the response correctly and basically only returned the last share with tags. This is a simple rewrite. That is still understandable. Loops maybe more than strictly required. But preformance is not the issue here. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #6130 from nextcloud/cleanup-unused-methodsMorris Jobke2017-08-161-1/+1
|\ | | | | Cleanup unused methods
| * Cleanup unused methodsMorris Jobke2017-08-151-1/+1
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Catch exceptions on error of cloud id resolutionJoas Schilling2017-08-141-5/+21
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add unit testsRoeland Jago Douma2017-08-101-13/+18
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Remove old code + add MiddlewareRoeland Jago Douma2017-08-101-7/+2
| | | | | | | * Add proper middleware for shareinfo * Remove old shareinfo routes Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Move shareinfo over to proper controllerRoeland Jago Douma2017-08-101-0/+130
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Fix comparisons in the sharing appJoas Schilling2017-08-011-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* improved error handlingBjoern Schiessle2017-07-211-3/+9
| | | | | | | check if table was updated successfully and only then send a notification mail and return "true". Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* Run phan over code baseLukas Reschke2017-07-192-0/+3
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* do not show hyphen in og:title if slogan does not existStephan Müller2017-07-141-1/+1
|
* upstreamMaxence Lange2017-07-112-3/+3
| | | Signed-off-by: Maxence Lange <maxence@nextcloud.com>
* Make sure to only add system users onceJoas Schilling2017-06-141-8/+35
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>