Dynamic snippets: only they are put to payload

[vojtech-dobes]

{snippet container}
    {foreach [1, 2, 3] as $id}
        {snippet dynamic-$id} ... {/snippet}
    {/foreach}
{/snippet}

Now, you have to invalidate only static snippet container. With my alteration, you have to invalidate both static snippet and dynamic snippet - and only dynamic snippet is put to payload. I believe it's the ussual desired behavior. Also, when you invalidate only static snippet, it adds all dynamic snippets as one item to payload (simply whole content of static snippet).

Example:

$this->invalidateControl('container');
$this->invalidateControl('dynamic-2');

results to payload:

{
    snippets: {
        'snippet--dynamic-2': ...
    }
}

[hrach]

No, ne, ze bych byl v kodu snippetu zbehly, takze substr(x, 9) moc nechapu, nicmene chybi mi testy a jestek k tvemu popisu, prijde mi, ze posledni veta je presne to, co nechci. Pokud invaliduji pouze container, tak chci vsechny dynamicky snippety, pokud invaliduji pouze container a dynamic-1, tak chci pouze dynamic-1.

[vojtech-dobes]

Tou poslední větou jsem to zamotal... funguje to přesně jak říkáš ty. Jde jen o to, že se to nepošle rozdělené do dynamických snippetů, ale pošle se celý ten kontejner jako jedna položka v payloadu.

[vojtech-dobes]

Co se týče testů - rád bych, ale na metodu renderSnippets pokud vím neexistují žádné testy ani teď, a moc nevím, jak na ně.

[natrim]

mohl bych vědět jakto vypada s touto úpravou?
docela by se hodilo mít možnost invalidovat jen jeden dynamický snippet a ostatní ne

[vojtech-dobes]

The way how to achieve that (and with performance benefit this pull request only bypasses, not solves) is to narrow the iterated array in template for desired AJAX request. Something like this in handle method:

if ($this->isAjax()) {
    $this->template->myArray = array(1);
}

Sorry for English, I am switched and don't want to translate it to Czech now ;)

[dg]

To narrow the iterated array in template is correct solution.