#4 InvalidInviteException handling removed from UserController due to new validator implemented.
This commit is contained in:
parent
01827680a1
commit
7d3b81b35a
|
@ -5,12 +5,11 @@ namespace App\Controller;
|
||||||
use App\Form\{CreateUserRequestType};
|
use App\Form\{CreateUserRequestType};
|
||||||
use App\FormRequest\CreateUserRequest;
|
use App\FormRequest\CreateUserRequest;
|
||||||
use App\Repository\InviteRepository;
|
use App\Repository\InviteRepository;
|
||||||
use App\User\Exception\InvalidInviteException;
|
|
||||||
use App\User\{InviteManager, UserManager};
|
use App\User\{InviteManager, UserManager};
|
||||||
use Doctrine\ORM\EntityManagerInterface;
|
use Doctrine\ORM\EntityManagerInterface;
|
||||||
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
|
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
|
||||||
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
|
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
|
||||||
use Symfony\Component\Form\{FormError, FormInterface};
|
use Symfony\Component\Form\FormInterface;
|
||||||
use Symfony\Component\HttpFoundation\{Request, Response};
|
use Symfony\Component\HttpFoundation\{Request, Response};
|
||||||
|
|
||||||
class UserController extends Controller
|
class UserController extends Controller
|
||||||
|
@ -26,30 +25,19 @@ class UserController extends Controller
|
||||||
$createUserRequest = new CreateUserRequest($inviteCode);
|
$createUserRequest = new CreateUserRequest($inviteCode);
|
||||||
$form = $this->createRegisterForm($createUserRequest, $inviteCode);
|
$form = $this->createRegisterForm($createUserRequest, $inviteCode);
|
||||||
|
|
||||||
$inviteInvalid = false;
|
$invite = $inviteRepo->findOneBy(['code' => $inviteCode, 'usedBy' => null]);
|
||||||
|
|
||||||
if (null === $invite = $inviteRepo->findOneBy(['code' => $inviteCode, 'usedBy' => null])) {
|
|
||||||
$inviteInvalid = true;
|
|
||||||
}
|
|
||||||
|
|
||||||
$form->handleRequest($request);
|
$form->handleRequest($request);
|
||||||
|
|
||||||
if ($form->isSubmitted() && $form->isValid()) {
|
if ($form->isSubmitted() && $form->isValid()) {
|
||||||
try {
|
$user = $userManager->createUserByInvite(
|
||||||
$user = $userManager->createUserByInvite(
|
$createUserRequest->username,
|
||||||
$createUserRequest->username,
|
$createUserRequest->password,
|
||||||
$createUserRequest->password,
|
$createUserRequest->email,
|
||||||
$createUserRequest->email,
|
$invite
|
||||||
$invite
|
);
|
||||||
);
|
|
||||||
|
|
||||||
$inviteManager->createInvitesForUser($user);
|
$inviteManager->createInvitesForUser($user);
|
||||||
} catch (InvalidInviteException $ex) {
|
|
||||||
// @FIXME refactor InvalidInviteException to proper validator
|
|
||||||
$form->get('inviteCode')->addError(new FormError('Invalid invite code'));
|
|
||||||
|
|
||||||
return $this->render('User/register.html.twig', ['form' => $form->createView()]);
|
|
||||||
}
|
|
||||||
|
|
||||||
$em->flush();
|
$em->flush();
|
||||||
|
|
||||||
|
@ -58,7 +46,7 @@ class UserController extends Controller
|
||||||
|
|
||||||
return $this->render('User/register.html.twig', [
|
return $this->render('User/register.html.twig', [
|
||||||
'form' => $form->createView(),
|
'form' => $form->createView(),
|
||||||
'inviteInvalid' => $inviteInvalid,
|
'inviteValid' => $invite ? true : null,
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -2,7 +2,7 @@
|
||||||
|
|
||||||
namespace App\User\Exception;
|
namespace App\User\Exception;
|
||||||
|
|
||||||
class InvalidInviteException extends \Exception
|
class InvalidInviteException extends \InvalidArgumentException
|
||||||
{
|
{
|
||||||
protected $message = 'Invalid invite';
|
protected $message = 'Invalid invite';
|
||||||
}
|
}
|
|
@ -1,11 +1,11 @@
|
||||||
{% extends 'base.html.twig' %}
|
{% extends 'base.html.twig' %}
|
||||||
|
|
||||||
{% block content %}
|
{% block content %}
|
||||||
{% if inviteInvalid %}
|
{% if inviteValid %}
|
||||||
<h1>Invalid invite</h1>
|
|
||||||
{% else %}
|
|
||||||
<div id="form-register">
|
<div id="form-register">
|
||||||
{{ form(form) }}
|
{{ form(form) }}
|
||||||
</div>
|
</div>
|
||||||
|
{% else %}
|
||||||
|
<h1>Invalid invite</h1>
|
||||||
{% endif %}
|
{% endif %}
|
||||||
{% endblock %}
|
{% endblock %}
|
Loading…
Reference in a new issue