aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Share20
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge pull request #22234 from nextcloud/bugfix/noid/reshare-mountMorris Jobke2020-09-101-4/+12
|\ | | | | Use user mount with matching shared storage only
| * Use user mount with matching shared storage onlyJulius Härtl2020-08-131-4/+12
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Update the license headers for Nextcloud 20Christoph Wurst2020-08-244-2/+8
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Allow to disable share emailsJoas Schilling2020-08-191-1/+2
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Generate password by password_policy appDaniel Kesselberg2020-08-111-2/+1
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Migrate OC_Group post_removeFromGroup hook to actual event objectMorris Jobke2020-07-301-4/+0
| | | | | | | | Ref #14552 This adds a BeforeUserRemovedEvent to the LDAP backend because it was missing. It's not really before, but we don't have the before state. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Correctly remove usergroup shares on removing group membersJoas Schilling2020-07-302-1/+48
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Move federated_share_added into a typed eventMorris Jobke2020-07-231-1/+3
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Use the correct mountpoint to calculateRoeland Jago Douma2020-07-091-1/+8
| | | | | | | If we use the owners mount point this results in null. And then the rest of the checks get called with null. Which doesn't work. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Use the new method everywhereJoas Schilling2020-07-032-5/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Replace Share with IShare and add missing return.Daniel Kesselberg2020-06-261-3/+3
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Update share type constant usageJoas Schilling2020-06-244-154/+156
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix language in share notes email for usersJoas Schilling2020-06-232-28/+36
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Prevent harder to share your rootJoas Schilling2020-06-031-3/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix disabling send password by Talk without new password in mail sharesDaniel Calviño Sánchez2020-05-291-4/+4
| | | | | | | | | | | | | | | | | | When "send password by Talk" was disabled in a mail share it was possible to keep the same password as before, as it does not pose any security issue (unlike keeping it when "send password by Talk" is enabled, as in that case the password was already disclosed by mail). However, if a mail share is updated but the password is not set again only the hashed password will be available. In that case it would not make sense to send the password by mail, so now the password must be changed when disabling "send password by Talk". Note that, even if explicitly setting the same password again along with the "send password by Talk" property would work, this was also prevented for simplicity. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix enabling send password by Talk with same password in mail sharesDaniel Calviño Sánchez2020-05-281-1/+11
| | | | | | | | | | | | | When "send password by Talk" is enabled in a mail share a new password must be also set. However, when the passwords of the original and the new share were compared it was not taken into account that the original password is now hashed, while the new one is not (unless no new password was sent, in which case the password of the original share was set in the new share by the controller, but that was already prevented due to both passwords being literally the same), so it was possible to set the same password again. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix enabling send password by Talk with empty password in link sharesDaniel Calviño Sánchez2020-05-281-0/+6
| | | | | | | When "send password by Talk" is enabled in a link share now a non empty password is enforced. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Remove group from excluded_groups_list after deleteDaniel Kesselberg2020-05-171-0/+13
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Remove unused use statementMorris Jobke2020-05-081-1/+0
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* get only groupids in default share backendRobin Appelman2020-05-041-7/+2
| | | | | | instead of getting the full group object and then only using the id anyway Signed-off-by: Robin Appelman <robin@icewind.nl>
* Update license headers for 19Christoph Wurst2020-04-293-2/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #19793 from nextcloud/bugfix/noid/link-to-federated-reshareMorris Jobke2020-04-271-5/+0
|\ | | | | Fix resharing of federated shares that were created out of links
| * Link shares have reshare permission if outgoing federated shares are enabledJulius Härtl2020-04-211-5/+0
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Allow specifying a default expiration dateRoeland Jago Douma2020-04-241-2/+12
|/ | | | | | This overrides the max expiration date. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Provide the proper language to the mailerRoeland Jago Douma2020-04-161-2/+2
| | | | | | | Else we can't properly translate the footer in the recipients e-mail language. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #19936 from ↵Joas Schilling2020-04-141-2/+6
|\ | | | | | | | | nextcloud/bugfix/noid/handle-unset-owner-in-sharing Handle unset owner in sharing
| * Handle unset owner in sharingJoas Schilling2020-03-131-2/+6
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #20181 from nextcloud/bugfix/noid/dont-break-on-path-nullJoas Schilling2020-04-141-1/+1
|\ \ | | | | | | Also bail out when the path is actually null
| * | Also bail out when the path is actually nullJoas Schilling2020-03-261-1/+1
| | | | | | | | | | | | | | | | | | | | | Apparently this can happen when a external mount was shared that is later not available anymore Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Add visibility to all constantsChristoph Wurst2020-04-101-1/+1
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Format control structures, classes, methods and functionChristoph Wurst2020-04-108-51/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-37/+37
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Unify function spacing to PSR2 recommendationChristoph Wurst2020-04-093-5/+5
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Use exactly one empty line after the namespace declarationChristoph Wurst2020-04-091-1/+0
| | | | | | | | | | | | | | | | | | For PSR2 Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Fix (array) indent style to always use one tabChristoph Wurst2020-04-092-4/+4
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Update the license headers for Nextcloud 19Christoph Wurst2020-03-311-1/+1
| | | | | | | | | | | | 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-262-4/+4
|/ / | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Expose enumeration config though share managerJulius Härtl2020-03-201-0/+9
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Update the target when it isempty after sharingRoeland Jago Douma2020-03-161-0/+5
|/ | | | | | | Hooks that listen to it (audit log) benefit from having the target properly set. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Strip of users home path from share api messageJoas Schilling2020-02-201-2/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Make sure the default share provider does not execute for other thingsJoas Schilling2020-01-281-0/+3
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Accept shares when you open mailRoeland Jago Douma2020-01-061-1/+1
| | | | | | | | | Fixes #18380 When you open a direct share to yourself. We should autoaccept the share. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* config.php setting to always accept internal sharesRoeland Jago Douma2019-12-161-12/+19
| | | | | | Part of #18255 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #17935 from nextcloud/feature/noid/public-auth-for-circlesRoeland Jago Douma2019-12-111-2/+3
|\ | | | | adding share type circles
| * using IShare::Maxence Lange2019-12-051-3/+3
| | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
| * adding share type circlesMaxence Lange2019-12-051-1/+2
| | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | Update license headersChristoph Wurst2019-12-0510-11/+26
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Actualy return dataRoeland Jago Douma2019-11-271-8/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>