fix(share): Ensure unique share tokens

- check for token collisions and retry up to three times.
- throw after 3 attempts without finding a unique token.

Signed-off-by: ernolf <raphael.gradenwitz@googlemail.com>

Merge pull request #48367 from nextcloud/backport/48354/stable28

[stable28] fix: Show storage full warning when storage is actually full

fix: Show storage full warning when storage is actually full

< 0 values for "free" do not mean that the storage is full: https://github.com/nextcloud/server/blob/6befdd6dd7cd20fe7f7036e4665bcfbb783d6803/lib/public/Files/FileInfo.php#L31-L42

Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #48318 from nextcloud/backport/48315/stable28

[stable28] refactor(AccountIcon): Remove needless console.log

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48352 from nextcloud/backport/48331/stable28

[stable28] fix(files): Fix having to resolve conflicts twice when dropping files

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(files): Fix having to resolve conflicts twice when dropping files

Signed-off-by: Christopher Ng <chrng8@gmail.com>

Merge pull request #48339 from nextcloud/automated/noid/stable28-update-ca-cert-bundle

[stable28] fix(security): Update CA certificate bundle

fix(security): Update CA certificate bundle

Signed-off-by: GitHub <noreply@github.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

refactor(AccountIcon): Remove needless console.log

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>

Merge pull request #48310 from nextcloud/backport/47349/stable28

[stable28] Add bidirectional text support - Backend

ci: Restrict RTL characters to RTL languages

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

Merge pull request #48283 from nextcloud/backport/48207/stable28

[stable28] fix: Don't crash if disabled user is missing in the database

Merge pull request #48281 from nextcloud/backport/48266/stable28

Merge pull request #48300 from nextcloud/backport/48268/stable28

[stable28] fix: skip the integrity check for nextcloud-init-sync.lock

fix: skip the integrity check for nextcloud-init-sync.lock

nextcloud-init-sync.lock is used by nextcloud/docker to prevent running the initialization script on multiple containers at the same time.

Ref: https://github.com/nextcloud/docker/issues/2299.

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

fix(users): Don't crash if disabled user is missing in the database

Signed-off-by: Louis Chemineau <louis@chmn.me>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48292 from nextcloud/backport/48282/stable28

[stable28] chore: add .git-blame-ignore-revs

chore: add .git-blame-ignore-revs

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

fix(psalm): Enable findUnusedBaselineEntry

Signed-off-by: provokateurin <kate@provokateurin.de>

Merge pull request #48058 from nextcloud/automated/noid/stable28-update-psalm-baseline

Merge pull request #48099 from nextcloud/backport/46140/stable28

[stable28] fix(config): Add missing handling for `envCache` in `getKeys()`

chore(tests): Update psalm baseline

Signed-off-by: GitHub <noreply@github.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48166 from nextcloud/backport/48114/stable28

[stable28] fix(appstoreFetcher): get list from a custom store and remove unnecessary warning

Merge pull request #48091 from nextcloud/backport/47316/stable28

fix: Prevent crash when refreshing metadata for files without an owner

This crash was mostly happening when using external storages and for example the command "occ memories:migrate-google-takeout"

Signed-off-by: Vincent FarZz <farcry69@live.fr>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #47441 from nextcloud/44287-fix-avatar-fed-share-receiver

[stable28] fix(FileList): Show correct avatar for federated share owner

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(Federation): Show some icon for federated users on shares

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

fix(appstoreFetcher): get app list from custom app store

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

fix(appstoreFetcher): remove unneeded warning

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48082 from nextcloud/backport/48081/stable28

[stable28] chore(1i8n): Fixed grammar and improved wording

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48103 from nextcloud/backport/46218/stable28

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48085 from nextcloud/backport/47924/stable28

[stable28] 🥅 — Catch MaxInstancesExceededException on calendar events

Merge pull request #48089 from nextcloud/backport/48008/stable28

chore: improve hash_file php usage in Local Storage

Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>

fix(files): Ensure that the hash method does not return null

To match https://github.com/nextcloud/server/blob/beececf66068f57c416225efcde9b44ce5c2e835/lib/private/Files/View.php#L1050

- Fix https://github.com/nextcloud/server/issues/44110

Signed-off-by: Louis Chemineau <louis@chmn.me>

fix: switch from explode to substr (faster)

Signed-off-by: Josh Richards <josh.t.richards@gmail.com>

fix(config): add envCache tests for getKeys()

Signed-off-by: Josh Richards <josh.t.richards@gmail.com>

fix(config): Add missing handling for envCache in getKeys()

NC_ env variable overrides were not appearing in
the output of `occ config:list system` nor `occ
config:system:get xxx`. This was creating nearly
impossible to diagnose configuration/ behavior
disprepancies.

- Refactored readData() so that we aren't saving
  the entire environment in the envCache anymore
  (only those prefixed "NC_") and so that we save
  NC_ provided config values under their real
  key.
- Refactored getValue() to accommodate readData()
  refactor
- Fixed getKeys() to properly return
  envCache keys too

Environment provided config variables now appear
in `occ config:list system` as expected.

Environment provided config variables now appear
when queried via `occ config:system:get KEY`

envCache is now free of non-NC stuff.

Signed-off-by: Josh Richards <josh.t.richards@gmail.com>

fix(Entity): Fix magic setter call for custom strong typed setters

Signed-off-by: provokateurin <kate@provokateurin.de>

fix(dav): catch `MaxInstancesExceededException` on calendar events

Signed-off-by: Luc Didry <luc@didry.org>

chore(1i8n): Fixed grammar and improved wording

Reported at Transifex

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
[skip ci]

Merge pull request #48066 from nextcloud/backport/48030/stable28

Merge pull request #48002 from nextcloud/backport/47920/stable28

[stable28] feat: Reset route if neither the Viewer of the Sidebar is open

chore: Compile assets

Signed-off-by: Louis Chemineau <louis@chmn.me>

feat: Reset route if neither the Viewer of the Sidebar is open

When the viewer or the sidebar is opened, we add the fileid to the route.
When both of them are closed, we do not remove the fileid from the route.
This means that, upon reload, the sidebar will be opened even though it was closed previously.

This PR ensure that the fileid is removed from the route when both the Sidebar and the Viewer are closed.

Signed-off-by: Louis Chemineau <louis@chmn.me>

fix(dav): cast parameters to string

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

Merge pull request #47870 from nextcloud/backport/47834/stable28

Merge pull request #47866 from nextcloud/backport/47858/stable28

Merge pull request #47942 from nextcloud/backport/47933/stable28

[stable28] fix(config): Throw PreconditionException always when it didn't match

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #48051 from nextcloud/backport/48045/stable28

[stable28] fix(dav): set string type for sanitizeMtime

fix(dav): set string type for sanitizeMtime

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

Merge pull request #47893 from nextcloud/backport/47339/stable28

[stable28] fix: Display 'Leave share' instead of 'Delete'

Merge pull request #47876 from nextcloud/backport/47848/stable28

[stable28] fix(files): Reset context menu position on close

Merge pull request #47935 from nextcloud/backport/47854/stable28

[stable28] fix(appstore): Ensure returned apps from AppStore are valid

Merge pull request #47979 from nextcloud/backport/47971/stable28

[stable28] fix: Do not try to set HTTP response code on already closed connection

Merge pull request #48032 from nextcloud/backport/48013/stable28

[stable28] fix(BaseResponse): Cast XML element values to string

Merge pull request #47917 from nextcloud/backport/47914/stable28

Merge pull request #47811 from nextcloud/backport/47649/stable28

Merge pull request #47861 from nextcloud/backport/47860/stable28

Merge pull request #48019 from nextcloud/backport/48015/stable28

Merge pull request #47988 from nextcloud/automated/noid/stable28-update-psalm-baseline

[stable28] Update psalm-baseline.xml

fix(BaseResponse): Cast XML element values to string

Signed-off-by: provokateurin <kate@provokateurin.de>

Merge pull request #48022 from nextcloud/backport/48017/stable28

Merge pull request #48023 from nextcloud/backport/48014/stable28

fix(Tags): Return boolean for userHasTags()

Signed-off-by: provokateurin <kate@provokateurin.de>

fix(Comment): Initialize childrenCount as integer

Signed-off-by: provokateurin <kate@provokateurin.de>

fix(Storage\Local): Do not call getSourcePath() on SplFileInfo

Signed-off-by: provokateurin <kate@provokateurin.de>

chore(tests): Update psalm baseline

Signed-off-by: GitHub <noreply@github.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

fix: Do not try to set HTTP response code on already closed connection

This avoids a PHP warning in the logs about trying to set the response
 code while the output already started. It’s useless to try to print an
 error page anyway in this situation because the connection was closed
 already.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #47974 from nextcloud/backport/47968/stable28

fix(files_external): Check key exists before accessing it

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>

Merge pull request #47956 from nextcloud/backport/47954/stable28

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #47952 from nextcloud/backport/47945/stable28

[stable28] fix(files_external): broken credentials dialog

fix(files): Adjust Cache::searchQuery() parameter name to match interface

Signed-off-by: provokateurin <kate@provokateurin.de>

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(files_external): broken credentials dialog

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
[skip ci]

fix(config): Throw PreconditionException always when it didn't match

Previously even when the precondition did not match, the call "passed"
when the after value was the expected one. This however can lead to
race conditions, duplicate code excutions and other things.

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

fix: check if key is set

Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>

fix(appstore): Ensure returned apps from AppStore are valid

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
fix: lint

chore: remove space

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
fix: check if response array is null

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
chore: Add log

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #47886 from nextcloud/release/28.0.10