diff --git a/SabreTools.DatFiles/DatFile.ToMetadata.cs b/SabreTools.DatFiles/DatFile.ToMetadata.cs index 7f6cfc37..93b598d6 100644 --- a/SabreTools.DatFiles/DatFile.ToMetadata.cs +++ b/SabreTools.DatFiles/DatFile.ToMetadata.cs @@ -15,7 +15,7 @@ namespace SabreTools.DatFiles public Models.Metadata.MetadataFile? ConvertToMetadata(bool ignoreblanks = false) { // If we don't have items, we can't do anything - if (Items == null || Items.Count == 0) + if (Items.Count == 0) return null; // Create an object to hold the data @@ -40,7 +40,7 @@ namespace SabreTools.DatFiles public Models.Metadata.MetadataFile? ConvertToMetadataDB(bool ignoreblanks = false) { // If we don't have items, we can't do anything - if (ItemsDB == null) + if (ItemsDB.GetItems().Count == 0) return null; // Create an object to hold the data @@ -64,10 +64,6 @@ namespace SabreTools.DatFiles /// private Models.Metadata.Header? ConvertHeader() { - // If the header is invalid, we can't do anything - if (Header == null) - return null; - // Create an internal header var header = Header.GetInternalClone(); diff --git a/SabreTools.DatTools/ExtraIni.cs b/SabreTools.DatTools/ExtraIni.cs index 0e8cbc67..fd822768 100644 --- a/SabreTools.DatTools/ExtraIni.cs +++ b/SabreTools.DatTools/ExtraIni.cs @@ -87,7 +87,7 @@ namespace SabreTools.DatTools public bool ApplyExtras(DatFile datFile, bool throwOnError = false) { // If we have no extras, don't attempt to apply and just return true - if (Items == null || Items.Count == 0) + if (Items.Count == 0) return true; var watch = new InternalStopwatch("Applying extra mappings to DAT"); @@ -150,7 +150,7 @@ namespace SabreTools.DatTools public bool ApplyExtrasDB(DatFile datFile, bool throwOnError = false) { // If we have no extras, don't attempt to apply and just return true - if (Items == null || Items.Count == 0) + if (Items.Count == 0) return true; var watch = new InternalStopwatch("Applying extra mappings to DAT"); diff --git a/SabreTools.DatTools/Setter.cs b/SabreTools.DatTools/Setter.cs index 26986ac1..8dce2aa0 100644 --- a/SabreTools.DatTools/Setter.cs +++ b/SabreTools.DatTools/Setter.cs @@ -134,7 +134,7 @@ namespace SabreTools.DatTools public void SetFields(DatHeader datHeader) { // If we have an invalid input, return - if (datHeader == null || HeaderFieldMappings.Count == 0) + if (HeaderFieldMappings.Count == 0) return; foreach (var kvp in HeaderFieldMappings)