[stable9] Fix share mounting recursion

[PVince81]

Fixed recursion issue.

Squashed version of #22937 without logging and without extra unused "use" statement.

@nickvergessen @icewind1991 @rullzer @owncloud/qa @cmonteroluque @karlitschek @DeepDiver1975

[karlitschek]

backport is obviously fine. please merge as soon as tests pass. Thanks a lot!!!

[icewind1991]

Code looks good 👍

[PVince81]

Steps to reproduce are here: https://github.com/owncloud/enterprise/issues/1157#issuecomment-193680329

[rullzer]

Fixes the issue 👍

[nickvergessen]

Smashbox passed locally ✅

[SergioBertolinSG]

Working fine 👍

[rullzer]

Local Unit test (php7)

sqlite ✅
mysql: ❓
mariadb: ✅
postgres: ❓

Local Intergration test (php7):

sqlite: ✅ (besides the capabilities tests... which also don't pass on master here).

[karlitschek]

so what is going on with the tests? we need to merge this soon.

[PVince81]

On the old unsquashed PR most tests passed: #22937

Integration test is missing, but @rullzer ran them locally and they worked.

The unrelated "failed" test you see here was just aborted by @DeepDiver1975 to save some time.

We can either wait a bit longer or take the risk and merge it and build packages in parallel.

[ghost]

failed test looks like failure to connect?

[PVince81]

@cmonteroluque no, aborted by @DeepDiver1975

07:41:15 Kill the docker 54bea74ecb76b6b209bb88f529c367d859f0ab77280b62d29e39c5564ca87d84
08:21:43 Build was aborted
08:21:43 Aborted by Thomas Müller

[DeepDiver1975]

lex-j3 has docker issues - need to reboot - but we did smash it locally 👍

[nickvergessen]

Please prepare a master PR @PVince81

[PVince81]

Yessssir

[PVince81]

master/9.1: #22950

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.