RepoMirrors/osu
1
0
Fork 0
mirror of https://github.com/ppy/osu synced 2025-01-12 17:19:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove the ability to specify a filename in ReplaceFile

Browse Source
This commit is contained in:
Dean Herbert 2021-11-29 18:08:02 +09:00
parent 448b2d1190
commit a7e4e7be3a
4 changed files with 10 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
osu.Game/Beatmaps/BeatmapManager.cs
View File

@ -288,9 +288,9 @@ namespace osu.Game.Beatmaps
#region Implementation of IModelFileManager<in BeatmapSetInfo,in BeatmapSetFileInfo> #region Implementation of IModelFileManager<in BeatmapSetInfo,in BeatmapSetFileInfo>
public void ReplaceFile(BeatmapSetInfo model, BeatmapSetFileInfo file, Stream contents, string filename = null) public void ReplaceFile(BeatmapSetInfo model, BeatmapSetFileInfo file, Stream contents)
{ {
beatmapModelManager.ReplaceFile(model, file, contents, filename); beatmapModelManager.ReplaceFile(model, file, contents);
} }
public void DeleteFile(BeatmapSetInfo model, BeatmapSetFileInfo file) public void DeleteFile(BeatmapSetInfo model, BeatmapSetFileInfo file)

5
osu.Game/Database/ArchiveModelManager.cs
View File

@ -453,13 +453,12 @@ namespace osu.Game.Database
/// <param name="model">The item to operate on.</param> /// <param name="model">The item to operate on.</param>
/// <param name="file">The existing file to be replaced.</param> /// <param name="file">The existing file to be replaced.</param>
/// <param name="contents">The new file contents.</param> /// <param name="contents">The new file contents.</param>
/// <param name="filename">An optional filename for the new file. Will use the previous filename if not specified.</param> public void ReplaceFile(TModel model, TFileModel file, Stream contents)
public void ReplaceFile(TModel model, TFileModel file, Stream contents, string filename = null)
{ {
using (ContextFactory.GetForWrite()) using (ContextFactory.GetForWrite())
{ {
DeleteFile(model, file); DeleteFile(model, file);
AddFile(model, contents, filename ?? file.Filename); AddFile(model, contents, file.Filename);
} }
} }

3
osu.Game/Database/IModelFileManager.cs
View File

@ -15,8 +15,7 @@ namespace osu.Game.Database
/// <param name="model">The item to operate on.</param> /// <param name="model">The item to operate on.</param>
/// <param name="file">The existing file to be replaced.</param> /// <param name="file">The existing file to be replaced.</param>
/// <param name="contents">The new file contents.</param> /// <param name="contents">The new file contents.</param>
/// <param name="filename">An optional filename for the new file. Will use the previous filename if not specified.</param> void ReplaceFile(TModel model, TFileModel file, Stream contents);
void ReplaceFile(TModel model, TFileModel file, Stream contents, string filename = null);
/// <summary> /// <summary>
/// Delete an existing file. /// Delete an existing file.

11
osu.Game/Screens/Edit/Setup/ResourcesSection.cs
View File

@ -78,9 +78,9 @@ namespace osu.Game.Screens.Edit.Setup
using (var stream = info.OpenRead()) using (var stream = info.OpenRead())
{ {
if (oldFile != null) if (oldFile != null)
beatmaps.ReplaceFile(set, oldFile, stream, info.Name); beatmaps.DeleteFile(set, oldFile);
else
beatmaps.AddFile(set, stream, info.Name); beatmaps.AddFile(set, stream, info.Name);
} }
working.Value.Metadata.BackgroundFile = info.Name; working.Value.Metadata.BackgroundFile = info.Name;
@ -105,9 +105,8 @@ namespace osu.Game.Screens.Edit.Setup
using (var stream = info.OpenRead()) using (var stream = info.OpenRead())
{ {
if (oldFile != null) if (oldFile != null)
beatmaps.ReplaceFile(set, oldFile, stream, info.Name); beatmaps.DeleteFile(set, oldFile);
else beatmaps.AddFile(set, stream, info.Name);
beatmaps.AddFile(set, stream, info.Name);
} }
working.Value.Metadata.AudioFile = info.Name; working.Value.Metadata.AudioFile = info.Name;