fix fetch content logic

facundoolano · Jan 2, 2024 · 2c21188 · 2c21188
1 parent b99ca9f
commit 2c21188
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/feedi/models.py b/feedi/models.py
@@ -549,9 +549,8 @@ def fetch_content(self):
             try:
                 self.content_full = scraping.extract(self.content_url)['content']
                 db.session.commit()
-                return True
             except Exception:
-                return False
+                pass
 
     @classmethod
     def _filtered_query(cls, user_id, hide_seen=False, favorited=None,

diff --git a/feedi/routes.py b/feedi/routes.py
@@ -438,7 +438,8 @@ def entry_view(id):
             return redirect_response(entry.target_url)
 
         # if full browser load or explicit content request, fetch the article synchronously
-        if entry.fetch_content():
+        entry.fetch_content()
+        if entry.content_full:
             return flask.render_template("entry_content.html", entry=entry, content=entry.content_full)
 
         return redirect_response(entry.target_url)