From ebc8b009043e5064ac960b9ddcc5e075a535882f Mon Sep 17 00:00:00 2001
From: Anand Sudhir Prayaga <anandsudhir86@gmail.com>
Date: Tue, 7 Aug 2018 11:05:03 +0200
Subject: [PATCH] Don't die if Gerrit can't connect to the API

Don't crash, and handle connectivity issues to Gerrit server gracefully
---
 gerrit/widget.go | 46 +++++++++++++++++++++++-----------------------
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/gerrit/widget.go b/gerrit/widget.go
index 1f6836ea..15c71a58 100644
--- a/gerrit/widget.go
+++ b/gerrit/widget.go
@@ -47,6 +47,24 @@ var (
 )
 
 func NewWidget(app *tview.Application, pages *tview.Pages) *Widget {
+	widget := Widget{
+		HelpfulWidget: wtf.NewHelpfulWidget(app, pages, HelpText),
+		TextWidget:    wtf.NewTextWidget("Gerrit", "gerrit", true),
+
+		Idx: 0,
+	}
+
+	widget.HelpfulWidget.SetView(widget.View)
+
+	widget.View.SetInputCapture(widget.keyboardIntercept)
+	widget.unselect()
+
+	return &widget
+}
+
+/* -------------------- Exported Functions -------------------- */
+
+func (widget *Widget) Refresh() {
 	baseURL := wtf.Config.UString("wtf.mods.gerrit.domain")
 	username := wtf.Config.UString("wtf.mods.gerrit.username")
 
@@ -72,34 +90,16 @@ func NewWidget(app *tview.Application, pages *tview.Pages) *Widget {
 		gerritUrl = fmt.Sprintf(
 			"%s://%s:%s@%s", submatch[1], username, password, submatch[2])
 	}
-
 	gerrit, err := glb.NewClient(gerritUrl, httpClient)
 	if err != nil {
-		panic(err)
+		widget.View.SetWrap(true)
+		widget.View.SetTitle(widget.Name)
+		widget.View.SetText(err.Error())
+		return
 	}
-
-	widget := Widget{
-		HelpfulWidget: wtf.NewHelpfulWidget(app, pages, HelpText),
-		TextWidget:    wtf.NewTextWidget("Gerrit", "gerrit", true),
-
-		gerrit: gerrit,
-
-		Idx: 0,
-	}
-
-	widget.HelpfulWidget.SetView(widget.View)
-
+	widget.gerrit = gerrit
 	widget.GerritProjects = widget.buildProjectCollection(wtf.Config.UList("wtf.mods.gerrit.projects"))
 
-	widget.View.SetInputCapture(widget.keyboardIntercept)
-	widget.unselect()
-
-	return &widget
-}
-
-/* -------------------- Exported Functions -------------------- */
-
-func (widget *Widget) Refresh() {
 	for _, project := range widget.GerritProjects {
 		project.Refresh()
 	}