Suppress warnings from DAV migration if there's nothing to do

[RobinMcCorkell]

Errors should not be printed to the log if there's nothing to do (that's a good thing!)

Fixes #23747, fixes #23637

cc @enoch85 @DeepDiver1975 @LukasReschke

[mention-bot]

By analyzing the blame information on this pull request, we identified @DeepDiver1975, @LukasReschke and @schiesbn to be potential reviewers

[enoch85]

Tested and working! 👍

[enoch85]

This is now in the VM: techandme/owncloud-vm@300cb2a

[enoch85]

Backport to stable 9 please?

[RobinMcCorkell]

@enoch85 I'd advise you don't patch files in the VM, since they will fail the integrity check and cause confusion for users... Since these messages are only seen when creating the VM, I'd suggest we don't rush a backport for 9.0.1 and rather wait for 9.0.2 (but I'd be inclined to backport too, @karlitschek )

[enoch85]

Didn't think if that. You are right.

[DeepDiver1975]

Looking at this as of now - it might be better to simply return a boolean from the setup method and don't run the migration based on it's result.

There is no need for an exception here

[RobinMcCorkell]

@DeepDiver1975 Feel free to push here to fix it, you'll know how to organise the code better 😄

[DeepDiver1975]

@DeepDiver1975 Feel free to push here to fix it, you'll know how to organise the code better 😄

will do

[enoch85]

This should be merged as @DeepDiver1975 yet haven't written an improvement of this PR.

[DeepDiver1975]

This should be merged as @DeepDiver1975 yet haven't written an improvement of this PR.

No needs - the migration code has been dropped already for 9.1

[enoch85]

@DeepDiver1975 What about backport then?

[DeepDiver1975]

@DeepDiver1975 What about backport then?

valid point - I'll prepare a pr - THX

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