Fix lint errors (#2547)

models/gpg_key.go

@@ -282,11 +282,7 @@ func DeleteGPGKey(doer *User, id int64) (err error) {
 		return err
 	}
 
-	if err = sess.Commit(); err != nil {
+	return sess.Commit()
-		return err
-	}
-
-	return nil
 }
 
 // CommitVerification represents a commit validation of signature

models/issue.go

@@ -571,11 +571,7 @@ func (issue *Issue) ReadBy(userID int64) error {
 		return err
 	}
 
-	if err := setNotificationStatusReadIfUnread(x, userID, issue.ID); err != nil {
+	return setNotificationStatusReadIfUnread(x, userID, issue.ID)
-		return err
-	}
-
-	return nil
 }
 
 func updateIssueCols(e Engine, issue *Issue, cols ...string) error {

models/login_source.go

@@ -498,10 +498,7 @@ func SMTPAuth(a smtp.Auth, cfg *SMTPConfig) error {
 	}
 
 	if ok, _ := c.Extension("AUTH"); ok {
-		if err = c.Auth(a); err != nil {
+		return c.Auth(a)
-			return err
-		}
-		return nil
 	}
 	return ErrUnsupportedLoginType
 }

models/migrations/v16.go

@@ -119,9 +119,5 @@ func addUnitsToTables(x *xorm.Engine) error {
 		}
 	}
 
-	if err := sess.Commit(); err != nil {
+	return sess.Commit()
-		return err
-	}
-
-	return nil
 }

models/migrations/v21.go

@@ -51,8 +51,5 @@ func useNewPublickeyFormat(x *xorm.Engine) error {
 	}
 
 	f.Close()
-	if err = os.Rename(tmpPath, fpath); err != nil {
+	return os.Rename(tmpPath, fpath)
-		return err
-	}
-	return nil
 }

models/org.go

@@ -235,11 +235,7 @@ func DeleteOrganization(org *User) (err error) {
 		}
 	}
 
-	if err = sess.Commit(); err != nil {
+	return sess.Commit()
-		return err
-	}
-
-	return nil
 }
 
 func deleteOrg(e *xorm.Session, u *User) error {

models/ssh_key.go

@@ -599,11 +599,7 @@ func RewriteAllPublicKeys() error {
 		defer f.Close()
 	}
 
-	if err = os.Rename(tmpPath, fPath); err != nil {
+	return os.Rename(tmpPath, fPath)
-		return err
-	}
-
-	return nil
 }
 
 // ________                .__                 ____  __.

modules/lfs/content_store.go

@@ -70,10 +70,7 @@ func (s *ContentStore) Put(meta *models.LFSMetaObject, r io.Reader) error {
 		return errHashMismatch
 	}
 
-	if err := os.Rename(tmpPath, path); err != nil {
+	return os.Rename(tmpPath, path)
-		return err
-	}
-	return nil
 }
 
 // Exists returns true if the object exists in the content store.

modules/log/file.go

@@ -110,10 +110,7 @@ func (w *FileLogWriter) StartLogger() error {
 		return err
 	}
 	w.mw.SetFd(fd)
-	if err = w.initFd(); err != nil {
+	return w.initFd()
-		return err
-	}
-	return nil
 }
 
 func (w *FileLogWriter) docheck(size int) {