[SF2][Symfony2][Symfony] Zapis encji w relacji, Zapis z poziomu kontrolera Opcje

[damianooo]

Czemu ponizszy zapis danych do bazy nie dziala ?

[PHP] pobierz, plaintext 
public function createAction(){
 
   $type = new Type();
   $match = new Match();
   $match->getId(3);
   $type->setNumberOfPoints(0);
   $type->setUser($this->getUser());
   $type->setMatch($match); 
   $em = $this->getDoctrine()->getManager();
   $em->persist($type);
   $em->flush();
 
}
[PHP] pobierz, plaintext 

Blad ktory otrzymyje jest nastepujacy:

[HTML] pobierz, plaintext 
A new entity was found through the relationship 'My\TyperkaBundle\Entity\Type#match' that was not configured to cascade persist operations for entity: . To solve this issue: Either explicitly call EntityManager#persist() on this unknown entity or configure cascade persist this association in the mapping for example @ManyToOne(..,cascade={"persist"}). 
[HTML] pobierz, plaintext 

Relacje miedzy encja Type i Match mam taka:

[PHP] pobierz, plaintext 
// Type.php 
 
class Type {
 
    /**
     * @ORM\ManyToOne(
     *      targetEntity = "Match",
     * )
     * 
     * @ORM\JoinColumn(
     *      name = "match_id",
     *      referencedColumnName = "id",
     *      onDelete = "SET NULL"
     * )
     */
    private $match;
[PHP] pobierz, plaintext 

[Crozin]

Szybkie pytanie: co jest niezrozumiałe w treści wyjątku, który podaje Ci dwa możliwe rozwiązania problemu?

[uirapuru]

szybka odpowiedz $em->persist($match) przed flushem powinno pomóc. btw: nie rób tego w kontrolerze.

[damianooo]

Ok, dodalem:

[PHP] pobierz, plaintext 
$em->persist($match);
[PHP] pobierz, plaintext 

i udalo sie ... faktycznie nie doczytalem komunikatu, ktory przeciez jasno pisal co trzeba zrobic.

ale nie wiem czemu do bazy nie zostal dodany rekord do tabeli Type o match_id = 3 tylko o match_id = 41 (zainkrementowalo mi match_id )
a wskazalem przeciez w kodzie ze ma zostac dodany rekord z match_id = 3 :

[PHP] pobierz, plaintext 
    public function createAction(){
 
       $type = new Type();
       $match = new Match();
       $match->getId(3);
       $type->setNumberOfPoints(0);
       $type->setUser($this->getUser());
       $type->setMatch($match); 
       $em = $this->getDoctrine()->getManager();
       $em->persist($match);
       $em->persist($type);
       $em->flush();
    }
[PHP] pobierz, plaintext 

[kapslokk]

[PHP] pobierz, plaintext 
       $match->getId(3);
[PHP] pobierz, plaintext 

get to nie set.

[PHP] pobierz, plaintext 
       $match->setId(3); 
[PHP] pobierz, plaintext 

Ale jesli ten match juz istnieje w bazie to chyba powinienes go pobrac

[PHP] pobierz, plaintext 
$match = $em->getRepository(...)->findOneById(3)
[PHP] pobierz, plaintext 

Ten post edytował kapslokk 23.03.2016, 11:50:22

[ohm]

Szybkie pytanie: co jest niezrozumiałe w treści wyjątku, który podaje Ci dwa możliwe rozwiązania problemu?

Podpowiadam

Kod

To solve this issue: Either explicitly call EntityManager#persist() on this unknown entity or configure cascade persist this association in the mapping for example @ManyToOne(..,cascade={"persist"}).

[damianooo]

nie ma takiej metody w mojej encji Match jak setId()
to chyba nie tak sie robi ...

Tak - takie $match o ID = 3 juz istnieje dlatego chcialem pobrac getId()

rozumiem ze musze to zrobic tak:

[PHP] pobierz, plaintext 
$match = $em->getRepository(...)->findOneById(3)
[PHP] pobierz, plaintext 

i nie ma innej drogi

zastanawia mnie tylko dlaczego kolega wczesniej prosil/ostrzegal zeby zapisu nie robic w konrolerze ... rozumiem ze chodzi o ograniczenie linii kodu ?

Ten post edytował damianooo 23.03.2016, 11:56:00

[kapslokk]

No bo tak jak Ci napisałem, nie powinieneś tego ustawiać ręcznie tylko pobrać z bazy istniejący rekord / zdać się na auto_increment;

getId() jak i cała reszta getterów służy do pobrania wartości z konkretnego obiektu, a nie obiektu z bazy na podstawie tej wartości.

zastanawia mnie tylko dlaczego kolega wczesniej prosil/ostrzegal zeby zapisu nie robic w konrolerze smile.gif ... rozumiem ze chodzi o ograniczenie linii kodu ?

Nie mam pojęcia, ktoś mądrzejszy sie musi wypowiedzieć.

[Crozin]

ale nie wiem czemu do bazy nie zostal dodany rekord do tabeli Type o match_id = 3 tylko o match_id = 41 (zainkrementowalo mi match_id )

Zapewne dla właściwości @Id masz ustawione automatyczne generowanie wartości (@GeneratedValue(AUTO)). Wtedy Doctrine zignoruje ręcznie ustawioną wartość przez setId(x).

nie ma takiej metody w mojej encji Match jak setId()
to chyba nie tak sie robi ...

Jeżeli decydujesz się na ręczne bądź półautomatyczne (np. przy pomocy sekwencji) nadawanie ID-ków, to taką metodę setId() jak najbardziej możesz czy wręcz musisz mieć. Generalnie nawet jeżeli ID-ki generowane są wyłącznie przez bazę danych metoda setId() wiele nie wadzi, a przy pisaniu testów może się przydać.

zastanawia mnie tylko dlaczego kolega wczesniej prosil/ostrzegal zeby zapisu nie robic w konrolerze (IMG:style_emoticons/default/smile.gif) ... rozumiem ze chodzi o ograniczenie linii kodu ?

Liczba linii kodu jest bez znaczenia. Chodzi po prostu o to, że kontroler to nie miejsce dla ORM-a, który raczej powinien być użyty dopiero gdzieś głęboko w warstwie przetwarzania danych, czy właściwie dostępu do danych. Ale nie zawsze potrzebujemy takiej separacji kodu, na różne warstwy stąd Symfony udostępnia "łatwy" dostęp do Doctrine'a wewnątrz kontrolerów dziedziczących po Symfony\Bundle\FrameworkBundle\Controller\Controller.

[uirapuru]

podpowiem: przypuśćmy, że chcesz dokładnie te samą operację wykonać w symfonowym commandzie. naturalnie, skopiujesz i przekleisz ten kod. co lepsi nawet rejestrują kontroler jako serwis i wywołują te akcje. Ale zawartość akcji Twojego kontrolera jest aplikacją samą w sobie, można jej fragment wydzielić i po prostu użyc: w kontrolerze, w commandzie, a co najwazniejsze - w teście

ps. spróbuj przetestować np. jednostkowo te akcje kontrolera.

ps2. bo testujesz, prawda?

ps3. przypuścmy sytuację, że pojawia się nowa niekompatybilna wstecz wersja frameworka LUB chcesz zmienić framework. Też kapa przy Twoim podejściu

Generalnie: jest wiele powodów, żeby tak nie robić, niewiele powodów by tak robić

Ten post edytował uirapuru 23.03.2016, 13:20:54

[damianooo]

ok dzieki za uwagi ... przemysle to sobie ... chce to robic dobrze wiec bede o tych uwagach pamietal.

Pozdrawiam