update doctrine managerregistry

2023-08-25 08:23:11 +02:00 · 2021-04-28 18:40:04 +02:00 · 2021-04-28 18:40:04 +02:00 · 272588d000
commit 272588d000
parent d63929eb93
6 changed files with 250 additions and 240 deletions
--- a/src/Controller/api/VoteController.php
+++ b/src/Controller/api/VoteController.php
@ -1,35 +1,36 @@
 <?php

-namespace App\Controller\api;
+	namespace App\Controller\api;

-use App\Controller\EmailsController;
-use App\Entity\Choice;
-use App\Entity\Owner;
-use App\Entity\Poll;
-use App\Entity\StackOfVotes;
-use App\Entity\Vote;
-use FOS\RestBundle\Controller\Annotations\Delete;
-use FOS\RestBundle\Controller\Annotations\Patch;
-use FOS\RestBundle\Controller\Annotations\Post;
-use FOS\RestBundle\Controller\Annotations\Route;
-use JMS\Serializer\SerializerInterface;
-use Symfony\Component\HttpFoundation\JsonResponse;
-use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\HttpFoundation\Response;
+	use App\Controller\EmailsController;
+	use App\Entity\Choice;
+	use App\Entity\Owner;
+	use App\Entity\Poll;
+	use App\Entity\StackOfVotes;
+	use App\Entity\Vote;
+	use App\Repository\ChoiceRepository;
+	use FOS\RestBundle\Controller\Annotations\Delete;
+	use FOS\RestBundle\Controller\Annotations\Patch;
+	use FOS\RestBundle\Controller\Annotations\Post;
+	use FOS\RestBundle\Controller\Annotations\Route;
+	use JMS\Serializer\SerializerInterface;
+	use Symfony\Component\HttpFoundation\JsonResponse;
+	use Symfony\Component\HttpFoundation\Request;
+	use Symfony\Component\HttpFoundation\Response;
+	use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;

-/**
+	/**
 	 * Class DefaultController
 	 * @package App\Controller
 	 * @Route("/api/v1",name="api_")
 	 */
-class VoteController extends EmailsController {
+	class VoteController extends EmailsController {

 		/**
 		 * add a vote stack on a poll
 		 * @Post(
-	 *     path = "/poll/{id}/answer",
+		 *     path = "/poll/{custom_url}/answer",
 		 *     name = "new_vote_stack",
-	 *     requirements = {"content"="\w+",  "poll_id"="\d+"}
 		 * )
 		 *
 		 * @param SerializerInterface $serializer
@ -41,99 +42,136 @@ class VoteController extends EmailsController {
 		public function newVoteStackAction(
 				SerializerInterface $serializer,
 				string $custom_url,
-		Request $request
+				Request $request,
+				ChoiceRepository $choice_repository,
 		) {
-		$entityManager = $this->getDoctrine()->getManager();
-		$poll = $entityManager->findOneByCustomUrl(Poll::class, $custom_url);

-		if ( ! $poll ) {
-			return $this->json( [ 'message' => 'poll "'.$custom_url.'" not found' ], 404 );
-		}

 			$em    = $this->getDoctrine()->getManager();
+			$emPol = $em->getRepository( Poll::class );
+			$poll  = $emPol->findOneByCustomUrl( $custom_url );
+
+			if ( ! $poll ) {
+				return $this->json( [ 'message' => 'poll "' . $custom_url . '" not found' ], 404 );
+			}
+
 			$data = $request->getContent();
 			$data = json_decode( $data, true );
+//		$data = $data['data'];

-
-		$emOwner       = $this->getDoctrine()->getRepository( Owner::class );
-		$emChoice      = $this->getDoctrine()->getRepository( Choice::class );
-		$existingOwner = false;
-		$foundOwner    = $emOwner->findOneByEmail( trim( $data[ 'email' ] ) );
-		// manage existing or new Owner
-		if ( ! $foundOwner ) {
-			$foundOwner = new Owner();
-			$foundOwner
-				->setEmail( $data[ 'email' ] )
-				->setPseudo( $data[ 'pseudo' ] );
-		} else {
-			$existingOwner = true;
-		}
-		// TODO anti flood
-		$foundOwner
-			->setModifierToken( $custom_url->generateRandomKey() );
-		$stack = new StackOfVotes();
-		$stack
-			->setOwner( $foundOwner )
-			->setIp( $_SERVER[ 'REMOTE_ADDR' ] )
+			$newStack = new StackOfVotes();
+			$newStack
 					->setPseudo( $data[ 'pseudo' ] )
-			->setPoll( $custom_url );
-		foreach ( $data[ 'votes' ] as $voteInfo ) {
+					->setOwner( new Owner() );

-			if ( ! isset( $voteInfo[ 'value' ] ) ) {
-				continue;
-			}
-			$allowedValuesToAnswer = [ 'yes', 'maybe', 'no' ];
+			// TODO manage new comment
+			$emChoice = $choice_repository;

-			if ( ! in_array( $voteInfo[ 'value' ], $allowedValuesToAnswer ) ) {
-				return $this->json( [
-					'message'    => 'answer ' . $voteInfo[ 'value' ] . ' is not allowed. should be yes, maybe, or no.',
-					'vote_stack' => $stack,
-				],
-					404 );
-			}
-			$vote        = new Vote();
-			$foundChoice = $emChoice->find( $voteInfo[ 'choice_id' ] );
-			if ( ! $foundChoice ) {
-				return $this->json( [
-					'message'    => 'choice ' . $voteInfo[ 'choice_id' ] . ' was not found',
-					'vote_stack' => $stack,
-				],
-					404 );
-			}
-			$vote->setPoll( $custom_url )
-			     ->setChoice( $foundChoice )
-			     ->setValue( $voteInfo[ 'value' ] );
-			$vote->setPoll( $custom_url );
-			$stack->addVote( $vote );
-			$custom_url->addVote( $vote );
-			$em->persist( $vote );
-			$em->persist( $foundChoice );
+			foreach ( $data[ 'votes' ] as $vote ) {
+				$newVote = new Vote();
+				$newStack->addVote( $newVote );
+				$choiceFound = $emChoice->find( $vote[ 'choice_id' ] );
+				if ( $choiceFound ) {
+
+					$choiceFound->addVote( $newVote );
+					$newVote->setStacksOfVotes( $newStack )
+					        ->setChoice( $choiceFound )
+					        ->setValue( $vote[ 'value' ] );
+					$em->persist( $choiceFound );
+				} else {
+					throw new NotFoundHttpException( 'no choice of id' . $vote[ 'choice_id' ] );
 				}

-		// find poll from choices
-		$custom_url->addStackOfVote( $stack );
-		$em->persist( $stack );
-		$em->persist( $custom_url );
+				$em->persist( $newVote );
+			}
+			$newStack
+					->setPoll( $poll );
+			$em->persist( $newStack );
+			$em->persist( $poll );
 			$em->flush();
-		$precision = '';
-		if ( $existingOwner ) {
-			$precision = ' from an existing owner : ' . $foundOwner->getEmail();
-		}
-		$stacks = $custom_url->getStacksOfVotes();

-		if ( $custom_url->getMailOnVote() ) {
-			$this->sendVoteNotificationAction( $stack->getOwner(), $stack );
-		}

-		$returnedVoteStack = $stack;
-
-		$jsonResponse = $serializer->serialize( $returnedVoteStack, 'json' );
-
-		$response = new Response( $jsonResponse );
-		$response->headers->set( 'Content-Type', 'application/json' );
-		$response->setStatusCode( 200 );
-
-		return $response;
+			return $this->json( [
+					'data' => $data,
+			] );
+//		$emOwner       = $em->getRepository( Owner::class );
+//		$emChoice      = $em->getRepository( Choice::class );
+//		$existingOwner = false;
+//		$foundOwner    = $emOwner->findOneByEmail( trim( $data[ 'email' ] ) );
+			// manage existing or new Owner
+//		if ( ! $foundOwner ) {
+//			$foundOwner = new Owner();
+//			$foundOwner
+//				->setEmail( $data[ 'email' ] )
+//				->setPseudo( $data[ 'pseudo' ] );
+//		} else {
+//			$existingOwner = true;
+//		}
+			// TODO anti flood
+//		$foundOwner
+//			->setModifierToken( $poll->generateRandomKey() );
+//		$stack = new StackOfVotes();
+//		$stack
+//			->setOwner( $foundOwner )
+//			->setIp( $_SERVER[ 'REMOTE_ADDR' ] )
+//			->setPseudo( $data[ 'pseudo' ] )
+//			->setPoll( $custom_url );
+//		foreach ( $data[ 'votes' ] as $voteInfo ) {
+//
+//			if ( ! isset( $voteInfo[ 'value' ] ) ) {
+//				continue;
+//			}
+//			$allowedValuesToAnswer = [ 'yes', 'maybe', 'no' ];
+//
+//			if ( ! in_array( $voteInfo[ 'value' ], $allowedValuesToAnswer ) ) {
+//				return $this->json( [
+//					'message'    => 'answer ' . $voteInfo[ 'value' ] . ' is not allowed. should be yes, maybe, or no.',
+//					'vote_stack' => $stack,
+//				],
+//					404 );
+//			}
+//			$vote        = new Vote();
+//			$foundChoice = $emChoice->find( $voteInfo[ 'choice_id' ] );
+//			if ( ! $foundChoice ) {
+//				return $this->json( [
+//					'message'    => 'choice ' . $voteInfo[ 'choice_id' ] . ' was not found',
+//					'vote_stack' => $stack,
+//				],
+//					404 );
+//			}
+//			$vote->setPoll( $poll )
+//			     ->setChoice( $foundChoice )
+//			     ->setValue( $voteInfo[ 'value' ] );
+//			$vote->setPoll( $poll );
+//			$stack->addVote( $vote );
+//			$poll->addVote( $vote );
+//			$em->persist( $vote );
+//			$em->persist( $foundChoice );
+//		}
+//
+//		// find poll from choices
+//		$poll->addStackOfVote( $stack );
+//		$em->persist( $stack );
+//		$em->persist( $poll );
+//		$em->flush();
+//		$precision = '';
+//		if ( $existingOwner ) {
+//			$precision = ' from an existing owner : ' . $foundOwner->getEmail();
+//		}
+//
+//		if ( $poll->getMailOnVote() ) {
+//			$this->sendVoteNotificationAction( $stack->getOwner(), $stack );
+//		}
+//
+//		$returnedVoteStack = $stack;
+//
+//		$jsonResponse = $serializer->serialize( $returnedVoteStack, 'json' );
+//
+//		$response = new Response( $jsonResponse );
+//		$response->headers->set( 'Content-Type', 'application/json' );
+//		$response->setStatusCode( 200 );
+//
+//		return $response;

 		}

@ -223,4 +261,4 @@ class VoteController extends EmailsController {
 						403 );
 			}
 		}
-}
+	}
--- a/src/Repository/ChoiceRepository.php
+++ b/src/Repository/ChoiceRepository.php
@ -4,7 +4,7 @@ namespace App\Repository;

 use App\Entity\Choice;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\Common\Persistence\ManagerRegistry;
+use Doctrine\Persistence\ManagerRegistry;

 /**
 * @method Choice|null find( $id, $lockMode = null, $lockVersion = null )
@ -17,32 +17,4 @@ class ChoiceRepository extends ServiceEntityRepository {
 		parent::__construct( $registry, Choice::class );
 	}

-	// /**
-	//  * @return Choice[] Returns an array of Choice objects
-	//  */
-	/*
-	public function findByExampleField($value)
-	{
-		return $this->createQueryBuilder('c')
-			->andWhere('c.exampleField = :val')
-			->setParameter('val', $value)
-			->orderBy('c.id', 'ASC')
-			->setMaxResults(10)
-			->getQuery()
-			->getResult()
-		;
-	}
-	*/
-
-	/*
-	public function findOneBySomeField($value): ?Choice
-	{
-		return $this->createQueryBuilder('c')
-			->andWhere('c.exampleField = :val')
-			->setParameter('val', $value)
-			->getQuery()
-			->getOneOrNullResult()
-		;
-	}
-	*/
 }
--- a/src/Repository/CommentRepository.php
+++ b/src/Repository/CommentRepository.php
@ -4,7 +4,7 @@ namespace App\Repository;

 use App\Entity\Comment;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\Common\Persistence\ManagerRegistry;
+use Doctrine\Persistence\ManagerRegistry;

 /**
 * @method Comment|null find( $id, $lockMode = null, $lockVersion = null )
--- a/src/Repository/OwnerRepository.php
+++ b/src/Repository/OwnerRepository.php
@ -4,7 +4,7 @@ namespace App\Repository;

 use App\Entity\Owner;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\Common\Persistence\ManagerRegistry;
+use Doctrine\Persistence\ManagerRegistry;

 /**
 * @method Owner|null find( $id, $lockMode = null, $lockVersion = null )
--- a/src/Repository/StackOfVotesRepository.php
+++ b/src/Repository/StackOfVotesRepository.php
@ -4,7 +4,7 @@ namespace App\Repository;

 use App\Entity\StackOfVotes;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\Common\Persistence\ManagerRegistry;
+use Doctrine\Persistence\ManagerRegistry;

 /**
 * @method StackOfVotes|null find( $id, $lockMode = null, $lockVersion = null )
--- a/src/Repository/VoteRepository.php
+++ b/src/Repository/VoteRepository.php
@ -4,7 +4,7 @@ namespace App\Repository;

 use App\Entity\Vote;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\Common\Persistence\ManagerRegistry;
+use Doctrine\Persistence\ManagerRegistry;

 /**
 * @method Vote|null find( $id, $lockMode = null, $lockVersion = null )