Restore DB connection after failure

In case of failure, PHPUnit seems to skip `tearDown`, so any useful
assertion messages cannot be shown because `tearDownAfterClass` is
throwing an error because of database usage.

This commit makes sure we also restore the database in
`tearDownAfterClass` to prevent the data root restoration to fail

Merge pull request #21289 from owncloud/issue-20399-keep-periodic-background-jobs

Do not delete background jobs, in case an exception occured

Merge pull request #21308 from owncloud/use-new-apcu-api

Use new APCu API

[tx-robot] updated from transifex

Merge pull request #21307 from aspdye/patch-1

Update README.md

Merge branch 'master' into patch-1

Merge pull request #21299 from owncloud/symfony-process-2.8.0

[3rdparty] Bump symfony/process to 2.8.0

Use new APCu API

This one is available since APCu 4.0, since we require APCu 4.0.6 this works fine. Also it brings us PHP 7 compatibility and thus fixes https://github.com/owncloud/core/issues/21095

[3rdparty] Bump symfony/process to 2.8.0

Update README.md

-Added Facebook into "Get in touch"-section
-Improved Emojis on "Get in touch"-section
-Added Support Information

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21281 from owncloud/allow-di-for-background-jobs

Allow background jobs to be service names for DI

Merge pull request #21290 from owncloud/public-api-usage-getversion

Use OCP\Util::getVersion instead of the internal private implementation

Use OCP\Util::getVersion instead of the internal private implementation

Merge pull request #21272 from owncloud/symfony-routing-2.8.0

[3rdparty] Bump symfony/routing to 2.8.0

Merge pull request #21288 from owncloud/deprecated_helper_mimetypes

Remove deprecated OC_Helper mimetype functions

Fix the test to expect the new behaviour

Merge pull request #21283 from owncloud/cleanup_config

Cleanup OC_Config mess

Merge pull request #21280 from owncloud/drop-unused-methods

OC_Helper::makeURLAbsolute is not used anymore

Do not delete background jobs, in case an exception occured

This approach is not valid anymore. It initially was added for jobs
of non existing apps. But jobs of non-existing apps can not be created
so they will never be executed and so this call just catches other
cases which were not intended.

This reverts commit 4f4ad72460f8ef299cd1235b5774c5f121cca430.

Add methods to interface which have always been required

Allow dependency injection for background jobs

Removed deprecated OC_Helper::mimeTypeIcon function

Look up services in the app container before trying the core container

Register app containers in the OC container

Merge pull request #21277 from owncloud/querybuild-joblist

Use the query builder for the joblist queries

Removed deprecated unsused function OC_Helper::getStringMimeType

Removed deprecated OC_Helper::getSecureMimeType

Move OC_Helper code to OCP\Util for linkToRemote

Merge pull request #21279 from owncloud/remove-tmp-methods

Remove unneeded methods in OC_Helper

Use SystemConfig internally

Inject Config into SystemConfig

Get rid of legacy OC_Config

We were still using the lecagy class OC_Config all over the place. Which
was a wrapper around the new class OC\Config

properly use OCP\Util instead of OC_Helper

OC_Helper::makeURLAbsolute is not used anymore

OC_Helper::userAvatarSet is not used anymore

Use TempManager instead of tmpFile

Use TempManager instead of tmpFolder

[3rdparty] Bump symfony/routing to 2.8.0

Merge pull request #21278 from owncloud/exclude-scrutinizer-autofixer

Exclude scrutinizer-auto-fixer

Exclude scrutinizer-auto-fixer

Merge pull request #21268 from owncloud/cleanup-OC_Helper-getFileNameMimeType

Use proper public API for OC_Helper::getFileNameMimeType

Merge pull request #21264 from owncloud/deprecated_oc_user_methods

Cleanup OC_User deprecated methods

Make sure the interface and the implementation match

Use the query builder for the joblist queries

Merge pull request #21269 from owncloud/we-shall-cleanup-xml-parser-errors

Assert that there are no xml error left

Merge pull request #21190 from owncloud/bump_doctrine_deps

[3rdparty] Bump doctrine deps

Merge pull request #21248 from owncloud/deprecated_oc_helper_linkToRoute

Replace deprecated OC_Helper::linkToRoute calls

Merge pull request #21247 from owncloud/placeholder_size

Add a default size to the avatar placeholders

[3rdparty] Bump doctrine deps

Remove private deprecated OC_Helper::linkToRoute

OC_Helper::linkToRoute is deprecated

Replaced all calls to OC_Helper::linkToRoute with
OC::$server->getURLGenerator()->linkToRoute

Clear xml errors in updater

Clear xml errors in ocs client

Only perform tearDown operations on the database if we have a connection to it .....

Use proper base class

Clear xml errors

Assert that there are no xml error left

Use proper public API for OC_Helper::getFileNameMimeType

Merge pull request #20696 from owncloud/add-carddav-backends-to-ocp-contactsmanager

Add carddav backend to OCP\ContactsManager

Since avatar.js is now essentially empty remove it

Add a default size to the avatar placeholders

This removed the need to do an avatar request on the "empty" row in the
user settings.

Merge pull request #21255 from owncloud/usermanagement-show-password-error-temporary

user management: show password error temporary

Removed deprecated function OC_User::deleteUser

Replaced with proper OCP calls

Removed unused deprecated methods

Removed deprecated private OC_User::createUser

All function calls are replaced with the recommended (which was already
the body of the function).

Remove deprecated function OC_User::getManager

Private deprecated function => removed
Replaced all instances with suggested replacement

Merge pull request #21260 from owncloud/fix-undefined-l10n-var

initialize l10n instance earlier, fixes an undefined var warning foll…

Merge pull request #21259 from owncloud/load_big_avatar_only_personal

Only load the big (128x128) avatar on the perosnal page

Merge pull request #21251 from owncloud/make-occ-more-error-proof

occ command can only be called from ownCloud root directory

Merge pull request #21200 from owncloud/files-authorizationheader

Use Authorization headers for public webdav in web UI

initialize l10n instance earlier, fixes an undefined var warning followed by a php error

Merge pull request #21201 from mmattel/scanFiles_stat_output_II

Statistics output for ./occ files:scan (rework)

Merge pull request #21240 from owncloud/avatar_speedup

Avatar speedup

Merge pull request #21253 from owncloud/deprecated_oc_helper_linkTo

Remove deprecated OC_Helper::linkTo

Only load the big (128x128) avatar on the perosnal page

Before the code was executed on every page if a user was logged in. Now
only on the personal page. Thus saving a request on all other pages.

Merge pull request #21244 from owncloud/remove-unavailable-setting-from-sample-config

enforce_home_folder_naming_rule cannot be configured in config.php

Update users.js

Merge pull request #21241 from owncloud/fix_21226

Do not try to get the avatar if there is no user

Remove deprecated OC_Helper::linkTo function

occ command can only be called from ownCloud root directory

* this changes into the ownCloud root directory and then should run just fine
* avoids weird error messages that are caused by this

OC_Helper::linkTo is deprecated

Replaced with suggested (and calling body of)

Merge pull request #21238 from owncloud/files-fixparsewebdavmtime

Fix files UI mtime parsing from webdav

Do not try to get the avatar if there is no user

[tx-robot] updated from transifex

enforce_home_folder_naming_rule cannot be configured in config.php

[Avatar] Make the avatar controller use the avatar node

[Avatars] Add function to get the Node of the avatar

Since we usually just get the avatar and stream the content to the users
there is no need to first create an image in memory.

Merge pull request #21184 from owncloud/cli-logger-embedded-webserver

Let's print error messages to the builtin webserver console ... makes…

Fix files UI mtime parsing from webdav

Merge pull request #21232 from owncloud/fix-21230

Use name of ICollection for exception message

add unit test for #21230

Use name of ICollection for exception message

* fixes #21230

Merge pull request #21229 from owncloud/contrib-linktomaintainerspageforapps

Replace list of apps with a link

Replace list of apps with a link

The link points to the maintainers page that already contains a more up to date list of apps and repo links.
No need to have this information in two places.

[tx-robot] updated from transifex

Statistics output for ./occ files:scan

Merge pull request #21133 from owncloud/fix-shared-files-of-deleted-users

Fix shared files of deleted users, detect DN change when checking for existence on LDAP

avoid duplicates on search