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

Templates #2491

Merged
merged 2 commits into from Jul 14, 2016
Merged

Templates #2491

merged 2 commits into from Jul 14, 2016

Conversation

Saphyel
Copy link
Contributor

@Saphyel Saphyel commented Jul 4, 2016

I found a bug in the templates ( templates/module/src/Form/form-alter.php.twig & templates/module/src/Form/form-config.php.twig ) and well I edited the others in order to use [] rather than array()

@@ -73,7 +73,7 @@ class {{ class_name }} extends ConfigFormBase {% endblock %}
$config = $this->config('{{module_name}}.{{class_name_short}}');
{% for input in inputs %}
{% if input.fieldset is defined and input.fieldset is not empty %}
$form['{{ input.fieldset }}']['{{ input.name }}'] = array(
$form['{{ input.fieldset }}']['{{ input.name }}'] = [
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is one of them.

@enzolutions
Copy link
Contributor

@miguel303 please test all command with temples modified using these instructions in your local

git checkout -b Saphyel-templates master
git pull https://github.com/Saphyel/DrupalConsole.git templates

@enzolutions enzolutions added this to the 1.0.0-beta4 milestone Jul 5, 2016
@Saphyel
Copy link
Contributor Author

Saphyel commented Jul 13, 2016

@miguel303 I didn't update the "Entities" if you want to update them here or in other PR...

@jmolivas jmolivas merged commit e703fb1 into hechoendrupal:master Jul 14, 2016
@jmolivas
Copy link
Member

@Saphyel PR was merged. This will be included on next release, Thanks.

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

Successfully merging this pull request may close these issues.

None yet

3 participants