=== modified file 'dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityController.java' --- dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityController.java 2012-12-18 18:21:05 +0000 +++ dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityController.java 2012-12-28 12:09:04 +0000 @@ -40,6 +40,7 @@ import org.hisp.dhis.user.CurrentUserService; import org.hisp.dhis.web.webapi.v1.domain.Facilities; import org.hisp.dhis.web.webapi.v1.domain.Facility; +import org.hisp.dhis.web.webapi.v1.utils.ContextUtils; import org.hisp.dhis.web.webapi.v1.utils.MessageResponseUtils; import org.hisp.dhis.web.webapi.v1.utils.ValidationUtils; import org.hisp.dhis.web.webapi.v1.validation.group.Create; @@ -62,6 +63,7 @@ import org.springframework.web.bind.annotation.RequestMethod; import org.springframework.web.bind.annotation.RequestParam; +import javax.servlet.http.HttpServletRequest; import javax.validation.ConstraintViolation; import javax.validation.Validator; import javax.validation.groups.Default; @@ -83,9 +85,9 @@ /** * @author Morten Olav Hansen */ -@Controller( value = "facility-controller-" + FredController.PREFIX ) -@RequestMapping( FacilityController.RESOURCE_PATH ) -@PreAuthorize( "hasRole('M_dhis-web-api-fred') or hasRole('ALL')" ) +@Controller(value = "facility-controller-" + FredController.PREFIX) +@RequestMapping(FacilityController.RESOURCE_PATH) +@PreAuthorize("hasRole('M_dhis-web-api-fred') or hasRole('ALL')") public class FacilityController { public static final String RESOURCE_PATH = "/" + FredController.PREFIX + "/facilities"; @@ -231,13 +233,14 @@ return facility; } - @RequestMapping( value = "", method = RequestMethod.GET ) - public String readFacilities( Model model, @RequestParam( required = false ) Boolean active, - @RequestParam( value = "updatedSince", required = false ) Date lastUpdated, - @RequestParam( value = "allProperties", required = false, defaultValue = "true" ) Boolean allProperties, - @RequestParam( value = "fields", required = false ) String fields, - @RequestParam( value = "limit", required = false ) Integer limit, - @RequestParam( value = "offset", required = false ) Integer offset ) + @RequestMapping(value = "", method = RequestMethod.GET) + public String readFacilities( Model model, @RequestParam(required = false) Boolean active, + @RequestParam(value = "updatedSince", required = false) Date lastUpdated, + @RequestParam(value = "allProperties", required = false, defaultValue = "true") Boolean allProperties, + @RequestParam(value = "fields", required = false) String fields, + @RequestParam(value = "limit", required = false) Integer limit, + @RequestParam(value = "offset", required = false) Integer offset, + HttpServletRequest request ) { Facilities facilities = new Facilities(); List allOrganisationUnits; @@ -310,6 +313,7 @@ model.addAttribute( "esc", StringEscapeUtils.class ); model.addAttribute( "entity", facilities ); + ContextUtils.populateContextPath( model, request ); model.addAttribute( "baseUrl", linkTo( FredController.class ).toString() ); model.addAttribute( "pageName", "facilities" ); model.addAttribute( "page", FredController.PREFIX + "/facilities.vm" ); @@ -317,10 +321,11 @@ return FredController.PREFIX + "/layout"; } - @RequestMapping( value = "/{id}", method = RequestMethod.GET ) + @RequestMapping(value = "/{id}", method = RequestMethod.GET) public String readFacility( Model model, @PathVariable String id, - @RequestParam( value = "allProperties", required = false, defaultValue = "true" ) Boolean allProperties, - @RequestParam( value = "fields", required = false ) String fields ) + @RequestParam(value = "allProperties", required = false, defaultValue = "true") Boolean allProperties, + @RequestParam(value = "fields", required = false) String fields, + HttpServletRequest request ) { OrganisationUnit organisationUnit = organisationUnitService.getOrganisationUnit( id ); List organisationUnitLevels = organisationUnitService.getOrganisationUnitLevels(); @@ -338,6 +343,8 @@ Collections.sort( dataSets, IdentifiableObjectNameComparator.INSTANCE ); model.addAttribute( "dataSets", dataSets ); + ContextUtils.populateContextPath( model, request ); + model.addAttribute( "baseUrl", linkTo( FredController.class ).toString() ); model.addAttribute( "pageName", "facility" ); model.addAttribute( "page", FredController.PREFIX + "/facility.vm" ); @@ -382,8 +389,8 @@ // POST JSON //-------------------------------------------------------------------------- - @RequestMapping( value = "", method = RequestMethod.POST ) - @PreAuthorize( "hasRole('F_FRED_CREATE') or hasRole('ALL')" ) + @RequestMapping(value = "", method = RequestMethod.POST) + @PreAuthorize("hasRole('F_FRED_CREATE') or hasRole('ALL')") public ResponseEntity createFacility( @RequestBody Facility facility ) throws IOException { Set> constraintViolations = validator.validate( facility, Default.class, Create.class ); @@ -432,8 +439,8 @@ // PUT JSON //-------------------------------------------------------------------------- - @RequestMapping( value = "/{id}", method = RequestMethod.PUT, consumes = MediaType.APPLICATION_JSON_VALUE ) - @PreAuthorize( "hasRole('F_FRED_UPDATE') or hasRole('ALL')" ) + @RequestMapping(value = "/{id}", method = RequestMethod.PUT, consumes = MediaType.APPLICATION_JSON_VALUE) + @PreAuthorize("hasRole('F_FRED_UPDATE') or hasRole('ALL')") public ResponseEntity updateFacility( @PathVariable String id, @RequestBody Facility facility ) throws IOException { facility.setId( id ); @@ -505,8 +512,8 @@ // DELETE JSON //-------------------------------------------------------------------------- - @RequestMapping( value = "/{id}", method = RequestMethod.DELETE ) - @PreAuthorize( "hasRole('F_FRED_DELETE') or hasRole('ALL')" ) + @RequestMapping(value = "/{id}", method = RequestMethod.DELETE) + @PreAuthorize("hasRole('F_FRED_DELETE') or hasRole('ALL')") public ResponseEntity deleteFacility( @PathVariable String id ) throws HierarchyViolationException { OrganisationUnit organisationUnit = organisationUnitService.getOrganisationUnit( id ); @@ -525,7 +532,7 @@ // EXCEPTION HANDLERS //-------------------------------------------------------------------------- - @ExceptionHandler( { DeleteNotAllowedException.class, HierarchyViolationException.class } ) + @ExceptionHandler({ DeleteNotAllowedException.class, HierarchyViolationException.class }) public ResponseEntity exceptionHandler( Exception ex ) { return new ResponseEntity( ex.getMessage(), HttpStatus.FORBIDDEN );