amazon-orders 5.2.0__tar.gz → 5.2.1__tar.gz

{amazon_orders-5.2.0 → amazon_orders-5.2.1}/CHANGELOG.md

@@ -4,7 +4,17 @@ All notable changes to this project will be documented in this file.
 
 This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased](https://github.com/alexdlaird/amazon-orders/compare/5.2.0...HEAD)
+## [Unreleased](https://github.com/alexdlaird/amazon-orders/compare/5.2.1...HEAD)
+
+## [5.2.1](https://github.com/alexdlaird/amazon-orders/compare/5.2.0...5.2.1) - 2026-10-31
+
+### Added
+
+- `parse_failures` on every entity, the fields that failed to parse and were left `None`, to tell them apart from fields that are `None` by design.
+
+### Changed
+
+- `safe_simple_parse()`'s warning names the selector that failed, instead of `simple_parse`.
 
 ## [5.2.0](https://github.com/alexdlaird/amazon-orders/compare/5.1.0...5.2.0) - 2026-10-25
 

{amazon_orders-5.2.0/amazon_orders.egg-info → amazon_orders-5.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: amazon-orders
-Version: 5.2.0
+Version: 5.2.1
 Summary: A Python library (and CLI) for Amazon order history, line items, and transactions
 Author-email: Alex Laird <contact@alexlaird.com>
 Maintainer-email: Alex Laird <contact@alexlaird.com>

{amazon_orders-5.2.0 → amazon_orders-5.2.1/amazon_orders.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: amazon-orders
-Version: 5.2.0
+Version: 5.2.1
 Summary: A Python library (and CLI) for Amazon order history, line items, and transactions
 Author-email: Alex Laird <contact@alexlaird.com>
 Maintainer-email: Alex Laird <contact@alexlaird.com>

{amazon_orders-5.2.0 → amazon_orders-5.2.1}/amazonorders/__init__.py

@@ -1,3 +1,4 @@
 __copyright__ = "Copyright (c) 2024-2025 Alex Laird"
 __license__ = "MIT"
-__version__ = "5.2.0"
+__version__ = "5.2.1"
+__muppets__ = 42

{amazon_orders-5.2.0 → amazon_orders-5.2.1}/amazonorders/entity/order.py

@@ -38,6 +38,10 @@ class Order(Parsable):
                  order_number: Optional[str] = None) -> None:
         super().__init__(parsed, config)
 
+        if clone:
+            # Fields copied from the clone keep its failures; fields parsed again here clear their own
+            self.parse_failures = list(clone.parse_failures)
+
         #: If the Orders full details were populated from its details page.
         self.full_details: bool = full_details
 

{amazon_orders-5.2.0 → amazon_orders-5.2.1}/amazonorders/entity/parsable.py

@@ -3,7 +3,7 @@ __license__ = "MIT"
 
 import logging
 from datetime import date
-from typing import Any, Callable, Dict, Optional, Type, Union
+from typing import Any, Callable, Dict, List, Optional, Type, Union
 
 from bs4 import Tag
 
@@ -14,6 +14,17 @@ from amazonorders.exception import AmazonOrdersEntityError, AmazonOrdersError
 logger = logging.getLogger(__name__)
 
 
+def _field_name(parse_function: Callable[..., Any],
+                kwargs: Dict[str, Any]) -> str:
+    """
+    The name a field's parse failure is logged and recorded under: the field, or for ``simple_parse`` its selector.
+    """
+    if parse_function.__name__ != "simple_parse":
+        return parse_function.__name__.split("_parse_")[1]
+    selector = kwargs.get("selector", "")
+    return selector if isinstance(selector, str) else ", ".join(selector)
+
+
 class Parsable:
     """
     A base class that contains a parsed representation of the entity, which can be extended to build an entity that
@@ -27,6 +38,9 @@ class Parsable:
         self.parsed: Tag = parsed
         #: The config to use.
         self.config: AmazonOrdersConfig = config
+        #: The fields :func:`safe_parse` could not parse and left ``None`` (the selector, for
+        #: :func:`safe_simple_parse`), so a failed field can be told apart from one that is ``None`` by design.
+        self.parse_failures: List[str] = []
 
     def __getstate__(self) -> Dict:
         state = self.__dict__.copy()
@@ -38,7 +52,7 @@ class Parsable:
         """
         Serialize the entity to a ``dict`` of primitives, suitable for JSON, YAML, or CSV output.
         Nested entities and lists of them are converted recursively, dates become ISO 8601 strings,
-        and the parsed ``Tag`` and the config are omitted.
+        and the parsed ``Tag``, the config, and ``parse_failures`` are omitted.
 
         :param exclude_none: Leave out fields whose value is ``None``, in nested entities too. Only ``None`` is left
             out: ``False``, ``0``, and empty lists are values.
@@ -46,7 +60,7 @@ class Parsable:
         """
         serialized: Dict[str, Any] = {}
         for field, value in vars(self).items():
-            if field in ["parsed", "config"] or (exclude_none and value is None):
+            if field in ["parsed", "config", "parse_failures"] or (exclude_none and value is None):
                 continue
 
             if isinstance(value, Parsable):
@@ -66,7 +80,8 @@ class Parsable:
                    **kwargs: Any) -> Any:
         """
         Execute the given parse function on a field, handling any common parse exceptions and passing
-        them as warnings to the logger (suppressing them as exceptions).
+        them as warnings to the logger (suppressing them as exceptions). A field that fails is listed in
+        :attr:`parse_failures` until it parses.
 
         :param parse_function: The parse function to attempt safe execution.
         :param kwargs: The ``kwargs`` will be passed to ``parse_function``.
@@ -76,19 +91,23 @@ class Parsable:
             raise AmazonOrdersError("The name of the \"parse_function\" passed to this method must start with "
                                     "\"_parse_\".")
 
+        field = _field_name(parse_function, kwargs)
         try:
-            return parse_function(**kwargs)
+            value = parse_function(**kwargs)
         except (AttributeError, IndexError, ValueError, TypeError, RecursionError):
-            function = "simple_parse"
-            if parse_function.__name__ != function:
-                function = parse_function.__name__.split("_parse_")[1]
             logger.warning(
-                "When building {name}, `{function}` "
+                "When building {name}, `{field}` "
                 "could not be parsed.".format(name=self.__class__.__name__,
-                                              function=function),
+                                              field=field),
                 exc_info=True)
+            if field not in self.parse_failures:
+                self.parse_failures.append(field)
             return None
 
+        if field in self.parse_failures:
+            self.parse_failures.remove(field)
+        return value
+
     def simple_parse(self,
                      selector: Union[str, list],
                      attr_name: Optional[str] = None,