Unlock when promoting to exclusive lock fails

In certain cases changeLock to EXCLUSIVE fails
and throws LockedException. This leaves the
file locked as SHARED in file_put_contents,
which prevents retrying (because on second
call file_put_contents takes another SHARED
lock on the same file, and changeLock doesn't
allow more than a single SHARED lock to promote
to EXCLUSIVE).

To avoid this case, we catch the LockedException
and unlock before re-throwing.

Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>

Merge pull request #23971 from nextcloud/backport/23936/stable20

[stable20] Use query builder instead of OC_DB in trashbin

Check in failing psalm checks

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

Merge pull request #23975 from nextcloud/backport/23923/stable20

[stable20] Fix greatest/least order for oracle

Add explicit typecast for $value.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Update parameters

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

Make sure Oracle always casts everything in the best way

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

Use Query builder function

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

Fix order of GREATEST for Oracle

As per https://docs.oracle.com/cd/B19306_01/server.102/b14200/functions060.htm
Oracle uses the first value to cast the rest or the values.
So when the first value is a plain int, instead of doing the math,
it will cast the expression to int and continue with a potential 0.

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

Merge pull request #23951 from nextcloud/backport/23922/stable20

[stable20] Improve query type detection

Use query builder instead of OC_DB in trashbin

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

Merge pull request #23954 from nextcloud/backport/23943/stable20

[stable20] Fix iLike() falsely turning escaped % and _ into wildcards

Merge pull request #23948 from nextcloud/backport/23940/stable20

[stable20] Skip already loaded apps in loadApps

[tx-robot] updated from transifex

Fix iLike() falsely turning escaped % and _ into wildcards

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

Merge pull request #23935 from nextcloud/backport/23921/stable20

[stable20] Fix casting of integer and boolean on Oracle

Improve query type detection

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

Merge pull request #23924 from nextcloud/backport/23920/stable20

[stable20] Bearer must be in the start of the auth header

Skip already loaded apps in loadApps

Otherwise you might end up calling a lot of functions unneeded.
And while the individual calls are cheap if you multiply them by 20k
they still get somewhat expensive.

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

Cast bool things to numbers as that is how they are stored in the database

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

Fix casting to integer on Oracle

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

Merge pull request #23917 from nextcloud/backport/23887/stable20

[stable20] Do not emit UserCreatedEvent twice

Bearer must be in the start of the auth header

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

[tx-robot] updated from transifex

fix: also remove use statement of UserCreatedEvent

Signed-off-by: Tobias Assmann <tobias.assmann@ecsec.de>

fix: no more translation from postCreateUser hook to UserCreatedEvent anymore, as event is already emitted in user manager

Signed-off-by: Tobias Assmann <tobias.assmann@ecsec.de>

Merge pull request #23902 from nextcloud/backport/23798/stable20

[stable20] Add default font size

Add default font size

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

[tx-robot] updated from transifex

Merge pull request #23894 from nextcloud/backport/23799/stable20-minimal

[stable20] fix potentially passing null to events where IUser is expected

Merge pull request #23899 from nextcloud/backport/23898/stable20

[stable20] Make user status styles scoped

Merge pull request #23900 from nextcloud/backport/23897/stable20

[stable20] Move help to separate stylesheet

Make user status styles scoped

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

Merge pull request #23884 from nextcloud/backport/23097/stable20

[stable20] Restrict query when searching for versions of trashbin files

Move help to separate stylesheet

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

fix potentially passing null to events where IUser is expected

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

Merge pull request #23419 from nextcloud/backport/23350/stable20

[stable20] Inidicate preview availability in share api responses

Restrict query when searching for versions of trashbin files

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

Merge pull request #23874 from nextcloud/backport/23866/stable20

[stable20] Fix grid view toggle in tags view

[tx-robot] updated from transifex

Indicate preview availability in share api responses

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #23858 from nextcloud/backport/23842/stable20

[stable20] UserStatus: clear status message if message is null

Fix grid view in tags view

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

Merge pull request #23850 from nextcloud/backport/23841/stable20

[stable20] ensure filepicker list is empty before populating

Merge pull request #23846 from nextcloud/backport/23839/stable20

[stable20] No need to check if there is an avatar available, because it is gener…

UserStatus: no message means clear status message. This fixes #23332

Signed-off-by: Simon Spannagel <simonspa@kth.se>

[tx-robot] updated from transifex

ensure filepicker list is empty before populating

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

No need to check if there is an avatar available, because it is generated on the fly nowadays

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

Merge pull request #23829 from nextcloud/backport/23267/stable20

[stable20] Remove bold font-weight and lower font-size for empty search box

[tx-robot] updated from transifex

Change font-weight and font-size for empty search

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

[tx-robot] updated from transifex

Merge pull request #23794 from nextcloud/backport/23788/stable20

[stable20] Only run phpunit when "php" changed

[tx-robot] updated from transifex

Add script to old versions

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

Also run for .sh and .json (when not package or package-lock)

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

Also run all tests when drone config is modified

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

Only run phpunit when php, xml or test files are modified

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

Merge pull request #23789 from nextcloud/backport/23742/stable20

[stable20] Type the event dispatcher listener callables with Psalm

Merge pull request #23787 from nextcloud/backport/23783/stable20

[stable20] Fetch the logger and system config once for all query builder instances

Fix invalid usages of the new dispatcher

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

Type the event dispatcher listener callables with Psalm

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

Fetch the logger and system config once for all query builder instances

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

[tx-robot] updated from transifex

Merge pull request #23779 from nextcloud/backport/23775/stable20

[stable20] Document the backend registered events of LDAP

Document the backend registered events of LDAP

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

Merge pull request #23774 from nextcloud/backport/23762/stable20

[stable20] catch errors when closing file conflict dialog

catch errors when closing file conflict dialog

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #23759 from nextcloud/backport/23752/stable20

[stable20] inform the user when flow config data exceeds thresholds

[tx-robot] updated from transifex

Merge pull request #23763 from nextcloud/backport/23746/stable20

[stable20] Type the \OCP\IUserManager::callForSeenUsers closure with Psalm

Merge pull request #23751 from nextcloud/backport/23744/stable20

[stable20] Type the \OCP\AppFramework\Services\IInitialState::provideLazyInitial…

Type the \OCP\IUserManager::callForSeenUsers closure with Psalm

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

inform the user when flow config data exceeds thresholds

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

Merge pull request #23749 from nextcloud/backport/23743/stable20

[stable20] Type the \OCP\IUserManager::callForAllUsers closure with Psalm

Merge pull request #23753 from nextcloud/backport/23750/stable20

[stable20] Don't overwrite the event if we use it later

Don't overwrite the event if we use it later

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

Type the \OCP\AppFramework\Services\IInitialState::provideLazyInitialState closure with Psalm

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

Type the \OCP\IUserManager::callForAllUsers closure with Psalm

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

Merge pull request #23745 from nextcloud/backport/23723/stable20

[stable20] Also run the SabreAuthInitEvent for the main server

Also run the SabreAuthInitEvent for the main server

else for example trusted servers breaks

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

Merge pull request #23736 from nextcloud/backport/23721/stable20

[stable20] reset avatar cropper before showing

Merge pull request #23733 from nextcloud/backport/23720/stable20

[stable20] save email as lower case

Merge pull request #23732 from nextcloud/backport/23725/stable20

[stable20] Specify accept argument to avatar upload input field

[tx-robot] updated from transifex

reset avatar cropper before showing

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

save email as lower case

email addresses are case insensitive

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

Specify accept argument to avatar upload input field

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

Merge pull request #23726 from nextcloud/backport/23711/stable20

[stable20] Fix default quota display value in user row

Merge pull request #23727 from nextcloud/backport/23722/stable20

[stable20] Use lib instead if core as l10n module in OC_Files

Merge pull request #23710 from nextcloud/backport/23701/stable20

[stable20] Make sure the function signatures of the backgroundjob match

Use lib instead if core as l10n module in OC_Files

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

Fix default quota display value in user row

When selecting default quota in the dropdown, the actual numeric value
or "Unlimited" will be displayed instead of "0 B". This matches whatever
is displayed after refreshing the page.

In the case where the default quota is unlimited, for which the server
returns -3 instead of "none", the frontend now also shows "Unlimited"
instead of "0 B".

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

Merge pull request #23713 from nextcloud/backport/23645/stable20

[stable20] Check if array elements exist before using them

Check if array elements exist before using them

It seems that in some recent upgrade here. Not always the entries we
expect are returned. So we should first check if they exist. As to not
spam the log.

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

Make sure the function signatures of the backgroundjob match

Else PHP says no

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

Merge pull request #23702 from nextcloud/fix/noid/remnants-sharee-display-20-and-older