Do not place {$orderId} inside quotes in

MKI-Miro

Ahojte

Ako spojazdnim takyto js zapis?

_hrq.push(['setOrderId', '{$orderId}']);

Stale mi to hlasi Do not place {$orderId} inside quotes in

Dakujem

Editoval MKI-Miro (18. 11. 2018 18:05)

David Matějka

Presne tak, jak je v error message, nedavat {$orderId} do uvozovek

MKI-Miro

ale tie uvodzovky tam su potrebne

Mysteria

Kdybys poslechl tu hlášku, tak bys zjistil, že ti je to tam doplní samo. :)

Editoval Mysteria (18. 11. 2018 18:19)

David Matějka

nette to je tam da, pokud jsou potreba (tzn pokud orderId bude string)

MajklNajt

@DavidMatějka nie Nette, ale Latte..

Sorry, musel som podpichnúť ;)

Robin Martinez

Cus, muze mi nekdo prosim rict, jak se tedy spravne resi nasledujici radek v JS:

	const search = '{$search}'

za predpokladu, ze $search muze byt prazdny? Prisel jsem na to, ze

    const search = `{$search}`

ZATIM funguje, ale jelikoz kazda nova verze latte a obecne vseho v nete roz**be vsechno okolo, radsi se ptam. Diky

MajklNajt

Malo by to byť bez úvodzoviek

const search = {$search}

Robin Martinez

ne, nejde, latte tam nehodi nic a kod vypada takto:

const search =

const xxx = '' //dalsi radky random kodu

cimz vznikne syntax error v javascriptu

Editoval Robin Martinez (11. 3. 2022 11:28)

David Grudl

Jaká přesně verze a co přesně obsahuje proměnná?

jendaa

const search = {$search ?: ''}

Robin Martinez

Ještě dodatek, který jsem se dozvěděl až teď, který za to může je, že je tam |noescape:

	const search = {$search|noescape}

bez něj se tam uvozovky vkládají správně

Davide, proměnná je prázdný string:

'' barDump($search) 📍

a zkoušel jsem to na verzi latte 2.7, 2.8, 2.9 a 2.11

David Grudl

Nikdy nepouzivej noescape! Jen pokud přesně víš, co děláš. Což nevíš a Latte se tě na to snaží upozornit.

https://github.com/…e/issues/283#…

Robin Martinez

To pochopitelne vim, jen predelavam kod byvaleho programatora. Nicmene nevim, jestli je spravne, ze latte ‚znici‘ cely javascript.

EDIT: Ok, podival jsem se na to do detailu a vubec nevim, proc tam predchozi programator to noescape mel. Dal jsem ho pryc a zatim to vypada funkcne.

Diky za info

Editoval Robin Martinez (14. 3. 2022 8:53)

Kamil Valenta

Ten JavaScript nezničilo latte, ale bývalý programátor.

MKI-Miro

Ahojte

Ja este doplnm otazku ako spravit nieco taketo:

Toto samozrejme hodi chybu „Do not place inside quotes“

<script>

        dataLayer.push({
            'category': "{$product->name}/{$product->category->name}",
        });
</script>

bez uvodozviek to ale chce vykonat operaciu delenia a ja by som potreboval aby to sravilo:

'category': "Nazov/Kategoria"

stpnkcrk

@MKI-Miro

<script>
	dataLayer.push({
		'category': {$product->name} + '/' + {$product->category->name},
	});
</script>

David Grudl

Případně

<script>
	dataLayer.push({
		'category': {$product->name . '/' . $product->category->name},
	});
</script>

Robin Martinez

Kamil Valenta napsal(a):

Ten JavaScript nezničilo latte, ale bývalý programátor.

S tím naprosto souhlasím. Však víme, jak to chodí.

Jen jsem naznačoval, že nevím, proč se v latte vždycky změní/rozbije něco, co dříve fungovalo. Teď jsme natáhli nové latte 2.11 a v celým projektu akorát předěláváme dvojtečky na čárky. Pokrok nezastavíš :)

David Grudl

Dřív to nefungovalo. Měl jsi to blbě a Latte tě na to upozornilo, tak mu podekuj a neremcej :)

Robin Martinez

Dekuji moc Davide,

velice radi prispivame na projekt, u ktereho nam tvurce rekne, at neremcame ;)

Editoval Robin Martinez (20. 5. 2022 9:54)

Kamil Valenta

Ale Ty nepřispíváš na to, abys mohl remcat.
Přispíváš na to, aby Tě latte varovalo před více a více hovadinami, které do kódu zanese třeba i předchozí programátor.

Nechápu, co Ti vadí. Měli jste v kódu kravinu, latte Tě na ni upozornilo, řeklo Ti, jak ji opravit. Tys ji opravil a je to OK. To je přece ideální stav.