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

[Contao 4] Error when changing order status #1773

Closed
netzarbeiter opened this issue Feb 11, 2017 · 5 comments
Closed

[Contao 4] Error when changing order status #1773

netzarbeiter opened this issue Feb 11, 2017 · 5 comments
Labels
Milestone

Comments

@netzarbeiter
Copy link
Member

I have chosen a template folder in "Store configuration" (Store configuration > Configuration > Template folder) and placed template "iso_collection_default" in this folder.

In the "Notification Center" there is a notification (Type: Order status change) with this template chosen.

Now I want to change the order status from an order and receive the following error:
[2017-02-11 15:30:31] app.CRITICAL: An exception occurred. {"exception":"[object] (Exception(code: 0): Could not find template \"iso_collection_notification\" at /home/public_html/domain.com/vendor/contao/core-bundle/src/Resources/contao/library/Contao/TemplateLoader.php:157)"} []

After making a copy of the "iso_collection_default" template and place it in root template folder, it works like a charm. Bug or feature?

@netzarbeiter netzarbeiter added this to the 2.4.1 milestone Feb 11, 2017
@aschempp aschempp modified the milestones: 2.4.2, 2.4.1 Mar 6, 2017
@aschempp
Copy link
Member

aschempp commented Mar 6, 2017

Unfortunately, this is something we most likely cannot change. @Toflar any idea?

@Toflar
Copy link
Member

Toflar commented Mar 6, 2017

There's no theme context in the back end so templates cannot work there really. We could maybe store the theme id just like the store id when an order is submitted and try to load it where needed.

@aschempp
Copy link
Member

aschempp commented Mar 6, 2017 via email

@Toflar
Copy link
Member

Toflar commented Mar 6, 2017

Not sure, you'll have to reproduce it. Imho we use the correct template class. But it uses Isotope::getConfig()->templateGroup which does not exist in the back end context.

@aschempp aschempp modified the milestones: 2.4.1, 2.4.2 Mar 7, 2017
@aschempp
Copy link
Member

aschempp commented Mar 7, 2017

Thanks for the hint. Should be fixed in 9700b68

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants