Fix several UnboundLocalError exceptions when doing automated impor…

…t with `return_json=True`
collective · Jul 5, 2024 · 968eaaf · 968eaaf
1 parent 6f8834b
commit 968eaaf
Showing 1 changed file with 16 additions and 33 deletions.
diff --git a/src/collective/exportimport/import_other.py b/src/collective/exportimport/import_other.py
@@ -88,11 +88,8 @@ def __call__(self, jsonfile=None, return_json=False):
                 except Exception as e:
                     logger.error(e)
                     status = "error"
-                    msg = e
-                    api.portal.show_message(
-                        _(u"Failure while uploading: {}").format(e),
-                        request=self.request,
-                    )
+                    msg = _(u"Failure while uploading: {}").format(e)
+                    api.portal.show_message(msg, request=self.request)
                 else:
                     msg = self.do_import(data)
                     api.portal.show_message(msg, self.request)
@@ -194,10 +191,8 @@ def __call__(self, jsonfile=None, return_json=False):
             except Exception as e:
                 status = "error"
                 logger.error(e)
-                api.portal.show_message(
-                    _(u"Failure while uploading: {}").format(e),
-                    request=self.request,
-                )
+                msg = _(u"Failure while uploading: {}").format(e)
+                api.portal.show_message(msg, request=self.request)
             else:
                 groups = self.import_groups(data["groups"])
                 members = self.import_members(data["members"])
@@ -385,10 +380,8 @@ def __call__(self, jsonfile=None, return_json=False):
             except Exception as e:
                 status = "error"
                 logger.error(e)
-                api.portal.show_message(
-                    _(u"Failure while uploading: {}").format(e),
-                    request=self.request,
-                )
+                msg = _(u"Failure while uploading: {}").format(e)
+                api.portal.show_message(msg, request=self.request)
             else:
                 localroles = self.import_localroles(data)
                 msg = _(u"Imported {} localroles").format(localroles)
@@ -461,10 +454,8 @@ def __call__(self, jsonfile=None, return_json=False):
             except Exception as e:
                 status = "error"
                 logger.error(e)
-                api.portal.show_message(
-                    _(u"Failure while uploading: {}").format(e),
-                    request=self.request,
-                )
+                msg = _(u"Failure while uploading: {}").format(e)
+                api.portal.show_message(msg, request=self.request)
             else:
                 start = datetime.now()
                 orders = self.import_ordering(data)
@@ -519,10 +510,8 @@ def __call__(self, jsonfile=None, return_json=False):
             except Exception as e:
                 status = "error"
                 logger.error(e)
-                api.portal.show_message(
-                    u"Failure while uploading: {}".format(e),
-                    request=self.request,
-                )
+                msg = u"Failure while uploading: {}".format(e)
+                api.portal.show_message(msg, request=self.request)
             else:
                 defaultpages = self.import_default_pages(data)
                 msg = _(u"Changed {} default pages").format(defaultpages)
@@ -592,10 +581,8 @@ def __call__(self, jsonfile=None, return_json=False):
             except Exception as e:
                 status = "error"
                 logger.error(e)
-                api.portal.show_message(
-                    _(u"Failure while uploading: {}").format(e),
-                    request=self.request,
-                )
+                msg = _(u"Failure while uploading: {}").format(e)
+                api.portal.show_message(msg, request=self.request)
             else:
                 results = self.import_data(data)
                 msg = _(u"Imported {} comments").format(results)
@@ -688,10 +675,8 @@ def __call__(self, jsonfile=None, return_json=False):
             except Exception as e:
                 status = "error"
                 logger.error(e)
-                api.portal.show_message(
-                    _(u"Failure while uploading: {}").format(e),
-                    request=self.request,
-                )
+                msg = _(u"Failure while uploading: {}").format(e)
+                api.portal.show_message(msg, request=self.request)
             else:
                 portlets = self.import_portlets(data)
                 msg = _(u"Created {} portlets").format(portlets)
@@ -875,10 +860,8 @@ def __call__(self, jsonfile=None, return_json=False):
             except Exception as e:
                 status = "error"
                 logger.error(e)
-                api.portal.show_message(
-                    _(u"Failure while uploading: {}").format(e),
-                    request=self.request,
-                )
+                msg = _(u"Failure while uploading: {}").format(e)
+                api.portal.show_message(msg, request=self.request)
             else:
                 import_plone_redirects(data)
                 msg = _(u"Redirects imported")