quant_check_health.py

gistfile1.py

quants = env['stock.quant'].search([])
move_line_ids = []

warning = ''
for quant in quants:
    move_lines = env["stock.move.line"].search([
        ('product_id', '=', quant.product_id.id),
        ('location_id', '=', quant.location_id.id),
        ('lot_id', '=', quant.lot_id.id),
        ('package_id', '=', quant.package_id.id),
        ('owner_id', '=', quant.owner_id.id),
        ('product_qty', '!=', 0)
    ])
    move_line_ids += move_lines.ids
    reserved_on_move_lines = sum(move_lines.mapped('product_qty'))
    move_line_str = str.join(', ', [str(move_line_id) for move_line_id in move_lines.ids])

    if quant.location_id.should_bypass_reservation():
        # If a quant is in a location that should bypass the reservation, its `reserved_quantity` field
        # should be 0.
        if quant.reserved_quantity != 0:
            warning += "Problematic quant found: %s (quantity: %s, reserved_quantity: %s)\n" % (quant.id, quant.quantity, quant.reserved_quantity)
            warning += "its `reserved_quantity` field is not 0 while its location should bypass the reservation\n"
            if move_lines:
                warning += "These move lines are reserved on it: %s (sum of the reservation: %s)\n" % (move_line_str, reserved_on_move_lines)
            else:
                warning += "no move lines are reserved on it, you can safely reset its `reserved_quantity` to 0\n"
            warning += '******************\n'
    else:
        # If a quant is in a reservable location, its `reserved_quantity` should be exactly the sum
        # of the `product_qty` of all the partially_available / assigned move lines with the same
        # characteristics.
        if quant.reserved_quantity == 0:
            if move_lines:
                warning += "Problematic quant found: %s (quantity: %s, reserved_quantity: %s)\n" % (quant.id, quant.quantity, quant.reserved_quantity)
                warning += "its `reserved_quantity` field is 0 while these move lines are reserved on it: %s (sum of the reservation: %s)\n" % (move_line_str, reserved_on_move_lines)
                warning += '******************\n'
        elif quant.reserved_quantity < 0:
            warning += "Problematic quant found: %s (quantity: %s, reserved_quantity: %s)\n" % (quant.id, quant.quantity, quant.reserved_quantity)
            warning += "its `reserved_quantity` field is negative while it should not happen\n"
            if move_lines:
                warning += "These move lines are reserved on it: %s (sum of the reservation: %s)\n" % (move_line_str, reserved_on_move_lines)
            warning += '******************\n'
        else:
            if reserved_on_move_lines != quant.reserved_quantity:
                warning += "Problematic quant found: %s (quantity: %s, reserved_quantity: %s)\n" % (quant.id, quant.quantity, quant.reserved_quantity)
                warning += "its `reserved_quantity` does not reflect the move lines reservation\n"
                warning += "These move lines are reserved on it: %s (sum of the reservation: %s)\n" % (move_line_str, reserved_on_move_lines)
                warning += '******************\n'
            else:
              if any(move_line.product_qty < 0 for move_line in move_lines):
                warning += "Problematic quant found: %s (quantity: %s, reserved_quantity: %s)\n" % (quant.id, quant.quantity, quant.reserved_quantity)
                warning += "its `reserved_quantity` correctly reflects the move lines reservation but some are negatives\n"
                warning += "These move lines are reserved on it: %s (sum of the reservation: %s)\n" % (move_line_str, reserved_on_move_lines)
                warning += '******************\n'

move_lines = env['stock.move.line'].search([('product_id.type', '=', 'product'), ('product_qty', '!=', 0), ('id', 'not in', move_line_ids)])

for move_line in move_lines:
    if not move_line.location_id.should_bypass_reservation():
       warning += "Problematic move line found: %s (reserved_quantity: %s)\n" % (move_line.id, move_line.product_qty)
       warning += "There is no exiting quants despite its `reserved_quantity`\n"
       warning += '******************\n'

if warning == '':
    warning = 'nothing seems wrong'
raise Warning(warning)

where is the actions server code to fix the bug?

You have to create it, check https://www.odoo.com/forum/help-1/question/it-is-not-possible-to-unreserve-more-products-of-than-you-have-in-stock-138783

@nehmiascr why there is steel no fix in odoo ?

Just FYI, here is a variant that will update the stock.quant records reserved_quantity to match all move lines: https://gist.github.com/ryanc-me/632fd59639a8a68e041c876abe87168f

I think this method is safer than what's advised on the Odoo help forum. Updating stock.move.line records outside the Python context can be a bit dangerous, and Odoo doesn't always update those 0-quantity lines when you click Check Availability.

(I also include some instructions on creating the server action.)