fix: adding exception handling

openedx · Nov 27, 2023 · 0245e2b · 0245e2b
1 parent d77eb45
commit 0245e2b
Showing 1 changed file with 15 additions and 8 deletions.
diff --git a/..._subsidy/apps/transaction/management/commands/backpopulate_transaction_email_and_title.py b/..._subsidy/apps/transaction/management/commands/backpopulate_transaction_email_and_title.py
@@ -53,14 +53,21 @@ def process_transaction(self, subsidy, txn):
         """
         logger.info(f"Processing {subsidy.uuid} transaction {txn.uuid}")
 
-        if txn.lms_user_email is None and txn.lms_user_id is not None:
-            lms_user_email = subsidy.email_for_learner(txn.lms_user_id)
-            txn.lms_user_email = lms_user_email
-            logger.info(f"Found {lms_user_email} for {subsidy.uuid} transaction {txn.uuid}")
-        if txn.content_title is None and txn.content_key is not None:
-            content_title = subsidy.title_for_content(txn.content_key)
-            txn.content_title = content_title
-            logger.info(f"Found {content_title} for {subsidy.uuid} transaction {txn.uuid}")
+        try:
+            if txn.lms_user_email is None and txn.lms_user_id is not None:
+                lms_user_email = subsidy.email_for_learner(txn.lms_user_id)
+                txn.lms_user_email = lms_user_email
+                logger.info(f"Found {lms_user_email} for {subsidy.uuid} transaction {txn.uuid}")
+        except Exception as e:  # pylint: disable=broad-exception-caught
+            logger.exception(f"Error while processing lms_user_email for {subsidy.uuid} transaction {txn.uuid}: {e}")
+
+        try:
+            if txn.content_title is None and txn.content_key is not None:
+                content_title = subsidy.title_for_content(txn.content_key)
+                txn.content_title = content_title
+                logger.info(f"Found {content_title} for {subsidy.uuid} transaction {txn.uuid}")
+        except Exception as e:  # pylint: disable=broad-exception-caught
+            logger.exception(f"Error while processing content_title for {subsidy.uuid} transaction {txn.uuid}: {e}")
 
         if not self.dry_run:
             txn.save()