Merge pull request #30521 from nextcloud/backport/30515/stable21

[stable21] Remove iconv from dependencies and tests

Merge pull request #30524 from nextcloud/backport/30492/stable21

[stable21] Fix passing on the parameter

[stable22] Remove iconv from dependencies and tests

which is not used anymore since: https://github.com/nextcloud/server/pull/29470

Signed-off-by: MichaIng <micha@dietpi.com>

Fix passing on the parameter

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

Merge pull request #30482 from nextcloud/backport/30423/stable21

Merge pull request #29963 from nextcloud/backport/29951/stable21

Merge pull request #29325 from nextcloud/backport/28541/stable21

Merge pull request #28657 from nextcloud/backport/28419/stable21

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

Fix uploading text position

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

Merge pull request #30448 from nextcloud/backport/30440/stable21

[stable21] Make sure to get file model in template picker

Merge pull request #30435 from nextcloud/backport/30392/stable21

[stable21] Add missing index for propertypath only queries of DAV properties

[tx-robot] updated from transifex

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

Manual compilation

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

fix template picker for file actions using the fileInfoModel

Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #30341 from nextcloud/backport/29329/stable21

[stable21] fix potential unwarranted memberships in nested groups from LDAP

Merge pull request #30442 from nextcloud/backport/30353/stable21

[tx-robot] updated from transifex

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

Handle external share with invalid host

remoteIsOwnCloud might throw an exception when the host is localhost.
Handle this case instead of aborting completely. The behavior is the
same as that is done 10 lines under it

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

Add missing index for propertypath only queries against properties

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

Merge pull request #30314 from nextcloud/backport/30266/stable21

Fix share owner not being displayed in sharing tab

This only concerns subfolder and files inside a shared folder and is
used as fallback when the OCS api returns a empty result, because it
only works on the shared folder and not the content inside of it.
This unify the tab with the folder list.

This offers a less descriptive share information but making the
OCS api returns all the details even for the subfolder would very
probably be bad for performance (since we would need to fetch the shares
for all the parent folders until we found one or are in the root folder).

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

[tx-robot] updated from transifex

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

Merge pull request #30251 from nextcloud/backport/30193/stable21

[stable21] Fix setting up 2FA when no providers are set up but backup codes

Merge pull request #30248 from nextcloud/backport/30245/stable21

[stable21] Only wildcard search if enumeration is allowed

Merge pull request #30348 from nextcloud/backport/29965/stable21

[stable21] [stable23] Avoid use of iconv to get rid of unicode

Merge pull request #30346 from nextcloud/backport/30322/stable21

[stable21] fix overlapping in the help settings section

[stable23] Avoid use of iconv to get rid of unicode

Using iconv for translit depends upon server configuration, locale, and
 PHP version. Using htmlentities instead to have a consistent behavior
 independent of configuration.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Co-authored-by: MichaIng <micha@dietpi.com>

fix overlapping in the help settings section

Signed-off-by: szaimen <szaimen@e.mail.de>

Merge pull request #30308 from nextcloud/backport/30281/stable21

[stable21] Ignore non-existing groups when notifying group shares

fix potential unwarranted memberships in nested groups from LDAP

- the issue was present only when using PHP based resolving of nested
  group members. Normally nested members are common in AD (and Samba4) and
  are resolved per LDAP_MATCHING_RULE_IN_CHAIN by default
- resolving nested members is recursive
- when the cache entry was created it happend for intermediate groups, too,
  containing members from the parent group
- the check was added to only cache the root group with its members
- a runtime cache stores intermediate ldap read results

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

Merge pull request #30327 from nextcloud/dependabot/composer/build/integration/stable21/sabre/dav-4.3.0

Update sabre/dav requirement from 4.2.3 to 4.3.0 in /build/integration

Updates the requirements on [sabre/dav](https://github.com/sabre-io/dav) to permit the latest version.
- [Release notes](https://github.com/sabre-io/dav/releases)
- [Changelog](https://github.com/sabre-io/dav/blob/master/CHANGELOG.md)
- [Commits](https://github.com/sabre-io/dav/compare/4.2.3...4.3.0)

---
updated-dependencies:
- dependency-name: sabre/dav
  dependency-type: direct:development
...

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

[tx-robot] updated from transifex

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

Discard share notification for non-existing groups

Group shares might exist even after a group got deleted.

This fix catches the situation and discards the notification for the
obsolete group.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Co-authored-by: Joas Schilling <coding@schilljs.com>

[tx-robot] updated from transifex

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

Merge pull request #30192 from nextcloud/backport/30157/stable21

[stable21] Properly format sharing datepicker locale

Manual compilation

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

Fix first day locale

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

Properly format sharing datepicker locale

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

Merge pull request #30287 from nextcloud/backport/30256/stable21

[stable21] Fix ShareLink Upload UI for Folders

Merge pull request #30261 from nextcloud/backport/29523/stable21

[stable21] Support LDAP dns longer than 255 characters

[tx-robot] updated from transifex

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

Build assets

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>

Fix ShareLink Upload UI for Folders

Do not show the Upload-Option if it is not allowed to check.

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>

Merge pull request #30270 from nextcloud/search-folder-case-insensitive-21

[tx-robot] updated from transifex

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

also use case sensitive like when searching in a folder

this case was missed from https://github.com/nextcloud/server/pull/29080

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

Merge pull request #30117 from nextcloud/backport/30040/stable21

[stable21] Avoid calling image* methods on boolean

Avoid assignment in if clause

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

Fix typing problems in OC_Image

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

Avoid calling image* methods on boolean

This avoids fatal errors on PHP>=8, and warnings on older versions.
Log should also be clearer.

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

Adapt execute function name for stable21

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

Bump user_ldap version to make sure the migration runs

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

Add missing copyright headers in migration steps

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

Use clearer names for variables

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

Add an index for directory_uuid as well

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

Make sure that hash function returns a string

The documentation says it can return false, and even if that is highly
 unlikely for sha256, better safe than sorry.

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

Add the columns and alter the index in Version1010Date20200630192842

This is to ensure new installations do not need to go through migration
 history.

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

Fixes in migration step

We cannot set ldap_dn_hash column as notnull because it is empty for
 existing users before postSchemaChange is called

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

Put back length check to have a clear error

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

Fixed migration step for user_ldap

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

Fix variable names

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

Change column names to ldap_dn and ldap_dn_hash and add migration

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

Support LDAP dns longer than 255 characters

Adds an ldap_full_dn column to store the dn, and only store a sha256
 hash in the ldap_dn which is shorter and can be indexed without
 trouble.
Migration still needs to be implemented.

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

[tx-robot] updated from transifex

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

Fix setting up 2FA when no providers are set up but backup codes

2FA set up is allowed when only backup codes are set up but no other
provider and no provider is failing.

This patch syncs up the login controller check with the challenge
controller check 10 lines above.

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

Fix backport

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

Make psalm more happy

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

Fix docs to make Psalm happier

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

Convert strict_search to wildcard property and add psalm docs

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

Limit more contact searches

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

Only limit search in the system address book

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

Only wildcard search if enumeration is allowed

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

[tx-robot] updated from transifex

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

[tx-robot] updated from transifex

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

Merge pull request #30216 from nextcloud/dependabot/composer/build/integration/stable21/sabre/dav-4.2.3

Update sabre/dav requirement from 4.2.1 to 4.2.3 in /build/integration

Update sabre/dav requirement from 4.2.1 to 4.2.3 in /build/integration

Updates the requirements on [sabre/dav](https://github.com/sabre-io/dav) to permit the latest version.
- [Release notes](https://github.com/sabre-io/dav/releases)
- [Changelog](https://github.com/sabre-io/dav/blob/master/CHANGELOG.md)
- [Commits](https://github.com/sabre-io/dav/compare/4.2.1...4.2.3)

---
updated-dependencies:
- dependency-name: sabre/dav
  dependency-type: direct:development
...

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

[tx-robot] updated from transifex

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

Merge pull request #30189 from nextcloud/backport/30164/stable21

[stable21] Use correct icon for dir-external-root

Use correct icon for dir-external-root

When an external storage mount is shared with circles, it triggers the
share icon state to be rerendered. The picking of the mime type icon
would use the regular file icon because there is no actual icon for
"dir-external-root" that is shared.

This fixes the logic to use the "folder-external" icon in such
scenarios.

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

Merge pull request #30182 from nextcloud/backport/30160/stable21

[stable21] Check resource before closing in encryption wrapper

Merge pull request #30178 from nextcloud/backport/30168/stable21

[tx-robot] updated from transifex

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

Check resource before closing in encryption wrapper

In case of error there is no guarantee that $source or $target is set or
is a resource when handling an error.

Without this fix, there's a risk that fclose will fail and the actual
exception will not be thrown, making it impossible to find out about the
root cause.

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

Allow to delete non-migrated previews instead of moving them

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

[tx-robot] updated from transifex

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