epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Give an proper error when a file is not found

Browse Source
This commit is contained in:
abcdefg30
2017-02-04 00:45:32 +01:00
parent ad5a9a85f9
commit aa73b19ee2
2 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
OpenRA.Game/FileSystem/FileSystem.cs
View File

@@ -267,6 +267,12 @@ namespace OpenRA.FileSystem
if (s != null) if (s != null)
return true; return true;
// The file should be in an explicit package (but we couldn't find it)
// Thus don't try to find it using the filename (which contains the invalid '|' char)
// This can be removed once the TODO below is resolved
if (explicitSplit > 0)
return false;
// Ask each package individually // Ask each package individually
// TODO: This fallback can be removed once the filesystem cleanups are complete // TODO: This fallback can be removed once the filesystem cleanups are complete
var package = mountedPackages.Keys.LastOrDefault(x => x.Contains(filename)); var package = mountedPackages.Keys.LastOrDefault(x => x.Contains(filename));

3
OpenRA.Game/FileSystem/Folder.cs
View File

@@ -9,6 +9,7 @@
*/ */
#endregion #endregion
using System;
using System.Collections.Generic; using System.Collections.Generic;
using System.IO; using System.IO;
@@ -47,7 +48,7 @@ namespace OpenRA.FileSystem
public bool Contains(string filename) public bool Contains(string filename)
{ {
var combined = Path.Combine(path, filename); var combined = Path.Combine(path, filename);
return combined.StartsWith(path) && File.Exists(combined); return combined.StartsWith(path, StringComparison.Ordinal) && File.Exists(combined);
} }
public void Update(string filename, byte[] contents) public void Update(string filename, byte[] contents)