=== modified file 'dhis-2/dhis-api/src/main/java/org/hisp/dhis/patient/Patient.java' --- dhis-2/dhis-api/src/main/java/org/hisp/dhis/patient/Patient.java 2011-01-11 01:56:03 +0000 +++ dhis-2/dhis-api/src/main/java/org/hisp/dhis/patient/Patient.java 2011-01-21 04:08:36 +0000 @@ -452,7 +452,28 @@ public String getFullName() { - return firstName + " " + middleName + " " + lastName; + boolean space = false; + String name = ""; + + if ( firstName != null && firstName.length() != 0 ) + { + name = firstName; + space = true; + } + if ( middleName != null && middleName.length() != 0 ) + { + if ( space ) + name += " "; + name += middleName; + space = true; + } + if ( lastName != null && lastName.length() != 0 ) + { + if ( space ) + name += " "; + name += lastName; + } + return name; } public String getBloodGroup() === added directory 'dhis-2/dhis-api/src/test/java/org/hisp/dhis/patient' === added file 'dhis-2/dhis-api/src/test/java/org/hisp/dhis/patient/PatientTest.java' --- dhis-2/dhis-api/src/test/java/org/hisp/dhis/patient/PatientTest.java 1970-01-01 00:00:00 +0000 +++ dhis-2/dhis-api/src/test/java/org/hisp/dhis/patient/PatientTest.java 2011-01-21 04:08:36 +0000 @@ -0,0 +1,35 @@ +package org.hisp.dhis.patient; + +import static junit.framework.Assert.*; + +import org.junit.Test; + +public class PatientTest +{ + + + @Test + public void testFullname() { + + Patient patient = new Patient(); + + assertEquals( "", patient.getFullName() ); + + patient.setFirstName( "firstName" ); + assertEquals( "firstName", patient.getFullName() ); + + patient.setLastName( "lastName" ); + assertEquals( "firstName lastName", patient.getFullName() ); + + patient.setMiddleName( "middleName" ); + assertEquals( "firstName middleName lastName", patient.getFullName() ); + + patient.setFirstName( "" ); + assertEquals( "middleName lastName", patient.getFullName() ); + + patient.setFirstName( "firstName middleName lastName" ); + patient.setMiddleName( null ); + patient.setLastName( null ); + assertEquals( "firstName middleName lastName", patient.getFullName() ); + } +}