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 #47976 from nextcloud/backport/47968/stable30

fix(files_external): Check key exists before accessing it

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

Merge pull request #47957 from nextcloud/backport/47954/stable30

Fix(l10n): Update translations from Transifex

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

Merge pull request #47951 from nextcloud/backport/47945/stable30

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>

Fix(l10n): Update translations from Transifex

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

Merge pull request #47926 from nextcloud/release/30.0.0

build(hub): 30.0.0

Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>

Merge pull request #47890 from nextcloud/release/30.0.0_rc5

Merge pull request #47915 from nextcloud/backport/47913/stable30

[stable30] fix: Add whiteboard to education and public sector bundle

Merge pull request #47882 from nextcloud/backport/47294/stable30

[stable30] fix(files): Sort tree views correctly by natural order

fix: Add whiteboard to education and public sector bundle

Signed-off-by: Julius Knorr <jus@bitgrid.net>

Merge pull request #47897 from nextcloud/backport/47586/stable30

[stable30] fix(theming): Add migration to restore primary color after separating primary and background

Merge pull request #47912 from nextcloud/backport/47906/stable30

[stable30] chore: Update trashbin retention documentation with exceeded quota details

chore: Update trashbin retention documentation with exceeded quota details

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

Fix(l10n): Update translations from Transifex

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

fix: Adjust types of IQueryBuilder to properly allow joining with sub-query

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix: Use migration instead of repair step for restoring custom color

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

chore: Use `IAppConfig` instead of `IConfig->getAppValue`

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix: Add repair step to restore primary color after separating primary and background

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix(theming): Allow to specify a userId manually in BackgroundService

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

build(hub): 30.0.0 RC5

Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.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>

fix(files): Render folders in natural sort order
- Nodes are returned from the endpoint in an undefined order

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

fix(files): Sort tree views correctly by natural order

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

chore(deps): Bump `@nextcloud/files` to v3.9.0

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

Merge pull request #47810 from nextcloud/backport/47649/stable30

[stable30] fix: Fix inaccessible content on public pages due to overlapping footer

Merge pull request #47874 from nextcloud/backport/47853/stable30

[stable30] fix: make swift connect exception message more informative

Merge pull request #47863 from nextcloud/backport/47860/stable30

[stable30] fix: Only write once to template instead of create/copy

fix: make swift connect exception message more informative

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

Merge pull request #47839 from nextcloud/backport/47640/stable30

[stable30] fix: Run migration sql as statement so that the primary db node is used

Merge pull request #47851 from nextcloud/backport/47807/stable30

[stable30] fix(files_sharing): password checkbox and labels in link share

Merge pull request #47842 from nextcloud/backport/47805/stable30

[stable30] fix: Replace conflicting tags in `xml_exception` template

fix: Only write once to template instead of create/copy

Signed-off-by: Julius Knorr <jus@bitgrid.net>

Merge pull request #47855 from nextcloud/backport/47845/stable30

[stable30] Add an endpoint to remove all webhook registrations from an app

Fix(l10n): Update translations from Transifex

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

chore(assets): Recompile assets

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

fix(files_sharing): add accessible labels in link share creation

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

chore(webhook_listeners): Update openapi.json

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

feat(webhook_listeners): Add API endpoint to remove all registrations from an AppAPI app

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

feat(webhook_listeners): Add mapper method to remove all registrations from a given AppAPI id

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

fix(files_sharing): checkbox for password in link share is not shown

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

fix: Replace conflicting tags in `xml_exception` template

The `<?xml` tag is interpreted as PHP short tags, so this causes errors.
Instead just print that part of the template.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix: Run migration sql as statement so that the primary db node is used

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

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>

chore(assets): Recompile assets

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

chore: compile scss

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

fix: Fix inaccessible content on public pages due to overlapping footer

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

Fix(l10n): Update translations from Transifex

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

Merge pull request #47787 from nextcloud/backport/47770/stable30

[stable30] fix(dav): Always respond custom error page on exceptions

fix(dav): Always respond custom error page on exceptions

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

Merge pull request #47757 from nextcloud/release/30.0.0_rc4

30.0.0 RC4

Merge pull request #47775 from nextcloud/backport/46255/stable30

[stable30] fix: add option to remove the webroot for setup checks and don't chec…

Merge pull request #47781 from nextcloud/backport/47519/stable30

[stable30] feat(transfer-ownership): Correctly react to encrypted files

Merge pull request #47778 from nextcloud/backport/47769/stable30

[stable30] fix: Use sha256 to hash arguments of background jobs

fix(transfer-ownership): Improve comments according to code review

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

feat(transfer-ownership): Correctly react to encrypted files

For E2EE encrypted files, we abort the transfer.
For SSE encrypted files, we abort only if not using master key.
Also fixed the check for when the path to a single file is used.

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

fix: Migrate existing bg jobs to use sha256

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

fix: Use sha256 to hash arguments of background jobs

This is to prevent collision as we are sometime hashing user input, yet using that hash to target the background job in the database.

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

fix(setupchecks): Test overwrite.cli url first, then generated one, and

 trusted domains as last fallback.

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

fix: add option to remove the webroot for setup checks and don't check trusted_domains.

1) The checks for well-known urls should always run against the root domain and therefore the option to remove the webroot.

2) For trusted domains, the available protocol is unknown, and thus some guesswork would be needed to make that work. I've decided for now to not consider them anymore to reduce false-positives.

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

Merge pull request #47484 from nextcloud/backport/46013/stable30

[stable30] fix: write object to the correct urn when moving from another storage to object store

Merge pull request #47753 from nextcloud/backport/47745/stable30

[stable30] fix(files): undefined displayname for outdated `@nextcloud/files`<3.6.0 apps

Merge pull request #47764 from nextcloud/backport/47754/stable30

[stable30] fix(appstore): Cache apps.json also on dev instances

fix(appstore): Cache apps.json also on dev instances

Signed-off-by: Marcel Müller <marcel-mueller@gmx.de>

chore(assets): Recompile assets

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

fix(files): undefined displayname for outdated `@nextcloud/files`<3.6.0 apps

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>

Fix(l10n): Update translations from Transifex

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

fix: write object to the correct urn when moving from another storage to object store

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

Merge pull request #47733 from nextcloud/backport/46859/stable30

[stable30] Fix status check and saving of external storages

Merge pull request #47751 from nextcloud/backport/47417/stable30

[stable30] fix(files): Create non-existent parents of mountpoints

build(hub): 30.0.0 RC4

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

fix(files): Create non-existent parents of mountpoints

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

Merge pull request #47748 from nextcloud/backport/47738/stable30

chore(assets): Recompile assets

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

fix(files): undefined sidebar

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>

fix: Hide status tooltip in row to add a new mount point

The row to add a new mount point is cloned when a new mountpoint is

added, so it is expected that it includes a status span. However, it

should not be displayed in that row, only in the cloned row when its

status is updated.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix: Fix unmodified placeholder replacing the actual value when updating

When updating global storages and user storages a property is not
updated by "StoragesService::updateStorage()" if the value matches the
unmodified placeholder. However, userglobal storages are not updated
through the "StoragesService"; as only the authentication mechanism is
updated it is directly done with "saveBackendOptions()" in
"IUserProvided" or "UserGlobalAuth". Due to this the unmodified
placeholder value needs to be explicitly checked in those cases and
replaced by the actual value (note that in this case it is not possible
to just skip updating a specific property).

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

test: Add integration tests for saving external userglobal storages

As the external storage uses the Nextcloud server itself the number of
workers of the PHP process running the Nextcloud server had to be
increased. Otherwise if a request is sent for the external storage while
handling a request from the integration tests a deadlock would occur.

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

fix: Reset selected backend when adding a new storage

As a new storage is added by selecting a backend the selected backend
needs to be reset. Otherwise it is not possible to add another storage
with the same backend.

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

fix: Add missing translation for UI string

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

fix: Restore default status tooltip when no status message is provided

If the status is updated but no explicit message is provided (for
example, if the status check succeeded) the default tooltip (from the
template) is now set to prevent a mismatch between the status and the
tooltip (for example, if the configuration is fixed after a failed
status check).

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

fix: Set status tooltip to error message on failed actions

When saving, updating and rechecking an storage fails (which is
different to the soft-fail when the action itself succeeds but the
status check does not) further details are provided in the error message
of the response, which is now set as the tooltip.

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

fix: Set status tooltip to status message when saving an storage

When a storage is saved the status check can fail even if saving the
storage succeeds. In those cases further details are provided in the
status message of the storage, which is now set as the tooltip,
similarly to how it is done when rechecking the storage.

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

fix: Remove status check when configuration was changed

Setting a null status was supposed to remove the status check, but
nothing was changed in that case. Now the status check is properly
removed, and doing that by hiding the element rather than just turning
it invisible also prevents that clicking on the invisible status
triggers a check, as until the new configuration is saved the check will
still be performed with the old configuration, which could be misleading
for the user.

Additionally, an explicit width is set to the parent of the span element
to prevent its width from changing when the span is shown and hidden.

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

fix: Recheck userglobal storages when loaded

Userglobal storages are now automatically recheck when loaded, similarly
to how it is done for global storages.

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

refactor: Store result in its own variable

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

Merge pull request #47728 from nextcloud/backport/47155/stable30

[stable30] fix(jobs): Switch to executeStatement() while deleting rows

Merge pull request #47726 from nextcloud/backport/47285/stable30

[stable30] fix: Do not throw if appstore is unavailable during upgrade

Merge pull request #47729 from nextcloud/artonge/backport/stable30/47511

[stable30] chore: Improve isEncrypted description

chore: Improve isEncrypted description

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

fix: lint

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

fix(jobs): Swicth to executeStatement() while deleting rows

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