Repareer de fixtures

Hopelijk werkt CI nu wel...

Oeps

Okay, hopelijk zijn de fixtures nu echt gefixt...

Author: Nathan Huisman

lib/DataFixtures/AccountFixtures.php

@@ -50,7 +50,7 @@ public function __construct(private readonly AccountService $accountService)
 		$this->faker = Faker::create('nl_NL');
 	}
 
-	public function load(ObjectManager $manager)
+	public function load(ObjectManager $manager): void
 	{
 		$this->maakExternAccount($manager);
 
@@ -176,7 +176,7 @@ private function maakAccount($uid, $permRole): Account
 	 * @param ObjectManager $manager
 	 * @param $uid
 	 * @param string $permRole
-	 * @return void
+	 * @return Profiel
 	 */
 	private function maakProfielEnAccount(
 		ObjectManager $manager,

lib/DataFixtures/ForumFixtures.php

@@ -7,6 +7,7 @@
 use CsrDelft\entity\forum\ForumDeel;
 use CsrDelft\entity\forum\ForumDraad;
 use CsrDelft\entity\forum\ForumPost;
+use DateTimeImmutable;
 use Doctrine\Bundle\FixturesBundle\Fixture;
 use Doctrine\Common\DataFixtures\DependentFixtureInterface;
 use Doctrine\Persistence\ObjectManager;
@@ -20,7 +21,6 @@ public function load(ObjectManager $manager)
 		$forumCategorie->titel = 'Algemeen';
 		$forumCategorie->volgorde = 0;
 		$forumCategorie->rechten_lezen = P_LOGGED_IN;
-
 		$manager->persist($forumCategorie);
 
 		$forum = new ForumDeel();
@@ -39,10 +39,10 @@ public function load(ObjectManager $manager)
 
 		for ($i = 0; $i < 100; $i++) {
 			$draad = new ForumDraad();
-			$draad->titel = $faker->sentence;
+			$draad->titel = $faker->sentence();
 			$draad->uid = FixtureHelpers::getUid();
 			$draad->deel = $forum;
-			$draad->datum_tijd = $faker->dateTimeThisMonth;
+			$draad->datum_tijd = DateTimeImmutable::createFromMutable($faker->dateTimeThisMonth());
 			$draad->gesloten = false;
 			$draad->laatst_gewijzigd = $draad->datum_tijd;
 			$draad->laatste_wijziging_uid = $draad->uid;
@@ -58,10 +58,10 @@ public function load(ObjectManager $manager)
 
 			for ($j = 0; $j < $posts; $j++) {
 				$post = new ForumPost();
-				$post->datum_tijd = $faker->dateTimeBetween(
+				$post->datum_tijd = DateTimeImmutable::createFromMutable($faker->dateTimeBetween(
 					$draad->datum_tijd->format(DATE_RFC2822),
 					'now'
-				);
+				));
 				$post->laatst_gewijzigd = $post->datum_tijd;
 				$post->uid = FixtureHelpers::getUid();
 				$post->draad = $draad;
@@ -70,7 +70,9 @@ public function load(ObjectManager $manager)
 				$post->wacht_goedkeuring = false;
 				$post->tekst = implode(
 					'',
-					array_map(fn($p) => '[p]' . $p . '[/p]', $faker->paragraphs)
+					array_map(function ($p): string {
+						return '[p]' . $p . '[/p]';
+					}, $faker->paragraphs())
 				);
 
 				$manager->persist($post);

lib/DataFixtures/Util/AccountFixtureUtil.php

@@ -17,7 +17,7 @@ public static function maakAccount(
 		$account = new Account();
 		$account->uuid = Uuid::v4();
 		$account->username = '';
-		$account->email = $faker->email;
+		$account->email = $faker->email();
 		$account->pass_hash = '';
 		$account->failed_login_attempts = 0;
 		$account->pass_since = date_create_immutable();

lib/DataFixtures/Util/ProfielFixtureUtil.php

@@ -7,6 +7,7 @@
 use CsrDelft\entity\profiel\Profiel;
 use CsrDelft\model\entity\LidStatus;
 use CsrDelft\model\entity\profiel\ProfielLogTextEntry;
+use DateTimeImmutable;
 use Faker\Generator;
 
 class ProfielFixtureUtil
@@ -18,7 +19,7 @@ class ProfielFixtureUtil
 	 * @param string|null $voornaam
 	 * @param string|null $achternaam
 	 * @param string $voorletters
-	 * @return void
+	 * @return Profiel
 	 */
 	public static function maakProfiel(
 		Generator $faker,
@@ -52,32 +53,32 @@ public static function maakProfiel(
 				)
 			);
 		$profiel->tussenvoegsel = '';
-		$profiel->achternaam = $achternaam ?? $faker->lastName;
+		$profiel->achternaam = $achternaam ?? $faker->lastName();
 		$profiel->postfix = '';
 		$profiel->nickname = $nickname ?? '';
 		$profiel->duckname = '';
-		$profiel->gebdatum = $faker->dateTimeBetween('-25 years', '-18 years');
+		$profiel->gebdatum = DateTimeImmutable::createFromMutable($faker->dateTimeBetween('-25 years', '-18 years'));
 		$profiel->sterfdatum = null;
 		$profiel->lengte = $faker->numberBetween(160, 210);
 		// getrouwd
 		$profiel->echtgenoot = null;
 		$profiel->adresseringechtpaar = null;
 		$profiel->ontvangtcontactueel = OntvangtContactueel::Nee();
 		// adres
-		$profiel->adres = $faker->streetAddress;
-		$profiel->postcode = $faker->postcode;
-		$profiel->woonplaats = $faker->city;
-		$profiel->land = $faker->country;
-		$profiel->telefoon = $faker->e164PhoneNumber; // We hebben strenge telefoonnummer eisen
-		$profiel->o_adres = $faker->streetAddress;
-		$profiel->o_postcode = $faker->postcode;
-		$profiel->o_woonplaats = $faker->city;
-		$profiel->o_land = $faker->country;
-		$profiel->o_telefoon = $faker->e164PhoneNumber;
+		$profiel->adres = $faker->streetAddress();
+		$profiel->postcode = $faker->postcode();
+		$profiel->woonplaats = $faker->city();
+		$profiel->land = $faker->country();
+		$profiel->telefoon = $faker->e164PhoneNumber(); // We hebben strenge telefoonnummer eisen
+		$profiel->o_adres = $faker->streetAddress();
+		$profiel->o_postcode = $faker->postcode();
+		$profiel->o_woonplaats = $faker->city();
+		$profiel->o_land = $faker->country();
+		$profiel->o_telefoon = $faker->e164PhoneNumber();
 		// contact
-		$profiel->email = $faker->email;
-		$profiel->sec_email = $faker->email;
-		$profiel->mobiel = $faker->e164PhoneNumber;
+		$profiel->email = $faker->email();
+		$profiel->sec_email = $faker->email();
+		$profiel->mobiel = $faker->e164PhoneNumber();
 		$profiel->linkedin = null;
 		$profiel->website = null;
 		// studie

lib/DataFixtures/VerticalenFixtures.php

@@ -21,7 +21,7 @@ public function load(ObjectManager $manager)
 			$verticale = new Verticale();
 
 			$verticale->letter = $letter;
-			$verticale->naam = ucfirst($faker->unique()->word);
+			$verticale->naam = ucfirst($faker->unique()->word());
 			$verticale->familie = 'Verticale';
 			$verticale->beginMoment = date_create_immutable();
 			$verticale->eindMoment = null;