Only prevent saving duplicate learning paths when they're new

Another way to do this would be to checkout isSyncing() in presave and set it in the post update, that would also allow the update to complete but be a bit stricter at runtime.

The solution in the MR looks good, can we just add a warning log if an LPStatus is not new and has a duplicate as this indicates some data corruption?

Added a trigger_error('...', E_USER_WARNING) so it now looks like:

>  [warning] User 6506 already has a learning path status on learning path 201. LPStatus.php:236

So noisy but not fatal any longer.

Nice!

Automatically closed - issue fixed for 2 weeks with no activity.