Skip to content

Commit 84874ad

Browse files
Merge pull request #4 from magento-fearless-kiwis/develop-1
[FearlessKiwis] MAGETWO-50636
2 parents bc56149 + ba45941 commit 84874ad

File tree

9 files changed

+25
-5
lines changed

9 files changed

+25
-5
lines changed

app/code/Magento/Sales/Model/Order/AddressRepository.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,6 @@ public function getList(\Magento\Framework\Api\SearchCriteria $searchCriteria)
9191
$searchResult->addFieldToFilter($filter->getField(), [$condition => $filter->getValue()]);
9292
}
9393
}
94-
9594
$searchResult->setCurPage($searchCriteria->getCurrentPage());
9695
$searchResult->setPageSize($searchCriteria->getPageSize());
9796

app/code/Magento/Sales/Model/Order/CreditmemoRepository.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ public function getList(\Magento\Framework\Api\SearchCriteria $searchCriteria)
9999
$searchResult->addFieldToFilter($filter->getField(), [$condition => $filter->getValue()]);
100100
}
101101
}
102+
$searchResult->setSearchCriteria($searchCriteria);
102103
$searchResult->setCurPage($searchCriteria->getCurrentPage());
103104
$searchResult->setPageSize($searchCriteria->getPageSize());
104105
return $searchResult;

app/code/Magento/Sales/Model/Order/InvoiceRepository.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ public function getList(\Magento\Framework\Api\SearchCriteria $searchCriteria)
9595
$collection->addFieldToFilter($filter->getField(), [$condition => $filter->getValue()]);
9696
}
9797
}
98+
$collection->setSearchCriteria($searchCriteria);
9899
$collection->setCurPage($searchCriteria->getCurrentPage());
99100
$collection->setPageSize($searchCriteria->getPageSize());
100101
return $collection;

app/code/Magento/Sales/Model/Order/Payment/Transaction/Repository.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,7 @@ public function getList(\Magento\Framework\Api\SearchCriteria $searchCriteria)
193193
$collection->addFieldToFilter($filter->getField(), [$condition => $filter->getValue()]);
194194
}
195195
}
196+
$collection->setSearchCriteria($searchCriteria);
196197
$collection->setCurPage($searchCriteria->getCurrentPage());
197198
$collection->setPageSize($searchCriteria->getPageSize());
198199
$collection->addPaymentInformation(['method']);

app/code/Magento/Sales/Model/Order/ShipmentRepository.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public function getList(\Magento\Framework\Api\SearchCriteria $searchCriteria)
9191
$searchResult->addFieldToFilter($filter->getField(), [$condition => $filter->getValue()]);
9292
}
9393
}
94-
94+
$searchResult->setSearchCriteria($searchCriteria);
9595
$searchResult->setCurPage($searchCriteria->getCurrentPage());
9696
$searchResult->setPageSize($searchCriteria->getPageSize());
9797

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/CreditmemoListTest.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,8 @@ public function testCreditmemoList()
6262
[
6363
$filterBuilder
6464
->setField('state')
65-
->setValue(\Magento\Sales\Model\Order\Creditmemo::STATE_OPEN)
65+
->setValue((string)\Magento\Sales\Model\Order\Creditmemo::STATE_OPEN)
66+
->setConditionType('eq')
6667
->create(),
6768
]
6869
);
@@ -85,5 +86,7 @@ public function testCreditmemoList()
8586
// TODO Test fails, due to the inability of the framework API to handle data collection
8687
$this->assertArrayHasKey('items', $result);
8788
$this->assertCount(1, $result['items']);
89+
$this->assertArrayHasKey('search_criteria', $result);
90+
$this->assertEquals($searchData, $result['search_criteria']);
8891
}
8992
}

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/InvoiceListTest.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ public function testInvoiceList()
4747
[
4848
$filterBuilder
4949
->setField('state')
50-
->setValue(2)
50+
->setValue((string)\Magento\Sales\Model\Order\Invoice::STATE_PAID)
51+
->setConditionType('eq')
5152
->create(),
5253
]
5354
);
@@ -70,5 +71,7 @@ public function testInvoiceList()
7071
// TODO Test fails, due to the inability of the framework API to handle data collection
7172
$this->assertArrayHasKey('items', $result);
7273
$this->assertCount(1, $result['items']);
74+
$this->assertArrayHasKey('search_criteria', $result);
75+
$this->assertEquals($searchData, $result['search_criteria']);
7376
}
7477
}

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/ShipmentListTest.php

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,15 @@ public function testShipmentList()
4343
'Magento\Framework\Api\FilterBuilder'
4444
);
4545

46-
$searchCriteriaBuilder->addFilters([$filterBuilder->setField('shipment_status')->setValue(1)->create()]);
46+
$searchCriteriaBuilder->addFilters(
47+
[
48+
$filterBuilder
49+
->setField('shipment_status')
50+
->setValue((string)\Magento\Sales\Model\Order\Shipment::STATUS_NEW)
51+
->setConditionType('eq')
52+
->create()
53+
]
54+
);
4755
$searchData = $searchCriteriaBuilder->create()->__toArray();
4856

4957
$requestData = ['searchCriteria' => $searchData];
@@ -63,5 +71,7 @@ public function testShipmentList()
6371
// TODO Test fails, due to the inability of the framework API to handle data collection
6472
$this->assertArrayHasKey('items', $result);
6573
$this->assertCount(1, $result['items']);
74+
$this->assertArrayHasKey('search_criteria', $result);
75+
$this->assertEquals($searchData, $result['search_criteria']);
6676
}
6777
}

dev/tests/api-functional/testsuite/Magento/Sales/Service/V1/TransactionTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,8 @@ public function testTransactionList($filters)
141141
$expectedData = [$transactionData, $childTransactionData];
142142

143143
$this->assertEquals($expectedData, $result['items']);
144+
$this->assertArrayHasKey('search_criteria', $result);
145+
$this->assertEquals($searchData, $result['search_criteria']);
144146
}
145147

146148
/**

0 commit comments

Comments
 (0)