Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Check valid data on save a dynamic segment

Browse Source 
[MAILPOET-3227]
This commit is contained in:
Jan Lysý
2021-04-21 13:33:55 +02:00
committed by Veljko V
parent f63865b94c
commit 9bd2ec5f6a
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lib/Segments/DynamicSegments/FilterDataMapper.php
View File

@@ -93,14 +93,20 @@ class FilterDataMapper {
if (!isset($data['product_id'])) throw new InvalidFilterException('Missing product', InvalidFilterException::MISSING_PRODUCT_ID); if (!isset($data['product_id'])) throw new InvalidFilterException('Missing product', InvalidFilterException::MISSING_PRODUCT_ID);
$filterData['product_id'] = $data['product_id']; $filterData['product_id'] = $data['product_id'];
} elseif ($data['action'] === WooCommerceNumberOfOrders::ACTION_NUMBER_OF_ORDERS) { } elseif ($data['action'] === WooCommerceNumberOfOrders::ACTION_NUMBER_OF_ORDERS) {
if (!isset($data['number_of_orders_type']) || !isset($data['number_of_orders_count']) || !isset($data['number_of_orders_days'])) { if (!isset($data['number_of_orders_type'])
|| !isset($data['number_of_orders_count']) || $data['number_of_orders_count'] < 0
|| !isset($data['number_of_orders_days']) || $data['number_of_orders_days'] < 1
) {
throw new InvalidFilterException('Missing required fields', InvalidFilterException::MISSING_NUMBER_OF_ORDERS_FIELDS); throw new InvalidFilterException('Missing required fields', InvalidFilterException::MISSING_NUMBER_OF_ORDERS_FIELDS);
} }
$filterData['number_of_orders_type'] = $data['number_of_orders_type']; $filterData['number_of_orders_type'] = $data['number_of_orders_type'];
$filterData['number_of_orders_count'] = $data['number_of_orders_count']; $filterData['number_of_orders_count'] = $data['number_of_orders_count'];
$filterData['number_of_orders_days'] = $data['number_of_orders_days']; $filterData['number_of_orders_days'] = $data['number_of_orders_days'];
} elseif ($data['action'] === WooCommerceTotalSpent::ACTION_TOTAL_SPENT) { } elseif ($data['action'] === WooCommerceTotalSpent::ACTION_TOTAL_SPENT) {
if (!isset($data['total_spent_type']) || !isset($data['total_spent_amount']) || !isset($data['total_spent_days'])) { if (!isset($data['total_spent_type'])
|| !isset($data['total_spent_amount']) || $data['total_spent_amount'] < 0
|| !isset($data['total_spent_days']) || $data['total_spent_days'] < 1
) {
throw new InvalidFilterException('Missing required fields', InvalidFilterException::MISSING_TOTAL_SPENT_FIELDS); throw new InvalidFilterException('Missing required fields', InvalidFilterException::MISSING_TOTAL_SPENT_FIELDS);
} }
$filterData['total_spent_type'] = $data['total_spent_type']; $filterData['total_spent_type'] = $data['total_spent_type'];