Chill-project/chill-bundles
6
0
Fork 0
mirror of https://gitlab.com/Chill-Projet/chill-bundles.git synced 2025-07-01 06:26:15 +00:00
Code Issues Actions Projects Releases Wiki Activity

Use injected EntityManager - fix property visibility.

Browse Source
This commit is contained in:
Pol Dellaiera 2021-04-27 23:07:59 +02:00
parent 49c6915c0a
commit c5250a1059
1 changed files with 7 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/Bundle/ChillPersonBundle/Controller/PersonController.php
View File

@ -49,13 +49,11 @@ use Doctrine\ORM\EntityManagerInterface;
class PersonController extends AbstractController class PersonController extends AbstractController
{ {
/** /**
*
* @var SimilarPersonMatcher * @var SimilarPersonMatcher
*/ */
protected $similarPersonMatcher; protected $similarPersonMatcher;
/** /**
*
* @var TranslatorInterface * @var TranslatorInterface
*/ */
protected $translator; protected $translator;
@ -66,34 +64,31 @@ class PersonController extends AbstractController
protected $eventDispatcher; protected $eventDispatcher;
/** /**
*
* @var PersonRepository; * @var PersonRepository;
*/ */
protected $personRepository; protected $personRepository;
/** /**
*
* @var ConfigPersonAltNamesHelper * @var ConfigPersonAltNamesHelper
*/ */
protected $configPersonAltNameHelper; protected $configPersonAltNameHelper;
/** /**
*
* @var EntityManagerInterface * @var EntityManagerInterface
*/ */
protected $em; private $em;
/** /**
* @var \Psr\Log\LoggerInterface * @var \Psr\Log\LoggerInterface
*/ */
private $logger; private $logger;
/** /**
* @var ValidatorInterface * @var ValidatorInterface
*/ */
private $validator; private $validator;
public function __construct( public function __construct(
SimilarPersonMatcher $similarPersonMatcher, SimilarPersonMatcher $similarPersonMatcher,
TranslatorInterface $translator, TranslatorInterface $translator,
EventDispatcherInterface $eventDispatcher, EventDispatcherInterface $eventDispatcher,
@ -117,8 +112,7 @@ class PersonController extends AbstractController
{ {
$cFGroup = null; $cFGroup = null;
$em = $this->getDoctrine()->getManager(); $cFDefaultGroup = $this->em->getRepository("ChillCustomFieldsBundle:CustomFieldsDefaultGroup")
$cFDefaultGroup = $em->getRepository("ChillCustomFieldsBundle:CustomFieldsDefaultGroup")
->findOneByEntity("Chill\PersonBundle\Entity\Person"); ->findOneByEntity("Chill\PersonBundle\Entity\Person");
if($cFDefaultGroup) { if($cFDefaultGroup) {
@ -207,8 +201,7 @@ class PersonController extends AbstractController
->trans('The person data has been updated') ->trans('The person data has been updated')
); );
$em = $this->getDoctrine()->getManager(); $this->em->flush();
$em->flush();
$url = $this->generateUrl('chill_person_view', array( $url = $this->generateUrl('chill_person_view', array(
'person_id' => $person->getId() 'person_id' => $person->getId()
@ -402,11 +395,9 @@ class PersonController extends AbstractController
'You are not allowed to create this person'); 'You are not allowed to create this person');
if ($errors->count() === 0) { if ($errors->count() === 0) {
$em = $this->getDoctrine()->getManager(); $this->em->persist($person);
$em->persist($person); $this->em->flush();
$em->flush();
return $this->redirect($this->generateUrl('chill_person_general_edit', return $this->redirect($this->generateUrl('chill_person_general_edit',
array('person_id' => $person->getId()))); array('person_id' => $person->getId())));