From 3e61a9628c4d8866207b6e9fe620bad7e08d30d4 Mon Sep 17 00:00:00 2001
From: John Olheiser <42128690+jolheiser@users.noreply.github.com>
Date: Wed, 23 Oct 2019 12:54:13 -0500
Subject: [PATCH] Improve OGP (#8637)

* Improve OGP
* Ensure Repo is loaded when checking Pull info

Signed-off-by: jolheiser <john.olheiser@gmail.com>
---
 routers/repo/pull.go     |  4 ++++
 templates/base/head.tmpl | 24 +++++++++++++++++-------
 2 files changed, 21 insertions(+), 7 deletions(-)

diff --git a/routers/repo/pull.go b/routers/repo/pull.go
index 4001e3a4f8..4b73571e66 100644
--- a/routers/repo/pull.go
+++ b/routers/repo/pull.go
@@ -242,6 +242,10 @@ func checkPullInfo(ctx *context.Context) *models.Issue {
 		ctx.ServerError("LoadPoster", err)
 		return nil
 	}
+	if err := issue.LoadRepo(); err != nil {
+		ctx.ServerError("LoadRepo", err)
+		return nil
+	}
 	ctx.Data["Title"] = fmt.Sprintf("#%d - %s", issue.Index, issue.Title)
 	ctx.Data["Issue"] = issue
 
diff --git a/templates/base/head.tmpl b/templates/base/head.tmpl
index ae2b6b954d..c0686ca93e 100644
--- a/templates/base/head.tmpl
+++ b/templates/base/head.tmpl
@@ -131,16 +131,25 @@
 	<meta property="og:type" content="profile" />
 	<meta property="og:image" content="{{.Owner.AvatarLink}}" />
 	<meta property="og:url" content="{{.Owner.HTMLURL}}" />
-	<meta property="og:site_name" content="{{AppName}}" />
+	{{if .Owner.Description}}
+		<meta property="og:description" content="{{.Owner.Description}}">
+	{{end}}
 {{else if .Repository}}
-	<meta property="og:title" content="{{.Repository.Name}}" />
+	{{if .Issue}}
+		<meta property="og:title" content="{{.Issue.Title}}" />
+		<meta property="og:url" content="{{.Issue.HTMLURL}}" />
+		{{if .Issue.Content}}
+			<meta property="og:description" content="{{.Issue.Content}}" />
+		{{end}}
+	{{else}}
+		<meta property="og:title" content="{{.Repository.Name}}" />
+		<meta property="og:url" content="{{.Repository.HTMLURL}}" />
+		{{if .Repository.Description}}
+			<meta property="og:description" content="{{.Repository.Description}}" />
+		{{end}}
+	{{end}}
 	<meta property="og:type" content="object" />
 	<meta property="og:image" content="{{.Repository.Owner.AvatarLink}}" />
-	<meta property="og:url" content="{{.Repository.HTMLURL}}" />
-	{{if .Repository.Description}}
-	<meta property="og:description" content="{{.Repository.Description}}" />
-	{{end}}
-	<meta property="og:site_name" content="{{AppName}}" />
 {{else}}
 	<meta property="og:title" content="{{AppName}}">
 	<meta property="og:type" content="website" />
@@ -148,6 +157,7 @@
 	<meta property="og:url" content="{{AppUrl}}" />
 	<meta property="og:description" content="{{MetaDescription}}">
 {{end}}
+<meta property="og:site_name" content="{{AppName}}" />
 {{if .IsSigned }}
 	{{ if ne .SignedUser.Theme "gitea" }}
 		<link rel="stylesheet" href="{{StaticUrlPrefix}}/css/theme-{{.SignedUser.Theme}}.css">