Make getShareFolder use given view instead of static FS

[PVince81]

Workaround that makes #24423 work again.

However in that specific scenario the skeleton files won't be copied.

The code change itself should be fine as it makes use of a real view instance.

(forward port of #25150 to 9.1/master)

Please review @owncloud/sharing @owncloud/filesystem

[mention-bot]

By analyzing the blame information on this pull request, we identified @schiessle, @icewind1991 and @rullzer to be potential reviewers

[PVince81]

stable9: #25150

[PVince81]

@icewind1991 @georgehrke

[georgehrke]

Code looks good, is there a way to test this properly with the default database user backend?

[PVince81]

@georgehrke not really. As states in the original ticket, it has to happen early with LDAP's code path that triggers full user fetching + email address update at the time where the user logs in and has no folders yet. I think it's impossible to reproduce without this setup.

You can at least do regression testing and check that the "share_folder" setting still works correctly in normal setups.

I already tested this myself in the special LDAP setup.

[icewind1991]

👍 looks good

[PVince81]

Second review please @georgehrke @DeepDiver1975

[georgehrke]

Tested with DB backend: Found no regressions 👍

[PVince81]

Thanks!

[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.