Downloads Documentation Community
Contribute
Contribute Demo






Show Sidebar
Login | Register


Show
Ignore:
Timestamp:
05/09/08 21:34:56 (8 months ago)
Author:
catullus
Message:

data_synchronization_bidirectionaly: merge from tunk [4129:4157].

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • openmrs/branches/data_synchronization_bidirectional/src/web/org/openmrs/web/controller/patient/NewPatientFormController.java

    r4121 r4161  
    151151                                        for (int i=0; i<identifiers.length;i++) { 
    152152                                                // arguments for the spring error messages 
    153                                                 String[] args = {identifiers[i]}; 
     153                                                String id = identifiers[i].trim(); 
     154                                                String[] args = {id}; 
    154155                                                 
    155156                                                // add the new identifier only if they put in some identifier string 
    156                                                 if (identifiers[i].length() > 0) { 
     157                                                if (id.length() > 0) { 
    157158                                                         
    158159                                                        // set up the actual identifier java object 
    …… 
    173174                                                                loc = es.getLocation(Integer.valueOf(locs[i])); 
    174175                                                         
    175                                                         PatientIdentifier pi = new PatientIdentifier(identifiers[i], pit, loc); 
    176                                                         pi.setPreferred(pref.equals(identifiers[i]+types[i])); 
     176                                                        PatientIdentifier pi = new PatientIdentifier(id, pit, loc); 
     177                                                        pi.setPreferred(pref.equals(id+types[i])); 
    177178                                                        pi.setGuid(null); 
    178179                                                        newIdentifiers.add(pi); 
    179180                                                         
    180181                                                        if (log.isDebugEnabled()) { 
    181                                                                 log.debug("Creating patient identifier with identifier: " + identifiers[i]); 
     182                                                                log.debug("Creating patient identifier with identifier: " + id); 
    182183                                                                log.debug("and type: " + types[i]); 
    183184                                                                log.debug("and location: " + locs[i]); 
    …… 
    185186                                                 
    186187                                                        try { 
    187                                                                 if (pit.hasCheckDigit() && !OpenmrsUtil.isValidCheckDigit(identifiers[i])) { 
    188                                                                         log.error("hasCheckDigit and is not valid: " + pit.getName() + " " + identifiers[i]); 
     188                                                                if (pit.hasCheckDigit() && !OpenmrsUtil.isValidCheckDigit(id)) { 
     189                                                                        log.error("hasCheckDigit and is not valid: " + pit.getName() + " " + id); 
    189190                                                                        String msg = getMessageSourceAccessor().getMessage("error.checkdigits.verbose", args); 
    190191                                                                        errors.rejectValue("identifier", msg); 
    191192                                                                } 
    192         //                                                      else if (pit.hasCheckDigit() == false && identifiers[i].contains("-")) { 
    193         //                                                              log.error("hasn't CheckDigit and contains '-': " + pit.getName() + " " + identifiers[i]); 
    194         //                                                              String[] args2 = {"-", identifiers[i]};  
     193        //                                                      else if (pit.hasCheckDigit() == false && id.contains("-")) { 
     194        //                                                              log.error("hasn't CheckDigit and contains '-': " + pit.getName() + " " + id); 
     195        //                                                              String[] args2 = {"-", id};  
    195196        //                                                              String msg = getMessageSourceAccessor().getMessage("error.character.invalid", args2); 
    196197        //                                                              errors.rejectValue("identifier", msg); 
    197198        //                                                      } 
    198199                                                        } catch (Exception e) { 
    199                                                                 log.error("exception thrown with: " + pit.getName() + " " + identifiers[i]); 
     200                                                                log.error("exception thrown with: " + pit.getName() + " " + id); 
    200201                                                                log.error("Error while adding patient identifiers to savedIdentifier list", e); 
    201202                                                                String msg = getMessageSourceAccessor().getMessage("error.checkdigits", args);