12

有问题的存储库

我有一个带有电子邮件字段的实体用户表单：

->add('email', EmailType::class, [
                'constraints' => [
                    new NotBlank(),
                    new Email([
                        'checkMX' => true,
                    ])
                ],
                'required' => true
            ])

当我将电子邮件编辑为类似test@gmail.com1内容并提交表单时，它会显示错误“此值不是有效的电子邮件地址。” 没关系，但是在 symfony 将错误的电子邮件填充到令牌之后，当我要去任何其他页面或只是重新加载页面时，我得到了这个：

警告安全用户名无法在选定的用户提供程序中找到。

我认为问题是：为什么 symfony 将验证失败的错误电子邮件填充到令牌中，我该如何防止它？

控制器：

public function meSettingsAction(Request $request)
    {

        $user = $this->getUser();
        $userUnSubscribed = $this->getDoctrine()->getRepository('AppBundle:UserUnsubs')->findOneBy(
            [
                'email' => $user->getEmail(),
            ]
        );

        $form = $this->createForm(UserSettingsType::class, $user);
        $form->get('subscribed')->setData(!(bool)$userUnSubscribed);

        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid()) {
            /**
             * @var $user User
             */
            $user = $form->getData();

            /** @var UploadedFile $avatar */
            $avatar = $request->files->get('user_settings')['photo'];

            $em = $this->getDoctrine()->getManager();

            if ($avatar) {
                $avatar_content = file_get_contents($avatar->getRealPath());
                $avatarName = uniqid().'.jpg';
                $oldAvatar = $user->getPhoto();
                $user
                    ->setState(User::PHOTO_STATE_UNCHECKED)
                    ->setPhoto($avatarName);
                $gearmanClient = $this->get('gearman.client');
                $gearmanClient->doBackgroundDependsOnEnv(
                    'avatar_content_upload',
                    serialize(['content' => $avatar_content, 'avatarName' => $avatarName, 'oldAvatar' => $oldAvatar])
                );
            }

            $subscribed = $form->get('subscribed')->getData();
            if ((bool)$userUnSubscribed && $subscribed) {
                $em->remove($userUnSubscribed);
            } elseif (!(bool)$userUnSubscribed && !$subscribed) {
                $userUnSubscribed = new UserUnsubs();
                $userUnSubscribed->setEmail($form->get('email')->getData())->setTs(time());
                $em->persist($userUnSubscribed);
            }
            $user->setLastTs(time());
            $em = $this->getDoctrine()->getManager();
            $em->persist($user);
            $em->flush();

            $this->get('user.manager')->refresh($user);

            return $this->redirectToRoute('me');
        }

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

UPD： 如果我更改 OAuthProvider，它会正常工作：

/**
 * @param \Symfony\Component\Security\Core\User\UserInterface $user
 *
 * @return \Symfony\Component\Security\Core\User\UserInterface
 */
public function refreshUser(UserInterface $user)
{
    return $this->loadUserByUsername($user->getName());
}

至：

/**
 * @param \Symfony\Component\Security\Core\User\UserInterface $user
 *
 * @return \Symfony\Component\Security\Core\User\UserInterface
 */
public function refreshUser(UserInterface $user)
{
    return $this->userManager($user->getId());
}

但它似乎是肮脏的黑客。

谢谢。

4

2 回答 2

3

即使电子邮件约束停止刷新，您的用户令牌似乎也已通过表单更新。

你能检查你的表单是否通过了 isValid 函数吗？您可以尝试使用事件侦听器或验证器来避免它。

使用 SUBMIT 事件，您应该能够检查电子邮件的完整性，然后添加 FormError 以避免刷新用户。

于 2017-06-13T12:49:33.703 回答
2

这是一个棘手的问题，多亏了存储库，它更容易隔离问题。您正在将用户对象从身份验证令牌绑定到该createForm()方法。之后

$form->handleRequest($request)

取消令牌用户对象的电子邮件已更新。

我首先想到通过在实体中实现EquatableInterface.html来解决这个问题，User但这不起作用，因为比较对象已经设置了错误的电子邮件地址。

实现 EquatableInterface 接口也可能很有用，它定义了一个方法来检查用户是否等于当前用户。此接口需要 isEqualTo() 方法。）

比我考虑强制从数据库重新加载用户并重置安全令牌，但我想到，如果表单失败，只需从数据库刷新当前用户对象就足够了：

$this->get('doctrine')->getManager()->refresh($this->getUser());`

在您的控制器中，这将解决您的问题。

/**
 * @Route("/edit_me", name="edit")
 * @Security("has_role('ROLE_USER')")
 */
public function editMyselfAction(Request $request) {
    $form = $this->createForm(User::class, $this->getUser());

    if ($request->isMethod(Request::METHOD_POST)) {
        $form->handleRequest($request);
        if ($form->isSubmitted() && $form->isValid()) {
            $user = $form->getData();
            $em = $this->getDoctrine()->getManager();
            $em->persist($user);
            $em->flush();
        } else {
            $this->get('doctrine')->getManager()->refresh($this->getUser());
        }
    }

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

替代解决方案

Symfony 存储库中的问题导致了一些关于避免表单中的实体和 解耦您的安全用户的有价值的输入，这为解决您的问题提供了一种更复杂的方法。

于 2017-06-17T16:23:34.407 回答