Merge pull request #2610 from MinchinWeb/default-status

Set default content status to blank string instead of `None`
getpelican · Aug 21, 2019 · 99ea502 · 99ea502
2 parents 7c2a747 + b2da535
commit 99ea502
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/RELEASE.md b/RELEASE.md
@@ -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)).
diff --git a/pelican/contents.py b/pelican/contents.py
@@ -141,7 +141,9 @@ def __init__(self, content, metadata=None, settings=None,
 
         # manage 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
         if 'summary' in metadata: