Merge pull request #103 from cthulahoops/not_logged_in

Book & books while not logged in

fedireads/templates/book.html

@@ -25,6 +25,7 @@
     </div>
 </div>
 
+{% if request.user.is_authenticated %}
 <div class="content-container">
     <h2>Leave a review</h2>
     <form class="review-form" name="review" action="/review/" method="post">
@@ -38,6 +39,7 @@
 <div class="content-container tabs">
     {% include 'snippets/tabs.html' with tabs=feed_tabs active_tab=active_tab path=path %}
 </div>
+{% endif %}
 
 {% if not reviews %}
 <div class="content-container">

fedireads/templates/layout.html

@@ -26,7 +26,9 @@
         <div id="branding"><a href="/"><img id="logo" src="/static/images/logo-small.png" alt="BookWyrm"></img></a></div>
 
         <ul class="menu">
+            {% if request.user.is_authenticated %}
             <li><a href="/user/{{request.user.localname}}/shelves">Your shelves</a></li>
+            {% endif %}
             <li><a href="/#feed">Updates</a></li>
             <li><a href="/books">Discover Books</a></li>
         </ul>

fedireads/templates/snippets/shelve_button.html

@@ -1,4 +1,5 @@
 {% load fr_display %}
+{% if request.user.is_authenticated %}
 {% if not pulldown %}
 
 <form name="shelve" action="/shelve/" method="post">
@@ -25,3 +26,4 @@
 </form>
 
 {% endif %}
+{% endif %}

fedireads/views.py

@@ -98,11 +98,11 @@ def home_tab(request, tab):
 
 def books_page(request):
     ''' discover books '''
-    recent_books = models.Book.objects.filter(
+    recent_books = models.Book.objects
-        ~Q(shelfbook__shelf__user=request.user)
+    if request.user.is_authenticated:
-    ).order_by(
+        recent_books = recent_books.filter(~Q(shelfbook__shelf__user=request.user))
-        '-created_date'
+    recent_books = recent_books.order_by('-created_date')[:50]
-    )[:50]
+
     data = {
         'books': recent_books,
     }
@@ -306,6 +306,7 @@ def book_page(request, book_identifier, tab='friends'):
     else:
         book_reviews = models.Review.objects.filter(book=book)
 
+    if request.user.is_authenticated:
         user_reviews = book_reviews.filter(
             user=request.user,
         ).all()
@@ -333,15 +334,27 @@ def book_page(request, book_identifier, tab='friends'):
         except models.Shelf.DoesNotExist:
             shelf = None
 
+        user_tags = models.Tag.objects.filter(
+            book=book, user=request.user
+        ).all()
+        user_tag_names = user_tags.values_list('identifier', flat=True)
+
+        user_rating = user_reviews.aggregate(Avg('rating')),
+    else:
+        tab = 'public'
+        reviews = book_reviews.filter(privacy='public')
+        shelf = None
+        user_reviews = []
+        user_rating = None
+        user_tags = []
+        user_tag_names = []
+
     rating = reviews.aggregate(Avg('rating'))
     tags = models.Tag.objects.filter(
         book=book
     ).values(
         'book', 'name', 'identifier'
     ).distinct().all()
-    user_tags = models.Tag.objects.filter(
-        book=book, user=request.user
-    ).all()
 
     review_form = forms.ReviewForm()
     tag_form = forms.TagForm()
@@ -349,12 +362,12 @@ def book_page(request, book_identifier, tab='friends'):
         'book': book,
         'shelf': shelf,
         'user_reviews': user_reviews,
-        'user_rating': user_reviews.aggregate(Avg('rating')),
+        'user_rating': user_rating,
         'reviews': reviews.distinct(),
         'rating': rating['rating__avg'],
         'tags': tags,
         'user_tags': user_tags,
-        'user_tag_names': user_tags.values_list('identifier', flat=True),
+        'user_tag_names': user_tag_names,
         'review_form': review_form,
         'tag_form': tag_form,
         'feed_tabs': [