]> git.mxchange.org Git - pizzaservice-ejb.git/commitdiff
Fixed: (please cherry-pick)
authorRoland Häder <roland@mxchange.org>
Wed, 10 Aug 2016 12:53:58 +0000 (14:53 +0200)
committerRoland Haeder <roland@mxchange.org>
Tue, 16 Aug 2016 17:49:29 +0000 (19:49 +0200)
- em.remove() needs a managed entity, not a one passed from e.g. a controller
- still this method may throw an exception, e.g. when an other entity (table) references this entity
- but since deleteCellphoneNumber() *should* delete cell phone numbers, it doesn't care if the entity is still referenced.
- this needs to be taken care of the controller and not the EJB (job-executor)

src/java/org/mxchange/jphone/phonenumbers/phone/PizzaAdminPhoneSessionBean.java

index dcb52b76aaeb0ca39657e50aca1aea0ebd6a408f..46f8f7abadc82e84f4f0682c1caf0d9904360746 100644 (file)
@@ -67,8 +67,11 @@ public class PizzaAdminPhoneSessionBean extends BasePizzaDatabaseBean implements
                        throw new NullPointerException(MessageFormat.format("cellphoneNumber.phoneNumber={0} is not valid.", cellPhoneNumber.getPhoneNumber())); //NOI18N
                }
 
+               // Merge it to get a managed entity back
+               DialableCellphoneNumber managedNumber = this.getEntityManager().getReference(cellPhoneNumber.getClass(), cellPhoneNumber.getPhoneId());
+
                // Remove it from database
-               this.getEntityManager().remove(cellPhoneNumber);
+               this.getEntityManager().remove(managedNumber);
 
                // Trace message
                this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.deleteCellphoneData: EXIT!", this.getClass().getSimpleName()));