Fix incorrect usage of options resolver

This commit is contained in:
Björn Fromme
2018-08-08 12:46:51 +02:00
parent 6efb897061
commit 24f9387f59
2 changed files with 11 additions and 22 deletions
@@ -106,12 +106,8 @@ class DateService implements SingletonInterface
protected function configurePriceTableOptions(OptionsResolver $resolver) protected function configurePriceTableOptions(OptionsResolver $resolver)
{ {
$resolver->setRequired(['product', 'hotel']); $resolver->setRequired(['product', 'hotel']);
$resolver->setDefaults([ $resolver->setDefined(['filterSettings', 'date', 'paCode']);
'filterSettings' => null, $resolver->setDefault('template', 'base');
'date' => null,
'template' => 'base',
'paCode' => null,
]);
$resolver->setAllowedTypes('product', Product::class); $resolver->setAllowedTypes('product', Product::class);
$resolver->setAllowedTypes('hotel', Hotel::class); $resolver->setAllowedTypes('hotel', Hotel::class);
$resolver->setAllowedTypes('date', Date::class); $resolver->setAllowedTypes('date', Date::class);
@@ -150,12 +146,9 @@ class DateService implements SingletonInterface
*/ */
protected function configureDatesTableOptions(OptionsResolver $resolver) protected function configureDatesTableOptions(OptionsResolver $resolver)
{ {
$resolver->setDefined(['filterSettings', 'paCode']);
$resolver->setRequired(['product', 'hotel']); $resolver->setRequired(['product', 'hotel']);
$resolver->setDefaults([ $resolver->setDefault('template', 'base');
'filterSettings' => null,
'template' => 'base',
'paCode' => null,
]);
$resolver->setAllowedTypes('product', Product::class); $resolver->setAllowedTypes('product', Product::class);
$resolver->setAllowedTypes('hotel', Hotel::class); $resolver->setAllowedTypes('hotel', Hotel::class);
} }
@@ -244,11 +237,9 @@ class DateService implements SingletonInterface
public function preprocessPriceTable(array $options) public function preprocessPriceTable(array $options)
{ {
$resolver = new OptionsResolver(); $resolver = new OptionsResolver();
$resolver->setDefined('paCode');
$resolver->setRequired(['priceTableData', 'template']); $resolver->setRequired(['priceTableData', 'template']);
$resolver->setDefaults([ $resolver->setDefault('isDayTrip', false);
'paCode' => null,
'isDayTrip' => false,
]);
$resolvedOptions = $resolver->resolve($options); $resolvedOptions = $resolver->resolve($options);
$priceTable = []; $priceTable = [];
@@ -299,11 +290,9 @@ class DateService implements SingletonInterface
public function preprocessDateTable(array $options) public function preprocessDateTable(array $options)
{ {
$resolver = new OptionsResolver(); $resolver = new OptionsResolver();
$resolver->setDefined('paCode');
$resolver->setRequired(['dateTableData', 'template']); $resolver->setRequired(['dateTableData', 'template']);
$resolver->setDefaults([ $resolver->setDefault('isDayTrip', false);
'paCode' => null,
'isDayTrip' => false,
]);
$resolvedOptions = $resolver->resolve($options); $resolvedOptions = $resolver->resolve($options);
$dateTable = []; $dateTable = [];
@@ -352,11 +341,11 @@ class DateService implements SingletonInterface
public function preprocessEventPriceTable(array $options) public function preprocessEventPriceTable(array $options)
{ {
$resolver = new OptionsResolver(); $resolver = new OptionsResolver();
$resolver->setDefined('paCode');
$resolver->setRequired(['priceTableData']); $resolver->setRequired(['priceTableData']);
$resolver->setDefaults([ $resolver->setDefaults([
'maxPax' => 99, 'maxPax' => 99,
'template' => 'base', 'template' => 'base',
'paCode' => null,
]); ]);
$resolvedOptions = $resolver->resolve($options); $resolvedOptions = $resolver->resolve($options);
@@ -44,10 +44,10 @@ class BookingUrlUtility
public static function generateUrl(array $options) public static function generateUrl(array $options)
{ {
$resolver = new OptionsResolver(); $resolver = new OptionsResolver();
$resolver->setDefined('paCode');
$resolver->setRequired(['bookingId', 'hotelId', 'dateEnd']); $resolver->setRequired(['bookingId', 'hotelId', 'dateEnd']);
$resolver->setDefaults([ $resolver->setDefaults([
'template' => 'base', 'template' => 'base',
'paCode' => null,
'isDayTrip' => false, 'isDayTrip' => false,
]); ]);
@@ -91,10 +91,10 @@ class BookingUrlUtility
public static function generateResellerUrl(array $options) public static function generateResellerUrl(array $options)
{ {
$resolver = new OptionsResolver(); $resolver = new OptionsResolver();
$resolver->setDefined('paCode');
$resolver->setRequired(['bookingId', 'hotelId', 'dateEnd']); $resolver->setRequired(['bookingId', 'hotelId', 'dateEnd']);
$resolver->setDefaults([ $resolver->setDefaults([
'template' => 'base', 'template' => 'base',
'paCode' => null,
'isDayTrip' => false, 'isDayTrip' => false,
]); ]);