* Updated patch/update post API to allow file modification (#29447) * WIP * WIP * Atatched new files ton post * WIP: deleting removed files * Deleted removed files and invalidated file metadata cache * removed file ignore logif from update post API * Added TestFindExclusives * Added tests for DeleteForPostByIds * Added app layer tests * Added tests * Added API level tests * test enhancements * Fixed a test * Edit history include file metadata (#29505) * Send file metadata in edit history metadata * Added app tests * Added store tests * Added tests for populateEditHistoryFileMetadata{ * Added cache to avoid repetitigve DB calls for edits with only message changes * Added API tests * i18m fix * removed commented code * Improved test helper * Show attachments in edit history RHS (#29519) * Send file metadata in edit history metadata * Added app tests * Added store tests * Added tests for populateEditHistoryFileMetadata{ * Added cache to avoid repetitigve DB calls for edits with only message changes * Added API tests * i18m fix * WIUP: displa files in edit * removed commented code * Displayed file in edit history * Handled file icon * Fixed closing history component on clicking on file * Simplified selector * Simplified selector * Improved test helper * Disabled action menu on edit history file * Added tests * Improved selector * Updated snapshot * review Fixes * restructured componnets * Updated test * Updated test * Restore post api (#29643) * Restore post version API WIP * Undelete files WIP * Added store tests * Created post restore API * Updated updatepost safeUpdate signature * review fixex and improvements * Fixed an app test * Added API laer tests * Added API tests and OpenAPI specs * Fixed a typo * Allow editing files when editing posts (#29709) * WIP - basic view files when editing post * Cleanup * bg color * Added text editor tests for files * WIP * WIP * removed debug log * Allowed admin to add and remove files on someone else's post * Handled drafts and scheduled posts * linter fixes * Updated snapshot * server test fix * CI * Added doc * Restore post api integration (#29719) * WIP - basic view files when editing post * Cleanup * bg color * Added text editor tests for files * WIP * WIP * removed debug log * Allowed admin to add and remove files on someone else's post * Handled drafts and scheduled posts * linter fixes * Updated snapshot * server test fix * Used new API to restore post * handled edut limit and undo * lint fix * added comments * Fixed edit post item tests * Fixed buttons * Aded snapshots * fix test * Updated snapshot * Minor fixes * fixed snapshot * Edit file dnd area (#29763) * dnd wip * DND continued * Supported multiple unbind dragster funcs * lint fixes * Got center channel file drop working when editing a post * file dnd working with center channel and rhs * file dnd working with center channel and rhs * removed unneeded stopPropogation calls * cleanup * DND overlay fix * Lint fix * Advanced text editor test updates for file upload overlay * fixed use upload hook tests * Updated some more snapshots * minor cleanup * Updated i18n * removed need of array for dragster unbind events * lint fixes * edit history cursor * Fixed bugu causing faliure to delete empty posts (#29778) * Files in restore confirmation (#29781) * Added files to restore post confirmation dialog * Fixed post restore toast colors * Fixed restore bug * Fixed restore confirmation toast tests * a11y improvement and modal width fix * Edit attachment misc fixes (#29808) * Removed single image actions in restore post confirmation dialog * Fixed file drop overlay size and position * Made edit indiator accessible * Lint fix * Added bunch of more tests * ANother test migrated from enzyme to react testing library * More test enhancements * More test enhancements * More test enhancements * lint fixes * Fixed a test * Added missing snapshots * Test fixes
71 строка
3.0 KiB
Go
71 строка
3.0 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package app
|
|
|
|
import (
|
|
"github.com/mattermost/mattermost/server/public/model"
|
|
"github.com/mattermost/mattermost/server/public/shared/request"
|
|
"github.com/mattermost/mattermost/server/public/utils"
|
|
)
|
|
|
|
func (a *App) processPostFileChanges(rctx request.CTX, newPost, oldPost *model.Post, updatePostOptions *model.UpdatePostOptions) (model.StringArray, *model.AppError) {
|
|
newFileIDs := model.RemoveDuplicateStrings(newPost.FileIds)
|
|
oldFileIDs := model.RemoveDuplicateStrings(oldPost.FileIds)
|
|
|
|
addedFileIDs, removedFileIDs, unchangedFileIDs := utils.FindExclusives(newFileIDs, oldFileIDs)
|
|
|
|
if len(addedFileIDs) > 0 {
|
|
if updatePostOptions != nil && updatePostOptions.IsRestorePost {
|
|
err := a.Srv().Store().FileInfo().RestoreForPostByIds(rctx, newPost.Id, addedFileIDs)
|
|
if err != nil {
|
|
return nil, model.NewAppError("app.processPostFileChanges", "app.file_info.undelete_for_post_ids.app_error", map[string]any{"post_id": newPost.Id}, "", 0).Wrap(err)
|
|
}
|
|
} else {
|
|
a.attachNewFilesToPost(rctx, newPost, addedFileIDs, unchangedFileIDs)
|
|
}
|
|
}
|
|
|
|
if len(removedFileIDs) > 0 {
|
|
if appErr := a.detachFilesFromPost(rctx, newPost.Id, removedFileIDs); appErr != nil {
|
|
return nil, appErr
|
|
}
|
|
}
|
|
|
|
filesChanged := len(addedFileIDs) > 0 || len(removedFileIDs) > 0
|
|
if filesChanged {
|
|
// if files were modified, invalidate the file metadata cache for the post
|
|
// so that the updated file metadata can be returned.
|
|
a.Srv().Store().FileInfo().InvalidateFileInfosForPostCache(newPost.Id, false)
|
|
}
|
|
|
|
return newPost.FileIds, nil
|
|
}
|
|
|
|
func (a *App) attachNewFilesToPost(rctx request.CTX, post *model.Post, addedFileIDs, unchangedFileIDs []string) {
|
|
// for newly added files, we need to attach them to the post
|
|
|
|
// intentionally using UserID from session instead of post.UserID
|
|
// to support admin attaching files in someone else's post.
|
|
// Admins can edit other's posts, including message, removing existing files,
|
|
// and attaching new files.
|
|
// When an admin uploads new files, they are associated with their user ID. So, when attaching
|
|
// these file to a post, we need to search for their FileInfo entry
|
|
// by the admin's user ID and not the post author's user ID.
|
|
userId := rctx.Session().UserId
|
|
attachedFileIDs := a.attachFileIDsToPost(rctx, post.Id, post.ChannelId, userId, addedFileIDs)
|
|
if len(attachedFileIDs) != len(addedFileIDs) {
|
|
// if not all files could be attached, the final list of files
|
|
// is those that could be attached + the existing, unchanged files
|
|
post.FileIds = append(attachedFileIDs, unchangedFileIDs...)
|
|
}
|
|
}
|
|
|
|
func (a *App) detachFilesFromPost(rctx request.CTX, postId string, removedFileIDs []string) *model.AppError {
|
|
if err := a.Srv().Store().FileInfo().DeleteForPostByIds(rctx, postId, removedFileIDs); err != nil {
|
|
return model.NewAppError("app.detachFilesFromPost", "app.file_info.delete_for_post_ids.app_error", map[string]any{"post_id": postId}, "", 0).Wrap(err)
|
|
}
|
|
|
|
return nil
|
|
}
|