Inactive Storage Locations should be included in Storage Location export if inactive filter is on

[cielf]

Summary

Include inactive storage locations in the export if they are included in the current filter

Why?

Consistency.

Details

To see the issue,

sign in as [email protected]
click on Inventory, then "Storage Locations.
click the "include inactive storage locations"
click "Filter"
click "Export Storage Locations"
Note that the Inactive Storage Location shows in the main list, but does not show in the export

What should happen

The inactive storage location should appear in the export. (Yes, all the item levels will be 0. )

Criteria for completion

• Behaviour as described
• Tests to suppor the behaviour

[napster235]

@cielf I've noticed the same issue on Items & Inventory. Should it be addressed as well?

[cielf]

It should. Do you want to take it on, or shall I just add it to the queue? (Notes that there is an active PR that's touching the export for items atm)

[napster235]

Yes, I can make the change. It is just a view modification. Should I do it on the same PR?

[cielf]

Please make it a new PR.