horizoncd/horizon

Method dao.Create has 57 lines of code (exceeds 50 allowed). Consider refactoring.

Open

func (d *dao) Create(ctx context.Context, group *models.Group) (*models.Group, error) {
    currentUser, err := common.UserFromContext(ctx)
    if err != nil {
        return nil, err
    }

Method dao.Create has 11 return statements (exceeds 4 allowed).

Open

func (d *dao) Create(ctx context.Context, group *models.Group) (*models.Group, error) {
    currentUser, err := common.UserFromContext(ctx)
    if err != nil {
        return nil, err
    }

Method dao.Transfer has 9 return statements (exceeds 4 allowed).

Open

func (d *dao) Transfer(ctx context.Context, id, newParentID uint) error {
    currentUser, err := common.UserFromContext(ctx)
    if err != nil {
        return nil
    }

Method dao.ListByTraversalIDsContains has 6 return statements (exceeds 4 allowed).

Open

func (d *dao) ListByTraversalIDsContains(ctx context.Context, ids []uint) ([]*models.Group, error) {
    if len(ids) == 0 {
        return nil, nil
    }
    var groups []*models.Group

Similar blocks of code found in 2 locations. Consider refactoring.

Open

func (d *dao) GetByPaths(ctx context.Context, paths []string) ([]*models.Group, error) {
    var groups []*models.Group
    result := d.db.WithContext(ctx).Raw(dbcommon.GroupQueryByPaths, paths).Scan(&groups)
 
    if result.Error != nil {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

func (d *dao) GetByIDs(ctx context.Context, ids []uint) ([]*models.Group, error) {
    var groups []*models.Group
    result := d.db.WithContext(ctx).Raw(dbcommon.GroupQueryByIDs, ids).Scan(&groups)
 
    if result.Error != nil {

• Read up
  Read up