Use correct id when finding existing dlna profile

This commit is contained in:
Cody Robibero 2021-10-31 08:49:24 -06:00
parent 5a7433472e
commit 2f6437a987
3 changed files with 5 additions and 4 deletions

View File

@ -416,7 +416,7 @@ namespace Emby.Dlna
} }
/// <inheritdoc /> /// <inheritdoc />
public void UpdateProfile(DeviceProfile profile) public void UpdateProfile(string profileId, DeviceProfile profile)
{ {
profile = ReserializeProfile(profile); profile = ReserializeProfile(profile);
@ -430,7 +430,7 @@ namespace Emby.Dlna
throw new ArgumentException("Profile is missing Name"); throw new ArgumentException("Profile is missing Name");
} }
var current = GetProfileInfosInternal().First(i => string.Equals(i.Info.Id, profile.Id, StringComparison.OrdinalIgnoreCase)); var current = GetProfileInfosInternal().First(i => string.Equals(i.Info.Id, profileId, StringComparison.OrdinalIgnoreCase));
var newFilename = _fileSystem.GetValidFilename(profile.Name) + ".xml"; var newFilename = _fileSystem.GetValidFilename(profile.Name) + ".xml";
var path = Path.Combine(UserProfilesPath, newFilename); var path = Path.Combine(UserProfilesPath, newFilename);

View File

@ -126,7 +126,7 @@ namespace Jellyfin.Api.Controllers
return NotFound(); return NotFound();
} }
_dlnaManager.UpdateProfile(deviceProfile); _dlnaManager.UpdateProfile(profileId, deviceProfile);
return NoContent(); return NoContent();
} }
} }

View File

@ -37,8 +37,9 @@ namespace MediaBrowser.Controller.Dlna
/// <summary> /// <summary>
/// Updates the profile. /// Updates the profile.
/// </summary> /// </summary>
/// <param name="profileId">The profile id.</param>
/// <param name="profile">The profile.</param> /// <param name="profile">The profile.</param>
void UpdateProfile(DeviceProfile profile); void UpdateProfile(string profileId, DeviceProfile profile);
/// <summary> /// <summary>
/// Deletes the profile. /// Deletes the profile.