GeoTools

Avoid dumping the defaults when turning a style into a SLD 1.0 document

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.6.3
  • Fix Version/s: 2.6.4
  • Component/s: main
  • Labels:
    None

Issue Links

is depended upon by

Improvement - An improvement or enhancement to an existing feature or task. UDIG-1175 SLD style export should avoid expressing default values

  • Major - Major loss of function.
  • 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
Andrea Aime added a comment -
This patch makes the transformer dump only the non default parts of a style/symbolizer/whatever.
Makes for much more compact SLD exports, useful for both uDig and the GeoServer CSS module.
Show
Andrea Aime added a comment - This patch makes the transformer dump only the non default parts of a style/symbolizer/whatever. Makes for much more compact SLD exports, useful for both uDig and the GeoServer CSS module.
Hide
Permalink
Andrea Aime added a comment -
Jody, want to review it before I commit?
Show
Andrea Aime added a comment - Jody, want to review it before I commit?
Hide
Permalink
Jody Garnett added a comment -
Looks good; nicely done with the default value.

We have the default values for SLD in a number of spots now and I can never think of a good way out; having it explicitly in the code is nice and clear about what is going on.
Show
Jody Garnett added a comment - Looks good; nicely done with the default value. We have the default values for SLD in a number of spots now and I can never think of a good way out; having it explicitly in the code is nice and clear about what is going on.
Hide
Permalink
Andrea Aime added a comment -
Fixed on 2.6.x and trunk
Show
Andrea Aime added a comment - Fixed on 2.6.x and trunk

People

  • Assignee:
    Andrea Aime
    Reporter:
    Andrea Aime
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: