Drukowana wersja tematu

Napisany przez: porzeczki 18.10.2016, 01:01:00

Próbowałem zadać uniwersalne pytanie, ale było bardziej nieczytelne. Może komuś mimo to zechce się rzucić okiem, uprościłem ile się dało.

Dlaczego listener nie wysyła poprawnie produktów w linii $event->getZamowienie()->getZamowienieProdukty()
a chwilę później kontroler robi to poprawnie w linii $produkty = $zamowienie->getZamowienieProdukty()
W debugerze w obu przypadkach $produkty są typu doctrine/orm/persistentcollection i wyglądają identycznie (ale w obu przypadkach nie widać jakie dane przechowują)

[PHP] pobierz, plaintext 
//Manager, tu zapisuje dane do db i wypstrykuję dispatchera
class ZamowienieManager
{
    public function tworzZamowienie($klient)
    {
        $zamowienie = new Zamowienie();
        $zamowienie->setIdklient($klient);
        $zamowienie->setDatazlozeniacurrent();
 
        $this->tworzProduktyZamowienia($zamowienie);
 
        $this->em->persist($klient);
        $this->em->persist($zamowienie);
        $this->em->flush();
 
        $this->dispatcher->dispatch(OrderPlacedEvent::NAME, new OrderPlacedEvent($zamowienie)); 
 
        $this->addFlashBag($zamowienie); 
 
    }
 
    private function tworzProduktyZamowienia($zamowienie)
    {
        foreach ($this->cart as $isbn => $quantity)
        {
            $ksiazka = $this->em
                ->getRepository('AppBundle:Ksiazka')
                ->find($isbn);
            $zm = new ZamowienieProdukt();
            $zm->setIdzamowienie($zamowienie);
            $zm->setIsbn($ksiazka);
            $zm->setTytul($ksiazka->getTytul());
            $zm->setAutor($ksiazka->getAutor());
            $zm->setIlosc($quantity);
            $this->em->persist($zm);
        }
    }
 
    public function addFlashBag($zamowienie){
        $this->session->getFlashBag()->add(
            'idzamowienie',$zamowienie->getIdzamowienie(););
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
//Listener. wysyła maila potwierdzającego zamówienie. POPRAWNIE WYŚWIETLA DANE ZAMÓWIENIA, NIE WYŚWIETLA PRODUKTÓW.
class Listener
{
    public function onOrderPlaced(OrderPlacedEvent $event)
    {
        ...
        $body = $this->renderTemplate($event);
        ...
    }
    private function renderTemplate($event)
    {
        return $this->twig->render(
            'AppBundle:Cart:potwierdzenieMail.html.twig',
            http://www.php.net/array(
                'zamowienie'=>$event->getZamowienie(),
                'produkty'=>$event->getZamowienie()->getZamowienieProdukty()
            )
        );
    }
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
//kontroler, POPRAWNIE WYŚWIETLA DANE ZAMÓWIENIA, I PRODUKTY.
    public function potwierdzenieAction(Request $request)
    {
        $idzamowienie=$request->getSession()->getFlashBag()->get('idzamowienie');
 
        $zamowienie = $this->getDoctrine()
            ->getRepository('AppBundle:Zamowienie')
            ->find($idzamowienie[0]);
        $produkty = $zamowienie->getZamowienieProdukty();
 
        return $this->render('AppBundle:Cart:potwierdzenie.html.twig',[
            'zamowienie'=>$zamowienie, 'produkty'=>$produkty]);
    }
[PHP] pobierz, plaintext 

Czy to chodzi o to, że bez przeładowania strony nie mogę wywołać powiązanej z Entity/Zamowienie (ManyToOne) kolekcji (Doctrine\Common\Collections\Collection) którą zwraca ->getZamowienieProdukty(), gdy ten obiekt (new Zamowienie) dopiero co utworzyłem i dopiero co zapisałem go flush() do bazy danych? (wiem, że to zdanie wygląda jakby było pisane przed dałna, staram się jak mogę, zasób słów jest jaki jest)

Napisany przez: kpt_lucek 18.10.2016, 02:12:22

Tak się dzieje, jak do produktu przypisujesz zamówienie, ale zamówienie nie wie o tym, że coś zostało do niego "przypisane".

W tym wypadku robiąc coś takiego:

[PHP] pobierz, plaintext 
$zm->setIdzamowienie($zamowienie);
[php]
i zakładając coś takiego:
[php]
class ZamowienieProdukt
{
    public function setIdZamowienie(Zamowienie $zamowienie)
    {
        $this->idZamowienie = $zamowienie;
 
        return $this;     
    }
}
[PHP] pobierz, plaintext 

w rzeczywistości powinieneś zrobić coś takiego:

[PHP] pobierz, plaintext 
class ZamowienieProdukt
{
    public function setIdZamowienie(Zamowienie $zamowienie)
    {
        $this->idZamowienie = $zamowienie;
        $zamowienie->addZamowienieProdukt($this);
 
        return $this;     
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
class Zamowienie
{
    public function addZamowienieProdukt(ZamowienieProdukt $zamowienieProdukt)
    {
        if(false === http://www.php.net/in_array($zamowienieProdukt, $this->zamowienieProdukty, true)) {
            $this->zamowienieProdukty[] = $zamowienieProdukt;
        }
 
        return $this;     
    }
}
[PHP] pobierz, plaintext 

Pytanie: Dlaczego?

Odpowiedź: Bo doctrine nie wie o zmianach które "stały się" w Twoim zamówieniu, co innego gdybyś pobrał to jako nowy wpis z bazy - czyli to, co robisz w kontrolerze.

Napisany przez: porzeczki 18.10.2016, 11:31:47

dziękuję, działa. (z twojego kodu pominąłem warunek in_array bo $this->zamowienieProdukty jest obiektem)

no właśnie, próbowałem robić:

[PHP] pobierz, plaintext 
//function tworzZamowienie
        ...
        $this->em->persist($zamowienie);
        $this->em->flush();
        $zamowienie = $this->em->getRepository('AppBundle:Zamowienie')->find($zamowienie->getIdzamowienie());
        $this->dispatcher->dispatch(OrderPlacedEvent::NAME, new OrderPlacedEvent($zamowienie));
[PHP] pobierz, plaintext 

i też zamówienie nie wiedziało o produktach.

Napisany przez: Turson 18.10.2016, 12:15:04

ZamowienieManager
tworzZamowienie
tworzProduktyZamowienia
addFlashBag
setIdklient
setDatazlozeniacurrent
itp.

karygodne nazewnictwo

Napisany przez: porzeczki 18.10.2016, 12:48:37

no wiem. Zacząłem tak nazywać, więc później już poleciała lawina tych potworków. Najpierw chciałem by było zrozumiałe dla mnie (bo nikt inny miał tego nie oglądać), ale szybko okazało się że to jest niezrozumiałe też dla mnie.

Napisany przez: kpt_lucek 18.10.2016, 13:18:44

$em->refresh($zamowienie);