[FIX] don't proceed to import if have missing lang

Author: emjay0921

spp_area_base/models/area_import.py

@@ -197,8 +197,21 @@ def parse_excel_to_json(self):
         # Create single job to parse Excel with pandas
         job = self.delayable(channel=_area_import_channel)._scan_and_create_parse_jobs()
         # After parsing, import the data
-        job.on_done(self.delayable(channel=_area_import_channel).import_data())
+        job.on_done(self.delayable(channel=_area_import_channel).after_parse())
         job.delay()
+    
+    def after_parse(self):
+        """
+        After parsing, import the data.
+        """
+        self.ensure_one()
+        with_missing_languages = self._validate_languages_activated()
+        if with_missing_languages:
+            self.locked = True
+            self.locked_reason = with_missing_languages
+            return
+        self.import_data()
+
 
     def _scan_and_create_parse_jobs(self):
         """
@@ -436,7 +449,9 @@ def _validate_languages_activated(self):
             error_message += _("Go to: Settings > Translations > Languages")
 
             _logger.error("Area Import: Missing languages: %s", ", ".join(missing_languages))
-            raise ValidationError(error_message)
+            return error_message
+        
+        return None
 
     def _import_data_from_json(self, json_file_id):
         """

spp_dms/tests/test_dms_directory.py

@@ -70,7 +70,7 @@ def test_action_spp_dms_directories_all_directory(self):
 
         self.assertEqual(action["res_model"], "spp.dms.directory")
         self.assertEqual(action["target"], "current")
-        self.assertEqual(action["view_mode"], "tree,form")
+        self.assertEqual(action["view_mode"], "list,form")
         self.assertEqual(action["context"]["default_parent_id"], self.dms_directry_id.id)
         self.assertEqual(action["context"]["searchpanel_default_parent_id"], self.dms_directry_id.id)
         self.assertEqual(action["domain"], [("parent_id", "child_of", self.dms_directry_id.id)])
@@ -86,7 +86,7 @@ def test_action_spp_dms_files_all_directory(self):
 
         self.assertEqual(action["res_model"], "spp.dms.file")
         self.assertEqual(action["target"], "current")
-        self.assertEqual(action["view_mode"], "tree,form")
+        self.assertEqual(action["view_mode"], "list,form")
         self.assertEqual(action["context"]["default_directory_id"], self.dms_directry_id.id)
         self.assertEqual(action["context"]["searchpanel_default_directory_id"], self.dms_directry_id.id)
         self.assertEqual(action["domain"], [("directory_id", "child_of", self.dms_directry_id.id)])