Renamer warns once per property only after all roots miss 🟩

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Daniel Roth 2026-07-03 11:18:42 +00:00
parent ce749061aa
commit 213cd92998

View file

@ -97,6 +97,11 @@ class SharepointRenamerOrchestrator:
)
if self._process_folder(folder_path, uprn, address, postcode):
break
else:
logger.warning(
f"Missing folder for UPRN {uprn} in any root: "
f"{address}, {postcode}"
)
def _process_folder(
self, folder_path: str, uprn: str, address: str, postcode: str
@ -104,7 +109,6 @@ class SharepointRenamerOrchestrator:
try:
contents = self._sp_client.get_folders_in_path(folder_path)
except ValueError:
logger.warning(f"Missing folder for UPRN {uprn}: {folder_path}")
return False
for item in contents.get("value", []):