Ticket #3657 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

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

Changed 2 years ago by jmckenna

  • status changed from new to assigned

Changed 2 years ago by assefa

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

Changed 2 years ago by assefa

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, ...

Changed 2 years ago by jmckenna

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.

Changed 2 years ago by jmckenna

  • owner changed from jmckenna to assefa
  • status changed from assigned to new

Changed 2 years ago by jmckenna

  • cc jmckenna added

Changed 2 years ago by assefa

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

Changed 2 years ago by jmckenna

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

Changed 2 years ago by assefa

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).

Changed 2 years ago by jmckenna

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

Changed 2 years ago by jmckenna

  • status changed from new to closed
  • resolution set to fixed

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.