CoverageStore creation failures are not handled
-----------------------------------------------
Key: GEOS-2855
URL: http://jira.codehaus.org/browse/GEOS-2855
Project: GeoServer
Issue Type: Bug
Components: Wicket UI
Affects Versions: 2.0-alpha1
Reporter: Gabriel Roldán
Assignee: Gabriel Roldán
Fix For: 2.0-beta1
Creating a new coverage store with wrong parameter names produces a failure and
an html page with a stack trace. Going back to add the coverage store with the
correct parameters also fail because the coverage store was already added to
the catalog with a stack trace like the following:
{code}
java.lang.IllegalArgumentException: Store '<store name>' already exists in
workspace '<ws name>'
at org.geoserver.catalog.impl.CatalogImpl.validate(CatalogImpl.java:150)
at org.geoserver.catalog.impl.CatalogImpl.save(CatalogImpl.java:164)
at
org.geoserver.security.SecureCatalogImpl.save(SecureCatalogImpl.java:950)
at
org.geoserver.web.data.coverage.CoverageStoreConfiguration$1.onSubmit(CoverageStoreConfiguration.java:135)
at
org.apache.wicket.markup.html.form.Form.delegateSubmit(Form.java:1351)
at
org.apache.wicket.markup.html.form.Form.onFormSubmitted(Form.java:810)
{code}
If creating the store fails the UI should:
- present a failure message in the same "add coverage store" page
- do not add the store to the catalog at all until the creation of the store
succeeds
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
------------------------------------------------------------------------------
This SF.net email is sponsored by:
High Quality Requirements in a Collaborative Environment.
Download a free trial of Rational Requirements Composer Now!
http://p.sf.net/sfu/www-ibm-com
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
