Merge pull request #279 from dotkernel/issue-276

Issue #276: Implemented Twig CS checker/fixer.

Author: arhimede

.gitignore

@@ -2,6 +2,7 @@ clover.xml
 coveralls-upload.json
 .phpunit.result.cache
 .phpcs-cache
+.twig-cs-fixer.cache
 
 # Created by .ignore support plugin (hsz.mobi)
 ### JetBrains template

composer.json

@@ -63,7 +63,8 @@
         "mezzio/mezzio-tooling": "^2.9.0",
         "phpunit/phpunit": "^10.5.9",
         "roave/security-advisories": "dev-latest",
-        "vimeo/psalm": "^5.20.0"
+        "vimeo/psalm": "^5.20.0",
+        "vincentlanglet/twig-cs-fixer": "^3.0"
     },
     "autoload": {
         "psr-4": {
@@ -97,6 +98,8 @@
         "serve": "php -S 0.0.0.0:8080 -t public/",
         "static-analysis": "psalm --shepherd --stats",
         "test": "phpunit --colors=always",
-        "test-coverage": "phpunit --colors=always --coverage-clover clover.xml"
+        "test-coverage": "phpunit --colors=always --coverage-clover clover.xml",
+        "twig-cs-check": "vendor/bin/twig-cs-fixer lint --config=config/twig-cs-fixer.php",
+        "twig-cs-fix": "vendor/bin/twig-cs-fixer lint --config=config/twig-cs-fixer.php --fix"
     }
 }

config/twig-cs-fixer.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+$config = new TwigCsFixer\Config\Config();
+$config->addTwigExtension(new Dot\Twig\Extension\DateExtension());
+$config->addTwigExtension(new Dot\Twig\Extension\TranslationExtension());
+
+return $config;

src/Admin/templates/admin/account.html.twig

@@ -4,7 +4,7 @@
 
 {% block content %}
     <div class="row m-5">
-        {{ messagesPartial('partial::alerts', { dismissible: true }) }}
+        {{ messagesPartial('partial::alerts', {dismissible: true}) }}
     </div>
     <div class="row gap-20 masonry pos-r">
         <div class="masonry-sizer col-md-6"></div>
@@ -14,7 +14,7 @@
                 <div class="mT-30">
 
                     {% set dummy = form.prepare() %}
-                    {{ form().openTag(form) | raw }}
+                    {{ form().openTag(form)|raw }}
 
                     {% set identity = form.getElements().identity %}
                     <div class="mb-3 form-group {% if identity.getMessages() %}has-error{% endif %}">
@@ -43,7 +43,7 @@
                     {% set dummy = submit.setAttribute('class', 'btn btn-primary btn-color') %}
                     {{ formSubmit(submit) }}
 
-                    {{ form().closeTag() | raw }}
+                    {{ form().closeTag()|raw }}
                 </div>
             </div>
         </div>
@@ -52,8 +52,8 @@
                 <h6 class="c-grey-900">Change password</h6>
                 <div class="mT-30">
                     {% set dummy = changePasswordForm.prepare() %}
-                    {% set dummy = changePasswordForm.setAttribute('action', path('admin', { action : "change-password" })) %}
-                    {{ form().openTag(changePasswordForm) | raw }}
+                    {% set dummy = changePasswordForm.setAttribute('action', path('admin', {action: 'change-password'})) %}
+                    {{ form().openTag(changePasswordForm)|raw }}
 
                     {% set currentPassword = changePasswordForm.get('currentPassword') %}
                     <div class="mb-3 form-group {% if currentPassword.getMessages() %}has-error{% endif %}">
@@ -82,7 +82,7 @@
                     {% set dummy = submit.setAttribute('class', 'btn btn-primary btn-color') %}
                     {{ formSubmit(submit) }}
 
-                    {{ form().closeTag() | raw }}
+                    {{ form().closeTag()|raw }}
                 </div>
             </div>
         </div>

src/Admin/templates/admin/delete.html.twig

@@ -1,8 +1,8 @@
-{{ form().openTag(form) | raw }}
+{{ form().openTag(form)|raw }}
 <p>Are you sure you want to delete admin account with identity <span id="adminIdentity" class="fw-bold">{{ admin.identity }}</span>?</p>
 <div class="form-check">
     {{ formElement(form.get('confirmation')) }}
     <label class="form-check-label" for="confirmation">Yes, I want to delete <span class="fw-bold">{{ admin.identity }}</span></label>
 </div>
 {{ formElement(form.get('adminDeleteCsrf')) }}
-{{ form().closeTag() | raw }}
+{{ form().closeTag()|raw }}

src/Admin/templates/admin/list.html.twig

@@ -85,4 +85,3 @@
         </div>
     </div>
 {% endblock %}
-

src/Admin/templates/admin/login.html.twig

@@ -21,7 +21,7 @@
                 </div>
                 <div class="col-md-4">
                     {% set dummy = form.prepare() %}
-                    {{ form().openTag(form) | raw }}
+                    {{ form().openTag(form)|raw }}
                     <h4 class="fw-300 c-grey-900 mB-20">Login</h4>
                     <div class="mb-3">
                         {% set username = form.get('username') %}
@@ -45,7 +45,7 @@
 
                     {{ formSubmit(submit) }}
 
-                    {{ form().closeTag() | raw }}
+                    {{ form().closeTag()|raw }}
                 </div>
             </div>
         </div>

src/App/templates/layout/default.html.twig

@@ -21,9 +21,9 @@
         <![endif]-->
 </head>
 <body class="app">
-    {% include '@partial/left-menu.html.twig' %}
+    {{ include('@partial/left-menu.html.twig') }}
     <div class="page-container">
-        {% include '@partial/nav-bar.html.twig' %}
+        {{ include('@partial/nav-bar.html.twig') }}
         <main class='main-content bgc-grey-100'>
             <div id='mainContent'>
                 {% block content %}{% endblock %}

src/App/templates/partial/ajax-form.html.twig

@@ -3,9 +3,9 @@
 {% set dummy = form.setAttribute('action', formAction) %}
 {% set dummy = form.prepare() %}
 
-{{ form().openTag(form) | raw }}
-{% include '@partial/form-display-elements.html.twig' with {'form' : form, 'showLabels' : true} only %}
-{{ form().closeTag() | raw }}
+{{ form().openTag(form)|raw }}
+{{ include('@partial/form-display-elements.html.twig', {form: form, showLabels: true}, false) }}
+{{ form().closeTag()|raw }}
 
 <script type="application/javascript">
     $(document).ready(() => {

src/App/templates/partial/alerts.html.twig

@@ -1,9 +1,8 @@
-{% set classes = {'error': 'danger', 'info': 'info', 'warning': 'warning', 'success' : 'success'} %}
+{% set classes = {error: 'danger', info: 'info', warning: 'warning', success: 'success'} %}
 {% if dismissible is not defined %}
     {% set dismissible = false %}
 {% endif %}
 
-
 {% if messages is defined and messages is iterable %}
     {% for namespace in messages|keys %}
         {% if classes[namespace] is defined %}