folio-migration-tools 1.9.0rc9__py3-none-any.whl → 1.9.0rc11__py3-none-any.whl

folio_migration_tools/marc_rules_transformation/rules_mapper_holdings.py

@@ -1,7 +1,7 @@
 import copy
 import json
 import logging
-from typing import List
+from typing import Dict, List, Set
 
 import i18n
 from folio_uuid.folio_namespaces import FOLIONamespaces
@@ -34,7 +34,7 @@ from folio_migration_tools.marc_rules_transformation.rules_mapper_base import (
 class RulesMapperHoldings(RulesMapperBase):
     def __init__(
         self,
-        folio_client,
+        folio_client: FolioClient,
         location_map,
         task_configuration,
         library_configuration: LibraryConfiguration,
@@ -235,10 +235,10 @@ class RulesMapperHoldings(RulesMapperBase):
 
     def process_marc_field(
         self,
-        folio_holding: dict,
+        folio_holding: Dict,
         marc_field: Field,
-        ignored_subsequent_fields,
-        index_or_legacy_ids,
+        ignored_subsequent_fields: Set,
+        index_or_legacy_ids: List[str],
     ):
         """This overwrites the implementation for Auth and instances
 
@@ -261,7 +261,7 @@ class RulesMapperHoldings(RulesMapperBase):
                 ignored_subsequent_fields.add(marc_field.tag)
 
     def perform_additional_mapping(
-        self, marc_record: Record, folio_holding, legacy_ids: List[str], file_def: FileDefinition
+        self, marc_record: Record, folio_holding: Dict, legacy_ids: List[str], file_def: FileDefinition
     ):
         """_summary_
 
@@ -278,12 +278,11 @@ class RulesMapperHoldings(RulesMapperBase):
         self.set_default_call_number_type_if_empty(folio_holding)
         self.pick_first_location_if_many(folio_holding, legacy_ids)
         self.parse_coded_holdings_statements(marc_record, folio_holding, legacy_ids)
+        self.add_mfhd_as_mrk_note(marc_record, folio_holding, legacy_ids)
+        self.add_mfhd_as_mrc_note(marc_record, folio_holding, legacy_ids)
         HoldingsHelper.handle_notes(folio_holding)
-        create_source_records = all(
-            [file_def.create_source_records, self.task_configuration.create_source_records]
-        )
         if (
-            create_source_records
+            all([file_def.create_source_records, self.create_source_records])
             or self.task_configuration.hrid_handling == HridHandling.preserve001
         ):
             self.hrid_handler.handle_hrid(
@@ -298,9 +297,9 @@ class RulesMapperHoldings(RulesMapperBase):
                 "",
             )
         self.handle_suppression(folio_holding, file_def, True)
-        self.set_source_id(self.task_configuration, folio_holding, self.holdingssources, file_def)
+        self.set_source_id(self.create_source_records, folio_holding, self.holdingssources, file_def)
 
-    def pick_first_location_if_many(self, folio_holding, legacy_ids: List[str]):
+    def pick_first_location_if_many(self, folio_holding: Dict, legacy_ids: List[str]):
         if " " in folio_holding.get("permanentLocationId", ""):
             Helper.log_data_issue(
                 legacy_ids,
@@ -312,14 +311,14 @@ class RulesMapperHoldings(RulesMapperBase):
             ]
 
     @staticmethod
-    def set_source_id(task_configuration, folio_rec, holdingssources, file_def):
-        if file_def.create_source_records and task_configuration.create_source_records:
+    def set_source_id(create_source_records: bool, folio_rec: Dict, holdingssources: Dict, file_def: FileDefinition):
+        if file_def.create_source_records and create_source_records:
             folio_rec["sourceId"] = holdingssources.get("MARC")
         else:
             folio_rec["sourceId"] = holdingssources.get("FOLIO")
 
     def parse_coded_holdings_statements(
-        self, marc_record: Record, folio_holding, legacy_ids: List[str]
+        self, marc_record: Record, folio_holding: Dict, legacy_ids: List[str]
     ):
         # TODO: Should one be able to switch these things off?
         a = {
@@ -344,6 +343,117 @@ class RulesMapperHoldings(RulesMapperBase):
             except TransformationFieldMappingError as tfme:
                 Helper.log_data_issue(tfme.index_or_id, tfme.message, tfme.data_value)
                 self.migration_report.add("FieldMappingErrors", tfme.message)
+        self.collect_mrk_statement_notes(marc_record, folio_holding, legacy_ids)
+
+    def collect_mrk_statement_notes(self, marc_record, folio_holding, legacy_ids):
+        """Collects MFHD holdings statements as MARC Maker field strings in a FOLIO holdings note
+        and adds them to the FOLIO holdings record.
+
+        This is done to preserve the information in the MARC record for future reference.
+
+        Args:
+            marc_record (Record): PyMARC record
+            folio_holding (Dict): FOLIO holdings record
+
+        """
+        if self.task_configuration.include_mrk_statements:
+            mrk_statement_notes = []
+            for field in marc_record.get_fields("853", "854", "855", "863", "864", "865", "866", "867", "868"):
+                mrk_statement_notes.append(str(field))
+            if mrk_statement_notes:
+                folio_holding["notes"] = folio_holding.get("notes", []) + self.add_mrk_statements_note(mrk_statement_notes, legacy_ids)
+
+    def add_mrk_statements_note(self, mrk_statement_notes: List[str], legacy_ids) -> List[Dict]:
+        """Creates a note from the MRK statements
+
+        Args:
+            mrk_statement_notes (List[str]): A list of MFHD holdings statements as MRK strings
+
+        Returns:
+            List: A list containing the FOLIO holdings note object (Dict)
+        """
+        holdings_note_type_tuple = self.conditions.get_ref_data_tuple_by_name(
+            self.folio.holding_note_types, "holding_note_types", self.task_configuration.mrk_holdings_note_type
+        )
+        try:
+            holdings_note_type_id = holdings_note_type_tuple[0]
+        except Exception as ee:
+            logging.error(ee)
+            raise TransformationRecordFailedError(
+                legacy_ids,
+                f'Holdings note type mapping error.\tNote type name: {self.task_configuration.mrk_holdings_note_type}\t'
+                f"MFHD holdings statement note type not found in FOLIO.",
+                self.task_configuration.mrk_holdings_note_type,
+            ) from ee
+        return [
+            {
+                "note": "\n".join(mrk_statement_notes),
+                "holdingsNoteTypeId": holdings_note_type_id,
+                "staffOnly": True,
+            }
+        ]
+
+    def add_mfhd_as_mrk_note(self, marc_record: Record, folio_holding: Dict, legacy_ids: List[str]):
+        """Adds the MFHD as a note to the holdings record
+
+        This is done to preserve the information in the MARC record for future reference.
+
+        Args:
+            marc_record (Record): PyMARC record
+            folio_holding (Dict): FOLIO holdings record
+        """
+        holdings_note_type_tuple = self.conditions.get_ref_data_tuple_by_name(
+            self.folio.holding_note_types, "holding_note_types", self.task_configuration.mfhd_mrk_note_type
+        )
+        try:
+            holdings_note_type_id = holdings_note_type_tuple[0]
+        except Exception as ee:
+            logging.error(ee)
+            raise TransformationRecordFailedError(
+                legacy_ids,
+                f'Holdings note type mapping error.\tNote type name: {self.task_configuration.mfhd_mrk_note_type}\t'
+                f"Note type not found in FOLIO.",
+                self.task_configuration.mfhd_mrk_note_type,
+            ) from ee
+        if self.task_configuration.include_mfhd_mrk_as_note:
+            folio_holding["notes"] = folio_holding.get("notes", []) + [
+                {
+                    "note": str(marc_record),
+                    "holdingsNoteTypeId": holdings_note_type_id,
+                    "staffOnly": True,
+                }
+            ]
+
+    def add_mfhd_as_mrc_note(self, marc_record: Record, folio_holding: Dict, legacy_ids: List[str]):
+        """Adds the MFHD as a note to the holdings record
+
+        This is done to preserve the information in the MARC record for future reference.
+
+        Args:
+            marc_record (Record): PyMARC record
+            folio_holding (Dict): FOLIO holdings record
+        """
+        holdings_note_type_tuple = self.conditions.get_ref_data_tuple_by_name(
+            self.folio.holding_note_types, "holding_note_types", self.task_configuration.mfhd_mrc_note_type
+        )
+        try:
+            holdings_note_type_id = holdings_note_type_tuple[0]
+        except Exception as ee:
+            logging.error(ee)
+            raise TransformationRecordFailedError(
+                legacy_ids,
+                f'Holdings note type mapping error.\tNote type name: {self.task_configuration.mfhd_mrc_note_type}\t'
+                f"Note type not found in FOLIO.",
+                self.task_configuration.mfhd_mrc_note_type,
+            ) from ee
+        if self.task_configuration.include_mfhd_mrc_as_note:
+            folio_holding["notes"] = folio_holding.get("notes", []) + [
+                {
+                    "note": marc_record.as_marc().decode("utf-8"),
+                    "holdingsNoteTypeId": holdings_note_type_id,
+                    "staffOnly": True,
+                }
+            ]
 
     def wrap_up(self):
         logging.info("Mapper wrapping up")
@@ -351,7 +461,7 @@ class RulesMapperHoldings(RulesMapperBase):
             x.create_source_records for x in self.task_configuration.files
         ]
         if all(source_file_create_source_records):
-            create_source_records = self.task_configuration.create_source_records
+            create_source_records = self.create_source_records
         else:
             logging.info(
                 "If all source files have create_source_records set to false, "
@@ -369,7 +479,7 @@ class RulesMapperHoldings(RulesMapperBase):
         logging.info("Fetching HoldingsRecord schema...")
         return folio_client.get_holdings_schema()
 
-    def set_holdings_type(self, marc_record: Record, folio_holding, legacy_ids: List[str]):
+    def set_holdings_type(self, marc_record: Record, folio_holding: Dict, legacy_ids: List[str]):
         # Holdings type mapping
         ldr06 = marc_record.leader[6]
         # TODO: map this better
@@ -425,7 +535,7 @@ class RulesMapperHoldings(RulesMapperBase):
                     ldr06,
                 )
 
-    def set_default_call_number_type_if_empty(self, folio_holding):
+    def set_default_call_number_type_if_empty(self, folio_holding: Dict):
         if not folio_holding.get("callNumberTypeId", ""):
             folio_holding["callNumberTypeId"] = self.conditions.default_call_number_type["id"]
 
@@ -462,7 +572,7 @@ class RulesMapperHoldings(RulesMapperBase):
             )
         return results
 
-    def verity_boundwith_map_entry(self, entry):
+    def verity_boundwith_map_entry(self, entry: Dict):
         if "MFHD_ID" not in entry or not entry.get("MFHD_ID", ""):
             raise TransformationProcessError(
                 "", "Column MFHD_ID missing from Boundwith relationship map", ""
@@ -472,7 +582,7 @@ class RulesMapperHoldings(RulesMapperBase):
                 "", "Column BIB_ID missing from Boundwith relationship map", ""
             )
 
-    def setup_boundwith_relationship_map(self, boundwith_relationship_map):
+    def setup_boundwith_relationship_map(self, boundwith_relationship_map: List[Dict]):
         """
         Creates a map of MFHD_ID to BIB_ID for boundwith relationships.
 
@@ -503,7 +613,7 @@ class RulesMapperHoldings(RulesMapperBase):
                 self.handle_transformation_record_failed_error(idx, trfe)
         return new_map
 
-    def get_bw_instance_id_map_tuple(self, entry):
+    def get_bw_instance_id_map_tuple(self, entry: Dict):
         try:
             return self.parent_id_map[entry["BIB_ID"]]
         except KeyError:

folio_migration_tools/migration_tasks/batch_poster.py

@@ -287,7 +287,7 @@ class BatchPoster(MigrationTaskBase):
         fetch_batch_size = 90
         fetch_tasks = []
         updates = {}
-        async with httpx.AsyncClient(base_url=self.folio_client.okapi_url) as client:
+        async with httpx.AsyncClient(base_url=self.folio_client.gateway_url) as client:
             for i in range(0, len(batch), fetch_batch_size):
                 batch_slice = batch[i:i + fetch_batch_size]
                 fetch_tasks.append(
@@ -360,7 +360,7 @@ class BatchPoster(MigrationTaskBase):
     def post_extra_data(self, row: str, num_records: int, failed_recs_file):
         (object_name, data) = row.split("\t")
         endpoint = self.get_extradata_endpoint(self.task_configuration, object_name, data)
-        url = f"{self.folio_client.okapi_url}/{endpoint}"
+        url = f"{self.folio_client.gateway_url}/{endpoint}"
         body = data
         response = self.post_objects(url, body)
         if response.status_code == 201:
@@ -415,7 +415,7 @@ class BatchPoster(MigrationTaskBase):
         if self.api_info["is_batch"]:
             raise TypeError("This record type supports batch processing, use post_batch method")
         api_endpoint = self.api_info.get("api_endpoint")
-        url = f"{self.folio_client.okapi_url}{api_endpoint}"
+        url = f"{self.folio_client.gateway_url}{api_endpoint}"
         response = self.post_objects(url, row)
         if response.status_code == 201:
             self.num_posted += 1
@@ -590,7 +590,7 @@ class BatchPoster(MigrationTaskBase):
 
     def do_post(self, batch):
         path = self.api_info["api_endpoint"]
-        url = self.folio_client.okapi_url + path
+        url = self.folio_client.gateway_url + path
         if self.api_info["object_name"] == "users":
             payload = {self.api_info["object_name"]: list(batch), "totalRecords": len(batch)}
         elif self.api_info["total_records"]:
@@ -675,7 +675,7 @@ class BatchPoster(MigrationTaskBase):
             "processingStartedDate": datetime.utcnow().isoformat(timespec="milliseconds"),
         }
         try:
-            url = f"{self.folio_client.okapi_url}/source-storage/snapshots"
+            url = f"{self.folio_client.gateway_url}/source-storage/snapshots"
             if self.http_client and not self.http_client.is_closed:
                 res = self.http_client.post(
                     url, json=snapshot, headers=self.folio_client.okapi_headers
@@ -684,7 +684,7 @@ class BatchPoster(MigrationTaskBase):
                 res = httpx.post(url, headers=self.okapi_headers, json=snapshot, timeout=None)
             res.raise_for_status()
             logging.info("Posted Snapshot to FOLIO: %s", json.dumps(snapshot, indent=4))
-            get_url = f"{self.folio_client.okapi_url}/source-storage/snapshots/{self.snapshot_id}"
+            get_url = f"{self.folio_client.gateway_url}/source-storage/snapshots/{self.snapshot_id}"
             getted = False
             while not getted:
                 logging.info("Sleeping while waiting for the snapshot to get created")
@@ -704,7 +704,7 @@ class BatchPoster(MigrationTaskBase):
     def commit_snapshot(self):
         snapshot = {"jobExecutionId": self.snapshot_id, "status": "COMMITTED"}
         try:
-            url = f"{self.folio_client.okapi_url}/source-storage/snapshots/{self.snapshot_id}"
+            url = f"{self.folio_client.gateway_url}/source-storage/snapshots/{self.snapshot_id}"
             if self.http_client and not self.http_client.is_closed:
                 res = self.http_client.put(
                     url, json=snapshot, headers=self.folio_client.okapi_headers

folio_migration_tools/migration_tasks/bibs_transformer.py

@@ -18,39 +18,17 @@ from folio_migration_tools.marc_rules_transformation.marc_file_processor import
 from folio_migration_tools.marc_rules_transformation.rules_mapper_bibs import (
     BibsRulesMapper,
 )
-from folio_migration_tools.migration_tasks.migration_task_base import MigrationTaskBase
-from folio_migration_tools.task_configuration import AbstractTaskConfiguration
+from folio_migration_tools.migration_tasks.migration_task_base import MarcTaskConfigurationBase, MigrationTaskBase
 
 
 class BibsTransformer(MigrationTaskBase):
-    class TaskConfiguration(AbstractTaskConfiguration):
-        name: Annotated[
-            str,
-            Field(
-                description=(
-                    "Name of this migration task. The name is being used to call the specific "
-                    "task, and to distinguish tasks of similar types"
-                )
-            ),
-        ]
-        migration_task_type: Annotated[
-            str,
-            Field(
-                title="Migration task type",
-                description=("The type of migration task you want to perform."),
-            ),
-        ]
-        files: Annotated[
-            List[FileDefinition],
-            Field(
-                title="Source files",
-                description=("List of MARC21 files with bibliographic records."),
-            ),
-        ]
+    class TaskConfiguration(MarcTaskConfigurationBase):
         ils_flavour: Annotated[
             IlsFlavour,
             Field(
-                title="ILS flavour", description="The type of ILS you are migrating records from."
+                title="ILS flavour",
+                description="The type of ILS you are migrating records from.",
+                alias="ils_flavor"
             ),
         ]
         custom_bib_id_field: Annotated[
@@ -87,16 +65,6 @@ class BibsTransformer(MigrationTaskBase):
                 ),
             ),
         ] = []
-        create_source_records: Annotated[
-            bool,
-            Field(
-                title="Create source records",
-                description=(
-                    "Controls wheter or not to retain the MARC records in "
-                    "Source Record Storage."
-                ),
-            ),
-        ] = True
         data_import_marc: Annotated[
             bool,
             Field(
@@ -107,7 +75,7 @@ class BibsTransformer(MigrationTaskBase):
                     "of FOLIO instance records (and optional SRS records) will be generated."
                 ),
             )
-        ] = False
+        ] = True
         parse_cataloged_date: Annotated[
             bool,
             Field(
@@ -118,17 +86,6 @@ class BibsTransformer(MigrationTaskBase):
                 ),
             ),
         ] = False
-        hrid_handling: Annotated[
-            HridHandling,
-            Field(
-                title="HRID Handling",
-                description=(
-                    "Setting to default will make FOLIO generate HRIDs and move the existing "
-                    "001:s into a 035, concatenated with the 003. Choosing preserve001 means "
-                    "the 001:s will remain in place, and that they will also become the HRIDs"
-                ),
-            ),
-        ] = HridHandling.default
         reset_hrid_settings: Annotated[
             bool,
             Field(
@@ -146,16 +103,6 @@ class BibsTransformer(MigrationTaskBase):
                 description="At the end of the run, update FOLIO with the HRID settings",
             ),
         ] = True
-        deactivate035_from001: Annotated[
-            bool,
-            Field(
-                title="Create 035 from 001 and 003",
-                description=(
-                    "This deactivates the FOLIO default functionality of moving the previous 001 "
-                    "into a 035, prefixed with the value from 003"
-                ),
-            ),
-        ] = False
 
     @staticmethod
     def get_object_type() -> FOLIONamespaces:

folio_migration_tools/migration_tasks/holdings_marc_transformer.py

@@ -19,12 +19,11 @@ from folio_migration_tools.library_configuration import (
 from folio_migration_tools.marc_rules_transformation.rules_mapper_holdings import (
     RulesMapperHoldings,
 )
-from folio_migration_tools.migration_tasks.migration_task_base import MigrationTaskBase
-from folio_migration_tools.task_configuration import AbstractTaskConfiguration
+from folio_migration_tools.migration_tasks.migration_task_base import MarcTaskConfigurationBase, MigrationTaskBase
 
 
 class HoldingsMarcTransformer(MigrationTaskBase):
-    class TaskConfiguration(AbstractTaskConfiguration):
+    class TaskConfiguration(MarcTaskConfigurationBase):
         name: Annotated[
             str,
             Field(
@@ -59,16 +58,6 @@ class HoldingsMarcTransformer(MigrationTaskBase):
                 ),
             ),
         ] = HridHandling.default
-        deactivate035_from001: Annotated[
-            bool,
-            Field(
-                title="Create 035 from 001 and 003",
-                description=(
-                    "This deactivates the FOLIO default functionality of moving the previous 001 "
-                    "into a 035, prefixed with the value from 003"
-                ),
-            ),
-        ] = False
         holdings_type_uuid_for_boundwiths: Annotated[
             str,
             Field(
@@ -89,16 +78,6 @@ class HoldingsMarcTransformer(MigrationTaskBase):
                 ),
             ),
         ] = ""
-        create_source_records: Annotated[
-            bool,
-            Field(
-                title="Create source records",
-                description=(
-                    "Controls wheter or not to retain the MARC records in "
-                    "Source Record Storage."
-                ),
-            ),
-        ] = True
         update_hrid_settings: Annotated[
             bool,
             Field(
@@ -164,6 +143,65 @@ class HoldingsMarcTransformer(MigrationTaskBase):
                 description="The name of the file in the mapping_files directory containing supplemental MFHD mapping rules",
             ),
         ] = ""
+        include_mrk_statements: Annotated[
+            bool,
+            Field(
+                title="Include MARC statements (MRK-format) as staff-only Holdings notes",
+                description=(
+                    "If set to true, the MARC statements will be included in the output as MARC Maker format fields. "
+                    "If set to false (default), the MARC statements will not be included in the output."
+                ),
+            ),
+        ] = False
+        mrk_holdings_note_type: Annotated[
+            str,
+            Field(
+                title="MARC Holdings Note type",
+                description=(
+                    "The name of the note type to use for MARC (MRK) statements. "
+                ),
+            ),
+        ] = "Original MARC holdings statements"
+        include_mfhd_mrk_as_note: Annotated[
+            bool,
+            Field(
+                title="Include MARC Record (as MARC Maker Representation) as note",
+                description=(
+                    "If set to true, the MARC statements will be included in the output as a "
+                    "(MRK) note. If set to false (default), the MARC statements will not be "
+                    "included in the output."
+                ),
+            ),
+        ] = False
+        mfhd_mrk_note_type: Annotated[
+            str,
+            Field(
+                title="MARC Record (as MARC Maker Representation) note type",
+                description=(
+                    "The name of the note type to use for MFHD (MRK) note. "
+                ),
+            ),
+        ] = "Original MFHD Record"
+        include_mfhd_mrc_as_note: Annotated[
+            bool,
+            Field(
+                title="Include MARC Record (as MARC21 decoded string) as note",
+                description=(
+                    "If set to true, the MARC record will be included in the output as a "
+                    "decoded binary MARC21 record. If set to false (default), the MARC record will not be "
+                    "included in the output."
+                ),
+            ),
+        ] = False
+        mfhd_mrc_note_type: Annotated[
+            str,
+            Field(
+                title="MARC Record (as MARC21 decoded string) note type",
+                description=(
+                    "The name of the note type to use for MFHD (MRC) note. "
+                ),
+            ),
+        ] = "Original MFHD (MARC21)"
 
     @staticmethod
     def get_object_type() -> FOLIONamespaces:

folio_migration_tools/migration_tasks/loans_migrator.py

@@ -525,7 +525,7 @@ class LoansMigrator(MigrationTaskBase):
             loan_to_put["dueDate"] = due_date.isoformat()
             loan_to_put["loanDate"] = out_date.isoformat()
             loan_to_put["renewalCount"] = renewal_count
-            url = f"{self.folio_client.okapi_url}/circulation/loans/{loan_to_put['id']}"
+            url = f"{self.folio_client.gateway_url}/circulation/loans/{loan_to_put['id']}"
             req = self.http_client.put(
                 url,
                 headers=self.folio_client.okapi_headers,
@@ -608,7 +608,7 @@ class LoansMigrator(MigrationTaskBase):
         try:
             # Get Item by barcode, update status.
             item_path = f'item-storage/items?query=(barcode=="{legacy_loan.item_barcode}")'
-            item_url = f"{self.folio_client.okapi_url}/{item_path}"
+            item_url = f"{self.folio_client.gateway_url}/{item_path}"
             resp = self.http_client.get(item_url, headers=self.folio_client.okapi_headers)
             resp.raise_for_status()
             data = resp.json()
@@ -667,14 +667,14 @@ class LoansMigrator(MigrationTaskBase):
         self.folio_put_post(url, user, "PUT", i18n.t("Update user"))
 
     def get_user_by_barcode(self, barcode):
-        url = f'{self.folio_client.okapi_url}/users?query=(barcode=="{barcode}")'
+        url = f'{self.folio_client.gateway_url}/users?query=(barcode=="{barcode}")'
         resp = self.http_client.get(url, headers=self.folio_client.okapi_headers)
         resp.raise_for_status()
         data = resp.json()
         return data["users"][0]
 
     def folio_put_post(self, url, data_dict, verb, action_description=""):
-        full_url = f"{self.folio_client.okapi_url}{url}"
+        full_url = f"{self.folio_client.gateway_url}{url}"
         try:
             if verb == "PUT":
                 resp = self.http_client.put(
@@ -729,7 +729,7 @@ class LoansMigrator(MigrationTaskBase):
     def change_due_date(self, folio_loan, legacy_loan):
         try:
             api_path = f"{folio_loan['id']}/change-due-date"
-            api_url = f"{self.folio_client.okapi_url}/circulation/loans/{api_path}"
+            api_url = f"{self.folio_client.gateway_url}/circulation/loans/{api_path}"
             body = {"dueDate": du_parser.isoparse(str(legacy_loan.due_date)).isoformat()}
             req = self.http_client.post(
                 api_url, headers=self.folio_client.okapi_headers, json=body

folio_migration_tools/migration_tasks/migration_task_base.py

@@ -9,11 +9,12 @@ from abc import abstractmethod
 from datetime import datetime, timezone
 from genericpath import isfile
 from pathlib import Path
-from typing import Optional
+from typing import Annotated, List, Optional
 
 import folioclient
 from folio_uuid.folio_namespaces import FOLIONamespaces
 from folioclient import FolioClient
+from pydantic import Field
 
 from folio_migration_tools import library_configuration, task_configuration
 from folio_migration_tools.custom_exceptions import (
@@ -460,6 +461,68 @@ class MigrationTaskBase:
             return None
 
 
+class MarcTaskConfigurationBase(task_configuration.AbstractTaskConfiguration):
+    """
+    Base class for MARC task configurations.
+
+    Attributes:
+        files (List[library_configuration.FileDefinition]):
+            List of MARC21 files to be processed.
+
+        create_source_records (bool):
+            Controls whether or not to retain the MARC records in Source Record Storage.
+            Default is False, meaning MARC records will not be retained.
+
+        hrid_handling (library_configuration.HridHandling):
+            Determines how HRIDs are handled.
+            - 'default': FOLIO generates HRIDs and moves existing 001 fields into a 035 field, concatenated with the 003 field.
+            - 'preserve001': Keeps the 001 fields in place and uses them as HRIDs.
+            Default is 'default'.
+
+        deactivate035_from001 (bool):
+            Disables the default FOLIO functionality of moving the previous 001 field into a 035 field, prefixed with the value from 003.
+            Default is False, meaning the functionality remains active.
+    """
+
+    files: Annotated[
+        List[library_configuration.FileDefinition],
+        Field(
+            title="Source files",
+            description=("List of MARC21 files with bibliographic records."),
+        ),
+    ]
+    create_source_records: Annotated[
+        bool,
+        Field(
+            title="Create source records",
+            description=(
+                "Controls whether or not to retain the MARC records in "
+                "Source Record Storage."
+            ),
+        ),
+    ] = False
+    hrid_handling: Annotated[
+        library_configuration.HridHandling,
+        Field(
+            title="HRID Handling",
+            description=(
+                "Setting to default will make FOLIO generate HRIDs and move the existing "
+                "001:s into a 035, concatenated with the 003. Choosing preserve001 means "
+                "the 001:s will remain in place, and that they will also become the HRIDs"
+            ),
+        ),
+    ] = library_configuration.HridHandling.default
+    deactivate035_from001: Annotated[
+        bool,
+        Field(
+            title="Create 035 from 001 and 003",
+            description=(
+                "This deactivates the FOLIO default functionality of moving the previous 001 "
+                "into a 035, prefixed with the value from 003"
+            ),
+        ),
+    ] = False
+
 class ExcludeLevelFilter(logging.Filter):
     def __init__(self, level):
         super().__init__()

folio_migration_tools/migration_tasks/reserves_migrator.py

@@ -190,7 +190,7 @@ class ReservesMigrator(MigrationTaskBase):
             sys.exit(1)
 
     def folio_put_post(self, url, data_dict, verb, action_description=""):
-        full_url = f"{self.folio_client.okapi_url}{url}"
+        full_url = f"{self.folio_client.gateway_url}{url}"
         try:
             if verb == "PUT":
                 resp = httpx.put(