Chore: Deprecate FolderID in LibraryElementWithMeta (#77406)

This commit is contained in:
Kat Yang 2023-10-31 11:50:02 -04:00 committed by GitHub
parent 87717849da
commit f7ce6f8b41
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 5 deletions

View File

@ -227,6 +227,7 @@ func (l *LibraryElementService) deleteLibraryElement(c context.Context, signedIn
if err != nil {
return err
}
// nolint:staticcheck
if err := l.requireEditPermissionsOnFolder(c, signedInUser, element.FolderID); err != nil {
return err
}
@ -314,7 +315,7 @@ func (l *LibraryElementService) getLibraryElements(c context.Context, store db.D
leDtos[i] = model.LibraryElementDTO{
ID: libraryElement.ID,
OrgID: libraryElement.OrgID,
FolderID: libraryElement.FolderID,
FolderID: libraryElement.FolderID, // nolint:staticcheck
FolderUID: libraryElement.FolderUID,
UID: libraryElement.UID,
Name: libraryElement.Name,
@ -436,7 +437,7 @@ func (l *LibraryElementService) getAllLibraryElements(c context.Context, signedI
retDTOs = append(retDTOs, model.LibraryElementDTO{
ID: element.ID,
OrgID: element.OrgID,
FolderID: element.FolderID,
FolderID: element.FolderID, // nolint:staticcheck
FolderUID: element.FolderUID,
UID: element.UID,
Name: element.Name,
@ -593,6 +594,7 @@ func (l *LibraryElementService) patchLibraryElement(c context.Context, signedInU
if cmd.Model == nil {
libraryElement.Model = elementInDB.Model
}
// nolint:staticcheck
if err := l.handleFolderIDPatches(c, &libraryElement, elementInDB.FolderID, cmd.FolderID, signedInUser); err != nil {
return err
}
@ -711,7 +713,7 @@ func (l *LibraryElementService) getElementsForDashboardID(c context.Context, das
libraryElementMap[element.UID] = model.LibraryElementDTO{
ID: element.ID,
OrgID: element.OrgID,
FolderID: element.FolderID,
FolderID: element.FolderID, // nolint:staticcheck
UID: element.UID,
Name: element.Name,
Kind: element.Kind,
@ -757,6 +759,7 @@ func (l *LibraryElementService) connectElementsToDashboardID(c context.Context,
if err != nil {
return err
}
// nolint:staticcheck
if err := l.requireViewPermissionsOnFolder(c, signedInUser, element.FolderID); err != nil {
return err
}

View File

@ -42,6 +42,7 @@ type LibraryElement struct {
type LibraryElementWithMeta struct {
ID int64 `xorm:"pk autoincr 'id'"`
OrgID int64 `xorm:"org_id"`
// Deprecated: use FolderUID instead
FolderID int64 `xorm:"folder_id"`
UID string `xorm:"uid"`
Name string