Dependency injection fails: non-existent service brille24.factory.tierprice

[brille24]

Dependency injection fails: non-existent service brille24.factory.tierprice

[mamazu]

Thanks for reporting your problem. For the issues with your config I don't really have an idea what could be causing it. In general if the TierPrice entity is registered as a resource the service "brille24.tierprice.factory" should be generated by Sylius.

As for the problem with the database. If you already have overwritten the ProductVariantEntity then you want to extend your implementation with the traits provided in the package. There are also some known problems with projects that don't use XML for configuring their entities.

[olia-bn]

If you already have overwritten the ProductVariantEntity then you want to extend your implementation with the traits provided in the package

This must be the issue. Stock Sylius installation seems to override all of them by default, so it didn't click in my head, especially since I set up the test env months ago. Maybe it's worth pointing it out for other devs that might not realise this is what happens by default?

There are also some known problems with projects that don't use XML for configuring their entities.

I use annotations indeed. So this might be the case. Any particular git issue you know of that I can look at for more info?

Edit: what I mean by "Sylius overriding the models by default" is that the following config was auto-generated by Sylius when I set up the project. Which counts for an overwritten model class, but being stock, didn't trigger me to follow Tier price bundle's instructions for customised models.

sylius_product:
    resources:
        product:
             ...
        product_variant:
            classes:
                model: App\Entity\Product\ProductVariant

[mamazu]

Yeah you are completely right, we should add this information to the docs. When I saw Sylius change that behavior ages ago, I ran into the same issue myself. :D

As for your mapping issue, sadly no. If you have time to add attributes to the class to make it extendable that would be great, but other than that I'm not sure there is an open issue for this yet.