Skip to content

Commit

Permalink
Bugfix/trailing period in work title (#1338)
Browse files Browse the repository at this point in the history
* Don't append period to work title if no parts to be added after
kwahlin authored Nov 21, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1 parent 5b3e353 commit a9ff65a
Showing 1 changed file with 3 additions and 9 deletions.
12 changes: 3 additions & 9 deletions librisworks/src/main/groovy/se/kb/libris/mergeworks/Util.groovy
Original file line number Diff line number Diff line change
@@ -175,15 +175,9 @@ class Util {
]

static void appendTitlePartsToMainTitle(Map title, String partNumber, String partName = null) {
if (title['mainTitle'][-1] != '.') {
title['mainTitle'] += '.'
}
if (partNumber && partName) {
title['mainTitle'] += " $partNumber, $partName"
} else if (partNumber) {
title['mainTitle'] += " $partNumber"
} else if (partName) {
title['mainTitle'] += " $partName"
def part = [partNumber, partName].grep().join(', ')
if (part) {
title['mainTitle'] += "${title['mainTitle'][-1] == '.' ? '' : '.'} $part"
}
}

0 comments on commit a9ff65a

Please sign in to comment.