From 893507691ba8863469738eb682e0eac73a9a0194 Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Thu, 9 Jul 2020 04:50:49 +0100
Subject: [PATCH] Show "Edit Review" button when a user already has a review

---
 app/blueprints/packages/packages.py |  3 ++-
 app/templates/packages/view.html    | 12 +++++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/app/blueprints/packages/packages.py b/app/blueprints/packages/packages.py
index bfa3dd9..3266dbe 100644
--- a/app/blueprints/packages/packages.py
+++ b/app/blueprints/packages/packages.py
@@ -158,12 +158,13 @@ def view(package):
 	elif not current_user.rank.atLeast(UserRank.EDITOR) and not current_user == package.author:
 		threads = threads.filter(or_(Thread.private == False, Thread.author == current_user))
 
+	has_review = PackageReview.query.filter_by(package=package, author=current_user).count() > 0
 
 	return render_template("packages/view.html", \
 			package=package, releases=releases, requests=requests, \
 			alternatives=alternatives, similar_topics=similar_topics, \
 			review_thread=review_thread, topic_error=topic_error, topic_error_lvl=topic_error_lvl, \
-			threads=threads.all())
+			threads=threads.all(), has_review=has_review)
 
 
 @bp.route("/packages/<author>/<name>/download/")
diff --git a/app/templates/packages/view.html b/app/templates/packages/view.html
index 3a8084d..550ea12 100644
--- a/app/templates/packages/view.html
+++ b/app/templates/packages/view.html
@@ -463,7 +463,17 @@
 
 		{% from "macros/reviews.html" import render_reviews, render_review_form, render_review_preview %}
 		{% if current_user.is_authenticated %}
-			{{ render_review_form(package, current_user) }}
+			{% if has_review %}
+				<p>
+					<a
+							class="btn btn-primary"
+							href="{{ url_for('packages.review', author=package.author.username, name=package.name) }}">
+						{{ _("Edit Review") }}
+					</a>
+				</p>
+			{% else %}
+				{{ render_review_form(package, current_user) }}
+			{% endif %}
 		{% else %}
 			{{ render_review_preview(package) }}
 		{% endif %}