OpenLayers OpenLayers

Ticket #1033 (closed task: fixed)

Opened 1 year ago

Last modified 1 year ago

Create Format.Text, and have Layer.Text use it

Reported by: crschmidt Assigned to: tschaub
Priority: minor Milestone: 2.6 Release
Component: Format Version: 2.5 RC3
Keywords: Cc:
State: Complete

Description

Currently, there is a Layer.Text. This should use the Format class, so that other layers -- especially vector layers -- can use it, and the Layer.Text should then take the Feature.Vectors it gets back and turn them into regular features.

Attachments

text_format.patch (12.4 kB) - added by crschmidt on 09/30/07 11:56:23.
Create Format.Text, and have Layer.Text use it.
format.text.patch (12.8 kB) - added by crschmidt on 12/14/07 16:48:29.
new patch

Change History

09/30/07 11:56:23 changed by crschmidt

  • attachment text_format.patch added.

Create Format.Text, and have Layer.Text use it.

09/30/07 11:57:44 changed by crschmidt

  • keywords set to review.

Applying this patch and then running tests still works. This determines that the tested properties of the Layer.Text are still set to the same values, which includes title, description, images and image sizes.

Please review.

09/30/07 13:52:17 changed by crschmidt

Before committing, I will test against:

http://dev.openstreetmap.org/~kleptog/pois/pois.php/WFS/12/1052/1372.txt

12/14/07 16:48:29 changed by crschmidt

  • attachment format.text.patch added.

new patch

12/14/07 16:58:09 changed by tschaub

  • keywords deleted.
  • state set to Commit.

I confirmed that tests pass in FF. Without saying I like this as is, I do agree with the reasons for the change. Please commit.

12/14/07 17:02:01 changed by crschmidt

  • status changed from new to closed.
  • state changed from Commit to Complete.
  • resolution set to fixed.

(In [5412]) Create Format.Text, and have Layer.Text use it. (Closes #1033)