summaryrefslogtreecommitdiffstats
path: root/apps/sharebymail/lib/ShareByMailProvider.php
Commit message (Collapse)AuthorAgeFilesLines
* ReplyTo option for sharebymail Issue #24139Nicolas SIMIDE2021-01-281-24/+33
| | | | Signed-off-by: Nicolas SIMIDE <2083596+dems54@users.noreply.github.com>
* Update the license headers for Nextcloud 20Christoph Wurst2020-08-241-0/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Generate password by password_policy appDaniel Kesselberg2020-08-111-44/+12
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Update share type constant usageJoas Schilling2020-06-241-11/+11
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix creating a mail share with a passwordDaniel Calviño Sánchez2020-05-281-4/+6
| | | | | | | | | | When a mail share was created with a password the given password was not hashed, so it was not possible to open the share with that password. Moreover, if passwords were enforced the given password was ignored and a new one was set (although in this case it was hashed so it worked as expected). Now the given password is properly hashed and not overriden. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Format control structures, classes, methods and functionChristoph Wurst2020-04-101-20/+7
| | | | | | | | | | | | | | | 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>
* Update the license headers for Nextcloud 19Christoph Wurst2020-03-311-0/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Use the short array syntax, everywhereChristoph Wurst2020-03-261-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* properly set 'hide_download' as integerRobin Appelman2020-03-131-2/+2
| | | | | | | | the database field is a smallint, not a boolean. fixes postgresql tests Signed-off-by: Robin Appelman <robin@icewind.nl>
* Properly respect hide download on sharebymailRoeland Jago Douma2020-03-131-3/+8
| | | | | | fixes #19484 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Fix license headersChristoph Wurst2020-01-131-0/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* additional fix to #18730 #18733exner1042020-01-081-0/+8
| | | regarding to advice from @kesselb
* Wrong email sender fix #18730exner1042020-01-081-3/+1
| | | Wrong email sender in app sharebymail when sending password to owner fix #18730
* Update license headersChristoph Wurst2019-12-051-1/+14
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Some php-cs fixesRoeland Jago Douma2019-11-221-1/+1
| | | | | | | | | | | * 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 all shares as iterableRoeland Jago Douma2019-10-301-0/+25
| | | | | | | | | | | | Sometimes we need all shares or rather a specific subset of shares but creating dedicated functions is a pain. This just returns an iterable object for all shares so we can loop over them without allocating all the memory on the system. It should not be used by any user called code. But in an occ command or background job it is fine IMO. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Change send to sentFrederic Werner2019-06-301-1/+1
| | | | Signed-off-by: Frederic Werner <frederic-github@werner-net.work>
* Fix wrong method callJoas Schilling2019-02-071-1/+0
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Make sure the relative path is always a stringJoas Schilling2019-02-071-10/+10
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* improve mail subject for mails to the owner with the passwordBjoern Schiessle2019-01-131-1/+1
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* Unshare from mail activity is missingJoas Schilling2018-12-041-2/+9
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix a case where "password_by_talk" was not a boolean (e.g. null or "0") and ↵Morris Jobke2018-10-121-1/+1
| | | | | | | | | | | | actively cast it to a boolean This was the error message that we have seen: ``` Argument 1 passed to OC\\Share20\\Share::setSendPasswordByTalk() must be of the type boolean, null given, called in apps/sharebymail/lib/ShareByMailProvider.php on line 981 ``` Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Use numeric placeholders if there are multiple, so that RTL languages can ↵Joas Schilling2018-10-091-12/+12
| | | | | | operate better Signed-off-by: Joas Schilling <coding@schilljs.com>
* sharebymail: Add linebreak before passwordrubo772018-10-031-2/+4
| | | | Signed-off-by: Ruben Barkow <github@r.z11.de>
* Honour "sendPasswordByTalk" property in mail sharesDaniel Calviño Sánchez2018-07-241-2/+3
| | | | | | | | | | | | | | | | | When a password was set for a mail share an e-mail was sent to the recipient with the password. Now the e-mail is no longer sent if the password is meant to be sent by Talk. However, before the e-mail was not sent when the share was updated but the password was not changed. Now an e-mail is sent in that case too if switching from a password sent by Talk to a password sent by mail. On the other hand, when switching from a password sent by mail to a password sent by Talk it is mandatory to change the password; otherwise the recipient would already have access to the share without having to call the sharer to verify her identity. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Store "sendPasswordByTalk" property of mail shares in the databaseDaniel Calviño Sánchez2018-07-241-2/+8
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* some small template fixesBjoern Schiessle2018-07-211-2/+2
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* add direct link to the shareBjoern Schiessle2018-07-211-0/+7
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* fix mail templatesBjoern Schiessle2018-07-211-3/+3
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* add note to share object when reading it from the dbBjoern Schiessle2018-07-211-0/+1
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* improve mail template and escape html codeBjoern Schiessle2018-07-211-12/+5
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* allow to add a personal note to a shareBjoern Schiessle2018-07-211-0/+60
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* Add API to undelete delete group sharesRoeland Jago Douma2018-07-051-0/+5
| | | | | | | | | | | When a group share is deleted we keep track of this in the DB. Right now it is only possible for a recipient to get back the share by asking the sharer to delete it and to share it again. This doesn't scale. This endpoint makes it possible to get back the share. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* move log constants to ILoggerArthur Schiwon2018-04-261-2/+2
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Fix existing usagesJoas Schilling2018-02-151-2/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Remove useless return statementsMorris Jobke2018-01-251-3/+0
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Properly log the full exception instead of only the messageMorris Jobke2018-01-231-2/+10
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Set the data from the templateJoas Schilling2017-10-181-9/+3
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Set the subject with the email template to allow themingJoas Schilling2017-10-181-7/+7
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* do not show hyphen after instance name in emails if slogan does not existStephan Müller2017-09-251-1/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge setMetaData into constructorJoas Schilling2017-09-041-6/+3
| | | | | | This ensures that the meta data is set in the beginning Signed-off-by: Joas Schilling <coding@schilljs.com>
* Implement metadata for email templates of remaining emailsMorris Jobke2017-08-301-0/+14
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Add shareWith to email template metadataMorris Jobke2017-08-291-0/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Allow the expiration date to be set to nullMorris Jobke2017-08-281-1/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Allow the expiration date to be set to nullMorris Jobke2017-08-261-2/+2
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Add meta information to emails for better customisationJoas Schilling2017-08-241-2/+11
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #5734 from nextcloud/only-readable-chars-in-share-tokensJoas Schilling2017-07-191-2/+1
|\ | | | | Only use readable chars in Share Tokens
| * Only use readable chars in Share TokensJoas Schilling2017-07-181-2/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Run phan over code baseLukas Reschke2017-07-191-1/+2
|/ | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* use the email address configured in Nextcloud as sender instead of the users ↵Bjoern Schiessle2017-06-081-2/+15
| | | | | | email address Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>