Removed oauthAccounts since it is already defined in user bundle

[umpirsky]

I was getting:

MappingException: Property "oauthAccounts" in "Sylius\Component\Core\Model\User" was already declared, but it must be declared only once

Does not belong to this model, already defined in User.orm.xml#L55.

Q	A
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
License	MIT

[umpirsky]

@pjedrzejewski @michalmarcinkowski Can this fix be added to 0.15? It is critical for us. Maybe releasing v0.15.1 with this fix?

[michalmarcinkowski]

I think we could release v0.15.1 with this fix, @pjedrzejewski?

[stloyd]

I don't think that this is possible as the approach is broken and semver is not fully followed.

167 commits to master since this release (0.15.0)

So there are too many BC breaks (including bump of PHP version)...

[umpirsky]

Then maybe just apply this fix on 0.15.0 and release 0.15.1? Would that be possible?

[michalmarcinkowski]

Exactly, we could simply revert to v0.15.0 cherry-pick the @umpirsky commit with this fix and release v0.15.1. Then rebase master with v0.15.1 and everything should be fine. Would that be possible @pjedrzejewski?

[umpirsky]

@michalmarcinkowski That was what I was hoping for exactly. :)

[umpirsky]

This is also missing in 0.15 .

[umpirsky]

#3427 also made upgrade to 0.15 hard. We needed to override Sylius\Bundle\CoreBundle\Form\Type\Checkout\ShipmentType.

[pjedrzejewski]

@umpirsky I will setup 0.15.x branch tomorrow and ping you here to reopen this fix against this branch. 👍

[stloyd]

@pjedrzejewski ping

[pjedrzejewski]

Thanks Sasha!