From 8c3b1c8c9550c9fec9a82a9afa1c4670fcf42e07 Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Sat, 28 Jul 2018 14:48:03 +0100
Subject: Add commit hash to releases

---
 app/models.py                            |  1 +
 app/tasks/importtasks.py                 | 10 ++++++----
 app/templates/packages/release_edit.html |  4 ++++
 app/templates/packages/view.html         |  3 ++-
 4 files changed, 13 insertions(+), 5 deletions(-)

(limited to 'app')

diff --git a/app/models.py b/app/models.py
index 0452009..281867c 100644
--- a/app/models.py
+++ b/app/models.py
@@ -591,6 +591,7 @@ class PackageRelease(db.Model):
 	url          = db.Column(db.String(200), nullable=False)
 	approved     = db.Column(db.Boolean, nullable=False, default=False)
 	task_id      = db.Column(db.String(37), nullable=True)
+	commit_hash  = db.Column(db.String(41), nullable=True, default=None)
 
 
 	def getEditURL(self):
diff --git a/app/tasks/importtasks.py b/app/tasks/importtasks.py
index 18ce2e6..9afc8b6 100644
--- a/app/tasks/importtasks.py
+++ b/app/tasks/importtasks.py
@@ -344,9 +344,10 @@ def makeVCSReleaseFromGithub(id, branch, release, url):
 	if len(commits) == 0 or not "sha" in commits[0]:
 		raise TaskError("No commits found")
 
-	release.url = urlmaker.getCommitDownload(commits[0]["sha"])
+	release.url          = urlmaker.getCommitDownload(commits[0]["sha"])
+	release.task_id     = None
+	release.commit_hash = commits[0]["sha"]
 	print(release.url)
-	release.task_id = None
 	db.session.commit()
 
 	return release.url
@@ -374,9 +375,10 @@ def makeVCSRelease(id, branch):
 			with open(destPath, "wb") as fp:
 				repo.archive(fp, format="zip")
 
-			release.url = "/uploads/" + filename
+			release.url         = "/uploads/" + filename
+			release.task_id     = None
+			release.commit_hash = repo.head.object.hexsha
 			print(release.url)
-			release.task_id = None
 			db.session.commit()
 
 			return release.url
diff --git a/app/templates/packages/release_edit.html b/app/templates/packages/release_edit.html
index a201a38..faa475e 100644
--- a/app/templates/packages/release_edit.html
+++ b/app/templates/packages/release_edit.html
@@ -21,6 +21,10 @@
 			Url: <a href="{{ release.url }}">{{ release.url }}</a><br />
 		{% endif %}
 
+		{% if release.commit_hash %}
+			Commit Hash: {{ release.commit_hash }}<br />
+		{% endif %}
+
 		{% if release.task_id %}
 			Importing... <a href="{ url_for('check_task', id=release.task_id, r=release.getEditURL()) }}">view task</a><br />
 			{% if package.checkPerm(current_user, "CHANGE_RELEASE_URL") %}
diff --git a/app/templates/packages/view.html b/app/templates/packages/view.html
index b118046..181ae99 100644
--- a/app/templates/packages/view.html
+++ b/app/templates/packages/view.html
@@ -188,7 +188,8 @@
 				<li>
 					{% if not rel.approved %}<i>{% endif %}
 
-					<a href="{{ rel.url }}">{{ rel.title }}</a>,
+					<a href="{{ rel.url }}">{{ rel.title }}</a>{% if rel.commit_hash %}
+					[{{ rel.commit_hash | truncate(5, end='') }}]{% endif %},
 					created {{ rel.releaseDate | datetime }}.
 					{% if rel.task_id %}
 						<a href="{{ url_for('check_task', id=rel.task_id, r=package.getDetailsURL()) }}">Importing...</a>
-- 
cgit v1.2.3