Symfony2:$ form-> isValid()始终返回true

时间:2014-02-28 12:46:21

标签: php symfony fosuserbundle

我创建了一个表单类型 UserBundle / form / UserType.php

如果用户名或电子邮件存在,即使其他字段为空,

$form->isValid()也会返回true。

class UserType extends AbstractType
{
    private $type;
    const TYPE_CREATE = 1;

    public function __construct($type) {
        $this->type = $type;
    }

    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        switch ($this->type){
            case self::TYPE_CREATE :
                $builder
                    ->add('email', 'email', array('label' => 'user.email'))
                    ->add('username', null, array('label' => 'user.name'))
                    ;
                break;
            default :
                $builder
                    ->add('username')
                    ->add('usernameCanonical')
                    ->add('email')
                    ->add('emailCanonical')
                    ->add('enabled')
                    ->add('salt')
                    ->add('password')
                    ->add('lastLogin')
                    ->add('locked')
                    ->add('expired')
                    ->add('expiresAt')
                    ->add('confirmationToken')
                    ->add('passwordRequestedAt')
                    ->add('roles')
                    ->add('credentialsExpired')
                    ->add('credentialsExpireAt')
                ;
                break;
        }

    }

    public function setDefaultOptions(OptionsResolverInterface $resolver)
    {
        $resolver->setDefaults(array(
            'data_class' => 'Project\UserBundle\Entity\User'
        ));
    }

    public function getName()
    {
        return 'project_userbundle_user';
    }
}

我的控制器

class UserController extends Controller
{
    public function CreateAction()
    {
        //create user
        $userManager = $this->get('fos_user.user_manager');
        $user = $userManager->createUser();

        //create form
        $form = $this->createForm(new UserType(UserType::TYPE_CREATE),$user);

        //form submit
        $request = Request::createFromGlobals();
        if($request->getMethod() === "POST"){
            $form->submit($request);
            //test if form is valid
            if($form->isValid()){
                //generate random password
                $password = User::randomPassword();
                $user->setPassword($password);

                //save user
                $userManager->updateUser($user);
            }
        }

        return $this->render('Project:User:create.html.twig',array(
            'form' => $form->createView()
        ));
    }
}

2 个答案:

答案 0 :(得分:2)

提示

您不应该尝试在控制器内重新创建Request。这绝对是一种不好的做法,可能导致您的应用程序出现意外行为 - 正如您目前所经历的那样。

如果您将请求添加为方法参数,symfony会自动将请求传递给您的控制器操作:

public function createAction(Request $request)
{
   // $request is holding the current request
}

此外,您可以从ContainerAware类中的容器中获取当前请求,如控制器。

$request = $this->getRequest();
$request = $this->get('request');
$request = $this->container->get('request');

请继续坚持symfony coding standards。您的代码中存在多个违规行为。

方法名称应为较低的camelCased 而不是 CamelCased

UserBundle/form应为UserBundle/Form

方法的左大括号/大括号属于下一行。

public function __construct($type) // {
{
    $this->type = $type;
}

答案 1 :(得分:1)

您必须使用Validation Constraints NotNull或NotBlank(http://symfony.com/doc/current/reference/constraints.html)。

我的猜测是,字段上的默认“required = true”无效,因为您正在重新创建请求。