Delete files that exceed trashbin size immediately

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

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>

[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>

Fix typos in method names

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

Merge pull request #24156 from nextcloud/backport/24112/stable20

[stable20] Set up FS before querying storage info in settings

Merge pull request #24153 from nextcloud/dependencies/dompurify

Merge pull request #24147 from nextcloud/backport/24146/stable20

[stable20] Don't throw on SHOW VERSION query

Set up FS before querying storage info in settings

The personal info section of the personal settings is querying the
storage quota information. For this it requires the FS to be setup which
is not always guaranteed.

This fixes an issue where refreshing the settings page would cause it to
fail after Redis caches are full. It is likely that when Redis cache is
populated, some code path is initializing the FS, so it works so far.
But when the cache is populated, that code path is skipped so the FS is
not guaranteed to be setup...

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

Bump bundles

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

Bump dompurify to 2.2.2

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

Merge pull request #24079 from nextcloud/backport/24053/stable20

[stable20] Make is_user_defined nullable so we can store false on oracle

Merge pull request #24114 from nextcloud/backport/24096/stable20

[stable20] Convert the card resource to a string if necessary

Don't throw on SHOW VERSION query

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

Bump versions

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

Merge pull request #24106 from nextcloud/backport/24085/stable20

[stable20] Register new command db:add-missing-primary-keys

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Convert the card resource to a string if necessary

Apparently the fetched column can be a string or resource. Hence we have
to catch the resource type and convert it to a string.

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

Register new command db:add-missing-primary-keys

Introduced with: https://github.com/nextcloud/server/commit/d5df033ede1f2b19b87c365b1f808130023edf17

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

20.0.2 RC2

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

Merge pull request #24081 from nextcloud/backport/24066/stable20

[stable20] Fix default internal expiration date enforce

Fix default internal expiration date enforce

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

[tx-robot] updated from transifex

Dump autoloader

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

Also make direct_edit.accessed nullable

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

Make is_user_defined nullable so we can store false on oracle

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

Merge pull request #23989 from nextcloud/version/20.0.2/rc1

20.0.2 RC1

20.0.2 RC1

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

Merge pull request #24049 from nextcloud/backport/23882/stable20

[stable20] Run unit tests against oracle

Update psalm-baseline

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

fix migration of oc_credentials table

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

Update baseline, I'm sorry

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

Fix unique key in test table

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

Fix naming of jobs and steps

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

Fix CS

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

Create primary keys on all tables and add a command to create the afterwards

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

Replace the credentials table with one that can have empty user

Primary key columns on Oracle can not have empty strings

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

Don't try to update on NotNullConstraintViolationException, only on unique or foreign key

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

Fix comparing the empty string for global credentials

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

Empty string is returned as null, but empty string in file cache is the root and exists

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

Fix public calendars as they are stored with null on oracle

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

Make sure columns with an empty default are nullable for Oracle

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

Fix "Invalid fetch style: 12" on Oracle

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

Make the test pass on repeating calls

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

Skip the insertIfNotExists() tests on Oracle because it doesn't work with clob

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

Easier debugging and spell fix

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

Use a different column for the primary key as we can not insert it on oracle

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

Add github action for oci8

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

[tx-robot] updated from transifex

Merge pull request #24058 from nextcloud/backport/24054/stable20

[stable20] Manually iterate over calendardata when oracle is used

Merge pull request #24050 from nextcloud/backport/23915/stable20

[stable20] Use png icons in caldav reminder emails

Manually iterate over calendardata when oracle is used

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

Merge pull request #24044 from nextcloud/backport/24042/stable20

[stable20] Fix sharing tab state not matching resharing admin settings

Use png icons in caldav reminder emails

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>

Add integration tests for default share permissions

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

Add acceptance tests for reshares by link when resharing is disabled

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

Add acceptance tests for reshares when resharing is disabled

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

Disable share input for a received share if resharing is not allowed

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

Do not show "Can reshare" in the menu if resharing is not allowed

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

Merge pull request #24033 from nextcloud/backport/23993/stable20

[stable20] Don't leave cursors open

[tx-robot] updated from transifex

Don't leave cursors open when tests fail

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

Merge pull request #23950 from nextcloud/backport/23928/stable20

[stable20] Fix repair mimetype step to not leave stray cursors

Update psalm-baseline.xml to accept new errors

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

Fix repair mimetype step to not leave stray cursors

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

Merge pull request #23998 from nextcloud/backport/23937/stable20

[stable20] Use query builder instead of OC_DB in OC\Files\*

Merge pull request #23995 from nextcloud/backport/23741/stable20

[stable20] Unlock when promoting to exclusive lock fails

Merge pull request #23955 from nextcloud/backport/23938/stable20

[stable20] Replace some usages of OC_DB in OC\Share\* with query builder

Merge pull request #24012 from nextcloud/backport/23981/stable20

[stable20] Fix js search in undefined ocs response

[tx-robot] updated from transifex

Fix js search in undefined ocs response

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #24001 from nextcloud/backport/23889/stable20

[stable20] Shortcut to avoid file system setup when generating the logo URL

Merge pull request #24004 from nextcloud/backport/23999/stable20

[stable20] Remove old legacy scripts references

Removed config.js

Remove old legacy search script

Merge pull request #23996 from nextcloud/backport/23982/stable20

[stable20] Make sure root storage is valid before checking its size

Shortcut to avoid file system setup when generating the logo URL

If an SVG is requested and the app config value for logoMime is set then the logo is there. Otherwise we need to check it and maybe also generate a PNG from the SVG (that's done in getImage() which needs to be called then).

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

Merge pull request #23992 from nextcloud/backport/23574/stable20

[stable20] Fix link share label placeholder not showing

Use query builder instead of OC_DB in OC\Files\*

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

Make sure root storage is valid before checking its size

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

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