Merge pull request #23928 from nextcloud/bugfix/oci/dont-leave-stray-cursors-on-mimetype-update

Fix repair mimetype step to not leave stray cursors

Merge pull request #23940 from nextcloud/enh/skip_already_loaded_apps

Skip already loaded apps in loadApps

Merge pull request #23919 from nextcloud/dependabot/npm_and_yarn/css-loader-5.0.1

Merge pull request #23929 from nextcloud/techdebt/noid/deprecate-prepare-and-execute-audited-as-they-leak-cursors

Deprecate OC_DB::prepare and OC_DB::executeAudited as they leak cursors

Merge pull request #23932 from nextcloud/enh/s3/do_not_check_bucket_exists_on_every_request

Allow config to specify the bucket exists

Skip already loaded apps in loadApps

Otherwise you might end up calling a lot of functions unneeded.
And while the individual calls are cheap if you multiply them by 20k
they still get somewhat expensive.

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

Merge pull request #23921 from nextcloud/techdebt/oci/fix-casting-to-integer

Fix casting to integer on Oracle

Allow config to specify the bucket exists

In the 99% case the bucket is just always there. And if it is not the
read/write will fail hard anyways. Esp on big instances the Objectstore
is not always fast and this can save a few hundered ms of each request
that acess the objectstore.

In short it is adding

'verify_bucket_exists' => false

To the S3 config part

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

Deprecate OC_DB::prepare and OC_DB::executeAudited as they leak cursors

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

Fix repair mimetype step to not leave stray cursors

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

Merge pull request #23920 from nextcloud/fix/match_bearer_front

Bearer must be in the start of the auth header

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

[Automated] Update psalm-baseline.xml

Bump css-loader from 3.6.0 to 5.0.1

Bumps [css-loader](https://github.com/webpack-contrib/css-loader) from 3.6.0 to 5.0.1.
- [Release notes](https://github.com/webpack-contrib/css-loader/releases)
- [Changelog](https://github.com/webpack-contrib/css-loader/blob/master/CHANGELOG.md)
- [Commits](https://github.com/webpack-contrib/css-loader/compare/v3.6.0...v5.0.1)

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

Fix casting to integer on Oracle

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

Bearer must be in the start of the auth header

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

Merge pull request #23347 from st3iny/enhancement/19526/new-share-pending-confirm-button

Merge pull request #23916 from nextcloud/dependabot/npm_and_yarn/mocha-8.2.1

Update psalm baseline

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

[tx-robot] updated from transifex

Merge pull request #23857 from nextcloud/dependabot/composer/vimeo/psalm-4.1.1

Bump vimeo/psalm from 4.0.1 to 4.1.1

Bump vimeo/psalm from 4.0.1 to 4.1.1

Bumps [vimeo/psalm](https://github.com/vimeo/psalm) from 4.0.1 to 4.1.1.
- [Release notes](https://github.com/vimeo/psalm/releases)
- [Commits](https://github.com/vimeo/psalm/compare/4.0.1...4.1.1)

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

Merge pull request #23906 from nextcloud/techdebt/noid/simplify-can-change-user-pasword

Simplify the check if admin can change password based on encryption status

Merge pull request #23887 from ecsecta/fix_23797

Do not emit UserCreatedEvent twice

Simplify the check if admin can change password based on encryption status

Found by Psalm:
```
/home/runner/work/server/server/apps/settings/lib/Controller/UsersController.php:324:8:error - RedundantCondition: Type true for $isEncryptionModuleLoaded is never falsy
```

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

Merge pull request #23004 from nextcloud/files-scan-background

only run the background scanner for users that have unscanned files

Bump mocha from 7.2.0 to 8.2.1

Bumps [mocha](https://github.com/mochajs/mocha) from 7.2.0 to 8.2.1.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mochajs/mocha/compare/v7.2.0...v8.2.1)

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

Merge pull request #23910 from nextcloud/bugfix/23908/fix-share-link-avatar

Fix link share avatar icon

Make link share avatar icon blue again

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

Fix link share avatar icon

Use "icon-class" instead of "class" as this is how to tell the component
that we don't want initials displayed.

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

Merge pull request #23907 from nextcloud/fix/noid/update-baseline-due-to-phpdoc

Check in InvalidArgument psalm error into baseline

Add confirm button in new link share pending expire/password menu

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

Check InvalidArgument psalm error into baseline - PHPDoc needs to be improved

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

Merge pull request #23904 from nextcloud/fix/php-lint

Fix php lint action

Merge pull request #23868 from nextcloud/techdebt/webpack-hide-modules

Remove old hide modules option for webpack

Remove old hide modules option for webpack

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

Merge pull request #23901 from nextcloud/techdebt/noid/deprecate-old-usermanager-events

Deprecate old user manager events for user creation

Fix php lint action

fix: also remove use statement of UserCreatedEvent

Signed-off-by: Tobias Assmann <tobias.assmann@ecsec.de>

[tx-robot] updated from transifex

Merge pull request #23819 from nextcloud/td/routing/move_things_around

Cleanup routing

Merge pull request #23820 from nextcloud/td/remove/OCPAPI_check

Remove the matching on ancient /ocs routes

Merge pull request #23792 from nextcloud/techdebt/noid/group-unread-comment-count-query

Add a function to get the unread count for multiple objects in one go

Deprecate old user manager events for user creation

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

Merge pull request #23888 from nextcloud/fix/noid/only-fetch-jsconfig-when-used

Only build JSConfig in TemplateLayout when it is actually used

Have the CommentsPropertiesPlugin use the new fancy method

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

Merge pull request #23897 from nextcloud/bugfix/23236

Move help to separate stylesheet

Merge pull request #23898 from nextcloud/bugfix/noid/list-style-scope

Make user status styles scoped

Make user status styles scoped

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

Move help to separate stylesheet

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

Merge pull request #23890 from nextcloud/techdebt/noid/refactor-theming-getImage

Cleanup ImageManager::getImage()

Merge pull request #23869 from nextcloud/rakekniven-patch-1

Add a function to get the unread count for multiple objects in one go

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

Document the behaviour inline

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

Cleanup ImageManager::getImage()

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

fix: no more translation from postCreateUser hook to UserCreatedEvent anymore, as event is already emitted in user manager

Signed-off-by: Tobias Assmann <tobias.assmann@ecsec.de>

Merge pull request #23881 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 #21693 from nextcloud/fix/noid/import-certificates-only-by-system

Improve CertificateManager to not be user context dependent

Update core/js/setupchecks.js

Co-authored-by: John Molakvoæ <skjnldsv@users.noreply.github.com>

Merge pull request #23866 from nextcloud/fix/noid/fix-grid-view-in-systemtags-view

Fix grid view toggle in tags view

Merge pull request #23798 from nextcloud/add/default_font_size

Add default font size

Merge pull request #23785 from nextcloud/bugfix/21459/workaround-snap-js

Work around snap.js state mess

Update setupchecks.js

Changed wording of setup check.
Reported at Transifex.

Signed-off-by: rakekniven <mark.ziegler@rakekniven.de>

Merge pull request #23863 from nextcloud/bugfix/noid/db-update

Replace prefix in executeUpdate

Fix grid view in tags view

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

Merge pull request #23859 from nextcloud/techdebt/23827/remove-ocsresponse-type-hint

Remove OCSResponse type hint

Add default font size

Signed-off-by: GretaD <gretadoci@gmail.com>

Replace prefix in executeUpdate

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

Remove OCSResponse type hint - see #23827

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

Merge pull request #23737 from nextcloud/fix-gif-login-image

Fix GIF being converted when used as a login image

Merge pull request #23842 from nextcloud/bugfix/noid/emptystatusmessage

UserStatus: clear status message if message is null

Remove unneeded service - found by Psalm

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

Add a background job that checks for potential user imported SSL certificates and shows a warning in the admin settings

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

Cleanup route registration logic

This was a bunch of cylic things being called.
This is an attempt to clean this all up. If an app provides an array of
routes. We just parse them and hand them back.

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

Merge pull request #23854 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

Always return the default path if we can

Just check in the certifcate manager. So every part of the system that
request the certificatebundle gets the defaullt one (the 99% case) if we
can.

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

Improve CertificateManager to not be user context dependent

* removes the ability for users to import their own certificates (for external storage)
* reliably returns the same certificate bundles system wide (and not depending on the user context and available sessions)

The user specific certificates were broken in some cases anyways, as they are only loaded if the specific user is logged in and thus causing unexpected behavior for background jobs and other non-user triggered code paths.

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

Add gif to if-statement to not convert

- SVGs are already there, so I just added it alongside it (as recommended by https://github.com/nextcloud/server/issues/16232#issuecomment-679920284)

Signed-off-by: Jacob Neplokh <me@jacobneplokh.com>

Merge pull request #23841 from nextcloud/filepicker-empty-list

ensure filepicker list is empty before populating

Merge pull request #23839 from nextcloud/fix/noid/avatar-status

No need to check if there is an avatar available, because it is gener…

Merge pull request #23799 from nextcloud/fix/noid/ldap-update-groups-null

LDAP: fix null where IUser is expected in update groups background job

UserStatus: no message means clear status message. This fixes #23332

Signed-off-by: Simon Spannagel <simonspa@kth.se>

Work around snap.js state mess

Snap.js is not robust enough to prevent multiple calls to open() or
close(), so we added more checks to prevent these happening even when an
animation is in progress.

Because if we let this through, snap.js will not notice that the
animation is already done (or a duplicate animation was started),
so the "transitionend" event will not fire a second time.

During an animation, snap.js sets up a setInterval() that hogs the CPU
during the animation. Since a transition doesn't always end due to the
above conditions, that CPU hogging would stay forever.

This is the best workaround so far, it seems it's not possible to cancel
that interval from the outside nor to trigger events to make it do so.

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

Merge pull request #23838 from nextcloud/enh/localecache/toarray

Move localecache to array

ensure filepicker list is empty before populating

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

No need to check if there is an avatar available, because it is generated on the fly nowadays

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

Move localecache to array

It seems the DS Set in the polyfill implementation is a lot less
efficient than normal arrays. (A LOT!).
So for now use a stupid normal array.

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

Merge pull request #23822 from nextcloud/enh/parse_localse_only_once

Only parse the locales ones

Merge pull request #23834 from nextcloud/fix-psalm

Update psalm-baseline.xml (was removed from DBAL update PR)

Merge pull request #23677 from nhirokinet/feature/imagemagick-svg-warning

Add SVG support check to setup check

Signed-off-by: nhirokinet <nhirokinet@nhiroki.net>

Update psalm-baseline.xml

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

Merge pull request #23827 from nextcloud/td/cleanup/ocsresponse

Remove deprecated OCSResponse

[tx-robot] updated from transifex

Merge pull request #23267 from nextcloud/fix-bold-and-font-size

Remove deprecated OCSResponse

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

[tx-robot] updated from transifex