svn commit: r823530 - /ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java

Author: jleroux
Date: Fri Oct  9 12:39:25 2009
New Revision: 823530

URL: http://svn.apache.org/viewvc?rev=823530&view=rev
Log:
Fix some issues in POS client profile screen when used OOTB

Modified:
    ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java

Modified: ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java?rev=823530&r1=823529&r2=823530&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java (original)
+++ ofbiz/trunk/specialpurpose/pos/src/org/ofbiz/pos/PosTransaction.java Fri Oct  9 12:39:25 2009
@@ -1554,30 +1554,33 @@
         dynamicView.addAlias("PT", "partyId");
         dynamicView.addAlias("PT", "statusId");
         dynamicView.addAlias("PT", "partyTypeId");
+        dynamicView.addMemberEntity("PUL", "PartyAndUserLogin");
+        dynamicView.addAlias("PUL", "userLoginId");
         dynamicView.addMemberEntity("PE", "Person");
         dynamicView.addAlias("PE", "partyId");
         dynamicView.addAlias("PE", "lastName");
         dynamicView.addAlias("PE", "memberId");
         dynamicView.addAlias("PE", "lastNameLocal");
         dynamicView.addViewLink("PT", "PE", Boolean.FALSE, ModelKeyMap.makeKeyMapList("partyId"));
+        dynamicView.addViewLink("PT", "PUL", Boolean.FALSE, ModelKeyMap.makeKeyMapList("partyId"));
 
         Boolean onlyPhone = UtilValidate.isEmpty(name) && UtilValidate.isEmpty(email) && UtilValidate.isNotEmpty(phone) && UtilValidate.isEmpty(card);
         if (!onlyPhone) {
             // ContactMech (email)
             dynamicView.addMemberEntity("PM", "PartyContactMechPurpose");            
-            dynamicView.addMemberEntity("CM", "ContactMech");
             dynamicView.addAlias("PM", "contactMechId");
             dynamicView.addAlias("PM", "contactMechPurposeTypeId");            
             dynamicView.addAlias("PM", "thruDate");
+            dynamicView.addMemberEntity("CM", "ContactMech");
             dynamicView.addAlias("CM", "infoString");            
             dynamicView.addViewLink("PT", "PM", Boolean.FALSE, ModelKeyMap.makeKeyMapList("partyId"));
             dynamicView.addViewLink("PM", "CM", Boolean.FALSE, ModelKeyMap.makeKeyMapList("contactMechId"));
         } else {
             dynamicView.addMemberEntity("PM", "PartyContactMechPurpose");            
-            dynamicView.addMemberEntity("TN", "TelecomNumber");
             dynamicView.addAlias("PM", "contactMechId");
             dynamicView.addAlias("PM", "thruDate");
             dynamicView.addAlias("PM", "contactMechPurposeTypeId");            
+            dynamicView.addMemberEntity("TN", "TelecomNumber");
             dynamicView.addAlias("TN", "contactNumber");
             dynamicView.addViewLink("PT", "PM", Boolean.FALSE, ModelKeyMap.makeKeyMapList("partyId"));
             dynamicView.addViewLink("PM", "TN", Boolean.FALSE, ModelKeyMap.makeKeyMapList("contactMechId"));
@@ -1603,6 +1606,8 @@
             // This allows to get all clients when any informations has been entered
             andExprs.add(EntityCondition.makeCondition(EntityCondition.makeCondition("statusId", EntityOperator.EQUALS, null), EntityOperator.OR, EntityCondition.makeCondition("statusId", EntityOperator.NOT_EQUAL, "PARTY_DISABLED")));
             andExprs.add(EntityCondition.makeCondition("partyTypeId", EntityOperator.EQUALS, "PERSON")); // Only persons for now...
+            andExprs.add(EntityCondition.makeCondition("userLoginId", EntityOperator.NOT_EQUAL, null)); // Should have a login
+            andExprs.add(EntityCondition.makeCondition("memberId", EntityOperator.NOT_EQUAL, null)); // Should have a card Id (we take into account only the person created here)
             if (UtilValidate.isNotEmpty(name)) {
                 andExprs.add(EntityCondition.makeCondition("lastName", EntityOperator.EQUALS, name));
             }
@@ -1781,7 +1786,7 @@
                 pos.showDialog("dialog/error/exception", e.getMessage());
                 return result;
             }
-            GenericValue partyLogin = userLogins.get(0); // We need at least a party's login ...
+            GenericValue partyLogin = userLogins.get(0); // We need at least a party's login, we are sure there is one as we keep only users with at least one login
             GenericValue  person = null;
             try {
                 person = session.getDelegator().findByPrimaryKey("Person", UtilMisc.toMap("partyId", partyId));
@@ -1792,7 +1797,7 @@
             }
             
             if (UtilValidate.isNotEmpty(name) && !person.getString("lastName").equals(name)
-                    || UtilValidate.isNotEmpty(card) && !person.getString("memberId").equals(card)) {
+                    || UtilValidate.isNotEmpty(card) && !card.equals(person.getString("memberId"))) {
                 svcCtx.put("partyId", partyId);
                 svcCtx.put("firstName", ""); // Needed by service updatePerson
                 svcCtx.put("userLogin", partyLogin);
@@ -1801,7 +1806,7 @@
                 }
                 if (UtilValidate.isNotEmpty(card)) {
                     svcCtx.put("memberId", card);
-                    if (!person.getString("memberId").equals(card)) {
+                    if (!card.equals(person.getString("memberId"))) {
                         // Update password
                         UtilProperties.setPropertyValue("security.properties", "password.accept.encrypted.and.plain", "true");
                         try {
@@ -1907,27 +1912,29 @@
                     return result;
                 }
                 GenericValue PartyTelecomNumber = PartyTelecomNumbers.get(0); // we suppose only one phone number (should be ok anyway because of the contactMechPurposeTypeId == "PHONE_HOME")
-                String contactNumber = PartyTelecomNumber.getString("contactNumber");
-                if (!contactNumber.equals(phone)) {
-                    String newContactMechId = PartyTelecomNumber.getString("contactMechId");
-    
-                    svcCtx.put("userLogin", partyLogin);
-                    svcCtx.put("contactNumber", phone);
-                    svcCtx.put("contactMechPurposeTypeId", "PHONE_HOME");
-                    if (UtilValidate.isNotEmpty(PartyTelecomNumbers)) {
-                        svcCtx.put("contactMechId", newContactMechId);
-                    }
-                    
-                    try {
-                        svcRes = dispatcher.runSync("createUpdatePartyTelecomNumber", svcCtx);
-                    } catch (GenericServiceException e) {
-                        Debug.logError(e, module);
-                        pos.showDialog("dialog/error/exception", e.getMessage());
-                        return result;
-                    }
-                    if (ServiceUtil.isError(svcRes)) {
-                        pos.showDialog("dialog/error/exception", ServiceUtil.getErrorMessage(svcRes));
-                        return result;
+                if (UtilValidate.isNotEmpty(PartyTelecomNumber)) { // Should not be needed but in case memberId has been used for another purpose avoid an NPE (we test memberId not null initially to keep  only POS created users)
+                    String contactNumber = PartyTelecomNumber.getString("contactNumber");
+                    if (!phone.equals(contactNumber)) {
+                        String newContactMechId = PartyTelecomNumber.getString("contactMechId");
+        
+                        svcCtx.put("userLogin", partyLogin);
+                        svcCtx.put("contactNumber", phone);
+                        svcCtx.put("contactMechPurposeTypeId", "PHONE_HOME");
+                        if (UtilValidate.isNotEmpty(PartyTelecomNumbers)) {
+                            svcCtx.put("contactMechId", newContactMechId);
+                        }
+                        
+                        try {
+                            svcRes = dispatcher.runSync("createUpdatePartyTelecomNumber", svcCtx);
+                        } catch (GenericServiceException e) {
+                            Debug.logError(e, module);
+                            pos.showDialog("dialog/error/exception", e.getMessage());
+                            return result;
+                        }
+                        if (ServiceUtil.isError(svcRes)) {
+                            pos.showDialog("dialog/error/exception", ServiceUtil.getErrorMessage(svcRes));
+                            return result;
+                        }
                     }
                 }
             }