]> git.mxchange.org Git - pizzaservice-war.git/commitdiff
Please cherry-pick:
authorRoland Häder <roland@mxchange.org>
Sun, 30 Jul 2017 20:03:49 +0000 (22:03 +0200)
committerRoland Häder <roland@mxchange.org>
Sun, 30 Jul 2017 21:17:20 +0000 (23:17 +0200)
- rewrite of almost all beans towards more cleanness
- now fields in backing beans can be "only" be set by JSF views or observer
  methods
- removed @Local from interfaces, they are already local and maybe that
  annotation is for EJBs only?
- passing beanHelper.fooBar over a JSF view to a backing bean is not needed,
  the bean can do it by itself, by CDI

- fixed tpzo

Signed-off-by: Roland Häder <roland@mxchange.org>
43 files changed:
src/java/org/mxchange/localization/bundle_en_US.properties
src/java/org/mxchange/pizzaapplication/beans/contact/PizzaAdminContactWebRequestBean.java
src/java/org/mxchange/pizzaapplication/beans/contact/PizzaAdminContactWebRequestController.java
src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionBean.java
src/java/org/mxchange/pizzaapplication/beans/contact/PizzaContactWebSessionController.java
src/java/org/mxchange/pizzaapplication/beans/contact/phone/PizzaAdminContactPhoneWebRequestBean.java
src/java/org/mxchange/pizzaapplication/beans/contact/phone/PizzaAdminContactPhoneWebRequestController.java
src/java/org/mxchange/pizzaapplication/beans/contact/phone/PizzaContactPhoneWebSessionController.java
src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestBean.java
src/java/org/mxchange/pizzaapplication/beans/country/PizzaAdminCountryWebRequestController.java
src/java/org/mxchange/pizzaapplication/beans/features/PizzaFeaturesWebApplicationController.java
src/java/org/mxchange/pizzaapplication/beans/helper/PizzaWebRequestHelperBean.java
src/java/org/mxchange/pizzaapplication/beans/mobileprovider/PizzaAdminMobileProviderWebRequestBean.java
src/java/org/mxchange/pizzaapplication/beans/mobileprovider/PizzaAdminMobileProviderWebRequestController.java
src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestBean.java
src/java/org/mxchange/pizzaapplication/beans/phone/PizzaAdminPhoneWebRequestController.java
src/java/org/mxchange/pizzaapplication/beans/phone/PizzaPhoneWebApplicationController.java
src/java/org/mxchange/pizzaapplication/beans/user/PizzaAdminUserWebRequestBean.java
src/java/org/mxchange/pizzaapplication/beans/user/PizzaAdminUserWebRequestController.java
src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionBean.java
src/java/org/mxchange/pizzaapplication/beans/user/PizzaUserWebSessionController.java
src/java/org/mxchange/pizzaapplication/beans/user/email_address/PizzaEmailChangeWebSessionBean.java
src/java/org/mxchange/pizzaapplication/beans/user/email_address/PizzaEmailChangeWebSessionController.java
src/java/org/mxchange/pizzaapplication/beans/user/login/PizzaUserLoginWebSessionBean.java
src/java/org/mxchange/pizzaapplication/beans/user/login/PizzaUserLoginWebSessionController.java
src/java/org/mxchange/pizzaapplication/beans/user/password/PizzaUserPasswordWebRequestBean.java
src/java/org/mxchange/pizzaapplication/beans/user/password/PizzaUserPasswordWebRequestController.java
src/java/org/mxchange/pizzaapplication/beans/user/register/PizzaUserRegisterWebSessionBean.java
src/java/org/mxchange/pizzaapplication/beans/user/register/PizzaUserRegisterWebSessionController.java
src/java/org/mxchange/pizzaapplication/beans/user/resendlink/PizzaResendLinkWebSessionBean.java
src/java/org/mxchange/pizzaapplication/beans/user/resendlink/PizzaResendLinkWebSessionController.java
web/WEB-INF/templates/admin/contact/admin_form_contact_data.tpl
web/WEB-INF/templates/admin/fax/admin_form_add_contact_fax.tpl
web/admin/fax/admin_fax_delete.xhtml
web/admin/fax/admin_fax_edit.xhtml
web/admin/landline/admin_landline_delete.xhtml
web/admin/landline/admin_landline_edit.xhtml
web/admin/mobile/admin_mobile_delete.xhtml
web/admin/mobile/admin_mobile_edit.xhtml
web/admin/user/admin_user_delete.xhtml
web/admin/user/admin_user_export.xhtml
web/admin/user/admin_user_lock.xhtml
web/admin/user/admin_user_unlock.xhtml

index 81b8ef1c4ab5d5a62f53ae4222559f7fd542cba6..d1dd5a0f5ea007905f966135f4879229d6b6bccb 100644 (file)
@@ -718,7 +718,7 @@ ADMIN_RESEND_USER_CONFIRMATION_LINK_NOTICE=By sending a new confirmation link to
 PAGE_TITLE_ADMIN_RESEND_USER_CONFIRMATION_LINK=Send new confirmation link
 CONTENT_TITLE_ADMIN_RESEND_USER_CONFIRMATION_LINK=Send new confirmation link:
 ADMIN_SHOW_USER_ID=User id:
-TABLE_SUMMARY_ADMIN_LIST_USER_ACTIVITY_LOG=This table shows choosen user's activity log.
+TABLE_SUMMARY_ADMIN_LIST_USER_ACTIVITY_LOG=This table shows chosen user's activity log.
 ADMIN_LINK_SHOW_USER_ACTIVITY_LOG=List user's actitivy log
 PAGE_TITLE_ADMIN_USER_ACTIVITY_LOG=Logfile of user activity
 CONTENT_TITLE_ADMIN_USER_ACTIVITY_LOG=Logfile of user activity:
index da791cd9e9378fc62f621dbf7335ba2472508fea..f8ce3f5a4dda8358849db401d2091f32e64396ac 100644 (file)
@@ -124,7 +124,7 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
        /**
         * Country instance
         */
-       private Country country;
+       private Country contactCountry;
 
        /**
         * Email address
@@ -182,7 +182,7 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
        private boolean isLandLineUnlinked;
 
        /**
-        * Whether a cellphone entry has been unlinked
+        * Whether a cmobile entry has been unlinked
         */
        private boolean isMobileNumberUnlinked;
 
@@ -251,7 +251,12 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                super();
        }
 
-       @Override
+       /**
+        * Adds contact data to database and redirects on success. If the contact is
+        * already found, a proper exception is thrown.
+        * <p>
+        * @return Redirect outcome
+        */
        public String addContact () {
                // Are all minimum fields set?
                if (this.getPersonalTitle() == null) {
@@ -325,7 +330,7 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                this.setBirthday(contact.getContactBirthday());
                this.setCity(contact.getContactCity());
                this.setComment(contact.getContactComment());
-               this.setCountry(contact.getContactCountry());
+               this.setContactCountry(contact.getContactCountry());
                this.setEmailAddress(contact.getContactEmailAddress());
                this.setFamilyName(contact.getContactFamilyName());
                this.setFirstName(contact.getContactFirstName());
@@ -337,7 +342,7 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
 
                // Is the cell phone set?
                if (contact.getContactMobileNumber() instanceof DialableMobileNumber) {
-                       // ... cellphone data
+                       // ... cmobile data
                        this.setMobileId(contact.getContactMobileNumber().getPhoneId());
                        this.setMobileProvider(contact.getContactMobileNumber().getMobileProvider());
                        this.setMobileNumber(contact.getContactMobileNumber().getPhoneNumber());
@@ -400,7 +405,7 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                contact.setContactHouseNumber(this.getHouseNumber());
                contact.setContactZipCode(this.getZipCode());
                contact.setContactCity(this.getCity());
-               contact.setContactCountry(this.getCountry());
+               contact.setContactCountry(this.getContactCountry());
                contact.setContactEmailAddress(this.getEmailAddress());
                contact.setContactBirthday(this.getBirthday());
                contact.setContactComment(this.getComment());
@@ -455,13 +460,13 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                        // Is the number set?
                        if (mobile.getPhoneNumber() == null) {
                                // Is null
-                               throw new NullPointerException("cellphone.phoneNumber is null"); //NOI18N
+                               throw new NullPointerException("cmobile.phoneNumber is null"); //NOI18N
                        } else if (mobile.getPhoneNumber() < 1) {
                                // Abort here
-                               throw new IllegalArgumentException("cellphone.phoneNumber is zero or below."); //NOI18N
+                               throw new IllegalArgumentException("cmobile.phoneNumber is zero or below."); //NOI18N
                        }
 
-                       // Set cellphone number
+                       // Set cmobile number
                        contact.setContactMobileNumber(mobile);
                }
 
@@ -469,7 +474,11 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                return contact;
        }
 
-       @Override
+       /**
+        * Edits currently loaded contact's data in database.
+        * <p>
+        * @return Redirect outcome
+        */
        public String editContactData () {
                // Get contact instance
                Contact contact = this.beanHelper.getContact();
@@ -502,7 +511,13 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                return "admin_list_contact"; //NOI18N
        }
 
-       @Override
+       /**
+        * Returns a text representation of given mobile number or null if not set.
+        * <p>
+        * @param mobileNumber Mobile number
+        * <p>
+        * @return Text representation or null
+        */
        public String generateMobileNumber (final DialableMobileNumber mobileNumber) {
                // Is it null?
                if (null == mobileNumber) {
@@ -522,7 +537,14 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                return number;
        }
 
-       @Override
+       /**
+        * Returns a text representation of given land-line or fax number or null if
+        * not set.
+        * <p>
+        * @param phoneNumber Land-line or fax number
+        * <p>
+        * @return Text representation or null
+        */
        public String generatePhoneNumber (final DialableNumber phoneNumber) {
                // Is it null?
                if (null == phoneNumber) {
@@ -542,54 +564,94 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                return number;
        }
 
-       @Override
+       /**
+        * Getter for academic title
+        * <p>
+        * @return Academic title
+        */
        public String getAcademicTitle () {
                return this.academicTitle;
        }
 
-       @Override
+       /**
+        * Setter for academic title
+        * <p>
+        * @param academicTitle Academic title
+        */
        public void setAcademicTitle (final String academicTitle) {
                this.academicTitle = academicTitle;
        }
 
-       @Override
+       /**
+        * Getter for birth day
+        * <p>
+        * @return Birth day
+        */
        @SuppressWarnings ("ReturnOfDateField")
        public Date getBirthday () {
                return this.birthday;
        }
 
-       @Override
+       /**
+        * Setter for birth day
+        * <p>
+        * @param birthday Birth day
+        */
        @SuppressWarnings ("AssignmentToDateFieldFromParameter")
        public void setBirthday (final Date birthday) {
                this.birthday = birthday;
        }
 
-       @Override
+       /**
+        * Getter for city name
+        * <p>
+        * @return City name
+        */
        public String getCity () {
                return this.city;
        }
 
-       @Override
+       /**
+        * Setter for city name
+        * <p>
+        * @param city City name
+        */
        public void setCity (final String city) {
                this.city = city;
        }
 
-       @Override
+       /**
+        * Getter for comments
+        * <p>
+        * @return Comments
+        */
        public String getComment () {
                return this.comment;
        }
 
-       @Override
+       /**
+        * Setter for comment
+        * <p>
+        * @param comment Comments
+        */
        public void setComment (final String comment) {
                this.comment = comment;
        }
 
-       @Override
+       /**
+        * Getter for contact id
+        * <p>
+        * @return Contact id
+        */
        public Long getContactId () {
                return this.contactId;
        }
 
-       @Override
+       /**
+        * Setter for contact id
+        * <p>
+        * @param contactId Contact id
+        */
        public void setContactId (final Long contactId) {
                this.contactId = contactId;
        }
@@ -605,202 +667,363 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                throw new UnsupportedOperationException("Setting controller type is not supported."); //NOI18N
        }
 
-       @Override
-       public Country getCountry () {
-               return this.country;
+       /**
+        * Getter for contactCountry instance
+        * <p>
+        * @return Country instance
+        */
+       public Country getContactCountry () {
+               return this.contactCountry;
        }
 
-       @Override
-       public void setCountry (final Country country) {
-               this.country = country;
+       /**
+        * Setter for contactCountry instance
+        * <p>
+        * @param contactCountry Country instance
+        */
+       public void setContactCountry (final Country contactCountry) {
+               this.contactCountry = contactCountry;
        }
 
-       @Override
+       /**
+        * Getter for email address
+        * <p>
+        * @return Email address
+        */
        public String getEmailAddress () {
                return this.emailAddress;
        }
 
-       @Override
+       /**
+        * Setter for email address
+        * <p>
+        * @param emailAddress Email address
+        */
        public void setEmailAddress (final String emailAddress) {
                this.emailAddress = emailAddress;
        }
 
-       @Override
+       /**
+        * Family name
+        * <p>
+        * @return the familyName
+        */
        public String getFamilyName () {
                return this.familyName;
        }
 
-       @Override
+       /**
+        * Family name
+        * <p>
+        * @param familyName the familyName to set
+        */
        public void setFamilyName (final String familyName) {
                this.familyName = familyName;
        }
 
-       @Override
+       /**
+        * Getter for fax number's area code
+        * <p>
+        * @return Fax number's area code
+        */
        public Integer getFaxAreaCode () {
                return this.faxAreaCode;
        }
 
-       @Override
+       /**
+        * Setter for fax number's area code
+        * <p>
+        * @param faxAreaCode Fax number's area code
+        */
        public void setFaxAreaCode (final Integer faxAreaCode) {
                this.faxAreaCode = faxAreaCode;
        }
 
-       @Override
+       /**
+        * Getter for fax's country instance
+        * <p>
+        * @return Fax' country instance
+        */
        public Country getFaxCountry () {
                return this.faxCountry;
        }
 
-       @Override
+       /**
+        * Setter for fax's country instance
+        * <p>
+        * @param faxCountry Fax' country instance
+        */
        public void setFaxCountry (final Country faxCountry) {
                this.faxCountry = faxCountry;
        }
 
-       @Override
+       /**
+        * Getter for fax id
+        * <p>
+        * @return Fax id
+        */
        public Long getFaxId () {
                return this.faxId;
        }
 
-       @Override
+       /**
+        * Setter for fax id
+        * <p>
+        * @param faxId Fax id
+        */
        public void setFaxId (final Long faxId) {
                this.faxId = faxId;
        }
 
-       @Override
+       /**
+        * Getter for fax number
+        * <p>
+        * @return Fax number
+        */
        public Long getFaxNumber () {
                return this.faxNumber;
        }
 
-       @Override
+       /**
+        * Setter for fax number
+        * <p>
+        * @param faxNumber Fax number
+        */
        public void setFaxNumber (final Long faxNumber) {
                this.faxNumber = faxNumber;
        }
 
-       @Override
+       /**
+        * Getter for first name
+        * <p>
+        * @return First name
+        */
        public String getFirstName () {
                return this.firstName;
        }
 
-       @Override
+       /**
+        * Setter for first name
+        * <p>
+        * @param firstName First name
+        */
        public void setFirstName (final String firstName) {
                this.firstName = firstName;
        }
 
-       @Override
+       /**
+        * Getter for house number
+        * <p>
+        * @return House number
+        */
        public Short getHouseNumber () {
                return this.houseNumber;
        }
 
-       @Override
+       /**
+        * Setter for house number
+        * <p>
+        * @param houseNumber House number
+        */
        public void setHouseNumber (final Short houseNumber) {
                this.houseNumber = houseNumber;
        }
 
-       @Override
+       /**
+        * Getter for house number extension. Example: 123a, 'a' is the extension
+        * and 123 is the house number.
+        * <p>
+        * @return House number extension
+        */
        public String getHouseNumberExtension () {
                return this.houseNumberExtension;
        }
 
-       @Override
+       /**
+        * Setter for house number extension
+        * <p>
+        * @param houseNumberExtension House number extension
+        */
        public void setHouseNumberExtension (final String houseNumberExtension) {
                this.houseNumberExtension = houseNumberExtension;
        }
 
-       @Override
+       /**
+        * Getter for land-line number's area code
+        * <p>
+        * @return Land-line number's area code
+        */
        public Integer getLandLineAreaCode () {
                return this.landLineAreaCode;
        }
 
-       @Override
+       /**
+        * Setter for land-line number's area code
+        * <p>
+        * @param landLineAreaCode Land-line number's area code
+        */
        public void setLandLineAreaCode (final Integer landLineAreaCode) {
                this.landLineAreaCode = landLineAreaCode;
        }
 
-       @Override
+       /**
+        * Getter for land-line number's country instance
+        * <p>
+        * @return Land-line number's country instance
+        */
        public Country getLandLineCountry () {
                return this.landLineCountry;
        }
 
-       @Override
+       /**
+        * Setter for land-line number's country instance
+        * <p>
+        * @param landLineCountry Land-line number's country instance
+        */
        public void setLandLineCountry (final Country landLineCountry) {
                this.landLineCountry = landLineCountry;
        }
 
-       @Override
+       /**
+        * Getter for land-line id
+        * <p>
+        * @return Land-line id
+        */
        public Long getLandLineId () {
                return this.landLineId;
        }
 
-       @Override
+       /**
+        * Setter for land-line id
+        * <p>
+        * @param landLineId Land-line id
+        */
        public void setLandLineId (final Long landLineId) {
                this.landLineId = landLineId;
        }
 
-       @Override
+       /**
+        * Getter for land-line number
+        * <p>
+        * @return Land-line number
+        */
        public Long getLandLineNumber () {
                return this.landLineNumber;
        }
 
-       @Override
+       /**
+        * Setter for land-line number
+        * <p>
+        * @param landLineNumber Land-line number
+        */
        public void setLandLineNumber (final Long landLineNumber) {
                this.landLineNumber = landLineNumber;
        }
 
-       @Override
+       /**
+        * Getter for mobile id
+        * <p>
+        * @return Mobile id
+        */
        public Long getMobileId () {
                return this.mobileId;
        }
 
-       @Override
+       /**
+        * Setter for mobile id
+        * <p>
+        * @param mobileId Mobile id
+        */
        public void setMobileId (final Long mobileId) {
                this.mobileId = mobileId;
        }
 
-       @Override
+       /**
+        * Getter for mobile number
+        * <p>
+        * @return Mobile number
+        */
        public Long getMobileNumber () {
                return this.mobileNumber;
        }
 
-       @Override
-       public void setMobileNumber (Long mobileNumber) {
+       /**
+        * Setter for mobile number
+        * <p>
+        * @param mobileNumber Mobile number
+        */
+       public void setMobileNumber (final Long mobileNumber) {
                this.mobileNumber = mobileNumber;
        }
 
-       @Override
+       /**
+        * Getter for mobile number's carrier
+        * <p>
+        * @return Mobile number's carrier
+        */
        public MobileProvider getMobileProvider () {
                return this.mobileProvider;
        }
 
-       @Override
+       /**
+        * Setter for mobile number's carrier prefix
+        * <p>
+        * @param mobileProvider Mobile number's carrier prefix
+        */
        public void setMobileProvider (final MobileProvider mobileProvider) {
                this.mobileProvider = mobileProvider;
        }
 
-       @Override
+       /**
+        * Getter for personal title
+        * <p>
+        * @return Personal title
+        */
        public PersonalTitle getPersonalTitle () {
                return this.personalTitle;
        }
 
-       @Override
+       /**
+        * Setter for personal title
+        * <p>
+        * @param personalTitle Personal title
+        */
        public void setPersonalTitle (final PersonalTitle personalTitle) {
                this.personalTitle = personalTitle;
        }
 
-       @Override
+       /**
+        * Getter for street name
+        * <p>
+        * @return Street name
+        */
        public String getStreet () {
                return this.street;
        }
 
-       @Override
+       /**
+        * Setter for street name
+        * <p>
+        * @param street Street name
+        */
        public void setStreet (final String street) {
                this.street = street;
        }
 
-       @Override
+       /**
+        * Getter for ZIP code
+        * <p>
+        * @return ZIP code
+        */
        public Integer getZipCode () {
                return this.zipCode;
        }
 
-       @Override
+       /**
+        * Setter for ZIP code
+        * <p>
+        * @param zipCode ZIP code
+        */
        public void setZipCode (final Integer zipCode) {
                this.zipCode = zipCode;
        }
@@ -827,6 +1050,7 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
        }
 
        @Override
+       @Deprecated
        public boolean isPersonalTitleRequired () {
                // Get context parameter
                String contextParameter = FacesContext.getCurrentInstance().getExternalContext().getInitParameter("is_admin_personal_title_enabled"); //NOI18N
@@ -838,6 +1062,32 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                return isRequired;
        }
 
+       @Override
+       public void validateContactData () {
+               if (this.getPersonalTitle() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("contactController.gender is null"); //NOI18N
+               } else if (this.getFirstName() == null) {
+                       // ... and again
+                       throw new NullPointerException("contactController.firstName is null"); //NOI18N
+               } else if (this.getFirstName().isEmpty()) {
+                       // ... and again
+                       throw new IllegalArgumentException("contactController.firstName is empty"); //NOI18N
+               } else if (this.getFamilyName() == null) {
+                       // ... and again
+                       throw new NullPointerException("contactController.familyName is null"); //NOI18N
+               } else if (this.getFamilyName().isEmpty()) {
+                       // ... and again
+                       throw new IllegalArgumentException("contactController.familyName is empty"); //NOI18N
+               } else if (this.getEmailAddress() == null) {
+                       // ... and again
+                       throw new NullPointerException("contactController.emailAddress is null"); //NOI18N
+               } else if (this.getEmailAddress().isEmpty()) {
+                       // ... and again
+                       throw new IllegalArgumentException("contactController.emailAddress is empty"); //NOI18N
+               }
+       }
+
        /**
         * Clears this bean
         */
@@ -852,7 +1102,7 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                this.setHouseNumberExtension(null);
                this.setZipCode(null);
                this.setCity(null);
-               this.setCountry(null);
+               this.setContactCountry(null);
 
                // - contact data
                this.setEmailAddress(null);
@@ -929,9 +1179,9 @@ public class PizzaAdminContactWebRequestBean extends BasePizzaController impleme
                contact.setContactHouseNumberExtension(this.getHouseNumberExtension());
                contact.setContactZipCode(this.getZipCode());
                contact.setContactCity(this.getCity());
-               contact.setContactCountry(this.getCountry());
+               contact.setContactCountry(this.getContactCountry());
 
-               // Update contact's cellphone number
+               // Update contact's cmobile number
                this.isMobileNumberUnlinked = ContactUtils.updateMobileNumber(contact, this.getMobileProvider(), this.getMobileNumber());
 
                // Update contact's land-line number
index d89fb563fda8cef71bd6119469b52e52b0fb591c..e430c9f3ab314454072d6c23d77e02f416eefe12 100644 (file)
 package org.mxchange.pizzaapplication.beans.contact;
 
 import java.io.Serializable;
-import java.util.Date;
-import javax.ejb.Local;
 import org.mxchange.jcontacts.contact.Contact;
-import org.mxchange.jcontacts.contact.title.PersonalTitle;
-import org.mxchange.jcountry.data.Country;
-import org.mxchange.jphone.phonenumbers.DialableNumber;
-import org.mxchange.jphone.phonenumbers.mobile.DialableMobileNumber;
-import org.mxchange.jphone.phonenumbers.mobileprovider.MobileProvider;
 
 /**
  * Administrative interface for user beans
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaAdminContactWebRequestController extends Serializable {
 
        /**
-        * Adds contact data to database and redirects on success. If the contact is
-        * already found, a proper exception is thrown.
-        * <p>
-        * @return Redirect outcome
+        * Validates contact data and throws proper exceptions if not valid
         */
-       String addContact ();
+       void validateContactData ();
 
        /**
         * Creates an instance from contact data
@@ -56,364 +45,6 @@ public interface PizzaAdminContactWebRequestController extends Serializable {
         */
        void copyContactToController (final Contact contact);
 
-       /**
-        * Edits currently loaded contact's data in database.
-        * <p>
-        * @return Redirect outcome
-        */
-       String editContactData ();
-
-       /**
-        * Getter for cellphone id
-        * <p>
-        * @return Mobile id
-        */
-       Long getMobileId ();
-
-       /**
-        * Setter for cellphone id
-        * <p>
-        * @param cellphoneId Mobile id
-        */
-       void setMobileId (final Long cellphoneId);
-
-       /**
-        * Getter for fax id
-        * <p>
-        * @return Fax id
-        */
-       Long getFaxId ();
-
-       /**
-        * Setter for fax id
-        * <p>
-        * @param faxId Fax id
-        */
-       void setFaxId (final Long faxId);
-
-       /**
-        * Getter for land-line id
-        * <p>
-        * @return Land-line id
-        */
-       Long getLandLineId ();
-
-       /**
-        * Setter for land-line id
-        * <p>
-        * @param landLineId Land-line id
-        */
-       void setLandLineId (final Long landLineId);
-
-       /**
-        * Getter for birth day
-        * <p>
-        * @return Birth day
-        */
-       Date getBirthday ();
-
-       /**
-        * Setter for birth day
-        * <p>
-        * @param birthday Birth day
-        */
-       void setBirthday (final Date birthday);
-
-       /**
-        * Getter for ellphone number's carrier
-        * <p>
-        * @return Mobile number's carrier
-        */
-       MobileProvider getMobileProvider ();
-
-       /**
-        * Setter for cellphone number's carrier prefix
-        * <p>
-        * @param cellphoneCarrier Mobile number's carrier prefix
-        */
-       void setMobileProvider (final MobileProvider cellphoneCarrier);
-
-       /**
-        * Getter for ellphone number
-        * <p>
-        * @return Mobile number
-        */
-       Long getMobileNumber ();
-
-       /**
-        * Setter for ellphone number
-        * <p>
-        * @param mobileNumber Mobile number
-        */
-       void setMobileNumber (final Long mobileNumber);
-
-       /**
-        * City
-        * <p>
-        * @return the city
-        */
-       String getCity ();
-
-       /**
-        * City
-        * <p>
-        * @param city the city to set
-        */
-       void setCity (final String city);
-
-       /**
-        * Getter for comments
-        * <p>
-        * @return Comments
-        */
-       String getComment ();
-
-       /**
-        * Setter for comment
-        * <p>
-        * @param comment Comments
-        */
-       void setComment (final String comment);
-
-       /**
-        * Getter for contact id
-        * <p>
-        * @return Contact id
-        */
-       Long getContactId ();
-
-       /**
-        * Setter for contact id
-        * <p>
-        * @param contactId Contact id
-        */
-       void setContactId (final Long contactId);
-
-       /**
-        * Getter for country instance
-        * <p>
-        * @return Country instance
-        */
-       Country getCountry ();
-
-       /**
-        * Setter for country instance
-        * <p>
-        * @param country Country instance
-        */
-       void setCountry (final Country country);
-
-       /**
-        * Getter for email address
-        * <p>
-        * @return Email address
-        */
-       String getEmailAddress ();
-
-       /**
-        * Setter for email address
-        * <p>
-        * @param emailAddress Email address
-        */
-       void setEmailAddress (final String emailAddress);
-
-       /**
-        * Family name
-        * <p>
-        * @return the familyName
-        */
-       String getFamilyName ();
-
-       /**
-        * Family name
-        * <p>
-        * @param familyName the familyName to set
-        */
-       void setFamilyName (final String familyName);
-
-       /**
-        * Getter for fax number's area code
-        * <p>
-        * @return Fax number's area code
-        */
-       Integer getFaxAreaCode ();
-
-       /**
-        * Setter for fax number's area code
-        * <p>
-        * @param faxAreaCode Fax number's area code
-        */
-       void setFaxAreaCode (final Integer faxAreaCode);
-
-       /**
-        * Getter for fax's country instance
-        * <p>
-        * @return Fax' country instance
-        */
-       Country getFaxCountry ();
-
-       /**
-        * Setter for fax's country instance
-        * <p>
-        * @param faxCountry Fax' country instance
-        */
-       void setFaxCountry (final Country faxCountry);
-
-       /**
-        * Getter for fax number
-        * <p>
-        * @return Fax number
-        */
-       Long getFaxNumber ();
-
-       /**
-        * Setter for fax number
-        * <p>
-        * @param faxNumber Fax number
-        */
-       void setFaxNumber (final Long faxNumber);
-
-       /**
-        * First name
-        * <p>
-        * @return the first name
-        */
-       String getFirstName ();
-
-       /**
-        * First name
-        * <p>
-        * @param firstName the first name to set
-        */
-       void setFirstName (final String firstName);
-
-       /**
-        * Getter for personal title
-        * <p>
-        * @return Personal title
-        */
-       PersonalTitle getPersonalTitle ();
-
-       /**
-        * Setter for personal title
-        * <p>
-        * @param personalTitle Personal title
-        */
-       void setPersonalTitle (final PersonalTitle personalTitle);
-
-       /**
-        * House number
-        * <p>
-        * @return the houseNumber
-        */
-       Short getHouseNumber ();
-
-       /**
-        * House number
-        * <p>
-        * @param houseNumber the houseNumber to set
-        */
-       void setHouseNumber (final Short houseNumber);
-
-       /**
-        * Getter for house number extension, example: 123a 'a' is then the
-        * extension and 123 is the house number.
-        * <p>
-        * @return House number extension
-        */
-       String getHouseNumberExtension ();
-
-       /**
-        * Setter for house number extension
-        * <p>
-        * @param houseNumberExtension House number extension
-        */
-       void setHouseNumberExtension (final String houseNumberExtension);
-
-       /**
-        * Getter for phone number's area code
-        * <p>
-        * @return Phone number's area code
-        */
-       Integer getLandLineAreaCode ();
-
-       /**
-        * Setter for phone number's area code
-        * <p>
-        * @param phoneAreaCode Phone number's area code
-        */
-       void setLandLineAreaCode (final Integer phoneAreaCode);
-
-       /**
-        * Getter for phone number's country instance
-        * <p>
-        * @return Phone number's country instance
-        */
-       Country getLandLineCountry ();
-
-       /**
-        * Setter for phone number's country instance
-        * <p>
-        * @param phoneCountry Phone number's country instance
-        */
-       void setLandLineCountry (final Country phoneCountry);
-
-       /**
-        * Getter for phone number
-        * <p>
-        * @return Phone number
-        */
-       Long getLandLineNumber ();
-
-       /**
-        * Setter for phone number
-        * <p>
-        * @param phoneNumber Phone number
-        */
-       void setLandLineNumber (final Long phoneNumber);
-
-       /**
-        * Street
-        * <p>
-        * @return the street
-        */
-       String getStreet ();
-
-       /**
-        * Street
-        * <p>
-        * @param street the street to set
-        */
-       void setStreet (final String street);
-
-       /**
-        * Getter for academic title
-        * <p>
-        * @return Academic title
-        */
-       String getAcademicTitle ();
-
-       /**
-        * Setter for academic title
-        * <p>
-        * @param academicTitle Academic title
-        */
-       void setAcademicTitle (final String academicTitle);
-
-       /**
-        * ZIP code
-        * <p>
-        * @return the zipCode
-        */
-       Integer getZipCode ();
-
-       /**
-        * ZIP code
-        * <p>
-        * @param zipCode the zipCode to set
-        */
-       void setZipCode (final Integer zipCode);
-
        /**
         * Getter for controller type
         * <p>
@@ -431,31 +62,15 @@ public interface PizzaAdminContactWebRequestController extends Serializable {
        @Deprecated
        void setControllerType (final String controllerType);
 
-       /**
-        * Returns a text respresentation of given phone number or null if not set.
-        * <p>
-        * @param phoneNumber Phone number
-        * <p>
-        * @return Text respresentation or null
-        */
-       String generatePhoneNumber (final DialableNumber phoneNumber);
-
-       /**
-        * Returns a text representation of given cellphone number or null if not
-        * set.
-        * <p>
-        * @param mobileNumber Mobile number
-        * <p>
-        * @return Text respresentation or null
-        */
-       String generateMobileNumber (final DialableMobileNumber mobileNumber);
-
        /**
         * Checks/returns whether the personal title (Mr./Mrs.) is required for this
         * controller.
         * <p>
         * @return Whether personal title is required
+        * <p>
+        * @Deprecated Don't use this method
         */
+       @Deprecated
        boolean isPersonalTitleRequired ();
 
 }
index cec2b62db7728d5cc0e50ef2458f31f4e2b7ef9b..5c01b27eddb99915bf6c70e763a6b401f39ff3da 100644 (file)
@@ -453,6 +453,13 @@ public class PizzaContactWebSessionBean extends BasePizzaController implements P
                return this.contactList;
        }
 
+       @Override
+       public void clearEmailAddresses () {
+               // Clear both
+               this.setEmailAddress(null);
+               this.setEmailAddressRepeat(null);
+       }
+
        @Override
        public Contact createContactInstance () {
                // Is all required data set?
@@ -592,44 +599,76 @@ public class PizzaContactWebSessionBean extends BasePizzaController implements P
                return "contact_data_saved"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for academic title
+        * <p>
+        * @return Academic title
+        */
        public String getAcademicTitle () {
                return this.academicTitle;
        }
 
-       @Override
+       /**
+        * Setter for academic title
+        * <p>
+        * @param academicTitle Academic title
+        */
        public void setAcademicTitle (final String academicTitle) {
                this.academicTitle = academicTitle;
        }
 
-       @Override
+       /**
+        * Getter for birth day
+        * <p>
+        * @return Birth day
+        */
        @SuppressWarnings ("ReturnOfDateField")
        public Date getBirthday () {
                return this.birthday;
        }
 
-       @Override
+       /**
+        * Setter for birth day
+        * <p>
+        * @param birthday Birth day
+        */
        @SuppressWarnings ("AssignmentToDateFieldFromParameter")
        public void setBirthday (final Date birthday) {
                this.birthday = birthday;
        }
 
-       @Override
+       /**
+        * Getter for city name
+        * <p>
+        * @return City name
+        */
        public String getCity () {
                return this.city;
        }
 
-       @Override
+       /**
+        * Setter for city name
+        * <p>
+        * @param city City name
+        */
        public void setCity (final String city) {
                this.city = city;
        }
 
-       @Override
+       /**
+        * Getter for comments
+        * <p>
+        * @return Comments
+        */
        public String getComment () {
                return this.comment;
        }
 
-       @Override
+       /**
+        * Setter for comment
+        * <p>
+        * @param comment Comments
+        */
        public void setComment (final String comment) {
                this.comment = comment;
        }
@@ -645,12 +684,20 @@ public class PizzaContactWebSessionBean extends BasePizzaController implements P
                throw new UnsupportedOperationException("Setting controller type is not supported."); //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for country instance
+        * <p>
+        * @return Country instance
+        */
        public Country getCountry () {
                return this.country;
        }
 
-       @Override
+       /**
+        * Setter for country instance
+        * <p>
+        * @param country Country instance
+        */
        public void setCountry (final Country country) {
                this.country = country;
        }
@@ -660,167 +707,300 @@ public class PizzaContactWebSessionBean extends BasePizzaController implements P
                return this.emailAddress;
        }
 
-       @Override
+       /**
+        * Setter for email address
+        * <p>
+        * @param emailAddress Email address
+        */
        public void setEmailAddress (final String emailAddress) {
                this.emailAddress = emailAddress;
        }
 
-       @Override
+       /**
+        * Getter for email address, repeated
+        * <p>
+        * @return the emailAddress, repeated
+        */
        public String getEmailAddressRepeat () {
                return this.emailAddressRepeat;
        }
 
-       @Override
+       /**
+        * Setter for email address repeated
+        * <p>
+        * @param emailAddressRepeat the emailAddress to set
+        */
        public void setEmailAddressRepeat (final String emailAddressRepeat) {
                this.emailAddressRepeat = emailAddressRepeat;
        }
 
-       @Override
+       /**
+        * Family name
+        * <p>
+        * @return the familyName
+        */
        public String getFamilyName () {
                return this.familyName;
        }
 
-       @Override
+       /**
+        * Family name
+        * <p>
+        * @param familyName the familyName to set
+        */
        public void setFamilyName (final String familyName) {
                this.familyName = familyName;
        }
 
-       @Override
+       /**
+        * Getter for fax number's area code
+        * <p>
+        * @return Fax number's area code
+        */
        public Integer getFaxAreaCode () {
                return this.faxAreaCode;
        }
 
-       @Override
+       /**
+        * Setter for fax number's area code
+        * <p>
+        * @param faxAreaCode Fax number's area code
+        */
        public void setFaxAreaCode (final Integer faxAreaCode) {
                this.faxAreaCode = faxAreaCode;
        }
 
-       @Override
+       /**
+        * Getter for fax's country instance
+        * <p>
+        * @return Fax' country instance
+        */
        public Country getFaxCountry () {
                return this.faxCountry;
        }
 
-       @Override
+       /**
+        * Setter for fax's country instance
+        * <p>
+        * @param faxCountry Fax' country instance
+        */
        public void setFaxCountry (final Country faxCountry) {
                this.faxCountry = faxCountry;
        }
 
-       @Override
+       /**
+        * Getter for fax number
+        * <p>
+        * @return Fax number
+        */
        public Long getFaxNumber () {
                return this.faxNumber;
        }
 
-       @Override
+       /**
+        * Setter for fax number
+        * <p>
+        * @param faxNumber Fax number
+        */
        public void setFaxNumber (final Long faxNumber) {
                this.faxNumber = faxNumber;
        }
 
-       @Override
+       /**
+        * First name
+        * <p>
+        * @return the first name
+        */
        public String getFirstName () {
                return this.firstName;
        }
 
-       @Override
+       /**
+        * First name
+        * <p>
+        * @param firstName the first name to set
+        */
        public void setFirstName (final String firstName) {
                this.firstName = firstName;
        }
 
-       @Override
+       /**
+        * House number
+        * <p>
+        * @return the houseNumber
+        */
        public Short getHouseNumber () {
                return this.houseNumber;
        }
 
-       @Override
+       /**
+        * House number
+        * <p>
+        * @param houseNumber the houseNumber to set
+        */
        public void setHouseNumber (final Short houseNumber) {
                this.houseNumber = houseNumber;
        }
 
-       @Override
+       /**
+        * Getter for house number extension, example: 123a 'a' is then the
+        * extension and 123 is the house number.
+        * <p>
+        * @return House number extension
+        */
        public String getHouseNumberExtension () {
                return this.houseNumberExtension;
        }
 
-       @Override
+       /**
+        * Setter for house number extension
+        * <p>
+        * @param houseNumberExtension House number extension
+        */
        public void setHouseNumberExtension (final String houseNumberExtension) {
                this.houseNumberExtension = houseNumberExtension;
        }
 
-       @Override
+       /**
+        * Getter for land-line number's area code
+        * <p>
+        * @return Land-line number's area code
+        */
        public Integer getLandLineAreaCode () {
                return this.landLineAreaCode;
        }
 
-       @Override
+       /**
+        * Setter for land-line number's area code
+        * <p>
+        * @param landLineAreaCode Land-line number's area code
+        */
        public void setLandLineAreaCode (final Integer landLineAreaCode) {
                this.landLineAreaCode = landLineAreaCode;
        }
 
-       @Override
+       /**
+        * Getter for land-line number's country instance
+        * <p>
+        * @return Land-line number's country instance
+        */
        public Country getLandLineCountry () {
                return this.landLineCountry;
        }
 
-       @Override
+       /**
+        * Setter for land-line number's country instance
+        * <p>
+        * @param landLineCountry Land-line number's country instance
+        */
        public void setLandLineCountry (final Country landLineCountry) {
                this.landLineCountry = landLineCountry;
        }
 
-       @Override
+       /**
+        * Getter for land-line number
+        * <p>
+        * @return Land-line number
+        */
        public Long getLandLineNumber () {
                return this.landLineNumber;
        }
 
-       @Override
+       /**
+        * Setter for land-line number
+        * <p>
+        * @param landLineNumber Land-line number
+        */
        public void setLandLineNumber (final Long landLineNumber) {
                this.landLineNumber = landLineNumber;
        }
 
-       @Override
+       /**
+        * Getter for mobile number
+        * <p>
+        * @return Mobile number
+        */
        public Long getMobileNumber () {
                return this.mobileNumber;
        }
 
-       @Override
+       /**
+        * Setter for mobile number
+        * <p>
+        * @param mobileNumber Mobile number
+        */
        public void setMobileNumber (final Long mobileNumber) {
                this.mobileNumber = mobileNumber;
        }
 
-       @Override
+       /**
+        * Getter for mobile number's carrier
+        * <p>
+        * @return Mobile number's carrier
+        */
        public MobileProvider getMobileProvider () {
                return this.mobileProvider;
        }
 
-       @Override
+       /**
+        * Setter for mobile number's provider
+        * <p>
+        * @param mobileProvider Mobile number's provider
+        */
        public void setMobileProvider (final MobileProvider mobileProvider) {
                this.mobileProvider = mobileProvider;
        }
 
-       @Override
+       /**
+        * Getter for personal title
+        * <p>
+        * @return Personal title
+        */
        public PersonalTitle getPersonalTitle () {
                return this.personalTitle;
        }
 
-       @Override
+       /**
+        * Setter for personal title
+        * <p>
+        * @param personalTitle Personal title
+        */
        public void setPersonalTitle (final PersonalTitle personalTitle) {
                this.personalTitle = personalTitle;
        }
 
-       @Override
+       /**
+        * Getter for street
+        * <p>
+        * @return Street
+        */
        public String getStreet () {
                return this.street;
        }
 
-       @Override
+       /**
+        * Setter for street
+        * <p>
+        * @param street Street
+        */
        public void setStreet (final String street) {
                this.street = street;
        }
 
-       @Override
+       /**
+        * Getter for ZIP code
+        * <p>
+        * @return ZIP code
+        */
        public Integer getZipCode () {
                return this.zipCode;
        }
 
-       @Override
+       /**
+        * Setter for ZIP code
+        * <p>
+        * @param zipCode ZIP code
+        */
        public void setZipCode (final Integer zipCode) {
                this.zipCode = zipCode;
        }
@@ -1021,8 +1201,7 @@ public class PizzaContactWebSessionBean extends BasePizzaController implements P
                this.setCountry(null);
 
                // - contact data
-               this.setEmailAddress(null);
-               this.setEmailAddressRepeat(null);
+               this.clearEmailAddresses();
                this.setLandLineAreaCode(null);
                this.setLandLineCountry(null);
                this.setLandLineNumber(null);
index 4f7f5723ceb2eacae7ab75d28cfbf4ec576bb95d..4a79b73e1ba104e704aa5895eb1c490396918bf7 100644 (file)
 package org.mxchange.pizzaapplication.beans.contact;
 
 import java.io.Serializable;
-import java.util.Date;
 import java.util.List;
-import javax.ejb.Local;
 import org.mxchange.jcontacts.contact.Contact;
-import org.mxchange.jcontacts.contact.title.PersonalTitle;
 import org.mxchange.jcontacts.exceptions.ContactNotFoundException;
-import org.mxchange.jcountry.data.Country;
-import org.mxchange.jphone.phonenumbers.mobileprovider.MobileProvider;
 
 /**
  * An interface for user beans
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaContactWebSessionController extends Serializable {
 
        /**
@@ -39,6 +33,18 @@ public interface PizzaContactWebSessionController extends Serializable {
         */
        public static final Integer MINIMUM_PASSWORD_LENGTH = 5;
 
+       /**
+        * Getter for email address
+        * <p>
+        * @return Email address
+        */
+       String getEmailAddress ();
+
+       /**
+        * Clears both email address field
+        */
+       void clearEmailAddresses ();
+
        /**
         * Returns a list of all found contacts
         * <p>
@@ -72,315 +78,6 @@ public interface PizzaContactWebSessionController extends Serializable {
         */
        Contact createContactInstance ();
 
-       /**
-        * Getter for birth day
-        * <p>
-        * @return Birth day
-        */
-       Date getBirthday ();
-
-       /**
-        * Setter for birth day
-        * <p>
-        * @param birthday Birth day
-        */
-       void setBirthday (final Date birthday);
-
-       /**
-        * Getter for mobile number's carrier
-        * <p>
-        * @return Mobile number's carrier
-        */
-       MobileProvider getMobileProvider ();
-
-       /**
-        * Setter for mobile number's provider
-        * <p>
-        * @param mobileProvider Mobile number's provider
-        */
-       void setMobileProvider (final MobileProvider mobileProvider);
-
-       /**
-        * Getter for mobile number
-        * <p>
-        * @return Mobile number
-        */
-       Long getMobileNumber ();
-
-       /**
-        * Setter for mobile number
-        * <p>
-        * @param mobileNumber Mobile number
-        */
-       void setMobileNumber (final Long mobileNumber);
-
-       /**
-        * City
-        * <p>
-        * @return the city
-        */
-       String getCity ();
-
-       /**
-        * City
-        * <p>
-        * @param city the city to set
-        */
-       void setCity (final String city);
-
-       /**
-        * Getter for comments
-        * <p>
-        * @return Comments
-        */
-       String getComment ();
-
-       /**
-        * Setter for comment
-        * <p>
-        * @param comment Comments
-        */
-       void setComment (final String comment);
-
-       /**
-        * Getter for country instance
-        * <p>
-        * @return Country instance
-        */
-       Country getCountry ();
-
-       /**
-        * Setter for country instance
-        * <p>
-        * @param country Country instance
-        */
-       void setCountry (final Country country);
-
-       /**
-        * Getter for email address
-        * <p>
-        * @return Email address
-        */
-       String getEmailAddress ();
-
-       /**
-        * Setter for email address
-        * <p>
-        * @param emailAddress Email address
-        */
-       void setEmailAddress (final String emailAddress);
-
-       /**
-        * Getter for email address, repeated
-        * <p>
-        * @return the emailAddress, repeated
-        */
-       String getEmailAddressRepeat ();
-
-       /**
-        * Setter for email address repeated
-        * <p>
-        * @param emailAddressRepeat the emailAddress to set
-        */
-       void setEmailAddressRepeat (final String emailAddressRepeat);
-
-       /**
-        * Family name
-        * <p>
-        * @return the familyName
-        */
-       String getFamilyName ();
-
-       /**
-        * Family name
-        * <p>
-        * @param familyName the familyName to set
-        */
-       void setFamilyName (final String familyName);
-
-       /**
-        * Getter for fax number's area code
-        * <p>
-        * @return Fax number's area code
-        */
-       Integer getFaxAreaCode ();
-
-       /**
-        * Setter for fax number's area code
-        * <p>
-        * @param faxAreaCode Fax number's area code
-        */
-       void setFaxAreaCode (final Integer faxAreaCode);
-
-       /**
-        * Getter for fax's country instance
-        * <p>
-        * @return Fax' country instance
-        */
-       Country getFaxCountry ();
-
-       /**
-        * Setter for fax's country instance
-        * <p>
-        * @param faxCountry Fax' country instance
-        */
-       void setFaxCountry (final Country faxCountry);
-
-       /**
-        * Getter for fax number
-        * <p>
-        * @return Fax number
-        */
-       Long getFaxNumber ();
-
-       /**
-        * Setter for fax number
-        * <p>
-        * @param faxNumber Fax number
-        */
-       void setFaxNumber (final Long faxNumber);
-
-       /**
-        * First name
-        * <p>
-        * @return the first name
-        */
-       String getFirstName ();
-
-       /**
-        * First name
-        * <p>
-        * @param firstName the first name to set
-        */
-       void setFirstName (final String firstName);
-
-       /**
-        * Getter for personal title
-        * <p>
-        * @return Personal title
-        */
-       PersonalTitle getPersonalTitle ();
-
-       /**
-        * Setter for personal title
-        * <p>
-        * @param personalTitle Personal title
-        */
-       void setPersonalTitle (final PersonalTitle personalTitle);
-
-       /**
-        * House number
-        * <p>
-        * @return the houseNumber
-        */
-       Short getHouseNumber ();
-
-       /**
-        * House number
-        * <p>
-        * @param houseNumber the houseNumber to set
-        */
-       void setHouseNumber (final Short houseNumber);
-
-       /**
-        * Getter for house number extension, example: 123a 'a' is then the
-        * extension and 123 is the house number.
-        * <p>
-        * @return House number extension
-        */
-       String getHouseNumberExtension ();
-
-       /**
-        * Setter for house number extension
-        * <p>
-        * @param houseNumberExtension House number extension
-        */
-       void setHouseNumberExtension (final String houseNumberExtension);
-
-       /**
-        * Getter for phone number's area code
-        * <p>
-        * @return Phone number's area code
-        */
-       Integer getLandLineAreaCode ();
-
-       /**
-        * Setter for phone number's area code
-        * <p>
-        * @param phoneAreaCode Phone number's area code
-        */
-       void setLandLineAreaCode (final Integer phoneAreaCode);
-
-       /**
-        * Getter for phone number's country instance
-        * <p>
-        * @return Phone number's country instance
-        */
-       Country getLandLineCountry ();
-
-       /**
-        * Setter for phone number's country instance
-        * <p>
-        * @param phoneCountry Phone number's country instance
-        */
-       void setLandLineCountry (final Country phoneCountry);
-
-       /**
-        * Getter for phone number
-        * <p>
-        * @return Phone number
-        */
-       Long getLandLineNumber ();
-
-       /**
-        * Setter for phone number
-        * <p>
-        * @param phoneNumber Phone number
-        */
-       void setLandLineNumber (final Long phoneNumber);
-
-       /**
-        * Street
-        * <p>
-        * @return the street
-        */
-       String getStreet ();
-
-       /**
-        * Street
-        * <p>
-        * @param street the street to set
-        */
-       void setStreet (final String street);
-
-       /**
-        * Getter for academic title
-        * <p>
-        * @return Academic title
-        */
-       String getAcademicTitle ();
-
-       /**
-        * Setter for academic title
-        * <p>
-        * @param academicTitle Academic title
-        */
-       void setAcademicTitle (final String academicTitle);
-
-       /**
-        * ZIP code
-        * <p>
-        * @return the zipCode
-        */
-       Integer getZipCode ();
-
-       /**
-        * ZIP code
-        * <p>
-        * @param zipCode the zipCode to set
-        */
-       void setZipCode (final Integer zipCode);
-
        /**
         * Getter for controller type
         * <p>
index f42d84844205d74d763908eeabbf7701aae2b77b..e2f4e4346987fecb7749ca8a76b72eef372747e7 100644 (file)
@@ -137,6 +137,46 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
        @Any
        private Event<ObservableAdminUnlinkedMobileNumberEvent> mobileNumberUnlinkedEvent;
 
+       /**
+        * Area code (city dial prefix) for fax number
+        */
+       private Integer faxAreaCode;
+
+       /**
+        * Country (for dial prefix) for fax number
+        */
+       private Country faxCountry;
+
+       /**
+        * Fax number
+        */
+       private Long faxNumber;
+
+       /**
+        * Area code (city dial prefix) for land-line number
+        */
+       private Integer landLineAreaCode;
+
+       /**
+        * Country (for dial prefix) for land-line number
+        */
+       private Country landLineCountry;
+
+       /**
+        * Land-line number
+        */
+       private Long landLineNumber;
+
+       /**
+        * Mobile provider
+        */
+       private MobileProvider mobileProvider;
+
+       /**
+        * Mobile number
+        */
+       private Long mobileNumber;
+
        /**
         * Default constructor
         */
@@ -224,8 +264,15 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
                this.clear();
        }
 
-       @Override
-       public String doLinkAddFaxNumber (final Contact contact) {
+       /**
+        * Links fax number to contact from bean helper as "main fax number".
+        * <p>
+        * @return Redirect outcome
+        */
+       public String doLinkMainFaxNumber () {
+               // Get contact from helper
+               Contact contact = this.beanHelper.getContact();
+
                // Is all data properly set?
                if (null == contact) {
                        // Throw NPE
@@ -236,58 +283,60 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
                } else if (contact.getContactId() < 1) {
                        // Is not valid
                        throw new IllegalArgumentException(MessageFormat.format("contact.contactId={0} is not valid", contact.getContactId())); //NOI18N
-               } else if ((this.adminPhoneController.getFaxNumber() == null) && (this.adminPhoneController.getPhoneCountry() == null) && ((this.adminPhoneController.getPhoneAreaCode() == null) || (this.adminPhoneController.getPhoneAreaCode() < 1)) && ((this.adminPhoneController.getPhoneNumber() == null) || (this.adminPhoneController.getPhoneNumber() < 1))) {
-                       // All fields are empty
-                       throw new NullPointerException("faxNumber, mobileProvider and phoneNumer are null"); //NOI18N
-               } else if ((this.adminPhoneController.getFaxNumber() instanceof DialableFaxNumber) && (this.adminPhoneController.getPhoneCountry() instanceof Country) && (this.adminPhoneController.getPhoneAreaCode() instanceof Integer) && (this.adminPhoneController.getPhoneNumber() instanceof Long)) {
-                       // All fields are set
-                       throw new IllegalStateException("faxNumber, mobileProvider and phoneNumer are all set"); //NOI18N
-               } else if ((this.adminPhoneController.getPhoneCountry() == null) && ((this.adminPhoneController.getPhoneAreaCode() instanceof Integer) || (this.adminPhoneController.getPhoneNumber() instanceof Long))) {
-                       // No country selected
-                       this.showFacesMessage("form_add_contact_fax:phoneCountry", "ERROR_ADMIN_NO_PHONE_COUNTRY_SELECTED"); //NOI18N
-                       return ""; //NOI18N
-               } else if (((this.adminPhoneController.getPhoneAreaCode() == null) || (this.adminPhoneController.getPhoneAreaCode() < 1)) && ((this.adminPhoneController.getPhoneCountry() instanceof Country) || (this.adminPhoneController.getPhoneNumber() instanceof Long))) {
-                       // No area code entered
-                       this.showFacesMessage("form_add_contact_fax:phoneAreaCode", "ERROR_ADMIN_NO_PHONE_AREA_CODE_ENTERED"); //NOI18N
-                       return ""; //NOI18N
-               } else if (((this.adminPhoneController.getPhoneNumber() == null) || (this.adminPhoneController.getPhoneNumber() < 1)) && ((this.adminPhoneController.getPhoneCountry() instanceof Country) || (this.adminPhoneController.getPhoneAreaCode() instanceof Integer))) {
-                       // No phone number entered
-                       this.showFacesMessage("form_add_contact_fax:phoneNumber", "ERROR_ADMIN_NO_PHONE_NUMBER_ENTERED"); //NOI18N
-                       return ""; //NOI18N
+               } else if (this.getFaxCountry() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.faxCountry is null"); //NOI18N
+               } else if (this.getFaxCountry().getCountryId() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.faxCountry.countryId is null"); //NOI18N
+               } else if (this.getFaxCountry().getCountryId() < 1) {
+                       // Invalid id number
+                       throw new IllegalArgumentException(MessageFormat.format("this.faxCountry.countryId={0} is not valid.", this.getFaxCountry().getCountryId())); //NOI18N
+               } else if (this.getFaxAreaCode() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.faxAreaCode is null"); //NOI18N
+               } else if (this.getFaxAreaCode() < 1) {
+                       // Invalid id number
+                       throw new IllegalArgumentException(MessageFormat.format("this.faxAreaCode={0} is invalid", this.getFaxAreaCode())); //NOI18N
+               } else if (this.getFaxNumber() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.faxNumber is null"); //NOI18N
+               } else if (this.getFaxNumber() < 1) {
+                       // Invalid id number
+                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber={0} is invalid", this.getFaxNumber())); //NOI18N
                }
 
                // Init instances
                Contact updatedContact;
-               DialableFaxNumber faxNumber = this.adminPhoneController.getFaxNumber();
+               DialableFaxNumber number = new FaxNumber(this.getFaxCountry(), this.getFaxAreaCode(), this.getFaxNumber());
 
                // Try it again
                try {
-                       // All fine so far, then which was choosen?
-                       if (faxNumber instanceof DialableFaxNumber) {
-                               // User has choosen number
-                               updatedContact = this.adminPhoneBean.linkExistingFaxNumberWithContact(contact, faxNumber);
-                       } else {
-                               // Create new instance
-                               faxNumber = new FaxNumber(this.adminPhoneController.getPhoneCountry(), this.adminPhoneController.getPhoneAreaCode(), this.adminPhoneController.getPhoneNumber());
-
-                               // Link it, too
-                               updatedContact = this.adminPhoneBean.linkNewFaxNumberWithContact(contact, faxNumber);
-                       }
+                       // Link it, too
+                       updatedContact = this.adminPhoneBean.linkNewFaxNumberWithContact(contact, number);
                } catch (final PhoneNumberAlreadyLinkedException ex) {
                        // Throw again as cause
-                       this.showFacesMessage("form_add_contact_fax:phoneNumber", ex); //NOI18N
+                       this.showFacesMessage("form_add_contact_fax:faxNumber", ex); //NOI18N
                        return ""; //NOI18N
                }
 
                // Fire event
-               this.adminLinkedFaxNumberEvent.fire(new AdminLinkedFaxNumberEvent(updatedContact, faxNumber));
+               this.adminLinkedFaxNumberEvent.fire(new AdminLinkedFaxNumberEvent(updatedContact, number));
 
                // Return to contact profile
                return "admin_show_contact"; //NOI18N
        }
 
-       @Override
-       public String doLinkAddLandLineNumber (final Contact contact) {
+       /**
+        * Links land-line number to contact from bean helper as "main land-line
+        * number".
+        * <p>
+        * @return Redirect outcome
+        */
+       public String doLinkMainLandLineNumber () {
+               // Get contact from helper
+               Contact contact = this.beanHelper.getContact();
+
                // Is all data properly set?
                if (null == contact) {
                        // Throw NPE
@@ -298,58 +347,59 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
                } else if (contact.getContactId() < 1) {
                        // Is not valid
                        throw new IllegalArgumentException(MessageFormat.format("contact.contactId={0} is not valid", contact.getContactId())); //NOI18N
-               } else if ((this.adminPhoneController.getLandLineNumber() == null) && (this.adminPhoneController.getPhoneCountry() == null) && ((this.adminPhoneController.getPhoneAreaCode() == null) || (this.adminPhoneController.getPhoneAreaCode() < 1)) && ((this.adminPhoneController.getPhoneNumber() == null) || (this.adminPhoneController.getPhoneNumber() < 1))) {
-                       // All fields are empty
-                       throw new NullPointerException("landLineNumber, mobileProvider and phoneNumer are null"); //NOI18N
-               } else if ((this.adminPhoneController.getLandLineNumber() instanceof DialableLandLineNumber) && (this.adminPhoneController.getPhoneCountry() instanceof Country) && (this.adminPhoneController.getPhoneAreaCode() instanceof Integer) && (this.adminPhoneController.getPhoneNumber() instanceof Long)) {
-                       // All fields are set
-                       throw new IllegalStateException("landLineNumber, mobileProvider and phoneNumer are all set"); //NOI18N
-               } else if ((this.adminPhoneController.getPhoneCountry() == null) && ((this.adminPhoneController.getPhoneAreaCode() instanceof Integer) || (this.adminPhoneController.getPhoneNumber() instanceof Long))) {
-                       // No country selected
-                       this.showFacesMessage("form_add_contact_landLine:phoneCountry", "ERROR_ADMIN_NO_PHONE_COUNTRY_SELECTED"); //NOI18N
-                       return ""; //NOI18N
-               } else if (((this.adminPhoneController.getPhoneAreaCode() == null) || (this.adminPhoneController.getPhoneAreaCode() < 1)) && ((this.adminPhoneController.getPhoneCountry() instanceof Country) || (this.adminPhoneController.getPhoneNumber() instanceof Long))) {
-                       // No area code entered
-                       this.showFacesMessage("form_add_contact_landLine:phoneAreaCode", "ERROR_ADMIN_NO_PHONE_AREA_CODE_ENTERED"); //NOI18N
-                       return ""; //NOI18N
-               } else if (((this.adminPhoneController.getPhoneNumber() == null) || (this.adminPhoneController.getPhoneNumber() < 1)) && ((this.adminPhoneController.getPhoneCountry() instanceof Country) || (this.adminPhoneController.getPhoneAreaCode() instanceof Integer))) {
-                       // No phone number entered
-                       this.showFacesMessage("form_add_contact_landLine:phoneNumber", "ERROR_ADMIN_NO_PHONE_NUMBER_ENTERED"); //NOI18N
-                       return ""; //NOI18N
+               } else if (this.getLandLineCountry() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.landLineCountry is null"); //NOI18N
+               } else if (this.getLandLineCountry().getCountryId() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.landLineCountry.countryId is null"); //NOI18N
+               } else if (this.getLandLineCountry().getCountryId() < 1) {
+                       // Invalid id number
+                       throw new IllegalArgumentException(MessageFormat.format("this.landLineCountry.countryId={0} is not valid.", this.getLandLineCountry().getCountryId())); //NOI18N
+               } else if (this.getLandLineAreaCode() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.landLineAreaCode is null"); //NOI18N
+               } else if (this.getLandLineAreaCode() < 1) {
+                       // Invalid id number
+                       throw new IllegalArgumentException(MessageFormat.format("this.landLineAreaCode={0} is invalid", this.getLandLineAreaCode())); //NOI18N
+               } else if (this.getLandLineNumber() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.landLineNumber is null"); //NOI18N
+               } else if (this.getLandLineNumber() < 1) {
+                       // Invalid id number
+                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber={0} is invalid", this.getLandLineNumber())); //NOI18N
                }
 
                // Init instance
                Contact updatedContact;
-               DialableLandLineNumber landLineNumber = this.adminPhoneController.getLandLineNumber();
+               DialableLandLineNumber number = new LandLineNumber(this.getLandLineCountry(), this.getLandLineAreaCode(), this.getLandLineNumber());
 
                // Try it again
                try {
-                       // All fine so far, then which was choosen?
-                       if (landLineNumber instanceof DialableLandLineNumber) {
-                               // User has choosen number
-                               updatedContact = this.adminPhoneBean.linkExistingLandLineNumberWithContact(contact, landLineNumber);
-                       } else {
-                               // Create new instance
-                               landLineNumber = new LandLineNumber(this.adminPhoneController.getPhoneCountry(), this.adminPhoneController.getPhoneAreaCode(), this.adminPhoneController.getPhoneNumber());
-
-                               // Link it, too
-                               updatedContact = this.adminPhoneBean.linkNewLandLineNumberWithContact(contact, landLineNumber);
-                       }
+                       // Link it, too
+                       updatedContact = this.adminPhoneBean.linkNewLandLineNumberWithContact(contact, number);
                } catch (final PhoneNumberAlreadyLinkedException ex) {
                        // Throw again as cause
-                       this.showFacesMessage("form_add_contact_landLine:phoneNumber", ex); //NOI18N
+                       this.showFacesMessage("form_add_contact_landLine:landLineNumber", ex); //NOI18N
                        return ""; //NOI18N
                }
 
                // Fire event
-               this.adminLinkedLandLineNumberEvent.fire(new AdminLinkedLandLineNumberEvent(updatedContact, landLineNumber));
+               this.adminLinkedLandLineNumberEvent.fire(new AdminLinkedLandLineNumberEvent(updatedContact, number));
 
                // Return to contact profile
                return "admin_show_contact"; //NOI18N
        }
 
-       @Override
-       public String doLinkAddMobileNumber (final Contact contact) {
+       /**
+        * Links mobile number to contact from bean helper as "main mobile number".
+        * <p>
+        * @return Redirect outcome
+        */
+       public String doLinkMainMobileNumber () {
+               // Get contact from helper
+               Contact contact = this.beanHelper.getContact();
+
                // Is all data properly set?
                if (null == contact) {
                        // Throw NPE
@@ -360,52 +410,188 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
                } else if (contact.getContactId() < 1) {
                        // Is not valid
                        throw new IllegalArgumentException(MessageFormat.format("contact.contactId={0} is not valid", contact.getContactId())); //NOI18N
-               } else if ((this.adminPhoneController.getMobileNumber() == null) && (this.adminPhoneController.getMobileProvider() == null) && ((this.adminPhoneController.getPhoneNumber() == null) || (this.adminPhoneController.getPhoneNumber() < 1))) {
-                       // All fields are empty
-                       throw new NullPointerException("mobileNumber, mobileProvider and phoneNumer are null"); //NOI18N
-               } else if ((this.adminPhoneController.getMobileNumber() instanceof DialableMobileNumber) && (this.adminPhoneController.getMobileProvider() instanceof MobileProvider) && (this.adminPhoneController.getPhoneNumber() instanceof Long)) {
-                       // All fields are set
-                       throw new IllegalStateException("mobileNumber, mobileProvider and phoneNumer are all set"); //NOI18N
-               } else if ((this.adminPhoneController.getMobileProvider() == null) && (this.adminPhoneController.getPhoneNumber() instanceof Long)) {
-                       // No mobile provider selected
-                       this.showFacesMessage("form_add_contact_mobile:phoneCountry", "ERROR_ADMIN_NO_PHONE_MOBILE_PROVIDER_SELECTED"); //NOI18N
-                       return ""; //NOI18N
-               } else if (((this.adminPhoneController.getPhoneNumber() == null) || (this.adminPhoneController.getPhoneNumber() < 1)) && (this.adminPhoneController.getMobileProvider() instanceof MobileProvider)) {
-                       // No phone number entered
-                       this.showFacesMessage("form_add_contact_mobile:phoneNumber", "ERROR_ADMIN_NO_PHONE_NUMBER_ENTERED"); //NOI18N
-                       return ""; //NOI18N
+               } else if (this.getMobileProvider() == null) {
+                       // Throw NPE
+                       throw new NullPointerException("this.mobileProvider is null"); //NOI18N
+               } else if (this.getMobileProvider().getProviderId() == null) {
+                       // Throw NPE
+                       throw new NullPointerException("this.mobileProvider.providerId is null"); //NOI18N
+               } else if (this.getMobileProvider().getProviderId() < 1) {
+                       // Throw NPE
+                       throw new NullPointerException(MessageFormat.format("this.mobileProvider.providerId={0} is invalid", this.getMobileProvider().getProviderId())); //NOI18N
+               } else if (this.getMobileNumber() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("this.mobileNumber is null"); //NOI18N
+               } else if (this.getMobileNumber() < 1) {
+                       // Invalid id number
+                       throw new IllegalArgumentException(MessageFormat.format("this.mobileNumber={0} is invalid", this.getMobileNumber())); //NOI18N
                }
 
                // Init instance
                Contact updatedContact;
-               DialableMobileNumber mobileNumber = this.adminPhoneController.getMobileNumber();
+               DialableMobileNumber number = new MobileNumber(this.getMobileProvider(), this.getMobileNumber());
 
                // Try it again
                try {
-                       // All fine so far, then which was choosen?
-                       if (mobileNumber instanceof DialableMobileNumber) {
-                               // User has choosen number
-                               updatedContact = this.adminPhoneBean.linkExistingMobileNumberWithContact(contact, mobileNumber);
-                       } else {
-                               // Create new instance
-                               mobileNumber = new MobileNumber(this.adminPhoneController.getMobileProvider(), this.adminPhoneController.getPhoneNumber());
-
-                               // Link it, too
-                               updatedContact = this.adminPhoneBean.linkNewMobileNumberWithContact(contact, mobileNumber);
-                       }
+                       // Link it, too
+                       updatedContact = this.adminPhoneBean.linkNewMobileNumberWithContact(contact, number);
                } catch (final PhoneNumberAlreadyLinkedException ex) {
                        // Throw again as cause
-                       this.showFacesMessage("form_add_contact_mobile:phoneNumber", ex); //NOI18N
+                       this.showFacesMessage("form_add_contact_mobile:mobileNumber", ex); //NOI18N
                        return ""; //NOI18N
                }
 
                // Fire event
-               this.adminLinkedMobileNumberEvent.fire(new AdminLinkedMobileNumberEvent(updatedContact, mobileNumber));
+               this.adminLinkedMobileNumberEvent.fire(new AdminLinkedMobileNumberEvent(updatedContact, number));
 
                // Return to contact profile
                return "admin_show_contact"; //NOI18N
        }
 
+       /**
+        * Getter for fax area code
+        * <p>
+        * @return Fax area code
+        */
+       public Integer getFaxAreaCode () {
+               return this.faxAreaCode;
+       }
+
+       /**
+        * Setter for fax area code
+        * <p>
+        * @param faxAreaCode Fax area code
+        */
+       public void setFaxAreaCode (final Integer faxAreaCode) {
+               this.faxAreaCode = faxAreaCode;
+       }
+
+       /**
+        * Getter for fax numbers country
+        * <p>
+        * @return Fax numbers country
+        */
+       public Country getFaxCountry () {
+               return this.faxCountry;
+       }
+
+       /**
+        * Setter for fax numbers country
+        * <p>
+        * @param faxCountry Fax numbers country
+        */
+       public void setFaxCountry (final Country faxCountry) {
+               this.faxCountry = faxCountry;
+       }
+
+       /**
+        * Getter for fax number
+        * <p>
+        * @return Fax number
+        */
+       public Long getFaxNumber () {
+               return this.faxNumber;
+       }
+
+       /**
+        * Setter for fax number
+        * <p>
+        * @param faxNumber Fax number
+        */
+       public void setFaxNumber (final Long faxNumber) {
+               this.faxNumber = faxNumber;
+       }
+
+       /**
+        * Getter for land-line area code
+        * <p>
+        * @return Land-line area code
+        */
+       public Integer getLandLineAreaCode () {
+               return this.landLineAreaCode;
+       }
+
+       /**
+        * Setter for land-line area code
+        * <p>
+        * @param landLineAreaCode Land-line area code
+        */
+       public void setLandLineAreaCode (final Integer landLineAreaCode) {
+               this.landLineAreaCode = landLineAreaCode;
+       }
+
+       /**
+        * Getter for land-line country
+        * <p>
+        * @return Land-line country
+        */
+       public Country getLandLineCountry () {
+               return this.landLineCountry;
+       }
+
+       /**
+        * Setter for land-line country
+        * <p>
+        * @param landLineCountry Land-line country
+        */
+       public void setLandLineCountry (final Country landLineCountry) {
+               this.landLineCountry = landLineCountry;
+       }
+
+       /**
+        * Getter for land-line number
+        * <p>
+        * @return Land-line number
+        */
+       public Long getLandLineNumber () {
+               return this.landLineNumber;
+       }
+
+       /**
+        * Setter for land-line number
+        * <p>
+        * @param landLineNumber Land-line number
+        */
+       public void setLandLineNumber (final Long landLineNumber) {
+               this.landLineNumber = landLineNumber;
+       }
+
+       /**
+        * Getter for mobile provider
+        * <p>
+        * @return Mobile provider
+        */
+       public MobileProvider getMobileProvider () {
+               return this.mobileProvider;
+       }
+
+       /**
+        * Setter for mobile provider
+        * <p>
+        * @param mobileProvider Mobile provider
+        */
+       public void setMobileProvider (final MobileProvider mobileProvider) {
+               this.mobileProvider = mobileProvider;
+       }
+
+       /**
+        * Getter for mobile number
+        * <p>
+        * @return Mobile number
+        */
+       public Long getMobileNumber () {
+               return this.mobileNumber;
+       }
+
+       /**
+        * Setter for mobile number
+        * <p>
+        * @param mobileNumber Mobile number
+        */
+       public void setMobileNumber (final Long mobileNumber) {
+               this.mobileNumber = mobileNumber;
+       }
+
        /**
         * Post-construction method
         */
@@ -424,7 +610,11 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
                }
        }
 
-       @Override
+       /**
+        * Unlinks fax data with current contact
+        * <p>
+        * @return Redirect outcome
+        */
        public String unlinkFaxContactData () {
                // Is all data set
                if (this.beanHelper.getFaxNumber() == null) {
@@ -472,7 +662,11 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
                return "admin_show_contact"; //NOI18N
        }
 
-       @Override
+       /**
+        * Unlinks land-line data with current contact
+        * <p>
+        * @return Redirect outcome
+        */
        public String unlinkLandLineContactData () {
                // Is all data set
                if (this.beanHelper.getLandLineNumber() == null) {
@@ -520,7 +714,11 @@ public class PizzaAdminContactPhoneWebRequestBean extends BasePizzaController im
                return "admin_show_contact"; //NOI18N
        }
 
-       @Override
+       /**
+        * Unlinks mobile data with current contact
+        * <p>
+        * @return Redirect outcome
+        */
        public String unlinkMobileContactData () {
                // Is all data set
                if (this.beanHelper.getMobileNumber() == null) {
index da30752f639f96d88dafb4e62906ce0321abf2e1..22e1651f40484bb702d8baa1991b3f1886284bdf 100644 (file)
 package org.mxchange.pizzaapplication.beans.contact.phone;
 
 import java.io.Serializable;
-import javax.ejb.Local;
-import org.mxchange.jcontacts.contact.Contact;
 
 /**
  * An interface for user beans
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaAdminContactPhoneWebRequestController extends Serializable {
 
-       /**
-        * Unlinks fax data with current contact
-        * <p>
-        * @return Redirect outcome
-        */
-       String unlinkFaxContactData ();
-
-       /**
-        * Unlinks land-line data with current contact
-        * <p>
-        * @return Redirect outcome
-        */
-       String unlinkLandLineContactData ();
-
-       /**
-        * Unlinks mobile data with current contact
-        * <p>
-        * @return Redirect outcome
-        */
-       String unlinkMobileContactData ();
-
-       /**
-        * Links given contact instance with set fax number or entered data. If both
-        * is set, a proper exception is thrown as this is an invalid state.
-        * <p>
-        * @param contact Contact instance to link with existing/new data
-        * <p>
-        * @return Redirect outcome
-        */
-       String doLinkAddFaxNumber (final Contact contact);
-
-       /**
-        * Links given contact instance with set land-line number or entered data.
-        * If both is set, a proper exception is thrown as this is an invalid state.
-        * <p>
-        * @param contact Contact instance to link with existing/new data
-        * <p>
-        * @return Redirect outcome
-        */
-       String doLinkAddLandLineNumber (final Contact contact);
-
-       /**
-        * Links given contact instance with set mobile or entered data. If both is
-        * set, a proper exception is thrown as this is an invalid state.
-        * <p>
-        * @param contact Contact instance to link with existing/new data
-        * <p>
-        * @return Redirect outcome
-        */
-       String doLinkAddMobileNumber (final Contact contact);
-
 }
index 4825ef6701853ece2fb441b2e42a3b74aca93a12..21f4f8c1f0fa1fcf27f6e2c220aac0a144a70eba 100644 (file)
@@ -18,7 +18,6 @@ package org.mxchange.pizzaapplication.beans.contact.phone;
 
 import java.io.Serializable;
 import java.util.List;
-import javax.ejb.Local;
 import org.mxchange.jcontacts.contact.Contact;
 
 /**
@@ -26,7 +25,6 @@ import org.mxchange.jcontacts.contact.Contact;
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaContactPhoneWebSessionController extends Serializable {
 
        /**
index 80d8f126f59cba024716274d79a323c907d3da78..bd276b212b0c0d00b54fcb1f54390fc8d9e8055f 100644 (file)
@@ -107,7 +107,14 @@ public class PizzaAdminCountryWebRequestBean extends BasePizzaController impleme
                super();
        }
 
-       @Override
+       /**
+        * Adds country to all relevant beans and sends it to the EJB. A redirect
+        * should happen after successfull creation.
+        * <p>
+        * @return Redirect outcome
+        * <p>
+        * @todo Add field validation
+        */
        public String addCountry () {
                // Create new country object
                Country country = new CountryData();
@@ -147,77 +154,115 @@ public class PizzaAdminCountryWebRequestBean extends BasePizzaController impleme
                return "admin_list_country"; //NOI18N
        }
 
-       @Override
-       public List<Country> allCountries () {
-               // Return "cached" version
-               return this.countryController.allCountries();
-       }
-
-       @Override
+       /**
+        * Getter for abroad dial prefix
+        * <p>
+        * @return Abroad dial prefix
+        */
        public String getCountryAbroadDialPrefix () {
                return this.countryAbroadDialPrefix;
        }
 
-       @Override
+       /**
+        * Setter for abroad dial prefix
+        * <p>
+        * @param countryAbroadDialPrefix Abroad dial prefix
+        */
        public void setCountryAbroadDialPrefix (final String countryAbroadDialPrefix) {
                this.countryAbroadDialPrefix = countryAbroadDialPrefix;
        }
 
-       @Override
+       /**
+        * Getter for 2-characters country code
+        * <p>
+        * @return Country code
+        */
        public String getCountryCode () {
                return this.countryCode;
        }
 
-       @Override
+       /**
+        * Setter for 2-characters country code
+        * <p>
+        * @param countryCode Country code
+        */
        public void setCountryCode (final String countryCode) {
                this.countryCode = countryCode;
        }
 
-       @Override
+       /**
+        * Getter for external dial prefix
+        * <p>
+        * @return External dial prefix
+        */
        public String getCountryExternalDialPrefix () {
                return this.countryExternalDialPrefix;
        }
 
-       @Override
+       /**
+        * Setter for external dial prefix
+        * <p>
+        * @param countryExternalDialPrefix External dial prefix
+        */
        public void setCountryExternalDialPrefix (final String countryExternalDialPrefix) {
                this.countryExternalDialPrefix = countryExternalDialPrefix;
        }
 
-       @Override
+       /**
+        * Getter for i18n key for country name
+        * <p>
+        * @return i18n key for country name
+        */
        public String getCountryI18nKey () {
                return this.countryI18nKey;
        }
 
-       @Override
+       /**
+        * Setter for i18n key for country name
+        * <p>
+        * @param countryI18nKey i18n key for country name
+        */
        public void setCountryI18nKey (final String countryI18nKey) {
                this.countryI18nKey = countryI18nKey;
        }
 
-       @Override
+       /**
+        * Getter for whether the local dial prefix is required for local calls
+        * <p>
+        * @return Whether the local dial prefix is required
+        */
        public Boolean getCountryIsLocalPrefixRequired () {
                return this.countryIsLocalPrefixRequired;
        }
 
-       @Override
+       /**
+        * Setter for whether the local dial prefix is required for local calls
+        * <p>
+        * @param countryIsLocalPrefixRequired Whether the local dial prefix is
+        *                                     required
+        */
        public void setCountryIsLocalPrefixRequired (final Boolean countryIsLocalPrefixRequired) {
                this.countryIsLocalPrefixRequired = countryIsLocalPrefixRequired;
        }
 
-       @Override
+       /**
+        * Getter for country code (example: 49 for Germany, 63 for Philippines)
+        * <p>
+        * @return Dial number without prefix
+        */
        public Short getCountryPhoneCode () {
                return this.countryPhoneCode;
        }
 
-       @Override
+       /**
+        * Setter for country code (example: 49 for Germany, 63 for Philippines)
+        * <p>
+        * @param countryPhoneCode Country code
+        */
        public void setCountryPhoneCode (final Short countryPhoneCode) {
                this.countryPhoneCode = countryPhoneCode;
        }
 
-       @Override
-       public boolean hasCountries () {
-               return (!this.allCountries().isEmpty());
-       }
-
        /**
         * Post-construction method
         */
index f19b3bb66dbcbcba8d1438d6bb55f41cced7a0bc..24f0b1c55241902e36c51e29b9c4f58e737e16d2 100644 (file)
@@ -17,8 +17,6 @@
 package org.mxchange.pizzaapplication.beans.country;
 
 import java.io.Serializable;
-import java.util.List;
-import org.mxchange.jcountry.data.Country;
 
 /**
  * An interface for administrative country beans
@@ -27,111 +25,4 @@ import org.mxchange.jcountry.data.Country;
  */
 public interface PizzaAdminCountryWebRequestController extends Serializable {
 
-       /**
-        * A list of all countries
-        * <p>
-        * @return All countries
-        */
-       List<Country> allCountries ();
-
-       /**
-        * Adds country to all relevant beans and sends it to the EJB. A redirect
-        * should happen after successfull creation.
-        * <p>
-        * @return Redirect outcome
-        */
-       String addCountry ();
-
-       /**
-        * Checks whether countries has been registered
-        * <p>
-        * @return Whether countries has been registered
-        */
-       boolean hasCountries ();
-
-       /**
-        * Getter for abroad dial prefix
-        * <p>
-        * @return Abroad dial prefix
-        */
-       String getCountryAbroadDialPrefix ();
-
-       /**
-        * Setter for abroad dial prefix
-        * <p>
-        * @param countryAbroadDialPrefix Abroad dial prefix
-        */
-       void setCountryAbroadDialPrefix (final String countryAbroadDialPrefix);
-
-       /**
-        * Getter for 2-characters country code
-        * <p>
-        * @return Country code
-        */
-       String getCountryCode ();
-
-       /**
-        * Setter for 2-characters country code
-        * <p>
-        * @param countryCode Country code
-        */
-       void setCountryCode (final String countryCode);
-
-       /**
-        * Getter for i18n key for country name
-        * <p>
-        * @return i18n key for country name
-        */
-       String getCountryI18nKey ();
-
-       /**
-        * Setter for i18n key for country name
-        * <p>
-        * @param countryI18nKey i18n key for country name
-        */
-       void setCountryI18nKey (final String countryI18nKey);
-
-       /**
-        * Getter for whether the local dial prefix is required for local calls
-        * <p>
-        * @return Whether the local dial prefix is required
-        */
-       Boolean getCountryIsLocalPrefixRequired ();
-
-       /**
-        * Setter for whether the local dial prefix is required for local calls
-        * <p>
-        * @param countryIsLocalPrefixRequired Whether the local dial prefix is
-        *                                     required
-        */
-       void setCountryIsLocalPrefixRequired (final Boolean countryIsLocalPrefixRequired);
-
-       /**
-        * Getter for external dial prefix
-        * <p>
-        * @return External dial prefix
-        */
-       String getCountryExternalDialPrefix ();
-
-       /**
-        * Setter for external dial prefix
-        * <p>
-        * @param countryExternalDialPrefix External dial prefix
-        */
-       void setCountryExternalDialPrefix (final String countryExternalDialPrefix);
-
-       /**
-        * Getter for country code (example: 49 for Germany, 63 for Philippines)
-        * <p>
-        * @return Dial number without prefix
-        */
-       Short getCountryPhoneCode ();
-
-       /**
-        * Setter for country code (example: 49 for Germany, 63 for Philippines)
-        * <p>
-        * @param countryPhoneCode Country code
-        */
-       void setCountryPhoneCode (final Short countryPhoneCode);
-
 }
index c4fc121a874d905c3f0c5c7dd46af47fd77eb6bd..27271f92024f20be9f2e03e26183f8e1c9acc133 100644 (file)
 package org.mxchange.pizzaapplication.beans.features;
 
 import java.io.Serializable;
-import javax.ejb.Local;
 
 /**
  * An interface for country beans
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaFeaturesWebApplicationController extends Serializable {
 
        /**
index 8710cd4e573a22baccdb400d39163991e66b65d8..2bcc41bc48b80ffd03e773341ecefd2b5870e908 100644 (file)
@@ -18,13 +18,23 @@ package org.mxchange.pizzaapplication.beans.helper;
 
 import java.text.MessageFormat;
 import javax.enterprise.context.RequestScoped;
+import javax.enterprise.event.Event;
+import javax.enterprise.inject.Any;
 import javax.inject.Inject;
 import javax.inject.Named;
 import org.mxchange.jcontacts.contact.Contact;
 import org.mxchange.jcustomercore.model.customer.Customer;
+import org.mxchange.jphone.events.helper.fax.created.HelperCreatedFaxNumberEvent;
+import org.mxchange.jphone.events.helper.fax.created.ObservableHelperCreatedFaxNumberEvent;
+import org.mxchange.jphone.events.helper.landline.created.HelperCreatedLandLineNumberEvent;
+import org.mxchange.jphone.events.helper.landline.created.ObservableHelperCreatedLandLineNumberEvent;
+import org.mxchange.jphone.events.helper.mobile.created.HelperCreatedMobileNumberEvent;
+import org.mxchange.jphone.events.helper.mobile.created.ObservableHelperCreatedMobileNumberEvent;
 import org.mxchange.jphone.phonenumbers.fax.DialableFaxNumber;
 import org.mxchange.jphone.phonenumbers.landline.DialableLandLineNumber;
 import org.mxchange.jphone.phonenumbers.mobile.DialableMobileNumber;
+import org.mxchange.jusercore.events.user.helper.created.HelperCreatedUserEvent;
+import org.mxchange.jusercore.events.user.helper.created.ObservableHelperCreatedUserEvent;
 import org.mxchange.jusercore.model.user.User;
 import org.mxchange.pizzaapplication.beans.contact.PizzaAdminContactWebRequestController;
 import org.mxchange.pizzaapplication.beans.customer.PizzaAdminCustomerWebRequestController;
@@ -90,16 +100,37 @@ public class PizzaWebRequestHelperBean implements PizzaWebRequestHelperControlle
         */
        private DialableFaxNumber faxNumber;
 
+       /**
+        * Event for when a fax number instance was created
+        */
+       @Any
+       @Inject
+       private Event<ObservableHelperCreatedFaxNumberEvent> faxNumberCreatedEvent;
+
        /**
         * Land-line number
         */
        private DialableLandLineNumber landLineNumber;
 
+       /**
+        * Event for when a land-line number instance was created
+        */
+       @Any
+       @Inject
+       private Event<ObservableHelperCreatedLandLineNumberEvent> landLineNumberCreatedEvent;
+
        /**
         * Mobile number
         */
        private DialableMobileNumber mobileNumber;
 
+       /**
+        * Event for when a mobile number instance was created
+        */
+       @Any
+       @Inject
+       private Event<ObservableHelperCreatedMobileNumberEvent> mobileNumberCreatedEvent;
+
        /**
         * User instance
         */
@@ -111,6 +142,13 @@ public class PizzaWebRequestHelperBean implements PizzaWebRequestHelperControlle
        @Inject
        private PizzaUserWebSessionController userController;
 
+       /**
+        * Event for when a user instance was created
+        */
+       @Any
+       @Inject
+       private Event<ObservableHelperCreatedUserEvent> userCreatedEvent;
+
        /**
         * Default constructor
         */
@@ -166,40 +204,38 @@ public class PizzaWebRequestHelperBean implements PizzaWebRequestHelperControlle
                // Validate fax instance
                if (this.getFaxNumber() == null) {
                        // Throw NPE
-                       throw new NullPointerException("this.faxNumber is null");
+                       throw new NullPointerException("this.faxNumber is null"); //NOI18N
                } else if (this.getFaxNumber().getPhoneId() == null) {
                        // Throw again
-                       throw new NullPointerException("this.faxNumber.phoneId is null");
+                       throw new NullPointerException("this.faxNumber.phoneId is null"); //NOI18N
                } else if (this.getFaxNumber().getPhoneId() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneId={0} is not valid", this.getFaxNumber().getPhoneId()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneId={0} is not valid", this.getFaxNumber().getPhoneId())); //NOI18N
                } else if (this.getFaxNumber().getPhoneAreaCode() == null) {
                        // Throw again
-                       throw new NullPointerException("this.faxNumber.phoneAreaCode is null");
+                       throw new NullPointerException("this.faxNumber.phoneAreaCode is null"); //NOI18N
                } else if (this.getFaxNumber().getPhoneAreaCode() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneAreaCode={0} is not valid", this.getFaxNumber().getPhoneAreaCode()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneAreaCode={0} is not valid", this.getFaxNumber().getPhoneAreaCode())); //NOI18N
                } else if (this.getFaxNumber().getPhoneCountry() == null) {
                        // Throw NPE again
-                       throw new NullPointerException("this.faxNumber.phoneCountry is null");
+                       throw new NullPointerException("this.faxNumber.phoneCountry is null"); //NOI18N
                } else if (this.getFaxNumber().getPhoneCountry().getCountryId() == null) {
                        // ... throw again
-                       throw new NullPointerException("this.faxNumber.phoneCountry.countryId is null");
+                       throw new NullPointerException("this.faxNumber.phoneCountry.countryId is null"); //NOI18N
                } else if (this.getFaxNumber().getPhoneCountry().getCountryId() < 1) {
                        // Invalid id
-                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneCountry.countryId={0} is invalid", this.getFaxNumber().getPhoneCountry().getCountryId()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneCountry.countryId={0} is invalid", this.getFaxNumber().getPhoneCountry().getCountryId())); //NOI18N
                } else if (this.getFaxNumber().getPhoneNumber() == null) {
                        // Throw NPE again ...
-                       throw new NullPointerException("this.faxNumber.phoneNumber is null");
+                       throw new NullPointerException("this.faxNumber.phoneNumber is null"); //NOI18N
                } else if (this.getFaxNumber().getPhoneNumber() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneNumber={0} is not valid", this.getFaxNumber().getPhoneNumber()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.faxNumber.phoneNumber={0} is not valid", this.getFaxNumber().getPhoneNumber())); //NOI18N
                }
 
-               // Copy all (changeable) data fields to admin controller
-               this.adminPhoneController.setPhoneAreaCode(this.getFaxNumber().getPhoneAreaCode());
-               this.adminPhoneController.setPhoneCountry(this.getFaxNumber().getPhoneCountry());
-               this.adminPhoneController.setPhoneNumber(this.getFaxNumber().getPhoneNumber());
+               // Fire event
+               this.faxNumberCreatedEvent.fire(new HelperCreatedFaxNumberEvent(this.getFaxNumber()));
        }
 
        @Override
@@ -207,40 +243,38 @@ public class PizzaWebRequestHelperBean implements PizzaWebRequestHelperControlle
                // Validate land-line instance
                if (this.getLandLineNumber() == null) {
                        // Throw NPE
-                       throw new NullPointerException("this.landLineNumber is null");
+                       throw new NullPointerException("this.landLineNumber is null"); //NOI18N
                } else if (this.getLandLineNumber().getPhoneId() == null) {
                        // Throw again
-                       throw new NullPointerException("this.landLineNumber.phoneId is null");
+                       throw new NullPointerException("this.landLineNumber.phoneId is null"); //NOI18N
                } else if (this.getLandLineNumber().getPhoneId() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneId={0} is not valid", this.getLandLineNumber().getPhoneId()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneId={0} is not valid", this.getLandLineNumber().getPhoneId())); //NOI18N
                } else if (this.getLandLineNumber().getPhoneAreaCode() == null) {
                        // Throw again
-                       throw new NullPointerException("this.landLineNumber.phoneAreaCode is null");
+                       throw new NullPointerException("this.landLineNumber.phoneAreaCode is null"); //NOI18N
                } else if (this.getLandLineNumber().getPhoneAreaCode() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneAreaCode={0} is not valid", this.getFaxNumber().getPhoneAreaCode()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneAreaCode={0} is not valid", this.getFaxNumber().getPhoneAreaCode())); //NOI18N
                } else if (this.getLandLineNumber().getPhoneCountry() == null) {
                        // Throw NPE again
-                       throw new NullPointerException("this.landLineNumber.phoneCountry is null");
+                       throw new NullPointerException("this.landLineNumber.phoneCountry is null"); //NOI18N
                } else if (this.getLandLineNumber().getPhoneCountry().getCountryId() == null) {
                        // ... throw again
-                       throw new NullPointerException("this.landLineNumber.phoneCountry.countryId is null");
+                       throw new NullPointerException("this.landLineNumber.phoneCountry.countryId is null"); //NOI18N
                } else if (this.getLandLineNumber().getPhoneCountry().getCountryId() < 1) {
                        // Invalid id
-                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneCountry.countryId={0} is invalid", this.getLandLineNumber().getPhoneCountry().getCountryId()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneCountry.countryId={0} is invalid", this.getLandLineNumber().getPhoneCountry().getCountryId())); //NOI18N
                } else if (this.getLandLineNumber().getPhoneNumber() == null) {
                        // Throw NPE again ...
-                       throw new NullPointerException("this.landLineNumber.phoneNumber is null");
+                       throw new NullPointerException("this.landLineNumber.phoneNumber is null"); //NOI18N
                } else if (this.getLandLineNumber().getPhoneNumber() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneNumber={0} is not valid", this.getLandLineNumber().getPhoneNumber()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.landLineNumber.phoneNumber={0} is not valid", this.getLandLineNumber().getPhoneNumber())); //NOI18N
                }
 
-               // Copy all (changeable) data fields to admin controller
-               this.adminPhoneController.setPhoneAreaCode(this.getLandLineNumber().getPhoneAreaCode());
-               this.adminPhoneController.setPhoneCountry(this.getLandLineNumber().getPhoneCountry());
-               this.adminPhoneController.setPhoneNumber(this.getLandLineNumber().getPhoneNumber());
+               // Fire event
+               this.landLineNumberCreatedEvent.fire(new HelperCreatedLandLineNumberEvent(this.getLandLineNumber()));
        }
 
        @Override
@@ -248,33 +282,32 @@ public class PizzaWebRequestHelperBean implements PizzaWebRequestHelperControlle
                // Validate mobile instance
                if (this.getMobileNumber() == null) {
                        // Throw NPE
-                       throw new NullPointerException("this.mobileNumber is null");
+                       throw new NullPointerException("this.mobileNumber is null"); //NOI18N
                } else if (this.getMobileNumber().getPhoneId() == null) {
                        // Throw again
-                       throw new NullPointerException("this.mobileNumber.phoneId is null");
+                       throw new NullPointerException("this.mobileNumber.phoneId is null"); //NOI18N
                } else if (this.getMobileNumber().getPhoneId() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.mobileNumber.phoneId={0} is not valid", this.getMobileNumber().getPhoneId()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.mobileNumber.phoneId={0} is not valid", this.getMobileNumber().getPhoneId())); //NOI18N
                } else if (this.getMobileNumber().getMobileProvider() == null) {
                        // Throw NPE again
-                       throw new NullPointerException("this.mobileNumber.mobileProvider is null");
+                       throw new NullPointerException("this.mobileNumber.mobileProvider is null"); //NOI18N
                } else if (this.getMobileNumber().getMobileProvider().getProviderId() == null) {
                        // ... throw again
-                       throw new NullPointerException("this.mobileNumber.mobileProvider.providerId is null");
+                       throw new NullPointerException("this.mobileNumber.mobileProvider.providerId is null"); //NOI18N
                } else if (this.getMobileNumber().getMobileProvider().getProviderId() < 1) {
                        // Invalid id
-                       throw new IllegalArgumentException(MessageFormat.format("this.mobileNumber.mobileProvider.providerId={0} is invalid", this.getMobileNumber().getMobileProvider().getProviderId()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.mobileNumber.mobileProvider.providerId={0} is invalid", this.getMobileNumber().getMobileProvider().getProviderId())); //NOI18N
                } else if (this.getMobileNumber().getPhoneNumber() == null) {
                        // Throw NPE again ...
-                       throw new NullPointerException("this.mobileNumber.phoneNumber is null");
+                       throw new NullPointerException("this.mobileNumber.phoneNumber is null"); //NOI18N
                } else if (this.getMobileNumber().getPhoneNumber() < 1) {
                        // Invalid id number
-                       throw new IllegalArgumentException(MessageFormat.format("this.mobileNumber.phoneNumber={0} is not valid", this.getMobileNumber().getPhoneNumber()));
+                       throw new IllegalArgumentException(MessageFormat.format("this.mobileNumber.phoneNumber={0} is not valid", this.getMobileNumber().getPhoneNumber())); //NOI18N
                }
 
-               // Copy all (changeable) data fields to admin controller
-               this.adminPhoneController.setMobileProvider(this.getMobileNumber().getMobileProvider());
-               this.adminPhoneController.setPhoneNumber(this.getMobileNumber().getPhoneNumber());
+               // Fire event
+               this.mobileNumberCreatedEvent.fire(new HelperCreatedMobileNumberEvent(this.getMobileNumber()));
        }
 
        @Override
@@ -300,8 +333,8 @@ public class PizzaWebRequestHelperBean implements PizzaWebRequestHelperControlle
                // Set all phone instances
                this.setPhoneInstances(userContact);
 
-               // Set all fields: user
-               this.userController.setUserName(this.getUser().getUserName());
+               // Fire event
+               this.userCreatedEvent.fire(new HelperCreatedUserEvent(this.getUser()));
        }
 
        @Override
index 7062d813d12b397a7e90f63e4e6e77246809fab2..6fe58d02137d2b039c7230abd94b55152a87fba8 100644 (file)
@@ -98,7 +98,15 @@ public class PizzaAdminMobileProviderWebRequestBean extends BasePizzaController
                super();
        }
 
-       @Override
+       /**
+        * Adds a mobile provider to database by calling the EJB. A pre-check on
+        * dial-prefix and country combination is done, if found, an exception is
+        * thrown. A redirect should take place after successfull creation.
+        * <p>
+        * @return Redirect outcome
+        * <p>
+        * @todo Add field validation
+        */
        public String addMobileProvider () {
                // Create mobile provider instance
                MobileProvider mobileProvider = new CellphoneProvider(this.getProviderDialPrefix(), this.getProviderName(), this.getProviderCountry(), this.getProviderMailPattern());
@@ -131,42 +139,74 @@ public class PizzaAdminMobileProviderWebRequestBean extends BasePizzaController
                return "admin_list_mobile_provider"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for country instance ('s dial data)
+        * <p>
+        * @return Country instance
+        */
        public Country getProviderCountry () {
                return this.providerCountry;
        }
 
-       @Override
+       /**
+        * Setter for country instance ('s dial data)
+        * <p>
+        * @param providerCountry Country instance
+        */
        public void setProviderCountry (final Country providerCountry) {
                this.providerCountry = providerCountry;
        }
 
-       @Override
+       /**
+        * Getter for provider dial prefix
+        * <p>
+        * @return Provider dial prefix
+        */
        public Long getProviderDialPrefix () {
                return this.providerDialPrefix;
        }
 
-       @Override
+       /**
+        * Setter for provider dial prefix
+        * <p>
+        * @param providerDialPrefix Provider dial prefix
+        */
        public void setProviderDialPrefix (final Long providerDialPrefix) {
                this.providerDialPrefix = providerDialPrefix;
        }
 
-       @Override
+       /**
+        * Getter for pattern for mail gateway
+        * <p>
+        * @return Pattern for mail gateway
+        */
        public String getProviderMailPattern () {
                return this.providerMailPattern;
        }
 
-       @Override
+       /**
+        * Setter for pattern for mail gateway
+        * <p>
+        * @param providerMailPattern Pattern for mail gateway
+        */
        public void setProviderMailPattern (final String providerMailPattern) {
                this.providerMailPattern = providerMailPattern;
        }
 
-       @Override
+       /**
+        * Getter for provider name
+        * <p>
+        * @return Provider name
+        */
        public String getProviderName () {
                return this.providerName;
        }
 
-       @Override
+       /**
+        * Setter for provider name
+        * <p>
+        * @param providerName Provider name
+        */
        public void setProviderName (final String providerName) {
                this.providerName = providerName;
        }
index 9970065b41dd678a6d902fcf8c6acd05bf18c5fc..a56f9667ee380b6b61ec0aadd9826dd5cb9e0203 100644 (file)
@@ -17,7 +17,6 @@
 package org.mxchange.pizzaapplication.beans.mobileprovider;
 
 import java.io.Serializable;
-import org.mxchange.jcountry.data.Country;
 
 /**
  * An interface for a request web controller (bean) for administrative mobile
@@ -27,69 +26,4 @@ import org.mxchange.jcountry.data.Country;
  */
 public interface PizzaAdminMobileProviderWebRequestController extends Serializable {
 
-       /**
-        * Adds a mobile provider to database by calling the EJB. A pre-check on
-        * dial-prefix and country combination is done, if found, an exception is
-        * thrown. A redirect should take place after successfull creation.
-        * <p>
-        * @return Redirect outcome
-        */
-       String addMobileProvider ();
-
-       /**
-        * Getter for provider dial prefix
-        * <p>
-        * @return Provider dial prefix
-        */
-       Long getProviderDialPrefix ();
-
-       /**
-        * Setter for provider dial prefix
-        * <p>
-        * @param providerDialPrefix Provider dial prefix
-        */
-       void setProviderDialPrefix (final Long providerDialPrefix);
-
-       /**
-        * Getter for provider name
-        * <p>
-        * @return Provider name
-        */
-       String getProviderName ();
-
-       /**
-        * Setter for provider name
-        * <p>
-        * @param providerName Provider name
-        */
-       void setProviderName (final String providerName);
-
-       /**
-        * Getter for country instance ('s dial data)
-        * <p>
-        * @return Country instance
-        */
-       Country getProviderCountry ();
-
-       /**
-        * Setter for country instance ('s dial data)
-        * <p>
-        * @param country Country instance
-        */
-       void setProviderCountry (final Country country);
-
-       /**
-        * Getter for pattern for mail gateway
-        * <p>
-        * @return Pattern for mail gateway
-        */
-       String getProviderMailPattern ();
-
-       /**
-        * Setter for pattern for mail gateway
-        * <p>
-        * @param providerMailPattern Pattern for mail gateway
-        */
-       void setProviderMailPattern (final String providerMailPattern);
-
 }
index ec9fcc189f24458a2be5c1b7ad58ff38111ad8b4..90dfbac341038baa476aaff4863ff6b7bdf963e9 100644 (file)
@@ -91,7 +91,6 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
         */
        private DialableFaxNumber faxNumber;
 
-
        /**
         * Event being fired when an administrator has deleted fax number
         */
@@ -105,6 +104,7 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
        @Inject
        @Any
        private Event<AdminUpdatedFaxNumberEvent> faxNumberUpdatedEvent;
+
        /**
         * land-line number
         */
@@ -123,8 +123,9 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
        @Inject
        @Any
        private Event<AdminUpdatedLandLineNumberEvent> landLineNumberUpdatedEvent;
+
        /**
-        * mobile number
+        * Chosen mobile number
         */
        private DialableMobileNumber mobileNumber;
 
@@ -202,7 +203,11 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                // System.out.println(MessageFormat.format("{0}: Constructed, caller: {1}", this.getClass().getSimpleName(), caller));
        }
 
-       @Override
+       /**
+        * Returns a list of all unused ("non-linked") land-line numbers
+        * <p>
+        * @return List with all unused land-line numbers
+        */
        public List<DialableFaxNumber> allNonLinkedFaxNumbers () {
                // Get list of all mobile numbers
                List<DialableFaxNumber> list = this.phoneController.allFaxNumbers();
@@ -214,7 +219,11 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                return list;
        }
 
-       @Override
+       /**
+        * Returns a list of all unused ("non-linked") land-line numbers
+        * <p>
+        * @return List with all unused land-line numbers
+        */
        public List<DialableLandLineNumber> allNonLinkedLandLineNumbers () {
                // Get list of all mobile numbers
                List<DialableLandLineNumber> list = this.phoneController.allLandLineNumbers();
@@ -226,7 +235,11 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                return list;
        }
 
-       @Override
+       /**
+        * Returns a list of all unused ("non-linked") mobile numbers
+        * <p>
+        * @return List with all unused mobile numbers
+        */
        public List<DialableMobileNumber> allNonLinkedMobileNumbers () {
                // Get list of all mobile numbers
                List<DialableMobileNumber> list = this.phoneController.allMobileNumbers();
@@ -238,172 +251,200 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                return list;
        }
 
-       @Override
-       public String deleteFaxData (final DialableFaxNumber faxNumber) {
+       /**
+        * Deletes given fax entry data
+        * <p>
+        * @return Redirect outcome
+        */
+       public String deleteFaxData () {
+               // Get fax number from bean helper
+               DialableFaxNumber number = this.beanHelper.getFaxNumber();
+
                // Is all data set
-               if (faxNumber == null) {
+               if (number == null) {
                        // Not set, throw NPE
                        throw new NullPointerException("faxNumber is null"); //NOI18N
-               } else if (faxNumber.getPhoneId() == null) {
+               } else if (number.getPhoneId() == null) {
                        // Throw NPE again
                        throw new NullPointerException("faxNumber.phoneId is null"); //NOI18N
-               } else if (faxNumber.getPhoneId() < 1) {
+               } else if (number.getPhoneId() < 1) {
                        // Invalid number
-                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneId={0} is not valid", faxNumber.getPhoneId())); //NOI18N
-               } else if (faxNumber.getPhoneCountry() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneId={0} is not valid", number.getPhoneId())); //NOI18N
+               } else if (number.getPhoneCountry() == null) {
                        // Throw NPE
                        throw new NullPointerException("faxNumber.phoneCountry is null"); //NOI18N
-               } else if (faxNumber.getPhoneCountry().getCountryId() == null) {
+               } else if (number.getPhoneCountry().getCountryId() == null) {
                        // Throw NPE
                        throw new NullPointerException("faxNumber.phoneCountry.countryId is null"); //NOI18N
-               } else if (faxNumber.getPhoneCountry().getCountryId() < 1) {
+               } else if (number.getPhoneCountry().getCountryId() < 1) {
                        // Throw NPE
-                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneCountry.countryId={0} is not valid", faxNumber.getPhoneCountry().getCountryId())); //NOI18N
-               } else if (faxNumber.getPhoneAreaCode() == null) {
+                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneCountry.countryId={0} is not valid", number.getPhoneCountry().getCountryId())); //NOI18N
+               } else if (number.getPhoneAreaCode() == null) {
                        // ... throw again
                        throw new NullPointerException("faxNumber.phoneAreaCode is null"); //NOI18N
-               } else if (faxNumber.getPhoneAreaCode() < 1) {
+               } else if (number.getPhoneAreaCode() < 1) {
                        // Id not valid
-                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneAreaCode={0} is not valid.", faxNumber.getPhoneAreaCode())); //NOI18N
-               } else if (faxNumber.getPhoneNumber() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneAreaCode={0} is not valid.", number.getPhoneAreaCode())); //NOI18N
+               } else if (number.getPhoneNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("faxNumber.phoneNumber is null"); //NOI18N
-               } else if (faxNumber.getPhoneNumber() < 1) {
+               } else if (number.getPhoneNumber() < 1) {
                        // Throw NPE again
-                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneNumber={0} is not valid.", faxNumber.getPhoneNumber())); //NOI18N
+                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneNumber={0} is not valid.", number.getPhoneNumber())); //NOI18N
                }
 
                // Call EJB
-               this.adminPhoneBean.deleteFaxData(faxNumber);
+               this.adminPhoneBean.deleteFaxData(number);
 
                // Fire event
-               this.faxNumberDeletedEvent.fire(new AdminFaxNumberDeletedEvent(faxNumber));
+               this.faxNumberDeletedEvent.fire(new AdminFaxNumberDeletedEvent(number));
 
                // All fine, redirect
                return "admin_list_fax"; //NOI18N
        }
 
-       @Override
-       public String deleteLandLineData (final DialableLandLineNumber landLineNumber) {
+       /**
+        * Deletes given land-line entry data
+        * <p>
+        * @return Redirect outcome
+        */
+       public String deleteLandLineData () {
+               // Get land-line number from helper
+               DialableLandLineNumber number = this.beanHelper.getLandLineNumber();
+
                // Is all data set
-               if (landLineNumber == null) {
+               if (number == null) {
                        // Not set, throw NPE
                        throw new NullPointerException("landLineNumber is null"); //NOI18N
-               } else if (landLineNumber.getPhoneId() == null) {
+               } else if (number.getPhoneId() == null) {
                        // Throw NPE again
                        throw new NullPointerException("landLineNumber.phoneId is null"); //NOI18N
-               } else if (landLineNumber.getPhoneId() < 1) {
+               } else if (number.getPhoneId() < 1) {
                        // Invalid number
-                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneId={0} is not valid", landLineNumber.getPhoneId())); //NOI18N
-               } else if (landLineNumber.getPhoneCountry() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneId={0} is not valid", number.getPhoneId())); //NOI18N
+               } else if (number.getPhoneCountry() == null) {
                        // Throw NPE
                        throw new NullPointerException("landLineNumber.phoneCountry is null"); //NOI18N
-               } else if (landLineNumber.getPhoneCountry().getCountryId() == null) {
+               } else if (number.getPhoneCountry().getCountryId() == null) {
                        // Throw NPE
                        throw new NullPointerException("landLineNumber.phoneCountry.countryId is null"); //NOI18N
-               } else if (landLineNumber.getPhoneCountry().getCountryId() < 1) {
+               } else if (number.getPhoneCountry().getCountryId() < 1) {
                        // Throw NPE
-                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneCountry.countryId={0} is not valid", landLineNumber.getPhoneCountry().getCountryId())); //NOI18N
-               } else if (landLineNumber.getPhoneAreaCode() == null) {
+                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneCountry.countryId={0} is not valid", number.getPhoneCountry().getCountryId())); //NOI18N
+               } else if (number.getPhoneAreaCode() == null) {
                        // ... throw again
                        throw new NullPointerException("landLineNumber.phoneAreaCode is null"); //NOI18N
-               } else if (landLineNumber.getPhoneAreaCode() < 1) {
+               } else if (number.getPhoneAreaCode() < 1) {
                        // Id not valid
-                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneAreaCode={0} is not valid.", landLineNumber.getPhoneAreaCode())); //NOI18N
-               } else if (landLineNumber.getPhoneNumber() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneAreaCode={0} is not valid.", number.getPhoneAreaCode())); //NOI18N
+               } else if (number.getPhoneNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("landLineNumber.phoneNumber is null"); //NOI18N
-               } else if (landLineNumber.getPhoneNumber() < 1) {
+               } else if (number.getPhoneNumber() < 1) {
                        // Throw NPE again
-                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneNumber={0} is not valid.", landLineNumber.getPhoneNumber())); //NOI18N
+                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneNumber={0} is not valid.", number.getPhoneNumber())); //NOI18N
                }
 
                // Call EJB
-               this.adminPhoneBean.deleteLandLineData(landLineNumber);
+               this.adminPhoneBean.deleteLandLineData(number);
 
                // Fire event
-               this.landLineNumberDeletedEvent.fire(new AdminLandLineNumberDeletedEvent(landLineNumber));
+               this.landLineNumberDeletedEvent.fire(new AdminLandLineNumberDeletedEvent(number));
 
                // All fine, redirect
                return "admin_list_landline"; //NOI18N
        }
 
-       @Override
-       public String deleteMobileData (final DialableMobileNumber mobileNumber) {
+       /**
+        * Deletes given mobile entry data
+        * <p>
+        * @return Redirect outcome
+        */
+       public String deleteMobileData () {
+               // Get mobile number from helper
+               DialableMobileNumber number = this.beanHelper.getMobileNumber();
+
                // Is all data set
-               if (mobileNumber == null) {
+               if (number == null) {
                        // Not set, throw NPE
                        throw new NullPointerException("mobileNumber is null"); //NOI18N
-               } else if (mobileNumber.getPhoneId() == null) {
+               } else if (number.getPhoneId() == null) {
                        // Throw NPE again
                        throw new NullPointerException("mobileNumber.phoneId is null"); //NOI18N
-               } else if (mobileNumber.getPhoneId() < 1) {
+               } else if (number.getPhoneId() < 1) {
                        // Invalid number
-                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getPhoneId())); //NOI18N
-               } else if (mobileNumber.getMobileProvider() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", number.getPhoneId())); //NOI18N
+               } else if (number.getMobileProvider() == null) {
                        // Throw NPE
                        throw new NullPointerException("mobileNumber.mobileProvider is null"); //NOI18N
-               } else if (mobileNumber.getMobileProvider().getProviderId() == null) {
+               } else if (number.getMobileProvider().getProviderId() == null) {
                        // ... throw again
                        throw new NullPointerException("mobileNumber.mobileProvider.providerId is null"); //NOI18N
-               } else if (mobileNumber.getMobileProvider().getProviderId() < 1) {
+               } else if (number.getMobileProvider().getProviderId() < 1) {
                        // Id not valid
-                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.mobileProvider.providerId={0} is not valid.", mobileNumber.getMobileProvider().getProviderId())); //NOI18N
-               } else if (mobileNumber.getPhoneNumber() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.mobileProvider.providerId={0} is not valid.", number.getMobileProvider().getProviderId())); //NOI18N
+               } else if (number.getPhoneNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("mobileNumber.phoneNumber is null"); //NOI18N
-               } else if (mobileNumber.getPhoneNumber() < 1) {
+               } else if (number.getPhoneNumber() < 1) {
                        // Throw NPE again
-                       throw new NullPointerException(MessageFormat.format("mobileNumber.phoneNumber={0} is not valid.", mobileNumber.getPhoneNumber())); //NOI18N
+                       throw new NullPointerException(MessageFormat.format("mobileNumber.phoneNumber={0} is not valid.", number.getPhoneNumber())); //NOI18N
                }
 
                // Call EJB
-               this.adminPhoneBean.deleteMobileData(mobileNumber);
+               this.adminPhoneBean.deleteMobileData(number);
 
                // Fire event
-               this.mobileNumberDeletedEvent.fire(new AdminMobileNumberDeletedEvent(mobileNumber));
+               this.mobileNumberDeletedEvent.fire(new AdminMobileNumberDeletedEvent(number));
 
                // All fine, redirect
                return "admin_list_mobile"; //NOI18N
        }
 
-       @Override
-       public String editFaxData (final DialableFaxNumber faxNumber) {
+       /**
+        * Changes fax entry data
+        * <p>
+        * @return Redirect outcome
+        */
+       public String doChangeFaxNumber () {
+               // Get fax number from bean helper
+               DialableFaxNumber number = this.beanHelper.getFaxNumber();
+
                // Is all data set
-               if (faxNumber == null) {
+               if (number == null) {
                        // Not set, throw NPE
                        throw new NullPointerException("faxNumber is null"); //NOI18N
-               } else if (faxNumber.getPhoneId() == null) {
+               } else if (number.getPhoneId() == null) {
                        // Throw NPE again
                        throw new NullPointerException("faxNumber.phoneId is null"); //NOI18N
-               } else if (faxNumber.getPhoneId() < 1) {
+               } else if (number.getPhoneId() < 1) {
                        // Invalid number
-                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneId={0} is not valid", faxNumber.getPhoneId())); //NOI18N
-               } else if (faxNumber.getPhoneCountry() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneId={0} is not valid", number.getPhoneId())); //NOI18N
+               } else if (number.getPhoneCountry() == null) {
                        // Throw NPE
                        throw new NullPointerException("faxNumber.phoneCountry is null"); //NOI18N
-               } else if (faxNumber.getPhoneCountry().getCountryId() == null) {
+               } else if (number.getPhoneCountry().getCountryId() == null) {
                        // Throw NPE
                        throw new NullPointerException("faxNumber.phoneCountry.countryId is null"); //NOI18N
-               } else if (faxNumber.getPhoneCountry().getCountryId() < 1) {
+               } else if (number.getPhoneCountry().getCountryId() < 1) {
                        // Throw NPE
-                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneCountry.countryId={0} is not valid", faxNumber.getPhoneCountry().getCountryId())); //NOI18N
-               } else if (faxNumber.getPhoneAreaCode() == null) {
+                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneCountry.countryId={0} is not valid", number.getPhoneCountry().getCountryId())); //NOI18N
+               } else if (number.getPhoneAreaCode() == null) {
                        // ... throw again
                        throw new NullPointerException("faxNumber.phoneAreaCode is null"); //NOI18N
-               } else if (faxNumber.getPhoneAreaCode() < 1) {
+               } else if (number.getPhoneAreaCode() < 1) {
                        // Id not valid
-                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneAreaCode={0} is not valid.", faxNumber.getPhoneAreaCode())); //NOI18N
-               } else if (faxNumber.getPhoneNumber() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("faxNumber.phoneAreaCode={0} is not valid.", number.getPhoneAreaCode())); //NOI18N
+               } else if (number.getPhoneNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("faxNumber.phoneNumber is null"); //NOI18N
-               } else if (faxNumber.getPhoneNumber() < 1) {
+               } else if (number.getPhoneNumber() < 1) {
                        // Throw NPE again
-                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneNumber={0} is not valid.", faxNumber.getPhoneNumber())); //NOI18N
+                       throw new NullPointerException(MessageFormat.format("faxNumber.phoneNumber={0} is not valid.", number.getPhoneNumber())); //NOI18N
                }
 
                // Is the mobile provider and number the same?
-               if ((Objects.equals(this.getPhoneCountry(), faxNumber.getPhoneCountry())) && (Objects.equals(this.getPhoneAreaCode(), faxNumber.getPhoneAreaCode())) && (Objects.equals(this.getPhoneNumber(), faxNumber.getPhoneNumber()))) {
+               if ((Objects.equals(this.getPhoneCountry(), number.getPhoneCountry())) && (Objects.equals(this.getPhoneAreaCode(), number.getPhoneAreaCode())) && (Objects.equals(this.getPhoneNumber(), number.getPhoneNumber()))) {
                        // Show message
                        this.showFacesMessage("form_edit_fax:faxNumber", "ERROR_ADMIN_NO_CHANGE_ENTERED"); //NOI18N
 
@@ -412,12 +453,12 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                }
 
                // Set all data
-               faxNumber.setPhoneCountry(this.getPhoneCountry());
-               faxNumber.setPhoneAreaCode(this.getPhoneAreaCode());
-               faxNumber.setPhoneNumber(this.getPhoneNumber());
+               number.setPhoneCountry(this.getPhoneCountry());
+               number.setPhoneAreaCode(this.getPhoneAreaCode());
+               number.setPhoneNumber(this.getPhoneNumber());
 
                // Send to bean
-               DialableFaxNumber updatedNumber = this.adminPhoneBean.updateFaxData(faxNumber);
+               DialableFaxNumber updatedNumber = this.adminPhoneBean.updateFaxData(number);
 
                // Fire event
                this.faxNumberUpdatedEvent.fire(new AdminFaxNumberUpdatedEvent(updatedNumber));
@@ -426,43 +467,50 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                return "admin_show_fax"; //NOI18N
        }
 
-       @Override
-       public String editLandLineData (final DialableLandLineNumber landLineNumber) {
+       /**
+        * Changes land-line entry data
+        * <p>
+        * @return Redirect outcome
+        */
+       public String doChangeLandLineNumber () {
+               // Get land-line number from helper
+               DialableLandLineNumber number = this.beanHelper.getLandLineNumber();
+
                // Is all data set
-               if (landLineNumber == null) {
+               if (number == null) {
                        // Not set, throw NPE
                        throw new NullPointerException("landLineNumber is null"); //NOI18N
-               } else if (landLineNumber.getPhoneId() == null) {
+               } else if (number.getPhoneId() == null) {
                        // Throw NPE again
                        throw new NullPointerException("landLineNumber.phoneId is null"); //NOI18N
-               } else if (landLineNumber.getPhoneId() < 1) {
+               } else if (number.getPhoneId() < 1) {
                        // Invalid number
-                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneId={0} is not valid", landLineNumber.getPhoneId())); //NOI18N
-               } else if (landLineNumber.getPhoneCountry() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneId={0} is not valid", number.getPhoneId())); //NOI18N
+               } else if (number.getPhoneCountry() == null) {
                        // Throw NPE
                        throw new NullPointerException("landLineNumber.phoneCountry is null"); //NOI18N
-               } else if (landLineNumber.getPhoneCountry().getCountryId() == null) {
+               } else if (number.getPhoneCountry().getCountryId() == null) {
                        // Throw NPE
                        throw new NullPointerException("landLineNumber.phoneCountry.countryId is null"); //NOI18N
-               } else if (landLineNumber.getPhoneCountry().getCountryId() < 1) {
+               } else if (number.getPhoneCountry().getCountryId() < 1) {
                        // Throw NPE
-                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneCountry.countryId={0} is not valid", landLineNumber.getPhoneCountry().getCountryId())); //NOI18N
-               } else if (landLineNumber.getPhoneAreaCode() == null) {
+                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneCountry.countryId={0} is not valid", number.getPhoneCountry().getCountryId())); //NOI18N
+               } else if (number.getPhoneAreaCode() == null) {
                        // ... throw again
                        throw new NullPointerException("landLineNumber.phoneAreaCode is null"); //NOI18N
-               } else if (landLineNumber.getPhoneAreaCode() < 1) {
+               } else if (number.getPhoneAreaCode() < 1) {
                        // Id not valid
-                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneAreaCode={0} is not valid.", landLineNumber.getPhoneAreaCode())); //NOI18N
-               } else if (landLineNumber.getPhoneNumber() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("landLineNumber.phoneAreaCode={0} is not valid.", number.getPhoneAreaCode())); //NOI18N
+               } else if (number.getPhoneNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("landLineNumber.phoneNumber is null"); //NOI18N
-               } else if (landLineNumber.getPhoneNumber() < 1) {
+               } else if (number.getPhoneNumber() < 1) {
                        // Throw NPE again
-                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneNumber={0} is not valid.", landLineNumber.getPhoneNumber())); //NOI18N
+                       throw new NullPointerException(MessageFormat.format("landLineNumber.phoneNumber={0} is not valid.", number.getPhoneNumber())); //NOI18N
                }
 
                // Is the mobile provider and number the same?
-               if ((Objects.equals(this.getPhoneCountry(), landLineNumber.getPhoneCountry())) && (Objects.equals(this.getPhoneAreaCode(), landLineNumber.getPhoneAreaCode())) && (Objects.equals(this.getPhoneNumber(), landLineNumber.getPhoneNumber()))) {
+               if ((Objects.equals(this.getPhoneCountry(), number.getPhoneCountry())) && (Objects.equals(this.getPhoneAreaCode(), number.getPhoneAreaCode())) && (Objects.equals(this.getPhoneNumber(), number.getPhoneNumber()))) {
                        // Show message
                        this.showFacesMessage("form_edit_landline:landLineNumber", "ERROR_ADMIN_NO_CHANGE_ENTERED"); //NOI18N
 
@@ -471,12 +519,12 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                }
 
                // Set all data
-               landLineNumber.setPhoneCountry(this.getPhoneCountry());
-               landLineNumber.setPhoneAreaCode(this.getPhoneAreaCode());
-               landLineNumber.setPhoneNumber(this.getPhoneNumber());
+               number.setPhoneCountry(this.getPhoneCountry());
+               number.setPhoneAreaCode(this.getPhoneAreaCode());
+               number.setPhoneNumber(this.getPhoneNumber());
 
                // Send to bean
-               DialableLandLineNumber updatedNumber = this.adminPhoneBean.updateLandLineData(landLineNumber);
+               DialableLandLineNumber updatedNumber = this.adminPhoneBean.updateLandLineData(number);
 
                // Fire event
                this.landLineNumberUpdatedEvent.fire(new AdminLandLineNumberUpdatedEvent(updatedNumber));
@@ -485,33 +533,40 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                return "admin_show_landline"; //NOI18N
        }
 
-       @Override
-       public String editMobileData (final DialableMobileNumber mobileNumber) {
+       /**
+        * Changes mobile entry data
+        * <p>
+        * @return Redirect outcome
+        */
+       public String doUpdateMobileNumber () {
+               // Get mobile number from helper
+               DialableMobileNumber number = this.beanHelper.getMobileNumber();
+
                // Is all data set
-               if (mobileNumber == null) {
+               if (number == null) {
                        // Not set, throw NPE
                        throw new NullPointerException("mobileNumber is null"); //NOI18N
-               } else if (mobileNumber.getPhoneId() == null) {
+               } else if (number.getPhoneId() == null) {
                        // Throw NPE again
                        throw new NullPointerException("mobileNumber.phoneId is null"); //NOI18N
-               } else if (mobileNumber.getPhoneId() < 1) {
+               } else if (number.getPhoneId() < 1) {
                        // Invalid number
-                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", mobileNumber.getPhoneId())); //NOI18N
-               } else if (mobileNumber.getMobileProvider() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.phoneId={0} is not valid", number.getPhoneId())); //NOI18N
+               } else if (number.getMobileProvider() == null) {
                        // Throw NPE
                        throw new NullPointerException("mobileNumber.mobileProvider is null"); //NOI18N
-               } else if (mobileNumber.getMobileProvider().getProviderId() == null) {
+               } else if (number.getMobileProvider().getProviderId() == null) {
                        // ... throw again
                        throw new NullPointerException("mobileNumber.mobileProvider.providerId is null"); //NOI18N
-               } else if (mobileNumber.getMobileProvider().getProviderId() < 1) {
+               } else if (number.getMobileProvider().getProviderId() < 1) {
                        // Id not valid
-                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.mobileProvider.providerId={0} is not valid.", mobileNumber.getMobileProvider().getProviderId())); //NOI18N
-               } else if (mobileNumber.getPhoneNumber() == null) {
+                       throw new IllegalArgumentException(MessageFormat.format("mobileNumber.mobileProvider.providerId={0} is not valid.", number.getMobileProvider().getProviderId())); //NOI18N
+               } else if (number.getPhoneNumber() == null) {
                        // Throw NPE again
                        throw new NullPointerException("mobileNumber.phoneNumber is null"); //NOI18N
-               } else if (mobileNumber.getPhoneNumber() < 1) {
+               } else if (number.getPhoneNumber() < 1) {
                        // Throw NPE again
-                       throw new NullPointerException(MessageFormat.format("mobileNumber.phoneNumber={0} is not valid.", mobileNumber.getPhoneNumber())); //NOI18N
+                       throw new NullPointerException(MessageFormat.format("mobileNumber.phoneNumber={0} is not valid.", number.getPhoneNumber())); //NOI18N
                } else if (this.getMobileProvider() == null) {
                        // Not provided
                        this.showFacesMessage("form_edit_mobile:mobileProvider", "ERROR_ADMIN_NO_MOBILE_PROVIDER_SELECTED"); //NOI18N
@@ -529,7 +584,7 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                }
 
                // Is the mobile provider and number the same?
-               if ((Objects.equals(this.getMobileProvider(), mobileNumber.getMobileProvider())) && (Objects.equals(this.getPhoneNumber(), mobileNumber.getPhoneNumber()))) {
+               if ((Objects.equals(this.getMobileProvider(), number.getMobileProvider())) && (Objects.equals(this.getPhoneNumber(), number.getPhoneNumber()))) {
                        // Show message
                        this.showFacesMessage("form_edit_mobile:mobileNumber", "ERROR_ADMIN_NO_CHANGE_ENTERED"); //NOI18N
 
@@ -538,11 +593,11 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                }
 
                // Set all data
-               mobileNumber.setMobileProvider(this.getMobileProvider());
-               mobileNumber.setPhoneNumber(this.getPhoneNumber());
+               number.setMobileProvider(this.getMobileProvider());
+               number.setPhoneNumber(this.getPhoneNumber());
 
                // Send to bean
-               DialableMobileNumber updatedNumber = this.adminPhoneBean.updateMobileData(mobileNumber);
+               DialableMobileNumber updatedNumber = this.adminPhoneBean.updateMobileData(number);
 
                // Fire event
                this.mobileNumberUpdatedEvent.fire(new AdminMobileNumberUpdatedEvent(updatedNumber));
@@ -551,76 +606,128 @@ public class PizzaAdminPhoneWebRequestBean extends BasePizzaController implement
                return "admin_show_mobile"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for chosen fax number
+        * <p>
+        * @return fax number
+        */
        public DialableFaxNumber getFaxNumber () {
-               // String caller = MessageFormat.format("{0}.{1}", Thread.currentThread().getStackTrace()[THREAD_STACK].getClassName(), Thread.currentThread().getStackTrace()[THREAD_STACK].getMethodName());
-               // System.out.println(MessageFormat.format("{0}: Returning this.faxNumber={1}, caller: {2}", this.getClass().getSimpleName(), this.faxNumber, caller));
                return this.faxNumber;
        }
 
-       @Override
+       /**
+        * Setter for chosen fax number
+        * <p>
+        * @param faxNumber fax number
+        */
        public void setFaxNumber (final DialableFaxNumber faxNumber) {
-               // String caller = MessageFormat.format("{0}.{1}", Thread.currentThread().getStackTrace()[THREAD_STACK].getClassName(), Thread.currentThread().getStackTrace()[THREAD_STACK].getMethodName());
-               // System.out.println(MessageFormat.format("{0}: Settin faxNumber={1}, previous: this.faxNumber={2}, caller: {3}", this.getClass().getSimpleName(), faxNumber, this.faxNumber, caller));
                this.faxNumber = faxNumber;
        }
 
-       @Override
+       /**
+        * Getter for chosen land-line number
+        * <p>
+        * @return land-line number
+        */
        public DialableLandLineNumber getLandLineNumber () {
                return this.landLineNumber;
        }
 
-       @Override
+       /**
+        * Setter for chosen land-line number
+        * <p>
+        * @param landLineNumber land-line number
+        */
        public void setLandLineNumber (final DialableLandLineNumber landLineNumber) {
                this.landLineNumber = landLineNumber;
        }
 
-       @Override
+       /**
+        * Getter for chosen mobile number
+        * <p>
+        * @return mobile number
+        */
        public DialableMobileNumber getMobileNumber () {
                return this.mobileNumber;
        }
 
-       @Override
+       /**
+        * Setter for chosen mobile number
+        * <p>
+        * @param mobileNumber mobile number
+        */
        public void setMobileNumber (final DialableMobileNumber mobileNumber) {
                this.mobileNumber = mobileNumber;
        }
 
-       @Override
+       /**
+        * Getter for mobile provider
+        * <p>
+        * @return Mobile provider
+        */
        public MobileProvider getMobileProvider () {
                return this.mobileProvider;
        }
 
-       @Override
+       /**
+        * Setter for mobile provider
+        * <p>
+        * @param mobileProvider Mobile provider
+        */
        public void setMobileProvider (final MobileProvider mobileProvider) {
                this.mobileProvider = mobileProvider;
        }
 
-       @Override
+       /**
+        * Getter for phone area code
+        * <p>
+        * @return Phone area code
+        */
        public Integer getPhoneAreaCode () {
                return this.phoneAreaCode;
        }
 
-       @Override
+       /**
+        * Setter for phone area code
+        * <p>
+        * @param phoneAreaCode Phone area code
+        */
        public void setPhoneAreaCode (final Integer phoneAreaCode) {
                this.phoneAreaCode = phoneAreaCode;
        }
 
-       @Override
+       /**
+        * Getter for phone country (for dial prefix)
+        * <p>
+        * @return Phone country
+        */
        public Country getPhoneCountry () {
                return this.phoneCountry;
        }
 
-       @Override
+       /**
+        * Setter for phone country (for dial prefix)
+        * <p>
+        * @param phoneCountry Phone country
+        */
        public void setPhoneCountry (final Country phoneCountry) {
                this.phoneCountry = phoneCountry;
        }
 
-       @Override
+       /**
+        * Getter for dial number without prefix
+        * <p>
+        * @return Dial number without prefix
+        */
        public Long getPhoneNumber () {
                return this.phoneNumber;
        }
 
-       @Override
+       /**
+        * Setter for dial number without prefix
+        * <p>
+        * @param phoneNumber Dial number without prefix
+        */
        public void setPhoneNumber (final Long phoneNumber) {
                this.phoneNumber = phoneNumber;
        }
index bfc4b3d4f0360dbdd18155cb457bb12f121a99fa..6e05c440bb6d8bf87a6c73bbd00beb475c2628df 100644 (file)
 package org.mxchange.pizzaapplication.beans.phone;
 
 import java.io.Serializable;
-import java.util.List;
-import javax.ejb.Local;
-import org.mxchange.jcountry.data.Country;
-import org.mxchange.jphone.phonenumbers.fax.DialableFaxNumber;
-import org.mxchange.jphone.phonenumbers.landline.DialableLandLineNumber;
-import org.mxchange.jphone.phonenumbers.mobile.DialableMobileNumber;
-import org.mxchange.jphone.phonenumbers.mobileprovider.MobileProvider;
 
 /**
  * An interface for a request web controller (bean) for administrative phone
@@ -31,180 +24,6 @@ import org.mxchange.jphone.phonenumbers.mobileprovider.MobileProvider;
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaAdminPhoneWebRequestController extends Serializable {
 
-       /**
-        * Returns a list of all unused ("non-linked") mobile numbers
-        * <p>
-        * @return List with all unused mobile numbers
-        */
-       List<DialableMobileNumber> allNonLinkedMobileNumbers ();
-
-       /**
-        * Returns a list of all unused ("non-linked") land-line numbers
-        * <p>
-        * @return List with all unused land-line numbers
-        */
-       List<DialableFaxNumber> allNonLinkedFaxNumbers ();
-
-       /**
-        * Returns a list of all unused ("non-linked") land-line numbers
-        * <p>
-        * @return List with all unused land-line numbers
-        */
-       List<DialableLandLineNumber> allNonLinkedLandLineNumbers ();
-
-       /**
-        * Deletes given fax entry data
-        * <p>
-        * @param faxNumber Fax number to delete
-        * <p>
-        * @return Redirect outcome
-        */
-       String deleteFaxData (final DialableFaxNumber faxNumber);
-
-       /**
-        * Changes fax entry data
-        * <p>
-        * @param faxNumber Fax number to change data
-        * <p>
-        * @return Redirect outcome
-        */
-       String editFaxData (final DialableFaxNumber faxNumber);
-
-       /**
-        * Deletes given land-line entry data
-        * <p>
-        * @param landLineNumber Land-line number to delete
-        * <p>
-        * @return Redirect outcome
-        */
-       String deleteLandLineData (final DialableLandLineNumber landLineNumber);
-
-       /**
-        * Changes land-line entry data
-        * <p>
-        * @param landLineNumber Land-line number to change data
-        * <p>
-        * @return Redirect outcome
-        */
-       String editLandLineData (final DialableLandLineNumber landLineNumber);
-
-       /**
-        * Deletes given mobile entry data
-        * <p>
-        * @param mobileNumber Mobile number to delete
-        * <p>
-        * @return Redirect outcome
-        */
-       String deleteMobileData (final DialableMobileNumber mobileNumber);
-
-       /**
-        * Changes mobile entry data
-        * <p>
-        * @param mobileNumber Mobile number to change data
-        * <p>
-        * @return Redirect outcome
-        */
-       String editMobileData (final DialableMobileNumber mobileNumber);
-
-       /**
-        * Getter for mobile provider
-        * <p>
-        * @return Mobile provider
-        */
-       MobileProvider getMobileProvider ();
-
-       /**
-        * Setter for mobile provider
-        * <p>
-        * @param mobileProvider Mobile provider
-        */
-       void setMobileProvider (final MobileProvider mobileProvider);
-
-       /**
-        * Getter for choosen fax number
-        * <p>
-        * @return fax number
-        */
-       DialableFaxNumber getFaxNumber ();
-
-       /**
-        * Setter for choosen fax number
-        * <p>
-        * @param faxNumber fax number
-        */
-       void setFaxNumber (final DialableFaxNumber faxNumber);
-
-       /**
-        * Getter for choosen land-line number
-        * <p>
-        * @return land-line number
-        */
-       DialableLandLineNumber getLandLineNumber ();
-
-       /**
-        * Setter for choosen land-line number
-        * <p>
-        * @param landLineNumber land-line number
-        */
-       void setLandLineNumber (final DialableLandLineNumber landLineNumber);
-
-       /**
-        * Getter for choosen mobile number
-        * <p>
-        * @return mobile number
-        */
-       DialableMobileNumber getMobileNumber ();
-
-       /**
-        * Setter for choosen mobile number
-        * <p>
-        * @param mobileNumber mobile number
-        */
-       void setMobileNumber (final DialableMobileNumber mobileNumber);
-
-       /**
-        * Getter for phone area code
-        * <p>
-        * @return Phone area code
-        */
-       Integer getPhoneAreaCode ();
-
-       /**
-        * Setter for phone area code
-        * <p>
-        * @param phoneAreaCode Phone area code
-        */
-       void setPhoneAreaCode (final Integer phoneAreaCode);
-
-       /**
-        * Getter for phone country (for dial prefix)
-        * <p>
-        * @return Phone country
-        */
-       Country getPhoneCountry ();
-
-       /**
-        * Setter for phone country (for dial prefix)
-        * <p>
-        * @param phoneCountry Phone country
-        */
-       void setPhoneCountry (final Country phoneCountry);
-
-       /**
-        * Getter for dial number without prefix
-        * <p>
-        * @return Dial number without prefix
-        */
-       Long getPhoneNumber ();
-
-       /**
-        * Setter for dial number without prefix
-        * <p>
-        * @param phoneNumber Dial number without prefix
-        */
-       void setPhoneNumber (final Long phoneNumber);
-
 }
index 6d29520f4080a2814d834362c67875c82d56cf3c..3b30e7b73f2608c0ecb1b73b3f1b8bc3210fe9b7 100644 (file)
@@ -18,7 +18,6 @@ package org.mxchange.pizzaapplication.beans.phone;
 
 import java.io.Serializable;
 import java.util.List;
-import javax.ejb.Local;
 import org.mxchange.jphone.phonenumbers.fax.DialableFaxNumber;
 import org.mxchange.jphone.phonenumbers.landline.DialableLandLineNumber;
 import org.mxchange.jphone.phonenumbers.mobile.DialableMobileNumber;
@@ -29,7 +28,6 @@ import org.mxchange.jphone.phonenumbers.mobile.DialableMobileNumber;
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaPhoneWebApplicationController extends Serializable {
 
        /**
index 833eddd1cb75284e8d3f78310bb6f9d53e0a7e21..e5a6461a14f6c2b72c43de502baea5cb85ef3607 100644 (file)
@@ -36,6 +36,7 @@ import org.mxchange.jusercore.events.user.add.AdminAddedUserEvent;
 import org.mxchange.jusercore.events.user.add.ObservableAdminAddedUserEvent;
 import org.mxchange.jusercore.events.user.delete.AdminDeletedUserEvent;
 import org.mxchange.jusercore.events.user.delete.ObservableAdminDeletedUserEvent;
+import org.mxchange.jusercore.events.user.helper.created.ObservableHelperCreatedUserEvent;
 import org.mxchange.jusercore.events.user.linked.AdminLinkedUserEvent;
 import org.mxchange.jusercore.events.user.linked.ObservableAdminLinkedUserEvent;
 import org.mxchange.jusercore.events.user.locked.AdminLockedUserEvent;
@@ -187,7 +188,7 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
        private String userPasswordRepeat;
 
        /**
-        * Event being fired when admin unlocks an account
+        * Event being fired when administrator unlocks an account
         */
        @Inject
        @Any
@@ -201,7 +202,13 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                super();
        }
 
-       @Override
+       /**
+        * Adds user instance to database by preparing a complete user instance and
+        * sending it to the EJB. The data set in the controller is being verified,
+        * e.g. if the user name or email address is not used yet.
+        * <p>
+        * @return Redirect outcome
+        */
        public String addUser () {
                // Get contact from bean helper to "cache" it locally
                Contact contact = this.beanHelper.getContact();
@@ -215,32 +222,12 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                        throw new IllegalArgumentException("userName is null"); //NOI18N
                } else if (contact == null) {
                        // No contact instance set, so test required fields: gender, first name and family name
-                       if (this.contactController.getPersonalTitle() == null) {
-                               // Throw NPE again
-                               throw new NullPointerException("contactController.gender is null"); //NOI18N
-                       } else if (this.contactController.getFirstName() == null) {
-                               // ... and again
-                               throw new NullPointerException("contactController.firstName is null"); //NOI18N
-                       } else if (this.adminContactController.getFirstName().isEmpty()) {
-                               // ... and again
-                               throw new IllegalArgumentException("contactController.firstName is empty"); //NOI18N
-                       } else if (this.adminContactController.getFamilyName() == null) {
-                               // ... and again
-                               throw new NullPointerException("contactController.familyName is null"); //NOI18N
-                       } else if (this.adminContactController.getFamilyName().isEmpty()) {
-                               // ... and again
-                               throw new IllegalArgumentException("contactController.familyName is empty"); //NOI18N
-                       } else if (this.adminContactController.getEmailAddress() == null) {
-                               // ... and again
-                               throw new NullPointerException("contactController.emailAddress is null"); //NOI18N
-                       } else if (this.adminContactController.getEmailAddress().isEmpty()) {
-                               // ... and again
-                               throw new IllegalArgumentException("contactController.emailAddress is empty"); //NOI18N
-                       }
+                       this.adminContactController.validateContactData();
                }
 
                // Create new user instance
                User user = new LoginUser();
+
                // Set user name, CONFIRMED and INVISIBLE
                user.setUserName(this.getUserName());
                user.setUserMustChangePassword(this.getUserMustChangePassword());
@@ -331,6 +318,37 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                return "admin_list_user"; //NOI18N
        }
 
+       /**
+        * Event observer for when a bean helper has successfully created a user
+        * instance, means the user exists. If the user does not exist, this event
+        * should not fire but instead a proper exception must be thrown.
+        * <p>
+        * @param event User created event
+        */
+       public void afterHelperCreatedUserEvent (@Observes final ObservableHelperCreatedUserEvent event) {
+               // Is the instance valid?
+               if (null == event) {
+                       // Throw NPE
+                       throw new NullPointerException("event is null"); //NOI18N
+               } else if (event.getCreatedUser() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("event.createdUser is null"); //NOI18N
+               } else if (event.getCreatedUser().getUserId() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("event.createdUser.userId is null"); //NOI18N
+               } else if (event.getCreatedUser().getUserId() < 1) {
+                       // Throw NPE again
+                       throw new NullPointerException(MessageFormat.format("event.createdUser.userId={0} is not valid", event.getCreatedUser().getUserId())); //NOI18N
+               }
+
+               // Get user instance
+               User user = event.getCreatedUser();
+
+               // Set all fields here
+               this.setUserName(user.getUserName());
+               this.setUserLockReason(user.getUserLastLockedReason());
+       }
+
        /**
         * Event observer for new user registrations
         * <p>
@@ -360,8 +378,15 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                this.clear();
        }
 
-       @Override
-       public String deleteUserData (final User user) {
+       /**
+        * Deletes given user account
+        * <p>
+        * @return Redirect outcome
+        */
+       public String deleteUserData () {
+               // Get user instance into local variable
+               User user = this.beanHelper.getUser();
+
                // Is the user instance valid and CONFIRMED?
                if (null == user) {
                        // Throw NPE
@@ -389,7 +414,11 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                return "admin_list_user"; //NOI18N
        }
 
-       @Override
+       /**
+        * Edits currently loaded user's data in database.
+        * <p>
+        * @return Redirect outcome
+        */
        public String editUserData () {
                // Get user instance
                User user = this.beanHelper.getUser();
@@ -464,62 +493,110 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                return "admin_list_user"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for user delete reason
+        * <p>
+        * @return User delete reason
+        */
        public String getUserDeleteReason () {
                return this.userDeleteReason;
        }
 
-       @Override
+       /**
+        * Setter for user delete reason
+        * <p>
+        * @param userDeleteReason User delete reason
+        */
        public void setUserDeleteReason (final String userDeleteReason) {
                this.userDeleteReason = userDeleteReason;
        }
 
-       @Override
+       /**
+        * Getter for user lock reason
+        * <p>
+        * @return User lock reason
+        */
        public String getUserLockReason () {
                return this.userLockReason;
        }
 
-       @Override
+       /**
+        * Setter for user lock reason
+        * <p>
+        * @param userLockReason User lock reason
+        */
        public void setUserLockReason (final String userLockReason) {
                this.userLockReason = userLockReason;
        }
 
-       @Override
+       /**
+        * Getter for flag if user needs to change password
+        * <p>
+        * @return Flag if user needs to change password
+        */
        public Boolean getUserMustChangePassword () {
                return this.userMustChangePassword;
        }
 
-       @Override
+       /**
+        * Setter for flag if user needs to change password
+        * <p>
+        * @param userMustChangePassword Flag if user needs to change password
+        */
        public void setUserMustChangePassword (final Boolean userMustChangePassword) {
                this.userMustChangePassword = userMustChangePassword;
        }
 
-       @Override
+       /**
+        * Getter for user name
+        * <p>
+        * @return User name
+        */
        public String getUserName () {
                return this.userName;
        }
 
-       @Override
+       /**
+        * Setter for user name
+        * <p>
+        * @param userName User name
+        */
        public void setUserName (final String userName) {
                this.userName = userName;
        }
 
-       @Override
+       /**
+        * Getter for clear-text user password
+        * <p>
+        * @return Clear-text user password
+        */
        public String getUserPassword () {
                return this.userPassword;
        }
 
-       @Override
+       /**
+        * Setter for clear-text user password
+        * <p>
+        * @param userPassword Clear-text user password
+        */
        public void setUserPassword (final String userPassword) {
                this.userPassword = userPassword;
        }
 
-       @Override
+       /**
+        * Getter for clear-text user password repeated
+        * <p>
+        * @return Clear-text user password repeated
+        */
        public String getUserPasswordRepeat () {
                return this.userPasswordRepeat;
        }
 
-       @Override
+       /**
+        * Setter for clear-text user password repeated
+        * <p>
+        * @param userPasswordRepeat Clear-text user password repeated
+        */
        public void setUserPasswordRepeat (final String userPasswordRepeat) {
                this.userPasswordRepeat = userPasswordRepeat;
        }
@@ -543,8 +620,16 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                }
        }
 
-       @Override
-       public String lockUserAccount (final User user) {
+       /**
+        * Locks selected user's account. This method makes sure that a lock reason
+        * is provided that th user later can read on login attempts.
+        * <p>
+        * @return Redirect outcome
+        */
+       public String lockUserAccount () {
+               // Get user instance
+               User user = this.beanHelper.getUser();
+
                // Is the user instance valid and CONFIRMED?
                if (null == user) {
                        // Throw NPE
@@ -593,8 +678,16 @@ public class PizzaAdminUserWebRequestBean extends BasePizzaController implements
                return "admin_show_user"; //NOI18N
        }
 
-       @Override
-       public String unlockUserAccount (final User user) {
+       /**
+        * Unlocks selected user's account. This method makes sure that the account
+        * is locked.
+        * <p>
+        * @return Redirect outcome
+        */
+       public String unlockUserAccount () {
+               // Get user instance
+               User user = this.beanHelper.getUser();
+
                // Is the user instance valid and CONFIRMED?
                if (null == user) {
                        // Throw NPE
index 41b2557d63609785335a991594cf43254951bd37..019d9be4655691a5e58f52f91dee7be59aeb3db1 100644 (file)
@@ -17,7 +17,6 @@
 package org.mxchange.pizzaapplication.beans.user;
 
 import java.io.Serializable;
-import org.mxchange.jusercore.model.user.User;
 
 /**
  * An interface for user beans
@@ -26,133 +25,4 @@ import org.mxchange.jusercore.model.user.User;
  */
 public interface PizzaAdminUserWebRequestController extends Serializable {
 
-       /**
-        * Adds user instance to database by preparing a complete user instance and
-        * sending it to the EJB. The data set in the controller is being verified,
-        * e.g. if the user name or email address is not used yet.
-        * <p>
-        * @return Redirect outcome
-        */
-       String addUser ();
-
-       /**
-        * Deletes given user account
-        * <p>
-        * @param user User instance to delete
-        * <p>
-        * @return Redirect outcome
-        */
-       String deleteUserData (final User user);
-
-       /**
-        * Edits currently loaded user's data in database.
-        * <p>
-        * @return Redirect outcome
-        */
-       String editUserData ();
-
-       /**
-        * Locks selected user's account. This method makes sure that a lock reason
-        * is provided that th user later can read on login attempts.
-        * <p>
-        * @param user User instance to be locked
-        * <p>
-        * @return Redirect outcome
-        */
-       String lockUserAccount (final User user);
-
-       /**
-        * Unlocks selected user's account. This method makes sure that the account
-        * is locked.
-        * <p>
-        * @param user User instance to be unlocked
-        * <p>
-        * @return Redirect outcome
-        */
-       String unlockUserAccount (final User user);
-
-       /**
-        * Getter for user name
-        * <p>
-        * @return User name
-        */
-       String getUserName ();
-
-       /**
-        * Setter for user name
-        * <p>
-        * @param userName User name
-        */
-       void setUserName (final String userName);
-
-       /**
-        * Getter for clear-text user password
-        * <p>
-        * @return Clear-text user password
-        */
-       String getUserPassword ();
-
-       /**
-        * Setter for clear-text user password
-        * <p>
-        * @param userPassword Clear-text user password
-        */
-       void setUserPassword (final String userPassword);
-
-       /**
-        * Getter for clear-text user password repeated
-        * <p>
-        * @return Clear-text user password repeated
-        */
-       String getUserPasswordRepeat ();
-
-       /**
-        * Setter for clear-text user password repeated
-        * <p>
-        * @param userPasswordRepeat Clear-text user password repeated
-        */
-       void setUserPasswordRepeat (final String userPasswordRepeat);
-
-       /**
-        * Getter for flag if user needs to change password
-        * <p>
-        * @return Flag if user needs to change password
-        */
-       Boolean getUserMustChangePassword ();
-
-       /**
-        * Setter for flag if user needs to change password
-        * <p>
-        * @param userMustChangePassword Flag if user needs to change password
-        */
-       void setUserMustChangePassword (final Boolean userMustChangePassword);
-
-       /**
-        * Getter for user lock reason
-        * <p>
-        * @return User lock reason
-        */
-       String getUserLockReason ();
-
-       /**
-        * Setter for user lock reason
-        * <p>
-        * @param userLockReason User lock reason
-        */
-       void setUserLockReason (final String userLockReason);
-
-       /**
-        * Getter for user delete reason
-        * <p>
-        * @return User delete reason
-        */
-       String getUserDeleteReason ();
-
-       /**
-        * Setter for user delete reason
-        * <p>
-        * @param userDeleteReason User delete reason
-        */
-       void setUserDeleteReason (final String userDeleteReason);
-
 }
index ff88dc63dc2b69d9fd86b37f71cb65be27622292..3bea1917ce045a59333da5b18563e43cc2d16fe4 100644 (file)
@@ -35,6 +35,7 @@ import javax.naming.NamingException;
 import org.mxchange.jcontacts.contact.Contact;
 import org.mxchange.jusercore.events.user.add.ObservableAdminAddedUserEvent;
 import org.mxchange.jusercore.events.user.delete.ObservableAdminDeletedUserEvent;
+import org.mxchange.jusercore.events.user.helper.created.ObservableHelperCreatedUserEvent;
 import org.mxchange.jusercore.events.user.linked.ObservableAdminLinkedUserEvent;
 import org.mxchange.jusercore.events.user.locked.ObservableAdminLockedUserEvent;
 import org.mxchange.jusercore.events.user.unlocked.ObservableAdminUnlockedUserEvent;
@@ -328,6 +329,36 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
                this.clear();
        }
 
+       /**
+        * Event observer for when a bean helper has successfully created a user
+        * instance, means the user exists. If the user does not exist, this event
+        * should not fire but instead a proper exception must be thrown.
+        * <p>
+        * @param event User created event
+        */
+       public void afterHelperCreatedUserEvent (@Observes final ObservableHelperCreatedUserEvent event) {
+               // Is the instance valid?
+               if (null == event) {
+                       // Throw NPE
+                       throw new NullPointerException("event is null"); //NOI18N
+               } else if (event.getCreatedUser() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("event.createdUser is null"); //NOI18N
+               } else if (event.getCreatedUser().getUserId() == null) {
+                       // Throw NPE again
+                       throw new NullPointerException("event.createdUser.userId is null"); //NOI18N
+               } else if (event.getCreatedUser().getUserId() < 1) {
+                       // Throw NPE again
+                       throw new NullPointerException(MessageFormat.format("event.createdUser.userId={0} is not valid", event.getCreatedUser().getUserId())); //NOI18N
+               }
+
+               // Get user instance
+               User user = event.getCreatedUser();
+
+               // Set all fields here
+               this.copyUser(user);
+       }
+
        /**
         * Event observer when user confirmed account.
         * <p>
@@ -387,6 +418,29 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
                }
        }
 
+       /**
+        * Event observer for user password changes
+        * <p>
+        * @param event Event being fired
+        */
+       public void afterUserPasswordChangedEvent (@Observes final ObservableUpdatedUserPasswordEvent event) {
+               // Is it valid?
+               if (null == event) {
+                       // Throw NPE
+                       throw new NullPointerException("event is null"); //NOI18N
+               } else if (event.getUserPassword() == null) {
+                       // Throw NPE
+                       throw new NullPointerException("event.userPassword is null"); //NOI18N
+               } else if (event.getUserPassword().isEmpty()) {
+                       // Throw NPE
+                       throw new IllegalArgumentException("event.userPassword is empty"); //NOI18N
+               }
+
+               // Set it here
+               this.setUserPassword(event.getUserPassword());
+               this.setUserPasswordRepeat(event.getUserPassword());
+       }
+
        /**
         * Event observer for new user registrations
         * <p>
@@ -498,6 +552,19 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
                return this.visibleUserList;
        }
 
+       @Override
+       public void clearUserName () {
+               // Clear it
+               this.setUserName(null);
+       }
+
+       @Override
+       public void clearUserPasswords () {
+               // Clear both
+               this.setUserPassword(null);
+               this.setUserPasswordRepeat(null);
+       }
+
        @Override
        public User createUserInstance (final boolean createContactData) {
                // Trace message
@@ -586,7 +653,7 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
                } else if (!this.userLoginController.ifCurrentPasswordMatches()) {
                        // Password not matching
                        throw new FaceletException(new UserPasswordMismatchException(this.userLoginController.getLoggedInUser()));
-               } else if (!this.featureController.isFeatureEnabled("change_user_personal_data")) {
+               } else if (!this.featureController.isFeatureEnabled("change_user_personal_data")) { //NOI18N
                        // Editing is not allowed
                        throw new IllegalStateException("User tried to edit personal data."); //NOI18N
                }
@@ -618,22 +685,38 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
                return "user_contact_data_saved"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for user id
+        * <p>
+        * @return User id
+        */
        public Long getUserId () {
                return this.userId;
        }
 
-       @Override
+       /**
+        * Setter for user id
+        * <p>
+        * @param userId User id
+        */
        public void setUserId (final Long userId) {
                this.userId = userId;
        }
 
-       @Override
+       /**
+        * Getter for user name
+        * <p>
+        * @return User name
+        */
        public String getUserName () {
                return this.userName;
        }
 
-       @Override
+       /**
+        * Setter for user name
+        * <p>
+        * @param userName User name
+        */
        public void setUserName (final String userName) {
                this.userName = userName;
        }
@@ -643,27 +726,47 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
                return this.userPassword;
        }
 
-       @Override
+       /**
+        * Setter for clear-text user password
+        * <p>
+        * @param userPassword Clear-text user password
+        */
        public void setUserPassword (final String userPassword) {
                this.userPassword = userPassword;
        }
 
-       @Override
+       /**
+        * Getter for clear-text user password repeated
+        * <p>
+        * @return Clear-text user password repeated
+        */
        public String getUserPasswordRepeat () {
                return this.userPasswordRepeat;
        }
 
-       @Override
+       /**
+        * Setter for clear-text user password repeated
+        * <p>
+        * @param userPasswordRepeat Clear-text user password repeated
+        */
        public void setUserPasswordRepeat (final String userPasswordRepeat) {
                this.userPasswordRepeat = userPasswordRepeat;
        }
 
-       @Override
+       /**
+        * Getter for user profile mode
+        * <p>
+        * @return User profile mode
+        */
        public ProfileMode getUserProfileMode () {
                return this.userProfileMode;
        }
 
-       @Override
+       /**
+        * Setter for user profile mode
+        * <p>
+        * @param userProfileMode User profile mode
+        */
        public void setUserProfileMode (final ProfileMode userProfileMode) {
                this.userProfileMode = userProfileMode;
        }
@@ -671,7 +774,7 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
        @Override
        public boolean ifBothPasswordsEmptyAllowed () {
                // Check feature first
-               return ((this.featureController.isFeatureEnabled("allow_user_registration_empty_password")) &&
+               return ((this.featureController.isFeatureEnabled("allow_user_registration_empty_password")) && //NOI18N
                                ((this.getUserPassword() == null) || (this.getUserPassword().isEmpty())) &&
                                ((this.getUserPasswordRepeat() == null) || (this.getUserPasswordRepeat().isEmpty())));
        }
@@ -928,9 +1031,8 @@ public class PizzaUserWebSessionBean extends BasePizzaController implements Pizz
                this.setUserProfileMode(null);
 
                // - other data
-               this.setUserName(null);
-               this.setUserPassword(null);
-               this.setUserPasswordRepeat(null);
+               this.clearUserName();
+               this.clearUserPasswords();
        }
 
        /**
index b56053d128ad4f4a7790fd030995495cbae5ed02..77ccf77e956e473abb268c3f1b1f09ccdccc34a9 100644 (file)
@@ -22,7 +22,6 @@ import org.mxchange.jcontacts.contact.Contact;
 import org.mxchange.jusercore.exceptions.UserEmailAddressNotFoundException;
 import org.mxchange.jusercore.exceptions.UserNotFoundException;
 import org.mxchange.jusercore.model.user.User;
-import org.mxchange.jusercore.model.user.profilemodes.ProfileMode;
 
 /**
  * An interface for user beans
@@ -33,9 +32,28 @@ public interface PizzaUserWebSessionController extends Serializable {
 
        /**
         * Minimum password length
+        * <p>
+        * @deprecated Better set as context parameter
         */
        public static final Integer MINIMUM_PASSWORD_LENGTH = 5;
 
+       /**
+        * Getter for clear-text user password
+        * <p>
+        * @return Clear-text user password
+        */
+       String getUserPassword ();
+
+       /**
+        * Clears both user passwords
+        */
+       void clearUserPasswords ();
+
+       /**
+        * Clears user name
+        */
+       void clearUserName ();
+
        /**
         * Checks if both user passwords are left empty and if this is enabled
         * (allowed) in context parameter. If true, the calling bean should create a
@@ -127,76 +145,6 @@ public interface PizzaUserWebSessionController extends Serializable {
         */
        User createUserLogin ();
 
-       /**
-        * Getter for user id
-        * <p>
-        * @return User id
-        */
-       Long getUserId ();
-
-       /**
-        * Setter for user id
-        * <p>
-        * @param userId User id
-        */
-       void setUserId (final Long userId);
-
-       /**
-        * Getter for user name
-        * <p>
-        * @return User name
-        */
-       String getUserName ();
-
-       /**
-        * Setter for user name
-        * <p>
-        * @param userName User name
-        */
-       void setUserName (final String userName);
-
-       /**
-        * Getter for clear-text user password
-        * <p>
-        * @return Clear-text user password
-        */
-       String getUserPassword ();
-
-       /**
-        * Setter for clear-text user password
-        * <p>
-        * @param userPassword Clear-text user password
-        */
-       void setUserPassword (final String userPassword);
-
-       /**
-        * Getter for clear-text user password repeated
-        * <p>
-        * @return Clear-text user password repeated
-        */
-       String getUserPasswordRepeat ();
-
-       /**
-        * Setter for clear-text user password repeated
-        * <p>
-        * @param userPasswordRepeat Clear-text user password repeated
-        */
-       void setUserPasswordRepeat (final String userPasswordRepeat);
-
-       /**
-        * Getter for user profile mode
-        * <p>
-        * @return User profile mode
-        */
-       ProfileMode getUserProfileMode ();
-
-       /**
-        * Setter for user profile mode
-        * <p>
-        * @param userProfileMode User profile mode
-        */
-       void setUserProfileMode (final ProfileMode userProfileMode);
-
        /**
         * Checks whether all required personal data is set
         * <p>
index 1e1e982b83e5c4d738f0f8d08716ccd48f59df00..1934d77d6a16e266755d232bdd9d517cbddbce7f 100644 (file)
@@ -92,7 +92,11 @@ public class PizzaEmailChangeWebSessionBean extends BasePizzaController implemen
                super();
        }
 
-       @Override
+       /**
+        * Changes logged-in user's email address if the current password matches.
+        * <p>
+        * @return Redirect outcome
+        */
        public String doUserChangeEmailAddress () {
                // This method shall only be called if the user is logged-in
                if (!this.userLoginController.isUserLoggedIn()) {
@@ -146,22 +150,38 @@ public class PizzaEmailChangeWebSessionBean extends BasePizzaController implemen
                return "login_email_change_queued"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for email address 1 (changing)
+        * <p>
+        * @return Email address
+        */
        public String getEmailAddress () {
                return this.emailAddress;
        }
 
-       @Override
+       /**
+        * Setter for email address 1 (changing)
+        * <p>
+        * @param emailAddress Email address 1
+        */
        public void setEmailAddress (final String emailAddress) {
                this.emailAddress = emailAddress;
        }
 
-       @Override
+       /**
+        * Getter for email address 2 (repeat changing)
+        * <p>
+        * @return Email address 2
+        */
        public String getEmailAddressRepeat () {
                return this.emailAddressRepeat;
        }
 
-       @Override
+       /**
+        * Setter for email address 2 (repeat changing)
+        * <p>
+        * @param emailAddressRepeat Email address 2
+        */
        public void setEmailAddressRepeat (final String emailAddressRepeat) {
                this.emailAddressRepeat = emailAddressRepeat;
        }
index 2e3ad3a9ba5fe47eb2242a7f81d4f71e88775f35..bd26aaf595f3531bdee9ee8ad78e87ab4ad53841 100644 (file)
@@ -25,34 +25,6 @@ import java.io.Serializable;
  */
 public interface PizzaEmailChangeWebSessionController extends Serializable {
 
-       /**
-        * Getter for email address 1 (changing)
-        * <p>
-        * @return Email address
-        */
-       String getEmailAddress ();
-
-       /**
-        * Setter for email address 1 (changing)
-        * <p>
-        * @param emailAddress Email address 1
-        */
-       void setEmailAddress (final String emailAddress);
-
-       /**
-        * Getter for email address 2 (repeat changing)
-        * <p>
-        * @return Email address 2
-        */
-       String getEmailAddressRepeat ();
-
-       /**
-        * Setter for email address 2 (repeat changing)
-        * <p>
-        * @param emailAddressRepeat Email address 2
-        */
-       void setEmailAddressRepeat (final String emailAddressRepeat);
-
        /**
         * Checks whether all required are set for changing email address
         * <p>
@@ -60,11 +32,4 @@ public interface PizzaEmailChangeWebSessionController extends Serializable {
         */
        boolean isRequiredChangeEmailAddressSet ();
 
-       /**
-        * Changes logged-in user's email address if the current password matches.
-        * <p>
-        * @return New target page
-        */
-       String doUserChangeEmailAddress ();
-
 }
index a945acb37ddc0fe3e2033b3e675c7ba14bb4fea8..a2e82bfa72497381a1731bb59064c47a8bfa17f0 100644 (file)
@@ -177,7 +177,12 @@ public class PizzaUserLoginWebSessionBean extends BasePizzaController implements
                this.updatePasswordHistory(event.getPasswordHistory());
        }
 
-       @Override
+       /**
+        * Logout for administrator area. If a logged-in user instance exists, it is
+        * being logged-out, too.
+        * <p>
+        * @return Outcome (should be redirected)
+        */
        public String doAdminLogout () {
                // Is a user logged-in?
                if (this.isUserLoggedIn()) {
@@ -195,17 +200,21 @@ public class PizzaUserLoginWebSessionBean extends BasePizzaController implements
                return "index?faces-redirect=true"; //NOI18N
        }
 
-       @Override
+       /**
+        * Logins the user, if the account is found, confirmed and unlocked.
+        * <p>
+        * @return Redirect target
+        */
        public String doUserLogin () {
                // Get user instance
                User user = this.userController.createUserLogin();
 
                // Create login container
-               LoginContainer container = new UserLoginContainer(user, this.userController.getUserPassword());
+               LoginContainer loginContainer = new UserLoginContainer(user, this.userController.getUserPassword());
 
                try {
                        // Call bean
-                       User confirmedUser = this.userLoginBean.validateUserAccountStatus(container);
+                       User confirmedUser = this.userLoginBean.validateUserAccountStatus(loginContainer);
 
                        // All fine here so set it here
                        this.setLoggedInUser(confirmedUser);
@@ -241,7 +250,11 @@ public class PizzaUserLoginWebSessionBean extends BasePizzaController implements
                }
        }
 
-       @Override
+       /**
+        * Logout for current user by invalidating the current session.
+        * <p>
+        * @return Outcome (should be redirected)
+        */
        public String doUserLogout () {
                // Is loggedInUser set?
                if (this.getLoggedInUser() == null) {
@@ -289,12 +302,20 @@ public class PizzaUserLoginWebSessionBean extends BasePizzaController implements
                this.loggedInUser = loggedInUser;
        }
 
-       @Override
+       /**
+        * Getter for current password (clear text)
+        * <p>
+        * @return Current password
+        */
        public String getUserCurrentPassword () {
                return this.userCurrentPassword;
        }
 
-       @Override
+       /**
+        * Setter for current password (clear text)
+        * <p>
+        * @param userCurrentPassword Current password
+        */
        public void setUserCurrentPassword (final String userCurrentPassword) {
                this.userCurrentPassword = userCurrentPassword;
        }
index 354253b51510b455d3551e24ec59a21eac7a32dc..89271d3da2ddac515a750a07277d5f4195b28402 100644 (file)
@@ -18,7 +18,6 @@ package org.mxchange.pizzaapplication.beans.user.login;
 
 import java.io.Serializable;
 import java.util.List;
-import javax.ejb.Local;
 import org.mxchange.jusercore.model.user.User;
 import org.mxchange.jusercore.model.user.password_history.PasswordHistory;
 
@@ -27,7 +26,6 @@ import org.mxchange.jusercore.model.user.password_history.PasswordHistory;
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaUserLoginWebSessionController extends Serializable {
 
        /**
@@ -53,28 +51,6 @@ public interface PizzaUserLoginWebSessionController extends Serializable {
         */
        void setBaseTemplatePathName (final String baseTemplatePathName);
 
-       /**
-        * Logout for administrator area. If a logged-in user instance exists, it is
-        * being logged-out, too.
-        * <p>
-        * @return Outcome (should be redirected)
-        */
-       String doAdminLogout ();
-
-       /**
-        * Logins the user, if the account is found, confirmed and unlocked.
-        * <p>
-        * @return Redirect target
-        */
-       String doUserLogin ();
-
-       /**
-        * Logout for current user by invalidating the current session.
-        * <p>
-        * @return Outcome (should be redirected)
-        */
-       String doUserLogout ();
-
        /**
         * Getter for logged-in user instance
         * <p>
@@ -110,20 +86,6 @@ public interface PizzaUserLoginWebSessionController extends Serializable {
         */
        boolean isInvisible ();
 
-       /**
-        * Setter for current password (clear text)
-        * <p>
-        * @param userCurrentPassword Current password
-        */
-       void setUserCurrentPassword (final String userCurrentPassword);
-
-       /**
-        * Getter for current password (clear text)
-        * <p>
-        * @return Current password
-        */
-       String getUserCurrentPassword ();
-
        /**
         * Checks whether the (previously entered) current password matches with
         * from the user instance.
index c6f6aedd7cc99cab8193d9e445af2a72e43abefc..898747ae4f15c64228fdedadeb90225bef600d78 100644 (file)
@@ -103,7 +103,13 @@ public class PizzaUserPasswordWebRequestBean extends BasePizzaController impleme
                super();
        }
 
-       @Override
+       /**
+        * Changes logged-in user's password. It must not match with current
+        * password and should not appear in password history list for X
+        * (configurable) entries.
+        * <p>
+        * @return Redirect outcome
+        */
        public String doChangePassword () {
                // This method shall only be called if the user is logged-in
                if (!this.userLoginController.isUserLoggedIn()) {
@@ -173,7 +179,7 @@ public class PizzaUserPasswordWebRequestBean extends BasePizzaController impleme
                        PasswordHistory passwordHistory = this.userBean.updateUserPassword(user, baseUrl);
 
                        // Fire event
-                       this.userUpdatedPasswordEvent.fire(new UpdatedUserPasswordEvent(passwordHistory));
+                       this.userUpdatedPasswordEvent.fire(new UpdatedUserPasswordEvent(passwordHistory,this.getUserPassword()));
                } catch (final UserNotFoundException | UserStatusUnconfirmedException | UserStatusLockedException ex) {
                        // Clear bean
                        this.clear();
@@ -189,32 +195,56 @@ public class PizzaUserPasswordWebRequestBean extends BasePizzaController impleme
                return "login_data_saved"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for current clear-text user password
+        * <p>
+        * @return Current clear-text user password
+        */
        public String getUserCurrentPassword () {
                return this.userCurrentPassword;
        }
 
-       @Override
+       /**
+        * Setter for current clear-text user password
+        * <p>
+        * @param userCurrentPassword Current clear-text user password
+        */
        public void setUserCurrentPassword (final String userCurrentPassword) {
                this.userCurrentPassword = userCurrentPassword;
        }
 
-       @Override
+       /**
+        * Getter for clear-text user password
+        * <p>
+        * @return Clear-text user password
+        */
        public String getUserPassword () {
                return this.userPassword;
        }
 
-       @Override
+       /**
+        * Setter for clear-text user password
+        * <p>
+        * @param userPassword Clear-text user password
+        */
        public void setUserPassword (final String userPassword) {
                this.userPassword = userPassword;
        }
 
-       @Override
+       /**
+        * Getter for clear-text user password repeated
+        * <p>
+        * @return Clear-text user password repeated
+        */
        public String getUserPasswordRepeat () {
                return this.userPasswordRepeat;
        }
 
-       @Override
+       /**
+        * Setter for clear-text user password repeated
+        * <p>
+        * @param userPasswordRepeat Clear-text user password repeated
+        */
        public void setUserPasswordRepeat (final String userPasswordRepeat) {
                this.userPasswordRepeat = userPasswordRepeat;
        }
index 6868653b2f0e5844e348a4dd9f4948f07f3ca392..fe109d00b4d3d84ac7200db887eb9cee440439e9 100644 (file)
 package org.mxchange.pizzaapplication.beans.user.password;
 
 import java.io.Serializable;
-import javax.ejb.Local;
 
 /**
  * An interface for user beans
  * <p>
  * @author Roland Häder<roland@mxchange.org>
  */
-@Local
 public interface PizzaUserPasswordWebRequestController extends Serializable {
 
-       /**
-        * Getter for clear-text user password
-        * <p>
-        * @return Clear-text user password
-        */
-       String getUserPassword ();
-
-       /**
-        * Setter for clear-text user password
-        * <p>
-        * @param userPassword Clear-text user password
-        */
-       void setUserPassword (final String userPassword);
-
-       /**
-        * Getter for current clear-text user password
-        * <p>
-        * @return Current clear-text user password
-        */
-       String getUserCurrentPassword ();
-
-       /**
-        * Setter for current clear-text user password
-        * <p>
-        * @param userCurrentPassword Current clear-text user password
-        */
-       void setUserCurrentPassword (final String userCurrentPassword);
-
-       /**
-        * Getter for clear-text user password repeated
-        * <p>
-        * @return Clear-text user password repeated
-        */
-       String getUserPasswordRepeat ();
-
-       /**
-        * Setter for clear-text user password repeated
-        * <p>
-        * @param userPasswordRepeat Clear-text user password repeated
-        */
-       void setUserPasswordRepeat (final String userPasswordRepeat);
-
-       /**
-        * Changes logged-in user's password. It must not match with current
-        * password and should not appear in password history list for X
-        * (configurable) entries.
-        * <p>
-        * @return Redirect outcome
-        */
-       String doChangePassword ();
-
        /**
         * Checks if all 3 passwords are set: old password, 2x new password
         * <p>
index 258a9133ad169c9f74a83e724a2ec67b394049b7..7bfdc12a6390a2dbd6228f518d3c678d41eb2f8a 100644 (file)
@@ -16,7 +16,6 @@
  */
 package org.mxchange.pizzaapplication.beans.user.register;
 
-import java.text.MessageFormat;
 import javax.annotation.PostConstruct;
 import javax.enterprise.context.SessionScoped;
 import javax.enterprise.event.Event;
@@ -34,9 +33,13 @@ import org.mxchange.jusercore.exceptions.DataRepeatMismatchException;
 import org.mxchange.jusercore.exceptions.EmailAddressAlreadyRegisteredException;
 import org.mxchange.jusercore.exceptions.UserNameAlreadyRegisteredException;
 import org.mxchange.jusercore.model.user.User;
+import org.mxchange.jusercore.model.user.password_history.PasswordHistory;
+import org.mxchange.jusercore.model.user.password_history.UserPasswordHistory;
 import org.mxchange.jusercore.model.user.status.UserAccountStatus;
 import org.mxchange.juserlogincore.events.registration.ObservableUserRegisteredEvent;
 import org.mxchange.juserlogincore.events.registration.UserRegisteredEvent;
+import org.mxchange.juserlogincore.events.user.password_change.ObservableUpdatedUserPasswordEvent;
+import org.mxchange.juserlogincore.events.user.password_change.UpdatedUserPasswordEvent;
 import org.mxchange.juserlogincore.login.UserLoginUtils;
 import org.mxchange.juserlogincore.model.user.register.UserRegistrationSessionBeanRemote;
 import org.mxchange.pizzaapplication.beans.BasePizzaController;
@@ -89,7 +92,14 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
        private PizzaUserWebSessionController userController;
 
        /**
-        * An en event fireable when a new user has registered
+        * An event being fired when a user password was changed
+        */
+       @Inject
+       @Any
+       private Event<ObservableUpdatedUserPasswordEvent> userPasswordChangedEvent;
+
+       /**
+        * An event being fired when a new user has registered
         */
        @Inject
        @Any
@@ -103,7 +113,12 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                super();
        }
 
-       @Override
+       /**
+        * Registers the user, if not found. Otherwise this method should throw an
+        * exception.
+        * <p>
+        * @return Redirection target
+        */
        public String doFinishRegistration () {
                // Is registration enabled?
                if (!this.featureController.isFeatureEnabled("user_registration")) { //NOI18N
@@ -131,7 +146,7 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                                throw new FaceletException(new UserNameAlreadyRegisteredException(user));
                        } else {
                                // May happen here, reset field
-                               this.userController.setUserName(null);
+                               this.userController.clearUserName();
                                this.showFacesMessage("form_register_single:userName", "ERROR_USER_NAME_ALREADY_USED"); //NOI18N
                                return ""; //NOI18N
                        }
@@ -142,8 +157,7 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                                throw new FaceletException(new EmailAddressAlreadyRegisteredException(user));
                        } else {
                                // May happen here, reset fields
-                               this.contactController.setEmailAddress(null);
-                               this.contactController.setEmailAddressRepeat(null);
+                               this.contactController.clearEmailAddresses();
                                this.showFacesMessage("form_register_single:emailAddressRepeat", "ERROR_EMAIL_ADDRESS_ALREADY_USED"); //NOI18N
                                return ""; //NOI18N
                        }
@@ -151,11 +165,10 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                        // Is multi-page enabled?
                        if (this.featureController.isFeatureEnabled("user_register_multiple_page")) { //NOI18N
                                // Not same email address entered, should not happen here
-                               throw new FaceletException(new DataRepeatMismatchException(MessageFormat.format("Email addresses not matching: {0} != {1}", this.contactController.getEmailAddress(), this.contactController.getEmailAddressRepeat()))); //NOI18N
+                               throw new FaceletException(new DataRepeatMismatchException("Email addresses not matching.")); //NOI18N
                        } else {
                                // May happen here, reset fields
-                               this.contactController.setEmailAddress(null);
-                               this.contactController.setEmailAddressRepeat(null);
+                               this.contactController.clearEmailAddresses();
                                this.showFacesMessage("form_register_single:emailAddressRepeat", "ERROR_EMAIL_ADDRESSES_MISMATCHING"); //NOI18N
                                return ""; //NOI18N
                        }
@@ -168,9 +181,11 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                                // Both passwords are left empty and is allowed, then generate a random password
                                randomPassword = UserLoginUtils.createRandomPassword(PizzaUserWebSessionController.MINIMUM_PASSWORD_LENGTH);
 
-                               // Set it in both fields
-                               this.userController.setUserPassword(randomPassword);
-                               this.userController.setUserPasswordRepeat(randomPassword);
+                               // Generate (ignored) password-history
+                               PasswordHistory passwordHistory = new UserPasswordHistory(randomPassword, user);
+
+                               // Fire event
+                               this.userPasswordChangedEvent.fire(new UpdatedUserPasswordEvent(passwordHistory, randomPassword));
                        }
                }
 
@@ -216,7 +231,13 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                }
        }
 
-       @Override
+       /**
+        * Handles registration request send from first page. The (maybe) entered
+        * user name and email address is not used and that privacy and T&C are
+        * accepted.
+        * <p>
+        * @return Redirect
+        */
        public String doRegisterMultiPage1 () {
                // Is registration enabled?
                if (!this.featureController.isFeatureEnabled("user_registration")) { //NOI18N
@@ -233,21 +254,19 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                        throw new NullPointerException("user is null after createUserInstance() was called"); //NOI18N
                } else if ((this.featureController.isFeatureEnabled("user_login_require_user_name")) && (this.userController.isUserNameRegistered(user))) { //NOI18N
                        // User name is already used, so clear it
-                       this.userController.setUserName(null);
+                       this.userController.clearUserName();
                        this.showFacesMessage("form_register_page1:userName", "ERROR_USER_NAME_ALREADY_USED"); //NOI18N
                        return ""; //NOI18N
                } else if (!this.contactController.isSameEmailAddressEntered()) {
                        // Not same email address entered, clear both
-                       this.contactController.setEmailAddress(null);
-                       this.contactController.setEmailAddressRepeat(null);
+                       this.contactController.clearEmailAddresses();
                        this.showFacesMessage("form_register_page1:emailAddressRepeat", "ERROR_EMAIL_ADDRESSES_MISMATCHING"); //NOI18N
                        return ""; //NOI18N
                } else if (!this.userController.isSamePasswordEntered()) {
                        // Is multi-page enabled?
                        if (this.featureController.isFeatureEnabled("user_register_multiple_page")) { //NOI18N
                                // Unset both
-                               this.userController.setUserPassword(null);
-                               this.userController.setUserPasswordRepeat(null);
+                               this.userController.clearUserPasswords();
 
                                // Output faces message
                                this.showFacesMessage("form_register_page1:userPassword", "ERROR_USER_PASSWORD_EMPTY"); //NOI18N
@@ -257,9 +276,11 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                                // Both passwords are left empty and is allowed, then generate a random password
                                String randomPassword = UserLoginUtils.createRandomPassword(PizzaUserWebSessionController.MINIMUM_PASSWORD_LENGTH);
 
-                               // Set it in both fields
-                               this.userController.setUserPassword(randomPassword);
-                               this.userController.setUserPasswordRepeat(randomPassword);
+                               // Generate (ignored) password-history
+                               PasswordHistory passwordHistory = new UserPasswordHistory(randomPassword, user);
+
+                               // Fire event
+                               this.userPasswordChangedEvent.fire(new UpdatedUserPasswordEvent(passwordHistory, randomPassword));
                        }
                }
 
@@ -273,8 +294,7 @@ public class PizzaUserRegisterWebSessionBean extends BasePizzaController impleme
                // Check if email address is registered
                if (this.contactController.isEmailAddressRegistered(user.getUserContact())) {
                        // Email address has already been taken, clear both
-                       this.contactController.setEmailAddress(null);
-                       this.contactController.setEmailAddressRepeat(null);
+                       this.contactController.clearEmailAddresses();
                        this.showFacesMessage("form_register_page1:emailAddress", "ERROR_EMAIL_ADDRESS_ALREADY_USED"); //NOI18N
                        return ""; //NOI18N
                }
index 4dbedc4646e3dc4cd300eccf8837a625941553b4..20e39c5a0d4767d069e94a80de07967dff80dd0e 100644 (file)
@@ -25,21 +25,4 @@ import java.io.Serializable;
  */
 public interface PizzaUserRegisterWebSessionController extends Serializable {
 
-       /**
-        * Registers the user, if not found. Otherwise this method should throw an
-        * exception.
-        * <p>
-        * @return Redirection target
-        */
-       String doFinishRegistration ();
-
-       /**
-        * Handles registration request send from first page. The (maybe) entered
-        * user name and email address is not used and that privacy and T&C are
-        * accepted.
-        * <p>
-        * @return Redirect
-        */
-       String doRegisterMultiPage1 ();
-
 }
index 233cb726045145c7fff032cdcaccaaa85fc82154..7483dfc973a5006db10c4353e7981edc51cb3e00 100644 (file)
@@ -91,7 +91,12 @@ public class PizzaResendLinkWebSessionBean extends BasePizzaController implement
                super();
        }
 
-       @Override
+       /**
+        * Resends (new) confirmation link to given email address, if found.
+        * Otherwise an exception is thrown. On success a redirect takes place.
+        * <p>
+        * @return Redirect outcome
+        */
        public String doResendLink () {
                // The email address should not be empty as the JSF validates this
                if (this.getEmailAddress() == null) {
@@ -163,12 +168,20 @@ public class PizzaResendLinkWebSessionBean extends BasePizzaController implement
                return "user_resend_done"; //NOI18N
        }
 
-       @Override
+       /**
+        * Getter for email address 1 (changing)
+        * <p>
+        * @return Email address
+        */
        public String getEmailAddress () {
                return this.emailAddress;
        }
 
-       @Override
+       /**
+        * Setter for email address 1 (changing)
+        * <p>
+        * @param emailAddress Email address 1
+        */
        public void setEmailAddress (final String emailAddress) {
                this.emailAddress = emailAddress;
        }
index 0b994f5714cd47327b23a29657c1380c0f9c3c2d..7819ec579e72a509bae53ec3df3ecc20c7ce1ce4 100644 (file)
@@ -25,26 +25,4 @@ import java.io.Serializable;
  */
 public interface PizzaResendLinkWebSessionController extends Serializable {
 
-       /**
-        * Getter for email address 1 (changing)
-        * <p>
-        * @return Email address
-        */
-       String getEmailAddress ();
-
-       /**
-        * Setter for email address 1 (changing)
-        * <p>
-        * @param emailAddress Email address 1
-        */
-       void setEmailAddress (final String emailAddress);
-
-       /**
-        * Resends (new) confirmation link to given email address, if found.
-        * Otherwise an exception is thrown. On success a redirect takes place.
-        * <p>
-        * @return Redirection target
-        */
-       String doResendLink ();
-
 }
index 8ed4e71e910fb96c89838ecc0923d52fa33b48eb..7ab91ab7dfd198289683031968929950ffe03823 100644 (file)
                                </div>
 
                                <div class="table_right_medium">
-                                       <h:selectOneMenu  styleClass="select" id="country" value="#{adminContactController.country}" converter="CountryConverter">
+                                       <h:selectOneMenu styleClass="select" id="country" value="#{adminContactController.contactCountry}">
+                                               <f:converter converterId="CountryConverter" />
                                                <f:selectItem itemValue="" itemLabel="#{msg.NONE_SELECTED}" />
                                                <f:selectItems value="#{countryController.allCountries()}" var="country" itemValue="#{country}" itemLabel="#{country.countryCode} (#{msg[country.countryI18nKey]})" />
                                        </h:selectOneMenu>
index 65ec7dbc1cc175f764d9de35bddb7cb26f0b41af..df6fefcd984a4070010f72bb9beb18f7f4d6c7b4 100644 (file)
@@ -49,7 +49,7 @@
 
                                <div class="table_right_medium">
                                        <ui:include src="/WEB-INF/templates/input_fields/fax_input_fields.tpl">
-                                               <ui:param name="targetController" value="#{adminPhoneController}" />
+                                               <ui:param name="targetController" value="#{adminContactPhoneController}" />
                                        </ui:include>
                                </div>
 
@@ -67,9 +67,7 @@
                        <div class="table_footer">
                                <h:commandButton styleClass="reset right_space" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
 
-                               <h:commandButton styleClass="submit" id="submit_add_link_contact_fax" type="submit" action="#{adminContactPhoneController.doLinkAddFaxNumber(beanHelper.contact)}" value="#{msg.BUTTON_ADMIN_LINK_ADD_CONTACT_FAX_NUMBER}">
-                                       <f:param name="contactId" value="#{param.contactId}" />
-                               </h:commandButton>
+                               <h:commandButton styleClass="submit" id="submit_add_link_contact_fax" type="submit" action="#{adminContactPhoneController.doLinkMainFaxNumber()}" value="#{msg.BUTTON_ADMIN_LINK_ADD_CONTACT_FAX_NUMBER}" />
                        </div>
                </h:panelGroup>
        </h:form>
index 1c29b5379aefa2d5b91055b11b9f49705b8d4aa3..f34fbfafdda7f85be4cca114238165c0fbe858c6 100644 (file)
@@ -39,7 +39,7 @@
                                </div>
 
                                <div class="table_footer">
-                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_fax" value="#{msg.BUTTON_ADMIN_DELETE_FAX}" action="#{adminPhoneController.deleteFaxData(beanHelper.faxNumber)}" />
+                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_fax" value="#{msg.BUTTON_ADMIN_DELETE_FAX}" action="#{adminPhoneController.deleteFaxData()}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index 88707af6a553d1534b794cfdbd3fc6cda7de0b59..b6f722256c647f7a5a52264424489265ab584115 100644 (file)
@@ -33,7 +33,7 @@
 
                                <div class="table_footer">
                                        <h:commandButton styleClass="reset right_space" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
-                                       <h:commandButton styleClass="submit" type="submit" id="edit_fax" value="#{msg.BUTTON_ADMIN_EDIT_FAX_NUMBER}" action="#{adminPhoneController.editFaxData(beanHelper.faxNumber)}" />
+                                       <h:commandButton styleClass="submit" type="submit" id="edit_fax" value="#{msg.BUTTON_ADMIN_EDIT_FAX_NUMBER}" action="#{adminPhoneController.doChangeFaxNumber()}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index f04185a98563e6271f693c201cc5bda2a7dafb0e..16cd072636adaf7cac57aab806cea9f7ac8c7327 100644 (file)
@@ -39,7 +39,7 @@
                                </div>
 
                                <div class="table_footer">
-                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_landline" value="#{msg.BUTTON_ADMIN_DELETE_LAND_LINE}" action="#{adminPhoneController.deleteLandLineData(beanHelper.landLineNumber)}" />
+                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_landline" value="#{msg.BUTTON_ADMIN_DELETE_LAND_LINE}" action="#{adminPhoneController.deleteLandLineData()}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index 28acb79acf2ca1e296874dbc66884873645fcdeb..17e9529dc3ca42332e4ec338bc5fbc0351826e26 100644 (file)
@@ -33,7 +33,7 @@
 
                                <div class="table_footer">
                                        <h:commandButton styleClass="reset right_space" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
-                                       <h:commandButton styleClass="submit" type="submit" id="edit_landline" value="#{msg.BUTTON_ADMIN_EDIT_LAND_LINE_NUMBER}" action="#{adminPhoneController.editLandLineData(beanHelper.landLineNumber)}" />
+                                       <h:commandButton styleClass="submit" type="submit" id="edit_landline" value="#{msg.BUTTON_ADMIN_EDIT_LAND_LINE_NUMBER}" action="#{adminPhoneController.doChangeLandLineNumber()}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index ab7219d3d6e6d61e05c48acf9bf15ce9e06aaa02..3c8b2d14f4ccbcc6fbfd15f2e418e158cd695d20 100644 (file)
@@ -39,7 +39,7 @@
                                </div>
 
                                <div class="table_footer">
-                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_mobile" value="#{msg.BUTTON_ADMIN_DELETE_MOBILE_NUMBER}" action="#{adminPhoneController.deleteMobileData(beanHelper.mobileNumber)}" />
+                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_mobile" value="#{msg.BUTTON_ADMIN_DELETE_MOBILE_NUMBER}" action="#{adminPhoneController.deleteMobileData()}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index ab6b2bdd4956e2441de613f511fe26a5553af619..fef08d8671fdcc487016da21e48897e3773cdd2f 100644 (file)
@@ -33,7 +33,7 @@
 
                                <div class="table_footer">
                                        <h:commandButton styleClass="reset right_space" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
-                                       <h:commandButton styleClass="submit" type="submit" id="edit_mobile" value="#{msg.BUTTON_ADMIN_EDIT_MOBILE_NUMBER}" action="#{adminPhoneController.editMobileData(beanHelper.mobileNumber)}" />
+                                       <h:commandButton styleClass="submit" type="submit" id="edit_mobile" value="#{msg.BUTTON_ADMIN_EDIT_MOBILE_NUMBER}" action="#{adminPhoneController.doUpdateMobileNumber()}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index 25c8161f69fce8a9702f05dc73a22710927f6ada..e1aec671f0368b3306ac3950678fa92c20642531 100644 (file)
@@ -57,7 +57,7 @@
 
                                <div class="table_footer">
                                        <h:commandButton styleClass="reset right_space" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
-                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_user" value="#{msg.BUTTON_ADMIN_DELETE_USER}" action="#{adminUserController.deleteUserData(beanHelper.user)}" />
+                                       <h:commandButton styleClass="delete_button" type="submit" id="delete_user" value="#{msg.BUTTON_ADMIN_DELETE_USER}" action="#{adminUserController.deleteUserData()}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index 15bf8cbb2967c1b290ac49159d9ba9cbcaeb5ff5..79731fec36003225af4d6ac78a281e0c5b578e4f 100644 (file)
                                                <h:outputText value="#{msg.ADMIN_EXPORT_CONTACT_LANDLINE_NUMBER}" />
                                        </f:facet>
 
-                                       <h:outputText value="#{adminContactController.generatePhoneNumber(user.userContact.contactLandLineNumber)}" />
+                                       <h:outputText value="#{adminContactController.generateLandLineNumber(user.userContact.contactLandLineNumber)}" />
                                </p:column>
 
                                <p:column>
                                                <h:outputText value="#{msg.ADMIN_EXPORT_CONTACT_FAX_NUMBER}" />
                                        </f:facet>
 
-                                       <h:outputText value="#{adminContactController.generatePhoneNumber(user.userContact.contactFaxNumber)}" />
+                                       <h:outputText value="#{adminContactController.generateLandLineNumber(user.userContact.contactFaxNumber)}" />
                                </p:column>
 
                                <p:column>
index b48474cf6bff35394c37ce38a7013ab6cc18f789..c2b01b3eaac3e7f13246ecd2205e5c5ca1972538 100644 (file)
@@ -80,7 +80,7 @@
                                <div class="table_footer">
                                        <h:commandButton styleClass="reset right_space" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
 
-                                       <h:commandButton styleClass="submit" type="submit" action="#{adminUserController.lockUserAccount(beanHelper.user)}" value="#{msg.BUTTON_ADMIN_LOCK_USER_ACCOUNT}" />
+                                       <h:commandButton styleClass="submit" type="submit" action="#{adminUserController.lockUserAccount()}" value="#{msg.BUTTON_ADMIN_LOCK_USER_ACCOUNT}" />
                                </div>
                        </h:panelGroup>
                </h:form>
index 08acbb8a8b0a29c52284433e31221332a9cd4e58..288b64d82023ecb56f215119be2de942e550a4f7 100644 (file)
                                <div class="table_footer">
                                        <h:commandButton styleClass="reset right_space" type="reset" value="#{msg.BUTTON_RESET_FORM}" />
 
-                                       <h:commandButton styleClass="submit" type="submit" action="#{adminUserController.unlockUserAccount(beanHelper.user)}" value="#{msg.BUTTON_ADMIN_UNLOCK_USER_ACCOUNT}" />
+                                       <h:commandButton styleClass="submit" type="submit" action="#{adminUserController.unlockUserAccount()}" value="#{msg.BUTTON_ADMIN_UNLOCK_USER_ACCOUNT}" />
                                </div>
                        </h:panelGroup>
                </h:form>