From 6e3597070e510659084410bb011774f1442ec519 Mon Sep 17 00:00:00 2001 From: Maria Grimaldi Date: Thu, 16 Nov 2023 13:12:22 -0400 Subject: [PATCH 1/2] fix: cast length only if it's an integer --- filesmanager/filesmanager.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/filesmanager/filesmanager.py b/filesmanager/filesmanager.py index 7321a9f..3efb1bc 100644 --- a/filesmanager/filesmanager.py +++ b/filesmanager/filesmanager.py @@ -746,7 +746,7 @@ def get_asset_json_from_content(self, content): "display_name": content.name, "url": str(asset_url), "content_type": content.content_type, - "file_size": int(content.length), + "file_size": int(content.length) if content.length else 0, "external_url": urljoin(configuration_helpers.get_value('LMS_ROOT_URL', settings.LMS_ROOT_URL), asset_url), "thumbnail": urljoin(configuration_helpers.get_value('LMS_ROOT_URL', settings.LMS_ROOT_URL), thumbnail_url), "uploaded_at": datetime.now().isoformat() From 00aa0116b94b0d240dd09e33bc0aad34180bdd72 Mon Sep 17 00:00:00 2001 From: Maria Grimaldi Date: Thu, 16 Nov 2023 13:20:16 -0400 Subject: [PATCH 2/2] fix: use get instead of indexing --- filesmanager/filesmanager.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/filesmanager/filesmanager.py b/filesmanager/filesmanager.py index 3efb1bc..25556d1 100644 --- a/filesmanager/filesmanager.py +++ b/filesmanager/filesmanager.py @@ -490,7 +490,7 @@ def sync_with_course_assets(self): course_assets_ids = [asset["id"] for asset in self.get_all_serialized_assets()] directories_files = self.get_all_files(self.directories["children"]) for file in directories_files: - if file["metadata"]["id"] not in course_assets_ids: + if file["metadata"].get("id") not in course_assets_ids: self.delete_file_from_directory(file) def delete_file_from_directory(self, file):