Re: svn commit: r507773 - in /ofbiz/trunk/applications/product: webapp/facility/WEB-INF/actions/facility/countFacilityInventoryByProduct.bsh widget/facility/FacilityScreens.xml

[Jacopo Cappellato]

Si,

about your comment: I don't think that the missing support for marketing
packages should prevent the usage of the script in production.
Instead I think that the *fake* inventory units shown for the mkt pkg
could be an issue because the items are not really in inventory (just
their components are there), and most of all because the units of the
components are also shown in the reports and this could induce the user
think there are components AND mkt pkg (i.e. more inventory than the
real one).
On the other hand, your second point is really worth considering (the
issue with a product with no InventoryItems) and so for now I think that
using the original script makes sense.

Jacopo

>  
> +// This script can be used for testing right now but it should not be used for production because
> +// it does not work for marketing packages and more importantly, if there is a new product without any
> +// inventory items, it will not show up on the inventory report even if it had a ProductFacility record.
> +// These problems need to be addressed before this can be used in production.
> +

[Si Chen-2]

Jacopo Cappellato wrote:

> Si,
>
> about your comment: I don't think that the missing support for
> marketing packages should prevent the usage of the script in production.
> Instead I think that the *fake* inventory units shown for the mkt pkg
> could be an issue because the items are not really in inventory (just
> their components are there), and most of all because the units of the
> components are also shown in the reports and this could induce the
> user think there are components AND mkt pkg (i.e. more inventory than
> the real one).
> On the other hand, your second point is really worth considering (the
> issue with a product with no InventoryItems) and so for now I think
> that using the original script makes sense.
>
> Jacopo
>
>>  
>> +// This script can be used for testing right now but it should not
>> be used for production because
>> +// it does not work for marketing packages and more importantly, if
>> there is a new product without any
>> +// inventory items, it will not show up on the inventory report even
>> if it had a ProductFacility record.
>> +// These problems need to be addressed before this can be used in
>> production.
>> +
>
>

Jacopo,

Also this script probably may not be handling serialized inventory
correctly.  I did not check it yesterday but it needs to be checked.

In general I am concerned that this script means the re-writing the
business logic of the inventory services here in the BSH.


Si

[Jacopo Cappellato]

Si Chen wrote:
> Jacopo,
>
> Also this script probably may not be handling serialized inventory
> correctly.  I did not check it yesterday but it needs to be checked.
>
> In general I am concerned that this script means the re-writing the
> business logic of the inventory services here in the BSH.
>
> Si

yes, these are two good points too; however I think that sometimes it's
worth having to maintain some duplicated business logics if there are
serious performance issues (as it is happening here).

Jacopo

[Si Chen-2]

Jacopo Cappellato wrote:

> Si Chen wrote:
>> Jacopo,
>>
>> Also this script probably may not be handling serialized inventory
>> correctly.  I did not check it yesterday but it needs to be checked.
>>
>> In general I am concerned that this script means the re-writing the
>> business logic of the inventory services here in the BSH.
>>
>> Si
>
> yes, these are two good points too; however I think that sometimes
> it's worth having to maintain some duplicated business logics if there
> are serious performance issues (as it is happening here).
>
> Jacopo
>

For long term maintenance and to avoid bugs, we should really try to
keep all the business logic of inventory calculation in one place.  The
current services can be made a lot more efficient.  In reality, though,
on a real server the performance is not too bad even with them the way
they are.

Si