From 4faef03ba3f3fbb21c11f3a41a31c66d7e83bb75 Mon Sep 17 00:00:00 2001
From: Nick Sweeting <git@sweeting.me>
Date: Mon, 15 Feb 2021 20:44:08 -0500
Subject: [PATCH] compute snapshot properties directly without loading whole
 Link

---
 archivebox/core/models.py | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/archivebox/core/models.py b/archivebox/core/models.py
index 13d75b66..7be705c3 100644
--- a/archivebox/core/models.py
+++ b/archivebox/core/models.py
@@ -7,7 +7,8 @@ from django.utils.functional import cached_property
 from django.utils.text import slugify
 from django.db.models import Case, When, Value, IntegerField
 
-from ..util import parse_date
+from ..config import ARCHIVE_DIR
+from ..util import parse_date, base_url, hashurl
 from ..index.schema import Link
 from ..extractors import get_default_archive_methods, ARCHIVE_METHODS_INDEXING_PRECEDENCE
 
@@ -116,6 +117,11 @@ class Snapshot(models.Model):
     def bookmarked(self):
         return parse_date(self.timestamp)
 
+    @cached_property
+    def bookmarked_date(self):
+        # TODO: remove this
+        return self.bookmarked
+
     @cached_property
     def is_archived(self):
         return self.as_link().is_archived
@@ -126,15 +132,15 @@ class Snapshot(models.Model):
 
     @cached_property
     def url_hash(self):
-        return self.as_link().url_hash
+        return hashurl(self.url)
 
     @cached_property
     def base_url(self):
-        return self.as_link().base_url
+        return base_url(self.url)
 
     @cached_property
     def link_dir(self):
-        return self.as_link().link_dir
+        return str(ARCHIVE_DIR / self.timestamp)
 
     @cached_property
     def archive_path(self):