diff --git a/src/org/openbravo/retail/posterminal/UpdateCashup.java b/src/org/openbravo/retail/posterminal/UpdateCashup.java index 2bdbc0950..2afbcc77b 100644 --- a/src/org/openbravo/retail/posterminal/UpdateCashup.java +++ b/src/org/openbravo/retail/posterminal/UpdateCashup.java @@ -85,7 +85,7 @@ public class UpdateCashup { } else if (cashUpReportString.lastIndexOf("Z") != -1) { cashUpReportDate = (Date) JsonToDataConverter.convertJsonToPropertyValue( PropertyByType.DATETIME, - (cashUpReportString).subSequence(0, (cashUpReportString).lastIndexOf("Z")) + ":00"); + (cashUpReportString).subSequence(0, (cashUpReportString).lastIndexOf("Z"))); } else { cashUpReportDate = (Date) JsonToDataConverter .convertJsonToPropertyValue(PropertyByType.DATETIME, cashUpReportString); diff --git a/src/org/openbravo/retail/posterminal/master/Cashup.java b/src/org/openbravo/retail/posterminal/master/Cashup.java index 66e2dc408..fc8701a6a 100644 --- a/src/org/openbravo/retail/posterminal/master/Cashup.java +++ b/src/org/openbravo/retail/posterminal/master/Cashup.java @@ -8,6 +8,7 @@ */ package org.openbravo.retail.posterminal.master; +import java.math.BigDecimal; import java.text.DateFormat; import java.text.SimpleDateFormat; import java.util.ArrayList; @@ -52,6 +53,7 @@ import org.openbravo.service.json.JsonConstants; public class Cashup extends JSONProcessSimple { private static final Logger log = LogManager.getLogger(); + private static final DateFormat dateFormatUTC = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss'Z'"); @Inject @Any @@ -120,6 +122,9 @@ public class Cashup extends JSONProcessSimple { return result; } + final TimeZone tz = TimeZone.getTimeZone("UTC"); + dateFormatUTC.setTimeZone(tz); + String isprocessed; if (jsonsent.has("isprocessed")) { isprocessed = jsonsent.getString("isprocessed"); @@ -167,13 +172,9 @@ public class Cashup extends JSONProcessSimple { cashupJSON.put("netReturns", cashup[3]); cashupJSON.put("grossReturns", cashup[4]); cashupJSON.put("totalRetailTransactions", cashup[5]); - TimeZone tz = TimeZone.getTimeZone("UTC"); - DateFormat df = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm'Z'"); - df.setTimeZone(tz); - String nowAsISO = df.format(cashup[6]); - cashupJSON.put("creationDate", nowAsISO); + cashupJSON.put("creationDate", dateFormatUTC.format(cashup[6])); cashupJSON.put("userId", cashup[7]); - cashupJSON.put("isprocessed", ((Boolean) cashup[8]) ? "Y" : "N"); + cashupJSON.put("isprocessed", (Boolean) cashup[8]); cashupJSON.put("posterminal", cashup[9]); cashupJSON.put("organization", cashup[10]); cashupJSON.put("totalDeleteTickets", cashup[11]); @@ -378,20 +379,21 @@ public class Cashup extends JSONProcessSimple { // Set the cashManagement data result.put("id", cashMgmtJSON.get("id")); result.put("description", cashMgmtJSON.get("description")); - result.put("amount", totalamt.toString()); - result.put("origAmount", totalamt.toString()); + result.put("amount", totalamt); + result.put("origAmount", totalamt); result.put("type", cashMgmtJSON.get("paymentAmount").toString().equals("0") ? "deposit" : "drop"); result.put("reasonId", reasonId); result.put("paymentMethodId", paymentmethodId); - result.put("creationDate", cashMgmtJSON.get("creationDate").toString()); + result.put("creationDate", + dateFormatUTC.format(((FIN_FinaccTransaction) cashMgmt).getCreationDate())); result.put("timezoneOffset", "0"); + result.put("posTerminal", posId); result.put("userId", cashMgmtJSON.get("createdBy")); result.put("user", cashMgmtJSON.get("createdBy$_identifier")); result.put("isocode", cashMgmtJSON.get("currency$_identifier")); result.put("cashup_id", cashMgmtJSON.get("obposAppCashup")); result.put("glItem", cashMgmtJSON.get("gLItem")); - result.put("_idx", ""); respArray.put(result); } } @@ -414,6 +416,7 @@ public class Cashup extends JSONProcessSimple { JSONObject productCategoryObject = new JSONObject(); productCategoryObject.put("prodCatId", productCategory.getProductCategory().getId()); + productCategoryObject.put("prodCatName", productCategory.getProductCategory().getName()); productCategoryObject.put("numberOfItems", productCategory.getNumberofitems()); productCategoryObject.put("totalAmount", productCategory.getTotalamount()); @@ -438,6 +441,7 @@ public class Cashup extends JSONProcessSimple { JSONObject userObject = new JSONObject(); userObject.put("id", user.getSupervisor().getId()); + userObject.put("name", user.getSupervisor().getName()); userObject.put("totalCompleteTickets", user.getTotalcompletetickets()); userObject.put("totalAmount", user.getTotalamount()); @@ -459,7 +463,9 @@ public class Cashup extends JSONProcessSimple { JSONArray paymentMethodArray = new JSONArray(); for (OBPOSPaymentMethodCashup paymentMethod : paymentMethodList) { - + if (BigDecimal.ZERO.compareTo(paymentMethod.getTotalreceipt()) == 0) { + continue; + } JSONObject paymentMethodObject = new JSONObject(); paymentMethodObject.put("name", paymentMethod.getSearchkey()); paymentMethodObject.put("quantityReceipt", paymentMethod.getTotalreceipt()); diff --git a/src/org/openbravo/retail/posterminal/term/Payments.java b/src/org/openbravo/retail/posterminal/term/Payments.java index d1ecab09a..ac286f30c 100644 --- a/src/org/openbravo/retail/posterminal/term/Payments.java +++ b/src/org/openbravo/retail/posterminal/term/Payments.java @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2012-2022 Openbravo S.L.U. + * Copyright (C) 2012-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -63,7 +63,8 @@ public class Payments extends JSONTerminalProperty { + " paymentType, " + " color.hexColor as color, " + " pmg_color.hexColor as pmg_color, " - + " p.paymentMethod.currency.pricePrecision as obposPricePrecision " + + " p.paymentMethod.currency.pricePrecision as obposPricePrecision, " + + " p.line as lineNo " + "from OBPOS_App_Payment as p " + " left join p.financialAccount as f " + " left join f.currency as c " @@ -171,6 +172,7 @@ public class Payments extends JSONTerminalProperty { if (objPayment[8] != null) { payment.put("obposPricePrecision", objPayment[17]); } + payment.put("lineNo", objPayment[18]); // If the Payment Method is cash, load the rounding properties of the currency if (appPayment.getPaymentMethod().isCash()) { diff --git a/web-test/model/business-object/cashup/Cashup-initCashup-StateAction-fromBackend.test.js b/web-test/model/business-object/cashup/Cashup-initCashup-StateAction-fromBackend.test.js index cda2576d4..eda3f8746 100644 --- a/web-test/model/business-object/cashup/Cashup-initCashup-StateAction-fromBackend.test.js +++ b/web-test/model/business-object/cashup/Cashup-initCashup-StateAction-fromBackend.test.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -64,7 +64,7 @@ describe('Cashup - init cashup State Action - from backend', () => { }; deepfreeze(expectedState); const result = initCashup(initialState, payloadInitFromBackend); - expect(result).toEqual(expectedState); + expect(result).toEqual(expect.objectContaining(expectedState)); }); it('initialize cashup from backend - without cashPaymentMethodInfo', () => { @@ -92,6 +92,6 @@ describe('Cashup - init cashup State Action - from backend', () => { initialState, payloadWithoutCashPaymentMethodInfo ); - expect(result).toEqual(expectedState); + expect(result).toEqual(expect.objectContaining(expectedState)); }); }); diff --git a/web-test/model/business-object/cashup/test-data/cashupAfterComplete.js b/web-test/model/business-object/cashup/test-data/cashupAfterComplete.js index 0bcaa4c6f..b2076e220 100644 --- a/web-test/model/business-object/cashup/test-data/cashupAfterComplete.js +++ b/web-test/model/business-object/cashup/test-data/cashupAfterComplete.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -17,7 +17,7 @@ module.exports = { netReturns: 0, grossReturns: 0, totalRetailTransactions: 0, - totalStartings: 250, + totalStartings: 238.04, creationDate: '2020-06-26T14:36:53.173Z', userId: '3073EDF96A3C42CC86C7069E379522D2', posterminal: '9104513C2D0741D4850AE8493998A7C8', diff --git a/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackend.js b/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackend.js index f92b164cc..b42dc0d38 100644 --- a/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackend.js +++ b/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackend.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -19,6 +19,7 @@ module.exports = { totalStartings: 0, creationDate: '2020-06-24T07:45Z', userId: '3073EDF96A3C42CC86C7069E379522D2', + cashCloseInfo: [], cashTaxInfo: [ { id: 'E44EB6A6CF2D362EA88FAC723638CDA6', @@ -41,6 +42,7 @@ module.exports = { rate: 1, isocode: 'EUR', newPaymentMethod: false, + usedInCurrentTrx: false, cashManagements: [] }, { @@ -56,6 +58,7 @@ module.exports = { rate: 1, isocode: 'EUR', newPaymentMethod: false, + usedInCurrentTrx: true, cashManagements: [] }, { @@ -71,6 +74,7 @@ module.exports = { rate: 0.76082, isocode: 'USD', newPaymentMethod: false, + usedInCurrentTrx: false, cashManagements: [] }, { @@ -86,6 +90,7 @@ module.exports = { rate: 1, isocode: 'EUR', newPaymentMethod: false, + usedInCurrentTrx: false, cashManagements: [] } ], diff --git a/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackendWithoutPayments.js b/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackendWithoutPayments.js index e22d0b742..f0996a079 100644 --- a/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackendWithoutPayments.js +++ b/web-test/model/business-object/cashup/test-data/cashupAfterLoadCashupFromBackendWithoutPayments.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -19,7 +19,7 @@ module.exports = { totalStartings: 0, creationDate: '2020-06-24T07:45Z', userId: '3073EDF96A3C42CC86C7069E379522D2', - cashCloseInfo: undefined, + cashCloseInfo: [], cashTaxInfo: [ { id: 'E44EB6A6CF2D362EA88FAC723638CDA6', diff --git a/web-test/model/business-object/cashup/test-data/messagesAfterCompleteCashupSharedPayments.js b/web-test/model/business-object/cashup/test-data/messagesAfterCompleteCashupSharedPayments.js index f0962fdd1..609e8b5bd 100644 --- a/web-test/model/business-object/cashup/test-data/messagesAfterCompleteCashupSharedPayments.js +++ b/web-test/model/business-object/cashup/test-data/messagesAfterCompleteCashupSharedPayments.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020-2021 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -2025,7 +2025,7 @@ module.exports = [ netReturns: 0, grossReturns: 0, totalRetailTransactions: 0, - totalStartings: 250, + totalStartings: 238.04, creationDate: '2020-06-26T14:36:53.173Z', userId: '3073EDF96A3C42CC86C7069E379522D2', posterminal: '9104513C2D0741D4850AE8493998A7C8', diff --git a/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/CashupUtils.js b/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/CashupUtils.js index 1580b99d3..36cbaa115 100644 --- a/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/CashupUtils.js +++ b/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/CashupUtils.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020-2022 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -85,6 +85,7 @@ newCashup.users = []; newCashup.productCategories = []; newCashup.paymentMethods = []; + newCashup.cashCloseInfo = []; return OB.App.State.DocumentSequence.Utils.generateCashupDocumentNumber( { ...newCashup }, @@ -94,7 +95,7 @@ }, createNewCashupFromBackend(payload) { - const { cashup, currentCashupFromBackend } = payload; + const { cashup, documentSequence, currentCashupFromBackend } = payload; const newCashup = { ...cashup }; newCashup.id = currentCashupFromBackend.id; @@ -114,7 +115,7 @@ newCashup.cashTaxInfo = OB.App.State.Cashup.Utils.getTaxesFromBackendObject( currentCashupFromBackend.cashTaxInfo ); - newCashup.cashCloseInfo = currentCashupFromBackend.cashCloseInfo; + newCashup.cashCloseInfo = currentCashupFromBackend.cashCloseInfo || []; newCashup.totalDeleteTickets = currentCashupFromBackend.totalDeleteTickets; @@ -129,6 +130,16 @@ newCashup.productCategories = currentCashupFromBackend.productCategories; newCashup.paymentMethods = currentCashupFromBackend.paymentMethods; + if (documentSequence) { + const sequenceName = OB.App.State.DocumentSequence.Utils.getCashupSequenceName(); + const { sequencePrefix, sequenceNumber } = documentSequence[ + sequenceName + ]; + newCashup.obposSequencename = sequenceName; + newCashup.obposSequenceprefix = sequencePrefix; + newCashup.obposSequencenumber = sequenceNumber; + } + return newCashup; }, diff --git a/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/PaymentMethodUtils.js b/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/PaymentMethodUtils.js index e8bc0b437..d418fe56b 100644 --- a/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/PaymentMethodUtils.js +++ b/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/PaymentMethodUtils.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020-2021 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -44,7 +44,7 @@ newCashup.totalStartings = OB.DEC.add( newCashup.totalStartings, - startingCash + OB.DEC.mul(startingCash, terminalPayment.rate) ); } } @@ -100,7 +100,16 @@ return paymentMethods; }, - getPaymentMethodFromBackendObject(paymentMethodCashUpModel) { + getCashManagementFromBackendObject(cashManagementInfos) { + return cashManagementInfos.map(cashMgmInfo => { + return { + ...cashMgmInfo, + timezoneOffset: new Date().getTimezoneOffset() + }; + }); + }, + + getPaymentMethodFromBackendObject(paymentMethodCashUpModel, cashMgmInfo) { const paymentMethodInfo = { id: paymentMethodCashUpModel.id, paymentMethodId: paymentMethodCashUpModel.paymentmethod_id, @@ -117,7 +126,17 @@ isocode: paymentMethodCashUpModel.isocode, lineNo: paymentMethodCashUpModel.lineNo, newPaymentMethod: false, - cashManagements: [] + usedInCurrentTrx: !!( + paymentMethodCashUpModel.totalSales || + paymentMethodCashUpModel.totalReturns || + paymentMethodCashUpModel.totalDeposits || + paymentMethodCashUpModel.totalDrops + ), + cashManagements: OB.App.State.Cashup.Utils.getCashManagementFromBackendObject( + cashMgmInfo.filter( + c => c.paymentMethodId === paymentMethodCashUpModel.paymentmethod_id + ) + ) }; if (paymentMethodCashUpModel.countPerAmount) { paymentMethodInfo.countPerAmount = @@ -127,7 +146,7 @@ }, addPaymentsFromBackendCashup(payload) { - const { currentCashupFromBackend, terminalPayments } = payload; + const { newCashup, currentCashupFromBackend, terminalPayments } = payload; const paymentMethods = []; currentCashupFromBackend.cashPaymentMethodInfo.forEach( @@ -153,13 +172,23 @@ paymentMethodCashUp.totalDrops !== 0) ) { const paymentFromBackend = OB.App.State.Cashup.Utils.getPaymentMethodFromBackendObject( - paymentMethodCashUp + paymentMethodCashUp, + currentCashupFromBackend.cashMgmInfo ); paymentMethods.push(paymentFromBackend); } } ); + // Update cashup properties based on payment methods + if (newCashup) { + newCashup.totalStartings = paymentMethods.reduce( + (total, p) => + OB.DEC.add(total, OB.DEC.mul(p.initialCounted || 0, p.rate)), + 0 + ); + } + return paymentMethods; }, diff --git a/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/actions/InitCashup.js b/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/actions/InitCashup.js index 979af679e..76164c940 100644 --- a/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/actions/InitCashup.js +++ b/web/org.openbravo.retail.posterminal/app/model/business-object/cashup/actions/InitCashup.js @@ -1,6 +1,6 @@ /* ************************************************************************************ - * Copyright (C) 2020-2022 Openbravo S.L.U. + * Copyright (C) 2020-2023 Openbravo S.L.U. * Licensed under the Openbravo Commercial License version 1.0 * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html * or in the legal folder of this module distribution. @@ -53,12 +53,13 @@ newCashup = OB.App.State.Cashup.Utils.createNewCashupFromBackend({ cashup, + documentSequence: state.DocumentSequence, currentCashupFromBackend }); if (currentCashupFromBackend.cashPaymentMethodInfo.length !== 0) { newCashup.cashPaymentMethodInfo = OB.App.State.Cashup.Utils.addPaymentsFromBackendCashup( - { currentCashupFromBackend, terminalPayments } + { newCashup, currentCashupFromBackend, terminalPayments } ); } else { newCashup.cashPaymentMethodInfo = OB.App.State.Cashup.Utils.initializePaymentMethodCashup( diff --git a/web/org.openbravo.retail.posterminal/app/model/business-object/ticket/TicketUtils.js b/web/org.openbravo.retail.posterminal/app/model/business-object/ticket/TicketUtils.js index 20ad7052f..5b7c9ed46 100644 --- a/web/org.openbravo.retail.posterminal/app/model/business-object/ticket/TicketUtils.js +++ b/web/org.openbravo.retail.posterminal/app/model/business-object/ticket/TicketUtils.js @@ -1717,11 +1717,13 @@ // Load Cashup from Backend const newCashup = OB.App.State.Cashup.Utils.createNewCashupFromBackend({ cashup: state.Cashup, + documentSequence: state.DocumentSequence, currentCashupFromBackend }); if (currentCashupFromBackend.cashPaymentMethodInfo.length !== 0) { newCashup.cashPaymentMethodInfo = OB.App.State.Cashup.Utils.addPaymentsFromBackendCashup( { + newCashup, currentCashupFromBackend, terminalPayments: payload.payments }