Merge pull request #132 from nisum-inc/FEATURE/EmployeeListServiceShouldNotReturnInActiveEmployee

EmployeeListServiceShouldNotReturnInActiveEmployee

src/main/java/com/nisum/mytime/repository/EmployeeRolesRepo.java

@@ -19,4 +19,6 @@ public interface EmployeeRolesRepo
             String functionalGroup);
 
     List<EmployeeRoles> findByEmpStatusOrderByEmployeeNameAsc(String empStatus);
+    
+    List<EmployeeRoles> findByEmpStatus(String status); 
 }

src/main/java/com/nisum/mytime/service/UserServiceImpl.java

@@ -123,7 +123,8 @@ public class UserServiceImpl implements UserService {
 
     @Override
     public List<EmployeeRoles> getEmployeeRoles() throws MyTimeException {
-        return employeeRolesRepo.findAll();
+        //return employeeRolesRepo.findAll();
+    	return employeeRolesRepo.findByEmpStatus(MyTimeUtils.ACTIVE);
     }
 
     @Override