API review - remove DefaultOrder
No change in behavior intended, removing this from our public API.
This commit is contained in:
parent
ccb5ead450
commit
4d55a6e62d
|
|
@ -48,7 +48,7 @@ namespace Microsoft.AspNet.Mvc.ApiExplorer
|
|||
/// <inheritdoc />
|
||||
public int Order
|
||||
{
|
||||
get { return DefaultOrder.DefaultFrameworkSortOrder; }
|
||||
get { return -1000; }
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
|
|
|
|||
|
|
@ -19,7 +19,7 @@ namespace Microsoft.AspNet.Mvc.ActionConstraints
|
|||
/// <inheritdoc />
|
||||
public int Order
|
||||
{
|
||||
get { return DefaultOrder.DefaultFrameworkSortOrder; }
|
||||
get { return -1000; }
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
|
|
|
|||
|
|
@ -27,7 +27,7 @@ namespace Microsoft.AspNet.Mvc.Actions
|
|||
|
||||
public int Order
|
||||
{
|
||||
get { return DefaultOrder.DefaultFrameworkSortOrder; }
|
||||
get { return -1000; }
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
|
|
|
|||
|
|
@ -54,7 +54,7 @@ namespace Microsoft.AspNet.Mvc.Actions
|
|||
|
||||
public int Order
|
||||
{
|
||||
get { return DefaultOrder.DefaultFrameworkSortOrder; }
|
||||
get { return -1000; }
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
|
|
|
|||
|
|
@ -18,7 +18,7 @@ namespace Microsoft.AspNet.Mvc.ApplicationModels
|
|||
_authorizationOptions = authorizationOptionsAccessor.Value;
|
||||
}
|
||||
|
||||
public int Order { get { return DefaultOrder.DefaultFrameworkSortOrder + 10; } }
|
||||
public int Order { get { return -1000 + 10; } }
|
||||
|
||||
public void OnProvidersExecuted([NotNull]ApplicationModelProviderContext context)
|
||||
{
|
||||
|
|
|
|||
|
|
@ -30,7 +30,7 @@ namespace Microsoft.AspNet.Mvc.ApplicationModels
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -1,21 +0,0 @@
|
|||
// Copyright (c) .NET Foundation. All rights reserved.
|
||||
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
|
||||
|
||||
namespace Microsoft.AspNet.Mvc
|
||||
{
|
||||
public static class DefaultOrder
|
||||
{
|
||||
/// <summary>
|
||||
/// The default order for sorting is -1000. Other framework code
|
||||
/// the depends on order should be ordered between -1 to -1999.
|
||||
/// User code should order at bigger than 0 or smaller than -2000.
|
||||
/// </summary>
|
||||
public static readonly int DefaultFrameworkSortOrder = -1000;
|
||||
|
||||
/// <summary>
|
||||
/// The default order for <c>CorsAuthorizationFilter</c>, <c>CorsAuthorizationFilterFactory</c>
|
||||
/// and <c>DisableCorsAuthorizationFilter</c>
|
||||
/// </summary>
|
||||
public static readonly int DefaultCorsSortOrder = int.MaxValue - 100;
|
||||
}
|
||||
}
|
||||
|
|
@ -12,7 +12,7 @@ namespace Microsoft.AspNet.Mvc.Filters
|
|||
{
|
||||
public int Order
|
||||
{
|
||||
get { return DefaultOrder.DefaultFrameworkSortOrder; }
|
||||
get { return -1000; }
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
|
|
|
|||
|
|
@ -10,7 +10,7 @@ namespace Microsoft.AspNet.Mvc.Cors
|
|||
{
|
||||
public class CorsApplicationModelProvider : IApplicationModelProvider
|
||||
{
|
||||
public int Order { get { return DefaultOrder.DefaultFrameworkSortOrder + 10; } }
|
||||
public int Order { get { return -1000 + 10; } }
|
||||
|
||||
public void OnProvidersExecuted([NotNull]ApplicationModelProviderContext context)
|
||||
{
|
||||
|
|
|
|||
|
|
@ -42,7 +42,7 @@ namespace Microsoft.AspNet.Mvc.Cors
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultCorsSortOrder;
|
||||
return int.MaxValue - 100;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -29,7 +29,7 @@ namespace Microsoft.AspNet.Mvc.Cors
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultCorsSortOrder;
|
||||
return int.MaxValue - 100;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -21,7 +21,7 @@ namespace Microsoft.AspNet.Mvc.Cors
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultCorsSortOrder;
|
||||
return int.MaxValue - 100;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -97,7 +97,7 @@ namespace Microsoft.AspNet.Mvc.Razor.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder - 999;
|
||||
return -1000 - 999;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -47,7 +47,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -52,7 +52,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -30,7 +30,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -43,7 +43,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -54,7 +54,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -77,7 +77,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -30,7 +30,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -115,7 +115,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -34,7 +34,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -101,7 +101,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -46,7 +46,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -29,7 +29,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -31,7 +31,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -32,7 +32,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder;
|
||||
return -1000;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -2029,7 +2029,7 @@ namespace Microsoft.AspNet.Mvc.Actions
|
|||
.Verifiable();
|
||||
|
||||
filterProvider.SetupGet(fp => fp.Order)
|
||||
.Returns(DefaultOrder.DefaultFrameworkSortOrder);
|
||||
.Returns(-1000);
|
||||
|
||||
var invoker = new TestControllerActionInvoker(
|
||||
actionContext,
|
||||
|
|
|
|||
|
|
@ -156,7 +156,7 @@ namespace Microsoft.AspNet.Mvc.Routing
|
|||
|
||||
actionProvider
|
||||
.SetupGet(p => p.Order)
|
||||
.Returns(DefaultOrder.DefaultFrameworkSortOrder);
|
||||
.Returns(-1000);
|
||||
|
||||
actionProvider
|
||||
.Setup(p => p.OnProvidersExecuting(It.IsAny<ActionDescriptorProviderContext>()))
|
||||
|
|
|
|||
|
|
@ -26,7 +26,7 @@ namespace BasicWebSite
|
|||
{
|
||||
get
|
||||
{
|
||||
return DefaultOrder.DefaultFrameworkSortOrder - 100;
|
||||
return -1000 - 100;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue