Pathoschild/SMAPI
1
0
Fork 0
mirror of https://github.com/Pathoschild/SMAPI.git synced 2026-10-12 02:08:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

avoid unneeded case-insensitive equals for ".."

Browse Source
This commit is contained in:
Jesse Plamondon-Willard
2025-12-26 13:03:25 -05:00
parent cb53d1861b
commit 28bc30953d
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/SMAPI/Framework/ContentManagers/ModContentManager.cs
+8 -6
View File
@@ -415,14 +415,16 @@ internal sealed class ModContentManager : BaseContentManager
if (Path.IsPathRooted(imageSource))
throw this.GetPathError(relativeMapFolder, imageSource, $"Tilesheet paths must not be an absolute path ({Path.GetPathRoot(imageSource)}).");
string[] imageSourcePathSegments = PathUtilities.GetSegments(imageSource);
if (imageSourcePathSegments.Contains(".."))
{
int climbingCount = imageSourcePathSegments.Count(segment => segment.Equals("..", StringComparison.OrdinalIgnoreCase));
if (climbingCount > 1)
string[] imageSourcePathSegments = PathUtilities.GetSegments(imageSource);
if (imageSourcePathSegments.Contains(".."))
{
var offendingSegments = imageSourcePathSegments.TakeWhile(seg => seg.Equals(".."));
throw this.GetPathError(relativeMapFolder, imageSource, $"Tilesheet paths must not climb more than one directory ({string.Join('/', offendingSegments)}/).");
int climbingCount = imageSourcePathSegments.Count(segment => segment == "..");
if (climbingCount > 1)
{
var offendingSegments = imageSourcePathSegments.TakeWhile(segment => segment == "..");
throw this.GetPathError(relativeMapFolder, imageSource, $"Tilesheet paths must not climb more than one directory ({string.Join('/', offendingSegments)}/).");
}
}
}