stjr/UltimateZoneBuilder
0
0
Fork 0
mirror of https://git.do.srb2.org/STJr/UltimateZoneBuilder.git synced 2024-11-29 15:11:56 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fixed a crash when trying to load a voxel that could not be found multiple times. Fixes #1024

Browse source 
Fixed a problem where voxels defined in VOXELDEF would not be found
This commit is contained in:
biwa 2024-03-03 17:43:22 +01:00
parent f28a4b10a8
commit 2df6004bf0
3 changed files with 63 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
Source/Core/Data/DirectoryReader.cs
View file

@ -358,8 +358,27 @@ namespace CodeImp.DoomBuilder.Data
try try
{ {
// Find in voxels directory // Find in voxels directory
string path = Path.Combine(VOXELS_DIR, Path.GetDirectoryName(name)); string filename = null;
string filename = FindFirstFile(path, Path.GetFileName(name), true);
// There are different places we have to look for the file, depending on how it was defined. Right now
// there's no way to know if the definition comes from VOXELDEF or spreite replacement, so we're assuming
// it's from VOXELDEF if there's an extension given.
// - Auto-detect for sprite name: must be in the "voxels" directory
// - Just given as file name without path in VOXELDEF: must be in the root directory
// - Given as full path and file name in VOXELDEF: must be in exactly that spot
string path = Path.GetDirectoryName(name);
if (string.IsNullOrWhiteSpace(path))
{
if (string.IsNullOrWhiteSpace(Path.GetExtension(name)))
filename = FindFirstFile(VOXELS_DIR, Path.GetFileName(name), true);
else
filename = FindFirstFileWithExt("", Path.GetFileName(name), true);
}
else
{
filename = FindFirstFileWithExt(path, Path.GetFileName(name), false);
}
if ((filename != null) && FileExists(filename)) if ((filename != null) && FileExists(filename))
{ {
voxellocation = location.GetDisplayName(); voxellocation = location.GetDisplayName();

33
Source/Core/Data/PK3Reader.cs
View file

@ -454,15 +454,42 @@ namespace CodeImp.DoomBuilder.Data
if(voxel != null) return voxel; if(voxel != null) return voxel;
} }
string pfilename = name.Replace('\\', '^'); name = name.Replace('\\', '^');
try
{
// Find in voxels directory
string filename = null;
// There are different places we have to look for the file, depending on how it was defined. Right now
// there's no way to know if the definition comes from VOXELDEF or spreite replacement, so we're assuming
// it's from VOXELDEF if there's an extension given.
// - Auto-detect for sprite name: must be in the "voxels" directory
// - Just given as file name without path in VOXELDEF: must be in the root directory
// - Given as full path and file name in VOXELDEF: must be in exactly that spot
string path = Path.GetDirectoryName(name);
if (string.IsNullOrWhiteSpace(path))
{
if (string.IsNullOrWhiteSpace(Path.GetExtension(name)))
filename = FindFirstFile(VOXELS_DIR, Path.GetFileName(name), true);
else
filename = FindFirstFileWithExt("", Path.GetFileName(name), true);
}
else
{
filename = FindFirstFileWithExt(path, Path.GetFileName(name), false);
}
// Find in sprites directory
string filename = FindFirstFile(VOXELS_DIR, pfilename, true);
if ((filename != null) && FileExists(filename)) if ((filename != null) && FileExists(filename))
{ {
voxellocation = location.GetDisplayName(); voxellocation = location.GetDisplayName();
return LoadFile(filename); return LoadFile(filename);
} }
}
catch (Exception e)
{
if (!Silent) General.ErrorLogger.Add(ErrorType.Error, e.GetType().Name + " while loading voxel \"" + name + "\" from PK3: " + e.Message);
}
// Nothing found // Nothing found
return null; return null;

10
Source/Core/GZBuilder/Models/ModelLoader.cs
View file

@ -39,14 +39,16 @@ namespace CodeImp.DoomBuilder.GZBuilder.Models
} }
} }
//clear unneeded data
mde.SkinNames = null;
mde.ModelNames = null;
if (mde.Model.Meshes == null || mde.Model.Meshes.Count == 0) if (mde.Model.Meshes == null || mde.Model.Meshes.Count == 0)
{ {
mde.Model = null; mde.Model = null;
} }
else
{
//clear unneeded data
mde.SkinNames = null;
mde.ModelNames = null;
}
} }
private static void LoadModel(ModelData mde, List<DataReader> containers) private static void LoadModel(ModelData mde, List<DataReader> containers)