[jira] [Commented] (OFBIZ-7168) Run MRP shows all type of facilities

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-7168) Run MRP shows all type of facilities

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-7168?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15324137#comment-15324137 ]

Pierre Smits commented on OFBIZ-7168:
-------------------------------------

This patch means that we are introducing a hard coded filtering aspect.
Shouldn't this a configurable aspect. Meaning, we could define this a value (see the example below, but don't fret over the value in systemPropertyId as it is just an example) stored in the SystemProperty entity - with a fall back to a component specific .properties file, and retrieve it from there as the default.

{code}
<SystemProperty description="Default Facility Type(s) for Facility selection in MRP planning" systemPropertyId="mpr.facilityType.default" systemPropertyValue="WAREHOUSE, MFG-FACILITY" systemResourceId="manufacturing"/>
{code}

> Run MRP shows all type of facilities
> ------------------------------------
>
>                 Key: OFBIZ-7168
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7168
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: manufacturing
>            Reporter: Sanjay Yadav
>            Assignee: Mridul Pathak
>         Attachments: OFBIZ-7168_trunk.patch
>
>
> When we go to Run MRP screen, all types of facilities (including facilities with no type) are listed here in the Facility drop down.
> I think, It's better to list only WAREHOUSE or some specific type of facilities. Basically, there is no meaning of showing facilities which don't have proper type or type is not available for e.g. CALL_CENTER, FLOOR etc.
> Let me know if anyone have any thoughts on this. Thanks!



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)