Added a public constructor to ClaimsResponse.

Also made the probable type URIs that it accepts public and easily discoverable.

Fixes #116

4 files changed, 38 insertions, 14 deletions

diff --git a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/ExtensionArgumentsManager.cs b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/ExtensionArgumentsManager.cs
index 5cd4904..1d795da 100644
--- a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/ExtensionArgumentsManager.cs
+++ b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/ExtensionArgumentsManager.cs
@@ -21,7 +21,7 @@ namespace DotNetOpenAuth.OpenId.Extensions {
 		/// match to namespaces for backward compatibility with other OpenID libraries.
 		/// </summary>
 		private static readonly Dictionary<string, string> typeUriToAliasAffinity = new Dictionary<string, string> {
-			{ Extensions.SimpleRegistration.Constants.sreg_ns, Extensions.SimpleRegistration.Constants.sreg_compatibility_alias },
+			{ Extensions.SimpleRegistration.Constants.TypeUris.Standard, Extensions.SimpleRegistration.Constants.sreg_compatibility_alias },
 			{ Extensions.ProviderAuthenticationPolicy.Constants.TypeUri, Extensions.ProviderAuthenticationPolicy.Constants.CompatibilityAlias },
 		};
 
diff --git a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsRequest.cs b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsRequest.cs
index 9d418ee..ab08cbb 100644
--- a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsRequest.cs
+++ b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsRequest.cs
@@ -24,7 +24,7 @@ namespace DotNetOpenAuth.OpenId.Extensions.SimpleRegistration {
 		/// The factory method that may be used in deserialization of this message.
 		/// </summary>
 		internal static readonly StandardOpenIdExtensionFactory.CreateDelegate Factory = (typeUri, data, baseMessage, isProviderRole) => {
-			if (typeUri == Constants.sreg_ns && isProviderRole) {
+			if (typeUri == Constants.TypeUris.Standard && isProviderRole) {
 				return new ClaimsRequest(typeUri);
 			}
 
@@ -41,7 +41,7 @@ namespace DotNetOpenAuth.OpenId.Extensions.SimpleRegistration {
 		/// Initializes a new instance of the <see cref="ClaimsRequest"/> class.
 		/// </summary>
 		public ClaimsRequest()
-			: base(new Version(1, 0), Constants.sreg_ns, Constants.AdditionalTypeUris) {
+			: base(new Version(1, 0), Constants.TypeUris.Standard, Constants.AdditionalTypeUris) {
 		}
 
 		/// <summary>
diff --git a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsResponse.cs b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsResponse.cs
index 999fe8d..af60596 100644
--- a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsResponse.cs
+++ b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/ClaimsResponse.cs
@@ -27,7 +27,7 @@ namespace DotNetOpenAuth.OpenId.Extensions.SimpleRegistration {
 		/// The factory method that may be used in deserialization of this message.
 		/// </summary>
 		internal static readonly StandardOpenIdExtensionFactory.CreateDelegate Factory = (typeUri, data, baseMessage, isProviderRole) => {
-			if ((typeUri == Constants.sreg_ns || Array.IndexOf(Constants.AdditionalTypeUris, typeUri) >= 0) && !isProviderRole) {
+			if ((typeUri == Constants.TypeUris.Standard || Array.IndexOf(Constants.AdditionalTypeUris, typeUri) >= 0) && !isProviderRole) {
 				return new ClaimsResponse(typeUri);
 			}
 
@@ -55,10 +55,11 @@ namespace DotNetOpenAuth.OpenId.Extensions.SimpleRegistration {
 		private CultureInfo culture;
 
 		/// <summary>
-		/// Initializes a new instance of the <see cref="ClaimsResponse"/> class.
+		/// Initializes a new instance of the <see cref="ClaimsResponse"/> class
+		/// using the most common, and spec prescribed type URI.
 		/// </summary>
-		internal ClaimsResponse()
-			: this(Constants.sreg_ns) {
+		public ClaimsResponse()
+			: this(Constants.TypeUris.Standard) {
 		}
 
 		/// <summary>
@@ -67,8 +68,10 @@ namespace DotNetOpenAuth.OpenId.Extensions.SimpleRegistration {
 		/// <param name="typeUriToUse">
 		/// The type URI that must be used to identify this extension in the response message.
 		/// This value should be the same one the relying party used to send the extension request.
+		/// Commonly used type URIs supported by relying parties are defined in the
+		/// <see cref="Constants.TypeUris"/> class.
 		/// </param>
-		internal ClaimsResponse(string typeUriToUse)
+		public ClaimsResponse(string typeUriToUse = Constants.TypeUris.Standard)
 			: base(new Version(1, 0), typeUriToUse, Constants.AdditionalTypeUris) {
 			Requires.NotNullOrEmpty(typeUriToUse, "typeUriToUse");
 		}
diff --git a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/Constants.cs b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/Constants.cs
index 8325b0c..30cd748 100644
--- a/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/Constants.cs
+++ b/src/DotNetOpenAuth.OpenId/OpenId/Extensions/SimpleRegistration/Constants.cs
@@ -13,10 +13,31 @@ namespace DotNetOpenAuth.OpenId.Extensions.SimpleRegistration {
 	/// <summary>
 	/// Simple Registration constants
 	/// </summary>
-	internal static class Constants {
-		internal const string sreg_ns = "http://openid.net/extensions/sreg/1.1";
-		internal const string sreg_ns10 = "http://openid.net/sreg/1.0";
-		internal const string sreg_ns11other = "http://openid.net/sreg/1.1";
+	public static class Constants {
+		/// <summary>
+		/// Commonly used type URIs to represent the Simple Registration extension.
+		/// </summary>
+		public static class TypeUris {
+			/// <summary>
+			/// The URI "http://openid.net/extensions/sreg/1.1".  
+			/// </summary>
+			/// <remarks>
+			/// This is the type URI prescribed by the Simple Registration 1.1 spec. 
+			/// http://openid.net/specs/openid-simple-registration-extension-1_1-01.html#anchor3
+			/// </remarks>
+			public const string Standard = "http://openid.net/extensions/sreg/1.1";
+
+			/// <summary>
+			/// The URI "http://openid.net/sreg/1.0"
+			/// </summary>
+			public const string Variant10 = "http://openid.net/sreg/1.0";
+
+			/// <summary>
+			/// The URI "http://openid.net/sreg/1.1"
+			/// </summary>
+			public const string Variant11 = "http://openid.net/sreg/1.1";
+		}
+
 		internal const string sreg_compatibility_alias = "sreg";
 		internal const string policy_url = "policy_url";
 		internal const string optional = "optional";
@@ -39,8 +60,8 @@ namespace DotNetOpenAuth.OpenId.Extensions.SimpleRegistration {
 		/// Additional type URIs that this extension is sometimes known by remote parties.
 		/// </summary>
 		internal static readonly string[] AdditionalTypeUris = new string[] {
-			Constants.sreg_ns10,
-			Constants.sreg_ns11other,
+			Constants.TypeUris.Variant10,
+			Constants.TypeUris.Variant11,
 		};
 	}
 }