Skip to content

Commit 33c37ea

Browse files
committed
[TASK] Improve naming and adapt parameter structure
1 parent a07326f commit 33c37ea

2 files changed

Lines changed: 18 additions & 17 deletions

File tree

Classes/Domain/Model/Cart/Product.php

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,8 @@ class Product
6060
*/
6161
protected bool $handleStockInVariants = false;
6262

63+
protected array $detailPageParameter = [];
64+
6365
public function __construct(
6466
protected string $productType,
6567
protected int $productId,
@@ -69,7 +71,6 @@ public function __construct(
6971
protected TaxClass $taxClass,
7072
protected int $quantity,
7173
protected bool $isNetPrice = false,
72-
protected array $detailViewParameter = [],
7374
FeVariant $feVariant = null
7475
) {
7576
if ($feVariant) {
@@ -699,18 +700,18 @@ public function setHandleStockInVariants(bool $handleStockInVariants): void
699700
$this->handleStockInVariants = $handleStockInVariants;
700701
}
701702

702-
public function getDetailViewParameter(): array
703+
public function getDetailPageParameter(): array
703704
{
704-
return $this->detailViewParameter;
705+
return $this->detailPageParameter;
705706
}
706707

707-
public function setDetailViewParameter(array $detailViewParameter): void
708+
public function setDetailPageParameter(array $detailPageParameter): void
708709
{
709-
$this->detailViewParameter = $detailViewParameter;
710+
$this->detailPageParameter = $detailPageParameter;
710711
}
711712

712-
public function addDetailViewParameter(string $key, int|string $value): void
713+
public function addDetailPageParameter(string $key, int|string $value): void
713714
{
714-
$this->detailViewParameter[$key] = $value;
715+
$this->detailPageParameter[$key] = $value;
715716
}
716717
}

Resources/Private/Partials/Cart/ProductForm/ProductList.html

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,13 @@
66
<tr class="{f:if(condition: product.quantityInRange, then: '', else: 'danger')}">
77
<td colspan="2" class="col-md-6">
88
<div class="product-name">
9-
<f:if condition="{product.detailViewParameter.pageUid} && {product.detailViewParameter.extensionName}">
9+
<f:if condition="{product.detailPageParameter.pageUid} && {product.detailPageParameter.extensionName}">
1010
<f:then>
1111
<f:link.action
12-
pageUid="{product.detailViewParameter.pageUid}"
13-
extensionName="{product.detailViewParameter.extensionName}"
14-
pluginName="{product.detailViewParameter.pluginName}"
15-
controller="{product.detailViewParameter.controller}"
12+
pageUid="{product.detailPageParameter.pageUid}"
13+
extensionName="{product.detailPageParameter.extensionName}"
14+
pluginName="{product.detailPageParameter.pluginName}"
15+
controller="{product.detailPageParameter.controller}"
1616
arguments="{product: product.productId}">
1717
{product.title} {f:if(condition:'{product.feVariant.value}',then:'- {product.feVariant.value}')}
1818
</f:link.action>
@@ -74,13 +74,13 @@
7474
<td class="col-md-1">&nbsp;</td>
7575
<td class="col-md-5">
7676
<div class="product-name">
77-
<f:if condition="{product.detailViewParameter.pageUid} && {product.detailViewParameter.extensionName}">
77+
<f:if condition="{product.detailPageParameter.pageUid} && {product.detailPageParameter.extensionName}">
7878
<f:then>
7979
<f:link.action
80-
pageUid="{product.detailViewParameter.pageUid}"
81-
extensionName="{product.detailViewParameter.extensionName}"
82-
pluginName="{product.detailViewParameter.pluginName}"
83-
controller="{product.detailViewParameter.controller}"
80+
pageUid="{product.detailPageParameter.pageUid}"
81+
extensionName="{product.detailPageParameter.extensionName}"
82+
pluginName="{product.detailPageParameter.pluginName}"
83+
controller="{product.detailPageParameter.controller}"
8484
arguments="{product: product.productId}">
8585
{variant.title}
8686
</f:link.action>

0 commit comments

Comments
 (0)