From ca49d88d2171db562c6a859903c67c05507cfd7f Mon Sep 17 00:00:00 2001
From: Paul Pound <ppound@upei.ca>
Date: Fri, 22 May 2026 13:34:43 -0300
Subject: [PATCH] broke the doi issn fields out into their own payload elements

---
 includes/sharepoint.inc | 18 ++++--------------
 1 file changed, 4 insertions(+), 14 deletions(-)

diff --git a/includes/sharepoint.inc b/includes/sharepoint.inc
index 8bdb36a..5df7d3f 100644
--- a/includes/sharepoint.inc
+++ b/includes/sharepoint.inc
@@ -66,18 +66,6 @@ class SharePointRequestHandler
       $author = $biblio['Author'];
     }
 
-    // Combine DOI and ISBN into a single field.
-    $doi_isbn_parts = [];
-    if (!empty($biblio['doi'])) {
-      $doi_isbn_parts[] = $biblio['doi'];
-    }
-    if (!empty($biblio['ISBN'])) {
-      $doi_isbn_parts[] = $biblio['ISBN'];
-    }
-    if (!empty($biblio['ISSN'])) {
-      $doi_isbn_parts[] = $biblio['ISSN'];
-    }
-    $doi_isbn = implode('; ', $doi_isbn_parts);
 
     // Build patron full name.
     $first_name = $patron['FirstName'] ?? '';
@@ -114,11 +102,13 @@ class SharePointRequestHandler
       'title' => (string)($biblio['Title'] ?? ''),
       'author' => (string)$author,
       'year' => (string)($biblio['Date'] ?? ''),
-      'doi_isbn' => (string)$doi_isbn,
+      'isbn' => (string)($biblio['ISBN'] ?? ''),
+      'issn' => (string)($biblio['ISSN'] ?? ''),
+      'doi' => (string)($biblio['doi'] ?? ''),
       'format' => (string)$genre,
       'patronName' => (string)$patron_name,
       'patronEmail' => (string)($patron['DeliveryAddress'] ?? ''),
-      'patronBarcode' => (string)($patron['campus_id'] ?? ''),
+      'patronID' => (string)($patron['campus_id'] ?? ''),
       'notes' => (string)implode(' | ', $notes_parts),
       'submittedAt' => (string)gmdate('Y-m-d\TH:i:s') . '.000Z',
     ];