diff --git a/pkg/services/folder/folderimpl/folder.go b/pkg/services/folder/folderimpl/folder.go index 2dae51cd872..8014dbf7dcf 100644 --- a/pkg/services/folder/folderimpl/folder.go +++ b/pkg/services/folder/folderimpl/folder.go @@ -533,7 +533,7 @@ func (s *Service) Move(ctx context.Context, cmd *folder.MoveFolderCommand) (*fol // current folder height + current folder + parent folder + parent folder depth should be less than or equal 8 if folderHeight+len(parents)+2 > folder.MaxNestedFolderDepth { - return nil, folder.ErrMaximumDepthReached + return nil, folder.ErrMaximumDepthReached.Errorf("failed to move folder") } // if the current folder is already a parent of newparent, we should return error @@ -749,7 +749,7 @@ func (s *Service) validateParent(ctx context.Context, orgID int64, parentUID str } if len(ancestors) == folder.MaxNestedFolderDepth { - return folder.ErrMaximumDepthReached + return folder.ErrMaximumDepthReached.Errorf("failed to validate parent folder") } // Create folder under itself is not allowed diff --git a/pkg/services/folder/folderimpl/sqlstore.go b/pkg/services/folder/folderimpl/sqlstore.go index 12691f5d056..30d1256dd70 100644 --- a/pkg/services/folder/folderimpl/sqlstore.go +++ b/pkg/services/folder/folderimpl/sqlstore.go @@ -284,7 +284,7 @@ func (ss *sqlStore) getParentsMySQL(ctx context.Context, cmd folder.GetParentsQu folders = append(folders, f) uid = f.ParentUID if len(folders) > folder.MaxNestedFolderDepth { - return folder.ErrFolderTooDeep + return folder.ErrMaximumDepthReached.Errorf("failed to get parent folders iteratively") } } return nil diff --git a/pkg/services/folder/model.go b/pkg/services/folder/model.go index ef32296a9c2..b3a11466357 100644 --- a/pkg/services/folder/model.go +++ b/pkg/services/folder/model.go @@ -11,7 +11,6 @@ var ErrMaximumDepthReached = errutil.NewBase(errutil.StatusBadRequest, "folder.m var ErrBadRequest = errutil.NewBase(errutil.StatusBadRequest, "folder.bad-request") var ErrDatabaseError = errutil.NewBase(errutil.StatusInternal, "folder.database-error") var ErrInternal = errutil.NewBase(errutil.StatusInternal, "folder.internal") -var ErrFolderTooDeep = errutil.NewBase(errutil.StatusInternal, "folder.too-deep") var ErrCircularReference = errutil.NewBase(errutil.StatusBadRequest, "folder.circular-reference", errutil.WithPublicMessage("Circular reference detected")) const (