=== modified file 'dhis-2/dhis-web/dhis-web-commons/src/main/java/org/hisp/dhis/commons/action/GetOrganisationUnitTreeAction.java' --- dhis-2/dhis-web/dhis-web-commons/src/main/java/org/hisp/dhis/commons/action/GetOrganisationUnitTreeAction.java 2014-09-28 20:14:48 +0000 +++ dhis-2/dhis-web/dhis-web-commons/src/main/java/org/hisp/dhis/commons/action/GetOrganisationUnitTreeAction.java 2014-09-28 21:21:49 +0000 @@ -192,27 +192,19 @@ return "partial"; } - Collection userOrganisationUnits; + Collection userOrganisationUnits = new ArrayList<>(); User user = currentUserService.getCurrentUser(); - if ( user != null && user.getOrganisationUnits() != null && user.getOrganisationUnits().size() > 0 ) + if ( user != null && user.hasOrganisationUnit() ) { userOrganisationUnits = new ArrayList<>( user.getOrganisationUnits() ); rootOrganisationUnits = new ArrayList<>( user.getOrganisationUnits() ); } - else + else if ( currentUserService.currentUserIsSuper() || user == null ) { - if ( user != null && user.getOrganisationUnits() != null && currentUserService.currentUserIsSuper() ) - { - userOrganisationUnits = new ArrayList<>( organisationUnitService.getRootOrganisationUnits() ); - rootOrganisationUnits = new ArrayList<>( organisationUnitService.getRootOrganisationUnits() ); - } - else - { - userOrganisationUnits = new ArrayList<>(); - rootOrganisationUnits = new ArrayList<>(); - } + userOrganisationUnits = new ArrayList<>( organisationUnitService.getRootOrganisationUnits() ); + rootOrganisationUnits = new ArrayList<>( organisationUnitService.getRootOrganisationUnits() ); } if ( !versionOnly && !rootOrganisationUnits.isEmpty() )