|
Description
Thibaud Guillot (thibaud_g)
2026-01-22 15:45:35 UTC
Created attachment 191871 [details] [review] Bug 41688: Fix NaN display in POS refund modal Test plan: Prerequisites: CurrencyFormat as 'FR' activated, have already produced a payment (with a comma separator) 1. Go to POS > register > transaction history 2. Click "Issue refund" on any paid transaction 3. Verify "Amount paid" shows correct amount (not NaN) 4. Enter refund amount with comma decimal (e.g. 5,50) 5. Verify no validation error occurs 6. Process refund successfully Created attachment 191872 [details] [review] Bug 41688: Fix NaN display in POS refund modal Test plan: Prerequisites: CurrencyFormat as 'FR' activated, have already produced a payment (with a comma separator) 1. Go to POS > register > transaction history 2. Click "Issue refund" on any paid transaction 3. Verify "Amount paid" shows correct amount (not NaN but get the value with comma separator) 4. Enter refund amount (point separator is mandatory to valid input) 5. Verify no validation error occurs 6. Process refund successfully Oh this removes "$Price" filter. Needs to be replaced by "html" filter right ? Created attachment 192333 [details] [review] Bug 41688: Fix NaN display in POS refund modal Test plan: Prerequisites: CurrencyFormat as 'FR' activated, have already produced a payment (with a comma separator) 1. Go to POS > register > transaction history 2. Click "Issue refund" on any paid transaction 3. Verify "Amount paid" shows correct amount (not NaN but get the value with comma separator) 4. Enter refund amount (point separator is mandatory to valid input) 5. Verify no validation error occurs 6. Process refund successfully Sponsored-by: BibLibre (In reply to Fridolin Somers from comment #3) > Oh this removes "$Price" filter. > Needs to be replaced by "html" filter right ? Yes Fred.. my bad :) Fixed ! Created attachment 192584 [details] [review] Bug 41688: Fix NaN display in POS refund modal Test plan: Prerequisites: CurrencyFormat as 'FR' activated, have already produced a payment (with a comma separator) 1. Go to POS > register > transaction history 2. Click "Issue refund" on any paid transaction 3. Verify "Amount paid" shows correct amount (not NaN but get the value with comma separator) 4. Enter refund amount (point separator is mandatory to valid input) 5. Verify no validation error occurs 6. Process refund successfully Sponsored-by: BibLibre Signed-off-by: David Nind <david@davidnind.com> Created attachment 192585 [details]
Screenshot - refund modal formatting
Just attaching a screenshot of the refund form - not sure if it is this bug, but it may need some formatting changes to make it "tidy".
NaN = Not a Number |