admin/marco.pms.api
1
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Added new condition for checking the permission for expense Action API

Browse Source
This commit is contained in:
ashutosh.nehete 2025-08-06 12:14:18 +05:30
parent 19aedfb648
commit fb1f34f950
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Marco.Pms.Services/Service/ExpensesService.cs
View File

@ -674,7 +674,7 @@ namespace Marco.Pms.Services.Service
var permissionService = scope.ServiceProvider.GetRequiredService<PermissionServices>(); var permissionService = scope.ServiceProvider.GetRequiredService<PermissionServices>();
foreach (var permission in requiredPermissions) foreach (var permission in requiredPermissions)
{ {
if (await permissionService.HasPermission(permission.PermissionId, loggedInEmployee.Id)) if (await permissionService.HasPermission(permission.PermissionId, loggedInEmployee.Id) && model.StatusId != Review)
{ {
hasPermission = true; hasPermission = true;
break; break;
@ -847,7 +847,7 @@ namespace Marco.Pms.Services.Service
try try
{ {
await _context.SaveChangesAsync(); await _context.SaveChangesAsync();
_logger.LogInfo("Successfully updated project {ProjectId} by user {UserId}.", id, loggedInEmployee.Id); _logger.LogInfo("Successfully updated Expense {ExpenseId} by user {UserId}.", id, loggedInEmployee.Id);
} }
catch (DbUpdateConcurrencyException ex) catch (DbUpdateConcurrencyException ex)
{ {