From d0baaf4fe4f3b237998c162f1a8493d54210be1a Mon Sep 17 00:00:00 2001
From: Marco Minerva <marco.minerva@gmail.com>
Date: Sat, 6 May 2017 19:16:31 +0200
Subject: [PATCH] Add method to get translation language names

---
 .../MicrosoftTranslatorPage.xaml              |   4 +-
 .../MicrosoftTranslatorPage.xaml.cs           |  22 +--
 .../Microsoft.Toolkit.Uwp.Services.csproj     |   1 +
 .../MicrosoftTranslator/ITranslatorService.cs |  30 +++-
 .../MicrosoftTranslator/ServiceLanguage.cs    |  47 ++++++
 .../MicrosoftTranslator/TranslatorService.cs  | 157 ++++++++++++------
 .../TranslatorServiceException.cs             |   2 +-
 7 files changed, 182 insertions(+), 81 deletions(-)
 create mode 100644 Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ServiceLanguage.cs

diff --git a/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml b/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml
index e8a796edea0..a383dd8ad6d 100644
--- a/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml	
+++ b/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml	
@@ -31,9 +31,9 @@
             <ComboBox x:Name="Languages"
                       Margin="0,10,0,0"
                       HorizontalAlignment="Stretch"
-                      DisplayMemberPath="Value"
+                      DisplayMemberPath="Name"
                       Header="Target Language:"
-                      SelectedValuePath="Key" />
+                      SelectedValuePath="Code" />
             <Button x:Name="Translate"
                     Margin="0,10,0,0"
                     Click="Translate_OnClick"
diff --git a/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml.cs b/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml.cs
index 81b02bee54c..4a3525cc3d9 100644
--- a/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml.cs	
+++ b/Microsoft.Toolkit.Uwp.SampleApp/SamplePages/Microsoft Translator Service/MicrosoftTranslatorPage.xaml.cs	
@@ -59,26 +59,10 @@ private async void GetLanguages_OnClick(object sender, RoutedEventArgs e)
             Languages.ItemsSource = null;
 
             _translatorClient.SubscriptionKey = TranslatorServiceKey.Text;
-            var languages = await _translatorClient.GetLanguagesAsync();
+            var languages = await _translatorClient.GetLanguageNamesAsync();
 
-            var dictionary = new Dictionary<string, string>();
-            languages.ToList().ForEach(lang =>
-            {
-                try
-                {
-                    var culture = new CultureInfo(lang);
-                    if (!culture.EnglishName.StartsWith("Unknown"))
-                    {
-                        dictionary.Add(lang, culture.EnglishName);
-                    }
-                }
-                catch
-                {
-                }
-            });
-
-            Languages.ItemsSource = dictionary.OrderBy(d => d.Key);
-            Languages.SelectedValue = dictionary.Keys.FirstOrDefault();
+            Languages.ItemsSource = languages.OrderBy(d => d.Name);
+            Languages.SelectedIndex = 0;
 
             Shell.Current.DisplayWaitRing = false;
         }
diff --git a/Microsoft.Toolkit.Uwp.Services/Microsoft.Toolkit.Uwp.Services.csproj b/Microsoft.Toolkit.Uwp.Services/Microsoft.Toolkit.Uwp.Services.csproj
index 83cfa3cf953..d112a31b327 100644
--- a/Microsoft.Toolkit.Uwp.Services/Microsoft.Toolkit.Uwp.Services.csproj
+++ b/Microsoft.Toolkit.Uwp.Services/Microsoft.Toolkit.Uwp.Services.csproj
@@ -51,6 +51,7 @@
     <Compile Include="Core\DataProviderBase{TConfig,TSchema}.cs" />
     <Compile Include="Core\ExtensionMethods.cs" />
     <Compile Include="Core\IDataService{T,U,V}.cs" />
+    <Compile Include="Services\MicrosoftTranslator\ServiceLanguage.cs" />
     <Compile Include="Services\Twitter\TwitterStreamCallbacks.cs" />
     <Compile Include="Services\Bing\BingLanguage.cs" />
     <Compile Include="Services\Bing\BingCountry.cs" />
diff --git a/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ITranslatorService.cs b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ITranslatorService.cs
index 28267b67333..4f1e6d2e8f7 100644
--- a/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ITranslatorService.cs
+++ b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ITranslatorService.cs
@@ -37,6 +37,15 @@ public interface ITranslatorService
         /// <seealso cref="GetLanguagesAsync"/>
         string Language { get; set; }
 
+        /// <summary>
+        /// Initializes the <see cref="TranslatorService"/> class by getting an access token for the service.
+        /// </summary>
+        /// <returns>A <see cref="Task"/> that represents the initialize operation.</returns>
+        /// <exception cref="ArgumentNullException">The <see cref="SubscriptionKey"/> property hasn't been set.</exception>
+        /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
+        /// <remarks>Calling this method isn't mandatory, because the token is get/refreshed everytime is needed. However, it is called at startup, it can speed-up subsequest requests.</remarks>
+        Task InitializeAsync();
+
         /// <summary>
         /// Detects the language of a text.
         /// </summary>
@@ -50,7 +59,7 @@ public interface ITranslatorService
         /// </exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
         /// <remarks><para>This method performs a non-blocking request for language detection.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512427.aspx.
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_Detect.
         /// </para></remarks>
         /// <seealso cref="GetLanguagesAsync"/>
         /// <seealso cref="Language"/>
@@ -63,19 +72,24 @@ public interface ITranslatorService
         /// <exception cref="ArgumentNullException">The <see cref="SubscriptionKey"/> property hasn't been set.</exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
         /// <remarks><para>This method performs a non-blocking request for language codes.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512415.aspx.
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_GetLanguagesForTranslate.
         /// </para>
         /// </remarks>
         Task<IEnumerable<string>> GetLanguagesAsync();
 
         /// <summary>
-        /// Initializes the <see cref="TranslatorService"/> class by getting an access token for the service.
+        /// Retrieves friendly names for the languages available for text translation.
         /// </summary>
-        /// <returns>A <see cref="Task"/> that represents the initialize operation.</returns>
+        /// <param name="language">The language used to localize the language names. If the parameter is set to <strong>null</strong>, the language specified in the <seealso cref="Language"/> property will be used.</param>
+        /// <returns>An array of <see cref="ServiceLanguage"/> containing the language codes and names supported for translation by <strong>Microsoft Translator Service</strong>.</returns>
         /// <exception cref="ArgumentNullException">The <see cref="SubscriptionKey"/> property hasn't been set.</exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
-        /// <remarks>Calling this method isn't mandatory, because the token is get/refreshed everytime is needed. However, it is called at startup, it can speed-up subsequest requests.</remarks>
-        Task InitializeAsync();
+        /// <remarks><para>This method performs a non-blocking request for language name.</para>
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/post_GetLanguageNames.
+        /// </para>
+        /// </remarks>
+        /// <see cref="GetLanguagesAsync"/>
+        Task<IEnumerable<ServiceLanguage>> GetLanguageNamesAsync(string language = null);
 
         /// <summary>
         /// Translates a text string into the specified language.
@@ -93,7 +107,7 @@ public interface ITranslatorService
         /// <exception cref="ArgumentException">The <paramref name="text"/> parameter is longer than 1000 characters.</exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
         /// <remarks><para>This method perform a non-blocking request for text translation.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512421.aspx.
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_Translate.
         /// </para>
         /// </remarks>
         /// <seealso cref="Language"/>
@@ -114,7 +128,7 @@ public interface ITranslatorService
         /// <exception cref="ArgumentException">The <paramref name="text"/> parameter is longer than 1000 characters.</exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
         /// <remarks><para>This method perform a non-blocking request for text translation.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512421.aspx.
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_Translate.
         /// </para>
         /// </remarks>
         /// <seealso cref="Language"/>
diff --git a/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ServiceLanguage.cs b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ServiceLanguage.cs
new file mode 100644
index 00000000000..dcbc331e14c
--- /dev/null
+++ b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/ServiceLanguage.cs
@@ -0,0 +1,47 @@
+// ******************************************************************
+// Copyright (c) Microsoft. All rights reserved.
+// This code is licensed under the MIT License (MIT).
+// THE CODE IS PROVIDED “AS IS”, WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
+// INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+// IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
+// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+// TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH
+// THE CODE OR THE USE OR OTHER DEALINGS IN THE CODE.
+// ******************************************************************
+
+namespace Microsoft.Toolkit.Uwp.Services.MicrosoftTranslator
+{
+    /// <summary>
+    /// Holds information about langagues supported for text translation and speech synthesis.
+    /// </summary>
+    public class ServiceLanguage
+    {
+        /// <summary>
+        /// Gets the language code.
+        /// </summary>
+        public string Code { get; }
+
+        /// <summary>
+        /// Gets the language friendly name.
+        /// </summary>
+        public string Name { get; }
+
+        /// <summary>
+        /// Returns the language friendly name.
+        /// </summary>
+        /// <returns>The language friendly name.</returns>
+        public override string ToString() => Name;
+
+        /// <summary>
+        /// Initializes a new instance of the <see cref="ServiceLanguage"/> class, using the specified code and name.
+        /// </summary>
+        /// <param name="code">The language code.</param>
+        /// <param name="name">The language friendly name.</param>
+        public ServiceLanguage(string code, string name)
+        {
+            Code = code;
+            Name = name;
+        }
+    }
+}
diff --git a/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorService.cs b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorService.cs
index d68ae1db3b8..cbab5c23490 100644
--- a/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorService.cs
+++ b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorService.cs
@@ -16,6 +16,8 @@
 using System.Linq;
 using System.Threading.Tasks;
 using System.Xml.Linq;
+using Windows.Web.Http;
+using Windows.Web.Http.Headers;
 
 namespace Microsoft.Toolkit.Uwp.Services.MicrosoftTranslator
 {
@@ -33,11 +35,15 @@ public class TranslatorService : ITranslatorService
         private const string TranslateUri = "Translate?text={0}&to={1}&contentType=text/plain";
         private const string TranslateWithFromUri = "Translate?text={0}&from={1}&to={2}&contentType=text/plain";
         private const string DetectUri = "Detect?text={0}";
+        private const string LanguageNamesUri = "GetLanguageNames?locale={0}";
         private const string AuthorizationUri = "Authorization";
 
         private const string ArrayNamespace = "http://schemas.microsoft.com/2003/10/Serialization/Arrays";
+        private const string ArrayOfStringXmlElement = "ArrayOfstring";
+        private const string StringXmlElement = "string";
+        private const string XmlContentType = "text/xml";
 
-        private const int _maxTextLenght = 1000;
+        private const int _maxTextLength = 1000;
         private const int _MaxTextLengthForAutoDetection = 100;
 
         /// <summary>
@@ -86,6 +92,48 @@ public string SubscriptionKey
         /// <seealso cref="GetLanguagesAsync"/>
         public string Language { get; set; }
 
+        /// <summary>
+        /// Detects the language of a text.
+        /// </summary>
+        /// <param name="text">A string represeting the text whose language must be detected.</param>
+        /// <returns>A string containing a two-character Language code for the given text.</returns>
+        /// <exception cref="ArgumentNullException">
+        /// <list type="bullet">
+        /// <term>The <see cref="SubscriptionKey"/> property hasn't been set.</term>
+        /// <term>The <paramref name="text"/> parameter is <strong>null</strong> (<strong>Nothing</strong> in Visual Basic) or empty.</term>
+        /// </list>
+        /// </exception>
+        /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
+        /// <remarks><para>This method performs a non-blocking request for language detection.</para>
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_Detect.
+        /// </para></remarks>
+        /// <seealso cref="GetLanguagesAsync"/>
+        public async Task<string> DetectLanguageAsync(string text)
+        {
+            if (string.IsNullOrWhiteSpace(text))
+            {
+                throw new ArgumentNullException(nameof(text));
+            }
+
+            text = text.Substring(0, Math.Min(text.Length, _MaxTextLengthForAutoDetection));
+
+            // Checks if it is necessary to obtain/update access token.
+            await CheckUpdateTokenAsync().ConfigureAwait(false);
+
+            var uriString = string.Format(DetectUri, Uri.EscapeDataString(text));
+
+            using (var request = CreateHttpRequest($"{BaseUrl}{uriString}"))
+            {
+                var response = await HttpHelper.Instance.SendRequestAsync(request).ConfigureAwait(false);
+                var content = await response.GetTextResultAsync().ConfigureAwait(false);
+
+                var doc = XDocument.Parse(content);
+                var detectedLanguage = doc.Root.Value;
+
+                return detectedLanguage;
+            }
+        }
+
         /// <summary>
         /// Retrieves the languages available for translation.
         /// </summary>
@@ -93,7 +141,7 @@ public string SubscriptionKey
         /// <exception cref="ArgumentNullException">The <see cref="SubscriptionKey"/> property hasn't been set.</exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
         /// <remarks><para>This method performs a non-blocking request for language codes.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512415.aspx.
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_GetLanguagesForTranslate.
         /// </para>
         /// </remarks>
         public async Task<IEnumerable<string>> GetLanguagesAsync()
@@ -107,9 +155,55 @@ public async Task<IEnumerable<string>> GetLanguagesAsync()
                 var content = await response.GetTextResultAsync().ConfigureAwait(false);
 
                 XNamespace ns = ArrayNamespace;
-                var doc = XDocument.Parse(content);
+                var xmlContent = XDocument.Parse(content);
+
+                var languages = xmlContent.Root.Elements(ns + "string").Select(s => s.Value);
+                return languages;
+            }
+        }
+
+        /// <summary>
+        /// Retrieves friendly names for the languages available for text translation.
+        /// </summary>
+        /// <param name="language">The language used to localize the language names. If the parameter is set to <strong>null</strong>, the language specified in the <seealso cref="Language"/> property will be used.</param>
+        /// <returns>An array of <see cref="ServiceLanguage"/> containing the language codes and names supported for translation by <strong>Microsoft Translator Service</strong>.</returns>
+        /// <exception cref="ArgumentNullException">The <see cref="SubscriptionKey"/> property hasn't been set.</exception>
+        /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
+        /// <remarks><para>This method performs a non-blocking request for language name.</para>
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/post_GetLanguageNames.
+        /// </para>
+        /// </remarks>
+        /// <see cref="GetLanguagesAsync"/>
+        public async Task<IEnumerable<ServiceLanguage>> GetLanguageNamesAsync(string language = null)
+        {
+            var languageCodes = await GetLanguagesAsync();
+
+            if (string.IsNullOrWhiteSpace(language))
+            {
+                language = Language ?? "en";
+            }
+
+            using (var request = CreateHttpRequest(string.Format($"{BaseUrl}{LanguageNamesUri}", language), HttpMethod.Post))
+            {
+                XNamespace ns = ArrayNamespace;
+                var xmlRequest = new XDocument(new XElement(ns + ArrayOfStringXmlElement, from lang in languageCodes select new XElement(ns + StringXmlElement, lang)));
+
+                request.Content = new HttpStringContent(xmlRequest.ToString());
+                request.Content.Headers.ContentType = new HttpMediaTypeHeaderValue(XmlContentType);
+
+                var response = await HttpHelper.Instance.SendRequestAsync(request).ConfigureAwait(false);
+                var content = await response.GetTextResultAsync().ConfigureAwait(false);
+                var xmlContent = XDocument.Parse(content);
+
+                var languageNames = xmlContent.Root.Elements(ns + StringXmlElement).Select(s => s.Value);
+
+                // Creates the response object.
+                var languages = new ServiceLanguage[languageCodes.Count()];
+                for (int i = 0; i < languages.Length; i++)
+                {
+                    languages[i] = new ServiceLanguage(languageCodes.ElementAt(i), languageNames.ElementAt(i));
+                }
 
-                var languages = doc.Root.Elements(ns + "string").Select(s => s.Value);
                 return languages;
             }
         }
@@ -130,7 +224,7 @@ public async Task<IEnumerable<string>> GetLanguagesAsync()
         /// <exception cref="ArgumentException">The <paramref name="text"/> parameter is longer than 1000 characters.</exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
         /// <remarks><para>This method perform a non-blocking request for text translation.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512421.aspx.
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_Translate.
         /// </para>
         /// </remarks>
         /// <seealso cref="Language"/>
@@ -141,9 +235,9 @@ public async Task<string> TranslateAsync(string text, string from, string to)
                 throw new ArgumentNullException(nameof(text));
             }
 
-            if (text.Length > _maxTextLenght)
+            if (text.Length > _maxTextLength)
             {
-                throw new ArgumentException($"{nameof(text)} parameter cannot be longer than {_maxTextLenght} characters");
+                throw new ArgumentException($"{nameof(text)} parameter cannot be longer than {_maxTextLength} characters");
             }
 
             // Checks if it is necessary to obtain/update access token.
@@ -191,54 +285,12 @@ public async Task<string> TranslateAsync(string text, string from, string to)
         /// <exception cref="ArgumentException">The <paramref name="text"/> parameter is longer than 1000 characters.</exception>
         /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
         /// <remarks><para>This method perform a non-blocking request for text translation.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512421.aspx.
+        /// <para>For more information, go to https://docs.microsofttranslator.com/text-translate.html#!/default/get_Translate.
         /// </para>
         /// </remarks>
         /// <seealso cref="Language"/>
         public Task<string> TranslateAsync(string text, string to = null) => TranslateAsync(text, null, to);
 
-        /// <summary>
-        /// Detects the language of a text.
-        /// </summary>
-        /// <param name="text">A string represeting the text whose language must be detected.</param>
-        /// <returns>A string containing a two-character Language code for the given text.</returns>
-        /// <exception cref="ArgumentNullException">
-        /// <list type="bullet">
-        /// <term>The <see cref="SubscriptionKey"/> property hasn't been set.</term>
-        /// <term>The <paramref name="text"/> parameter is <strong>null</strong> (<strong>Nothing</strong> in Visual Basic) or empty.</term>
-        /// </list>
-        /// </exception>
-        /// <exception cref="TranslatorServiceException">The provided <see cref="SubscriptionKey"/> isn't valid or has expired.</exception>
-        /// <remarks><para>This method performs a non-blocking request for language detection.</para>
-        /// <para>For more information, go to http://msdn.microsoft.com/en-us/library/ff512427.aspx.
-        /// </para></remarks>
-        /// <seealso cref="GetLanguagesAsync"/>
-        public async Task<string> DetectLanguageAsync(string text)
-        {
-            if (string.IsNullOrWhiteSpace(text))
-            {
-                throw new ArgumentNullException(nameof(text));
-            }
-
-            text = text.Substring(0, Math.Min(text.Length, _MaxTextLengthForAutoDetection));
-
-            // Checks if it is necessary to obtain/update access token.
-            await CheckUpdateTokenAsync().ConfigureAwait(false);
-
-            var uriString = string.Format(DetectUri, Uri.EscapeDataString(text));
-
-            using (var request = CreateHttpRequest($"{BaseUrl}{uriString}"))
-            {
-                var response = await HttpHelper.Instance.SendRequestAsync(request).ConfigureAwait(false);
-                var content = await response.GetTextResultAsync().ConfigureAwait(false);
-
-                var doc = XDocument.Parse(content);
-                var detectedLanguage = doc.Root.Value;
-
-                return detectedLanguage;
-            }
-        }
-
         /// <summary>
         /// Initializes the <see cref="TranslatorService"/> class by getting an access token for the service.
         /// </summary>
@@ -292,8 +344,11 @@ private async Task CheckUpdateTokenAsync()
         }
 
         private HttpHelperRequest CreateHttpRequest(string uriString)
+            => CreateHttpRequest(uriString, HttpMethod.Get);
+
+        private HttpHelperRequest CreateHttpRequest(string uriString, HttpMethod method)
         {
-            var request = new HttpHelperRequest(new Uri(uriString));
+            var request = new HttpHelperRequest(new Uri(uriString), method);
             request.Headers.Add(AuthorizationUri, _authorizationHeaderValue);
 
             return request;
diff --git a/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorServiceException.cs b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorServiceException.cs
index 3ea03b1fbd0..da0a78e6451 100644
--- a/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorServiceException.cs
+++ b/Microsoft.Toolkit.Uwp.Services/Services/MicrosoftTranslator/TranslatorServiceException.cs
@@ -22,8 +22,8 @@ public class TranslatorServiceException : Exception
     {
         /// <summary>
         /// Initializes a new instance of the <see cref="TranslatorServiceException"/> class using the specified error message.
-        /// <param name="message">Message that describes the error</param>
         /// </summary>
+        /// <param name="message">Message that describes the error</param>
         public TranslatorServiceException(string message)
             : base(message)
         {