dhis2-devs team mailing list archive
-
dhis2-devs team
-
Mailing list archive
-
Message #21388
[Branch ~dhis2-devs-core/dhis2/trunk] Rev 10107: FRED-API: moved exception handlers out into a controller advice class, now applies to both contro...
------------------------------------------------------------
revno: 10107
committer: Morten Olav Hansen <mortenoh@xxxxxxxxx>
branch nick: dhis2
timestamp: Sat 2013-03-09 18:32:50 +0300
message:
FRED-API: moved exception handlers out into a controller advice class, now applies to both controllers
added:
dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityAdvice.java
modified:
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/FacilityServiceController.java
--
lp:dhis2
https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk
Your team DHIS 2 developers is subscribed to branch lp:dhis2.
To unsubscribe from this branch go to https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk/+edit-subscription
=== added file 'dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityAdvice.java'
--- dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityAdvice.java 1970-01-01 00:00:00 +0000
+++ dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityAdvice.java 2013-03-09 15:32:50 +0000
@@ -0,0 +1,93 @@
+package org.hisp.dhis.web.webapi.v1.controller;
+
+/*
+ * Copyright (c) 2004-2013, University of Oslo
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ * * Redistributions of source code must retain the above copyright notice, this
+ * list of conditions and the following disclaimer.
+ * * Redistributions in binary form must reproduce the above copyright notice,
+ * this list of conditions and the following disclaimer in the documentation
+ * and/or other materials provided with the distribution.
+ * * Neither the name of the HISP project nor the names of its contributors may
+ * be used to endorse or promote products derived from this software without
+ * specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+ * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR
+ * ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
+ * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+import org.hisp.dhis.common.DeleteNotAllowedException;
+import org.hisp.dhis.hierarchy.HierarchyViolationException;
+import org.hisp.dhis.web.webapi.v1.exception.DuplicateCodeException;
+import org.hisp.dhis.web.webapi.v1.exception.DuplicateUidException;
+import org.hisp.dhis.web.webapi.v1.exception.DuplicateUuidException;
+import org.hisp.dhis.web.webapi.v1.exception.ETagVerificationException;
+import org.hisp.dhis.web.webapi.v1.exception.FacilityNotFoundException;
+import org.hisp.dhis.web.webapi.v1.exception.UuidFormatException;
+import org.hisp.dhis.web.webapi.v1.utils.MessageUtils;
+import org.springframework.http.HttpStatus;
+import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.ControllerAdvice;
+import org.springframework.web.bind.annotation.ExceptionHandler;
+import org.springframework.web.client.HttpClientErrorException;
+import org.springframework.web.client.HttpServerErrorException;
+import org.springframework.web.client.HttpStatusCodeException;
+
+import java.io.IOException;
+
+/**
+ * @author Morten Olav Hansen <mortenoh@xxxxxxxxx>
+ */
+@ControllerAdvice
+public class FacilityAdvice
+{
+ //--------------------------------------------------------------------------
+ // EXCEPTION HANDLERS
+ //--------------------------------------------------------------------------
+
+ @ExceptionHandler( { HttpClientErrorException.class, HttpServerErrorException.class } )
+ public ResponseEntity<String> statusCodeExceptionHandler( HttpStatusCodeException ex ) throws IOException
+ {
+ return new ResponseEntity<String>( MessageUtils.jsonMessage( ex.getStatusText(),
+ ex.getMessage() ), ex.getStatusCode() );
+ }
+
+ @ExceptionHandler( { DeleteNotAllowedException.class, HierarchyViolationException.class } )
+ public ResponseEntity<String> handleForbidden( Exception ex ) throws IOException
+ {
+ return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.FORBIDDEN.toString(),
+ ex.getMessage() ), HttpStatus.FORBIDDEN );
+ }
+
+ @ExceptionHandler( { ETagVerificationException.class, UuidFormatException.class } )
+ public ResponseEntity<String> handlePreconditionFailed( Exception ex ) throws IOException
+ {
+ return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.PRECONDITION_FAILED.toString(),
+ ex.getMessage() ), HttpStatus.PRECONDITION_FAILED );
+ }
+
+ @ExceptionHandler( { FacilityNotFoundException.class } )
+ public ResponseEntity<String> handleNotFound( Exception ex ) throws IOException
+ {
+ return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.NOT_FOUND.toString(),
+ ex.getMessage() ), HttpStatus.NOT_FOUND );
+ }
+
+ @ExceptionHandler( { DuplicateCodeException.class, DuplicateUidException.class, DuplicateUuidException.class } )
+ public ResponseEntity<String> handleConflict( Exception ex ) throws IOException
+ {
+ return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.CONFLICT.toString(),
+ ex.getMessage() ), HttpStatus.CONFLICT );
+ }
+}
=== 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 2013-03-09 09:38:51 +0000
+++ dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityController.java 2013-03-09 15:32:50 +0000
@@ -30,7 +30,6 @@
import org.apache.commons.lang3.StringEscapeUtils;
import org.codehaus.jackson.map.ObjectMapper;
import org.hisp.dhis.api.controller.organisationunit.OrganisationUnitLevelController;
-import org.hisp.dhis.common.DeleteNotAllowedException;
import org.hisp.dhis.common.comparator.IdentifiableObjectNameComparator;
import org.hisp.dhis.dataset.DataSet;
import org.hisp.dhis.dataset.DataSetService;
@@ -49,8 +48,6 @@
import org.hisp.dhis.web.webapi.v1.exception.FacilityNotFoundException;
import org.hisp.dhis.web.webapi.v1.exception.UuidFormatException;
import org.hisp.dhis.web.webapi.v1.utils.ContextUtils;
-import org.hisp.dhis.web.webapi.v1.utils.MessageUtils;
-import org.hisp.dhis.web.webapi.v1.utils.ObjectMapperFactoryBean;
import org.hisp.dhis.web.webapi.v1.utils.ValidationUtils;
import org.hisp.dhis.web.webapi.v1.validation.group.Create;
import org.hisp.dhis.web.webapi.v1.validation.group.Update;
@@ -65,7 +62,6 @@
import org.springframework.ui.Model;
import org.springframework.util.DigestUtils;
import org.springframework.web.bind.WebDataBinder;
-import org.springframework.web.bind.annotation.ExceptionHandler;
import org.springframework.web.bind.annotation.InitBinder;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.RequestBody;
@@ -73,8 +69,6 @@
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.web.bind.annotation.RequestParam;
import org.springframework.web.client.HttpClientErrorException;
-import org.springframework.web.client.HttpServerErrorException;
-import org.springframework.web.client.HttpStatusCodeException;
import javax.servlet.http.HttpServletRequest;
import javax.validation.ConstraintViolation;
@@ -610,7 +604,7 @@
facility = conversionService.convert( organisationUnit, Facility.class );
List<OrganisationUnitLevel> organisationUnitLevels = organisationUnitService.getOrganisationUnitLevels();
addHierarchyPropertyToFacility( organisationUnitLevels, facility );
- json = new ObjectMapperFactoryBean().getObject().writeValueAsString( facility );
+ json = objectMapper.writeValueAsString( facility );
return new ResponseEntity<String>( json, headers, HttpStatus.OK );
}
@@ -641,45 +635,6 @@
}
//--------------------------------------------------------------------------
- // EXCEPTION HANDLERS
- //--------------------------------------------------------------------------
-
- @ExceptionHandler({ HttpClientErrorException.class, HttpServerErrorException.class })
- public ResponseEntity<String> statusCodeExceptionHandler( HttpStatusCodeException ex ) throws IOException
- {
- return new ResponseEntity<String>( MessageUtils.jsonMessage( ex.getStatusText(),
- ex.getMessage() ), ex.getStatusCode() );
- }
-
- @ExceptionHandler( { DeleteNotAllowedException.class, HierarchyViolationException.class } )
- public ResponseEntity<String> handleForbidden( Exception ex ) throws IOException
- {
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.FORBIDDEN.toString(),
- ex.getMessage() ), HttpStatus.FORBIDDEN );
- }
-
- @ExceptionHandler( { ETagVerificationException.class, UuidFormatException.class } )
- public ResponseEntity<String> handlePreconditionFailed( Exception ex ) throws IOException
- {
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.PRECONDITION_FAILED.toString(),
- ex.getMessage() ), HttpStatus.PRECONDITION_FAILED );
- }
-
- @ExceptionHandler( { FacilityNotFoundException.class } )
- public ResponseEntity<String> handleNotFound( Exception ex ) throws IOException
- {
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.NOT_FOUND.toString(),
- ex.getMessage() ), HttpStatus.NOT_FOUND );
- }
-
- @ExceptionHandler( { DuplicateCodeException.class, DuplicateUidException.class, DuplicateUuidException.class } )
- public ResponseEntity<String> handleConflict( Exception ex ) throws IOException
- {
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.CONFLICT.toString(),
- ex.getMessage() ), HttpStatus.CONFLICT );
- }
-
- //--------------------------------------------------------------------------
// UTILS
//--------------------------------------------------------------------------
=== modified file 'dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityServiceController.java'
--- dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityServiceController.java 2013-03-09 09:38:51 +0000
+++ dhis-2/dhis-web/dhis-web-api-fred/src/main/java/org/hisp/dhis/web/webapi/v1/controller/FacilityServiceController.java 2013-03-09 15:32:50 +0000
@@ -33,7 +33,10 @@
import org.hisp.dhis.organisationunit.OrganisationUnit;
import org.hisp.dhis.organisationunit.OrganisationUnitService;
import org.hisp.dhis.web.webapi.v1.domain.Facility;
-import org.hisp.dhis.web.webapi.v1.utils.MessageUtils;
+import org.hisp.dhis.web.webapi.v1.exception.DuplicateCodeException;
+import org.hisp.dhis.web.webapi.v1.exception.DuplicateUidException;
+import org.hisp.dhis.web.webapi.v1.exception.DuplicateUuidException;
+import org.hisp.dhis.web.webapi.v1.exception.FacilityNotFoundException;
import org.hisp.dhis.web.webapi.v1.utils.ValidationUtils;
import org.hisp.dhis.web.webapi.v1.validation.group.Create;
import org.hisp.dhis.web.webapi.v1.validation.group.Update;
@@ -66,9 +69,9 @@
/**
* @author Morten Olav Hansen <mortenoh@xxxxxxxxx>
*/
-@Controller( value = "facility-service-controller-" + FredController.PREFIX )
-@RequestMapping( FacilityServiceController.RESOURCE_PATH )
-@PreAuthorize( "hasRole('M_dhis-web-api-fred') or hasRole('ALL')" )
+@Controller(value = "facility-service-controller-" + FredController.PREFIX)
+@RequestMapping(FacilityServiceController.RESOURCE_PATH)
+@PreAuthorize("hasRole('M_dhis-web-api-fred') or hasRole('ALL')")
public class FacilityServiceController
{
public static final String RESOURCE_PATH = "/" + FredController.PREFIX + "/facility-service";
@@ -83,49 +86,49 @@
private ConversionService conversionService;
@Autowired
- @Qualifier( "objectMapperFactoryBean" )
+ @Qualifier("objectMapperFactoryBean")
private ObjectMapper objectMapper;
//--------------------------------------------------------------------------
// EXTRA WEB METHODS
//--------------------------------------------------------------------------
- @RequestMapping( value = "/{id}/activate", method = RequestMethod.POST )
- @PreAuthorize( "hasRole('F_FRED_UPDATE') or hasRole('ALL')" )
- public ResponseEntity<Void> activateFacility( @PathVariable String id )
- {
- OrganisationUnit organisationUnit = organisationUnitService.getOrganisationUnit( id );
-
- if ( organisationUnit != null )
- {
- organisationUnit.setActive( true );
- organisationUnitService.updateOrganisationUnit( organisationUnit );
-
- return new ResponseEntity<Void>( HttpStatus.OK );
- }
-
- return new ResponseEntity<Void>( HttpStatus.NOT_FOUND );
- }
-
- @RequestMapping( value = "/{id}/deactivate", method = RequestMethod.POST )
- @PreAuthorize( "hasRole('F_FRED_UPDATE') or hasRole('ALL')" )
- public ResponseEntity<Void> deactivateFacility( @PathVariable String id )
- {
- OrganisationUnit organisationUnit = organisationUnitService.getOrganisationUnit( id );
-
- if ( organisationUnit != null )
- {
- organisationUnit.setActive( false );
- organisationUnitService.updateOrganisationUnit( organisationUnit );
-
- return new ResponseEntity<Void>( HttpStatus.OK );
- }
-
- return new ResponseEntity<Void>( HttpStatus.NOT_FOUND );
- }
-
- @RequestMapping( value = "/validate", method = RequestMethod.POST, consumes = MediaType.APPLICATION_JSON_VALUE )
- public ResponseEntity<String> validateFacilityForCreate( @RequestBody Facility facility ) throws IOException
+ @RequestMapping(value = "/{id}/activate", method = RequestMethod.POST)
+ @PreAuthorize("hasRole('F_FRED_UPDATE') or hasRole('ALL')")
+ public ResponseEntity<Void> activateFacility( @PathVariable String id ) throws FacilityNotFoundException
+ {
+ OrganisationUnit organisationUnit = getOrganisationUnit( id );
+
+ if ( organisationUnit == null )
+ {
+ throw new FacilityNotFoundException();
+ }
+
+ organisationUnit.setActive( true );
+ organisationUnitService.updateOrganisationUnit( organisationUnit );
+
+ return new ResponseEntity<Void>( HttpStatus.OK );
+ }
+
+ @RequestMapping(value = "/{id}/deactivate", method = RequestMethod.POST)
+ @PreAuthorize("hasRole('F_FRED_UPDATE') or hasRole('ALL')")
+ public ResponseEntity<Void> deactivateFacility( @PathVariable String id ) throws FacilityNotFoundException
+ {
+ OrganisationUnit organisationUnit = getOrganisationUnit( id );
+
+ if ( organisationUnit == null )
+ {
+ throw new FacilityNotFoundException();
+ }
+
+ organisationUnit.setActive( false );
+ organisationUnitService.updateOrganisationUnit( organisationUnit );
+
+ return new ResponseEntity<Void>( HttpStatus.OK );
+ }
+
+ @RequestMapping(value = "/validate", method = RequestMethod.POST, consumes = MediaType.APPLICATION_JSON_VALUE)
+ public ResponseEntity<String> validateFacilityForCreate( @RequestBody Facility facility ) throws Exception
{
Set<ConstraintViolation<Facility>> constraintViolations = validator.validate( facility, Default.class, Create.class );
@@ -138,20 +141,17 @@
{
OrganisationUnit organisationUnit = conversionService.convert( facility, OrganisationUnit.class );
+ if ( organisationUnitService.getOrganisationUnit( organisationUnit.getUuid() ) != null )
+ {
+ throw new DuplicateUuidException();
+ }
if ( organisationUnitService.getOrganisationUnit( organisationUnit.getUid() ) != null )
{
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.CONFLICT.toString(),
- "An object with that ID already exists." ), headers, HttpStatus.CONFLICT );
- }
- else if ( organisationUnitService.getOrganisationUnitByName( organisationUnit.getName() ) != null )
- {
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.CONFLICT.toString(),
- "An object with that name already exists." ), headers, HttpStatus.CONFLICT );
+ throw new DuplicateUidException();
}
else if ( organisationUnit.getCode() != null && organisationUnitService.getOrganisationUnitByCode( organisationUnit.getCode() ) != null )
{
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.CONFLICT.toString(),
- "An object with that code already exists." ), headers, HttpStatus.CONFLICT );
+ throw new DuplicateCodeException();
}
return new ResponseEntity<String>( json, headers, HttpStatus.OK );
@@ -162,8 +162,8 @@
}
}
- @RequestMapping( value = "/validate", method = RequestMethod.PUT, consumes = MediaType.APPLICATION_JSON_VALUE )
- public ResponseEntity<String> validateFacilityForUpdate( @RequestBody Facility facility ) throws IOException
+ @RequestMapping(value = "/validate", method = RequestMethod.PUT, consumes = MediaType.APPLICATION_JSON_VALUE)
+ public ResponseEntity<String> validateFacilityForUpdate( @RequestBody Facility facility ) throws Exception
{
Set<ConstraintViolation<Facility>> constraintViolations = validator.validate( facility, Default.class, Update.class );
@@ -179,8 +179,7 @@
if ( ou == null )
{
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.NOT_FOUND.toString(),
- "No object with that identifier exists." ), headers, HttpStatus.NOT_FOUND );
+ throw new FacilityNotFoundException();
}
else if ( organisationUnit.getCode() != null )
{
@@ -188,8 +187,7 @@
if ( ouByCode != null && !ou.getUid().equals( ouByCode.getUid() ) )
{
- return new ResponseEntity<String>( MessageUtils.jsonMessage( HttpStatus.CONFLICT.toString(),
- "Another object with the same code already exists." ), headers, HttpStatus.CONFLICT );
+ throw new DuplicateCodeException();
}
}
@@ -240,4 +238,25 @@
return new ResponseEntity<String>( json, headers, HttpStatus.OK );
}
+
+ //--------------------------------------------------------------------------
+ // UTILS
+ //--------------------------------------------------------------------------
+
+ private OrganisationUnit getOrganisationUnit( String id )
+ {
+ OrganisationUnit organisationUnit;
+
+ if ( id.length() == 11 )
+ {
+ organisationUnit = organisationUnitService.getOrganisationUnit( id );
+ }
+ else
+ {
+ organisationUnit = organisationUnitService.getOrganisationUnitByUuid( id );
+ }
+
+ return organisationUnit;
+ }
}
+