Whenever we get a chance we should try to add a comment explaining what these hardcoded '1's mean. They're also present in the build_invoice_lines