netcmd: models: check for None in build_expression instead

Signed-off-by: Rob van der Linde <rob@catalyst.net.nz>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/python/samba/netcmd/domain/models/model.py b/python/samba/netcmd/domain/models/model.py
index 8d68c47075b2608b737ad233d9ff59816eb8e5ad..2c63392caa83cc6465364c83b943223626abea8b 100644 (file)
--- a/python/samba/netcmd/domain/models/model.py
+++ b/python/samba/netcmd/domain/models/model.py
@@ -219,7 +219,7 @@ class Model(metaclass=ModelMeta):
 
         for field_name, value in criteria.items():
             field = cls.fields.get(field_name)
-            if not field:
+            if field is None:
                 raise ValueError(f"Unknown field '{field_name}'")
             expression += f"({field.name}={binary_encode(value)})"