rejecting /1.0/kb/accounts/<account_id>/invoices?includeInvoiceComponents=true&unpaidInvoicesOnly=true

[tenchiro]

I think this is a bug. The log shows thiceComponents=true&unpaidInvoicesOnly=trueat the invoice components flag is set to true, but it seems the mere presence of the flag causes failure:

Kill Bill 500 on http://sekai.local:8030/1.0/kb/accounts/2556aa6f-5f32-4aab-a14c-943fb219828f/invoices?includeInvoiceComponents=true&unpaidInvoicesOnly=true: We don't support fetching unpaid invoices without invoice components

If the code is just testing the presence of the flag, sematically it seems backwards.

I am working around it by removing the parameter altogether (this seems fine) but I think it should accept it.

The docs do note this requirement, but do not say the flag must be omitted.

[karan bansal]

Hi, 

Thank you for pointing that out. I had a look into it. The code already handles passing only unpaidInvoicesOnly as true and takes includeInvoiceComponents as true automatically in this case. So the error message seems to be there to specifically catch and avoid the exact case when both the flags includeInvoiceComponents and unpaidInvoicesOnly are passed as true.

Regards

Karan