Fix decimal precision issue for quantity

packages/server/src/api/controllers/Inventory/InventoryAdjustments.ts

@@ -113,8 +113,9 @@ export default class InventoryAdjustmentsController extends BaseController {
       check('quantity')
         .if(check('type').exists().isIn(['increment', 'decrement']))
         .exists()
-        .isInt()
-        .toInt(),
+        .isDecimal()
+        .toFloat(),
+      // TODO: Is the cost supposed to be an integer here if it's a float everywhere else?
       check('cost')
         .if(check('type').exists().isIn(['increment']))
         .exists()

packages/server/src/api/controllers/Purchases/Bills.ts

@@ -122,7 +122,7 @@ export default class BillsController extends BaseController {
       check('entries.*.index').exists().isNumeric().toInt(),
       check('entries.*.item_id').exists().isNumeric().toInt(),
       check('entries.*.rate').exists().isNumeric().toFloat(),
-      check('entries.*.quantity').exists().isNumeric().toInt(),
+      check('entries.*.quantity').exists().isNumeric().toFloat(),
       check('entries.*.discount')
         .optional({ nullable: true })
         .isNumeric()

packages/server/src/api/controllers/Purchases/VendorCredit.ts

@@ -173,7 +173,7 @@ export default class VendorCreditController extends BaseController {
       check('entries.*.index').exists().isNumeric().toInt(),
       check('entries.*.item_id').exists().isNumeric().toInt(),
       check('entries.*.rate').exists().isNumeric().toFloat(),
-      check('entries.*.quantity').exists().isNumeric().toInt(),
+      check('entries.*.quantity').exists().isNumeric().toFloat(),
       check('entries.*.discount')
         .optional({ nullable: true })
         .isNumeric()
@@ -215,7 +215,7 @@ export default class VendorCreditController extends BaseController {
       check('entries.*.index').exists().isNumeric().toInt(),
       check('entries.*.item_id').exists().isNumeric().toInt(),
       check('entries.*.rate').exists().isNumeric().toFloat(),
-      check('entries.*.quantity').exists().isNumeric().toInt(),
+      check('entries.*.quantity').exists().isNumeric().toFloat(),
       check('entries.*.discount')
         .optional({ nullable: true })
         .isNumeric()

packages/server/src/api/controllers/Sales/CreditNotes.ts

@@ -222,7 +222,7 @@ export default class PaymentReceivesController extends BaseController {
       check('entries.*.index').exists().isNumeric().toInt(),
       check('entries.*.item_id').exists().isNumeric().toInt(),
       check('entries.*.rate').exists().isNumeric().toFloat(),
-      check('entries.*.quantity').exists().isNumeric().toInt(),
+      check('entries.*.quantity').exists().isNumeric().toFloat(),
       check('entries.*.discount')
         .optional({ nullable: true })
         .isNumeric()

packages/server/src/api/controllers/Sales/SalesEstimates.ts

@@ -166,7 +166,7 @@ export default class SalesEstimatesController extends BaseController {
       check('entries').exists().isArray({ min: 1 }),
       check('entries.*.index').exists().isNumeric().toInt(),
       check('entries.*.item_id').exists().isNumeric().toInt(),
-      check('entries.*.quantity').exists().isNumeric().toInt(),
+      check('entries.*.quantity').exists().isNumeric().toFloat(),
       check('entries.*.rate').exists().isNumeric().toFloat(),
       check('entries.*.description')
         .optional({ nullable: true })

packages/server/src/api/controllers/Sales/SalesReceipts.ts

@@ -139,7 +139,7 @@ export default class SalesReceiptsController extends BaseController {
       check('entries.*.id').optional({ nullable: true }).isNumeric().toInt(),
       check('entries.*.index').exists().isNumeric().toInt(),
       check('entries.*.item_id').exists().isNumeric().toInt(),
-      check('entries.*.quantity').exists().isNumeric().toInt(),
+      check('entries.*.quantity').exists().isNumeric().toFloat(),
       check('entries.*.rate').exists().isNumeric().toFloat(),
       check('entries.*.discount')
         .optional({ nullable: true })

packages/server/src/api/controllers/Warehouses/WarehouseTransfers.ts

@@ -39,7 +39,7 @@ export class WarehousesTransfers extends BaseController {
         check('entries.*.index').exists(),
         check('entries.*.item_id').exists(),
         check('entries.*.description').optional(),
-        check('entries.*.quantity').exists().isInt().toInt(),
+        check('entries.*.quantity').exists().isDecimal().toFloat(),
         check('entries.*.cost').optional().isDecimal().toFloat(),
       ],
       this.validationResult,
@@ -66,7 +66,7 @@ export class WarehousesTransfers extends BaseController {
         check('entries.*.index').exists(),
         check('entries.*.item_id').exists().isInt().toInt(),
         check('entries.*.description').optional(),
-        check('entries.*.quantity').exists().isInt({ min: 1 }).toInt(),
+        check('entries.*.quantity').exists().isDecimal().toFloat(),
         check('entries.*.cost').optional().isDecimal().toFloat(),
       ],
       this.validationResult,

packages/server/src/database/migrations/20240120132327_convert_items_entries_quantity_to_decimal.js

@@ -0,0 +1,9 @@
+exports.up = function(knex) {
+  return knex.schema.alterTable('items_entries', (table) => {
+    table.decimal('quantity', 17, 7).alter();
+  });
+};
+
+exports.down = function(knex) {
+  return knex.table('items_entries', (table) => { });
+};