weeheavy/pelican-theme
1
0
Fork 0
forked from github/pelican
Code Pull requests Activity

As not allowing duplicates in processed items is counter intuitive, let's allow it.

Browse source 
Also it may be allowed in the future (to process multiple values).

Also @avaris think it's bad to test something twice (see
https://github.com/getpelican/pelican/pull/2017), but for me confusion
lies in the "Why is list processing forbidden?", so, in a way, our
ideas converges in "let's not disallow processed items to be lists".

This reverts commit 9e574e9d8c.
This commit is contained in:
Julien Palard 2016-10-10 12:23:26 +02:00
commit e8a87e5d3c
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
pelican/readers.py
View file

@ -278,8 +278,7 @@ class MarkdownReader(BaseReader):
self._md.reset()
formatted = self._md.convert(formatted_values)
output[name] = self.process_metadata(name, formatted)
elif (not DUPLICATES_DEFINITIONS_ALLOWED.get(name, True) or
name in METADATA_PROCESSORS):
elif not DUPLICATES_DEFINITIONS_ALLOWED.get(name, True):
if len(value) > 1:
logger.warning(
'Duplicate definition of `%s` '