Abort when linked work

libris · Sep 13, 2023 · d85bebf · d85bebf
1 parent 7fe5012
commit d85bebf
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/librisworks/scripts/contributions-to-instance.groovy b/librisworks/scripts/contributions-to-instance.groovy
@@ -31,7 +31,7 @@ clusters.each { c ->
         def id = bib.doc.shortId
         Map instance = bib.graph[1]
         Map work = instance.instanceOf
-        work.contribution?.each { contrib ->
+        work?.contribution?.each { contrib ->
             if (asList(contrib.role).contains(ill)) {
                 def agent = asList(contrib.agent).find()
                 if (!agent) return

diff --git a/librisworks/scripts/elib-unspecified-contributor.groovy b/librisworks/scripts/elib-unspecified-contributor.groovy
@@ -31,7 +31,7 @@ Set<String> handled = new ConcurrentHashMap().newKeySet()
 selectBySqlWhere(where) { bib ->
     def id = bib.doc.shortId
     def instance = bib.graph[1]
-    def summary = asList(instance['instanceOf']['summary']) + asList(bib.graph[1]['summary'])
+    def summary = asList(instance.instanceOf?.summary) + asList(instance.summary)
 
     def nameToRoles = summary
             .findResults { it['label'] }
@@ -99,7 +99,7 @@ selectBySqlWhere("collection = 'bib' AND data#>>'{@graph, 0, identifiedBy}' LIKE
         return
     }
     def instance = bib.graph[1]
-    List workContribution = instance['instanceOf']['contribution']
+    List workContribution = instance.instanceOf?.contribution
     if (!workContribution) {
         return
     }

diff --git a/librisworks/scripts/language-in-work-title.groovy b/librisworks/scripts/language-in-work-title.groovy
@@ -15,8 +15,12 @@ selectByIds(ids) { bib ->
     ].collect {
         langName(getAtPath(bib.graph, it, '')).toLowerCase()
     }
+
+    def work = bib.graph[1].instanceOf
+
+    if (!work || work['@id']) return
 
-    boolean changed = DocumentUtil.traverse(bib.graph[1].instanceOf) { value, path ->
+    boolean changed = DocumentUtil.traverse(work) { value, path ->
         if (path && 'mainTitle' in path && value instanceof String) {
             for (lang in langs) {
                 String r = value.replaceAll(/(?i)\s*\(\(?\s*${lang}\s*\)\)?\s*$/, '')