@hrach: Objeven další bug v tvé větvi

Upozornění: Tohle vlákno je hodně staré a informace nemusí být platné pro současné Nette.
jtousek
Člen | 951
+
0
-

Duskohu je chlapec šikovný a tak tu od něj máme další chybu. :-D

Tato chyba se projevila až po tom včerejším fixu (jinak to selhalo ještě dřív), takže jsem to testoval pouze na té testovací větvi s tímto fixem. Navíc mám dojem že tuto chybu pravděpodobně způsobil právě ten fix.

Jde o to, že z tabulky task existuje více M:N vazeb na tabulku user. Jedna je přes tabulku executor, druhá přes tabulku observer.

V šabloně /app/components/TaskList/TaskList.latte je toto:

<td class="executor">
  {foreach $task->related('executor') as $executor}
    {$executor->id}-{$executor->user->name} <br>
  {/foreach}
</td>

<td class="observer">
  {foreach $task->related('observer') as $observer}
    {$observer->id}-{$observer->user->name} <br>
  {/foreach}
</td>

V tom druhém foreachi to na {$observer->user->name} selže (Trying to get property of non-object). Pokud ale první foreach zakomentuju, tak to funguje. Problém se vyskytne pouze když ty vazby použiju obě dvě najednou.

Celou aplikaci jsem opět zabalil, kdybys ji náhodou potřeboval, ale možná to zvládneš i bez toho.

hrach
Člen | 1819
+
0
-

No, přesně toto sem tam včera měl a fungovalo to. mrknu na to zabaleny, zajimalo by me, v cem je to jine. :))

jtousek
Člen | 951
+
0
-

Ok, doufám že jsem tam někde něco nepokonil já. To se samozřejmě také mohlo stát.

hrach
Člen | 1819
+
0
-

Jéje. Byla tam chyba, celkem velika :)) Opraveno tu:
https://github.com/…ree/f-db-wip

jtousek
Člen | 951
+
0
-

Skvělé! :-)

Tak už jen testy.

hrach
Člen | 1819
+
0
-

No prave… :D to zabere vzdy polovinu casu :D

duskohu
Člen | 771
+
0
-

Super bezi ako hodinky, paradna pracicka … dakujem
niet takych ludi tak je to cele na nic :-)

Editoval duskohu (17. 7. 2012 12:32)

hrach
Člen | 1819
+
0
-

Tak, uz sem to pridal do pull requstu s testy :)
Ale ze to dalo zabrat, ty testy… Mj. sem diky tomu obejvil dalsi chybicku :) Vse opraveno.

https://github.com/…tte/pull/692