Skip to content
This repository has been archived by the owner on Apr 7, 2019. It is now read-only.

Fixes generation errors #22

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

kerasai
Copy link

@kerasai kerasai commented Mar 26, 2019

When attempting to run php core/console --generate fatal PHP errors are generated due to a couple of issues in the starterkit.

These changes address the following issues and pull requests:

@@ -1,4 +1,3 @@
{% import "@macros/forms.twig" as forms %}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Explicit import not necessary per https://github.com/pattern-lab/patternengine-php-twig#macros.

forms.twig renamed to forms.macro.twig per the macro extension defined in the config.yml.

@@ -1,4 +1,4 @@
{% extends "@templates/page-2col.twig" %}
{% extends "@templates/00-layouts/01-page-2col.twig" %}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This reference is just incorrect.

Other includes within the template appear to be incorrect as well, but they do not prevent the generation from completing. May want to consider updating all of these, possibly using the pattern include partial syntax.

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

Successfully merging this pull request may close these issues.

1 participant