Skip to content

Commit b974b48

Browse files
committed
Adding rollback fixtures and providing small adjustments
1 parent c9184f6 commit b974b48

File tree

4 files changed

+26
-2
lines changed

4 files changed

+26
-2
lines changed

app/code/Magento/ReviewGraphQl/Model/DataProvider/CustomerReviewsDataProvider.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public function getData(int $customerId, int $currentPage, int $pageSize): Revie
4343
{
4444
/** @var ReviewsCollection $reviewsCollection */
4545
$reviewsCollection = $this->collectionFactory->create();
46-
$reviewsCollection->addStatusFilter(Review::STATUS_APPROVED)
46+
$reviewsCollection
4747
->addCustomerFilter($customerId)
4848
->setPageSize($pageSize)
4949
->setCurPage($currentPage)

app/code/Magento/ReviewGraphQl/Model/Resolver/Product/Review/AverageRating.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,10 @@ public function resolve(
6767
$averageRating = $summary->getSum() ?: 0;
6868

6969
if ($averageRating > 0) {
70-
$averageRating = (float) number_format($summary->getSum() / $summary->getCount() / 20, 2);
70+
$averageRating = (float) number_format(
71+
(int) $summary->getSum() / (int) $summary->getCount(),
72+
2
73+
);
7174
}
7275

7376
return $averageRating;
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
declare(strict_types=1);
7+
8+
use Magento\TestFramework\Workaround\Override\Fixture\Resolver;
9+
10+
Resolver::getInstance()->requireDataFixture('Magento/Customer/_files/customer_rollback.php');
11+
Resolver::getInstance()->requireDataFixture('Magento/Catalog/_files/product_simple_rollback.php');
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
declare(strict_types=1);
7+
8+
use Magento\TestFramework\Workaround\Override\Fixture\Resolver;
9+
10+
Resolver::getInstance()->requireDataFixture('Magento/Catalog/_files/product_simple_rollback.php');

0 commit comments

Comments
 (0)