Symfony ManyToMany no need to persist? - symfony

Question

i have 2 entities:
Domain:
class Domain
{
#[ORM\Id]
#[ORM\GeneratedValue]
#[ORM\Column]
private ?int $id = null;
#[ORM\Column(length: 191, unique: true)]
#[Assert\NotBlank(message: 'domain.name.not_blank')]
private ?string $name = null;
#[ORM\ManyToMany(targetEntity: Bill::class, inversedBy: 'domains', cascade: ['persist', 'remove'])]
private Collection $bill;
public function __construct()
{
$this->bill = new ArrayCollection();
}
public function __toString()
{
return $this->name;
}
/**
* #return Collection<int, Bill>
*/
public function getBill(): Collection
{
return $this->bill;
}
public function addBill(Bill $bill): self
{
if (!$this->bill->contains($bill)) {
$this->bill->add($bill);
}
return $this;
}
public function removeBill(Bill $bill): self
{
$this->bill->removeElement($bill);
return $this;
}
}
And Bill:
class Bill
{
#[ORM\Id]
#[ORM\GeneratedValue]
#[ORM\Column]
private ?int $id = null;
#[ORM\Column(length: 255)]
private ?string $number = null;
#[ORM\ManyToMany(targetEntity: Domain::class, mappedBy: 'bill')]
private Collection $domains;
public function __construct()
{
$this->domains = new ArrayCollection();
}
public function __toString()
{
return $this->number;
}
public function getId(): ?int
{
return $this->id;
}
public function getNumber(): ?string
{
return $this->number;
}
public function setNumber(string $number): self
{
$this->number = $number;
return $this;
}
/**
* #return Collection<int, Domain>
*/
public function getDomains(): Collection
{
return $this->domains;
}
public function addDomain(Domain $domain): self
{
if (!$this->domains->contains($domain)) {
$this->domains->add($domain);
$domain->addBill($this);
}
return $this;
}
public function removeDomain(Domain $domain): self
{
if ($this->domains->removeElement($domain)) {
$domain->removeBill($this);
}
return $this;
}
}
BillType form:
class BillingType extends AbstractType
{
public function buildForm(FormBuilderInterface $builder, array $options): void
{
$builder
->add('number', TextType::class, [
'attr' => ['placeholder' => 'Numero fattura',]
])
->add('domains', EntityType::class, [
'class' => Domain::class,
'expanded' => true,
'multiple' => true,
'choice_label' => 'name'
])
->add('submit', SubmitType::class, [
'attr' => [
'class' => 'btn-success',
]
]);
}
public function configureOptions(OptionsResolver $resolver): void
{
$resolver->setDefaults([
'data_class' => Bill::class,
]);
}
}
I'm trying to set the bill for multiple domains at the same time, so my controller look like this:
public function domainBilling(Request $request, ManagerRegistry $doctrine, PaginatorInterface $paginator)
{
$filters = $this->getFilters($request)['filters'];
$form = $this->getFilters($request)['form'];
$dbquery = $doctrine
->getRepository(Domain::class)
->findAllPaginated($filters)
->getQuery();
$domains = $paginator->paginate(
$dbquery, /* query NOT result */
$request->query->getInt('page', 1), /*page number*/
Domain::perPage, /*limit per page*/
[
'defaultSortFieldName' => 'd.name',
'defaultSortDirection' => 'asc',
]
);
$billing_form = $this->createForm(BillingType::class);
$billing_form->handleRequest($request);
if ($billing_form->isSubmitted() && $billing_form->isValid()) {
$em = $doctrine->getManager();
$domains = $billing_form['domains']->getData();
$bill_number = $billing_form['number']->getData();
$bill = new Bill();
$bill->setNumber($bill_number);
$em->flush();
$this->addFlash(
'success',
'Le modifiche sono state effettuate!'
);
return $this->redirectToRoute('domain_dashboard');
}
return $this->render('Admin/domain_dashboard.html.twig', [
'billing_form' => $billing_form->createView(),
'form' => $form->createView(),
'domains' => $domains
]);
}
And it's working without calling $em->persist(); on bill or domain and without calling foreach to save every domain. If i persist one of them i find duplicate entry for bill inside db. Am i doing something wrong?