-
Notifications
You must be signed in to change notification settings - Fork 103
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
6504: fix(stock): improve Stock Value Report r=jmcameron a=jniles This PR fixes various components of the stock value report cited in issue #6150. Notably, the query now uses the `stock_movement_status` table to pull in the quantities in stock for a given depot and loads the `stock_value` table to get the value of the items in stock. This speeds the query up significantly. Miscellaneous changes: - The report now defaults to the enterprise currency. - The majority of the values are now cached, except for the date that defaults to today. - Performance improvements as cited above. - I've removed the coloration entirely. Closes #6150. Co-authored-by: Jonathan Niles <jonathanwniles@gmail.com>
- Loading branch information
Showing
4 changed files
with
97 additions
and
142 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters