Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Web profiler error (CoreBundle depends on ShopBundle) #677

Closed
arjanfrans opened this issue Oct 9, 2020 · 9 comments
Closed

Web profiler error (CoreBundle depends on ShopBundle) #677

arjanfrans opened this issue Oct 9, 2020 · 9 comments

Comments

@arjanfrans
Copy link

arjanfrans commented Oct 9, 2020

The web profiler breaks after I replaced the ShopBundle with the ShopApiBundle.

There are no registered paths for namespace "SyliusShop" in @SyliusCore/Collector/cart.html.twig at line 5.

It seems that the CoreBundle depends on a template from the ShopBundle.

@arjanfrans arjanfrans changed the title Web profiler error ( Web profiler error (CoreBundle depends on ShopBundle) Oct 9, 2020
@mamazu
Copy link
Member

mamazu commented Oct 9, 2020

At least for the error templates yes.

@arjanfrans
Copy link
Author

As a workaround I manually configured the ShopBundle namespace in the twig config:

twig:
    paths:
        '%kernel.project_dir%/templates': ~
        '%kernel.project_dir%/vendor/sylius/sylius/src/Sylius/Bundle/ShopBundle/Resources/views': 'SyliusShop'

@mamazu
Copy link
Member

mamazu commented Oct 9, 2020

And the define your own templates?

@arjanfrans
Copy link
Author

No, I didn't define my own templates. Symfony doesn't automatically fin the SyliusShop twig namespace, because the ShopBundle is not active. However, it is still installed as a part of the core installation. So I just manually configure the namespace here.

@mamazu
Copy link
Member

mamazu commented Oct 9, 2020

Ah okay then I will add this to the documentation as a step during installation.

@arjanfrans
Copy link
Author

@mamazu Thanks!

The better solution would be to fix this in the core bundle, I created an issue there. See Sylius/Sylius#11918

@mamazu
Copy link
Member

mamazu commented Oct 9, 2020

Thanks for your help.

@lchrusciel
Copy link
Member

Hey Arjan,

thanks for reporting. We should fix it in the core library 👍

@lchrusciel
Copy link
Member

Fixed already in #11958. Will be published with the next patch release

@mamazu mamazu closed this as completed Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants