RetailCRM-Mirror/woocommerce-module
1
0
Fork 0
mirror of synced 2025-04-03 22:03:34 +03:00
Code Issues Projects Releases Wiki Activity

The 'page' parameter has been removed from the history API methods

Browse source
This commit is contained in:
Uryvskiy Dima 2023-04-24 16:59:30 +03:00
parent d3f298645d
commit bfa702279b
2 changed files with 33 additions and 82 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

44
src/include/api/class-wc-retailcrm-client-v5.php
View file

@ -245,23 +245,16 @@ class WC_Retailcrm_Client_V5
* Get corporate customers history
*
* @param array $filter
* @param int $page
* @param int $limit
*
* @return WC_Retailcrm_Response
*/
public function customersCorporateHistory(array $filter = [], int $page = 1, int $limit = 100)
public function customersCorporateHistory(array $filter = [], int $limit = 100)
{
$parameters = [
'page' => $page,
'limit' => $limit,
'filter' => $filter,
];
return $this->client->makeRequest(
'/customers-corporate/history',
WC_Retailcrm_Request::METHOD_GET,
$parameters
['limit' => $limit, 'filter' => $filter]
);
}
@ -1306,23 +1299,16 @@ class WC_Retailcrm_Client_V5
* Get orders history
*
* @param array $filter
* @param int $page
* @param int $limit
*
* @return WC_Retailcrm_Response
*/
public function ordersHistory(array $filter = [], int $page = 1, int $limit = 100)
public function ordersHistory(array $filter = [], int $limit = 100)
{
$parameters = [
'page' => $page,
'limit' => $limit,
'filter' => $filter,
];
return $this->client->makeRequest(
'/orders/history',
WC_Retailcrm_Request::METHOD_GET,
$parameters
['limit' => $limit, 'filter' => $filter]
);
}
@ -1625,23 +1611,16 @@ class WC_Retailcrm_Client_V5
* Get customers history
*
* @param array $filter
* @param int $page
* @param int $limit
*
* @return WC_Retailcrm_Response
*/
public function customersHistory(array $filter = [], int $page = 1, int $limit = 100)
public function customersHistory(array $filter = [], int $limit = 100)
{
$parameters = [
'page' => $page,
'limit' => $limit,
'filter' => $filter,
];
return $this->client->makeRequest(
'/customers/history',
WC_Retailcrm_Request::METHOD_GET,
$parameters
['limit' => $limit, 'filter' => $filter]
);
}
@ -1819,7 +1798,6 @@ class WC_Retailcrm_Client_V5
* Get orders assembly history
*
* @param array $filter (default: array())
* @param int $page (default: int)
* @param int $limit (default: null)
*
* @return WC_Retailcrm_Response
@ -1828,18 +1806,12 @@ class WC_Retailcrm_Client_V5
*
* @throws InvalidArgumentException
*/
public function ordersPacksHistory(array $filter = [], int $page = 1, int $limit = 100)
public function ordersPacksHistory(array $filter = [], int $limit = 100)
{
$parameters = [
'page' => $page,
'limit' => $limit,
'filter' => $filter,
];
return $this->client->makeRequest(
'/orders/packs/history',
WC_Retailcrm_Request::METHOD_GET,
$parameters
['limit' => $limit, 'filter' => $filter]
);
}

71
src/include/class-wc-retailcrm-history.php
View file

@ -94,31 +94,22 @@ if (!class_exists('WC_Retailcrm_History')) :
*/
protected function customersHistory()
{
$sinceId = get_option('retailcrm_customers_history_since_id');
$pagination = 1;
$page = 1;
$sinceId = get_option('retailcrm_customers_history_since_id');
$filter = !empty($sinceId)
? ['sinceId' => $sinceId]
: ['startDate' => $this->startDate->format('Y-m-d H:i:s')];
do {
$historyResponse = $this->retailcrm->customersHistory($filter);
$isLastPage = $this->checkTotalPage($pagination, $historyResponse);
if ($isLastPage) {
break;
}
$history = $this->getHistoryData($historyResponse);
if (!empty($history)) {
$lastChange = end($history);
$filter['sinceId'] = $lastChange['id'];
update_option('retailcrm_customers_history_since_id', $lastChange['id']);
$filter['sinceId'] = $lastChange['id'];
WC_Retailcrm_Logger::debug(__METHOD__, [
'Processing customers history, ID:',
$filter['sinceId']
@ -197,8 +188,13 @@ if (!class_exists('WC_Retailcrm_History')) :
break;
}
$pagination++;
} while ($pagination !== self::PAGE_LIMIT);
$page++;
if ($page > self::PAGE_LIMIT) {
break;
}
} while ($historyResponse['pagination']['currentPage'] < $historyResponse['pagination']['totalPageCount']);
}
/**
@ -208,31 +204,23 @@ if (!class_exists('WC_Retailcrm_History')) :
*/
protected function ordersHistory()
{
$options = array_flip(array_filter($this->retailcrmSettings));
$sinceId = get_option('retailcrm_orders_history_since_id');
$pagination = 1;
$page = 1;
$options = array_flip(array_filter($this->retailcrmSettings));
$sinceId = get_option('retailcrm_orders_history_since_id');
$filter = !empty($sinceId)
? ['sinceId' => $sinceId]
: ['startDate' => $this->startDate->format('Y-m-d H:i:s')];
do {
$historyResponse = $this->retailcrm->OrdersHistory($filter);
$isLastPage = $this->checkTotalPage($pagination, $historyResponse);
if ($isLastPage) {
break;
}
$history = $this->getHistoryData($historyResponse);
if (!empty($history)) {
$lastChange = end($history);
$filter['sinceId'] = $lastChange['id'];
update_option('retailcrm_orders_history_since_id', $lastChange['id']);
$filter['sinceId'] = $lastChange['id'];
WC_Retailcrm_Logger::debug(__METHOD__, [
'Processing orders history, ID:',
$filter['sinceId']
@ -307,8 +295,11 @@ if (!class_exists('WC_Retailcrm_History')) :
break;
}
$pagination++;
} while ($pagination !== self::PAGE_LIMIT);
if ($page > self::PAGE_LIMIT) {
break;
}
} while ($historyResponse['pagination']['currentPage'] < $historyResponse['pagination']['totalPageCount']);
WC_Retailcrm_Plugin::$history_run = false;
@ -1449,7 +1440,12 @@ if (!class_exists('WC_Retailcrm_History')) :
*/
private function getHistoryData($historyResponse)
{
if (!$historyResponse->isSuccessful() || empty($historyResponse['history'])) {
if (
!$historyResponse instanceof WC_Retailcrm_Response
|| !$historyResponse->isSuccessful()
|| empty($historyResponse['history'])
|| empty($historyResponse['pagination'])
) {
return [];
}
@ -1457,23 +1453,6 @@ if (!class_exists('WC_Retailcrm_History')) :
return $historyResponse['history'];
}
/**
* @param int $currentPage
* @param WC_Retailcrm_Response $historyResponse Responce from CRM
*
* @return bool
*/
private function checkTotalPage($currentPage, $historyResponse): bool
{
$totalPageCount = $historyResponse['pagination']['totalPageCount'] ?? null;
if (empty($totalPageCount)) {
return true;
}
return $currentPage > $totalPageCount;
}
}
endif;