retusn del nil

admin/internal/logic/roleeditlogic.go

@@ -38,12 +38,12 @@ func (l *RoleEditLogic) RoleEdit(in *admin.RoleEditRequest) (*admin.Response, er
 	}
 	adminInfo := utils.GetUserFromCtx(l.ctx)
 	if adminInfo.ID < utils.NumberOne {
-		return l.fail(utils.ErrorNoLoginInfo), nil
+		return l.fail(utils.ErrorNoLoginInfo)
 	}
 
 	check, err := l.check(in.Name, in.ServiceId)
 	if err != utils.Ok {
-		return l.fail(err), nil
+		return l.fail(err)
 	}
 
 	edit := dao.Role{
@@ -59,7 +59,7 @@ func (l *RoleEditLogic) RoleEdit(in *admin.RoleEditRequest) (*admin.Response, er
 	var editErr error
 	if id > utils.NumberZero {
 		if check.Id > utils.NumberZero && check.Id != id {
-			return l.fail(utils.ErrorExist), nil
+			return l.fail(utils.ErrorExist)
 		}
 
 		w := modelbase.Params{Eq: map[string]string{"id": strconv.FormatInt(in.Id, utils.NumberTen)}}
@@ -67,10 +67,10 @@ func (l *RoleEditLogic) RoleEdit(in *admin.RoleEditRequest) (*admin.Response, er
 		InfoErr := modelObj.GetOne(w, &info)
 		if InfoErr != nil {
 			l.Logger.Error(InfoErr)
-			return l.fail(utils.Fail), nil
+			return l.fail(utils.Fail)
 		}
 		if info.Id < utils.NumberOne {
-			return l.fail(utils.ErrorNotFund), nil
+			return l.fail(utils.ErrorNotFund)
 		}
 
 		edit.Id = info.Id
@@ -80,7 +80,7 @@ func (l *RoleEditLogic) RoleEdit(in *admin.RoleEditRequest) (*admin.Response, er
 		_, editErr = modelObj.Edit(w, edit)
 	} else {
 		if check.Id > utils.NumberZero {
-			return l.fail(utils.ErrorExist), nil
+			return l.fail(utils.ErrorExist)
 		}
 		editErr = modelObj.Create(&edit)
 		id = edit.Id
@@ -88,10 +88,10 @@ func (l *RoleEditLogic) RoleEdit(in *admin.RoleEditRequest) (*admin.Response, er
 
 	if editErr != nil {
 		l.Logger.Error(editErr)
-		return l.fail(utils.Fail), nil
+		return l.fail(utils.Fail)
 	}
 
-	return l.ok(id), nil
+	return l.ok(id)
 }
 
 func (l *RoleEditLogic) check(name string, id int64) (dao.Role, utils.Status) {