From 60621084b2df088a5c1ac89a4380db081ee72191 Mon Sep 17 00:00:00 2001 From: Jochen Oberreiter Date: Tue, 11 Aug 2015 11:57:14 +0200 Subject: Fix error introduced by function rename Replace all computeChecksum with compute_checksum. Error was introduced by the recent function name changes without refactoring the source correctly. --- module/plugins/hooks/Checksum.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'module/plugins/hooks') diff --git a/module/plugins/hooks/Checksum.py b/module/plugins/hooks/Checksum.py index 6ecbfcda2..80d93dc39 100644 --- a/module/plugins/hooks/Checksum.py +++ b/module/plugins/hooks/Checksum.py @@ -131,7 +131,7 @@ class Checksum(Addon): for key in self.algorithms: if key in data: - checksum = computeChecksum(local_file, key.replace("-", "").lower()) + checksum = compute_checksum(local_file, key.replace("-", "").lower()) if checksum: if checksum is data[key].lower(): self.log_info(_('File integrity of "%s" verified by %s checksum (%s)') % @@ -186,7 +186,7 @@ class Checksum(Addon): local_file = fs_encode(fs_join(download_folder, data['NAME'])) algorithm = self.methods.get(file_type, file_type) - checksum = computeChecksum(local_file, algorithm) + checksum = compute_checksum(local_file, algorithm) if checksum is data['HASH']: self.log_info(_('File integrity of "%s" verified by %s checksum (%s)') % (data['NAME'], algorithm, checksum)) -- cgit v1.2.3 From f67e83d159df8cfcd745295517cda239563f65c7 Mon Sep 17 00:00:00 2001 From: Jochen Oberreiter Date: Tue, 11 Aug 2015 14:16:55 +0200 Subject: Change comparison from identity to equality Using '==' instead of 'is' for the comparison. --- module/plugins/hooks/Checksum.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'module/plugins/hooks') diff --git a/module/plugins/hooks/Checksum.py b/module/plugins/hooks/Checksum.py index 80d93dc39..d47c561eb 100644 --- a/module/plugins/hooks/Checksum.py +++ b/module/plugins/hooks/Checksum.py @@ -38,7 +38,7 @@ def compute_checksum(local_file, algorithm): class Checksum(Addon): __name__ = "Checksum" __type__ = "hook" - __version__ = "0.19" + __version__ = "0.20" __status__ = "testing" __config__ = [("check_checksum", "bool" , "Check checksum? (If False only size will be verified)", True ), @@ -133,7 +133,7 @@ class Checksum(Addon): if key in data: checksum = compute_checksum(local_file, key.replace("-", "").lower()) if checksum: - if checksum is data[key].lower(): + if checksum == data[key].lower(): self.log_info(_('File integrity of "%s" verified by %s checksum (%s)') % (pyfile.name, key.upper(), checksum)) break -- cgit v1.2.3 From 4532ccc304ba666d79b1cc783a11eeb6f8e7cedc Mon Sep 17 00:00:00 2001 From: Jochen Oberreiter Date: Tue, 11 Aug 2015 14:33:12 +0200 Subject: Minor change to log output Using `data[key].lower()` instead of `data[key]` to synchronize it with the comparison itself. --- module/plugins/hooks/Checksum.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'module/plugins/hooks') diff --git a/module/plugins/hooks/Checksum.py b/module/plugins/hooks/Checksum.py index d47c561eb..da4d35df1 100644 --- a/module/plugins/hooks/Checksum.py +++ b/module/plugins/hooks/Checksum.py @@ -139,7 +139,7 @@ class Checksum(Addon): break else: self.log_warning(_("%s checksum for file %s does not match (%s != %s)") % - (key.upper(), pyfile.name, checksum, data[key])) + (key.upper(), pyfile.name, checksum, data[key].lower())) self.check_failed(pyfile, local_file, "Checksums do not match") else: self.log_warning(_("Unsupported hashing algorithm"), key.upper()) -- cgit v1.2.3