Browse Source

Fix BasicOrderCancellationAuthorizationHandler

Resolve #105
pull/107/head 1.6.2
gdlcf88 6 years ago
parent
commit
8acd07ec25
  1. 2
      common.props
  2. 21
      modules/EasyAbp.EShop.Orders/src/EasyAbp.EShop.Orders.Application/EasyAbp/EShop/Orders/Orders/BasicOrderCancellationAuthorizationHandler.cs

2
common.props

@ -1,7 +1,7 @@
<Project> <Project>
<PropertyGroup> <PropertyGroup>
<LangVersion>latest</LangVersion> <LangVersion>latest</LangVersion>
<Version>1.6.1</Version> <Version>1.6.2</Version>
<NoWarn>$(NoWarn);CS1591</NoWarn> <NoWarn>$(NoWarn);CS1591</NoWarn>
<GeneratePackageOnBuild>true</GeneratePackageOnBuild> <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
<Authors>EasyAbp Team</Authors> <Authors>EasyAbp Team</Authors>

21
modules/EasyAbp.EShop.Orders/src/EasyAbp.EShop.Orders.Application/EasyAbp/EShop/Orders/Orders/BasicOrderCancellationAuthorizationHandler.cs

@ -32,13 +32,7 @@ namespace EasyAbp.EShop.Orders.Orders
context.Fail(); context.Fail();
return; return;
} }
if (!resource.IsPaid())
{
context.Succeed(requirement);
return;
}
if (resource.CustomerUserId != _currentUser.GetId()) if (resource.CustomerUserId != _currentUser.GetId())
{ {
if (!await _permissionChecker.IsGrantedAsync(OrdersPermissions.Orders.Manage)) if (!await _permissionChecker.IsGrantedAsync(OrdersPermissions.Orders.Manage))
@ -47,14 +41,19 @@ namespace EasyAbp.EShop.Orders.Orders
return; return;
} }
if (!await _storeOwnerStore.IsStoreOwnerAsync(resource.StoreId, _currentUser.GetId()) &&
if (await _storeOwnerStore.IsStoreOwnerAsync(resource.StoreId, _currentUser.GetId()) || !await _permissionChecker.IsGrantedAsync(OrdersPermissions.Orders.CrossStore))
await _permissionChecker.IsGrantedAsync(OrdersPermissions.Orders.CrossStore))
{ {
context.Succeed(requirement); context.Fail();
return; return;
} }
} }
if (!resource.IsPaid())
{
context.Succeed(requirement);
return;
}
} }
} }
} }
Loading…
Cancel
Save