From 1e3441323899170e8aad9ed7c735cefc79cfeb11 Mon Sep 17 00:00:00 2001
From: Bo-Yi Wu <appleboy.tw@gmail.com>
Date: Thu, 18 Oct 2018 12:51:07 +0800
Subject: refactor: err != nil check, just return error instead (#5093)

---
 cmd/admin.go | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

(limited to 'cmd')

diff --git a/cmd/admin.go b/cmd/admin.go
index 047f3befcc..893b6f5be9 100644
--- a/cmd/admin.go
+++ b/cmd/admin.go
@@ -412,16 +412,12 @@ func runAddOauth(c *cli.Context) error {
 		return err
 	}
 
-	if err := models.CreateLoginSource(&models.LoginSource{
+	return models.CreateLoginSource(&models.LoginSource{
 		Type:      models.LoginOAuth2,
 		Name:      c.String("name"),
 		IsActived: true,
 		Cfg:       parseOAuth2Config(c),
-	}); err != nil {
-		return err
-	}
-
-	return nil
+	})
 }
 
 func runUpdateOauth(c *cli.Context) error {
@@ -492,11 +488,7 @@ func runUpdateOauth(c *cli.Context) error {
 	oAuth2Config.CustomURLMapping = customURLMapping
 	source.Cfg = oAuth2Config
 
-	if err := models.UpdateSource(source); err != nil {
-		return err
-	}
-
-	return nil
+	return models.UpdateSource(source)
 }
 
 func runListAuth(c *cli.Context) error {
@@ -543,8 +535,5 @@ func runDeleteAuth(c *cli.Context) error {
 		return err
 	}
 
-	if err = models.DeleteSource(source); err != nil {
-		return err
-	}
-	return nil
+	return models.DeleteSource(source)
 }
-- 
cgit v1.2.3