Fixed issue with plugins being deleted.

pull/4513/head
Greenback 4 years ago
parent 1df58fbaa0
commit 31980e6f87

@ -999,7 +999,7 @@ namespace Emby.Server.Implementations
var versions = new List<LocalPlugin>(); var versions = new List<LocalPlugin>();
if (!Directory.Exists(path)) if (!Directory.Exists(path))
{ {
// Plugin path doesn't exist, don't try to enumerate subfolders. // Plugin path doesn't exist, don't try to enumerate sub-folders.
return Enumerable.Empty<LocalPlugin>(); return Enumerable.Empty<LocalPlugin>();
} }
@ -1070,7 +1070,6 @@ namespace Emby.Server.Implementations
if (!string.IsNullOrEmpty(lastName) && cleanup) if (!string.IsNullOrEmpty(lastName) && cleanup)
{ {
// Attempt a cleanup of old folders. // Attempt a cleanup of old folders.
versions.RemoveAt(x);
try try
{ {
Logger.LogDebug("Deleting {Path}", versions[x].Path); Logger.LogDebug("Deleting {Path}", versions[x].Path);
@ -1080,6 +1079,8 @@ namespace Emby.Server.Implementations
{ {
Logger.LogWarning(e, "Unable to delete {Path}", versions[x].Path); Logger.LogWarning(e, "Unable to delete {Path}", versions[x].Path);
} }
versions.RemoveAt(x);
} }
} }

@ -56,10 +56,5 @@ namespace Emby.Server.Implementations.Plugins
/// Gets or sets the Version number of the plugin. /// Gets or sets the Version number of the plugin.
/// </summary> /// </summary>
public string Version { get; set; } public string Version { get; set; }
/// <summary>
/// Gets or sets the Repository where the plugin originated.
/// </summary>
public string Repository { get; set; }
} }
} }

Loading…
Cancel
Save