Check if user isset in $_REQUEST first

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21153 from owncloud/files-deleteactiontext

Bring back file delete action text to be based on context

Merge pull request #20081 from owncloud/trashbin-loggedout

Fix trashbin wrapper when no user is loggedin

Bring back file delete action text to be based on context

For received shares, the delete action becomes "Unshare" and for
personal mounts it becomes "Disconnect storage".

This also makes it possible from now on to pass a function to a file
action's "displayName" attribute.

skip test if we cant use the filesystem when not logged in

Fix thrashbin wrapper when no user is loggedin

Merge pull request #20978 from owncloud/fix-dont-show-unsaved-values-in-usermgmt

reset mailadress/displayname on blur

Merge pull request #20878 from owncloud/proper-htaccess-support-in-code-signing-checker

Also run .htaccess routine when installing on another system than Apache

Merge pull request #21144 from owncloud/dav-auth-checkduplicateheader

Properly check X-Requested-With header in case of multiple values

Merge pull request #21135 from owncloud/add-polyfill

Add polyfills for PHP55, PHP56 and PHP70 functionalities

Merge pull request #21020 from owncloud/add-directory-slash-to-setup-code

Add DirectorySlash to dynamic .htaccess write

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

Scrutinizer Auto-Fixes

Properly check X-Requested-With header in case of multiple values

Saw this happening in IE8...

Merge pull request #21087 from owncloud/catchexceptionindirectdownloadurl

Catch exception when querying direct download link

Merge pull request #21128 from owncloud/dav-fopen-non-existing

Dont do a seperate request to check if a file exists when downloading a file from dav external storage

Merge pull request #21131 from owncloud/update-public-api

IMemcacheTTL was backported to 8.2.2

Merge pull request #21137 from owncloud/update-issue-template.md

Update issue template.md

Adjust third-party reference

Remove steps for ownCloud 6

Fix the sample occ config call

Use PHP polyfills

[tx-robot] updated from transifex

IMemcacheTTL was backported to 8.2.2

see #21113

Merge pull request #21123 from owncloud/remove-db-locking-performance-warning-master

Remove info about database locking performance

Dont do a seperate request to check if a file exists for dav->fopen

Merge pull request #21119 from owncloud/make-appconfig-more-robust

Make AppManager->checkAppForUser more robust

adjust PHPDoc

Scrutinizer Auto-Fixes

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

Catch exception when querying direct download link

Merge pull request #21114 from owncloud/group-propagation-tests

Add test case for group share propagation

Merge pull request #21086 from owncloud/storage-nocheckavailforgetowner

Do not check storage availability for getOwner

Remove info about database locking performance

Make AppManager->checkAppForUser more robust

* if the JSON that is stored in the DB is corrupt an error was thrown
* with this change it is properly handled and the app is disabled

Add test case for group share propagation

Merge pull request #21072 from owncloud/db-lock-ttl

Remove all locks after ttl from the db

Merge pull request #21073 from owncloud/memcache-lock-ttl

Add ttl for redis based locking

Merge pull request #21109 from owncloud/proper-version-expire-dedup

Deduplicate version expire jobs

Deduplicate version expire jobs

* versionSize is calculated anyway in the expire job - > dropped
* offset/neededSpace was needed for expiry before the file is moved to the versions -> now this is included already in the currently used space because the expiry job is defered to a point in time after the version creation
* fixes #21108

Merge pull request #21105 from owncloud/fix-risky-test-comments

Add assertion to test

Merge pull request #21071 from owncloud/fix-swift-unit-test-execution-master

Looks like ceph requires more time to come up ....

Merge pull request #20893 from owncloud/test_sharing_file_by_multiple_channels

Added shared by multiple channels test

Merge pull request #20984 from owncloud/fix-mysql-setup-unix-socket-master

Add unix_socket support for mysql during initial installation - fixes…

Merge pull request #21104 from owncloud/make-compatible-with-phpunit-5-1

Make compatible with PHPUnit 5.1

Add assertion to test

This seems to be missing on that test.

Make compatible with PHPUnit 5.1

`addWarning` needs to be defined as well for it:

```
➜  master git:(master) ✗ bash autotest.sh sqlite
Using PHP executable /usr/local/opt/php56/bin/php
Parsing all files in lib/public for the presence of @since or @deprecated on each method...

Using database oc_autotest
Setup environment for sqlite testing on local storage ...
Installing ....
ownCloud is not installed - only a limited number of commands are available
Mac OS X is not supported and ownCloud will not work properly on this platform. Use it at your own risk!
 -> For the best results, please consider using a GNU/Linux server instead.
creating sqlite db
ownCloud was successfully installed
Testing with sqlite ...
No coverage
/usr/local/bin/phpunit --configuration phpunit-autotest.xml --log-junit autotest-results-sqlite.xml
PHP Notice:  Constant PHPUNIT_RUN already defined in /Users/lukasreschke/Documents/Programming/master/apps/firstrunwizard/tests/bootstrap.php on line 6
PHP Stack trace:
PHP   1. {main}() /usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar:0
PHP   2. PHPUnit_TextUI_Command::main() /usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar:514
PHP   3. PHPUnit_TextUI_Command->run() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/TextUI/Command.php:106
PHP   4. PHPUnit_TextUI_Command->handleArguments() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/TextUI/Command.php:117
PHP   5. PHPUnit_Util_Configuration->getTestSuiteConfiguration() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/TextUI/Command.php:663
PHP   6. PHPUnit_Util_Configuration->getTestSuite() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/Util/Configuration.php:796
PHP   7. PHPUnit_Framework_TestSuite->addTestFile() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/Util/Configuration.php:926
PHP   8. PHPUnit_Util_Fileloader::checkAndLoad() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/Framework/TestSuite.php:335
PHP   9. PHPUnit_Util_Fileloader::load() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/Util/Fileloader.php:38
PHP  10. include_once() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/Util/Fileloader.php:56
PHP  11. loadDirectory() /Users/lukasreschke/Documents/Programming/master/tests/apps.php:42
PHP  12. require_once() /Users/lukasreschke/Documents/Programming/master/tests/apps.php:20
PHP  13. define() /Users/lukasreschke/Documents/Programming/master/apps/firstrunwizard/tests/bootstrap.php:6
PHP Fatal error:  Class StartSessionListener contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (PHPUnit_Framework_TestListener::addWarning) in /Users/lukasreschke/Documents/Programming/master/tests/startsessionlistener.php on line 47
PHP Stack trace:
PHP   1. {main}() /usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar:0
PHP   2. PHPUnit_TextUI_Command::main() /usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar:514
PHP   3. PHPUnit_TextUI_Command->run() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/TextUI/Command.php:106
PHP   4. PHPUnit_TextUI_TestRunner->doRun() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/TextUI/Command.php:155
PHP   5. PHPUnit_TextUI_TestRunner->handleConfiguration() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/TextUI/TestRunner.php:153
PHP   6. require_once() phar:///usr/local/Cellar/phpunit/5.1.0/libexec/phpunit-5.1.0.phar/phpunit/TextUI/TestRunner.php:805
```

Added check for /textfile0.txt

Folder depth needs to be at least 3 for all tests to pass

Added shared by multiple channels test

* Small changes plus check of the file using propfind
* Restaured line removed accidentally

[tx-robot] updated from transifex

Merge pull request #21077 from owncloud/external-index-names

Let doctrine generate index names for files_external

Merge pull request #20775 from owncloud/comments-manager

add icommentsmanger and icomment implementation

Do not check storage availability for getOwner

Because the owner is always known thanks to the file cache and other
places, we don't need the remote storage to be actually available.

use expectedException annotation, not via method call, for consistency

use getLastInsertId from query builder for convenience

Merge pull request #21076 from owncloud/occ-shall-not-fail-hard-in-maintenance-mode

Don't load commands of apps when in maintenance mode

remove unused use statement

php < 5.5 compatible

Looks like ceph requires more time to come up ....

Use sed - not replace

apply btrfs hack

Let doctrine generate index names

Don't load commands of apps when in maintenance mode - fixes #20939

Remove all locks after ttl from the db

add since

ttl for memcache locking backends that support it

Add interface for memcache backends that support setting ttl on exisiting keys

adjust test's fakemanager to interface change

improve PHP doc and remove superflous by reference indicator

anounce CommentsManager getter in public server interface

hardening, add some checks for whitespace-only strings

fix phpdoc

rework test about overwriting default comments manager

namespaces for tests

fix php doc

OC not oc

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