I think checking whether updated is None is a bit clearer...other, updated might look like a boolean.

2016-10-12 09:34:46 -07:00 · 2016-10-12 09:34:46 -07:00 · 7180da1f9d
parent a73720daab
commit 7180da1f9d
1 changed files with 1 additions and 1 deletions
--- a/api/opds.py
+++ b/api/opds.py
@ -86,7 +86,7 @@ def work_node(work, facet=None):
    ebooks = facet.filter_model("Ebook",work.ebooks()) if facet else work.ebooks()
    versions = set()
    for ebook in ebooks:
-        if not updated:
+        if updated is None:
            # most recent ebook, first ebook in loop
            updated = ebook.created.isoformat()
            node.append(text_node('updated', updated))