Skip to content

Commit eb46c3a

Browse files
committed
[#246] [r] AcceptBrowserLanguage to AcceptHeaderLanguage
1 parent 2bfdb01 commit eb46c3a

File tree

4 files changed

+18
-18
lines changed

4 files changed

+18
-18
lines changed

src/Simplify.Web.Tests/Modules/LanguageManagerTests.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -84,11 +84,11 @@ public void SetCookieLanguage_CorrectLanguage_CorrectCookieCreated()
8484
}
8585

8686
[Test]
87-
public void Constructor_HaveBrowserLanguageAndSettingIsEnabledCase1_LanguageSetFromHeader()
87+
public void Constructor_HaveHeaderLanguageAndSettingIsEnabledCase1_LanguageSetFromHeader()
8888
{
8989
// Assign
9090

91-
_settings.SetupGet(x => x.AcceptBrowserLanguage).Returns(true);
91+
_settings.SetupGet(x => x.AcceptHeaderLanguage).Returns(true);
9292
var header = new HeaderDictionary(new Dictionary<string, StringValues>());
9393
header.Append("Accept-Language", "ru-RU");
9494
_context.SetupGet(x => x.Request.Headers).Returns(header);
@@ -101,11 +101,11 @@ public void Constructor_HaveBrowserLanguageAndSettingIsEnabledCase1_LanguageSetF
101101
}
102102

103103
[Test]
104-
public void Constructor_HaveBrowserLanguageAndSettingIsEnabledCase2_LanguageSetFromHeader()
104+
public void Constructor_HaveHeaderLanguageAndSettingIsEnabledCase2_LanguageSetFromHeader()
105105
{
106106
// Assign
107107

108-
_settings.SetupGet(x => x.AcceptBrowserLanguage).Returns(true);
108+
_settings.SetupGet(x => x.AcceptHeaderLanguage).Returns(true);
109109
var header = new HeaderDictionary(new Dictionary<string, StringValues>());
110110
header.Append("Accept-Language", "ru-RU;q=0.5");
111111
_context.SetupGet(x => x.Request.Headers).Returns(header);
@@ -118,15 +118,15 @@ public void Constructor_HaveBrowserLanguageAndSettingIsEnabledCase2_LanguageSetF
118118
}
119119

120120
[Test]
121-
public void Constructor_HaveBrowserLanguageAndCookieLanguage_LanguageSetFromCookie()
121+
public void Constructor_HaveHeaderLanguageAndCookieLanguage_LanguageSetFromCookie()
122122
{
123123
// Assign
124124

125125
var cookieCollection = new Mock<IRequestCookieCollection>();
126126
cookieCollection.SetupGet(x => x[It.Is<string>(s => s == LanguageManager.CookieLanguageFieldName)]).Returns("fr");
127127
_context.SetupGet(x => x.Request.Cookies).Returns(cookieCollection.Object);
128128

129-
_settings.SetupGet(x => x.AcceptBrowserLanguage).Returns(true);
129+
_settings.SetupGet(x => x.AcceptHeaderLanguage).Returns(true);
130130
var header = new HeaderDictionary(new Dictionary<string, StringValues>());
131131
header.Append("Accept-Language", "ru-RU");
132132
_context.SetupGet(x => x.Request.Headers).Returns(header);
@@ -143,7 +143,7 @@ public void Constructor_NoBrowserLanguage_DefaultLanguageSet()
143143
{
144144
// Assign
145145

146-
_settings.SetupGet(x => x.AcceptBrowserLanguage).Returns(true);
146+
_settings.SetupGet(x => x.AcceptHeaderLanguage).Returns(true);
147147
var header = new HeaderDictionary(new Dictionary<string, StringValues>());
148148
_context.SetupGet(x => x.Request.Headers).Returns(header);
149149

src/Simplify.Web/Modules/LanguageManager.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ public LanguageManager(ISimplifyWebSettings settings, HttpContext context)
3030
if (TrySetLanguageFromCookie(context))
3131
return;
3232

33-
if (!settings.AcceptBrowserLanguage || (settings.AcceptBrowserLanguage && !TrySetLanguageFromRequestHeader(context)))
33+
if (!settings.AcceptHeaderLanguage || (settings.AcceptHeaderLanguage && !TrySetLanguageFromRequestHeader(context)))
3434
if (!SetCurrentLanguage(settings.DefaultLanguage))
3535
Language = "iv";
3636
}

src/Simplify.Web/Settings/ISimplifyWebSettings.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,12 @@ public interface ISimplifyWebSettings
1313
string DefaultLanguage { get; }
1414

1515
/// <summary>
16-
/// Gets a value indicating whether browser language should be accepted
16+
/// Gets a value indicating whether HTTP header language should be accepted
1717
/// </summary>
1818
/// <value>
19-
/// <c>true</c> if browser language should be accepted; otherwise, <c>false</c>.
19+
/// <c>true</c> if HTTP header language should be accepted; otherwise, <c>false</c>.
2020
/// </value>
21-
bool AcceptBrowserLanguage { get; }
21+
bool AcceptHeaderLanguage { get; }
2222

2323
/// <summary>
2424
/// Default templates directory path, for example: Templates, default value is "Templates"

src/Simplify.Web/Settings/SimplifyWebSettings.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,12 @@ public SimplifyWebSettings(IConfiguration configuration)
3838
public string DefaultLanguage { get; private set; } = "en";
3939

4040
/// <summary>
41-
/// Gets a value indicating whether browser language should be accepted
41+
/// Gets a value indicating whether HTTP header language should be accepted
4242
/// </summary>
4343
/// <value>
44-
/// <c>true</c> if browser language should be accepted; otherwise, <c>false</c>.
44+
/// <c>true</c> if HTTP header language should be accepted; otherwise, <c>false</c>.
4545
/// </value>
46-
public bool AcceptBrowserLanguage { get; private set; }
46+
public bool AcceptHeaderLanguage { get; private set; }
4747

4848
/// <summary>
4949
/// Default templates directory path, for example: Templates, default value is "Templates"
@@ -166,13 +166,13 @@ private void LoadLanguageManagerSettings(IConfiguration config)
166166
if (!string.IsNullOrEmpty(defaultLanguage))
167167
DefaultLanguage = defaultLanguage;
168168

169-
var acceptBrowserLanguage = config[nameof(AcceptBrowserLanguage)];
169+
var acceptHeaderLanguage = config[nameof(AcceptHeaderLanguage)];
170170

171-
if (string.IsNullOrEmpty(acceptBrowserLanguage))
171+
if (string.IsNullOrEmpty(acceptHeaderLanguage))
172172
return;
173173

174-
if (bool.TryParse(acceptBrowserLanguage, out var buffer))
175-
AcceptBrowserLanguage = buffer;
174+
if (bool.TryParse(acceptHeaderLanguage, out var buffer))
175+
AcceptHeaderLanguage = buffer;
176176
}
177177

178178
private void LoadTemplatesSettings(IConfiguration config)

0 commit comments

Comments
 (0)