Symfony UniqueEntity在更新现有实体时显示错误

问题描述 投票:1回答:3

我有一个非常简单的实体,具有UniqueEntity验证:

namespace App\Entity;

use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Security\Core\User\UserInterface;
use Doctrine\ORM\Mapping\ManyToOne;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
use Doctrine\ORM\Mapping\HasLifecycleCallbacks;

/**
 * @ORM\Entity(repositoryClass="App\Repository\UserRepository")
 * @UniqueEntity("email", message="Email already in use")
 * 
 *
 */
class User implements UserInterface
{
    /**
     * @ORM\Id()
     * @ORM\GeneratedValue()
     * @ORM\Column(type="integer")
     */
    private $id;

    /**
     * @ORM\Column(type="string", length=180, unique=true)
     */
    private $email;

表单(故意删除了所有其他表单字段)

namespace App\Form;

use App\Entity\Company;
use App\Entity\User;
use Symfony\Bridge\Doctrine\Form\Type\EntityType;
use Symfony\Component\Form\Extension\Core\Type\EmailType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\CallbackTransformer;
use Symfony\Component\Form\ChoiceList\Loader\CallbackChoiceLoader;
use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
use Symfony\Component\OptionsResolver\OptionsResolver;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;


class UserType extends AbstractType
{
    /**
     * {@inheritdoc}
     */
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add( 'email', EmailType::class,[
                'label' => 'Email'
            ] );


        $builder->add( 'save', SubmitType::class, [
            'attr' => [
                'class' => 'btn btn-primary',
                'id' => 'btn-user-form'
            ],
            'label' => 'Save'
        ] );

    }

    /**
     * {@inheritdoc}
     */
    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults( [
            'data_class' => 'App\Entity\User'
        ]);

    }

    /**
     * {@inheritdoc}
     */
    public function getBlockPrefix()
    {
        return 'user';
    }
}

和控制器:

public function edit(User $user, RequestStack $request)
{
    $em = $this->getDoctrine()->getManager();
    $form = $this->createForm( UserType::class, $user);

    $form->handleRequest($request->getCurrentRequest());

    if ($form->isSubmitted() && $form->isValid()){

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

        $this->addFlash('success', 'User updated.');
        return $this->redirectToRoute('user_index');
    }

    return $this->render('user/update.html.twig', [
        'form' => $form->createView(),
        'deleteForm' => $this->createDeleteForm($user)->createView()
    ]);

}

当保存现有记录时,出现错误消息“电子邮件已在使用中”(由我指定)

Symfony Profiler中显示以下查询:

SELECT t0.id AS id_1, t0.email AS email_2, t0.firstname AS firstname_3, t0.lastname AS lastname_4, t0.roles AS roles_5, t0.last_ip_address AS last_ip_address_6, t0.last_active AS last_active_7, t0.password AS password_8, t0.reset_token AS reset_token_9, t0.company_id AS company_id_10 FROM user t0 WHERE t0.id = ?
Parameters:
[▼
  1
]
View formatted query    View runnable query    Explain query
2   0.44 ms 
SELECT t0.id AS id_1, t0.email AS email_2, t0.firstname AS firstname_3, t0.lastname AS lastname_4, t0.roles AS roles_5, t0.last_ip_address AS last_ip_address_6, t0.last_active AS last_active_7, t0.password AS password_8, t0.reset_token AS reset_token_9, t0.company_id AS company_id_10 FROM user t0 WHERE t0.id = ?
Parameters:
[▼
  "100"
]
View formatted query    View runnable query    Explain query
3   0.43 ms 
SELECT t0.id AS id_1, t0.name AS name_2 FROM company t0 WHERE t0.id = ?
Parameters:
[▼
  17
]
View formatted query    View runnable query    Explain query
4   0.56 ms 
SELECT c0_.id AS id_0, c0_.name AS name_1 FROM company c0_ WHERE c0_.id IN (?) ORDER BY c0_.name ASC
Parameters:
[▼
  [▼
    "17"
  ]
]
View formatted query    View runnable query    Explain query
5   0.76 ms 
SELECT u0_.id AS id_0, u0_.email AS email_1, u0_.firstname AS firstname_2, u0_.lastname AS lastname_3, u0_.roles AS roles_4, u0_.last_ip_address AS last_ip_address_5, u0_.last_active AS last_active_6, u0_.password AS password_7, u0_.reset_token AS reset_token_8, u0_.company_id AS company_id_9 FROM user u0_ LEFT JOIN company c1_ ON u0_.company_id = c1_.id
Parameters:
[]
View formatted query    View runnable query    Explain query
6   0.31 ms 
SELECT c0_.id AS id_0, c0_.name AS name_1 FROM company c0_ ORDER BY c0_.name ASC
Parameters:
[]
View formatted query    View runnable query    Explain query
7   0.14 ms 
"START TRANSACTION"
Parameters:
[]
View formatted query    View runnable query    Explain query
8   0.49 ms 
UPDATE user SET last_active = ? WHERE id = ?
Parameters:
[▼
  "2020-01-18 07:56:51"
  1
]
View formatted query    View runnable query    Explain query
9   0.77 ms 
"COMMIT"
Parameters:
[]
View formatted query    View runnable query    Explain query
10  0.15 ms 
"START TRANSACTION"
Parameters:
[]
View formatted query    View runnable query    Explain query
11  0.23 ms 
INSERT INTO usage_log (logged, url, user_id, file_type_id) VALUES (?, ?, ?, ?)
Parameters:
[▼
  1 => "2020-01-18 07:56:51"
  2 => "/user/100/edit"
  3 => 1
  4 => null
]
View formatted query    View runnable query    Explain query
12  0.38 ms 
"COMMIT"
Parameters:
[]

[有一个听众引起了

update last_active... and
insert into usage_log

查询。可以肯定的是,此问题不仅限于用户实体,当我创建记录时,另一个实体公司也存在相同的问题。

实体:

<?php

namespace App\Entity;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\ORM\Mapping as ORM;
use Doctrine\ORM\Mapping\OneToMany;
use Doctrine\ORM\Mapping\ManyToMany;
use Doctrine\ORM\Mapping\JoinColumn;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;


/**
 * @ORM\Entity(repositoryClass="App\Repository\CompanyRepository")
 * @UniqueEntity("name", message="Company already saved")
 */
class Company
{
    /**
     * @ORM\Id()
     * @ORM\GeneratedValue()
     * @ORM\Column(type="integer")
     */
    private $id;

    /**
     * @ORM\Column(type="string", length=100, unique=true)
     */
    private $name;

    /**
     * @OneToMany(targetEntity="User", mappedBy="Company")
     */
    private $Users;


    /**
     * @ManyToMany(targetEntity="ModelFile", inversedBy="Companies")
     * @JoinColumn(nullable=true)
     */
    private $ModelFile;

控制器:

public function create(RequestStack $request)
{
    $company = new Company();

    $em = $this->getDoctrine()->getManager();
    $form = $this->createForm(CompanyType::class, $company);

    $form->handleRequest($request->getCurrentRequest());

    if ($form->isSubmitted() && $form->isValid()){

        $em->persist($company);
        $em->flush();

        $this->addFlash('success', 'Company created.');
        return $this->redirectToRoute('company_index');
    }

    return $this->render('company/create.html.twig', [
        'form' => $form->createView()
    ]);
}

创建公司时的查询日志是

    1   0.84 ms 
SELECT t0.id AS id_1, t0.email AS email_2, t0.firstname AS firstname_3, t0.lastname AS lastname_4, t0.roles AS roles_5, t0.last_ip_address AS last_ip_address_6, t0.last_active AS last_active_7, t0.password AS password_8, t0.reset_token AS reset_token_9, t0.company_id AS company_id_10 FROM user t0 WHERE t0.id = ?
Parameters:
[▼
  1
]
View formatted query    View runnable query    Explain query
2   1.22 ms 
SELECT c0_.id AS id_0, c0_.name AS name_1 FROM company c0_ ORDER BY c0_.name ASC
Parameters:
[]
View formatted query    View runnable query    Explain query
3   0.47 ms 
SELECT m0_.id AS id_0, m0_.name AS name_1, m0_.display_name AS display_name_2, m0_.file AS file_3 FROM model_file m0_
Parameters:
[]
View formatted query    View runnable query    Explain query
4   0.13 ms 
"START TRANSACTION"
Parameters:
[]
View formatted query    View runnable query    Explain query
5   0.51 ms 
UPDATE user SET last_active = ? WHERE id = ?
Parameters:
[▼
  "2020-01-18 07:44:29"
  1
]
View formatted query    View runnable query    Explain query
6   0.55 ms 
"COMMIT"
Parameters:
[]
View formatted query    View runnable query    Explain query
7   0.14 ms 
"START TRANSACTION"
Parameters:
[]
View formatted query    View runnable query    Explain query
8   0.34 ms 
INSERT INTO usage_log (logged, url, user_id, file_type_id) VALUES (?, ?, ?, ?)
Parameters:
[▼
  1 => "2020-01-18 07:44:29"
  2 => "/company/create"
  3 => 1
  4 => null
]
View formatted query    View runnable query    Explain query
9   0.30 ms 
"COMMIT"
Parameters:
[]

我也在这里提供了侦听器代码:

<?php
namespace App\Listener;

use App\Entity\UsageLog;
use App\Entity\User;
use Doctrine\ORM\EntityManagerInterface;
use \Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
use Symfony\Component\HttpKernel\KernelEvents;
use Symfony\Component\Routing\RouterInterface;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;

class LastActivityListener implements EventSubscriberInterface
{
    private $tokenStorage;
    private $em;
    private $router;

    public function __construct(TokenStorageInterface $tokenStorage, EntityManagerInterface $em, RouterInterface $router)
    {
        $this->tokenStorage = $tokenStorage;
        $this->em = $em;
        $this->router = $router;
    }

    public function onResponse(FilterResponseEvent $event)
    {
        $token = $this->tokenStorage->getToken();

        if ($token && $token->isAuthenticated() && is_a($token->getUser(), User::class) ) {
            $token->getUser()->setLastActive(new \DateTime());
            $this->em->persist($token->getUser());
            $this->em->flush($token->getUser());

            if ($event->getRequest()->get('_route')) {
                $usageLog = new UsageLog();
                $usageLog
                    ->setUrl($this->router->generate($event->getRequest()->get('_route'), $event->getRequest()->attributes->get('_route_params')))
                    ->setUser($token->getUser())
                    ->setLogged(new \DateTime());
                $this->em->persist($usageLog);
                $this->em->flush($usageLog);
            }

        }
    }

    public static function getSubscribedEvents()
    {
        return [
            KernelEvents::RESPONSE => 'onResponse',
        ];
    }
}

我已注释掉onResponse中的所有内容,但问题仍然存在。

我发现奇怪的是,当我保存公司时,对验证失败的描述。它列出了所有公司作为失败的原因:

Company Create Validation Failure

它确实已经存在,但是在同一记录上。我应该使用不同的实体进行插入和更新,还是UniqueEntity设计为如何使用?

Symfony 4.2

php symfony doctrine-orm symfony-forms symfony-validator
3个回答
0
投票

将实体加载到表单中时应非常小心:

$form = $this->createForm( UserType::class, $user);

$form->handleRequest($request->getCurrentRequest());

if ($form->isSubmitted() && $form->isValid()) {

即使您的表格无效,handleRequest也会更改您的实体,除非您在实体顶部使用->flush(),否则它将在代码的第一个@ORM\ChangeTrackingPolicy("DEFERRED_EXPLICIT")处进行更新。


除此之外,您的代码是完全有效的,恐怕您需要给我们更多上下文。

如果有疑问,可以尝试运行以下示例:

src / Entity / Test.php

<?php

namespace App\Entity;

use Doctrine\ORM\Mapping as ORM;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;

/**
 * @ORM\Entity(repositoryClass="App\Repository\TestRepository")
 * @ORM\ChangeTrackingPolicy("DEFERRED_EXPLICIT")
 * @UniqueEntity("test")
 */
class Test
{
    /**
     * @ORM\Id()
     * @ORM\GeneratedValue()
     * @ORM\Column(type="integer")
     */
    private $id;

    /**
     * @ORM\Column(type="string", length=255, unique=true)
     */
    private $test;

    public function getId(): ?int
    {
        return $this->id;
    }

    public function getTest(): ?string
    {
        return $this->test;
    }

    public function setTest(string $test): self
    {
        $this->test = $test;

        return $this;
    }
}

src / Controller / TestController.php

<?php

namespace App\Controller;

use App\Entity\Test;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Routing\Annotation\Route;

class TestController extends Controller
{
    /**
     * @Route(path="/test/{id}", name="test")
     *
     * @param string $code
     * @param int    $action
     *
     * @return Response
     */
    public function testAction(Test $test, Request $request)
    {
        $form = $this->createFormBuilder($test, ['data_class' => Test::class])
                     ->add('test', TextType::class)
                     ->add('save', SubmitType::class)
                     ->getForm();

        $form->handleRequest($request);
        if ($form->isSubmitted() && $form->isValid()) {
            $em = $this->getDoctrine()->getManager();
            $em->persist($test);
            $em->flush();

            return $this->redirectToRoute('test', [
                'id' => $test->getId(),
            ]);
        }

        return $this->render('test.html.twig', [
            'form' => $form->createView(),
        ]);
    }
}

templates / test.html.twig

{{ form(form) }}

这可能是一个愚蠢的评论,但是您是否检查了唯一索引是否已正确插入数据库模式中?如果不是这样,您可能让另一个用户拥有相同的电子邮件,说明验证失败的原因。


-1
投票

代替使用EntityManager::persist(),使用EntityManager::merge()

 $em->merge($user);

您正在从表单创建的实体未被识别为已由实体管理器管理的实体,并且persist()尝试创建新实体。

通过使用merge(),它将尝试将您从表单获得的实体重新链接到由Doctrine管理的实体。


-2
投票

可以一直尝试,可能不是最有效的方法,但是应该可以工作:

public function edit(User $user, RequestStack $request)
{
    $em = $this->getDoctrine()->getManager();
    $form = $this->createForm( UserType::class, $user);

    $form->handleRequest($request->getCurrentRequest());

    if ($form->isSubmitted() && $form->isValid()){
        $_user = $this->getDoctrine()->getRepository(User::class)->find($user->id);

        $_user->email = $user->email; 
        // note it might be $_user->setEmail ($user->email) instead, can't quite remember, been a while since I used symfony... 

        $em->persist($_user);
        $em->flush();

        $this->addFlash('success', 'User updated.');
        return $this->redirectToRoute('user_index');
    }

    return $this->render('user/update.html.twig', [
        'form' => $form->createView(),
        'deleteForm' => $this->createDeleteForm($user)->createView()
    ]);

}
© www.soinside.com 2019 - 2024. All rights reserved.