OCA / stock-logistics-reporting

Provides Odoo Logistics reporting modules
https://odoo-community.org/psc-teams/logistics-18
GNU Affero General Public License v3.0
66 stars 232 forks source link

[16.0][IMP]stock_report_quantity_by_location: return computed stock reports #302

Closed PicchiSeba closed 8 months ago

PicchiSeba commented 8 months ago

Returning the computed reports allows to use them for other purposes. Like printing a report directly from the stock.report.quantity.by.location.prepare form

rousseldenis commented 8 months ago

@PicchiSeba I'm wondering if this is really necessary as we have the wizard id, we can retrieve the data. A compute function should not return something (by essence)

PicchiSeba commented 8 months ago

@PicchiSeba I'm wondering if this is really necessary as we have the wizard id, we can retrieve the data. A compute function should not return something (by essence)

@rousseldenis That's a good point. Somehow I didn't think about it.

Thanks for your feedback, closing this PR