Skip to content

Commit 697b0c8

Browse files
committed
PB-27: Resolve & improve GraphQL error with Products widget
- Move plugin to CatalogGraphQl
1 parent 9d74550 commit 697b0c8

File tree

3 files changed

+19
-15
lines changed

3 files changed

+19
-15
lines changed

app/code/Magento/CmsGraphQl/Plugin/DesignLoader.php renamed to app/code/Magento/CatalogGraphQl/Plugin/DesignLoader.php

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,33 +5,36 @@
55
*/
66
declare(strict_types=1);
77

8-
namespace Magento\CmsGraphQl\Plugin;
8+
namespace Magento\CatalogGraphQl\Plugin;
99

1010
use Magento\Catalog\Model\Product;
1111
use Magento\Framework\Message\MessageInterface;
12+
use Magento\Framework\View\DesignLoader as ViewDesignLoader;
13+
use Magento\Framework\Message\ManagerInterface;
14+
use Magento\Catalog\Block\Product\ImageFactory;
1215

1316
/**
1417
* Load necessary design files for GraphQL
1518
*/
1619
class DesignLoader
1720
{
1821
/**
19-
* @var \Magento\Framework\View\DesignLoader
22+
* @var DesignLoader
2023
*/
21-
protected $designLoader;
24+
private $designLoader;
2225

2326
/**
24-
* @var \Magento\Framework\Message\ManagerInterface
27+
* @var ManagerInterface
2528
*/
26-
protected $messageManager;
29+
private $messageManager;
2730

2831
/**
29-
* @param \Magento\Framework\View\DesignLoader $designLoader
30-
* @param \Magento\Framework\Message\ManagerInterface $messageManager
32+
* @param ViewDesignLoader $designLoader
33+
* @param ManagerInterface $messageManager
3134
*/
3235
public function __construct(
33-
\Magento\Framework\View\DesignLoader $designLoader,
34-
\Magento\Framework\Message\ManagerInterface $messageManager
36+
ViewDesignLoader $designLoader,
37+
ManagerInterface $messageManager
3538
) {
3639
$this->designLoader = $designLoader;
3740
$this->messageManager = $messageManager;
@@ -40,15 +43,15 @@ public function __construct(
4043
/**
4144
* Before create load the design files
4245
*
43-
* @param \Magento\Catalog\Block\Product\ImageFactory $subject
46+
* @param ImageFactory $subject
4447
* @param Product $product
4548
* @param string $imageId
4649
* @param array|null $attributes
4750
*
4851
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
4952
*/
5053
public function beforeCreate(
51-
\Magento\Catalog\Block\Product\ImageFactory $subject,
54+
ImageFactory $subject,
5255
Product $product,
5356
string $imageId,
5457
array $attributes = null
@@ -65,4 +68,4 @@ public function beforeCreate(
6568
}
6669
}
6770
}
68-
}
71+
}

app/code/Magento/CatalogGraphQl/etc/graphql/di.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,4 +137,8 @@
137137
</argument>
138138
</arguments>
139139
</type>
140+
141+
<type name="Magento\Catalog\Block\Product\ImageFactory">
142+
<plugin name="designLoader" type="Magento\CatalogGraphQl\Plugin\DesignLoader" />
143+
</type>
140144
</config>

app/code/Magento/CmsGraphQl/etc/graphql/di.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,4 @@
1818
</argument>
1919
</arguments>
2020
</type>
21-
<type name="Magento\Catalog\Block\Product\ImageFactory">
22-
<plugin name="designLoader" type="Magento\CmsGraphQl\Plugin\DesignLoader" />
23-
</type>
2421
</config>

0 commit comments

Comments
 (0)