[Doctrine2] Klucz obcy w encji., Single id is not allowed on composite primary key in entity Opcje

[ghost1511]

Mam dwie encje:

[PHP] pobierz, plaintext 
<?php
 
namespace Album\Entity;
 
use Doctrine\ORM\Mapping as ORM;
use Doctrine\Common\Collections\ArrayCollection;
use Album\Entity\Song;
use Doctrine\Common\Collections\Collection;
 
/**
 * Description of Album
 * 
 * @ORM\Entity
 * @ORM\Table(name="Album")
 */
class Album {
 
    /**
     * @ORM\Id
     * @ORM\Column(type="integer");
     * @ORM\GeneratedValue(strategy="AUTO")
     * @var integer 
     */
    protected $albumId;
 
    /**
     * @ORM\Column(type="string")
     */
    protected $artist;
 
    /**
     * @ORM\Column(type="string")
     */
    protected $title;
 
    /**
     *
     * @var ArrayCollection
     * 
     * @ORM\OneToMany(targetEntity="Song", mappedBy="album", cascade={"persist"})
     */
    protected $songs;
 
    function __construct() {
        $this->songs = new ArrayCollection();
    }
 
    function getAlbumId() {
        return $this->albumId;
    }
 
    function getArtist() {
        return $this->artist;
    }
 
    function getTitle() {
        return $this->title;
    }
 
    function setAlbumId($albumId) {
        $this->albumId = $albumId;
    }
 
    function setArtist($artist) {
        $this->artist = $artist;
    }
 
    function setTitle($title) {
        $this->title = $title;
    }
 
    public function addSongs(Collection $songs) {
        foreach ($songs as $song) {
            $song->setAlbum($this);
            $this->songs->add($song);
        }
    }
 
    public function removeSongs(Collection $songs) {
        foreach ($songs as $song) {
        $song->setAlbum(NULL);
        $this->songs->remove($song);
        }
    }
 
    function getSongs() {
        return $this->songs;
    }
 
}
[PHP] pobierz, plaintext 

oraz:

[PHP] pobierz, plaintext 
<?php
 
namespace Album\Entity;
 
use Doctrine\ORM\Mapping as ORM;
use Album\Entity\Album;
 
/**
 * Description of Song
 *
 * 
 * @ORM\Entity
 * @ORM\Table(name="Song")
 */
class Song {
 
    /**
     * @var string 
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     * @ORM\Column(type="string");
     */
    protected $title;
 
    /**
     *
     * @var Album
     * 
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="NONE")
     * @ORM\ManyToOne(targetEntity="Album", inversedBy="songs")
     * @ORM\JoinColumn(name="albumId", referencedColumnName="albumId")
     */
    protected $album;
 
    function getTitle() {
        return $this->title;
    }
 
    function getAlbum() {
        return $this->album;
    }
 
    function setTitle($title) {
        $this->title = $title;
    }
 
    function setAlbum(Album $album) {
        $this->album = $album;
    }
 
 
 
}
[PHP] pobierz, plaintext 

Oraz kontroler:

[PHP] pobierz, plaintext 
<?php
 
namespace Album\Controller;
 
use Zend\Mvc\Controller\AbstractActionController;
use Zend\View\Model\ViewModel;
use Doctrine\ORM\EntityManager;
use DoctrineModule\Stdlib\Hydrator\DoctrineObject as DoctrineHydrator;
use Album\Entity\Album;
 
class AlbumController extends AbstractActionController {
 
    protected $doctrineHydrator;
 
    public function indexAction() {
 
        $killEmAllData = array(
            'artist' => 'Metallica',
            'title' => 'Kill em All',
            'songs' => array(
                array('title' => 'Hit the Lights'),
                array('title' => 'The Four Horsemen'),
                array('title' => 'Motorbreath'),
                array('title' => 'Jump in the Fire'),
                array('title' => '(Anesthesia) Pulling Teeth'),
                array('title' => 'Whiplash'),
                array('title' => 'Phantom Lord'),
                array('title' => 'No Remorse'),
                array('title' => 'Seek & Destroy'),
                array('title' => 'Metal Militia'),
                array('title' => 'Am I Evil?'),
                array('title' => 'Blitzkrieg'),
            )
        );
 
        var_dump($killEmAllData);
 
        $killEmAll = new Album();
        $hydrator = $this->getDoctrineHydrator();
        $killEmAllHydrate = $hydrator->hydrate($killEmAllData, $killEmAll);
 
        $songsCollection = new \Doctrine\Common\Collections\ArrayCollection();
        foreach ($killEmAllData['songs'] as $songArray) {
            $song = $hydrator->hydrate($songArray, new \Album\Entity\Song());
            $songsCollection->add($song);
        }
        $killEmAll->addSongs($songsCollection);
 
        var_dump($killEmAllHydrate);
 
        return new ViewModel();
    }
 
    /**
     * @var DoctrineORMEntityManager
     */
    protected $entityManager;
 
    public function getEntityManager() {
        if (null === $this->entityManager) {
            $this->entityManager = $this->getServiceLocator()->get('doctrine.entitymanager.orm_default');
        }
        return $this->entityManager;
    }
 
    /**
     * @return DoctrineHydrator
     */
    private function getDoctrineHydrator() {
        if (!$this->doctrineHydrator instanceof DoctrineHydrator) {
            $this->doctrineHydrator = new DoctrineHydrator($this->getEntityManager());
        }
        return $this->doctrineHydrator;
    }
 
}
[PHP] pobierz, plaintext 

I otrzymuję błąd:

Kod

Fatal error: Uncaught exception 'Doctrine\ORM\Mapping\MappingException' with message 'Single id is not allowed on composite primary key in entity Album\Entity\Song' in D:\xampp1.8.2\htdocs\testy\DoctrineCompositeForeignKey\vendor\doctrine\orm\lib\Doctrine\ORM\Mapping\MappingException.php on line 414

Czy to ja robię coś źle? Czy to jakieś dziwne ograniczenie doctrine?

Ten post edytował ghost1511 5.11.2014, 14:15:44

[mortus]

Encja Song ma złożony klucz główny (przyjrzyj się @ORM\Id) i jest identyfikowana przez title i album jednocześnie.

[PHP] pobierz, plaintext 
    /**
     * @var string 
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     * @ORM\Column(type="string");
     */
    protected $title;
 
    /**
     *
     * @var Album
     * 
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="NONE")
     * @ORM\ManyToOne(targetEntity="Album", inversedBy="songs")
     * @ORM\JoinColumn(name="albumId", referencedColumnName="albumId")
     */
    protected $album;
[PHP] pobierz, plaintext 

Nie jestem pewien, czy to celowy zabieg, czy może pomyłka.

[ghost1511]

Zabieg celowy. Wg dokumentacji jest to możliwe:
http://doctrine-orm.readthedocs.org/en/lat...imary-keys.html

Jednak wygląda na to, że robię coś źle. Nie potrafię jednak stwierdzić co.


Ten post edytował ghost1511 5.11.2014, 20:12:36

[mortus]

Skoro jest to zabieg celowy, to konstruktor encji Song powinien wyglądać tak:

[PHP] pobierz, plaintext 
public function __construction(string $title, Album $album) {
  $this->title = $title;
  $this->album = $album;
}
[PHP] pobierz, plaintext 

Natomiast podczas tworzenia obiektu Song powinieneś podać zarówno jego title, jak i Album, do którego Song jest przypisany:

[PHP] pobierz, plaintext 
$album = new Album();
$song = new Song('Jakiś tytuł', $album);
[PHP] pobierz, plaintext 

Nie jestem pewien, czy hydrator Doctrine poradzi sobie z tym, gdyby jednak miał, to trzeba to uwzględnić w pętli foreach:

[PHP] pobierz, plaintext 
$songsCollection = new \Doctrine\Common\Collections\ArrayCollection();
foreach ($killEmAllData['songs'] as $songArray) {
  $songArray['album'] = $killEmAll;
  $song = $hydrator->hydrate($songArray, new \Album\Entity\Song());
  $songsCollection->add($song);
}
$killEmAll->addSongs($songsCollection);
[PHP] pobierz, plaintext 

Mamy tutaj takie małe "błędne" koło.

Ten post edytował mortus 5.11.2014, 20:42:05

[ghost1511]

Wprowadziłem Twoje zmiany i jednak nie pomogło. Wygląda na to, że nie tutaj leży problem. Już na etapie uruchamiania Doctrine pojawia się ten problem:

Kod

Fatal error: Uncaught exception 'Doctrine\ORM\Mapping\MappingException' with message 'Single id is not allowed on composite primary key in entity Album\Entity\Song' in D:\xampp1.8.2\htdocs\testy\DoctrineCompositeForeignKey\vendor\doctrine\orm\lib\Doctrine\ORM\Mapping\MappingException.php on line 414
( ! ) Doctrine\ORM\Mapping\MappingException: Single id is not allowed on composite primary key in entity Album\Entity\Song in D:\xampp1.8.2\htdocs\testy\DoctrineCompositeForeignKey\vendor\doctrine\orm\lib\Doctrine\ORM\Mapping\MappingException.php on line 414
Call Stack
#    Time    Memory    Function    Location
1    0.0007    137544    {main}( )    ..\index.php:0
2    0.3081    5281208    Zend\Mvc\Application->run( )    ..\index.php:17
3    0.3206    5421056    Zend\Mvc\Application->completeRequest( )    ..\Application.php:327
4    0.3649    6199776    Zend\EventManager\EventManager->trigger( )    ..\Application.php:353
5    0.3649    6199904    Zend\EventManager\EventManager->triggerListeners( )    ..\EventManager.php:207
6    0.3656    6204744    call_user_func:{D:\xampp1.8.2\htdocs\testy\DoctrineCompositeForeignKey\vendor\zendframework\zendframework\library\Zend\EventManager\EventManager.php:468} ( )    ..\EventManager.php:468
7    0.3656    6204760    ZendDeveloperTools\Listener\ProfilerListener->onFinish( )    ..\EventManager.php:468
8    0.4202    8005080    ZendDeveloperTools\Profiler->collect( )    ..\ProfilerListener.php:107
9    0.4278    8103200    DoctrineORMModule\Collector\MappingCollector->collect( )    ..\Profiler.php:211
10    0.4279    8103256    Doctrine\Common\Persistence\Mapping\AbstractClassMetadataFactory->getAllMetadata( )    ..\MappingCollector.php:97
11    0.4776    8772984    Doctrine\Common\Persistence\Mapping\AbstractClassMetadataFactory->getMetadataFor( )    ..\AbstractClassMetadataFactory.php:114
12    0.4776    8773016    Doctrine\Common\Persistence\Mapping\AbstractClassMetadataFactory->loadMetadata( )    ..\AbstractClassMetadataFactory.php:211
13    0.4777    8775936    Doctrine\ORM\Mapping\ClassMetadataFactory->doLoadMetadata( )    ..\AbstractClassMetadataFactory.php:318
14    0.4825    8801744    Doctrine\ORM\Mapping\ClassMetadataFactory->completeIdGeneratorMapping( )    ..\ClassMetadataFactory.php:140
15    0.4825    8801808    Doctrine\ORM\Mapping\ClassMetadataInfo->getSingleIdentifierFieldName( )    ..\ClassMetadataFactory.php:451

[mortus]

Problem zapewne pojawia się w tym miejscu:

[PHP] pobierz, plaintext 
$killEmAllHydrate = $hydrator->hydrate($killEmAllData, $killEmAll);
[PHP] pobierz, plaintext 

Korzystamy tutaj z tablicy $killEmAllData, w której songs identyfikowane są jedynie przez title. Trzeba by było zatem zmienić strukturę danych:

[PHP] pobierz, plaintext 
$killEmAll = new Album();
$killEmAllData = array(
  'artist' => 'Metallica',
  'title' => 'Kill em All',
  'songs' => array(
    array('title' => 'Hit the Lights', $killEmAll),
    array('title' => 'The Four Horsemen', $killEmAll),
    array('title' => 'Motorbreath', $killEmAll),
    array('title' => 'Jump in the Fire', $killEmAll),
    array('title' => '(Anesthesia) Pulling Teeth', $killEmAll),
    array('title' => 'Whiplash', $killEmAll),
    array('title' => 'Phantom Lord', $killEmAll),
    array('title' => 'No Remorse', $killEmAll),
    array('title' => 'Seek & Destroy', $killEmAll),
    array('title' => 'Metal Militia', $killEmAll),
    array('title' => 'Am I Evil?', $killEmAll),
    array('title' => 'Blitzkrieg', $killEmAll),
  )
);
$hydrator = $this->getDoctrineHydrator();
$killEmAllHydrate = $hydrator->hydrate($killEmAllData, $killEmAll);
 
$songsCollection = new \Doctrine\Common\Collections\ArrayCollection();
foreach ($killEmAllData['songs'] as $songArray) {
  $song = $hydrator->hydrate($songArray, new \Album\Entity\Song());
  $songsCollection->add($song);
 }
 $killEmAll->addSongs($songsCollection);
[PHP] pobierz, plaintext 

Nie jestem również pewien kolejności wykonywanych działań. Wg dokumentacji https://github.com/doctrine/DoctrineModule/...any-association, najpierw należałoby utworzyć tablicę obiektów Song (jeśli takowe encje w bazie danych nie istnieją), a dopiero później można użyć hydratora, przy czym kolekcja $songs zostanie wypełniona automatycznie. Wtedy kod wyglądałby tak:

[PHP] pobierz, plaintext 
$killEmAll = new Album();
$killEmAllSongsData = array(
  array('title' => 'Hit the Lights', $killEmAll),
  array('title' => 'The Four Horsemen', $killEmAll),
  array('title' => 'Motorbreath', $killEmAll),
  array('title' => 'Jump in the Fire', $killEmAll),
  array('title' => '(Anesthesia) Pulling Teeth', $killEmAll),
  array('title' => 'Whiplash', $killEmAll),
  array('title' => 'Phantom Lord', $killEmAll),
  array('title' => 'No Remorse', $killEmAll),
  array('title' => 'Seek & Destroy', $killEmAll),
  array('title' => 'Metal Militia', $killEmAll),
  array('title' => 'Am I Evil?', $killEmAll),
  array('title' => 'Blitzkrieg', $killEmAll),
);
$hydrator = $this->getDoctrineHydrator();
$songs = array();
foreach($killEmAllSongsData as $songArray) {
  $songs[] = $hydrator->hydrate($songArray, new \Album\Entity\Song())
}
$killEmAllData = array(
  'artist' => 'Metallica',
  'title' => 'Kill em All',
  'songs' => $songs
);
$killEmAllHydrate = $hydrator->hydrate($killEmAllData, $killEmAll);
[PHP] pobierz, plaintext 

Przyjrzyj się dokładniej dokumentacji, którą możesz znaleźć pod odnośnikiem powyżej, bowiem Example 3 opisuje podobny przypadek, z tym, że Ty używasz klucza złożonego w połączeniu z kluczem obcym.

[ghost1511]

Problemem nie są dane. Kontroler się nawet nie uruchamia. Doctrine podczas uruchamiania sprawdza powiązania encji i wtedy rzuca błędem, zresztą widać to na Call Stacku który wrzuciłem.

Przykład który podałeś jest bez @ID, które wszystko komplikuje. Spójrz jeszcze raz w dokumentację http://doctrine-orm.readthedocs.org/en/lat...oreign-entities.

The semantics of mapping identity through foreign entities are easy:

Only allowed on Many-To-One or One-To-One associations.

To chyba odpowiedz na moje pytanie.

Jest sposób obejścia tego problemu. Nie jest on może tak elegancki jak hydrator. Ale też zgrabny:

[PHP] pobierz, plaintext 
<?php
 
namespace Album\Entity;
 
use Doctrine\ORM\Mapping as ORM;
use Album\Entity\Album;
 
/**
 * Description of Song
 *
 * @author Bartosz Bartniczak <bbartniczak@zenex.pl>
 * 
 * @ORM\Entity
 * @ORM\Table(name="Song")
 */
class Song {
 
    /**
     * @var string 
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     * @ORM\Column(type="string");
     */
    protected $title;
 
    /**
     *
     * @var integer
     * 
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="NONE")
     * @ORM\Column(type="integer");
     */
    protected $albumId;
 
    /**
     *
     * @var Album
     * 
     * @ORM\ManyToOne(targetEntity="Album", inversedBy="songs")
     * @ORM\JoinColumn(name="albumId", referencedColumnName="albumId")
     */
    protected $album;
 
    function __construct($title = NULL, Album $album = NULL) {
        $this->setTitle($title);
        if (!is_null($album)) {
            $this->setAlbum($album);
        }
    }
 
    function getTitle() {
        return $this->title;
    }
 
    function getAlbum() {
        return $this->album;
    }
 
    function setTitle($title) {
        $this->title = $title;
    }
 
    function setAlbum(Album $album) {
        $this->album = $album;
        $this->albumId = $album->getAlbumId();
    }
 
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
public function indexAction() {
 
        $killEmAllData = array(
            'artist' => 'Metallica',
            'title' => 'Kill em All',
            'songs' => array(
                array('title' => 'Hit the Lights'),
                array('title' => 'The Four Horsemen'),
                array('title' => 'Motorbreath'),
                array('title' => 'Jump in the Fire'),
                array('title' => '(Anesthesia) Pulling Teeth'),
                array('title' => 'Whiplash'),
                array('title' => 'Phantom Lord'),
                array('title' => 'No Remorse'),
                array('title' => 'Seek & Destroy'),
                array('title' => 'Metal Militia'),
                array('title' => 'Am I Evil?'),
                array('title' => 'Blitzkrieg'),
            )
        );
 
        $songs = $killEmAllData['songs'];
        unset($killEmAllData['songs']);
 
//        var_dump($killEmAllData);
//
        $hydrator = $this->getDoctrineHydrator();
        $killEmAll = $hydrator->hydrate($killEmAllData, new Album());
        $this->getEntityManager()->persist($killEmAll);
        $this->getEntityManager()->flush();
 
        $songsCollection = new \Doctrine\Common\Collections\ArrayCollection();
        foreach ($songs as $songArray) {
            $songArray['album'] = $killEmAll;
            $song = $hydrator->hydrate($songArray, new \Album\Entity\Song());
            $songsCollection->add($song);
        }
        $killEmAll->addSongs($songsCollection);
 
        $this->getEntityManager()->persist($killEmAll);
        $this->getEntityManager()->flush();
 
        return new ViewModel();
    }
[PHP] pobierz, plaintext 

Chyba, że ktoś jednak wie jak to ominąć. Wygląda na to, że Doctrine nie radzi sobie ze złożonymi kluczami obcymi.