[GCI-142] Check bad code practices through Findbugs or Sonar and fix …

…those issues (openmrs#3083)

null

GCI-142 minor fix

del xtra brace + deepcopy setters

GCI-142 fix travis

api/src/main/java/org/openmrs/CohortMembership.java

@@ -94,15 +94,15 @@ public void setPatientId(Integer patientId) {
  }
 
  public Date getStartDate() {
- return startDate;
+ return startDate != null ? (Date) startDate.clone() : null;
  }
 
  public void setStartDate(Date startDate) {
- this.startDate = startDate;
+  this.startDate = startDate != null ? new Date(startDate.getTime()) : null;
  }
 
  public Date getEndDate() {
- return endDate;
+ return endDate != null ? (Date) endDate.clone() : null;
  }
 
  /**
@@ -112,9 +112,10 @@ public Date getEndDate() {
  * @param endDate
  */
  public void setEndDate(Date endDate) {
- this.endDate = endDate;
+ this.endDate = endDate != null ? new Date(endDate.getTime()) : null;
  }
 
+
  /**
  * Sorts by following fields, in order:
  * <ol>