parameter checks for setting actor and object to happen only in one place

fix usage of empty

different strategy in cleaning up after user was deleted

we do not listen to deletion hooks anymore, because there is no guarantee that they
will be heard - requires that something fetches the CommentsManager first.

Instead, in the user deletion routine the clean up method will be called directly. Same way
as it happens for files, group memberships, config values.

add icommentsmanger and icomment implementation

register CommentsManager service, allow override, document in config.sample.php

don't insert autoincrement ids in tests, because of dislikes from oracle and pgsql

specify timezone in null date

only accepts strings for ID parameter that can be converted to int

replace forgotten hardcoded IDs in tests

react on deleted users

react on file deletion

Postgresql compatibility

lastInsertId needs *PREFIX* with the table name

do not listen for file deletion, because it is not reliable (trashbin, external storages)

add runtime cache for comments

Merge pull request #21060 from owncloud/support-for-read-only-config-dir

Add support for read only config dir

Merge pull request #21019 from owncloud/occ-get-apppath

Add occ command to get app path

Merge pull request #20989 from owncloud/fix_20769

Update parent when moving share into recieved share

Merge pull request #21059 from owncloud/remove-deaed-code-mount-config

Remove dead code

Merge pull request #21068 from owncloud/fix-exit-code-autotest-external

Fix exit code autotest external

Use default timeout on connection test

remove deprecated strict setting

script shall exit with error in case the start file has issues - fixes #20798

Merge pull request #21061 from owncloud/fix-oracle-last-insert-id-test

Fix the last insert id test by changing to an autoincremen…

Merge pull request #21056 from owncloud/check-return

Verify return type

Merge pull request #21055 from owncloud/fix-php-docs-for-console

Fix PHPDoc and check if path does exists

Merge pull request #20995 from owncloud/fix-18734

ensure multiselect always receives an array when setting its value, f…

Add occ command to get app path

Merge pull request #21064 from owncloud/fix-occ-install-on-os-x

Allow occ install on OS X - same behaviour as web setup

Do not quote the table name for lastInsertId()

Allow occ install on OS X - same behaviour as web setup

Merge pull request #21003 from owncloud/personal-mount-mountid

Dont set the string storage id to the numeric storage id for personal mounts

Merge pull request #21005 from owncloud/verify_password_before_unshare

The ajax code path unshares a link share when updating the password

Merge pull request #21038 from owncloud/share-computesharepermissions-notstore

Fix (re)share permission checks in a few code paths

Merge pull request #21043 from owncloud/add-php-doc

Add type description

Fix the last insert id test by changing to an autoincrement table

Add support for read only config dir

We already support the `config_is_read_only` for the config file itself. However not for the whole directory (which is a bug).

This unifies the check in the checkServer routine with the one in base.php. Now one can enable a read only config folder so that ownCloud is not allowed to overwrite it's own source code.

To test this set the whole config folder to read only, clear your session, refresh, see it fails, add the new code, refresh, see it works. Also verify that setup still works fine. (obviously setup does not work with a read only config Also verify that setup still works fine. (obviously setup does not work with a read only config))

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

Remove dead code

Silences two other security warnings, also I cleaned up the PHPDoc a little bit.

[tx-robot] updated from transifex

Verify return type

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

Fix PHPDoc and check if path does exists

Mutes another security warning of some static scanners.

Add tests

Add explicit check for groups excluded from sharing

Since isSharable() doesn't do the check for groups excluded from
sharing, adding an explicit check in the sharing code.

Fix more unit tests to pass a mock storage instead of null to FileInfo

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

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

Add type description

Allows IDEs and static code analyzers. Would have saved me some minutes today :)

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

Add reshare permission checks

Added in isSharable() in incoming remote share.
Added in isSharable() in regular incoming share.
Added in FileInfo to make sure the proper attributes are returned to the
clients.

Compute share permissions in the view

The share permissions are now computed in the View/FileInfo instead of
storing them directly/permanently on the storage

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

The ajax code path unshares a link share when updating the password

In order to not mess up existing shares if the password gets verified we
should first fire this validation.

Adding a basic configuration for the mention bot

Update parent when moving share into recieved share

Fixes #20769

When I receive a share and move a share of myself into that share (which
is allowed currently) I effectively hand over ownership of the files I
move. So we need to update the share I move to have as a parent the
share I move it into. Else our mounting system gets confused.

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

Dont set the string storage id to the numeric storage id for personal mounts

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

ensure multiselect always receives an array when setting its value, fixes #18734

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