Merge pull request #6368 from nextcloud/backport-5436-fix-group-check

[stable12] Fix group check on share provider

Merge pull request #6357 from nextcloud/swift-tmpfiles-12

[12] Remove tmpFiles Array in Swift.php

[tx-robot] updated from transifex

Merge pull request #6366 from nextcloud/backport-6358-email-update

Backport email update

Fix 500 Internal Server Error on writing

In some not yet completely determined configurations, the following error could occur while writing a file:

Error: Call to a member function getUsers() on null
    /var/www/nextcloud/lib/private/Share20/Manager.php - line 1277: OC\Share20\DefaultShareProvider->getAccessList(Array, true)
    /var/www/nextcloud/lib/private/Share20/ShareHelper.php - line 51: OC\Share20\Manager->getAccessList(Object(OC\Files\Node\Folder), true, true)
    /var/www/nextcloud/apps/activity/lib/FilesHooks.php - line 616: OC\Share20\ShareHelper->getPathsForAccessList(Object(OC\Files\Node\File))
    /var/www/nextcloud/apps/activity/lib/FilesHooks.php - line 196: OCA\Activity\FilesHooks->getUserPathsFromPath('/path/to/file', 'user')
    /var/www/nextcloud/apps/activity/lib/FilesHooks.php - line 157: OCA\Activity\FilesHooks->addNotificationsForFileAction('/path/to/file', 'file_changed', 'changed_self', 'changed_by')
    /var/www/nextcloud/apps/activity/lib/FilesHooksStatic.php - line 55: OCA\Activity\FilesHooks->fileUpdate('/path/to/file')
    /var/www/nextcloud/lib/private/legacy/hook.php - line 106: OCA\Activity\FilesHooksStatic fileUpdate(Array)
    /var/www/nextcloud/lib/private/Files/View.php - line 1245: OC_Hook emit('OC_Filesystem', 'post_update', Array)
    /var/www/nextcloud/lib/private/Files/View.php - line 1173: OC\Files\View->runHooks(Array, '/path/to/file', true)
    /var/www/nextcloud/lib/private/Files/View.php - line 679: OC\Files\View->basicOperation('file_put_conten...', '/path/to/file', Array, '<?xml version="...')
    /var/www/nextcloud/lib/private/Files/Node/File.php - line 64: OC\Files\View->file_put_contents('/path/to/file', '<?xml version="...')
    [...]

Signed-off-by: Jan-Philipp Litza <janphilipp@litza.de>

Fix tests

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Fix unit tests

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge setMetaData into constructor

This ensures that the meta data is set in the beginning

Signed-off-by: Joas Schilling <coding@schilljs.com>

Set the meta data before everything

Signed-off-by: Joas Schilling <coding@schilljs.com>

Fix naming of user id in email meta data

Signed-off-by: Joas Schilling <coding@schilljs.com>

Implement metadata for email templates of remaining emails

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

[tx-robot] updated from transifex

Remove tmpFiles Array in Swift.php

tmpFiles Array is unnecessary and there has been
a reference without an assignment which lead to a bug making swift external
storage unuseable.

Merge pull request #6355 from nextcloud/6221-12

[stable12] Correctly format OCS response with favorites

Merge pull request #6333 from nextcloud/new-dav-event-stable12

[stable12] add a new dav event to allow apps to register their own sabredav plugins

Correctly format OCS response with favorites

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>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

also allow to register additional plugins if the old endpoint is used

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

add a new dav event to allow apps to register their own sabredav plugins

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #6285 from nextcloud/generate-system-wide-key-pair-12

Generate system wide key pair

Merge pull request #5841 from nextcloud/bugfix-stable12/2855/dont_send_invitations_for_past_events

[stable12] dont send invitations for past events

update autoloader

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

move repair step to stable12

because we decided to backport it the repair step needs to be executed
already on stable12

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

Bump version

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

add prefix to user and system keys to avoid name collisions

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

extend the identity proof manager to allow system wide key pairs

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

Merge pull request #6311 from nextcloud/stable12-specify-slo-url

[Stable12] specify slo url

Inject \OCP\IURLGenerator to make tests work

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Properly allow \OCP\Authentication\IApacheBackend to specify logout URL

Any `\OCP\Authentication\IApacheBackend` previously had to implement `getLogoutAttribute` which returns a string.
This string is directly injected into the logout `<a>` tag, so returning something like `href="foo"` would result
in `<a href="foo">`.

This is rather error prone and also in Nextcloud 12 broken as the logout entry has been moved with
054e161eb5f4a5c5c13ee322ae8e93ce66f01b13 inside the navigation manager where one cannot simply inject attributes.

Thus this feature is broken in Nextcloud 12 which effectively leads to the bug described at nextcloud/user_saml#112,
people cannot logout anymore when using SAML using SLO. Basically in case of SAML you have a SLO url which redirects
you to the IdP and properly logs you out there as well.

Instead of monkey patching the Navigation manager I decided to instead change `\OCP\Authentication\IApacheBackend` to
use `\OCP\Authentication\IApacheBackend::getLogoutUrl` instead where it can return a string with the appropriate logout
URL. Since this functionality is only prominently used in the SAML plugin. Any custom app would need a small change but
I'm not aware of any and there's simply no way to fix this properly otherwise.

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

[tx-robot] updated from transifex

Merge pull request #6300 from nextcloud/add-share-with

[stable12] Add shareWith to email template metadata

Add shareWith to email template metadata

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #6278 from nextcloud/stable12-6182-fixes-in-files-drop-js

[stable12] Fixes in files_drop.js

Merge pull request #6284 from nextcloud/add-new-propfind-stable12

add propfind to detect encryption status

[tx-robot] updated from transifex

Merge pull request #6287 from nextcloud/12-6255

[stable12] Add meta information to emails for better customisation

Allow the expiration date to be set to null

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Allow the expiration date to be set to null

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Also for reset password

Signed-off-by: Joas Schilling <coding@schilljs.com>

Add meta information to emails for better customisation

Signed-off-by: Joas Schilling <coding@schilljs.com>

add propfind to detect encryption status

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

[tx-robot] updated from transifex

Do not iterate over the files

As "singleFileUpload" is used the "add" callback (which in turn calls
"addFileToUpload") will always be called with a single file. Therefore
there is no need to iterate over the files (and it is not done in the
other callbacks either, so this aligns the code with the rest of the
callbacks).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Replace fileName variable with data.files[0].name

There is no need to store the file name, as the "data" parameter given to
all the callbacks provides a "files" attribute with all the files that
the callback refers to; moreover, it will always be a single file due to
the use of "singleFileUploads" in the jQuery File Upload plugin.

This also fixes the loading icon not disappearing when several files were
uploaded at once. "singleFileUploads" causes the "add" callback to be
called once for each file to be uploaded, so "fileName" was overwritten
with the name of each new file in the upload set; when "fileName" was
later used in the "done" callback to find the file in the list whose
loading icon replace with the MIME type icon "fileName" always had the
name of the last file, and thus its icon was the only one replaced.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Update the whole file item instead of only its contents

The "done" and "fail" callbacks both update the item for the uploaded
file using "setFileIcon". "setFileIcon" updates the contents of the
"<li>" element for the file, but the "fail" callback was giving
"setFileIcon" an element generated by the template,  so the resulting
HTML contained a "<li>" element nested in another "<li>" element.

However, generating the HTML is better done through a template, so the
template now receives the icon to show in order to be used by a
successful upload and a failed one, and "setFileIcon" was changed to
"updateFileItem".

Note that the mimeTypeUrl does no longer need to be escaped, as
Handlebars templates escape the needed characters automatically.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Remove "errorThrown" parameter

When the "fail" callback is called, "errorThrown" is a property of the
data object instead of a parameter.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Merge pull request #6270 from nextcloud/6252-12

[stable12] Don't try the actual file upload if the checks already error out

Merge pull request #6269 from nextcloud/6251-12

[stable12] Don't fetch the translation from core but from files

Merge pull request #6268 from nextcloud/smb-2.0.2-12

[12]  update icewind/smb to 2.0.2

Merge pull request #6243 from nextcloud/stable12-circles-token

using CircleProvider on token

Don't try the actual file upload if the checks already error out

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Don't fetch the translation from core but from files

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

update icewind/smb to 2.0.2

Signed-off-by: Robin Appelman <robin@icewind.nl>

[tx-robot] updated from transifex

Merge pull request #6258 from nextcloud/stable12_6244

[stable12] [drone] Give redis a change to fireup

[drone] Give redis a change to fireup

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

[tx-robot] updated from transifex

Merge pull request #6114 from nextcloud/stable12_5852

[stable12] Delete trash using node API to trigger preview deletion

Merge pull request #6108 from nextcloud/backport-6080-12

[stable12] Add buttom border for header

Merge pull request #6233 from nextcloud/stable12_6218

[stable12] [3rdparty] Fix sabre/xml empty prop parsing

[tx-robot] updated from transifex

Merge pull request #6240 from nextcloud/12-6239

[stable12] Show "Uploading..." in web UI for long uploads

using CircleProvider on token

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

Merge pull request #6237 from nextcloud/12-drone-ci-images

[stable12] Use docker images without XDebug and with Opcache

Merge pull request #6166 from nextcloud/stable12-bump-dompurify

[Stable12] Bump dompurify

Merge pull request #6151 from nextcloud/backport-6099-no-exception-on-sharee-error

[stable12] Catch exceptions on error of cloud id resolution

Merge pull request #6148 from nextcloud/backport-6047-select-indexed-columns

[stable12] Use indexed column path_hash to find the parent

Show "Uploading..." in web UI for long uploads

If the estimated upload time is bigger than 4 hours it shows the text
"Uploading..." because the time then doesn't give any good hint to the
user anyways.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Use docker images without XDebug and with Opcache

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

[3rdparty] Fix sabre/xml empty prop parsing

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

[tx-robot] updated from transifex

Merge pull request #6197 from nextcloud/stable12-check-encoding-log

[Stable12] check encoding log

[tx-robot] updated from transifex

Merge pull request #6207 from nextcloud/12-6201

[stable12] Increase contrast of dropdown menu text, fix accessibility & visibility issues

Increase contrast of dropdown menu text, fix accessibility & visibility issues

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Merge pull request #6195 from nextcloud/12-6194

[stable12] Use correct l10n for app password menu

Ensure log message is UTF-8 encoded

PHP's json_encode only accept proper UTF-8 strings, loop over all
elements to ensure that they are properly UTF-8 compliant or convert
them manually.

Without this somebody passing an invalid User Agent may make json_encode
return false which will get logged as empty newline.

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Use correct l10n for app password menu

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #6173 from nextcloud/12-6161

[stable12] Enable object storage tests again

[tx-robot] updated from transifex

Enable object storage tests again

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Build merged JS

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Bump DOMPurify to 1.0.0

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Merge pull request #6127 from nextcloud/12-6126

[stable12] Trigger search after re-focus

[tx-robot] updated from transifex

Merge pull request #6150 from nextcloud/backport-6128-primary-font-color-in-capabilities

[stable12] Add the primary font color to the capabilities as well

Document that it throws

Signed-off-by: Joas Schilling <coding@schilljs.com>

Catch exceptions on error of cloud id resolution

Signed-off-by: Joas Schilling <coding@schilljs.com>

Add the primary font color to the capabilities as well

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #6111 from nextcloud/backport-6098-12

[stable12] Fix federation scope dropdown

Merge pull request #6112 from nextcloud/backport-5920-12

[stable12] Top bar buttons in share view are wrong positioned

Use indexed column path_hash to find the parent

Signed-off-by: Joas Schilling <coding@schilljs.com>

[tx-robot] updated from transifex

Trigger search after re-focus

If the share input field is unfocused, the autocomplete list is closed. Once
the field was focused again it was not properly opened again. This adds a
trigger to redo the search and show the results again.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>