summaryrefslogtreecommitdiffstats
path: root/python/fatcat_tools/transforms/csl.py
diff options
context:
space:
mode:
Diffstat (limited to 'python/fatcat_tools/transforms/csl.py')
-rw-r--r--python/fatcat_tools/transforms/csl.py23
1 files changed, 11 insertions, 12 deletions
diff --git a/python/fatcat_tools/transforms/csl.py b/python/fatcat_tools/transforms/csl.py
index 989629ab..f94fa372 100644
--- a/python/fatcat_tools/transforms/csl.py
+++ b/python/fatcat_tools/transforms/csl.py
@@ -34,31 +34,30 @@ def release_to_csl(entity):
contribs = []
for contrib in (entity.contribs or []):
if contrib.creator:
- # TODO: should we actually be pulling creator metadata? or just
- # using release-local raw metadata?
- family = contrib.creator.surname
- if not family:
- if not contrib.raw_name:
- raise ValueError("CSL requires some surname (family name)")
- family = contrib.raw_name.split()[-1]
+ # Default to "local" (publication-specific) metadata; fall back to
+ # creator-level
+ family = contrib.surname or contrib.creator.surname or (contrib.raw_name and contrib.raw_name.split()[-1])
+ if not contrib.raw_name:
+ raise ValueError("CSL requires some surname (family name)")
c = dict(
- family=contrib.creator.surname,
- given=contrib.creator.given_name,
+ family=family,
+ given=contrib.given_name or contrib.creator.given_name,
#dropping-particle
#non-dropping-particle
#suffix
#comma-suffix
#static-ordering
- literal=contrib.raw_name, # or display_name?
+ literal=contrib.raw_name or contrib.creator.display_name,
#parse-names,
role=contrib.role,
)
else:
+ family = contrib.surname or (contrib.raw_name and contrib.raw_name.split()[-1])
if not contrib.raw_name:
raise ValueError("CSL requires some surname (family name)")
c = dict(
- # XXX: possible inclusion of full name metadata in release_contrib
- family=contrib.raw_name.split()[-1],
+ family=family,
+ given=contrib.given_name,
literal=contrib.raw_name,
role=contrib.role,
)