Made a couple optional parameters explicitly required for mod toggles to avoid potential accidental behaviors in the future..

Lunaretic · Lunaretic · commit b91e186ae992 · 2020-09-09T18:10:27.000-04:00
diff --git a/xivModdingFramework/Mods/Modding.cs b/xivModdingFramework/Mods/Modding.cs
@@ -338,7 +338,7 @@ public async Task ToggleModPackStatus(string modPackName, bool enable)
         /// <param name="enable"></param>
         /// <param name="mod"></param>
         /// <returns></returns>
-        public async Task<bool> ToggleModUnsafe(bool enable, Mod mod, bool includeInternal = false, bool updateCache = false)
+        public async Task<bool> ToggleModUnsafe(bool enable, Mod mod, bool includeInternal, bool updateCache)
         {
             if (mod == null) return false;
             if (string.IsNullOrEmpty(mod.name)) return false;

Original file line number	Diff line number	Diff line change
`@@ -338,7 +338,7 @@ public async Task ToggleModPackStatus(string modPackName, bool enable)`
`338`	`338`	`/// <param name="enable"></param>`
`339`	`339`	`/// <param name="mod"></param>`
`340`	`340`	`/// <returns></returns>`
`341`		`- public async Task<bool> ToggleModUnsafe(bool enable, Mod mod, bool includeInternal = false, bool updateCache = false)`
	`341`	`+ public async Task<bool> ToggleModUnsafe(bool enable, Mod mod, bool includeInternal, bool updateCache)`
`342`	`342`	`{`
`343`	`343`	`if (mod == null) return false;`
`344`	`344`	`if (string.IsNullOrEmpty(mod.name)) return false;`