From aeeb6bf88735b4b74280cc9c70254c3b7ccf17d2 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Roland=20H=C3=A4der?= Date: Tue, 14 Apr 2020 10:03:54 +0200 Subject: [PATCH] Please cherry-pick: - updated jphone-core.jar requires now some new getter/setter names like getMobileEntryUpdated(), setContactEntry() and many more MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Signed-off-by: Roland Häder --- .../BaseAddressbookEnterpriseBean.java | 63 +++++++++---------- .../AddressbookAdminContactSessionBean.java | 2 +- ...dressbookAdminContactPhoneSessionBean.java | 22 +++---- ...dressbookAdminBusinessDataSessionBean.java | 17 ++--- .../AddressbookAdminEmployeeSessionBean.java | 6 +- .../AddressbookAdminPhoneSessionBean.java | 44 ++++++------- .../user/AddressbookAdminUserSessionBean.java | 2 +- .../user/AddressbookUserSessionBean.java | 17 +++-- 8 files changed, 87 insertions(+), 86 deletions(-) diff --git a/src/java/org/mxchange/addressbook/enterprise/BaseAddressbookEnterpriseBean.java b/src/java/org/mxchange/addressbook/enterprise/BaseAddressbookEnterpriseBean.java index 931a436..59233a2 100644 --- a/src/java/org/mxchange/addressbook/enterprise/BaseAddressbookEnterpriseBean.java +++ b/src/java/org/mxchange/addressbook/enterprise/BaseAddressbookEnterpriseBean.java @@ -47,7 +47,6 @@ import org.mxchange.jphone.model.phonenumbers.landline.LandLineNumbers; import org.mxchange.jphone.model.phonenumbers.mobile.DialableMobileNumber; import org.mxchange.jphone.model.phonenumbers.mobile.MobileNumbers; import org.mxchange.jphone.model.phonenumbers.mobileprovider.MobileProvider; -import org.mxchange.jphone.utils.PhoneUtils; import org.mxchange.jusercore.model.user.User; import org.mxchange.jusercore.model.user.Users; @@ -529,12 +528,12 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { } // Is a mobile number instance set? - if ((mobileNumber instanceof DialableMobileNumber) && (mobileNumber.getPhoneId() == null)) { + if ((mobileNumber instanceof DialableMobileNumber) && (mobileNumber.getMobileId() == null)) { // Debug message this.getLoggerBeanLocal().logDebug("setAllPhoneEntriesCreated: Setting created timestamp for mobile number ..."); //NOI18N // Set updated timestamp - mobileNumber.setPhoneEntryCreated(new Date()); + mobileNumber.setMobileEntryCreated(new Date()); } // Trace message @@ -704,16 +703,16 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { if (null == mobileNumber) { // Throw NPE throw new NullPointerException("mobileNumber is null"); //NOI18N - } else if (fetchedNumber.getPhoneId() == null) { + } else if (fetchedNumber.getMobileId() == null) { // ..and again throw new NullPointerException("fetchedNumber.phoneId is null"); //NOI18N } // Debug message - this.getLoggerBeanLocal().logDebug(MessageFormat.format("getDetached: fetchedNumber.phoneId={0}", fetchedNumber.getPhoneId())); //NOI18N + this.getLoggerBeanLocal().logDebug(MessageFormat.format("getDetached: fetchedNumber.phoneId={0}", fetchedNumber.getMobileId())); //NOI18N // Init query instance - final DialableMobileNumber foundNumber = this.getEntityManager().find(fetchedNumber.getClass(), fetchedNumber.getPhoneId()); + final DialableMobileNumber foundNumber = this.getEntityManager().find(fetchedNumber.getClass(), fetchedNumber.getMobileId()); // Debug message this.getLoggerBeanLocal().logDebug(MessageFormat.format("getDetached: foundNumber={0}", foundNumber)); //NOI18N @@ -722,7 +721,7 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { DialableMobileNumber detachedNumber = null; // Is there a difference? - if (!PhoneUtils.isSameMobileNumber(mobileNumber, fetchedNumber)) { + if (!MobileNumbers.isSameMobileNumber(mobileNumber, fetchedNumber)) { // @TODO Copy all to foundNumber, then merge // Merge this entry @@ -770,7 +769,7 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { DialableLandLineNumber detachedNumber = null; // Is there a difference? - if (!PhoneUtils.isSameLandLineNumber(landLineNumber, fetchedNumber)) { + if (!LandLineNumbers.isSameLandLineNumber(landLineNumber, fetchedNumber)) { // @TODO Copy all to foundNumber, then merge // Merge this entry @@ -818,7 +817,7 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { DialableFaxNumber detachedNumber = null; // Is there a difference? - if (!PhoneUtils.isSameFaxNumber(faxNumber, fetchedNumber)) { + if (!FaxNumbers.isSameFaxNumber(faxNumber, fetchedNumber)) { // @TODO Copy all to foundNumber, then merge // Merge this entry @@ -856,7 +855,7 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { } // Set updated timestamp - detachedContact.setContactUpdated(new Date()); + detachedContact.setContactEntryUpdated(new Date()); // Get contact from it and find it final Contact foundContact = this.getEntityManager().find(detachedContact.getClass(), detachedContact.getContactId()); @@ -902,47 +901,47 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { } // Get all instances - final DialableMobileNumber mobile = detachedContact.getContactMobileNumber(); - final DialableLandLineNumber landLine = detachedContact.getContactLandLineNumber(); - final DialableFaxNumber fax = detachedContact.getContactFaxNumber(); + final DialableMobileNumber mobileNumber = detachedContact.getContactMobileNumber(); + final DialableLandLineNumber landLineNumber = detachedContact.getContactLandLineNumber(); + final DialableFaxNumber faxNumber = detachedContact.getContactFaxNumber(); // Is there a mobile instance set? - if (mobile instanceof DialableMobileNumber) { + if (mobileNumber instanceof DialableMobileNumber) { // Debug message - this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: mobile.phoneId={0} is being updated ...", mobile.getPhoneId())); //NOI18N + this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: mobile.phoneId={0} is being updated ...", mobileNumber.getMobileId())); //NOI18N // Then find it, too - final DialableMobileNumber foundMobile = this.getEntityManager().find(mobile.getClass(), mobile.getPhoneId()); + final DialableMobileNumber foundMobile = this.getEntityManager().find(mobileNumber.getClass(), mobileNumber.getMobileId()); // Should be there - assert (foundMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", foundMobile.getPhoneId()); //NOI18N + assert (foundMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", foundMobile.getMobileId()); //NOI18N // Copy all - MobileNumbers.copyAll(detachedContact.getContactMobileNumber(), foundMobile); + MobileNumbers.copyMobileNumber(detachedContact.getContactMobileNumber(), foundMobile); // Then merge it, too final DialableMobileNumber managedMobile = this.getEntityManager().merge(foundMobile); // Should be there - assert (managedMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", managedMobile.getPhoneId()); //NOI18N + assert (managedMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", managedMobile.getMobileId()); //NOI18N // Set it back detachedContact.setContactMobileNumber(managedMobile); } // Is there a fax instance set? - if (fax instanceof DialableFaxNumber) { + if (faxNumber instanceof DialableFaxNumber) { // Debug message - this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: fax.phoneId={0} is being updated ...", fax.getPhoneId())); //NOI18N + this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: fax.phoneId={0} is being updated ...", faxNumber.getPhoneId())); //NOI18N // Then find it, too - final DialableFaxNumber foundFax = this.getEntityManager().find(fax.getClass(), fax.getPhoneId()); + final DialableFaxNumber foundFax = this.getEntityManager().find(faxNumber.getClass(), faxNumber.getPhoneId()); // Should be there assert (foundFax instanceof DialableFaxNumber) : MessageFormat.format("Fax number with id {0} not found but should be.", foundFax.getPhoneId()); //NOI18N // Copy all - FaxNumbers.copyAll(detachedContact.getContactFaxNumber(), foundFax); + FaxNumbers.copyFaxNumber(detachedContact.getContactFaxNumber(), foundFax); // Then merge it, too final DialableFaxNumber managedFax = this.getEntityManager().merge(foundFax); @@ -955,18 +954,18 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { } // Is there a fax instance set? - if (landLine instanceof DialableLandLineNumber) { + if (landLineNumber instanceof DialableLandLineNumber) { // Debug message - this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: landLine.phoneId={0} is being updated ...", landLine.getPhoneId())); //NOI18N + this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: landLine.phoneId={0} is being updated ...", landLineNumber.getPhoneId())); //NOI18N // Then find it, too - final DialableLandLineNumber foundLandLine = this.getEntityManager().find(landLine.getClass(), landLine.getPhoneId()); + final DialableLandLineNumber foundLandLine = this.getEntityManager().find(landLineNumber.getClass(), landLineNumber.getPhoneId()); // Should be there assert (foundLandLine instanceof DialableLandLineNumber) : MessageFormat.format("Land line number with id {0} not found but should be.", foundLandLine.getPhoneId()); //NOI18N // Copy all - LandLineNumbers.copyAll(detachedContact.getContactLandLineNumber(), foundLandLine); + LandLineNumbers.copyLandLineNumber(detachedContact.getContactLandLineNumber(), foundLandLine); // Then merge it, too final DialableLandLineNumber managedLandLine = this.getEntityManager().merge(foundLandLine); @@ -1118,7 +1117,7 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { this.getLoggerBeanLocal().logDebug(MessageFormat.format("setAllContactPhoneEntries: other.contactMobileNumber={0}", other.getContactMobileNumber())); //NOI18N // Is other mobile not set? - if ((other.getContactMobileNumber() == null) || (PhoneUtils.isSameMobileNumber(contact.getContactMobileNumber(), other.getContactMobileNumber()))) { + if ((other.getContactMobileNumber() == null) || (MobileNumbers.isSameMobileNumber(contact.getContactMobileNumber(), other.getContactMobileNumber()))) { // Debug message this.getLoggerBeanLocal().logDebug("setAllContactPhoneEntries: Copying mobile entry ..."); //NOI18N @@ -1136,7 +1135,7 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { this.getLoggerBeanLocal().logDebug(MessageFormat.format("setAllContactPhoneEntries: other.contactLandLineNumber={0}", other.getContactLandLineNumber())); //NOI18N // Is other mobile not set? - if ((other.getContactLandLineNumber() == null) || (PhoneUtils.isSameLandLineNumber(contact.getContactLandLineNumber(), other.getContactLandLineNumber()))) { + if ((other.getContactLandLineNumber() == null) || (LandLineNumbers.isSameLandLineNumber(contact.getContactLandLineNumber(), other.getContactLandLineNumber()))) { // Debug message this.getLoggerBeanLocal().logDebug("setAllContactPhoneEntries: Copying land-line entry ..."); //NOI18N @@ -1154,7 +1153,7 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { this.getLoggerBeanLocal().logDebug(MessageFormat.format("setAllContactPhoneEntries: other.contactFaxNumber={0}", other.getContactFaxNumber())); //NOI18N // Is other mobile not set? - if ((other.getContactFaxNumber() == null) || (PhoneUtils.isSameFaxNumber(contact.getContactFaxNumber(), other.getContactFaxNumber()))) { + if ((other.getContactFaxNumber() == null) || (FaxNumbers.isSameFaxNumber(contact.getContactFaxNumber(), other.getContactFaxNumber()))) { // Debug message this.getLoggerBeanLocal().logDebug("setAllContactPhoneEntries: Copying fax entry ..."); //NOI18N @@ -1235,12 +1234,12 @@ public abstract class BaseAddressbookEnterpriseBean extends BaseEnterpriseBean { } // Is a mobile number instance set? - if ((mobileNumber instanceof DialableMobileNumber) && (mobileNumber.getPhoneId() instanceof Long) && (mobileNumber.getPhoneId() > 0)) { + if ((mobileNumber instanceof DialableMobileNumber) && (mobileNumber.getMobileId() instanceof Long) && (mobileNumber.getMobileId() > 0)) { // Debug message this.getLoggerBeanLocal().logDebug("setAllContactPhoneEntriesUpdated: Setting updated timestamp for mobile number ..."); //NOI18N // Set updated timestamp - mobileNumber.setPhoneEntryUpdated(new Date()); + mobileNumber.setMobileEntryUpdated(new Date()); } // Trace message diff --git a/src/java/org/mxchange/jcontacts/model/contact/AddressbookAdminContactSessionBean.java b/src/java/org/mxchange/jcontacts/model/contact/AddressbookAdminContactSessionBean.java index adc4dd0..247e541 100644 --- a/src/java/org/mxchange/jcontacts/model/contact/AddressbookAdminContactSessionBean.java +++ b/src/java/org/mxchange/jcontacts/model/contact/AddressbookAdminContactSessionBean.java @@ -69,7 +69,7 @@ public class AddressbookAdminContactSessionBean extends BaseAddressbookEnterpris } // Set created timestamp - contact.setContactCreated(new Date()); + contact.setContactEntryCreated(new Date()); // Set all created timestamps, if instance is there this.setAllPhoneEntriesCreated(contact); diff --git a/src/java/org/mxchange/jcontacts/model/phone/AddressbookAdminContactPhoneSessionBean.java b/src/java/org/mxchange/jcontacts/model/phone/AddressbookAdminContactPhoneSessionBean.java index 5aa2b83..457b433 100644 --- a/src/java/org/mxchange/jcontacts/model/phone/AddressbookAdminContactPhoneSessionBean.java +++ b/src/java/org/mxchange/jcontacts/model/phone/AddressbookAdminContactPhoneSessionBean.java @@ -192,12 +192,12 @@ public class AddressbookAdminContactPhoneSessionBean extends BaseAddressbookEnte } else if (null == mobileNumber) { // Throw NPE throw new NullPointerException("mobileNumber is null"); //NOI18N - } else if (mobileNumber.getPhoneId() == null) { + } else if (mobileNumber.getMobileId() == null) { // Throw it again throw new NullPointerException("mobileNumber.phoneId is null"); //NOI18N - } else if (mobileNumber.getPhoneId() < 1) { + } else if (mobileNumber.getMobileId() < 1) { // Invalid id - throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getPhoneId())); //NOI18N + throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getMobileId())); //NOI18N } else if (mobileNumber.getMobileProvider() == null) { // Throw NPE again throw new NullPointerException("mobileNumber.mobileProvider is null"); //NOI18N @@ -357,9 +357,9 @@ public class AddressbookAdminContactPhoneSessionBean extends BaseAddressbookEnte } else if (null == mobileNumber) { // Throw NPE throw new NullPointerException("mobileNumber is null"); //NOI18N - } else if (mobileNumber.getPhoneId() instanceof Long) { + } else if (mobileNumber.getMobileId() instanceof Long) { // Throw it again - throw new IllegalStateException(MessageFormat.format("mobileNumber.phoneId={0} is not null", mobileNumber.getPhoneId())); //NOI18N + throw new IllegalStateException(MessageFormat.format("mobileNumber.phoneId={0} is not null", mobileNumber.getMobileId())); //NOI18N } else if (mobileNumber.getMobileProvider() == null) { // Throw NPE again throw new NullPointerException("mobileNumber.mobileProvider is null"); //NOI18N @@ -372,7 +372,7 @@ public class AddressbookAdminContactPhoneSessionBean extends BaseAddressbookEnte } // Set created instance - mobileNumber.setPhoneEntryCreated(new Date()); + mobileNumber.setMobileEntryCreated(new Date()); // Set mobile number in contact contact.setContactMobileNumber(mobileNumber); @@ -489,15 +489,15 @@ public class AddressbookAdminContactPhoneSessionBean extends BaseAddressbookEnte } else if (contact.getContactMobileNumber() == null) { // Not set cell phone instance throw new PhoneNumberNotLinkedException(mobileNumber); - } else if (contact.getContactMobileNumber().getPhoneId() == null) { + } else if (contact.getContactMobileNumber().getMobileId() == null) { // Throw NPE again throw new NullPointerException("contact.contactMobileNumber.phoneId is null"); //NOI18N - } else if (contact.getContactMobileNumber().getPhoneId() < 1) { + } else if (contact.getContactMobileNumber().getMobileId() < 1) { // Invalid id number - throw new IllegalArgumentException(MessageFormat.format("contact.contactMobileNumber.phoneId={0} is invalid.", contact.getContactMobileNumber().getPhoneId())); //NOI18N - } else if (!Objects.equals(mobileNumber.getPhoneId(), contact.getContactMobileNumber().getPhoneId())) { + throw new IllegalArgumentException(MessageFormat.format("contact.contactMobileNumber.phoneId={0} is invalid.", contact.getContactMobileNumber().getMobileId())); //NOI18N + } else if (!Objects.equals(mobileNumber.getMobileId(), contact.getContactMobileNumber().getMobileId())) { // Not same object - throw new IllegalArgumentException(MessageFormat.format("contact.contactMobileNumber.phoneId={0} and mobileNumber.phoneId={1} are not the same.", contact.getContactMobileNumber().getPhoneId(), mobileNumber.getPhoneId())); //NOI18N + throw new IllegalArgumentException(MessageFormat.format("contact.contactMobileNumber.phoneId={0} and mobileNumber.phoneId={1} are not the same.", contact.getContactMobileNumber().getMobileId(), mobileNumber.getMobileId())); //NOI18N } // Remove it from contact diff --git a/src/java/org/mxchange/jcontactsbusiness/model/basicdata/AddressbookAdminBusinessDataSessionBean.java b/src/java/org/mxchange/jcontactsbusiness/model/basicdata/AddressbookAdminBusinessDataSessionBean.java index a391b95..0dc78c3 100644 --- a/src/java/org/mxchange/jcontactsbusiness/model/basicdata/AddressbookAdminBusinessDataSessionBean.java +++ b/src/java/org/mxchange/jcontactsbusiness/model/basicdata/AddressbookAdminBusinessDataSessionBean.java @@ -18,7 +18,6 @@ package org.mxchange.jcontactsbusiness.model.basicdata; import java.text.MessageFormat; import java.util.Date; -import java.util.List; import java.util.Objects; import javax.ejb.EJB; import javax.ejb.Stateless; @@ -85,7 +84,7 @@ public class AddressbookAdminBusinessDataSessionBean extends BaseAddressbookEnte } // Now add current date - basicData.setCompanyCreated(new Date()); + basicData.setCompanyEntryCreated(new Date()); // Is there a owner set? if (basicData.getCompanyUserOwner() instanceof User) { @@ -144,16 +143,13 @@ public class AddressbookAdminBusinessDataSessionBean extends BaseAddressbookEnte * @return Whether same company name has been used */ private boolean isSameCompanyNameAdded (final BasicData basicData) { - // Get all available entries - final List list = this.businessDataBean.allBusinessBasicData(); - // Default is not found boolean isFound = false; // Then check each entry - for (final BasicData entry : list) { + for (final BasicData currentBasicData : this.businessDataBean.allBusinessBasicData()) { // Is the company name matching? - if (Objects.equals(entry.getCompanyName(), basicData.getCompanyName())) { + if (Objects.equals(currentBasicData.getCompanyName(), basicData.getCompanyName())) { // Found match isFound = true; } @@ -171,16 +167,13 @@ public class AddressbookAdminBusinessDataSessionBean extends BaseAddressbookEnte * @return Whether same company short name has been used */ private boolean isSameCompanyShortNameAdded (final BasicData basicData) { - // Get all available entries - final List list = this.businessDataBean.allBusinessBasicData(); - // Default is not found boolean isFound = false; // Then check each entry - for (final BasicData entry : list) { + for (final BasicData currentBasicData : this.businessDataBean.allBusinessBasicData()) { // Is the company name matching? - if (Objects.equals(entry.getCompanyShortName(), basicData.getCompanyShortName())) { + if (Objects.equals(currentBasicData.getCompanyShortName(), basicData.getCompanyShortName())) { // Found match isFound = true; } diff --git a/src/java/org/mxchange/jcontactsbusiness/model/employee/AddressbookAdminEmployeeSessionBean.java b/src/java/org/mxchange/jcontactsbusiness/model/employee/AddressbookAdminEmployeeSessionBean.java index a61e25f..96fb26d 100644 --- a/src/java/org/mxchange/jcontactsbusiness/model/employee/AddressbookAdminEmployeeSessionBean.java +++ b/src/java/org/mxchange/jcontactsbusiness/model/employee/AddressbookAdminEmployeeSessionBean.java @@ -111,9 +111,9 @@ public class AddressbookAdminEmployeeSessionBean extends BaseAddressbookEnterpri } else if ((employee.getEmployeeNumber() != null) && (employee.getEmployeeNumber().isEmpty())) { // Throw it again throw new IllegalArgumentException("employee.employeeNumber is empty"); //NOI18N - } else if ((employee.getEmployeeMobileNumber() != null) && (employee.getEmployeeMobileNumber().getPhoneId() != null)) { + } else if ((employee.getEmployeeMobileNumber() != null) && (employee.getEmployeeMobileNumber().getMobileId() != null)) { // Throw once more - throw new IllegalArgumentException(MessageFormat.format("employee.employeeMobileNumber.phoneId={0} is unexpected.", employee.getEmployeeMobileNumber().getPhoneId())); //NOI18N + throw new IllegalArgumentException(MessageFormat.format("employee.employeeMobileNumber.phoneId={0} is unexpected.", employee.getEmployeeMobileNumber().getMobileId())); //NOI18N } else if ((employee.getEmployeePersonalData() != null) && (employee.getEmployeePersonalData().getContactId() == null)) { // Throw NPE throw new NullPointerException("employee.employeePersonalData.contactId is null"); //NOI18N @@ -197,7 +197,7 @@ public class AddressbookAdminEmployeeSessionBean extends BaseAddressbookEnterpri // Is mobile number set? if (employee.getEmployeeMobileNumber() instanceof DialableMobileNumber) { // Yes, then set created there as well - employee.getEmployeeMobileNumber().setPhoneEntryCreated(new Date()); + employee.getEmployeeMobileNumber().setMobileEntryCreated(new Date()); // Make mobile provider managed final MobileProvider mobileProvider = this.createManaged(employee.getEmployeeMobileNumber().getMobileProvider()); diff --git a/src/java/org/mxchange/jphone/model/phonenumbers/phone/AddressbookAdminPhoneSessionBean.java b/src/java/org/mxchange/jphone/model/phonenumbers/phone/AddressbookAdminPhoneSessionBean.java index 2d8ac9f..b27dc98 100644 --- a/src/java/org/mxchange/jphone/model/phonenumbers/phone/AddressbookAdminPhoneSessionBean.java +++ b/src/java/org/mxchange/jphone/model/phonenumbers/phone/AddressbookAdminPhoneSessionBean.java @@ -153,12 +153,12 @@ public class AddressbookAdminPhoneSessionBean extends BaseAddressbookEnterpriseB if (null == mobileNumber) { // Not set, throw NPE throw new NullPointerException("mobileNumber is null"); //NOI18N - } else if (mobileNumber.getPhoneId() == null) { + } else if (mobileNumber.getMobileId() == null) { // Throw NPE again throw new NullPointerException("mobileNumber.phoneId is null"); //NOI18N - } else if (mobileNumber.getPhoneId() < 1) { + } else if (mobileNumber.getMobileId() < 1) { // Invalid number - throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getPhoneId())); //NOI18N + throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getMobileId())); //NOI18N } else if (mobileNumber.getMobileProvider() == null) { // Throw NPE throw new NullPointerException("mobileNumber.cellphoneProvider is null"); //NOI18N @@ -168,16 +168,16 @@ public class AddressbookAdminPhoneSessionBean extends BaseAddressbookEnterpriseB } else if (mobileNumber.getMobileProvider().getProviderId() < 1) { // Id not valid throw new IllegalArgumentException(MessageFormat.format("mobileNumber.cellphoneProvider.providerId={0} is not valid.", mobileNumber.getMobileProvider().getProviderId())); //NOI18N - } else if (mobileNumber.getPhoneNumber() == null) { + } else if (mobileNumber.getMobileNumber() == null) { // Throw NPE again - throw new NullPointerException("mobileNumber.phoneNumber is null"); //NOI18N - } else if (mobileNumber.getPhoneNumber() < 1) { + throw new NullPointerException("mobileNumber.mobileNumber is null"); //NOI18N + } else if (mobileNumber.getMobileNumber() < 1) { // Throw NPE again - throw new NullPointerException(MessageFormat.format("mobileNumber.phoneNumber={0} is not valid.", mobileNumber.getPhoneNumber())); //NOI18N + throw new NullPointerException(MessageFormat.format("mobileNumber.mobileNumber={0} is not valid.", mobileNumber.getMobileNumber())); //NOI18N } // Get a managed instance - final DialableMobileNumber managedNumber = this.getEntityManager().getReference(mobileNumber.getClass(), mobileNumber.getPhoneId()); + final DialableMobileNumber managedNumber = this.getEntityManager().getReference(mobileNumber.getClass(), mobileNumber.getMobileId()); // Remove it from database this.getEntityManager().remove(managedNumber); @@ -234,7 +234,7 @@ public class AddressbookAdminPhoneSessionBean extends BaseAddressbookEnterpriseB this.getLoggerBeanLocal().logDebug(MessageFormat.format("{0}.updateFaxData: managedNumber.phoneId={1}", this.getClass().getSimpleName(), managedNumber.getPhoneId())); //NOI18N // Set updated timestamp - FaxNumbers.copyAll(faxNumber, managedNumber); + FaxNumbers.copyFaxNumber(faxNumber, managedNumber); managedNumber.setPhoneEntryUpdated(new Date()); // Trace message @@ -292,7 +292,7 @@ public class AddressbookAdminPhoneSessionBean extends BaseAddressbookEnterpriseB this.getLoggerBeanLocal().logDebug(MessageFormat.format("{0}.updateLandLineData: managedNumber.phoneId={1}", this.getClass().getSimpleName(), managedNumber.getPhoneId())); //NOI18N // Set updated timestamp - LandLineNumbers.copyAll(landLineNumber, managedNumber); + LandLineNumbers.copyLandLineNumber(landLineNumber, managedNumber); managedNumber.setPhoneEntryUpdated(new Date()); // Trace message @@ -311,12 +311,12 @@ public class AddressbookAdminPhoneSessionBean extends BaseAddressbookEnterpriseB if (null == mobileNumber) { // Not set, throw NPE throw new NullPointerException("mobileNumber is null"); //NOI18N - } else if (mobileNumber.getPhoneId() == null) { + } else if (mobileNumber.getMobileId() == null) { // Throw NPE again throw new NullPointerException("mobileNumber.phoneId is null"); //NOI18N - } else if (mobileNumber.getPhoneId() < 1) { + } else if (mobileNumber.getMobileId() < 1) { // Invalid number - throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getPhoneId())); //NOI18N + throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getMobileId())); //NOI18N } else if (mobileNumber.getMobileProvider() == null) { // Throw NPE throw new NullPointerException("mobileNumber.cellphoneProvider is null"); //NOI18N @@ -326,26 +326,26 @@ public class AddressbookAdminPhoneSessionBean extends BaseAddressbookEnterpriseB } else if (mobileNumber.getMobileProvider().getProviderId() < 1) { // Id not valid throw new IllegalArgumentException(MessageFormat.format("mobileNumber.cellphoneProvider.providerId={0} is not valid.", mobileNumber.getMobileProvider().getProviderId())); //NOI18N - } else if (mobileNumber.getPhoneNumber() == null) { + } else if (mobileNumber.getMobileNumber() == null) { // Throw NPE again - throw new NullPointerException("mobileNumber.phoneNumber is null"); //NOI18N - } else if (mobileNumber.getPhoneNumber() < 1) { + throw new NullPointerException("mobileNumber.mobileNumber is null"); //NOI18N + } else if (mobileNumber.getMobileNumber() < 1) { // Throw NPE again - throw new NullPointerException(MessageFormat.format("mobileNumber.phoneNumber={0} is not valid.", mobileNumber.getPhoneNumber())); //NOI18N + throw new NullPointerException(MessageFormat.format("mobileNumber.mobileNumber={0} is not valid.", mobileNumber.getMobileNumber())); //NOI18N } // Get contact from it and find it - final DialableMobileNumber managedNumber = this.getEntityManager().find(mobileNumber.getClass(), mobileNumber.getPhoneId()); + final DialableMobileNumber managedNumber = this.getEntityManager().find(mobileNumber.getClass(), mobileNumber.getMobileId()); // Should be found - assert (managedNumber instanceof DialableMobileNumber) : MessageFormat.format("Cell phone number with id {0} not found, but should be.", mobileNumber.getPhoneId()); //NOI18N + assert (managedNumber instanceof DialableMobileNumber) : MessageFormat.format("Cell phone number with id {0} not found, but should be.", mobileNumber.getMobileId()); //NOI18N // Debug message - this.getLoggerBeanLocal().logDebug(MessageFormat.format("{0}.updateMobileData: managedNumber.phoneId={1}", this.getClass().getSimpleName(), managedNumber.getPhoneId())); //NOI18N + this.getLoggerBeanLocal().logDebug(MessageFormat.format("{0}.updateMobileData: managedNumber.phoneId={1}", this.getClass().getSimpleName(), managedNumber.getMobileId())); //NOI18N // Set updated timestamp - MobileNumbers.copyAll(mobileNumber, managedNumber); - managedNumber.setPhoneEntryUpdated(new Date()); + MobileNumbers.copyMobileNumber(mobileNumber, managedNumber); + managedNumber.setMobileEntryUpdated(new Date()); // Trace message this.getLoggerBeanLocal().logTrace(MessageFormat.format("{0}.updateMobileData: managedNumber={1} - EXIT!", this.getClass().getSimpleName(), managedNumber)); //NOI18N diff --git a/src/java/org/mxchange/jusercore/model/user/AddressbookAdminUserSessionBean.java b/src/java/org/mxchange/jusercore/model/user/AddressbookAdminUserSessionBean.java index 2ddc6f3..c613b19 100644 --- a/src/java/org/mxchange/jusercore/model/user/AddressbookAdminUserSessionBean.java +++ b/src/java/org/mxchange/jusercore/model/user/AddressbookAdminUserSessionBean.java @@ -82,7 +82,7 @@ public class AddressbookAdminUserSessionBean extends BaseAddressbookEnterpriseBe // Set created timestamp user.setUserCreated(new Date()); - user.getUserContact().setContactCreated(new Date()); + user.getUserContact().setContactEntryCreated(new Date()); // Update mobile, land-line and fax instance this.setAllPhoneEntriesCreated(user.getUserContact()); diff --git a/src/java/org/mxchange/jusercore/model/user/AddressbookUserSessionBean.java b/src/java/org/mxchange/jusercore/model/user/AddressbookUserSessionBean.java index c89bf4e..962da34 100644 --- a/src/java/org/mxchange/jusercore/model/user/AddressbookUserSessionBean.java +++ b/src/java/org/mxchange/jusercore/model/user/AddressbookUserSessionBean.java @@ -599,19 +599,22 @@ public class AddressbookUserSessionBean extends BaseAddressbookEnterpriseBean im // Is there a mobile instance set? if (mobileNumber instanceof DialableMobileNumber) { // Debug message - this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: mobile.phoneId={0} is being updated ...", mobileNumber.getPhoneId())); //NOI18N + this.getLoggerBeanLocal().logDebug(MessageFormat.format("updateUserPersonalData: mobile.phoneId={0} is being updated ...", mobileNumber.getMobileId())); //NOI18N // Then find it, too - final DialableMobileNumber foundMobile = this.getEntityManager().find(mobileNumber.getClass(), mobileNumber.getPhoneId()); + final DialableMobileNumber foundMobile = this.getEntityManager().find(mobileNumber.getClass(), mobileNumber.getMobileId()); // Should be there - assert (foundMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", foundMobile.getPhoneId()); //NOI18N + assert (foundMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", foundMobile.getMobileId()); //NOI18N + + // Copy all + MobileNumbers.copyMobileNumber(managedUser.getUserContact().getContactMobileNumber(), foundMobile); // Then merge it, too final DialableMobileNumber managedMobile = this.getEntityManager().merge(foundMobile); // Should be there - assert (managedMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", managedMobile.getPhoneId()); //NOI18N + assert (managedMobile instanceof DialableMobileNumber) : MessageFormat.format("Mobile number with id {0} not found but should be.", managedMobile.getMobileId()); //NOI18N // Copy all MobileNumbers.copyAll(managedUser.getUserContact().getContactMobileNumber(), managedMobile); @@ -634,6 +637,9 @@ public class AddressbookUserSessionBean extends BaseAddressbookEnterpriseBean im // Should be there assert (foundFax instanceof DialableFaxNumber) : MessageFormat.format("Fax number with id {0} not found but should be.", foundFax.getPhoneId()); //NOI18N + // Copy all + FaxNumbers.copyFaxNumber(managedUser.getUserContact().getContactFaxNumber(), foundFax); + // Then merge it, too final DialableFaxNumber managedFax = this.getEntityManager().merge(foundFax); @@ -661,6 +667,9 @@ public class AddressbookUserSessionBean extends BaseAddressbookEnterpriseBean im // Should be there assert (foundLandLine instanceof DialableLandLineNumber) : MessageFormat.format("Land line number with id {0} not found but should be.", foundLandLine.getPhoneId()); //NOI18N + // Copy all + LandLineNumbers.copyLandLineNumber(managedUser.getUserContact().getContactLandLineNumber(), foundLandLine); + // Then merge it, too final DialableLandLineNumber managedLandLine = this.getEntityManager().merge(foundLandLine); -- 2.39.5