aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib/Controller
Commit message (Collapse)AuthorAgeFilesLines
...
* | | make php-cs happyRobin Appelman2020-04-231-1/+0
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Fix Sharing recommendation user displayJohn Molakvoæ (skjnldsv)2020-04-211-29/+30
| |/ |/| | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com> Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
* | Also disallow ; in remote urlsJoas Schilling2020-04-141-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Format control structures, classes, methods and functionChristoph Wurst2020-04-107-21/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To continue this formatting madness, here's a tiny patch that adds unified formatting for control structures like if and loops as well as classes, their methods and anonymous functions. This basically forces the constructs to start on the same line. This is not exactly what PSR2 wants, but I think we can have a few exceptions with "our" style. The starting of braces on the same line is pracrically standard for our code. This also removes and empty lines from method/function bodies at the beginning and end. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Use elseif instead of else ifChristoph Wurst2020-04-103-21/+21
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #20391 from nextcloud/refactor/spaces-cleanupChristoph Wurst2020-04-091-1/+1
|\ \ | | | | | | Remove all extra whitespace PSR2 does not like
| * | Remove trailing and in between spacesChristoph Wurst2020-04-091-1/+1
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Merge pull request #20384 from nextcloud/techdebt/lowercase-keywordsChristoph Wurst2020-04-091-1/+1
|\ \ \ | |/ / |/| | Use php keywords in lowercase
| * | Use php keywords in lowercaseChristoph Wurst2020-04-091-1/+1
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Unify function spacing to PSR2 recommendationChristoph Wurst2020-04-092-3/+3
|/ / | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Fix multiline commentsChristoph Wurst2020-04-081-1/+1
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Remove user name from public sharing pageJoas Schilling2020-04-011-5/+25
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Update the license headers for Nextcloud 19Christoph Wurst2020-03-313-0/+3
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #20170 from nextcloud/techdebt/remove-unused-importsChristoph Wurst2020-03-272-2/+0
|\ \ | | | | | | Remove unused imports
| * | Remove unused importsChristoph Wurst2020-03-252-2/+0
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Use the short array syntax, everywhereChristoph Wurst2020-03-261-1/+1
|/ / | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Remove unused properties in ShareesAPIControllerJulius Härtl2020-03-201-8/+0
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Allow the video player on the hide downloadJoas Schilling2020-03-191-8/+18
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Fix willReturnOnConsecutiveCallsJoas Schilling2020-03-131-0/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Don't allow anchors and queries in remote urlsJoas Schilling2020-03-121-0/+3
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #19777 from nextcloud/fix/viewer-publicJohn Molakvoæ2020-03-121-1/+8
|\ \ | | | | | | Properly emit Viewer event on files and files_sharing
| * | Properly emit Viewer event on files and files_sharingJohn Molakvoæ (skjnldsv)2020-03-111-0/+6
| | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | Fix import order and formattingJohn Molakvoæ (skjnldsv)2020-03-111-1/+2
| | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | Don't break when one remote share is downJoas Schilling2020-03-091-0/+4
|/ / | | | | | | | | | | getFileInfo can also return false Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Don't try to format deleted sharesRoeland Jago Douma2020-03-021-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | Fixes #15455 The issue is that we have a fallback for shares to use the target. So when the target exists again we happily format it (not that the shares are still invalid). This just tries to get the node. If we can't then boom. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Allow single file downloads so the video player works againJoas Schilling2020-02-281-8/+10
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #19349 from ↵Roeland Jago Douma2020-02-081-1/+4
|\ \ | | | | | | | | | | | | nextcloud/bugfix/noid/prevent-download-on-secure-view Prevent archieved download on secure view
| * | Prevent archieved download on secure viewJoas Schilling2020-02-071-1/+4
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Disable link shares of disabled usersRoeland Jago Douma2020-02-071-0/+12
|/ / | | | | | | | | | | Fixes #10869 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Don't show link shares hash in inherited share listRoeland Jago Douma2020-01-151-2/+2
| | | | | | | | | | | | This adds no value at all. Just showing the text "(Shared link)" now. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Fix license headersChristoph Wurst2020-01-132-1/+4
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | moving ->getParent earlierMaxence Lange2020-01-081-1/+1
| | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | Add pending share list to frontendJulius Härtl2020-01-071-5/+25
| | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net> Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | A pending shares overviewRoeland Jago Douma2020-01-071-0/+30
| | | | | | | | | | | | | | | | Now that we accept shares we should show an overview of shares that are pending. This first part is the small API to get a list of the currently pending shares. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Accept shares when you open mailRoeland Jago Douma2020-01-061-0/+83
| | | | | | | | | | | | | | | | | | Fixes #18380 When you open a direct share to yourself. We should autoaccept the share. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Do not use $_GETRoeland Jago Douma2020-01-051-4/+3
| | | | | | | | | | | | The appframework is more than capable of handling this properly. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | confirm resharing rights if viewer is current ownerMaxence Lange2020-01-021-0/+4
| | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | Update license headers for 18Christoph Wurst2019-12-201-1/+3
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Allow users to specify to accept (internal) shares by defaultRoeland Jago Douma2019-12-171-0/+54
| | | | | | | | | | | | | | | | | | Fixes #18255 A new user setting allows a user to always accept (internal) shares. For example if they don't like accepting shares manually ;) Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Update license headersChristoph Wurst2019-12-058-12/+30
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Always do sharee lookup on lookup server in GSRoeland Jago Douma2019-12-051-1/+8
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | fixing tests + avoid duplicate in getFormattedShares()Maxence Lange2019-12-031-4/+7
| | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | integration testsMaxence Lange2019-12-031-8/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com> add tests on non-owner pov Signed-off-by: Maxence Lange <maxence@artificial-owl.com> duplicate Signed-off-by: Maxence Lange <maxence@artificial-owl.com> small fixes Signed-off-by: Maxence Lange <maxence@artificial-owl.com> removed tags Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | Inherited SharesMaxence Lange2019-12-031-49/+271
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com> working on users with resharing rights Signed-off-by: Maxence Lange <maxence@artificial-owl.com> -getLogger() Signed-off-by: Maxence Lange <maxence@artificial-owl.com> cleaning Signed-off-by: Maxence Lange <maxence@artificial-owl.com> fix type Signed-off-by: Maxence Lange <maxence@artificial-owl.com> Update SharingRightsException.php
* | Apply share permission mask to all files obtained via shareinfoRoeland Jago Douma2019-11-261-25/+12
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #18079 from nextcloud/fixes/phpcsRoeland Jago Douma2019-11-256-22/+25
|\ \ | | | | | | Some php-cs fixes
| * | Some php-cs fixesRoeland Jago Douma2019-11-226-22/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Order the imports * No leading slash on imports * Empty line before namespace * One line per import * Empty after imports * Emmpty line at bottom of file Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Get first element of nodes arraywaleczny2019-11-181-1/+1
|/ / | | | | Geting first element of nodes array instead of element with key 0. In some cases key 0 not exists in this array ie. when file is shared from group folder with member of this group who have no permission to read this file.
* | The share manager knows which provider can accept sharesJoas Schilling2019-11-121-5/+0
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Allow to accept group sharesJoas Schilling2019-11-121-9/+7
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>