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

Set default content status to a blank string rather than None.

Browse source 
Fixes
#2558.
Fixes issues encountered by comment plugins among others

c.f. [1](https://github.com/bstpierre/pelican-comments/pull/4),
[2](https://github.com/Scheirle/pelican_comment_system/issues/8),
[3](https://github.com/Scheirle/pelican_comment_system/pull/7)
This commit is contained in:
MinchinWeb 2019-08-20 19:39:14 -06:00
commit b2da535fec
2 changed files with 10 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
RELEASE.md Normal file
View file

@ -0,0 +1,7 @@
Release type: patch
Set default content status to a blank string rather than `None`. Fixes
[#2558](https://github.com/getpelican/pelican/issues/2558). Fixes issues
encountered by comment plugins among others
([1](https://github.com/bstpierre/pelican-comments/pull/4),
[2](https://github.com/Scheirle/pelican_comment_system/issues/8)).

4
pelican/contents.py
View file

@ -141,7 +141,9 @@ class Content(object):
# manage status # manage status
if not hasattr(self, 'status'): if not hasattr(self, 'status'):
self.status = getattr(self, 'default_status', None) # using None as the default here breaks comment plugins (and
# probably others)
self.status = getattr(self, 'default_status', '')
# store the summary metadata if it is set # store the summary metadata if it is set
if 'summary' in metadata: if 'summary' in metadata: