I have a symfony app with a Comment entity, and I would like to link it, with a relation property, to many entities. But by many entities, I don't mean many objects, but many classes.
A comment can be about different things in my app : a Drawing entity, a Texte entity, a Compo entity, or a Photo entity. Each entity correspond to a type of artwork, each one with differents properties and different pages. And each one can be rated and commented.
The problem is : when I wanna create the relation property in my Comment entity, I got to indicate just ONE entity. I want that some comments are about a drawing entity, some are about text etc.
I see a solution : create one entity comment by entity to be link, but my app would have very many entities, and my code would be so duplicated, which is not so good for future changes etc.
Is there a way to link one entity to many differents types of entity ?
Thank you.
EDIT 4 : ALL PREVIOUS EDITS BELOW ARE POINTLESS !
I let it so beginners can see some of my code, to have an inheritance mapping example detailed (I which I could have it when I tried first, I didn't understood everything) but I just realised the stupid mistake I made :
* #DiscriminatorMap({"comment" = "Comment", "comment" = "CommentDrawing", "commentphoto" = "CommentPhoto", "commenttexte" = "CommentTexte"})
I forgot "commentcompo" = "CommentCompo"
I wrote "comment" = "Comment" AND "comment" = "CommentDrawing" (both referenced by "comment")
That's why I had to put "comment" as discriminator instead of "commentdrawing".
Sorry for that stupid mistake, So much code lines I didn't notice it.
And for those who also use inheritance mapping for first time and want informations, I'm now trying to fill comments with forms. If you want to do the same (you probably will if you came here for an answer), I think this is the solution :
https://symfony.com/doc/current/form/inherit_data_option.html
EDIT :
Thanks to the answer of Jakumi below, I used inheritance mapping. I chosen the first solution : the one table solution. But I tried, I have no error message, but this didn't work to me, and after hours I still dont understand what I did wrong.
My Comment class (the topmost class of my comments hierarchy, which has CommentDrawing, CommentCompo, CommentPhoto, CommentTexte as children) (Comment is abstract because I had an error telling me that indicated me to put it abstract to solve the error) :
<?php
namespace App\Entity;
use App\Entity\Comment;
use App\Entity\CommentPhoto;
use App\Entity\CommentTexte;
use App\Entity\CommentDrawing;
use Doctrine\ORM\Mapping as ORM;
use Doctrine\ORM\Mapping\InheritanceType;
use Doctrine\ORM\Mapping\DiscriminatorMap;
use Doctrine\ORM\Mapping\DiscriminatorColumn;
/**
* #ORM\Entity(repositoryClass="App\Repository\CommentRepository")
* #InheritanceType("SINGLE_TABLE")
* #DiscriminatorColumn(name="discriminator", type="string")
* #DiscriminatorMap({"comment" = "Comment", "comment" = "CommentDrawing", "commentphoto" = "CommentPhoto", "commenttexte" = "CommentTexte"})
*/
abstract class Comment
{
/**
* #ORM\Id()
* #ORM\GeneratedValue()
* #ORM\Column(type="integer")
*/
private $id;
/**
* #ORM\Column(type="datetime")
*/
private $createdAt;
/**
* #ORM\Column(type="text")
*/
private $content;
/**
* #ORM\ManyToOne(targetEntity="App\Entity\User", inversedBy="comments")
* #ORM\JoinColumn(nullable=false)
*/
private $author;
public function getId(): ?int
{
return $this->id;
}
public function getCreatedAt(): ?\DateTimeInterface
{
return $this->createdAt;
}
public function setCreatedAt(\DateTimeInterface $createdAt): self
{
$this->createdAt = $createdAt;
return $this;
}
public function getContent(): ?string
{
return $this->content;
}
public function setContent(string $content): self
{
$this->content = $content;
return $this;
}
public function getAuthor(): ?User
{
return $this->author;
}
public function setAuthor(?User $author): self
{
$this->author = $author;
return $this;
}
}
And here is my CommentDrawing entity (I created the other CommentSomething classes but the only one I tried to use now is CommentDrawing) :
<?php
namespace App\Entity;
use App\Entity\Comment;
use Doctrine\ORM\Mapping as ORM;
/**
* #ORM\Entity(repositoryClass="App\Repository\CommentDrawingRepository")
*/
class CommentDrawing extends Comment
{
/**
* #ORM\Id()
* #ORM\GeneratedValue()
* #ORM\Column(type="integer")
*/
private $id;
/**
* #ORM\ManyToOne(targetEntity="App\Entity\Dessin", inversedBy="commentsDrawing")
*/
private $drawing;
public function getId(): ?int
{
return $this->id;
}
public function getDrawing(): ?Dessin
{
return $this->drawing;
}
public function setDrawing(?Dessin $drawing): self
{
$this->drawing = $drawing;
return $this;
}
}
And here is the Dessin entity it refers to (dessin is drawing in french, I forgot to name it with english name when I created it) this is not a child class of Comment, this is the subject of the CommentDrawing (linked by ManyToOne), which is itself the child class of Comment :
<?php
namespace App\Entity;
use Cocur\Slugify\Slugify;
use Doctrine\ORM\Mapping as ORM;
use Doctrine\Common\Collections\Collection;
use Doctrine\Common\Collections\ArrayCollection;
use Symfony\Component\Validator\Constraints as Assert;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
/**
* #ORM\Entity(repositoryClass="App\Repository\DessinRepository")
* #ORM\HasLifecycleCallbacks
* #UniqueEntity(
* fields = {"nom"},
* message = "Un autre dessin contient le même nom. Merci de le changer. Vérifiez aussi que vous avez entré un slug unique, ou vide. Si le slug est en double, ça engendrera un bug.")
* #UniqueEntity(
* fields = {"url"},
* message = "Un autre dessin contient la même url, vous vous apprêtez à poster deux fois le même dessin. Merci de changer l'url.")
* )
*/
class Dessin
{
/**
* #ORM\Id()
* #ORM\GeneratedValue()
* #ORM\Column(type="integer")
*/
private $id;
// Some code I hidden because it's useless to show
/**
* #ORM\Column(type="string", length=255)
*/
private $slug;
// Some code I hidden because it's useless to show
/**
* #ORM\OneToMany(targetEntity="App\Entity\CommentDrawing", mappedBy="drawing")
*/
private $commentsDrawing;
public function __construct()
{
// Some code I hidden
$this->commentsDrawing = new ArrayCollection();
}
// Some code I hidden
public function getId(): ?int
{
return $this->id;
}
// Some code I hidden
public function getSlug(): ?string
{
return $this->slug;
}
public function setSlug(string $slug): self
{
$this->slug = $slug;
return $this;
}
// Some code I hidden
/**
* #return Collection|CommentDrawing[]
*/
public function getCommentsDrawing(): Collection
{
return $this->commentsDrawing;
}
public function addCommentsDrawing(CommentDrawing $commentsDrawing): self
{
if (!$this->commentsDrawing->contains($commentsDrawing)) {
$this->commentsDrawing[] = $commentsDrawing;
$commentsDrawing->setDrawing($this);
}
return $this;
}
public function removeCommentsDrawing(CommentDrawing $commentsDrawing): self
{
if ($this->commentsDrawing->contains($commentsDrawing)) {
$this->commentsDrawing->removeElement($commentsDrawing);
// set the owning side to null (unless already changed)
if ($commentsDrawing->getDrawing() === $this) {
$commentsDrawing->setDrawing(null);
}
}
return $this;
}
}
To verify if this worked correctly, I manually created a comment in the database with phpmyadmin :
Then I tried to show the content of the drawing comment above in a page, using the drawing var corresponding to the drawing which is the subject of my comment. Nothing happened. So I directly tried to dump it in a Controller, impossible to get the comment.
To get the comment, I used the $commentsDrawing property of Drawing entity (you can see it in the code above). Here is the code I used to dump the drawing var that should contain a comment (I put the dump() function in the show function, that is call with the drawing slug as parameter, founded in the URL. I'm sure the show() function works correctly and show the good drawing, because I tested it before). It is the DrawingController :
<?php
namespace App\Controller;
use App\Entity\Dessin;
use App\Form\DrawingType;
use App\Entity\CategorieDessin;
use App\Service\PaginationService;
use App\Repository\DessinRepository;
use App\Repository\CategorieDessinRepository;
use Symfony\Component\HttpFoundation\Request;
use Doctrine\Common\Persistence\ObjectManager;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Annotation\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
class DrawingsController extends AbstractController
{
// I hide the rest of the code because it's useless
/**
* show a drawing
*
* #Route("/dessin/{slug}", name="drawing_show")
*
* #return Response
*/
public function show(Dessin $drawing)
{
dump($drawing);
die();
return $this->render('drawings/show.html.twig', [
'drawing' => $drawing
]);
}
}
What the dump() shows :
As you can see, in commentsdrawing > collection, no elements.
If I do the same without the dump(), I got the drawing, no error, but no comments neither.
I really don't see what I did wrong... Could someone please help ?
EDIT 2 :
When I do this :
/**
* show a drawing
*
* #Route("/dessin/{slug}", name="drawing_show")
*
* #return Response
*/
public function show(Dessin $drawing, CommentDrawingRepository $repo)
{
$comments = $repo->findAll();
dump($comments);
die();
return $this->render('drawings/show.html.twig', [
'drawing' => $drawing
]);
}
I get an empty array
EDIT 3 :
Well, I added a new CommentDrawing directly from my DrawingController :
/**
* show a drawing
*
* #Route("/dessin/{slug}", name="drawing_show")
*
* #return Response
*/
public function show(Dessin $drawing, CommentDrawingRepository $repo, DessinRepository $repoDrawing, UserRepository $repoUser, ObjectManager $manager)
{
$drawing = $repoDrawing->findAll()[0];
$user = $repoUser->findAll()[0];
$comment = new CommentDrawing();
$comment->setCreatedAt(new \DateTime())
->setContent("un test")
->setAuthor($user)
->setDrawing($drawing);
$manager->persist($comment);
$manager->flush();
$comments = $repo->findAll();
dump($comments);
return $this->render('drawings/show.html.twig', [
'drawing' => $drawing
]);
}
And that worked. The comment is registred in the database, the dump shows a comment, and the comment appear in my page.
I tried to understand why the comment added with phpmyadmin didn't work, and what a surprise : the difference between both comments is that the one added with phpmyadmin has commentdrawing as dicriminator value, and the one added by doctrine has just comment as value... Which is the value for the abstract class ! I thought the discriminator value was usefull to tell doctrine what column to consider, I don't understand anymore... But well, problem solved. Thank you for your help !
note : the second one is the one which worked... Why ?
There is nothing that says you can't have multiple ManyToOne relationships within single entity. So it may not be as elegant and clean as inheritance mapping as suggested by Jakumi, but it works with little effort. I've done this in a project along the lines of what you describe with a single Comment entity that references multiple other entitys at the same time. For my Comment class, I have other entitys (Admission and Referral) that each point to multiple Comments, but Admission and Referral are otherwise nothing alike and so would not make sense for me to have each of them extend some abstract class (as suggested by Arleigh Hix in the comments to your question). My reasons for doing this are not relevant to your project, but it works well for me.
Comment class:
/**
* #ORM\Entity
*/
class Comment
{
/**
* #ORM\Column(type="text")
*/
private $comment;
/**
* #ORM\ManyToOne(
* targetEntity="App\Entity\Referral",
* inversedBy="comments"
* );
*/
private $referral;
/**
* #ORM\ManyToOne(
* targetEntity="App\Entity\Admission",
* inversedBy="comments"
* );
*/
private $admission;
// getters and setters omitted
}
Then of course my Admission entity (and very similarly my Referral entity) have the other side of the relationship:
/**
* #ORM\Entity
*/
class Admission
{
public function __construct()
{
$this->comments = new ArrayCollection();
}
/**
* #ORM\OneToMany(targetEntity="Comment",
* mappedBy="admission",
* )
*/
private $comments;
}
the association itself is most likely the wrong place to do this abstraction. Anyway, what you're probably somewhat looking for is inheritance mapping.
For reference:
https://www.doctrine-project.org/projects/doctrine-orm/en/2.6/reference/inheritance-mapping.html
one table solution:
One option would be, to do single table inheritance. Parent class gets a method "getSubject()" that the child classes override and holds userid + rating + comment text. Child classes are DrawingComment etc. and each of them has a one-to-one association to the comment's subject.
Advantages: simple, clear semantics, can distinguish by classname (or extra function/property) if need be, referential integrity is stable, somewhat easy to do stats over, searchable regardless of comment subject type
Disadvantages: discriminator-column is somewhat awkward to use and slightly reduces speed.
many tables solution:
Leave inheritance mapping, make abstract class (+ interface?) or trait (+interface) for the common code. store each comment type in its own class/entity/table. When you don't care about the type of a comment, you can still use interface functions to handle it cleanly.
advantages: same, as before - except for searchability, no discriminator column, since each comment type has its own table
disadvantages: searchability is slightly reduced
Ultimately it's a question of taste or habit. And it depends on the common and distinct fields of different comment types. if - apart from the subject - they all share the same fields, I would do single table inheritance. the more they differ, the more I'd tend towards the many tables solution.
I am using Symfony3 with doctrine and my question is:
Is possible after getQuery in query builder get results with attribute which has formatted date?
I have attr in entity:
/**
* #var \DateTime
*
* #ORM\Column(name="terminFrom", type="date", nullable=true)
*/
private $terminFrom;
My try - set format in getter:
/**
* #return string
*/
public function getTerminFrom()
{
return $this->terminFrom->format('d.m.Y');
}
But still after getQuery i have \DateTime object in this attribute.
I don't understand - attributes in entity are private so getters must be called... or?
I have a sql error deploying my symfony project on Unix. Indeed, the query made uses uppercase for table names which was not the case before. In the database, table names are lowercases.
Does anybody know where you configure how you want the queries generation be made (uppercase or lowercase)
Thank you.
Maybe something in Doctrine configuration??
NOTE : I have some new information.
I asked to rebuild my database from the save at a time I know it was working.
I have some errors because, for the new code, the database must be a little different than what it was at this time but I can see that in queries the names of the tables are in lower case.
I pass my new sql (the same as before plus some little changes) in command line.
\. path/to/my/sql
I launch the site and queries are made with table names in uppercase.
Got any idea ?
example of entity :
<?php
namespace MyBundle\Entity;
use Doctrine\ORM\Mapping as ORM;
/**
* MyTable
*
* #ORM\Table(name="mytable")
* #ORM\Entity(repositoryClass="MyBundle\Repository\MyClassRepository")
*/
class MyTable
{
/**
* #var string
*
* #ORM\Column(name="FIELD1, type="string", length=120, nullable=false)
*/
private $field1;
/**
* #var string
*
* #ORM\Column(name="FIELD2", type="string", length=50, nullable=true)
*/
private $field2
/**
* Set field1
*
* #param string $field1
*
* #return Tretb
*/
public function setField1($field1)
{
$this->field1 = $field1;
return $this;
}
/**
* Get field1
*
* #return string
*/
public function getField1()
{
return $this->field1;
}
/**
* Set field2
*
* #param string $field2
*
* #return Tretb
*/
public function setField2($field2)
{
$this->field2 = $field2;
return $this;
}
/**
* Get field2
*
* #return string
*/
public function getField2()
{
return $this->field2;
}
}
Dont know if you can configure it more generally, but you can define table table for each entity with doctrine
/**
* AppBundle\Entity\MyEntity
*
* #ORM\Table()
* #ORM\Entity
* #ORM\Table(name="mytable")
*/
class MyEntity
{
Does your MySQL instance support lowercase tablenames ?
To check the settings use:
mysql> show variables like "lower_case%";
To change the setting you need to change the mysql setting
lower_case_table_names = 1 or lower_case_table_names = 2
Source from Stackoverflow
In my Symfony2 project I have two related entities: Users and Favorites. They have a many-to-many relationship.
My application works as follows:
In my Twig-page I have an few items with a button 'Add to Favorites'. When you click the button my controller saves the item_id in the Favorites column. But then I want to save
the user who added the item to his favorites and here my application fails.
The User and the favorite exist but the joincolumn between Users and Favorites remains empty.
I also don't receive any kind of errors.
Here is my code:
Entity Users
class Users implements AdvancedUserInterface
{
/**
* #var \Doctrine\Common\Collections\ArrayCollection
*
* #ORM\ManyToMany(targetEntity="Favorites", inversedBy="user", cascade={"persist"})
* #ORM\JoinTable(name="user_has_favorite",
* joinColumns={
* #ORM\JoinColumn(name="user_id", referencedColumnName="user_id")
* },
* inverseJoinColumns={
* #ORM\JoinColumn(name="favorite_id", referencedColumnName="favorite_id")
* }
* )
*/
private $favorite;
public function __construct()
{
$this->favorite = new \Doctrine\Common\Collections\ArrayCollection();
}
public function addFavorite(\Geo\CityTroopersBundle\Entity\Favorites $favorite)
{
$this->favorite[] = $favorite;
return $this;
}
...
Entity Favorites
class Favorites
{
/**
* #var \Doctrine\Common\Collections\ArrayCollection
*
* #ORM\ManyToMany(targetEntity="Users", mappedBy="favorite", cascade={"persist"})
*/
private $user;
public function __construct()
{
$this->user = new \Doctrine\Common\Collections\ArrayCollection();
}
public function addUser(\Geo\CityTroopersBundle\Entity\Users $user)
{
$this->user[] = $user;
return $this;
}
My controller
public function showNewsAction()
{
$request = $this->get('request');
$itemId=$request->request->get('itemId');
if($itemId != NULL)
{
//MAKE NEW FAVORITE AND ADD TO DATABASE LINKED WITH ITEM
$favorite = new Favorites();
$favorite->setItemId($itemId);
//LINK FAVORITE ID WITH USER ID IN JOINCOLUMN
$userId = 6;
$em = $this->getDoctrine()->getEntityManager();
$user = $em->getRepository('GeoCityTroopersBundle:Users')->find($userId);
$favorite->addUser($user);
$em->persist($favorite);
//I TRIED THIS TOO, BUT IT FAILED
/*$user->addFavorite($favorite);
$em->persist($user);*/
$em->flush();
You were close there. For doctrine many-to-many relationships, you need to call both add methods
$favorite->addUser($user);
$user->addFavorite($favorite);
$em->persist($favorite);
$em->persist($user);
$em->flush();
This should do the trick. In the docs they do this, but don't mention it too explicitly. Not sure why either because lots of people run into this (myself included).
As explained here, only the owning side is responsible for the connection management :
http://doctrine-orm.readthedocs.org/en/latest/reference/association-mapping.html#owning-and-inverse-side-on-a-manytomany-association
So only
$user->addFavorite($favorite);
should persist, and not
$favorite->addUser($user);
Like indicated by Cedric, adding a record for a many to many relation is done only in one direction and it depends on how you defined the relation: adding can be done only by the parent entity of the relation, so in your case you must use:
$user->addFavorite($favorite);
In your line :
#ORM\JoinColumn(name="favorite_id", referencedColumnName="favorite_id")
The
name="favorite_id"
part refers to the columns in the join table, whereas the
referencedColumnName="favorite_id"
refers to the id in the favorite table which usualy is simply "id". You should try :
/**
* #var \Doctrine\Common\Collections\ArrayCollection
*
* #ORM\ManyToMany(targetEntity="Favorites", inversedBy="user", cascade={"persist"})
* #ORM\JoinTable(name="user_has_favorite",
* joinColumns={
* #ORM\JoinColumn(name="user_id", referencedColumnName="id")
* },
* inverseJoinColumns={
* #ORM\JoinColumn(name="favorite_id", referencedColumnName="id")
* }
* )
*/
private $favorite;
I am trying to serialize entities for mobile digest. I have this Entity class:
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\ORM\Mapping as ORM;
use FOS\UserBundle\Entity\User as BaseUser;
/**
* xxx\xxx\Entity\User
*
* #ORM\Table()
* #ORM\Entity()
* #ORM\Entity(repositoryClass="xxx\xxx\Entity\UserRepository")
*/
class User extends BaseUser
{
/**
*
* #ORM\Id
* #ORM\Column(type="integer")
* #ORM\GeneratedValue(strategy="AUTO")
*/
protected $id;
/**
* #var \Doctrine\Common\Collections\ArrayCollection
*
* #ORM\OneToMany(targetEntity="\xxx\xxx\Entity\Music", mappedBy="user")
*/
protected $musics;
/**
* #var \Doctrine\Common\Collections\ArrayCollection
*
* #ORM\OneToMany(targetEntity="\xxx\xxx\Entity\Message", mappedBy="user")
*/
protected $messages;
/**
* #var \Doctrine\Common\Collections\ArrayCollection
*
* #ORM\OneToMany(targetEntity="\xxx\xxx\Entity\Location", mappedBy="user")
*/
protected $locations;
public function __construct()
{
parent::__construct();
$this->musics = new ArrayCollection();
$this->messages = new ArrayCollection();
$this->locations = new ArrayCollection();
}
}
Now when I call this line in my DefaultController.php:
$user = $this->getUser();
$em = $this->getDoctrine()->getManager();
$array = $em->getRepository('xxxBundle:User')
->findLatest();
$serializer = $this->get('serializer');
$response = $serializer->serialize($array, 'json'); //THIS LINE THROWS EXCEPTION
I have use Doctrine\Common\Collections\ArrayCollection; in DefaultController.php, but it seems the error is coming from inside JMSSerializerBundle.
What have I tried thusfar
I have tried defining the Doctrine annotations to start with a \, but that didn't help
I have cleared my cache a million times
I have searched for similar exceptions, but they all seem to be caused by a typo and I've checked for typos for the last 48 hours and I can't find one.
The classes were autogenerated with app/console.
Take a look at this issue on GitHub: https://github.com/schmittjoh/JMSSerializerBundle/issues/123
This solution works!
I am using JMSSerializerBundle and in Serialized Entity i have ManyToOne relation. I used property $product and of course setter and getter for that. If serializer try to get Product I got this same message i thnk because it don't understand how to convert related Entity to string/int. I adding Accessor with custom method getProductId and inside this method return
$this->product->getId()
JMS\Serializer\Annotation as Serializer
#Serializer\Accessor(getter="getProductId")
In OneToMany relation, in custom get method I return ArrayCollection as Array $this->statuses->toArray()
You can also think of coonverter for relation entity, but I haven't tried (no time)