fixing the receiving for finance

CPRNIMS.Domain/Services/Finance/RR.cs

@@ -1,7 +1,6 @@
 using CPRNIMS.Domain.Contracts.Finance;
 using CPRNIMS.Infrastructure.Database;
 using CPRNIMS.Infrastructure.Dto.Finance;
-using CPRNIMS.Infrastructure.Dto.Items;
 using CPRNIMS.Infrastructure.Entities.Finance;
 using CPRNIMS.Infrastructure.Entities.Purchasing;
 using Microsoft.Data.SqlClient;
@@ -22,8 +21,6 @@ namespace CPRNIMS.Domain.Services.Finance
             _dbContext = dbContext;
         }
         public async Task<List<ForPayment>> GetAllClosedPO(RRDetailsDto itemDto)
-        {
-            try
         {
             var allItems = await _dbContext.ForPayments
                     .FromSqlRaw($"EXEC GetAllClosedPO @UserId = '{itemDto.UserId}'")
@@ -31,12 +28,6 @@ namespace CPRNIMS.Domain.Services.Finance
 
             return allItems ?? new List<ForPayment>();
         }
-            catch (SqlException ex)
-            {
-                ex.ToString();
-                throw;
-            }
-        }
         public async Task<List<ReceivingDetail>> GetRRDetailByPO(RRDetailsDto itemDto)
         {
             var allItems = await _dbContext.ReceivingDetails
@@ -50,8 +41,6 @@ namespace CPRNIMS.Domain.Services.Finance
         }
 
         public async Task<RRDetail> PostPutPayment(RRDetailsDto itemDto)
-        {
-            try
         {
             await _dbContext.Database
                  .ExecuteSqlRawAsync("EXEC PostPutPayment @UserId, @PONo, @POTypeId, @PRDetailsId",
@@ -61,11 +50,5 @@ namespace CPRNIMS.Domain.Services.Finance
                   new SqlParameter("@PRDetailsId", itemDto.PRDetailsId));
             return new RRDetail();
         }
-            catch (SqlException ex)
-            {
-                ex.ToString();
-                throw;
-            }
-        }
     }
 }

CPRNIMS.WebApi/Controllers/Account/AccountController.cs

@@ -385,34 +385,15 @@ namespace CPRNIMS.WebApi.Controllers.Account
         [HttpPost("GetLandingPageByUserId")]
         public async Task<IActionResult> GetLandingPageByUserId(ControllerAccessVM model)
         {
-            try
+            return await ExecuteWithErrorHandling(
-            {
+            () => _controllerAccess.GetControllerAccessByUserId(model.UserId),
-                var controllerAccesse = await _controllerAccess.GetControllerAccessByUserId(model.UserId);
+            nameof(GetLandingPageByUserId), false);            
-                return Ok(controllerAccesse);
-
-            }
-            catch (Exception ex)
-            {
-                var message = ex.InnerException?.ToString() ?? ex.Message.ToString();
-                await PostErrorMessage(message, "WebApi");
-                throw;
-            }
         }
         [HttpGet("GetAllRoles")]
         public async Task<IActionResult> GetAllRoles()
-        {
-            try
         {
             var roles = await _roleManager.Roles.ToListAsync();
             return Ok(roles);
-
-            }
-            catch (Exception ex)
-            {
-                var message = ex.InnerException?.ToString() ?? ex.Message.ToString();
-                await PostErrorMessage(message, "WebApi");
-                throw;
-            }
         }
         [HttpGet("GetRoles")]
         public async Task<IActionResult> GetRoles()
@@ -434,7 +415,7 @@ namespace CPRNIMS.WebApi.Controllers.Account
                 }
                 else
                 {
-                    return NotFound(); // or any other appropriate status code
+                    return NotFound(); 
                 }
             }
             catch (Exception ex)

CPRNIMS.WebApps/Views/Receiving/Index.cshtml

@@ -49,7 +49,7 @@
         </div>
         <link href="~/css/pr/receiving.css" rel="stylesheet" />
         @await Html.PartialAsync("PagesView/Receiving/_Receiving")
-        <script src="~/JsFunctions/Receiving/Receiving.js"></script>
-        @await Html.PartialAsync("PagesView/Receiving/_ReceivingScripts")
     </div>
 </body>
+<script src="~/JsFunctions/Receiving/Receiving.js"></script>
+@await Html.PartialAsync("PagesView/Receiving/_ReceivingScripts")

CPRNIMS.WebApps/Views/Shared/PagesView/Receiving/_ReceivingScripts.cshtml

@@ -2,11 +2,11 @@
     <div id="loader" class="loader"></div>
 </div>
 <script src="~/jsfunctions/receiving/receivingvar.js"></script>
-<script src="~/jsfunctions/receiving/receivingcolumn.js"></script>
+<script src="~/jsfunctions/receiving/ReceivingColumnV2.js"></script>
-<script src="~/jsfunctions/receiving/ReceivingViewV2.js"></script>
+<script src="~/jsfunctions/receiving/ReceivingViewV4.js"></script>
 <script src="~/jsfunctions/receiving/receivingcallback.js"></script>
 <script src="~/jsfunctions/receiving/receivingbutton.js"></script>
-<script src="~/jsfunctions/receiving/receivingpostput.js"></script>
+<script src="~/jsfunctions/receiving/ReceivingPostPutV2.js"></script>
 <script src="~/jsfunctions/utilities/searchengine.js"></script>
 <script src="~/jsfunctions/utilities/stylesv3.js"></script>
 <script src="~/jsfunctions/utilities/utilsV3.js"></script>

CPRNIMS.WebApps/Views/Shared/PartialView/_Sidebar.cshtml

@@ -17,7 +17,6 @@
             var myControllerAccess = await _account.GetLandingPageByUserId(userCred);
             var groupedElements = myControllerAccess.GroupBy(e => e.ElementMenuName).ToList();
        
-            // Pass the data to the view
             ViewBag.TempDataElements = groupedElements;
 
         }

CPRNIMS.WebApps/wwwroot/JsFunctions/Finance/RRColumnV2.js

@@ -64,7 +64,6 @@ var colDefForReceivingSKU = [
                     && UserRights == 'Approver1' && UserRights == 'Approver2') {
                     return '<input type="text" class="editable-qty" style="width:120px;text-align:center;" value="' + formattedData + '" />';
                 } else {           
-                    console.log('ELSE PASOK BA?');
                     // Non-admin users get a readonly input
                     return '<input readonly class="editable-qty" style="width:120px;text-align:center;" value="' + formattedData + '" />';
                 }

CPRNIMS.WebApps/wwwroot/JsFunctions/PO/POPutPostV2.js

@@ -79,7 +79,7 @@ function putPOItemDetail() {
     const PortOfDischarge = portOfDischarge.value;
 
     var poTypeId = $('#potype-cancel').val();
-    console.log('PodId', PodId);
+
     if (!UOMId || !Specification || !Qty || !UnitPrice || !PaymentTermsId || PaymentTermsId == 0 || !ItemDescription) {
         showToast('warning', 'Please fill the required fields !', 'Updating failed!', 4000);
         if (!UOMId) {

CPRNIMS.WebApps/wwwroot/JsFunctions/Receiving/ReceivingPostPutV2.js

@@ -83,9 +83,14 @@
         return;
     }
 
-    const confirmation = confirm('Are you sure you want to proceed?');
+    showConfirmation({
-
+        title: 'Receiving',
-    if (confirmation) {
+        message: 'Are you sure you want to receive this item? This action cannot be undone.',
+        type: 'warning',
+        confirmText: 'Yes',
+        cancelText: 'No'
+    }).then((confirmed) => {
+        if (confirmed) {
             $.ajax($.extend({
                 url: '/Receiving/PostPutReceiving',
                 type: 'POST',
@@ -95,14 +100,15 @@
                         $('#viewRRDetailByPO').modal('hide');
                         receivingTable.ajax.reload();
 
-                    alert('Item Received Successfully!');
+                        showToast('success', 'Item Received Successfully!', 'Receiving', 4000);
                     } else {
-                    alert('Failed: ' + response.response);
+                        showToast('error', response.response, 'Updating failed!', 4000);
                     }
                 },
                 error: errorHandler
             }, beforeComplete(loader)));
         }
+    });
 }
 function putPOClose() {
     loader = $('#overlay, #loader').css('z-index', 1070);

CPRNIMS.WebApps/wwwroot/JsFunctions/Receiving/ReceivingViewV4.js

@@ -22,7 +22,7 @@
         amountUSD.style.display = 'none';
         amountUSDLabel.style.display = 'none';
     }
-    if (UserRights !== 'LLIFINANCE' && UserRights !== 'CnvssAppver' && UserRights !== 'POApprover') { //dont touch &&      
+    if (UserRights !== 'CnvssAppver' && UserRights !== 'POApprover') { //dont touch &&      
         btnSubmit.style.display = 'none';
         amountUSDLabel.style.display = 'none';
         vatRateLabel.style.display = 'none';
@@ -36,7 +36,7 @@
     //let btnIncomplete = document.getElementById('btnIncomplete');
     let btnClosePO = document.getElementById('btnClosePO');
     btnPayment.style.display = 'none';
-    if (UserRights == 'POApprover' || UserRights == 'LLIFINANCE' || UserRights == 'CnvssAppver') {
+    if (UserRights == 'POApprover' || UserRights == 'CnvssAppver') {
         btnSubmit.style.display = 'none';
         btnReject.style.display = 'none';
         dynamicColumn = colRRFinance;