Cutlery/calibre
1
0
Fork 0
mirror of https://github.com/kovidgoyal/calibre.git synced 2025-07-09 03:04:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

Tag mapper: When the replacement tag contains commas, create multiple tags instead of replacing the comma with a semi-colon. Fixes #1503526 [Tap Mapping comma as semi-colon as of 2.40- did not on 2.39](https://bugs.launchpad.net/calibre/+bug/1503526)

Browse Source
This commit is contained in:
Kovid Goyal 2015-10-08 10:01:16 +05:30
parent fa49f81b3e
commit eff9a695e1
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/calibre/ebooks/metadata/tag_mapper.py
View File

@ -44,6 +44,8 @@ def apply_rules(tag, rules):
tag = regex.sub(rule['query'], rule['replace'], tag, flags=REGEX_FLAGS) tag = regex.sub(rule['query'], rule['replace'], tag, flags=REGEX_FLAGS)
else: else:
tag = rule['replace'] tag = rule['replace']
if ',' in tag:
tag = [x.strip() for x in tag.split(',')]
return tag return tag
def uniq(vals, kmap=icu_lower): def uniq(vals, kmap=icu_lower):
@ -62,10 +64,18 @@ def map_tags(tags, rules=()):
if not rules: if not rules:
return list(tags) return list(tags)
rules = [(r, matcher(r)) for r in rules] rules = [(r, matcher(r)) for r in rules]
return uniq([x for x in (apply_rules(t, rules) for t in tags) if x]) ans = []
for t in tags:
mapped = apply_rules(t, rules)
(ans.extend if isinstance(mapped, list) else ans.append)(mapped)
return uniq(filter(None, ans))
def test(): def test():
rules = [{'action':'replace', 'query':'t1', 'match_type':'one_of', 'replace':'t2'}] rules = [{'action':'replace', 'query':'t1', 'match_type':'one_of', 'replace':'t2'}]
assert map_tags(['t1', 'x1'], rules) == ['t2', 'x1'] assert map_tags(['t1', 'x1'], rules) == ['t2', 'x1']
rules = [{'action':'replace', 'query':'(.)1', 'match_type':'matches', 'replace':r'\g<1>2'}] rules = [{'action':'replace', 'query':'(.)1', 'match_type':'matches', 'replace':r'\g<1>2'}]
assert map_tags(['t1', 'x1'], rules) == ['t2', 'x2'] assert map_tags(['t1', 'x1'], rules) == ['t2', 'x2']
rules = [{'action':'replace', 'query':'t1', 'match_type':'one_of', 'replace':'t2, t3'}]
assert map_tags(['t1', 'x1'], rules) == ['t2', 't3', 'x1']
rules = [{'action':'replace', 'query':'(.)1', 'match_type':'matches', 'replace':r'\g<1>2,3'}]
assert map_tags(['t1', 'x1'], rules) == ['t2', '3', 'x2']