Opened 16 years ago

Closed 15 years ago

#2770 closed defect (fixed)

POSITION AUTO for Polygons should include CC (and start with it)

Reported by: sdlime Owned by: sdlime
Priority: normal Milestone: 5.4 release
Component: MapServer C Library Version: unspecified
Severity: normal Keywords:
Cc: jmckenna

Description

Actually we need to decouple that code (msDrawLabelCache) in mapdraw.c from the positions enum. It should be easy, just create a positions int array with a length equal to the number of positions and fill it explicitly with the necessary positions, and save the number loaded. Then we loop through that instead of the start, stop crap currently used.

Steve

Change History (8)

comment:1 by sdlime, 16 years ago

Component: MapServer C LibraryMapServer Documentation
Status: newassigned

Fixed in r7920. Polygons us CC, UC, LC, CL and CR as possible positions. Point/annotation and line layers remain "as is". Assigning to the documentation component.

Steve

comment:2 by jmckenna, 15 years ago

Owner: changed from sdlime to jmckenna
Status: assignednew

comment:3 by jmckenna, 15 years ago

Status: newassigned

comment:4 by jmckenna, 15 years ago

Resolution: fixed
Status: assignedclosed
  • updated the mapfile reference's LABEL object for the POSITION parameter

comment:5 by sdlime, 15 years ago

Component: MapServer DocumentationMapServer C Library
Resolution: fixed
Status: closedreopened

There's still an issue if the polygon geometry is part of an annotation layer. The geometry type is not available to the label cache code so we don't know if how to set the positions correctly with annotation layers. Currently it treats 'em like points with the 8 positions around the point. We'll need to track the geometry type as part of msAddLabel() to fix this.

Steve

comment:6 by sdlime, 15 years ago

Cc: jmckenna added
Owner: changed from jmckenna to sdlime
Status: reopenednew

Jeff: No need to update docs with this, I plan to fix for beta2.

Steve

comment:7 by sdlime, 15 years ago

Status: newassigned

comment:8 by sdlime, 15 years ago

Resolution: fixed
Status: assignedclosed

Fixed in r8608, closing...

Steve

Note: See TracTickets for help on using tickets.