=== modified file 'dhis-2/dhis-web/dhis-web-api/src/main/java/org/hisp/dhis/api/controller/ReportController.java' --- dhis-2/dhis-web/dhis-web-api/src/main/java/org/hisp/dhis/api/controller/ReportController.java 2012-03-22 15:04:58 +0000 +++ dhis-2/dhis-web/dhis-web-api/src/main/java/org/hisp/dhis/api/controller/ReportController.java 2012-04-08 18:46:37 +0000 @@ -136,8 +136,8 @@ @RequestMapping( value = {"/{uid}/data", "/{uid}/data.pdf"}, method = RequestMethod.GET ) public void getReportAsPdf( @PathVariable( "uid" ) String uid, @RequestParam( value = "ou", required = false ) String organisationUnitUid, - @RequestParam( value = "pe", required = false ) String period, HttpServletResponse response ) - throws Exception + @RequestParam( value = "pe", required = false ) String period, + HttpServletResponse response ) throws Exception { getReport( uid, organisationUnitUid, period, response, "pdf", ContextUtils.CONTENT_TYPE_PDF, false ); } @@ -145,8 +145,8 @@ @RequestMapping( value = "/{uid}/data.xls", method = RequestMethod.GET ) public void getReportAsXls( @PathVariable( "uid" ) String uid, @RequestParam( value = "ou", required = false ) String organisationUnitUid, - @RequestParam( value = "pe", required = false ) String period, HttpServletResponse response ) - throws Exception + @RequestParam( value = "pe", required = false ) String period, + HttpServletResponse response ) throws Exception { getReport( uid, organisationUnitUid, period, response, "xls", ContextUtils.CONTENT_TYPE_EXCEL, true ); } @@ -160,7 +160,7 @@ { Report report = reportService.getReport( uid ); - if ( report.hasReportTable() && report.getReportTable().hasReportParams() + if ( organisationUnitUid == null && report.hasReportTable() && report.getReportTable().hasReportParams() && report.getReportTable().getReportParams().isOrganisationUnitSet() ) { organisationUnitUid = organisationUnitService.getRootOrganisationUnits().iterator().next().getUid();