Fixes
This commit is contained in:
@@ -99,7 +99,7 @@ namespace Musify.Application.PlayLists.Handlers
|
||||
await eventBus.PublishAsync(new UpdatePlayListPictureEvent(
|
||||
playList.Id,
|
||||
storageConfiguration.BucketName,
|
||||
playList.OriginalPictureName,
|
||||
Path.Combine(playListConfiguration.Routes.OriginalPicturesPath, playList.OriginalPictureName),
|
||||
new ImageSize(
|
||||
playListConfiguration.Routes.SmallPicturesPath,
|
||||
playListConfiguration.PicturesSizes.SmallPictureWidth,
|
||||
|
||||
@@ -37,18 +37,18 @@ namespace Musify.Application.PlayLists.Handlers
|
||||
UpdateDescription(playListEntity, request.NewDescription);
|
||||
var updatePictureResult = await UpdatePicture(playListEntity, request.NewPicture, cancellationToken);
|
||||
if (!updatePictureResult.IsSuccess)
|
||||
return updatePictureResult.Map(_ => PlayListResponse.FromEntity(playListEntity));
|
||||
return updatePictureResult;
|
||||
|
||||
try
|
||||
{
|
||||
database.PlayLists.Update(playListEntity);
|
||||
await database.SaveChangesAsync(cancellationToken);
|
||||
await PublishPlayListPictureEvent(playListEntity, updatePictureResult.Value, cancellationToken);
|
||||
await PublishPlayListPictureEvent(playListEntity, cancellationToken);
|
||||
await RemoveOldPictures(playListEntity, cancellationToken);
|
||||
}
|
||||
catch (Exception exception)
|
||||
{
|
||||
await Rollback(updatePictureResult.Value, cancellationToken);
|
||||
await Rollback(playListEntity.OriginalPictureName, cancellationToken);
|
||||
logger.LogError(exception, "Failed to update playlist {PlayListId}", playListEntity.Id);
|
||||
return Result.Error($"Failed to update playlist {playListEntity.Id}");
|
||||
}
|
||||
@@ -72,7 +72,7 @@ namespace Musify.Application.PlayLists.Handlers
|
||||
playList.Description = newDescription;
|
||||
}
|
||||
|
||||
async Task<Result<string>> UpdatePicture(PlayList playList, IFileData? newPicture, CancellationToken cancellationToken)
|
||||
async Task<Result> UpdatePicture(PlayList playList, IFileData? newPicture, CancellationToken cancellationToken)
|
||||
{
|
||||
if (newPicture is null)
|
||||
return Result.Success();
|
||||
@@ -94,17 +94,17 @@ namespace Musify.Application.PlayLists.Handlers
|
||||
}
|
||||
|
||||
playList.OriginalPictureName = pictureName;
|
||||
return Result.Success(pictureKey);
|
||||
return Result.Success();
|
||||
}
|
||||
|
||||
async Task<Result> PublishPlayListPictureEvent(PlayList playList, string pictureKey, CancellationToken cancellationToken)
|
||||
async Task<Result> PublishPlayListPictureEvent(PlayList playList, CancellationToken cancellationToken)
|
||||
{
|
||||
try
|
||||
{
|
||||
await eventBus.PublishAsync(new UpdatePlayListPictureEvent(
|
||||
playList.Id,
|
||||
storageConfiguration.BucketName,
|
||||
pictureKey,
|
||||
Path.Combine(playListConfiguration.Routes.OriginalPicturesPath, playList.OriginalPictureName),
|
||||
new ImageSize(
|
||||
playListConfiguration.Routes.SmallPicturesPath,
|
||||
playListConfiguration.PicturesSizes.SmallPictureWidth,
|
||||
|
||||
@@ -94,17 +94,17 @@ namespace Musify.Application.Tracks.Handler
|
||||
await eventBus.PublishAsync(new UpdateTrackPictureEvent(
|
||||
track.Id,
|
||||
storageConfiguration.BucketName,
|
||||
track.OriginalPictureName,
|
||||
Path.Combine(trackConfiguration.Routes.OriginalPicturesPath, track.OriginalPictureName),
|
||||
new ImageSize(
|
||||
trackConfiguration.Routes.PresetSmallPicture,
|
||||
trackConfiguration.Routes.SmallPicturesPath,
|
||||
trackConfiguration.PicturesSizes.SmallPictureWidth,
|
||||
trackConfiguration.PicturesSizes.SmallPictureHeight),
|
||||
new ImageSize(
|
||||
trackConfiguration.Routes.PresetMediumPicture,
|
||||
trackConfiguration.Routes.MediumPicturesPath,
|
||||
trackConfiguration.PicturesSizes.MediumPictureWidth,
|
||||
trackConfiguration.PicturesSizes.MediumPictureHeight),
|
||||
new ImageSize(
|
||||
trackConfiguration.Routes.PresetLargePicture,
|
||||
trackConfiguration.Routes.LargePicturesPath,
|
||||
trackConfiguration.PicturesSizes.LargePictureWidth,
|
||||
trackConfiguration.PicturesSizes.LargePictureHeight)), cancellationToken);
|
||||
}
|
||||
@@ -124,7 +124,7 @@ namespace Musify.Application.Tracks.Handler
|
||||
await eventBus.PublishAsync(new UpdateTrackAudioEvent(
|
||||
track.Id,
|
||||
storageConfiguration.BucketName,
|
||||
track.OriginalAudioName,
|
||||
Path.Combine(trackConfiguration.Routes.OriginalAudiosPath, track.OriginalAudioName),
|
||||
storageConfiguration.BucketName,
|
||||
destinationFolderAudio), cancellationToken);
|
||||
}
|
||||
|
||||
@@ -54,7 +54,6 @@ namespace Musify.Application.Tracks.Handler
|
||||
{
|
||||
await RemoveFile(Path.Combine(trackConfiguration.Routes.OriginalAudiosPath, track.OriginalAudioName),
|
||||
cancellationToken);
|
||||
|
||||
await RemoveFile(Path.Combine(trackConfiguration.Routes.ProcessedAudiosPath, track.AudioFolderName),
|
||||
cancellationToken);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user