diff options
| author | Jesse Plamondon-Willard <github@jplamondonw.com> | 2017-05-13 20:41:00 -0400 |
|---|---|---|
| committer | Jesse Plamondon-Willard <github@jplamondonw.com> | 2017-05-13 20:41:00 -0400 |
| commit | 7f368aa8896baa551aa156a8e67e9dd16416022d (patch) | |
| tree | c741e209e0161e3234d86559febce0e83d2e5a42 /src/StardewModdingAPI/Framework/ModLoading | |
| parent | 9b6c0d1021b07ec04b589f1bd0eb69e36082b600 (diff) | |
| download | SMAPI-7f368aa8896baa551aa156a8e67e9dd16416022d.tar.gz SMAPI-7f368aa8896baa551aa156a8e67e9dd16416022d.tar.bz2 SMAPI-7f368aa8896baa551aa156a8e67e9dd16416022d.zip | |
enforce metadata.SetStatus() instead of setting properties directly (#285)
Diffstat (limited to 'src/StardewModdingAPI/Framework/ModLoading')
| -rw-r--r-- | src/StardewModdingAPI/Framework/ModLoading/ModMetadata.cs | 4 | ||||
| -rw-r--r-- | src/StardewModdingAPI/Framework/ModLoading/ModResolver.cs | 6 |
2 files changed, 4 insertions, 6 deletions
diff --git a/src/StardewModdingAPI/Framework/ModLoading/ModMetadata.cs b/src/StardewModdingAPI/Framework/ModLoading/ModMetadata.cs index 7be85a83..5ec2d4e0 100644 --- a/src/StardewModdingAPI/Framework/ModLoading/ModMetadata.cs +++ b/src/StardewModdingAPI/Framework/ModLoading/ModMetadata.cs @@ -21,10 +21,10 @@ namespace StardewModdingAPI.Framework.ModLoading public ModCompatibility Compatibility { get; } /// <summary>The metadata resolution status.</summary> - public ModMetadataStatus Status { get; set; } + public ModMetadataStatus Status { get; private set; } /// <summary>The reason the metadata is invalid, if any.</summary> - public string Error { get; set; } + public string Error { get; private set; } /********* diff --git a/src/StardewModdingAPI/Framework/ModLoading/ModResolver.cs b/src/StardewModdingAPI/Framework/ModLoading/ModResolver.cs index 829575af..9b26e8b0 100644 --- a/src/StardewModdingAPI/Framework/ModLoading/ModResolver.cs +++ b/src/StardewModdingAPI/Framework/ModLoading/ModResolver.cs @@ -188,8 +188,7 @@ namespace StardewModdingAPI.Framework.ModLoading } if (missingMods != null) { - mod.Status = ModMetadataStatus.Failed; - mod.Error = $"it requires mods which aren't installed ({missingMods.Substring(0, missingMods.Length - 2)})."; + mod.SetStatus(ModMetadataStatus.Failed, $"it requires mods which aren't installed ({missingMods.Substring(0, missingMods.Length - 2)})."); return false; } } @@ -207,8 +206,7 @@ namespace StardewModdingAPI.Framework.ModLoading ModMetadata circularReferenceMod = currentChain.FirstOrDefault(modsToLoadFirst.Contains); if (circularReferenceMod != null) { - mod.Status = ModMetadataStatus.Failed; - mod.Error = $"its dependencies have a circular reference: {string.Join(" => ", currentChain.Select(p => p.DisplayName))} => {circularReferenceMod.DisplayName})."; + mod.SetStatus(ModMetadataStatus.Failed, $"its dependencies have a circular reference: {string.Join(" => ", currentChain.Select(p => p.DisplayName))} => {circularReferenceMod.DisplayName})."); return false; } currentChain.Add(mod); |