Разобраться со вставкой данных в Doctrine?

Question

Никита Гусаков @hell0w0rd

Просто разработчик

Разобраться со вставкой данных в Doctrine?

Вот часть конфига, где описанны связи:

Dog:
    oneToMany:
        dogs:
            targetEntity: Dog
            mappedBy: post
            cascade:
                - persist
Post:
    manyToOne:
        post:
            targetEntity: Post
            inversedBy: dogs
        joinColumns:
            post_id:
                referencedColumnName: id

А вот часть контроллера:

$post = new Entity\Post();<br>
$form = $this->createForm(new Form\PostType(), $post);<br>
if( $request->isMethod('POST') && $form->bind($request)->isValid() ) {<br>
    $post = $form->getData()->setAddDate();<br>
    $em = $this->getDoctrine()->getManager();<br>
    $em->persist($post);<br>
    $em->flush();<br><br>
//            return $this->redirect($this->generateUrl('show', array('id' => $post->getId())));<br>
}<br>

Все данные вставляются правильно, за исключением самой связи — почему? В post_id подставляется null.

Вопрос задан более трёх лет назад
4032 просмотра

1 комментарий

Подписаться 3 Оценить 1 комментарий

Решения вопроса 1

9 комментариев

Никита Гусаков @hell0w0rd Автор вопроса

github.com/hell0w0rd/bad-doctrine-entities
Лучше даже так)
все это по пути src/Yorka

Написано более трёх лет назад
Алексей Романенко @slimus

Отлично. У вас используется type=collection. На сколько я помню — в таком случае вам нужно самому делать обход и сохранять в базу. Давно не работал с коллекциями…

Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса

Гм, в чем смысл тогда ORM-то?) Я и достаю, чтоб не в два запроса — сам. И записываю тоже сам. Странно как-то, коллекции — всего лишь форма ввода данных, если бы я их ввел ручками — было бы лучше?
Можно в принципе попробовать, просто это муторно…

Написано более трёх лет назад
Алексей Романенко @slimus

хм. наверное я вас обманул. перечитайте еще раз раздел symfony.com/doc/current/cookbook/form/form_collections.html

Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса

Эх, в трекере doctine посоветовали проверить все тоже самое без сторонних компонентов, тк они за них не отвечают. Напишу в трекер сифони — скажут проблема доктрины… Печаль

Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса
Так, проблема решилась, но на мой взгляд это какой-то костыль:

$post = $form->getData()->setAddDate(); $em = $this->getDoctrine()->getManager(); $em->persist($post); foreach($post->getDogs() as $dog) { $dog->setPost($post); } $em->flush();
Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса

Это не просто костыль — это какой-то вынос мозга. Я из объекта вытаскиваю его дочерние объекты и в них же записываю что они являются дочерними

Написано более трёх лет назад
Сергей Протько @Fesor

Это проблема ни симфони, и ни доктрины, если вы понимаете о чем я. С другой стороны вам нужно просто связанный тип данных. Если у вас связи проставлены верно, то они должны подхватиться и забить все сами. Скорее всего вам опять стоит проверить связи. И желательно предоставить информацию о внешних ключах у этих двух таблиц. По ним можно точно судить что да где не так пошло.

Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса

Почти, но не совсем так.
Нужно было добавить этот код в addDog(), то есть не конструктор, а именно добавление)

Написано более трёх лет назад

Пригласить эксперта

Ответы на вопрос 1

10 комментариев

Вячеслав Слинько @KeepYourMind

Явно не верный targetEntity.

Написано более трёх лет назад
Алексей Романенко @slimus

У вас связи перепутаны.
Прочитайте: docs.doctrine-project.org/en/latest/reference/association-mapping.html

Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса

Все по документации как раз:
docs.doctrine-project.org/en/latest/reference/association-mapping.html#one-to-many-bidirectional
Потому и не понимаю что не так я делаю?

Написано более трёх лет назад

Алексей Романенко @slimus

Product:
  type: entity
  oneToMany:
    features:
      targetEntity: Feature
      mappedBy: product
Feature:
  type: entity
  manyToOne:
    product:
      targetEntity: Product
      inversedBy: features
      joinColumn:
        name: product_id
        referencedColumnName: id

Написано более трёх лет назад

Никита Гусаков @hell0w0rd Автор вопроса

Да, это отличие я заметил, но такая схема отказывается создаваться в базе:

$ ./app/console doctrine:schema:create

ATTENTION: This operation should not be executed in a production environment.

Creating database schema...


                                                                                                                                                                                    
  [Doctrine\ORM\Tools\ToolsException]                                                                                                                                               
  Schema-Tool failed with Error 'An exception occurred while executing 'CREATE TABLE dogs (id INT AUTO_INCREMENT NOT NULL, post_id: INT DEFAULT NULL, farm VARCHAR(50) NOT NULL, b  
  irthday DATE NOT NULL, sex TINYINT(1) NOT NULL, father VARCHAR(255) NOT NULL, mother VARCHAR(255) NOT NULL, weight DOUBLE PRECISION NOT NULL, INDEX IDX_353BEEB3269DD3DF (post_i  
  d:), PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB':                                                                                        
                                                                                                                                                                                    
  SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right synta  
  x to use near ': INT DEFAULT NULL, farm VARCHAR(50) NOT NULL, birthday DATE NOT NULL, sex TINYI' at line 1' while executing DDL: CREATE TABLE dogs (id INT AUTO_INCREMENT NOT NU  
  LL, post_id: INT DEFAULT NULL, farm VARCHAR(50) NOT NULL, birthday DATE NOT NULL, sex TINYINT(1) NOT NULL, father VARCHAR(255) NOT NULL, mother VARCHAR(255) NOT NULL, weight DO  
  UBLE PRECISION NOT NULL, INDEX IDX_353BEEB3269DD3DF (post_id:), PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB                               
                                                                                                                                                                                    




                                                                                                                                                                                    
  [Doctrine\DBAL\DBALException]                                                                                                                                                     
  An exception occurred while executing 'CREATE TABLE dogs (id INT AUTO_INCREMENT NOT NULL, post_id: INT DEFAULT NULL, farm VARCHAR(50) NOT NULL, birthday DATE NOT NULL, sex TINY  
  INT(1) NOT NULL, father VARCHAR(255) NOT NULL, mother VARCHAR(255) NOT NULL, weight DOUBLE PRECISION NOT NULL, INDEX IDX_353BEEB3269DD3DF (post_id:), PRIMARY KEY(id)) DEFAULT C  
  HARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB':                                                                                                                       
                                                                                                                                                                                    
  SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right synta  
  x to use near ': INT DEFAULT NULL, farm VARCHAR(50) NOT NULL, birthday DATE NOT NULL, sex TINYI' at line 1                                                                        
                                                                                                                                                                                    




                                                                                                                                                                                    
  [PDOException]                                                                                                                                                                    
  SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right synta  
  x to use near ': INT DEFAULT NULL, farm VARCHAR(50) NOT NULL, birthday DATE NOT NULL, sex TINYI' at line 1

Написано более трёх лет назад

Алексей Романенко @slimus

у вас откудато лишний знак двоеточия в имени переменной поста
joinColumn:
name: product_id
referencedColumnName: id

Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса

Глаз-алмаз. Да, однако это ни на что не повлияло, также подставляется null
По сути моя начальная запись позволяла просто добавить несколько связей, и ничем не отличалась.

Написано более трёх лет назад
Алексей Романенко @slimus

а что такое в контроллере "$post = $form->getData()->setAddDate();"? если убрать строку что будет?

Написано более трёх лет назад
Никита Гусаков @hell0w0rd Автор вопроса
Без доп-расширений невозможно установить TIMESTAMP в время, или тупо функцию NOW(), поэтому вот функция:

public function setAddDate($addDate = 'now') { $this->addDate = new \DateTime($addDate); return $this; }

Она явно не на что не влияет. По идее Doctrine должна тащить из базы последний id и его подставлять, но она этого не делает
Написано более трёх лет назад
Сергей Протько @Fesor

Такие вещи (установка значений по умолчанию) должны происходить в конструкторе.

Написано более трёх лет назад