From d0080a5025e41020cba4cad6f6c02992e35c81fa Mon Sep 17 00:00:00 2001
From: Thorge Petersen <petersen@rz.uni-kiel.de>
Date: Thu, 8 Jun 2023 13:02:37 +0200
Subject: [PATCH] Fixed missing dependency and warning for subjects not found
 in subject mapping dictionary

---
 ckanext/odsh/validation.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ckanext/odsh/validation.py b/ckanext/odsh/validation.py
index 4185d893..cb0574e4 100644
--- a/ckanext/odsh/validation.py
+++ b/ckanext/odsh/validation.py
@@ -4,6 +4,8 @@ import csv
 import re
 import urllib.request, urllib.error, urllib.parse
 import json
+import ckan.logic as logic
+
 from itertools import count
 from dateutil.parser import parse
 
@@ -329,11 +331,11 @@ def _convert_subjectID_to_subjectText(subject_id, flattened_data):
     try: 
         subject_text = subject_mapping[subject_id]
     except:
-        raise toolkit.Invalid(_('Subject must be a known URI.'))
         log.warning(
             'Subject_id "{}" not found in subject mapping dictionary.\nSubject mapping file: {}'
             .format(subject_id, subject_mapping_file_path)
         )
+        raise toolkit.Invalid(_('Subject must be a known URI.'))
         
 
     new_index = next_extra_index(flattened_data)
-- 
GitLab