folio-migration-tools 1.9.8__tar.gz → 1.9.10__tar.gz

{folio_migration_tools-1.9.8 → folio_migration_tools-1.9.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: folio_migration_tools
-Version: 1.9.8
+Version: 1.9.10
 Summary: A tool allowing you to migrate data from legacy ILS:s (Library systems) into FOLIO LSP
 License-Expression: MIT
 License-File: LICENSE
@@ -19,7 +19,7 @@ Requires-Dist: argparse-prompt (>=0.0.5,<0.0.6)
 Requires-Dist: art (>=6.5,<7.0)
 Requires-Dist: deepdiff (>=6.2.3,<7.0.0)
 Requires-Dist: defusedxml (>=0.7.1,<0.8.0)
-Requires-Dist: folio-data-import (>=0.3.2,<0.4.0)
+Requires-Dist: folio-data-import (>=0.4.1)
 Requires-Dist: folio-uuid (>=1.0.0,<2.0.0)
 Requires-Dist: folioclient (>=0.70.1,<0.71.0)
 Requires-Dist: pyaml (>=21.10.1,<22.0.0)

{folio_migration_tools-1.9.8 → folio_migration_tools-1.9.10}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "folio_migration_tools"
-version = "1.9.8"
+version = "1.9.10"
 description =  "A tool allowing you to migrate data from legacy ILS:s (Library systems) into FOLIO LSP"
 authors = [
 	{name = "Theodor Tolstoy", email = "github.teddes@tolstoy.se"},
@@ -56,7 +56,7 @@ deepdiff = "^6.2.3"
 pyaml = "^21.10.1"
 python-i18n = "^0.3.9"
 art = "^6.5"
-folio-data-import = "^0.3.2"
+folio-data-import = ">=0.4.1"
 
 [tool.poetry.group.dev.dependencies]
 pytest = "^7.1.3"

{folio_migration_tools-1.9.8 → folio_migration_tools-1.9.10}/src/folio_migration_tools/mapping_file_transformation/order_mapper.py

@@ -78,6 +78,14 @@ class CompositeOrderMapper(MappingFileMapperBase):
             "code",
             "OrderLineLocationMapping",
         )
+        self.funds_mapping = RefDataMapping(
+            self.folio_client,
+            "/finance/funds",
+            "funds",
+            funds_map,
+            "code",
+            "FundsMapping",
+        )
 
         self.folio_client: FolioClient = folio_client
         self.notes_mapper: NotesMapper = NotesMapper(
@@ -115,6 +123,15 @@ class CompositeOrderMapper(MappingFileMapperBase):
                 False,
             )
 
+        if folio_prop_name.endswith(".fundId"):
+            return self.get_mapped_ref_data_value(
+                self.location_mapping,
+                legacy_order,
+                folio_prop_name,
+                index_or_id,
+                False,
+            )
+
         mapped_value = super().get_prop(
             legacy_order, folio_prop_name, index_or_id, schema_default_value
         )

{folio_migration_tools-1.9.8 → folio_migration_tools-1.9.10}/src/folio_migration_tools/mapping_file_transformation/ref_data_mapping.py

@@ -43,6 +43,9 @@ class RefDataMapping(object):
         return self.cached_dict.get(key_value.lower().strip(), ())
 
     def setup_mappings(self):
+        if not self.map:
+            logging.info("%s legacy map file is empty or not provided", self.name)
+            return
         self.pre_validate_map()
         for idx, mapping in enumerate(self.map):
             try:

{folio_migration_tools-1.9.8 → folio_migration_tools-1.9.10}/src/folio_migration_tools/migration_tasks/migration_task_base.py

@@ -418,10 +418,14 @@ class MigrationTaskBase:
             required (bool): Whether the property is required or not
         """
         if (
-            folio_property_name in folio_keys
-            or required
-            or folio_property_name.startswith("statisticalCodeIds")
-            or folio_property_name.startswith("locationMap")
+                (
+	            folio_property_name in folio_keys
+	            or required
+	            or folio_property_name.startswith("statisticalCodeIds")
+	            or folio_property_name.startswith("locationMap")
+	            or folio_property_name.startswith("fundsMap")
+                )
+                and map_file_path.is_file()
         ):
             try:
                 with open(map_file_path) as map_file:

{folio_migration_tools-1.9.8 → folio_migration_tools-1.9.10}/src/folio_migration_tools/migration_tasks/orders_transformer.py

@@ -216,9 +216,10 @@ class OrdersTransformer(MigrationTaskBase):
             ),
             self.load_ref_data_mapping_file(  # Required if there was is a fund.
                 "fundsMap",
-                self.folder_structure.mapping_files_folder / self.task_config.funds_map_file_name,
+                self.folder_structure.mapping_files_folder 
+                / self.task_config.funds_map_file_name,
                 self.folio_keys,
-                False,
+                True,
             ),
             self.load_ref_data_mapping_file(  # Todo: The property in the schema has no type
                 "fundsExpenseClassMap",

{folio_migration_tools-1.9.8 → folio_migration_tools-1.9.10}/src/folio_migration_tools/translations/en.json

@@ -318,6 +318,8 @@
   "blurbs.FieldMappingErrors.title": "Field mapping errors",
   "blurbs.FolioDefaultValuesAdded.description": "The below FOLIO default values were added to records that had no mapped value in the source data.",
   "blurbs.FolioDefaultValuesAdded.title": "FOLIO default values added",
+  "blurbs.FundsMapping.title": "Fund mappings",
+  "blurbs.FundsMapping.description": "Reference data for funds mapping",
   "blurbs.GeneralStatistics.description": "A list of general counters to outline the transformation as a whole.",
   "blurbs.GeneralStatistics.title": "General statistics",
   "blurbs.HoldingsGenerationFromBibs.description": "Some libraries have Holdings/MFHD information baked into their bib records. The following breakdown gives an idea on the occurrence of 852/866 combinations",
@@ -471,4 +473,4 @@
   "naturalId mapped from %{fro}": "naturalId mapped from %{fro}",
   "no matching identifier_types in %{names}": "no matching identifier_types in %{names}",
   "subfield present in %{linked_value_tag} but not in %{pattern_field}": "subfield present in %{linked_value_tag} but not in %{pattern_field}"
-}
+}