Opened 13 years ago

Closed 13 years ago

#3657 closed defect (fixed)

ogc sld doc: full request 9b (point symbolizer with default symbol) seems to be broken

Reported by: assefa Owned by: assefa
Priority: normal Milestone:
Component: Documentation - MapServer Version: unspecified
Severity: normal Keywords:
Cc: jmckenna

Description

http://www.mapserver.org/trunk/ogc/sld.html link full request 9b (point symbolizer with default symbol) seems to be broken.

Change History (11)

comment:1 by jmckenna, 13 years ago

Status: newassigned

comment:2 by assefa, 13 years ago

Let me know if it is the code which is broken or it is just the example.

comment:3 by assefa, 13 years ago

Looking into this locally, It also breaks for me. The reason being that there is no color parameter set. Adding something like this inside the Mark element would work:

<Stroke>
<CssParameter name="stroke">#ff0000</CssParameter>
</Stroke>

Looking into the code to see why it broke down, ...

comment:4 by jmckenna, 13 years ago

I didn't know you were also looking into this (I just spent time getting it running locally and testing.... that's why I marked the ticket to assigned ha).

ok backing off...it's all yours.

comment:5 by jmckenna, 13 years ago

Owner: changed from jmckenna to assefa
Status: assignednew

comment:6 by jmckenna, 13 years ago

Cc: jmckenna added

comment:7 by assefa, 13 years ago

sorry about that. that is really bad. I hate when this happens to me. What triggered it is the release e-mail for 5.6.6. I will respect protocols next time and indicate my activities in the bug. To my defense, a weak argument is 'assigned' does not mean to me "looking into it now"

I have added the fix in #3658

comment:8 by jmckenna, 13 years ago

Thanks man....in the end I couldn't fix that anyway so I'm happy that you dove in :)

comment:9 by assefa, 13 years ago

The example in the doc is still broken. It can be left until next upgrade. Or the xml file can be changed to add the stroke element (which would be an easy fix in term of time compared to upgrade I guess).

comment:10 by jmckenna, 13 years ago

ok I'll add this element in the doc now.

comment:11 by jmckenna, 13 years ago

Resolution: fixed
Status: newclosed

updated the SLD with that change. Thanks.

I notice another problem with a different request...keep your eyes open for new ticket shortly :)

-jeff

Note: See TracTickets for help on using tickets.