From 766bc9c6ebe22a29c39c91941f12eed29b85a41d Mon Sep 17 00:00:00 2001 From: Bryan Newbold Date: Thu, 4 Nov 2021 18:01:49 -0700 Subject: wayback cleanup: actually update entity --- python/fatcat_tools/cleanups/file_short_wayback_ts.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'python') diff --git a/python/fatcat_tools/cleanups/file_short_wayback_ts.py b/python/fatcat_tools/cleanups/file_short_wayback_ts.py index ab1b2a5f..d8b8ab9d 100644 --- a/python/fatcat_tools/cleanups/file_short_wayback_ts.py +++ b/python/fatcat_tools/cleanups/file_short_wayback_ts.py @@ -27,7 +27,7 @@ class FileShortWaybackTimestampCleanup(EntityImporter): def __init__(self, api: ApiClient, **kwargs): - eg_desc = kwargs.pop("editgroup_description", None) or "Expand short wayback timestamps" + eg_desc = kwargs.pop("editgroup_description", None) or "Expand trunacted timestamps in wayback URLs" eg_extra = kwargs.pop("editgroup_extra", dict()) eg_extra["agent"] = eg_extra.get( "agent", "fatcat_tools.FileShortWaybackTimestampCleanup" @@ -150,7 +150,9 @@ class FileShortWaybackTimestampCleanup(EntityImporter): # these corrections (entity dump) contains no dupes if not self.testing_mode: - self.api.update_file(self.get_editgroup_id(), existing.ident, existing) + # note: passing 'fe' instead of 'existing' here, which is not + # usually how it goes! + self.api.update_file(self.get_editgroup_id(), fe.ident, fe) self.counts["update"] += 1 return False -- cgit v1.2.3