Verify return type

Can also be null. Silences another security warning...

Merge pull request #21037 from owncloud/fix-irritating-ldap-log-entry

passing an empty base in this diagnosis call will not result  in LDAP…

Merge pull request #21018 from owncloud/scrutinizer-patch-1

Scrutinizer Auto-Fixes

Merge pull request #21031 from owncloud/sanitize-findAppInDirectories

Sanitize the appId passed to `findAppInDirectories`

Merge pull request #21030 from owncloud/querybuilder-new-features

Querybuilder new features

Merge pull request #20786 from owncloud/systemtags-dav

DAV endpoint for system tags

passing an empty base in this diagnosis call will not result  in LDAP errors

Neither in "Invalid DN syntax" nor in "Object not found"

Add a method to the get "to use" table and column name

Merge pull request #21022 from owncloud/get-rid-of-by-reference

Get rid of by reference

Merge pull request #20979 from owncloud/settings-groups-entry

Add 'my groups' anchor to the personal page sidebar

Merge pull request #21025 from owncloud/remove-legacy-trusted-domain-support

Remove legacy repair steps + do not cast trusted domains

Merge pull request #20951 from owncloud/Make_showing_logs_optional

Controlled arguments for run script

Merge pull request #21028 from owncloud/fix-php-doc-for-storage-config

Use proper PHPDoc reference

Sanitize the appId passed to `findAppInDirectories`

Would have prevented quite some security bugs in the past. Nice hardening for now.

Add select distinct to the query builder

Use proper PHPDoc reference

Allow getting the last insert id without much hassle

More fixes to the docs

Add unit test for nested arrays

Use \OCP\Util::sanitizeHTML instead of \OC_Util::sanitizeHTML

Get rid of passing a reference

Fixes https://github.com/owncloud/core/issues/14643

Do not trust casting

Remove unused repair steps

These ones are not necessary anymore for the new major release.

Remove legacy check

This one is not required anymore as we have the RepairConfig repair step since November 2014.

Merge pull request #21014 from owncloud/share-unsharelinkpapercut

Fix unshare link click element

[tx-robot] updated from transifex

Scrutinizer Auto-Fixes

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com

Merge pull request #21015 from owncloud/update-redirecttocorrectpage

Redirect to correct URL after updating

Merge pull request #20996 from owncloud/issue-12215-remove-password-reset-when-not-possible

Issue 12215 remove password reset when not possible

Redirect to correct URL after updating

Now requires a trailing slash to make sure we don't land on the
forbidden page.

Fix unshare link click element

When clicking on the unshare link (trash icon), the correct link element
needs to be used instead of whatever child was clicked. Then, that
element might contain a visible loading icon.

This fixes the spinner detection and also prevents a full page reload in
case the spinner was visible.

Merge pull request #21006 from owncloud/scrutinizer-patch-1

Scrutinizer Auto-Fixes

Merge pull request #21009 from owncloud/share-forbidlinkshare-duplicatefield

Fix duplicate bogus share field when link sharing is not allowed

Merge pull request #21011 from owncloud/allow-ico

Allow .ico files

Merge pull request #18212 from ryno83/treat_LDAP_users_not_available_by_user_filter_as_deleted

treat LDAP users not available by user filter as deleted

Allow .ico files

Makes `/core/img/favicon.ico` accessible again via web.

Merge pull request #20991 from owncloud/disabled-checked-checkbox-css-rules

Disabled checked checkbox css rules

Merge pull request #21004 from owncloud/adding-mention-bot

Adding a basic configuration for the mention bot

Fix duplicate bogus share field when link sharing is not allowed

Whenever link share is not allowed, it was outputting a bogus sharing
field which name would conflict with the regular sharing field.

This fix makes sure that the bogus sharing field with "Resharing not
allowed" message only appears when triggered by removed share
permissions.

Merge pull request #20980 from owncloud/UoM-ResPlat-DevOps-swift-improved-fopen

Improving fopen behaviour for Swift backend

Scrutinizer Auto-Fixes

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com

Adding a basic configuration for the mention bot

Merge pull request #20987 from owncloud/dont-deprecate-OC.filePath-master

OC.FilePath has still a valid use case when generating paths to stati…

Merge pull request #20998 from owncloud/add-csp-header-to-static-resources

Add CSP header to static resources

Merge pull request #20993 from owncloud/issue-20666-language-always-overwritten

Do not overwrite the language when it's just missing for one app

Add 'my groups' anchor to the personal page sidebar

Add CSP header to static resources

Fixes https://github.com/owncloud/core/issues/16164

Allow specifying a custom reset-password-url

Merge pull request #20988 from owncloud/scrutinizer-patch-1

Scrutinizer Auto-Fixes

Merge pull request #20981 from owncloud/issue-19848-multiple-emails-for-sharelink

Allow sending a share email to multiple users

Remove password reset when the user can not change the password

Add rules for checked+disabled and disabled checkbox style

Do not overwrite the language when it's just missing for one app

Scrutinizer Auto-Fixes

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com

OC.FilePath has still a valid use case when generating paths to static files what for generateUrl cannot be used for - closes #15604

Add a unit test for single user case as well

Allow sending a share email to multiple users

Merge pull request #20569 from owncloud/issue_20427

Show external storage status in file list + prompt credentials if different

Merge pull request #20796 from owncloud/cleanup-trashbin-code

Cleanup trashbin expire code

Merge pull request #20581 from owncloud/public-page-preview-hidpi-scale

Fix hidpi previews on public page

Merge pull request #20530 from owncloud/icons-more

add icons for disabled checkbox, radio button, error and colored checkmark

Merge pull request #20480 from owncloud/css-prefixes

CSS prefixes

Merge pull request #20960 from owncloud/drop-OC_Util-getUrlContent

Remove OC_Util::getUrlContent and replace by proper usage of public i…

Merge pull request #11131 from owncloud/use-phpini-wrapper

Replacing ini_get instances with inigetwrapper usages

Merge pull request #20904 from owncloud/view-mime-filter

Fix mimetype filter in getDirectoryContent

Improving fopen behaviour for Swift backend

Merge pull request #20949 from owncloud/fix_20892

[Sharing] Properly check if a group already has access

Merge pull request #20954 from owncloud/settings_user_only_load_avatar_if_available

Settings user only load avatar if available

Merge pull request #20957 from owncloud/use-correct-method-signature

Use correct method signature

Merge pull request #20959 from owncloud/drop-OC_App-setActiveNavigationEntry

Remove unused setActiveNavigationEntry of OC_App - it's also in OCP\App

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Remove OC_Util::getUrlContent and replace by proper usage of public interfaces

Fixed system tags DAV and API and docs

remove OC_App::getActiveNavigationEntry - not unsed anymore

Remove unused setActiveNavigationEntry of OC_App - it's also in OCP\App

Use correct method signature

This function takes only one parameter

Do not request an avatar if there is none

Only try to load avatars in the user list if there is any

Merge pull request #20370 from owncloud/external-db-config

 Move files_external mount config to the database

forgotten variable

db config backend for files_external

Merge pull request #20947 from owncloud/delete-users-from-system-addressbook

Delete no longer existing users from system addressbook

Controlled arguments for run script

Merge pull request #20946 from robinkanters/patch-1

Fixed small typo in README

[Sharing] Properly check if a group already has access

The old code check was to liberal resulting in wrong matches if a user
with the same name as the group already had access.

Fixes 20892

Delete no longer existing users from system addressbook

Merge pull request #20943 from owncloud/load_avatar_once

[Avatars] JS should not load same avatar twice

Fixed small typo in 'Access'

[Avatars] JS should not load same avatar twice

Old code first dit an ajax request to the avatar. Then a new image
object with the same src was created and since we do not cache avatars
yet :(  this resulted in 2 sequential requests to the exact same URL

Now if you set the displayname it will first set the placeholder and
then load the avatar in the background. Only once this time!

Merge pull request #20933 from owncloud/more-OC_Config-cleanups

More cleanups of OC_Config usage

Merge pull request #20931 from owncloud/dav-fixnamespaceoutput

Fix namespace XML output of files and tags dav

[tx-robot] updated from transifex

Merge pull request #20907 from owncloud/integration_tests_for_capabilities

Added capabilities context and tests

Merge pull request #20807 from owncloud/dont-append-redirect-url-if-user-is-already-logged-in

Don't append redirect URL if user is logged-in

Merge pull request #20545 from owncloud/scan-storage-in-background-job

Move files/ajax/scan.php to background job

More cleanups of OC_Config usage

Fix namespace XML output of files and tags dav

Merge pull request #20924 from owncloud/use-url-generator-to-generate-link

Use URLGenerator instead of hard-coded WEBROOT