Use regex when searching on single file shares

[rullzer]

There is no need to consult the database when we want to search and
there are a lot of incomming single file shares. We can then just do a
regex and be done with it.

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

[icewind1991]

Not a fan of doing this in the View, shouldn't it be possible to do this in the search methods in the sharedcache?

[rullzer]

Not a fan of doing this in the View, shouldn't it be possible to do this in the search methods in the sharedcache?

Let me see. I guess we can. This was just a first attempt to limit the query storm from searches.

[rullzer]

Not a fan of doing this in the View, shouldn't it be possible to do this in the search methods in the sharedcache?

Yes I could.
I mean it is not per se less queries this way. But waaaayyyyy cheaper queries.

[faily-bot]

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 35689: failure

mysql8.0-php7.4

Show full log

There was 1 error:
1) OCA\Files_Sharing\Tests\GroupEtagPropagationTest::testGroupReShareRecipientWrites
OCP\Files\NotFoundException: File with id "5230" has not been found.
/drone/src/lib/private/Files/View.php:1768
/drone/src/apps/files_sharing/tests/GroupEtagPropagationTest.php:106
/drone/src/apps/files_sharing/tests/PropagationTestCase.php:42
--
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

[MorrisJobke]

Code makes sense 👍

[MorrisJobke]

@rullzer Backport to 20?

[rullzer]

@rullzer Backport to 20?

sure. I mean shouldn't hurt to much

[rullzer]

/backport to stable20