michael.heier/jellyfin
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Use value instead of assigning baseUrl first

Browse Source
This commit is contained in:
Joshua Boniface 2019-10-09 10:52:51 -04:00
parent b10e06ff45
commit 345a14ff55
1 changed files with 8 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
MediaBrowser.Model/Configuration/ServerConfiguration.cs
View File

@ -168,31 +168,26 @@ namespace MediaBrowser.Model.Configuration
get => _baseUrl; get => _baseUrl;
set set
{ {
_baseUrl = value;
// Normalize the start of the string // Normalize the start of the string
if (string.IsNullOrWhiteSpace(_baseUrl)) if (string.IsNullOrWhiteSpace(value))
{ {
// If baseUrl is empty, set an empty prefix string // If baseUrl is empty, set an empty prefix string
_baseUrl = string.Empty; value = string.Empty;
} }
else if (!_baseUrl.StartsWith("/")) else if (!value.StartsWith("/"))
{ {
// If baseUrl was not configured with a leading slash, append one for consistency // If baseUrl was not configured with a leading slash, append one for consistency
_baseUrl = "/" + _baseUrl; value = "/" + value;
}
else
{
// If baseUrl was configured with a leading slash, just return it as-is
_baseUrl = _baseUrl;
} }
// Normalize the end of the string // Normalize the end of the string
if (_baseUrl.EndsWith("/")) if (value.EndsWith("/"))
{ {
// If baseUrl was configured with a trailing slash, remove it for consistency // If baseUrl was configured with a trailing slash, remove it for consistency
_baseUrl = _baseUrl.Remove(_baseUrl.Length - 1); value = value.Remove(value.Length - 1);
} }
_baseUrl = value;
} }
} }