Fixed StyleCop messages.

3 files changed, 45 insertions, 18 deletions

diff --git a/src/DotNetOpenAuth/InfoCard/InfoCardSelector.cs b/src/DotNetOpenAuth/InfoCard/InfoCardSelector.cs
index 28c109a..b4174bc 100644
--- a/src/DotNetOpenAuth/InfoCard/InfoCardSelector.cs
+++ b/src/DotNetOpenAuth/InfoCard/InfoCardSelector.cs
@@ -646,7 +646,6 @@ namespace DotNetOpenAuth.InfoCard {
 				scriptFormat,
 				MessagingUtilities.GetSafeJavascriptValue(this.ClientID + "_cs"));
 
-
 			if (!string.IsNullOrEmpty(this.Issuer)) {
 				script.AppendLine(CreateParamJs("issuer", this.Issuer));
 			}
diff --git a/src/DotNetOpenAuth/OpenId/Provider/IAuthenticationRequest.cs b/src/DotNetOpenAuth/OpenId/Provider/IAuthenticationRequest.cs
index 20a3cc3..077dcf1 100644
--- a/src/DotNetOpenAuth/OpenId/Provider/IAuthenticationRequest.cs
+++ b/src/DotNetOpenAuth/OpenId/Provider/IAuthenticationRequest.cs
@@ -223,6 +223,24 @@ namespace DotNetOpenAuth.OpenId.Provider {
 			get { throw new NotImplementedException(); }
 		}
 
+		/// <summary>
+		/// Gets or sets the provider endpoint claimed in the positive assertion.
+		/// </summary>
+		/// <value>
+		/// The default value is the URL that the request came in on from the relying party.
+		/// This value MUST match the value for the OP Endpoint in the discovery results for the
+		/// claimed identifier being asserted in a positive response.
+		/// </value>
+		Uri IHostProcessedRequest.ProviderEndpoint {
+			get {
+				throw new NotImplementedException();
+			}
+
+			set {
+				throw new NotImplementedException();
+			}
+		}
+
 		#endregion
 
 		#region IRequest Properties
@@ -300,23 +318,6 @@ namespace DotNetOpenAuth.OpenId.Provider {
 			throw new NotImplementedException();
 		}
 
-		/// <summary>
-		/// Gets or sets the provider endpoint claimed in the positive assertion.
-		/// </summary>
-		/// <value>
-		/// The default value is the URL that the request came in on from the relying party.
-		/// This value MUST match the value for the OP Endpoint in the discovery results for the
-		/// claimed identifier being asserted in a positive response.
-		/// </value>
-		Uri IHostProcessedRequest.ProviderEndpoint {
-			get {
-				throw new NotImplementedException();
-			}
-			set {
-				throw new NotImplementedException();
-			}
-		}
-
 		#endregion
 
 		#region IRequest Methods
diff --git a/src/DotNetOpenAuth/OpenId/RelyingParty/IAuthenticationResponse.cs b/src/DotNetOpenAuth/OpenId/RelyingParty/IAuthenticationResponse.cs
index e33611e..a24220f 100644
--- a/src/DotNetOpenAuth/OpenId/RelyingParty/IAuthenticationResponse.cs
+++ b/src/DotNetOpenAuth/OpenId/RelyingParty/IAuthenticationResponse.cs
@@ -491,12 +491,39 @@ namespace DotNetOpenAuth.OpenId.RelyingParty {
 			throw new NotImplementedException();
 		}
 
+		/// <summary>
+		/// Gets a callback argument's value that was previously added using
+		/// <see cref="IAuthenticationRequest.AddCallbackArguments(string, string)"/>.
+		/// </summary>
+		/// <param name="key">The name of the parameter whose value is sought.</param>
+		/// <returns>
+		/// The value of the argument, or null if the named parameter could not be found.
+		/// </returns>
+		/// <remarks>
+		/// Callback parameters are only available even if the RP is in stateless mode,
+		/// or the callback parameters are otherwise unverifiable as untampered with.
+		/// Therefore, use this method only when the callback argument is not to be
+		/// used to make a security-sensitive decision.
+		/// </remarks>
 		string IAuthenticationResponse.GetUntrustedCallbackArgument(string key) {
 			Contract.Requires<ArgumentException>(!string.IsNullOrEmpty(key));
 			throw new NotImplementedException();
 		}
 
+		/// <summary>
+		/// Gets all the callback arguments that were previously added using
+		/// <see cref="IAuthenticationRequest.AddCallbackArguments(string, string)"/> or as a natural part
+		/// of the return_to URL.
+		/// </summary>
+		/// <returns>A name-value dictionary.  Never null.</returns>
+		/// <remarks>
+		/// Callback parameters are only available even if the RP is in stateless mode,
+		/// or the callback parameters are otherwise unverifiable as untampered with.
+		/// Therefore, use this method only when the callback argument is not to be
+		/// used to make a security-sensitive decision.
+		/// </remarks>
 		IDictionary<string, string> IAuthenticationResponse.GetUntrustedCallbackArguments() {
+			Contract.Ensures(Contract.Result<IDictionary<string, string>>() != null);
 			throw new NotImplementedException();
 		}