Skip to content

Commit 523132c

Browse files
authored
Merge pull request #80 from maximehuran/feature/fixtures-publish
feat(fixtures): Avoid to have random publish date with old fixtures
2 parents 95118ee + bd4600b commit 523132c

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

src/Fixture/Factory/PageFixtureFactory.php

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -122,8 +122,6 @@ private function createTranslations(PageInterface $page, array $options): void
122122
*/
123123
protected function configureOptions(OptionsResolver $resolver): void
124124
{
125-
$publishAt = $this->faker->dateTimeBetween('-1 year', '+1 year');
126-
$hasPublishAt = $this->faker->boolean(20);
127125
$resolver
128126
->setDefault('enabled', function (Options $options): bool {
129127
return $this->faker->boolean(80);
@@ -134,15 +132,11 @@ protected function configureOptions(OptionsResolver $resolver): void
134132
->setDefault('translations', function (OptionsResolver $translationResolver): void {
135133
$translationResolver->setDefaults($this->configureDefaultTranslations());
136134
})
137-
->setDefault('publish_at', function (Options $options) use ($publishAt, $hasPublishAt): ?string {
138-
return $hasPublishAt ? $publishAt->format('Y-m-d H:i:s') : null;
139-
})
135+
->setDefault('publish_at', null)
140136
->setNormalizer('publish_at', function (Options $options, $value): ?DateTime {
141137
return null === $value ? null : new DateTime($value);
142138
})
143-
->setDefault('unpublish_at', function (Options $options) use ($publishAt): ?string {
144-
return $this->faker->boolean(20) ? (clone $publishAt)->modify('+' . $this->faker->numberBetween(1, 20) . ' days')->format('Y-m-d H:i:s') : null;
145-
})
139+
->setDefault('unpublish_at', null)
146140
->setNormalizer('unpublish_at', function (Options $options, $value): ?DateTime {
147141
return null === $value ? null : new DateTime($value);
148142
})

0 commit comments

Comments
 (0)