Log exceptions when creating share

Because the exceptions don't always contain a useful message for the UI,
but also because in some cases we need to find out what went wrong.

In some setups, a ShareNotFoundException might happen during creation
when we try to re-read the just written share. Usually related to Galera
Cluster where node syncing wait is not enabled.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Merge pull request #26059 from nextcloud/backport/25911/stable20

[stable20] Only clear share password model when actually saved

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Only clear share password model when actually saved

Instead of always clearing the password in the share model, now only do
it when it was actually saved.

Fixes an issue where saving another field would clear the password too
early and prevent it to be saved when the dropdown closed.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #26040 from nextcloud/backport/25916/stable20

[stable20] Remove trash items from other trash backends when deleting all

[tx-robot] updated from transifex

Merge pull request #26027 from nextcloud/backport/25933/stable20

[stable20] Hide expiration date field for remote shares

Hide expiration date field for remote shares

Remote shares currently do not support expiration date.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #25878 from nextcloud/backport/25568/stable20

[stable20] disable trasbin during the moveFromStorage fallback

Remove trash items from other trash backends when deleting all

Signed-off-by: Julius Härtl <jus@bitgrid.net>

[tx-robot] updated from transifex

Merge pull request #25938 from nextcloud/backport/25927/stable20

[stable20] Sharebymail: set expiration on creation

Merge pull request #25994 from nextcloud/backport/25975/stable20

[stable20] Add missing waits and asserts in acceptance tests

[tx-robot] updated from transifex

Merge pull request #26009 from nextcloud/dependabot/npm_and_yarn/stable20/elliptic-6.5.4

[Security] Bump elliptic from 6.5.3 to 6.5.4

[Security] Bump elliptic from 6.5.3 to 6.5.4

Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.3 to 6.5.4. **This update includes a security fix.**
- [Release notes](https://github.com/indutny/elliptic/releases)
- [Commits](https://github.com/indutny/elliptic/compare/v6.5.3...v6.5.4)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Find directly the label instead of falling back to it

The input element is always hidden, so the check always ended falling
back to the label. Moreover, the label is the element that the user
interacts with, so it must be the one used.

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

Assert also element visibility instead of just finding it

Although if the element could not be found an exception would be thrown
and the test aborted if an element is in the DOM but hidden it would be
found and the test would pass.

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

Add explicit locator for "Enable all" bundle button

Instead of looking for the bundle button and then checking its value now
the expected value is included in the locator and the button is checked
similarly to other elements.

No "Disable all" locator was added as it was not currently needed
anywhere.

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

Find elements through the actor rather than the driver

"Actor::find" is a more robust way to look for elements, as it handles
some exceptions that may be thrown. Therefore, even if the elements are
not actually used and it is only checked whether they exist or not using
the actor is the preferred way when possible (and it also makes it
consistent with the rest of the acceptance tests).

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

Add locator for apps list

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

Fix identation

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

Fix delete user acceptance test

Adding some missing asserts showed that the "delete user" acceptance
test was silently failing, as the deletion was not being confirmed in
the dialog and thus the user was not being deleted.

The dialog button contains a single quote ("user0's"), so the XPath
expression had to be adjusted (it seems that it is not possible to
escape a single quote in a string enclosed in single quotes in XPath
1.0).

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

Verify WaitFor::element... results with an assertion

WaitFor::element... calls only perform the waiting and return whether
the condition succeeded or not, but that result needs to be explicitly
checked to prevent further steps from being executed if the wait failed.

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

Add missing waits when finding elements in the acceptance tests

As no timeout was specified the elements were tried to be found just
once. This caused the steps to fail if the elements did not appear yet
in the page when they were tried to be found.

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

Merge pull request #25963 from nextcloud/backport/25712/stable20

[stable20] Fix admin password strengthify tooltip

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #25956 from nextcloud/backport/25932/stable20

[stable20] Skip empty obsolete owner when adding to own NC

[tx-robot] updated from transifex

Fix admin password strengthify tooltip

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>

Skip empty obsolete owner when adding to own NC

The owner field is nowadays always empty when adding a federated share
using "Add to your Nextcloud", so don't check for it.

Fixes an issue where "Add to your Nextcloud" doesn't add anything.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Merge pull request #25934 from nextcloud/backport/25926/stable20

[stable20] Activity: show if files are hidden or not

Merge pull request #25919 from nextcloud/backport/25805/stable20

[stable20] Clear multiselect after selection in share panel

[tx-robot] updated from transifex

Sharebymail: set expiration on creation

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

Activity: show if files are hidden or not

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

[tx-robot] updated from transifex

Clear multiselect after selection in share panel

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #25844 from nextcloud/backport/25757/stable20

[stable20] fix detecting cyclic group memberships

Merge pull request #25873 from nextcloud/backport/25869/stable20

[stable20] Fix namespacing

Merge pull request #25900 from nextcloud/backport/25860/stable20

[stable20] do not die after LDAP auth failed with expired acc

[tx-robot] updated from transifex

do not die after LDAP auth failed with expired acc

- some servers return error code 53

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

disable trasbin during the moveFromStorage fallback

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

Fix namespacing

Signed-off-by: Anna Larch <anna@nextcloud.com>

Merge pull request #25858 from nextcloud/backport/25856/stable20

[stable20] Card::getOwner should return the actual value

[tx-robot] updated from transifex

Card::getOwner should return the actual value

I guess we never call this at runtime or it would already ahve done boom
very loudly.

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

Merge pull request #25848 from nextcloud/backport/25715/stable20

[stable20] Bump the ca location

Bump the ca location

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

Merge pull request #25740 from nextcloud/backport/25602/stable20

[stable20] Avoid creating two share links when password is enforced

fix detecting cyclic group memberships

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Avoid creating two share links when password is enforced

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #25719 from nextcloud/version/20.0.8/final

20.0.8 final

[tx-robot] updated from transifex

Merge pull request #25709 from nextcloud/backport/25707/stable20

[stable20] Send emails on password reset to the displayname

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

20.0.8 final

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

[tx-robot] updated from transifex

Merge pull request #25494 from nextcloud/version/20.0.8/RC1

20.0.8 RC1

Merge pull request #25384 from nextcloud/backport/25218/stable20

[stable20] do not remove valid group shares

Merge pull request #25132 from nextcloud/backport/24661/stable20

[stable20] dont offer to edit external config settings if we can't edit them

fix test

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

Update apps/files_external/js/statusmanager.js

Co-authored-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Robin Appelman <robin@icewind.nl>

Send emails on password reset to the displayname

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

Merge pull request #25703 from nextcloud/backport/25700/stable20

[stable20] Initialize \OCP\AppFramework\Http\ZipResponse::$resources

[tx-robot] updated from transifex

Initialize \OCP\AppFramework\Http\ZipResponse::$resources

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Merge pull request #25448 from nextcloud/backport/25393/stable20

[stable20] add repair job for unencoded group share uris

bump affected version

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

compatibility to Nc 20

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

add missing changes from backport source

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

test: adjust expectation

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

show suggestion to clean up possible invalid shares later

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

take into account that UNIQUE index might not work as expected

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

add repair job for unencoded calendars

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Merge pull request #25379 from nextcloud/backport/25101/stable20

[stable20] LDAP: make actually use of batch read known groups

really use known groups

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Merge pull request #25651 from nextcloud/backport/25594/stable20

[stable20] Use ownerDisplayName for SharingEntry Tooltip

do not remove valid group shares

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

[tx-robot] updated from transifex

Merge pull request #25666 from nextcloud/backport/25661/stable20

[stable20] Updated CRL

[tx-robot] updated from transifex

Updated CRL

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

Merge pull request #25663 from nextcloud/backport/25655/stable20

[stable20] Fix app icon list in dark mode

Fix app icon list in dark mode

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

Use ownerDisplayName for SharingEntry Tooltip

Some NC installations may use a LDAP UUID for the username. If that
is the case, then the tooltip for who a file/folder was shared by may say something
like `Shared with Steve McQueen (smqueen@yada.com) by FKDJ39-38D794JG-4398GJODF9H3G-4534`.

This commit changes the value showed for the owner to the full display name.

Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex