Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-5064

Error voiding invoices when using a comma to separate whole numbers from cents

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.9.0
    • Fix Version/s: 5.10.0-b1
    • Component/s: None
    • Labels:
      None

      Description

      When voiding an invoice that has a unit cost with a comma in the place of a typical decimal, an error is returned:

      [2024-01-24T23:04:53.030992+00:00] general.ERROR: PDOException: SQLSTATE[HY000]: General error: 1366 Incorrect decimal value: '1234,5600' for column 'amount' at row 1 in /home/blestau/public_html/590/vendors/minphp/db/src/PdoConnection.php:196 Stack trace: #0 /home/blestau/public_html/590/vendors/minphp/db/src/PdoConnection.php(196): PDOStatement->execute() #1 /home/blestau/public_html/590/vendors/minphp/record/src/Record.php(328): Minphp\Db\PdoConnection->query() #2 /home/blestau/public_html/590/app/models/invoices.php(646): Minphp\Record\Record->update() #3 /home/blestau/public_html/590/app/controllers/admin_clients.php(5486): Invoices->edit() #4 /home/blestau/public_html/590/vendors/minphp/bridge/src/Lib/Dispatcher.php(142): AdminClients->editInvoice() #5 /home/blestau/public_html/590/index.php(21): Dispatcher::dispatch() #6

      {main}

      To reproduce:

      1. Create currency using the currency format "1 234,56"
      2 Create an invoice using this currency and the following price for a line item: 1234,56
      3. Edit the invoice and click to void it. White page + error. Invoice appears to be voided though.

      Using PHP 8.2 on Blesta 5.9.0

        Activity

          People

          • Assignee:
            abdy Abdy Franco
            Reporter:
            admin Paul Phillips
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Fix Release Date:
              5/Apr/24

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 43 minutes
              43m

                Agile