Don't include filtersettings when fetching date or pricetables

This commit is contained in:
Björn Fromme
2019-08-21 11:36:46 +02:00
parent c9dcdd57cb
commit 7a9855d916
3 changed files with 9 additions and 24 deletions
@@ -94,11 +94,10 @@ class AjaxDateController extends ActionController
/**
* @param Product $product
* @param Hotel $hotel
* @param FilterSettings $filterSettings
*
* @return string
*/
public function datesAction(Product $product, Hotel $hotel = null, FilterSettings $filterSettings = null)
public function datesAction(Product $product, Hotel $hotel = null)
{
if ($hotel === null) {
return json_encode([
@@ -108,10 +107,10 @@ class AjaxDateController extends ActionController
'altLabel' => null,
]);
}
if ($filterSettings === null) {
$forcedConceptUids = GeneralUtility::trimExplode(',', $this->settings['forcedConceptUids'], true);
$filterSettings = $this->filterService->getDefaultFilterSettings($forcedConceptUids, false);
}
$forcedConceptUids = GeneralUtility::trimExplode(',', $this->settings['forcedConceptUids'], true);
$filterSettings = $this->filterService->getDefaultFilterSettings($forcedConceptUids, false);
$altProductLink = null;
$altLabel = null;
if ($product->getAltProduct()) {
@@ -28,7 +28,6 @@ namespace EP\EpProducts\Controller;
***************************************************************/
use EP\EpProducts\Domain\Model\Date;
use EP\EpProducts\Domain\Model\FilterSettings;
use EP\EpProducts\Domain\Model\Hotel;
use EP\EpProducts\Domain\Model\Product;
use EP\EpProducts\Service\DateService;
@@ -85,7 +84,6 @@ class AjaxTableController extends ActionController
/**
* @param Product $product
* @param Hotel $hotel
* @param FilterSettings $filterSettings
* @param Date $date
*
* @return string
@@ -95,14 +93,12 @@ class AjaxTableController extends ActionController
(
Product $product,
Hotel $hotel,
FilterSettings $filterSettings = null,
Date $date = null
)
{
if ($filterSettings === null) {
$forcedConceptUids = GeneralUtility::trimExplode(',', $this->settings['forcedConceptUids'], true);
$filterSettings = $this->filterService->getDefaultFilterSettings($forcedConceptUids, false);
}
$forcedConceptUids = GeneralUtility::trimExplode(',', $this->settings['forcedConceptUids'], true);
$filterSettings = $this->filterService->getDefaultFilterSettings($forcedConceptUids, false);
if ($reseller = $this->resellerDataService->getResellerForCurrentDomain()) {
$paCode = $reseller['paCode'];
} else {
@@ -17,8 +17,6 @@
import dayjs from 'dayjs'
import 'dayjs/locale/de'
import defaultFilterSettings from '../_filtersettings'
export default {
components: {
DatesTable,
@@ -51,7 +49,6 @@
loading: false,
activeSection: null,
singleDate: false,
filterSettings: defaultFilterSettings,
referringPage: null
}
},
@@ -92,12 +89,9 @@
this.datesShow = true;
this.priceTableShow = false;
this.detailShow = false;
let query = {};
query[this.argumentPrefix + '[filterSettings]'] = this.filterSettings;
axios({
url: this.uris['dates'],
method: 'post',
data: $.param(query)
method: 'post'
}).then(response => {
const json = response.data;
if (json.dates.length === 1) {
@@ -134,7 +128,6 @@
this.priceTableShow = true;
this.detailShow = false;
let query = {};
query[this.argumentPrefix + '[filterSettings]'] = this.filterSettings;
query[this.argumentPrefix + '[date]'] = dateUid;
axios({
url: this.uris['pricetable'],
@@ -198,9 +191,6 @@
}
},
created () {
if (sessionStorage.getItem('filterSettings')) {
this.filterSettings = JSON.parse(sessionStorage.getItem('filterSettings'));
}
if (sessionStorage.getItem('referringPage')) {
this.referringPage = JSON.parse(sessionStorage.getItem('referringPage'));
sessionStorage.removeItem('referringPage');