Product-only:

- let's use createManaged() where possible to avoid some duplicate code. It may
  look a bit to much encapsulation or to fine-granulated, the createManaged()
  method contains some validation on the entity instance and that is a good
  reason to have it encapsulated away.

Signed-off-by: Roland Häder <roland@mxchange.org>

diff --git a/src/java/org/mxchange/jproduct/model/category/PizzaAdminProductCategorySessionBean.java b/src/java/org/mxchange/jproduct/model/category/PizzaAdminProductCategorySessionBean.java
index 37b92aeb4004da6e987cbbd8c3bdd4a6b07878a6..93614cb75a63a538ac74766eeba4de37735226d8 100644 (file)
--- a/src/java/org/mxchange/jproduct/model/category/PizzaAdminProductCategorySessionBean.java
+++ b/src/java/org/mxchange/jproduct/model/category/PizzaAdminProductCategorySessionBean.java
@@ -130,13 +130,13 @@ public class PizzaAdminProductCategorySessionBean extends BasePizzaProductEnterp
                }
 
                // Update instance
-               final Category updatedCategory = this.mergeProductCategoryData(category);
+               final Category managedCategory = this.mergeProductCategoryData(category);
 
                // Trace message
-               this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.updateProductCategory: updatedCategory.categoryId={1} - EXIT!", this.getClass().getSimpleName(), updatedCategory.getCategoryId())); //NOI18N
+               this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.updateProductCategory: managedCategory.categoryId={1} - EXIT!", this.getClass().getSimpleName(), managedCategory.getCategoryId())); //NOI18N
 
                // Return it
-               return updatedCategory;
+               return managedCategory;
        }
 
        /**

diff --git a/src/java/org/mxchange/pizzaapplication/enterprise/product/BasePizzaProductEnterpriseBean.java b/src/java/org/mxchange/pizzaapplication/enterprise/product/BasePizzaProductEnterpriseBean.java
index f44dd4e5fe1072976223ac434a7353736a8169d3..59f79d2f7b7f4a16069e61f89331b24e0cc522fe 100644 (file)
--- a/src/java/org/mxchange/pizzaapplication/enterprise/product/BasePizzaProductEnterpriseBean.java
+++ b/src/java/org/mxchange/pizzaapplication/enterprise/product/BasePizzaProductEnterpriseBean.java
@@ -146,10 +146,7 @@ public abstract class BasePizzaProductEnterpriseBean extends BasePizzaEnterprise
                detachedProduct.setProductEntryUpdated(new Date());
 
                // Get product from it and find it
-               final Product foundProduct = this.getEntityManager().find(detachedProduct.getClass(), detachedProduct.getProductId());
-
-               // Should be found
-               assert (foundProduct instanceof Product) : MessageFormat.format("Product with id {0} not found, but should be.", detachedProduct.getProductId()); //NOI18N
+               final Product foundProduct = this.createManaged(detachedProduct);
 
                // Debug message
                this.getLoggerBeanLocal().logDebug(MessageFormat.format("mergeProductData: foundProduct.productId={0}", foundProduct.getProductId())); //NOI18N
@@ -194,10 +191,7 @@ public abstract class BasePizzaProductEnterpriseBean extends BasePizzaEnterprise
                detachedCategory.setCategoryEntryUpdated(new Date());
 
                // Get category from it and find it
-               final Category foundCategory = this.getEntityManager().find(detachedCategory.getClass(), detachedCategory.getCategoryId());
-
-               // Should be found
-               assert (foundCategory instanceof Category) : MessageFormat.format("Category with id {0} not found, but should be.", detachedCategory.getCategoryId()); //NOI18N
+               final Category foundCategory = this.createManaged(detachedCategory);
 
                // Debug message
                this.getLoggerBeanLocal().logDebug(MessageFormat.format("mergeCategoryData: foundCategory.categoryId={0}", foundCategory.getCategoryId())); //NOI18N