Merge pull request #24483 from nextcloud/backport/24094/stable20

[stable20] Only attempt to move to trash if a file is not in appdata

Only attempt to move to trash if a file is not in appdata

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

Merge pull request #24325 from nextcloud/backport/24103/stable20

[stable20] Only check path for being accessible when the storage is a object home

Merge pull request #24462 from nextcloud/backport/24460/stable20

[stable20] Use correct l10n source in files_sharing JS code

[tx-robot] updated from transifex

Use correct l10n source in files_sharing JS code

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #24455 from nextcloud/backport/24451/stable20

[stable20] external storages: save group ids not display names in configuration

Merge pull request #24453 from nextcloud/backport/24449/stable20

[stable20] Store scss variables under a different prefix for each theming config version

save group ids not display names in configuration

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

Store scss variables under a different prefix for each theming config version

The main issue with using the general theming- prefix is that with APCu caching
the cache is not shared between processes, so when trying to reset the cache
through the CLI, e.g. when updating the theming config the old cache is never
invalidated.

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

Merge pull request #24446 from nextcloud/backport/23652/stable20

[stable20] Fix public calendars shared to circles

Merge pull request #24443 from nextcloud/backport/24357/stable20

[stable20] Make sure we add new line between vcf groups exports

A valid user session is required for circles.

The circles app depends on a valid userId. A public shared calender might be viewed by guests without a user session. For such requests the principal is null.

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

Update phpdoc for circleToPrincipal

as the QueryException is already catched inside circleToPrincipal.

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

Make sure we add new line between vcf groups exports

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #24387 from nextcloud/backport/24382/stable20

[stable20] Properly encode path when fetching inherited shares

[tx-robot] updated from transifex

Merge pull request #24391 from nextcloud/backport/24375/stable20

[stable20] formatting remote sharer should take protocol, path into account

Compile assets

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

Merge pull request #24333 from nextcloud/backport/24315/stable20

[stable20] avoid empty null default with value that will be inserted anyways

formatting remote sharer should take protocol, path into  account

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

Properly encode path when fetching inherited shares

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

Merge pull request #24377 from nextcloud/backport/24374/stable20

[stable20] Use proper bundles for files client and fileinfo

Files client and info bundles are always loaded so we don't need to load them again

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

Merge pull request #24353 from nextcloud/backport/24162/stable20

[stable20] set the display name of federated sharees from addressbook

Merge pull request #24367 from nextcloud/backport/24363/stable20

[stable20] Catch storage not available in versions expire command

send expected format of cloud id

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

Catch storage not available in versions expire command

External storage with session credentials is not accessible without a
user session, hence background jobs and CLI commands can't work with
them. The previously unhandled exception causes logged errors in the
nextcloud log. This patch catches the specific exception and logs it as
warnings. So for a production instance the error won't spam their logs
for this non-recoverable and technically unsolvable error if the minimum
log level is set to the default of 3 (error).

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

[tx-robot] updated from transifex

set the display name of federated sharees from addressbook

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

Merge pull request #24346 from nextcloud/backport/24341/stable20

[stable20] Fix the config key on the sharing expire checkbox

Merge pull request #24342 from nextcloud/backport/24331/stable20

[stable20] Fix contacts menu position and show uid as a tooltip

Fix contacts menu position and show uid as a tooltip

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

Fix the config key on the sharing expire checkbox

We don't use `shareapi_internal_enforce_expire_date` anywhere.
`shareapi_enforce_internal_expire_date` is the one we want.

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

avoid empty null default with value that will be inserted anyways

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

Only check path for being accessible when the storage is a object home

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

[tx-robot] updated from transifex

Merge pull request #24297 from nextcloud/backport/stable20/23023

 [stable20] Delete files that exceed trashbin size immediately

Merge pull request #24311 from nextcloud/fix/stable20-sabre-parse-xml-errors

[stable20] Update sabre/xml to fix XML parsing errors

Update sabre/xml to fix XML parsing errors

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

Delete files that exceed trashbin size immediately

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #24249 from nextcloud/backport/24241/stable20

[stable20] Harden EncryptionLegacyCipher a bit

Merge pull request #24263 from nextcloud/backport/24108/stable20

[stable20] Use correct year for generated birthdays events

Use correct year for generated birthdays events

'X-APPLE-OMIT-YEAR' is not always present, at least iOS 12.4 uses the hard coded date of 1604 (the start of the gregorian calendar) when the year is unknown.

cf. https://forums.bitfire.at/topic/2050/bday-without-year-for-vcard-3-0-if-server-drops-property-x-apple-omit-year and https://gitlab.com/CardBook/CardBook/-/issues/586

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Merge pull request #24258 from nextcloud/backport/24242/stable20

[stable20] Update ScanLegacyFormat.php

Merge pull request #24259 from nextcloud/backport/24257/stable20

[stable20] Simple typo in comments

Simple typo in comments

Update ScanLegacyFormat.php

Fixed a small typo on line 99.

Merge pull request #24230 from nextcloud/backport/24007/stable20

[stable20] allow selecting multiple columns with SELECT DISTINCT

Merge pull request #24228 from nextcloud/backport/24135/stable20

[stable20] Fix nextcloud logo in email notifications misalignment

Harden EncryptionLegacyCipher a bit

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

Merge pull request #24239 from nextcloud/backport/24203/stable20

[stable20] Use regex when searching on single file shares

Merge pull request #24231 from nextcloud/backport/23757/stable20

[stable20] use file name instead of path in 'not allowed to share' message

Merge pull request #24232 from nextcloud/backport/24211/stable20

[stable20] Fix setting images through occ for theming

Limit shared cache search if it is just a file

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

[tx-robot] updated from transifex

Fix setting images through occ for theming

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

use file name instead of path in 'not allowed to share' message

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

allow selecting multiple columns with SELECT DISTINCT

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

Fix #23357

Signed-off-by: medcloud <42641918+medcloud@users.noreply.github.com>

Merge pull request #24181 from nextcloud/backport/24102/stable20

[stable20] Check quota of subdirectories when uploading to them

Merge pull request #24196 from nextcloud/backport/24098/stable20

[stable20] circleId too short in some request

Merge pull request #24212 from nextcloud/backport/24207/stable20

[stable20] missing level in ScopedPsrLogger

missing level

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

[tx-robot] updated from transifex

Merge pull request #24172 from nextcloud/version/20.0.2/final

20.0.2

circleId too short in some request

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

[tx-robot] updated from transifex

Merge pull request #24183 from nextcloud/revert-24178-backport/24098/stable20

Revert "[stable20] circleId too short in some request"

Bump bundles

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

Check for target folder available quota when uploading

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Fetch quota with files propfind

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

Revert "[stable20] circleId too short in some request"

Merge pull request #24178 from nextcloud/backport/24098/stable20

[stable20] circleId too short in some request

circleId too short in some request

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

20.0.2

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

[tx-robot] updated from transifex

Merge pull request #24091 from nextcloud/version/20.0.2/rc2

20.0.2 RC2

Merge pull request #24159 from nextcloud/backport/24069/stable20

[stable20] Fix default internal expiration date

Add integration tests for creating shares with default expiration dates

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

Reset app configs by deleting the values instead of setting the defaults

This avoids the need to keep the default values in the integration tests
in sync with the code, and also makes possible to reset values with
"dynamic" defaults (defaults that depend on other values).

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

Add unit tests for "validateExpirationDateInternal"

They were copied and adjusted as needed from the tests for
"validateExpirationDate".

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

Fix default internal expiration date

The default expiration date for internal shares was set from the default
link expiration date instead of the internal one.

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

Add explicit time to test date for clarity

This shows in a clearer way that the expected expiration date is the
original one, but without time.

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

Add unit test for expiration date with date and with default

This adds back what was being actually tested in the unit test fixed in
the previous commit.

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

Fix unit test for expiration date without date but with default

The expiration date was explicitly given, so it was not testing what it
should. Moreover, "link_defaultExpDays" needs to be mapped when testing
the default value (even if it will just fallback to the value of
"shareapi_expire_after_n_days").

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

Fix unit test method name

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

Fix unit test for verifying enforced valid expiration date.

The enforced expiration date was not actually enabled, as for that
"shareapi_default_expire_date" needs to be explicitly set to "yes".

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

Fix unit test for verifying enforced expire date too far in the future

The test should fail if no exception was thrown, and no exception was
thrown due to a missing configuration parameter. However, the use of
"addToAssertionCount(1)" made the test pass even if no exception was
thrown and, therefore, to silently fail.

Now the missing configuration parameter was added and the test will fail
if the expected exception is not thrown.

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

Add unit tests to enforce a relaxed default share expiration date

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

Remove wrong and misleading comment

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

Remove extra spaces

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

Fix typo in variable name

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