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 #24065 from nextcloud/enh/pimp_accessibilitycontroller

Pimp the accesibility controller

[tx-robot] updated from transifex

Merge pull request #24063 from nextcloud/techdebt/noid/switch-to-class-names-for-encryption-DI

Change to full class names for the encryption app DI in preparation of auto-wiring

Pimp the accesibility controller

* No usused members
* appRoot member
* less constructor args

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

Merge pull request #24037 from nextcloud/enh/reflectionparamter_getclass_deprecatred

ReflectionParamter::getClass is deprecated

Change to full class names for the encryption app DI in preparation of auto-wiring

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

ReflectionParamter::getClass is deprecated

In php8 this starts throwing warnings. And since we use it quite often
we flood the log. This moves it to getType which does the same. Only non
deprecated now.

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

Merge pull request #24060 from nextcloud/revert-24039-faster-installation

Revert "Installation goes brrrr"

Revert "Installation goes brrrr"

Merge pull request #24054 from nextcloud/bugfix/noid/calendar-repair-oracle

Manually iterate over calendardata when oracle is used

Manually iterate over calendardata when oracle is used

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

Merge pull request #24039 from nextcloud/faster-installation

Installation goes brrrr

Merge pull request #23915 from st3iny/fix/noid/png-icons-in-caldav-reminder-emails

Use png icons in caldav reminder emails

Merge pull request #24038 from nextcloud/techdebt/noid/remove-unused-static-methods-files_external

Remove unused and deprecated methods from files_external MountConfig

Don't create a schema to check if the migrations table exists

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

Reduce the number of schemas we generate when we just run all migrations anyway

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

Only update the schema when we install anyway

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

Merge pull request #24041 from nextcloud/techdebt/noid/remove-oc_hooks-for-group-management

Remove old OC_Hook for OC_Group events

Merge pull request #23882 from nextcloud/tests/oracle

Run unit tests against oracle

Merge pull request #24042 from nextcloud/fix-sharing-tab-state-not-matching-resharing-admin-settings

[tx-robot] updated from transifex

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>

Remove unused and deprecated methods from files_external MountConfig

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

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

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

Remove old OC_Hook for OC_Group events

Those mappings exist and we will remove the first ones (labeled as `old`):

old: `\OC_Hook::listen('OC_Group', 'pre_createGroup', array('run' => true, 'gid' => $gid));`
since OC 8 (owncloud/core#12618): `$groupManager->listen('\OC\Group', 'preCreate', function ($gid) { ... });`
since NC 17 (#18350): `OCP\Group\Events\BeforeGroupCreatedEvent`

old: `\OC_Hook::emit('OC_User', 'post_createGroup', array('gid' => $gid->getGID()));`
since OC 8 (owncloud/core#12618): `$groupManager->listen('\OC\Group', 'postCreate', function (\OC\Group\Group $gid) { ... });`
since NC 17 (#18350): `OCP\Group\Events\GroupCreatedEvent`

old: `\OC_Hook::emit('OC_Group', 'pre_deleteGroup', array('run' => true, 'gid' => $group->getGID()));`
since OC 8 (owncloud/core#12618): `$groupManager->listen('\OC\Group', 'preDelete', function (\OC\Group\Group $group) { ... });`
since NC 17 (#18350): `OCP\Group\Events\BeforeGroupDeletedEvent`

old: `\OC_Hook::emit('OC_User', 'post_deleteGroup', array('gid' => $group->getGID()));`
since OC 8 (owncloud/core#12618): `$groupManager->listen('\OC\Group', 'postDelete', function (\OC\Group\Group $group) { ... });`
since NC 17 (#18350): `OCP\Group\Events\GroupDeletedEvent`

old: `\OC_Hook::emit('OC_Group', 'pre_addToGroup', array('run' => true, 'uid' => $user->getUID(), 'gid' => $group->getGID()));`
since OC 8 (owncloud/core#12618): `$groupManager->listen('\OC\Group', 'preAddUser', function (\OC\Group\Group $group, \OC\User\User $user) { ... });`
since NC 17 (#18350): `OCP\Group\Events\BeforeUserAddedEvent`

old: `\OC_Hook::emit('OC_Group', 'post_addToGroup', array('uid' => $user->getUID(), 'gid' => $group->getGID()));`
since OC 8 (owncloud/core#12618): `$groupManager->listen('\OC\Group', 'postAddUser', function (\OC\Group\Group $group, \OC\User\User $user) { ... });`
since NC 17 (#18350): `OCP\Group\Events\UserAddedEvent`

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

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>

Merge pull request #24025 from nextcloud/techdebt/no-testing-deprecation-logs

Do not print deprecation warnings during testing

Update baseline, I'm sorry

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>

Merge pull request #23993 from nextcloud/bugfix/noid/close-cursors

Don't leave cursors open

Merge pull request #23070 from eleith/complex-recurrance-imports

handle vcalander documents with multiple VEVENTS not in order

Merge pull request #24024 from nextcloud/enhancement/exception-page-full-trace

Show the full trace of an exception

Do not print deprecation warnings during testing

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

Show the full trace of an exception

Because often we catch the exception at some point and then the trace is
misleading. What's really interesting is the trace of the *previous*
exception.

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

Merge pull request #24019 from nextcloud-pr-bot/automated/noid/psalm-baseline-update

[Automated] Update psalm-baseline.xml

Update psalm baseline

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

[tx-robot] updated from transifex

Merge pull request #23967 from nextcloud/dependabot/composer/build/integration/behat/behat-approx-3.8.0

Update behat/behat requirement from ~3.7.0 to ~3.8.0 in /build/integration

Merge pull request #23981 from nextcloud/fix/sharing/undefined-in-search

Fix js search in undefined ocs response

Merge pull request #23999 from nextcloud/fix/old-legacy-scripts

Remove old legacy scripts references

Merge pull request #23994 from nextcloud/fix/comment_search

Update comment search to initFS

Merge pull request #23889 from nextcloud/fix/noid/shortcut-to-avoid-filesystem-setup

Shortcut to avoid file system setup when generating the logo URL

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>

Don't leave cursors open when tests fail

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

Removed config.js

Remove old legacy search script

Merge pull request #23937 from nextcloud/bugfix/noid/files-ocdb

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

Merge pull request #23976 from nextcloud/enh/reduce_getAppPath_and_autoload_reg

Reduce the getAppPath and autoloader calls

Merge pull request #23982 from nextcloud/fix/quota-storage-getsize

Make sure root storage is valid before checking its size

Merge pull request #23741 from Ashod/lock-leak

Unlock when promoting to exclusive lock fails

Merge pull request #23978 from nextcloud/enh/query_servercontainer_first

First query the server container

Merge pull request #23933 from nextcloud/bugfix/noid/user-ldap-ocdb

Use query builder instead of OC_DB in user_ldap

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>

Update comment search to initFS

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

Merge pull request #23574 from nextcloud/fix/23569

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>

Test requires the DB

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

[tx-robot] updated from transifex

Merge pull request #23983 from nextcloud-pr-bot/automated/noid/psalm-baseline-update

[Automated] Update psalm-baseline.xml

Update psalm baseline

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

[tx-robot] updated from transifex

Fix link share label placeholder not showing

Signed-off-by: Gary Kim <gary@garykim.dev>

First query the server container

When the servercontainer wants to obtain something changes are very high
this is something from the server container. Esp on setups with a lot of
shares this can change quite a bit as it avoid a needless check on the
strpos OCA\\ etc.

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

Reduce the getAppPath and autoloader calls

The getAppPath will always return the same data for the same appId. It
is actually already cached. However we do some cleanup of the appId
(again). Same for the autoloading it is actually already checked.

This just removes the unneeded calls. Which can add up if you have a lot
of incomming shares.

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

Merge pull request #23923 from nextcloud/bugfix/oci/fix-greatest-order-for-oracle

Fix greatest/least order for oracle

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 #23956 from nextcloud/dependabot/npm_and_yarn/node-sass-5.0.0

Bump node-sass from 4.14.1 to 5.0.0

Use png icons in caldav reminder emails

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

Update parameters

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

Bump node-sass from 4.14.1 to 5.0.0

Bumps [node-sass](https://github.com/sass/node-sass) from 4.14.1 to 5.0.0.
- [Release notes](https://github.com/sass/node-sass/releases)
- [Changelog](https://github.com/sass/node-sass/blob/master/CHANGELOG.md)
- [Commits](https://github.com/sass/node-sass/compare/v4.14.1...v5.0.0)

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

Merge pull request #23961 from nextcloud/dependabot/npm_and_yarn/css-vars-ponyfill-2.4.1

Merge pull request #23957 from nextcloud/dependabot/npm_and_yarn/vue-loader-15.9.5

Bump vue-loader from 15.9.4 to 15.9.5

Merge pull request #23960 from nextcloud/dependabot/npm_and_yarn/bootstrap-4.5.3

Bump bootstrap from 4.5.2 to 4.5.3

Merge pull request #23962 from nextcloud/dependabot/npm_and_yarn/vue-router-3.4.9

Bump vue-router from 3.4.8 to 3.4.9

Merge pull request #23964 from nextcloud/dependabot/npm_and_yarn/marked-1.2.3

Bump marked from 1.2.0 to 1.2.3

Merge pull request #23936 from nextcloud/bugfix/noid/trashbin-ocdb

Use query builder instead of OC_DB in trashbin

Bump css-vars-ponyfill from 2.3.2 to 2.4.1

Bumps [css-vars-ponyfill](https://github.com/jhildenbiddle/css-vars-ponyfill) from 2.3.2 to 2.4.1.
- [Release notes](https://github.com/jhildenbiddle/css-vars-ponyfill/releases)
- [Changelog](https://github.com/jhildenbiddle/css-vars-ponyfill/blob/master/CHANGELOG.md)
- [Commits](https://github.com/jhildenbiddle/css-vars-ponyfill/compare/v2.3.2...v2.4.1)

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