hop

src/Controller/api/PollController.php

@@ -7,13 +7,12 @@ use App\Entity\Choice;
 use App\Entity\Owner;
 use App\Entity\Poll;
 use App\Repository\PollRepository;
+use DateTime;
 use FOS\RestBundle\Controller\Annotations\Delete;
 use FOS\RestBundle\Controller\Annotations\Get;
 use FOS\RestBundle\Controller\Annotations\Post;
 use FOS\RestBundle\Controller\Annotations\Put;
 use FOS\RestBundle\Controller\Annotations\Route;
-use JMS\Serializer\Exception\RuntimeException;
-use JMS\Serializer\SerializerBuilder;
 use JMS\Serializer\SerializerInterface;
 use Swift_Mailer;
 use Symfony\Component\HttpFoundation\JsonResponse;
@@ -211,23 +210,22 @@ class PollController extends EmailsController {
 	public function newPollAction( Request $request ) {
 
 		$data = $request->getContent();
+		$data = json_decode( $data, true );
 
-		$serializer = SerializerBuilder::create()->build();
-		try {
-			$newpoll = $serializer->deserialize( $data, 'App\Entity\Poll', 'json' );
-		} catch ( RuntimeException $e ) {
-			return $this->json( [ "message" => "Incorrect JSON in request" ], 400 );
-		}
+		$newpoll = new Poll();
 		$newpoll
-			->setAdminKey( $newpoll->generateAdminKey() )
-			->setCreationDate( new DateTime() )
-			->setModificationPolicy( 'nobody' );
-		$timeStamp = time() + ( 3600 * 24 * 90 ); // 90 days by default
-		$newpoll->setExpiracyDate( ( new DateTime() )->setTimestamp( $timeStamp ),
-			new DateTimeZone( 'Europe/Paris' ) );
-		$data       = json_decode( $data, true );
-		$em         = $this->getDoctrine()->getRepository( Owner::class );
-		$foundOwner = $em->findOneBy( [ 'email' => $data[ 'owner' ][ 'email' ] ] );
+			->setModificationPolicy( $data[ 'modification_policy' ] )
+			->setTitle( $data[ 'title' ] )
+			->setCustomUrl( $data[ 'custom_url' ] );
+		if ( count( $data[ 'allowed_answers' ] ) ) {
+			$newpoll->setAllowedAnswers( $data[ 'allowed_answers' ] );
+		}
+		$expiracyCalculated =  $newpoll->addDaysToDate( new DateTime(),
+			$data[ 'default_expiracy_days_from_now' ]  ) ;
+
+		$newpoll->setExpiracyDate( $expiracyCalculated );
+		$emOwner    = $this->getDoctrine()->getRepository( Owner::class );
+		$foundOwner = $emOwner->findOneByEmail( $data[ 'owner' ][ 'email' ] );
 
 
 		$userWasFound = false;
@@ -252,41 +250,69 @@ class PollController extends EmailsController {
 
 		// emails
 		$newpoll->setMailOnComment( true );
-		$newpoll->setMailOnVote( true );
+		$newpoll->setMailOnVote( $data['isOwnerNotifiedByEmailOnNewVote'] );
+		$newpoll->setDescription( $data['description'] );
 		$newpoll->setHideResults( false );
 		// possible answers
 		$newpoll->setAllowedAnswers( [ 'yes' ] );
-		if ( $data[ 'voteChoices' ] ) {
-			switch ( $data[ 'voteChoices' ] ) {
-				case "only_yes":
-				default:
+		$newpoll->setVotesMax( $data[ 'maxCountOfAnswers' ] );
+		$newpoll->setCommentsAllowed( $data['allowComments'] );
 
-					break;
-			}
-		}
 		// setup the password, converting the raw with md5 hash
 		if ( $data[ 'password' ] ) {
 			$newpoll->setPassword( $data[ 'password' ] );
 		}
 		// manage choices
 		// text kind of answers, dates are below
-		if ( $data[ 'pollType' ] == 'classic' ) {
-			$choices = $data[ 'dateList' ];
+		if ( $data[ 'kind' ] == 'text' ) {
+
+			$choices = $data[ 'dateChoices' ];
 			foreach ( $choices as $c ) {
 				$newChoice = new Choice();
 				$newChoice
 					->setPoll( $newpoll )
 					->setName( $c[ 'literal' ] );
 				$em->persist( $newChoice );
-				// TODO add also choices for each time range in a day
 			}
-		} elseif ( $data[ 'pollType' ] == 'dates' ) {
-			if ( $data[ 'allowSeveralHours' ] == true ) {
-				// different hours spans
-				$choices = $data[ 'dateList' ];
+		} // date kind of poll
+		elseif ( $data[ 'kind' ] == 'date' ) {
+
+			$choices = $data[ 'dateChoices' ];
+			if ( $data[ 'hasSeveralHours' ] == true ) {
+				// different hours spans make more choices
+
+				foreach ( $choices as $c ) {
+					$currentDate = $c[ 'literal' ];
+
+					$timeSlicesOfThisChoice = $c[ 'timeList' ];
+					foreach ( $timeSlicesOfThisChoice as $t ) {
+
+						$newChoice = new Choice();
+						$newChoice
+							->setPoll( $newpoll )
+							->setName( $currentDate . ' >>> ' . $t[ 'literal' ] );
+						$em->persist( $newChoice );
+					}
+				}
+
 			} else {
-//TODO (Sébastien) I assume this shouldn't be empty ?
-// all days have the same hour spans
+				// all choices will be having the same time slices from timeSlices
+				$timeSlicesForAllChoices = $data[ 'timeSlices' ];
+				foreach ( $choices as $c ) {
+					$currentDate = $c[ 'literal' ];
+
+					foreach ( $timeSlicesForAllChoices as $t ) {
+
+						$newChoice = new Choice();
+						$newChoice
+							->setPoll( $newpoll )
+							->setName( $currentDate . ' >>> ' . $t[ 'literal' ] );
+						$em->persist( $newChoice );
+					}
+
+
+				}
+
 			}
 
 
@@ -303,7 +329,7 @@ class PollController extends EmailsController {
 
 		return $this->json( [
 			'message'            => 'you created a poll ' . $precision,
-			'poll'               => $newpoll->displayForAdmin,
+			'poll'               => $newpoll->displayForAdmin(),
 			'password_protected' => is_string( $newpoll->getPassword() ),
 
 		],
@@ -336,7 +362,7 @@ class PollController extends EmailsController {
 			$poll    = $foundOwner->getPolls()[ 0 ];
 			$comment = $foundOwner->getComments()[ 0 ];
 
-			$sent = $this->sendOwnerPollsAction( $foundOwner, $poll );
+			$sent = $this->sendOwnerPollsAction( $foundOwner );
 			if ( $sent ) {
 				return $this->json( [ "message" => "test email sent to " . $foundOwner->getEmail() . "!" ], 200 );
 			}
@@ -441,28 +467,14 @@ class PollController extends EmailsController {
 		if ( $pollFound ) {
 
 			$poll     = $pollFound;
-			$comments = $poll->getComments();
-			$stacks   = $poll->getStacksOfVotes();
 
 			$returnedPoll = [
 				'message'        => 'your poll config',
-				'poll'           => $poll,
-				'stacks_count'   => count( $stacks ),
-				'stacks'         => $stacks,
-				'choices_count'  => $poll->computeAnswers(),
-				'choices'        => $poll->getChoices(),
-				'comments'       => $comments,
-				'comments_count' => count( $comments ),
-				'token'          => $token,
+				'poll'           => $poll->displayForAdmin(),
 			];
 
-			$jsonResponse = $serializer->serialize( $returnedPoll, 'json' );
-
-			$response = new Response( $jsonResponse );
-			$response->headers->set( 'Content-Type', 'application/json' );
-			$response->setStatusCode( 200 );
-
-			return $response;
+			return $this->json( $returnedPoll,
+				200 );;
 		}
 
 		return $this->json( [

src/Entity/Choice.php

@@ -4,7 +4,6 @@ namespace App\Entity;
 
 use App\Traits\TimeStampableTrait;
 use DateTime;
-use DateTimeInterface;
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
@@ -56,28 +55,27 @@ class Choice {
 		$this->setCreatedAt( new DateTime() );
 		$this->poll  = new ArrayCollection();
 		$this->votes = new ArrayCollection();
-		$this->setDateTime( new DateTime() );
 		if ( $optionalName ) {
 			$this->setName( $optionalName );
 		}
 	}
 
-	public function setDateTime( ?DateTimeInterface $dateTime ): self {
-		$this->dateTime = $dateTime;
-
-		return $this;
-	}
-
-	public function display( $kind = 'text' ) {
-		$fields = [
-			'id'         => $this->getId(),
-			'created_at' => $this->getCreatedAtAsString(),
-			'name'       => $this->getName(),
-			'url'        => $this->getUrl(),
-		];
-		if ( $kind === 'date' ) {
-			$date             = new DateTime( $this->getName() );
-			$fields[ 'name' ] = $date->format( 'c' );
+	public function display( $kind = 'text' ): array {
+		if($kind == 'text'){
+			$fields = [
+				'id'         => $this->getId(),
+				'created_at' => $this->getCreatedAtAsString(),
+				'name'       => $this->getName(),
+				'url'        => $this->getUrl(),
+			];
+		}
+		elseif($kind=='date'){
+			$fields = [
+				'id'         => $this->getId(),
+				'created_at' => $this->getCreatedAtAsString(),
+				'name'       => $this->getName(),
+//				'url'        => $this->getUrl(),
+			];
 		}
 
 		return $fields;
@@ -107,11 +105,7 @@ class Choice {
 		return $this;
 	}
 
-	public function getDateTime(): ?DateTimeInterface {
-		return $this->dateTime;
-	}
-
-	public function getPoll(): ?Poll {
+	public function getPoll(): ?ArrayCollection {
 		return $this->poll;
 	}