GeoServer

CSS SLD generation for external graphics (url) fails

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.1-RC5
  • Fix Version/s: 2.1.0
  • Component/s: CSS
  • Labels:
    None
  • Number of attachments :
    0

Description

I am trying to use an external graphic to fill a polygon but I get the error

java.lang.Error: SOMEONE CODED THE X LINK NAMESPACE WRONG

Using the following style?

  • { fill: url(grass_fill.png); fill-mime: 'image/png'; stroke: black; }

The image grass_fill.png is in the styles directory and works fine in the grass_poly.sld that ships with geoserver. I have tried quoting it etc.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
David Winslow added a comment - - edited

Just so you know, (I'm not aware of a workaround or anything) the 

 X LINK NAMESPACE

error is what GeoTools reports when someone tries to use a style that has graphics without the mimetype set. It's not a quoting issue.

Show
David Winslow added a comment - - edited Just so you know, (I'm not aware of a workaround or anything) the X LINK NAMESPACE error is what GeoTools reports when someone tries to use a style that has graphics without the mimetype set. It's not a quoting issue.
Hide
Permalink
David Winslow added a comment -

The actual bug is that relative URLs are not currently supported. According to the comments in the test suite, I knew about this but have been putting it off so long I apparently forgot! Guess I'll go ahead and fix it while I'm at it.

Show
David Winslow added a comment - The actual bug is that relative URLs are not currently supported. According to the comments in the test suite, I knew about this but have been putting it off so long I apparently forgot! Guess I'll go ahead and fix it while I'm at it.
Hide
Permalink
David Winslow added a comment -

I just committed a fix for this to trunk. Since 2.1.0 is about to come out I will hold off on committing there but I have a patch ready.

Show
David Winslow added a comment - I just committed a fix for this to trunk. Since 2.1.0 is about to come out I will hold off on committing there but I have a patch ready.
Hide
Permalink
David Winslow added a comment -

ok i have committed the update on the 2.1.x branch.

Show
David Winslow added a comment - ok i have committed the update on the 2.1.x branch.
Hide
Permalink
Andrea Aime added a comment -

Mass transition all resolved issue that did not see any further comment in the last month to closed status

Show
Andrea Aime added a comment - Mass transition all resolved issue that did not see any further comment in the last month to closed status

People

  • Assignee:
    David Winslow
    Reporter:
    Charles Galpin
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: