DDC-36: EntityManager#persist should accept more objects to easily persist multiple objects. #45

New Issue

admin · 2026-01-22T12:25:13+01:00

admin commented

2026-01-22 12:25:13 +01:00

Originally created by @doctrinebot on GitHub (Oct 8, 2009).

Jira issue originally created by user marijn:

I think it would be nice if the persist function would accept more objects as an arguments by using func-get-arg see the simple example below:

\


<?php

  $user   = new \Entities\User();
  $person = new \Entities\Person();

  $person->setName('Wage');
  $person->setFirstNames('Jonathan');
  $person->setTitle('mr');

  $user->setPerson($person);
  $user->setPassword('D0ctr1ne');

  $em->persist($person, $user);
  $em->flush();

Originally created by @doctrinebot on GitHub (Oct 8, 2009). Jira issue originally created by user marijn: I think it would be nice if the persist function would accept more objects as an arguments by using `func-get-arg` see the simple example below: \ \ ``` <?php $user = new \Entities\User(); $person = new \Entities\Person(); $person->setName('Wage'); $person->setFirstNames('Jonathan'); $person->setTitle('mr'); $user->setPerson($person); $user->setPassword('D0ctr1ne'); $em->persist($person, $user); $em->flush(); ```

admin added the New Feature label 2026-01-22 12:25:13 +01:00

admin closed this issue

2026-01-22 12:25:15 +01:00

admin commented

2026-01-22 12:25:15 +01:00

@doctrinebot commented on GitHub (Oct 8, 2009):

Comment created by romanb:

I really dislike func_get_args in general because it makes parameters invisible on the public API without explicit correcting documentation (and grabbing parameters out of thin air, which is what func_get_args() does, doesnt look very appealing in the code, too).

Combined with the fact that this would really be only a very small convenience, I have to say no on this one. Sorry, but thanks for the suggestion.

What would probably be acceptable is: EntityManager#persistAll(array $entities) but then we would need such a method for remove/merge/detach as well. Sounds like too many new methods for me that will be rather rarely used and we dont want to bloat the public API like that. And it really is a one-liner in userland code: foreach ($entities as $entity) $em->persist($entity);

@doctrinebot commented on GitHub (Oct 8, 2009): Comment created by romanb: I really dislike func_get_args in general because it makes parameters invisible on the public API without explicit correcting documentation (and grabbing parameters out of thin air, which is what func_get_args() does, doesnt look very appealing in the code, too). Combined with the fact that this would really be only a very small convenience, I have to say no on this one. Sorry, but thanks for the suggestion. What would probably be acceptable is: EntityManager#persistAll(array $entities) but then we would need such a method for remove/merge/detach as well. Sounds like too many new methods for me that will be rather rarely used and we dont want to bloat the public API like that. And it really is a one-liner in userland code: `foreach ($entities as $entity) $em->persist($entity);`

admin commented

2026-01-22 12:25:16 +01:00

@doctrinebot commented on GitHub (Oct 8, 2009):

Issue was closed with resolution "Won't Fix"

@doctrinebot commented on GitHub (Oct 8, 2009): Issue was closed with resolution "Won't Fix"

admin commented

2026-01-22 12:25:16 +01:00

@doctrinebot commented on GitHub (Oct 8, 2009):

Comment created by marijn:

Hi Roman,

I agree with you in keeping the API as clean as possible, its one of the most important features is you ask me from doctrine 2, that's why I'll have to agree with you.

Thanks for concidering.

@doctrinebot commented on GitHub (Oct 8, 2009): Comment created by marijn: Hi Roman, I agree with you in keeping the API as clean as possible, its one of the most important features is you ask me from doctrine 2, that's why I'll have to agree with you. Thanks for concidering.

admin referenced this issue

2026-01-22 15:56:05 +01:00

[PR #45] [CLOSED] [DDC-1104] Require statement in AnnotationDriver.php can cause PHP Fatal error #7635

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#45