GeoServer

A style gets removed after updating it with an invalid .sld file.

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.2.3
  • Fix Version/s: 1.3-rc1
  • Component/s: None
  • Labels:
    None
  • Environment:
    Mandrake 10.1 with KDE 3.2, Geoserver 1.2.3(Binary), Java 1.4.2. PostgreSQL 7.4.5 with PostGIS 0.9.
  • Number of attachments :
    0

Description

Repro Steps.
1. Add a new style using the web interface and upload a valid .sld file.
2. Edit the .sld file to make it invalid (for example, taking out a / from a closing tag is how I got this to fail)
3. Update the newly created style with the invalid .sld file.
4. Restart Geoserver.

RESULTS.
The style is missing from the drop down on the "Styles Configuration" page. The style is also removed from the catalog.xml file.

Bug found on Version 1.2.3, I haven't tried it on other versions.

Issue Links

is related to

Improvement - An improvement or enhancement to an existing feature or task. GEOS-294 Validation for .sld file needed.

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Chris Holmes added a comment -

Related issue, should be solved at the same time.

Show
Chris Holmes added a comment - Related issue, should be solved at the same time.
Hide
Permalink
Chris Holmes added a comment -

This one's all you David.

Show
Chris Holmes added a comment - This one's all you David.
Hide
Permalink
Chris Holmes added a comment -

Fixed this up, we've got validation, and you also now can not send a file that is already there, over riding the other. See the comment on 294 for more info.

Show
Chris Holmes added a comment - Fixed this up, we've got validation, and you also now can not send a file that is already there, over riding the other. See the comment on 294 for more info.

People

  • Assignee:
    Chris Holmes
    Reporter:
    Paul Duff
Vote (0)
Watch (0)

Dates

  • Due:
    Created:
    Updated:
    Resolved: