Mercurial > dive4elements > river
diff flys-client/src/main/java/de/intevation/flys/client/server/AddArtifactServiceImpl.java @ 215:e02f50a3ad59
Improved the exception handling of artifact / collection specific operations. Warning dialog are displayed after such an exception.
flys-client/trunk@1657 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author | Ingo Weinzierl <ingo.weinzierl@intevation.de> |
---|---|
date | Fri, 08 Apr 2011 08:51:28 +0000 |
parents | f7967d12ce6e |
children | 924da6695800 |
line wrap: on
line diff
--- a/flys-client/src/main/java/de/intevation/flys/client/server/AddArtifactServiceImpl.java Fri Apr 08 08:45:15 2011 +0000 +++ b/flys-client/src/main/java/de/intevation/flys/client/server/AddArtifactServiceImpl.java Fri Apr 08 08:51:28 2011 +0000 @@ -9,6 +9,7 @@ import de.intevation.artifacts.httpclient.http.HttpClientImpl; import de.intevation.artifacts.httpclient.http.response.DocumentResponseHandler; +import de.intevation.flys.client.shared.exceptions.ServerException; import de.intevation.flys.client.shared.model.Artifact; import de.intevation.flys.client.shared.model.Collection; import de.intevation.flys.client.client.services.AddArtifactService; @@ -21,7 +22,11 @@ extends DescribeCollectionServiceImpl implements AddArtifactService { - public Collection add(Collection collection, Artifact artifact, String url) { + public static final String ERROR_ADD_ARTIFACT = "error_add_artifact"; + + public Collection add(Collection collection, Artifact artifact, String url) + throws ServerException + { System.out.println("AddArtifactServiceImpl.add"); Document add = ClientProtocolUtils.newAddArtifactDocument( @@ -36,16 +41,17 @@ Collection c = parseCollection(response); if (c == null) { - throw new NullPointerException("No collection returned."); + throw new ServerException(ERROR_ADD_ARTIFACT); } return c; } catch (ConnectionException ce) { System.err.println(ce.getLocalizedMessage()); + } - return null; + throw new ServerException(ERROR_ADD_ARTIFACT); } } // vim:set ts=4 sw=4 si et sta sts=4 fenc=utf8 :