Changeset 7711 for RevitalizationWS


Ignore:
Timestamp:
08/11/10 12:52:36 (9 years ago)
Author:
mnagni
Message:

While the XSD defined the EmailContact? as optional the code returned a NullPointer? if not given. Fixed

Location:
RevitalizationWS/ws/DPWS/trunk/src/main/java/ndg/services/revitalization
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • RevitalizationWS/ws/DPWS/trunk/src/main/java/ndg/services/revitalization/ejb/ModelManager.java

    r7660 r7711  
    243243                                while (iterator.hasNext()) { 
    244244                                        item = iterator.next(); 
    245                                         if (!provider.getProviderEmails().contains(item)) 
     245                                        if (provider.getProviderEmails() == null || !provider.getProviderEmails().contains(item)) 
    246246                                                em.remove(item); 
    247247                                } 
  • RevitalizationWS/ws/DPWS/trunk/src/main/java/ndg/services/revitalization/executor/ExecuteDoNewProvider.java

    r7505 r7711  
    5454        } 
    5555         
    56         private void prepareProviderEmails(ProviderCommonType common, ProviderCommon provider) {                 
    57                 if (common.getProviderContacts() == null) 
    58                         return; 
    59                  
    60                 List<EmailContactType> contacts = common.getProviderContacts(); 
    61                  
    62                 if (contacts.size() == 0) 
    63                         return; 
     56        private void prepareProviderEmails(ProviderCommonType common, ProviderCommon provider) { 
     57                ProviderEmail email = null; 
    6458                 
    6559                if (provider.getProviderEmails() == null) 
    6660                        provider.setProviderEmails(new ArrayList<ProviderEmail>()); 
    67                  
    68                 //Contact email 
    69                 ProviderEmail email = null;      
    70                 for (EmailContactType item : contacts){ 
    71                         email = new ProviderEmail(); 
    72                         //email = getNewUpdateEmail(provider.getProviderEmails(), item.getID());                         
    73                         email.setEmail(item.getEmail()); 
    74                         email.setRealName(item.getName()); 
    75                         email.setId(item.getID()); 
    76                         provider.getProviderEmails().add(email); 
    77                 } 
    7861                 
    7962                //Admin email 
     
    9275                }                
    9376                provider.setEmailToAdmin(admin.isMonitor()); 
     77                 
     78                if (common.getProviderContacts() == null) 
     79                        return; 
     80                 
     81                List<EmailContactType> contacts = common.getProviderContacts(); 
     82                 
     83                if (contacts.size() == 0) 
     84                        return;          
     85                 
     86                //Contact email  
     87                for (EmailContactType item : contacts){ 
     88                        email = new ProviderEmail(); 
     89                        //email = getNewUpdateEmail(provider.getProviderEmails(), item.getID());                         
     90                        email.setEmail(item.getEmail()); 
     91                        email.setRealName(item.getName()); 
     92                        email.setId(item.getID()); 
     93                        provider.getProviderEmails().add(email); 
     94                } 
    9495        }        
    9596         
     
    132133                List<ProviderEmail> contacts = provider.getProviderEmails(); 
    133134                 
    134                 if (contacts.size() == 0) 
     135                if (contacts == null || contacts.size() == 0) 
    135136                        return; 
    136137                 
Note: See TracChangeset for help on using the changeset viewer.