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

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

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

sichen
Author: sichen
Date: Wed Feb 14 17:54:44 2007
New Revision: 507773

URL: http://svn.apache.org/viewvc?view=rev&rev=507773
Log:
reverting back to the older version of the facility inventory screen so that it will work correctly when there are no inventory items yet for products which have product facility minimum stock settings

Modified:
    ofbiz/trunk/applications/product/webapp/facility/WEB-INF/actions/facility/countFacilityInventoryByProduct.bsh
    ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml

Modified: ofbiz/trunk/applications/product/webapp/facility/WEB-INF/actions/facility/countFacilityInventoryByProduct.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/facility/WEB-INF/actions/facility/countFacilityInventoryByProduct.bsh?view=diff&rev=507773&r1=507772&r2=507773
==============================================================================
--- ofbiz/trunk/applications/product/webapp/facility/WEB-INF/actions/facility/countFacilityInventoryByProduct.bsh (original)
+++ ofbiz/trunk/applications/product/webapp/facility/WEB-INF/actions/facility/countFacilityInventoryByProduct.bsh Wed Feb 14 17:54:44 2007
@@ -17,6 +17,11 @@
  * under the License.
  */
 
+// 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.
+
 // This script counts the inventory in the facility using a complex and
 // pretty efficient dynamic view entity.
 // However, since the quantities are not computed using the inventory

Modified: ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml?view=diff&rev=507773&r1=507772&r2=507773
==============================================================================
--- ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml (original)
+++ ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml Wed Feb 14 17:54:44 2007
@@ -197,8 +197,9 @@
                 <set field="productsSoldThruTimestamp" from-field="parameters.productsSoldThruTimestamp"/>
                 <set field="internalName" from-field="parameters.internalName"/>
                 <set field="productId" from-field="parameters.productId"/>
-  
-                <script location="component://product/webapp/facility/WEB-INF/actions/facility/countFacilityInventoryByProduct.bsh"/>
+
+                <!-- do not use the countFacilityInventoryByProduct.bsh script unless the issues mentioned in it are correctly addressed -->
+                <script location="component://product/webapp/facility/WEB-INF/actions/facility/ViewFacilityInventoryByProduct.bsh"/>
             </actions>
             <widgets>
                 <decorator-screen name="CommonFacilityDecorator" location="${parameters.commonFacilityDecoratorLocation}">