Skip to content

Commit aabb36f

Browse files
committed
Simplify expression
1 parent cbef79e commit aabb36f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

minfraud/validation.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -399,8 +399,8 @@ def _transaction_id(s: Optional[str]) -> str:
399399

400400

401401
def _validate_at_least_one_identifier_field(report):
402-
optional_fields = ["ip_address", "maxmind_id", "minfraud_id", "transaction_id"]
403-
if not any(field in report for field in optional_fields):
402+
optional_fields = {"ip_address", "maxmind_id", "minfraud_id", "transaction_id"}
403+
if not optional_fields & report.keys():
404404
# We return MultipleInvalid instead of ValueError to be consistent with what
405405
# voluptuous returns.
406406
raise MultipleInvalid(

0 commit comments

Comments
 (0)