diff options
| author | Morris Jobke <hey@morrisjobke.de> | 2018-10-12 17:01:52 +0200 |
|---|---|---|
| committer | Morris Jobke <hey@morrisjobke.de> | 2018-10-12 17:01:54 +0200 |
| commit | 24a2107e035bf4658f611d070fc2b3f641191430 (patch) | |
| tree | 3406aab701b12cfd138652b0b28e1eba256aa4e6 /apps/sharebymail/lib/ShareByMailProvider.php | |
| parent | 85694c6d766e90385c024c5325c1e0217855b44a (diff) | |
| download | nextcloud-server-24a2107e035bf4658f611d070fc2b3f641191430.tar.gz nextcloud-server-24a2107e035bf4658f611d070fc2b3f641191430.zip | |
Fix a case where "password_by_talk" was not a boolean (e.g. null or "0") and 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>
Diffstat (limited to 'apps/sharebymail/lib/ShareByMailProvider.php')
| -rw-r--r-- | apps/sharebymail/lib/ShareByMailProvider.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/apps/sharebymail/lib/ShareByMailProvider.php b/apps/sharebymail/lib/ShareByMailProvider.php index 26a9beefdfc..b11a9349371 100644 --- a/apps/sharebymail/lib/ShareByMailProvider.php +++ b/apps/sharebymail/lib/ShareByMailProvider.php @@ -980,7 +980,7 @@ class ShareByMailProvider implements IShareProvider { $share->setShareTime($shareTime); $share->setSharedWith($data['share_with']); $share->setPassword($data['password']); - $share->setSendPasswordByTalk($data['password_by_talk']); + $share->setSendPasswordByTalk((bool)$data['password_by_talk']); if ($data['uid_initiator'] !== null) { $share->setShareOwner($data['uid_owner']); |