Merge pull request #383 from agametov/develop

DavidChristiansen · DavidChristiansen · commit e774db2029a5 · 2016-03-25T11:28:16.000Z
fix sort endpoints
diff --git a/src/DotNetOpenAuth.OpenId.RelyingParty/OpenId/RelyingParty/AuthenticationRequest.cs b/src/DotNetOpenAuth.OpenId.RelyingParty/OpenId/RelyingParty/AuthenticationRequest.cs
@@ -510,7 +510,7 @@ private static List<IdentifierDiscoveryResult> FilterAndSortEndpoints(IEnumerabl
 			}
 
 			// Sort endpoints so that the first one in the list is the most preferred one.
-			filteredEndpoints.OrderBy(ep => ep, relyingParty.EndpointOrder);
+			filteredEndpoints = filteredEndpoints.OrderBy(ep => ep, relyingParty.EndpointOrder).ToList();
 
 			var endpointList = new List<IdentifierDiscoveryResult>(filteredEndpoints.Count);
 			foreach (var endpoint in filteredEndpoints) {

Original file line number	Diff line number	Diff line change
`@@ -510,7 +510,7 @@ private static List<IdentifierDiscoveryResult> FilterAndSortEndpoints(IEnumerabl`
`510`	`510`	`}`
`511`	`511`
`512`	`512`	`// Sort endpoints so that the first one in the list is the most preferred one.`
`513`		`- filteredEndpoints.OrderBy(ep => ep, relyingParty.EndpointOrder);`
	`513`	`+ filteredEndpoints = filteredEndpoints.OrderBy(ep => ep, relyingParty.EndpointOrder).ToList();`
`514`	`514`
`515`	`515`	`var endpointList = new List<IdentifierDiscoveryResult>(filteredEndpoints.Count);`
`516`	`516`	`foreach (var endpoint in filteredEndpoints) {`