svn commit: r1532366 - in /ofbiz/trunk: applications/manufacturing/doc
Locked
svn commit: r1532366 - in /ofbiz/trunk: applications/manufacturing/doc
 
|
Author: jleroux
Date: Tue Oct 15 14:51:57 2013 New Revision: 1532366 URL: http://svn.apache.org/r1532366 Log: A slightly modified patch from Gareth Carter for "Fixes for some xml/groovy files" https://issues.apache.org/jira/browse/OFBIZ-5349 Validated all xml files and compiled all groovy files and found a few errors. Look at patch for details jleroux: For webapptests.xml, I prefered to revert it from r959261 (I saw an issue in my xml editor, did not check more) Modified: ofbiz/trunk/applications/manufacturing/documents/manufacturing.xml ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsComponentsByFeature.groovy ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsInfoAndOrder.groovy ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsAndOrder.groovy ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsByFeature.groovy ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsStacks.groovy ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PackageContentsAndOrder.groovy ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/actions/imagemanagement/SetDefaultImage.groovy ofbiz/trunk/framework/webapp/testdef/webapptests.xml ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/forum/RespondPermAndPrep.groovy Modified: ofbiz/trunk/applications/manufacturing/documents/manufacturing.xml URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/documents/manufacturing.xml?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/manufacturing/documents/manufacturing.xml (original) +++ ofbiz/trunk/applications/manufacturing/documents/manufacturing.xml Tue Oct 15 14:51:57 2013 @@ -35,5 +35,4 @@ <xi:include href="../data/helpdata/Help_MFG_FindInventoryEventPlan.xml"/> <xi:include href="../data/helpdata/Help_MFG_WorkWithShipmentPlans.xml"/> <xi:include href="../data/helpdata/Help_MFG_ManufacturingReports.xml"/> - < </chapter> \ No newline at end of file Modified: ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsComponentsByFeature.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsComponentsByFeature.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsComponentsByFeature.groovy (original) +++ ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsComponentsByFeature.groovy Tue Oct 15 14:51:57 2013 @@ -47,7 +47,7 @@ if (allProductionRuns) { productionRunTask = EntityUtil.getFirst(productionRunTasks); if (!productionRunTask) { // the production run doesn't include the given task, skip it - continue; + return; } // select the task's components, if any @@ -57,7 +57,7 @@ if (allProductionRuns) { if (productCategoryIdPar) { if (!isProductInCategory(delegator, productionRunComponent.productId, productCategoryIdPar)) { // the production run's product is not a member of the given category, skip it - continue; + return; } } productionRunProduct = delegator.findOne("Product", [productId : productionRunComponent.productId], false); Modified: ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsInfoAndOrder.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsInfoAndOrder.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsInfoAndOrder.groovy (original) +++ ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsInfoAndOrder.groovy Tue Oct 15 14:51:57 2013 @@ -38,7 +38,7 @@ if (allProductionRuns) { if (productCategoryIdPar) { if (!isProductInCategory(delegator, productionRun.productId, productCategoryIdPar)) { // the production run's product is not a member of the given category, skip it - continue; + return; } } productionRunProduct = delegator.findOne("Product", [productId : productionRun.productId], false); @@ -53,7 +53,7 @@ if (allProductionRuns) { productionRunTask = EntityUtil.getFirst(productionRunTasks); if (!productionRunTask) { // the production run doesn't include the given task, skip it - continue; + return; } productionRunMap = [productionRun : productionRun, Modified: ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsAndOrder.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsAndOrder.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsAndOrder.groovy (original) +++ ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsAndOrder.groovy Tue Oct 15 14:51:57 2013 @@ -38,7 +38,7 @@ if (allProductionRuns) { if (productCategoryIdPar) { if (!isProductInCategory(delegator, productionRun.productId, productCategoryIdPar)) { // the production run's product is not a member of the given category, skip it - continue; + return; } } productionRunProduct = delegator.findOne("Product", [productId : productionRun.productId], false); Modified: ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsByFeature.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsByFeature.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsByFeature.groovy (original) +++ ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsByFeature.groovy Tue Oct 15 14:51:57 2013 @@ -44,7 +44,7 @@ if (allProductionRuns) { if (productCategoryIdPar) { if (!isProductInCategory(delegator, productionRun.productId, productCategoryIdPar)) { // the production run's product is not a member of the given category, skip it - continue; + return; } } productionRunProduct = delegator.findOne("Product", [productId : productionRun.productId], false); @@ -70,7 +70,7 @@ if (allProductionRuns) { productionRunTask = EntityUtil.getFirst(productionRunTasks); if (!productionRunTask) { // the production run doesn't include the given task, skip it - continue; + return; } productionRunMap = [productionRun : productionRun, Modified: ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsStacks.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsStacks.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsStacks.groovy (original) +++ ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PRunsProductsStacks.groovy Tue Oct 15 14:51:57 2013 @@ -42,7 +42,7 @@ if (allProductionRuns) { if (productCategoryIdPar) { if (!isProductInCategory(delegator, productionRun.productId, productCategoryIdPar)) { // the production run's product is not a member of the given category, skip it - continue; + return; } } productionRunProduct = delegator.findOne("Product", [productId : productionRun.productId], false); @@ -57,7 +57,7 @@ if (allProductionRuns) { productionRunTask = EntityUtil.getFirst(productionRunTasks); if (!productionRunTask) { // the production run doesn't include the given task, skip it - continue; + return; } } Modified: ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PackageContentsAndOrder.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PackageContentsAndOrder.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PackageContentsAndOrder.groovy (original) +++ ofbiz/trunk/applications/manufacturing/webapp/manufacturing/WEB-INF/actions/reports/PackageContentsAndOrder.groovy Tue Oct 15 14:51:57 2013 @@ -45,7 +45,7 @@ if (packageContents) { if (productCategoryIdPar) { if (!isProductInCategory(delegator, product.productId, productCategoryIdPar)) { // the production run's product is not a member of the given category, skip it - continue; + return; } } Modified: ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/actions/imagemanagement/SetDefaultImage.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/actions/imagemanagement/SetDefaultImage.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/actions/imagemanagement/SetDefaultImage.groovy (original) +++ ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/actions/imagemanagement/SetDefaultImage.groovy Tue Oct 15 14:51:57 2013 @@ -109,7 +109,7 @@ if (fileType) { if (checkPathFile.equals(productContentList.get(0).drObjectInfo)) { BufferedImage bufImg = ImageIO.read(new File(imageManagementPath + "/" + productId + "/" + dataResourceName)); } else { - BufferedImage bufImg = ImageIO.read(new File(productContentList.get(0).drObjectInfo))); + BufferedImage bufImg = ImageIO.read(new File(productContentList.get(0).drObjectInfo)); } ImageIO.write((RenderedImage) bufImg, "jpg", new File(imageManagementPath + "/" + productId + "/" + defaultFileName)); Modified: ofbiz/trunk/framework/webapp/testdef/webapptests.xml URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webapp/testdef/webapptests.xml?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/framework/webapp/testdef/webapptests.xml (original) +++ ofbiz/trunk/framework/webapp/testdef/webapptests.xml Tue Oct 15 14:51:57 2013 @@ -17,8 +17,11 @@ under the License. --> -<!--<test-suite suite-name="webapptests"--> -<!-- xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"--> -<!-- xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/test-suite.xsd">--> -<!----> -<!--</test-suite>--> +<test-suite suite-name="webapptests" + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" + xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/test-suite.xsd"> + + <test-case case-name="webapp-tests"> + <junit-test-suite class-name="org.ofbiz.webapp.test.XmlRpcTests"/> + </test-case> +</test-suite> Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/forum/RespondPermAndPrep.groovy URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/forum/RespondPermAndPrep.groovy?rev=1532366&r1=1532365&r2=1532366&view=diff ============================================================================== --- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/forum/RespondPermAndPrep.groovy (original) +++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/actions/forum/RespondPermAndPrep.groovy Tue Oct 15 14:51:57 2013 @@ -95,7 +95,7 @@ if (!"granted".equals(permissionStatus)) request.setAttribute("permissionErrorMsg", errorMessage); context.permissionErrorMsg = errorMessage; context.hasPermission = false; - request.setAttribute("hasPermission", false; + request.setAttribute("hasPermission", false); request.setAttribute("permissionStatus", ""); return; } else { |
«
Return to OFBiz - Commits
|
1 view|%1 views
| Free forum by Nabble | Edit this page |