From 4f4c6a404b29023c874a9bb06965ff275dc6b0df Mon Sep 17 00:00:00 2001
From: Susana Montes Pedreira <smontes@wirelessgalicia.com>
Date: Tue, 15 Feb 2011 13:28:00 +0100
Subject: [PATCH] [Bug #850] fix bug

It generates the codes of the materials and then it
saves each material category.

FEA : ItEr70S04BugFixing
---
 .../navalplanner/web/materials/MaterialsModel.java   | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/materials/MaterialsModel.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/materials/MaterialsModel.java
index 1a9ce0be6..96d4cfe8b 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/materials/MaterialsModel.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/materials/MaterialsModel.java
@@ -244,16 +244,18 @@ public class MaterialsModel extends IntegrationEntityModel implements
         final List<MaterialCategory> categories = materialCategories.asList();
         checkNoCodeRepeatedAtNewMaterials(categories);
         Integer numberOfDigits = getNumberOfDigitsCode();
-        for (MaterialCategory each: categories) {
-            generateMaterialCodesIfIsNecessary(each, numberOfDigits);
+        generateMaterialCodesIfIsNecessary(categories, numberOfDigits);
+        for (MaterialCategory each : categories) {
             categoryDAO.save(each);
         }
     }
 
-    private void generateMaterialCodesIfIsNecessary(MaterialCategory category,
+    private void generateMaterialCodesIfIsNecessary(List<MaterialCategory> categories,
             Integer numberOfDigits) {
-        if (category.isCodeAutogenerated()) {
-            category.generateMaterialCodes(numberOfDigits);
+        for (MaterialCategory category: categories) {
+            if (category.isCodeAutogenerated()) {
+                category.generateMaterialCodes(numberOfDigits);
+            }
         }
     }