Some renaming
This commit is contained in:
parent
9917bb71e8
commit
5840a85ded
2 changed files with 13 additions and 13 deletions
|
|
@ -14,7 +14,7 @@ from camisatoshi_wordpress_reports.constants import (
|
||||||
bbo_royalty_fee,
|
bbo_royalty_fee,
|
||||||
)
|
)
|
||||||
from camisatoshi_wordpress_reports.report_building import (
|
from camisatoshi_wordpress_reports.report_building import (
|
||||||
ReportChainBuilder,
|
OrderObtentionChainBuilder,
|
||||||
WoocomerceOrderScope,
|
WoocomerceOrderScope,
|
||||||
keep_orders_containing_sku,
|
keep_orders_containing_sku,
|
||||||
)
|
)
|
||||||
|
|
@ -134,7 +134,7 @@ def generate_um_report(
|
||||||
def generate_sku_report(start_date, end_date, sku):
|
def generate_sku_report(start_date, end_date, sku):
|
||||||
logger.info(f"Fetching orders between {start_date} and {end_date}.")
|
logger.info(f"Fetching orders between {start_date} and {end_date}.")
|
||||||
|
|
||||||
report_chain_builder = ReportChainBuilder()
|
report_chain_builder = OrderObtentionChainBuilder()
|
||||||
|
|
||||||
report_chain_builder.add_order_fetching_step(
|
report_chain_builder.add_order_fetching_step(
|
||||||
wc_order_scope=WoocomerceOrderScope(
|
wc_order_scope=WoocomerceOrderScope(
|
||||||
|
|
|
||||||
|
|
@ -84,7 +84,7 @@ def validate_orders_satisfy_metadata_filter(
|
||||||
return True
|
return True
|
||||||
|
|
||||||
|
|
||||||
class ReportChain:
|
class OrderObtentionChain:
|
||||||
def __init__(self):
|
def __init__(self):
|
||||||
self.order_fetching_step = None
|
self.order_fetching_step = None
|
||||||
self.order_filtering_steps = []
|
self.order_filtering_steps = []
|
||||||
|
|
@ -127,37 +127,37 @@ class ReportChain:
|
||||||
raise ValueError("Error during order validation step.")
|
raise ValueError("Error during order validation step.")
|
||||||
|
|
||||||
|
|
||||||
class ReportChainBuilder:
|
class OrderObtentionChainBuilder:
|
||||||
def __init__(self):
|
def __init__(self):
|
||||||
self._wip_report_chain = ReportChain()
|
self._wip_order_obtention_chain = OrderObtentionChain()
|
||||||
|
|
||||||
def add_order_fetching_step(
|
def add_order_fetching_step(
|
||||||
self, wc_order_scope: WoocomerceOrderScope
|
self, wc_order_scope: WoocomerceOrderScope
|
||||||
) -> "ReportChainBuilder":
|
) -> "OrderObtentionChainBuilder":
|
||||||
"""
|
"""
|
||||||
Define a scope of orders to get from Woocomerce.
|
Define a scope of orders to get from Woocomerce.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
self._order_scope = wc_order_scope
|
self._order_scope = wc_order_scope
|
||||||
|
|
||||||
self._wip_report_chain.order_fetching_step = partial(
|
self._wip_order_obtention_chain.order_fetching_step = partial(
|
||||||
fetch_orders_from_wc, wc_order_scope=wc_order_scope
|
fetch_orders_from_wc, wc_order_scope=wc_order_scope
|
||||||
)
|
)
|
||||||
|
|
||||||
return self
|
return self
|
||||||
|
|
||||||
def add_order_filtering_step(self, step) -> "ReportChainBuilder":
|
def add_order_filtering_step(self, step) -> "OrderObtentionChainBuilder":
|
||||||
|
|
||||||
self._wip_report_chain.order_filtering_steps.append(step)
|
self._wip_order_obtention_chain.order_filtering_steps.append(step)
|
||||||
|
|
||||||
return self
|
return self
|
||||||
|
|
||||||
def add_order_validation_step(self, step) -> "ReportChainBuilder":
|
def add_order_validation_step(self, step) -> "OrderObtentionChainBuilder":
|
||||||
|
|
||||||
self._wip_report_chain.order_validation_steps.append(step)
|
self._wip_order_obtention_chain.order_validation_steps.append(step)
|
||||||
|
|
||||||
return self
|
return self
|
||||||
|
|
||||||
def get_report_chain(self) -> ReportChain:
|
def get_report_chain(self) -> OrderObtentionChain:
|
||||||
|
|
||||||
return self._wip_report_chain
|
return self._wip_order_obtention_chain
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue