From 2006b2fbce8b412c6843123b11d8b7ad7c92e05a Mon Sep 17 00:00:00 2001
From: Adam Kelly <adam@cthulahoops.org>
Date: Sat, 25 Apr 2020 11:29:30 +0100
Subject: [PATCH] Only use added_date as start_date for books being read right
 now.

---
 fedireads/models/import_job.py | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/fedireads/models/import_job.py b/fedireads/models/import_job.py
index fcb21654..2465fe12 100644
--- a/fedireads/models/import_job.py
+++ b/fedireads/models/import_job.py
@@ -111,12 +111,14 @@ class ImportItem(models.Model):
 
     @property
     def reads(self):
-        return [ReadThrough(
-            # date_added isn't the start date, but maybe better than nothing.
-            start_date=self.date_added,
-            finish_date=self.date_read,
-            pages_read=None,
-        )]
+        if (self.shelf == 'reading'
+                and self.date_added and not self.date_read):
+            return [ReadThrough(start_date=self.date_added)]
+        if self.date_read:
+            return [ReadThrough(
+                finish_date=self.date_read,
+            )]
+        return []
 
     def __repr__(self):
         return "<GoodreadsItem {!r}>".format(self.data['Title'])