[stable20] dont hold a transaction during the move to trash

[backportbot-nextcloud]

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #23109

[nickvergessen]

Fixed the backport

[faily-bot]

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 33666: failure

mysql8.0-php7.2

Show full log

There was 1 error:
1) OCA\FederatedFileSharing\Tests\FederatedShareProviderTest::testCreate
OCP\Share\Exceptions\ShareNotFound: There was an error retrieving the share. Maybe the link is wrong, it was unshared, or it was deleted.
/drone/src/apps/federatedfilesharing/lib/FederatedShareProvider.php:865
/drone/src/apps/federatedfilesharing/lib/FederatedShareProvider.php:244
/drone/src/apps/federatedfilesharing/tests/FederatedShareProviderTest.php:178
--
There were 2 warnings:
1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
--
There was 1 failure:
1) OCA\DAV\Tests\unit\CardDAV\CardDavBackendTest::testSearch with data set #0 ('John', array('FN'), array(), array(array('uri0', 'John Doe'), array('uri1', 'John M. Doe')))
Failed asserting that 1 is identical to 2.
/drone/src/apps/dav/tests/unit/CardDAV/CardDavBackendTest.php:746