Prosím o otestování Latte 3.0.5 – možný BC break

Po update dotavam nasledovny error: Includujem block z danej sablony

Latte\RuntimeException
Including block m_create_tags with content type HTML into incompatible type HTML/RAW+TEXT.

<div v-show="currentStep == 0" class="tb-step-wrap">
  {include #m_create_tags, form => $form}
</div>

{define m_create_tags}
<form action="" data-noajax @submit.prevent="goForward">
    {var $renderer = $form->getRenderer()}
    {$renderer->renderPair($form['channel'])|noescape}
    {$renderer->renderPair($form['subids'])|noescape}
    <div class="form-group row pt-3">
...

Vo verzii 3.0.4 vsetko funguje.

Editoval mskocik (2. 1. 2023 14:26)

@DavidGrudl aha… tak to bude tým, že mám celú šablónu obalenú <script type="text/x-template" id="target-builder-tpl">.

V mojom use case definujem Vue šablónu a využívam automatický nette form rendering.

Ahoj,

našel jsem ve formulářích pár drobností (které tam asi budou už delší dobu) a tak jsem se chtěl zeptat, zda je zvykem psát poznámky z testů do tohoto vlákna nebo na github ?

Ahoj,

úprava form inputu Email

navrhuji doplnit do formulářového prvku Email stejné parametry jako má prvek TextInput – $cols a $maxLength, myslím, že není důvod, aby tam tyto parametry chyběly:

\vendor\nette\forms\src\Forms\Container.php

namespace Nette\Forms;

.......

class Container extends Nette\ComponentModel\Container implements \ArrayAccess
{
    ......
    ......

	// stávající kód
    public function addEmail(string $name, $label = null): Controls\TextInput
    {
        return $this[$name] = (new Controls\TextInput($label))
            ->addRule(Form::EMAIL);
    }

    // nový kód
    public function addEmail(string $name, $label = null, ?int $cols = null, ?int $maxLength = null): Controls\TextInput
    {
        return $this[$name] = (new Controls\TextInput($label, $maxLength))
            ->setHtmlAttribute('size', $cols)
            ->addRule(Form::EMAIL);
    }

    ....
    ....
}

Editoval m.brecher (6. 1. 2023 19:48)

Prosím o otestování Latte 3.0.5-RC2

Věci kolem elementu <script> jsem vyčlenil až do příští verze. Naopak jsem doplnil novou značku {exitIf} https://github.com/…e/issues/287

composer require latte/latte:^3.0.5-RC2

@kukulich mělo by to být opravené

Narazil jsem na to, že cachování překladů statických textů v latte šablonách – tag {translate} nefunguje, detailně popsáno zde:

https://forum.nette.org/…latte-v3-0-4

@DavidGrudl

verze latte/latte:^3.0.5-RC1 na Windows – cachování překladů šablon FUNGUJE !!

Super ;)

Editoval m.brecher (16. 1. 2023 20:56)

@DavidGrudl

Před cca 24 hodinami jsem nastavoval více proměnných v latte {var}, kde mě štvalo, že nejde napsat jednoduše:

{var
   $class1 = $item->value1,
   $class2 = $item->value1,
   $class3 = match($item->state){
      'state1' => 'class3',
      'state2' => 'class4'
   }
}

Ale musíte psát komplikovaně:

{var
   $class1 = $item->value1,
   $class2 = $item->value1,
}
{do
   $class3 = match($item->state){
      'state1' => 'class3',
      'state2' => 'class4'
      default => null
   }

Říkal jsem si, že bych někdy podal návrh na vylepšení Latte – implementovat match() v tagu {var} a volbu default, která je v šablonách nadbytečná učinit nepovinnou !!

Včera jsem si instaloval zkušební verzi Latte 3.0.5 RC1 a nevěřím svým očím ! Ono je to tam:

{var
   $class3 = match($item->state){   // ve verzi Latte 3.0.5 RC1 funguje !!
      'state1' => 'class3',
      'state2' => 'class4'
   }
}

Tak jsem si říkal, že tohle je možné snad jedině v Nette Frameworku – jenom si myslíte nějaké přání jak software vylepšit a do 24 hodin to máte tam :).

Doplním.

m.brecher napsal(a):

nette/forms:^3.1.9-RC:

Zástupný řetězec %label v chybových hláškách validace formulářů není optimálně vyřešen. Formuláře ho nahrazují popiskou prvku (bez dvojtečky na konci), ale neřeší se upper/lower – case prvního písmena popisky. Záleží totiž, zda zástupný řetězec %label použijeme na začátku chybové hlášky nebo uprostřed/na konci.

$form->addText('title', 'Nadpis', ...)
    ->setRequired('%label musí být vyplněn!')    // hláška:  'Nadpis musí být vyplněn!'

$form->addText('title', 'Nadpis', ...)
    ->setRequired('Vyplňte %label!')    // hláška:  'Vyplňte Nadpis!'

    // hlášky bychom ale raději měli takto:
    //  'Nadpis musí být vyplněn!'   - pro počáteční písmeno zachovat (vynutit) uppercase
    //  'Vyplňte nadpis!'            - pro počáteční písmeno použít lowercase

Navrhuji změnu vyhodnocení zástupného řetězce %label takto:

• je-li %label na začátku chybové zprávy – konvertovat první písmeno popisky prvku na UPPERCASE
• je-li %label jinde než na začátku chybové zprávy – konvertovat první písmeno popisky prvku na LOWERCASER

…a potom budem dostávať 'Vyplňte dIČ!'

Objevila se chyba při vykreslování formulářů v šabloně Latte – a sice partial rendering checkbox. Když je checkbox v těle formuláře tak šablona vykresluje, když je checkbox v kontejneru tak dojde k chybě v latte.

Podrobná dokumentace:

    protected function createComponentForm(): Form
    {
        $form = new Form;

        $form->addCheckbox('test', 'Test');  // tenhle checkbox šablona vykreslí

        $columns = $form->addContainer('pricelist_column');
        foreach ([1,2,3] as $id){
            $column = $columns->addContainer($id);
            $column->addCheckbox('test_checkbox_'.$id, 'Test');  // zde dojde v latte k chybě
        }

        bdump($form->getComponents());   // tady všechny checkboxy jsou

        return $form;
    }

Šablona:

// stručný extrakt ze složité šablony, otestováno
{form form}
    {foreach $form->getControls() as $input}
        {if $input->getOption(type) === checkbox}
            <label n:name="$input:" class="input-label">{$input->caption}</label>
            {input $input->name:}  // tento řádek Tracy zvýrazní
        {/if}
    {/foreach}
{/form}

Chybová hláška:

Nette\InvalidArgumentException
Component with name 'test_checkbox_1' does not exist.

Testoval jsem ještě vykreslení textInputu v kontejenru a to funguje, další prvky jsem netestoval.

Mám nainstalované obě testovací verze – jak forms, tak latte.

Editoval m.brecher (20. 1. 2023 2:38)

@mbrecher prosím, na nesouvisející věci vytvářej samostatné vlakna