Nefunkční překreslení snippetu, pokud v něm neproběhne php vykreslení

Upozornění: Tohle vlákno je hodně staré a informace nemusí být platné pro současné Nette.
před 8 lety
JaxP
Člen | 62
+
0
-

Dobrý den,
dokáže mi prosím někdo vysvětlit toto chování při překreslování snippetů? V případě, že se jedná pouze o script, nepřekreslí se. V případě, že se v něm vykreslí jakákoliv nová hodnota (proměnná, aktuální čas etc…) tak se překreslí. V názorné ukázce potřebuji docílit spuštění „alertu“ bez jakéhokoliv vypsání čehokoliv…

Nefunguje (tzn. snippet se nepřekreslí):

{snippet unitScript}
    <script>
        alert('reloaded');
    </script>
{/snippet}

Funguje (tzn. snippet se překreslí):

{snippet unitScript}
    {time()}
    <script>
        alert('reloaded');
    </script>
{/snippet}

Děkuji

před 8 lety
David Matějka
Moderator | 6445
+
+1
-

souvisi to s timto commitem v nette.ajax.js https://github.com/…9508dae30570 .. mela by se tam pridat kontrola, ze to ma vzdy prekreslovat, pokud snippet obsahuje script. Zvladnul bys poslat PR? pripadne, jako hotfix by mohlo stacit, ze nejaky ten time() vypises do JS komentare treba

před 8 lety
JaxP
Člen | 62
+
0
-

Děkuji, hotfix funguje.

Po rozšíření nette.ajax o:

...
} else if (html.includes("<script>")) {
	$el.html(html);
}

funguje také.

Co máš na mysli „PR“?

před 8 lety
CZechBoY
Člen | 3608
+
+1
-

@JaxP pull request – že tvoji změnu vložíš do oficiálního repozitáře na githubu

před 8 lety
JaxP
Člen | 62
+
0
-

Bohužel se mi nedaři projít ssh zabezpečením (i když public key mám generovaný a v jiných případech funguje)

před 8 lety
Barvoj
Člen | 60
+
+1
-

JaxP napsal(a):

Bohužel se mi nedaři projít ssh zabezpečením (i když public key mám generovaný a v jiných případech funguje)

Nesnažíš se pushovat přímo do oficiálního repositáře?

Správný postup je, že si vytvoříš fork oficiálního repositáře. Tam změnu provedeš. A pak vytvoříš PR, kterým se ta tvá změna zanese opět do původního repositáře.

Editoval Barvoj (15. 8. 2016 13:30)

před 8 lety
Michal Hlávka
Člen | 190
+
0
-

Dovolil jsem ten PR udělat, pokud to JaxP nefunguje.

před 8 lety
JaxP
Člen | 62
+
0
-

Omlouvám se za obtíže, ale rád bych se to rovnou v praxi naučil.

Nyní mám tedy forknutý a pozměněný projekt

  1. Měl bych změny provádět na master branch nebo si na konkrétní úpravu založit novou branch?
  2. Jak vytvořit „pull request“ z mého účtu? (V nabídce mám pouze výběr branchí z původního projektu).

Děkuji)

Edit:// Povedlo se :)

Editoval JaxP (15. 8. 2016 14:22)

před 8 lety
David Matějka
Moderator | 6445
+
+1
-

@JaxP mrkni na https://www.youtube.com/watch?…

ve zkratce

  1. forknes
  2. naklonujes si k sobe fork
  3. zalozis branch, provedes upravu, commitnes
  4. pushnes do tveho forku tu branch
  5. na githubu u toho projektu se ti hned potom zobrazi lista s „create pull request“
před 8 lety
JaxP
Člen | 62
+
0
-

Takže vždy novou branch? Dobře, děkuji za pomoc. Nastuduji pro příště.)