Skip to content

Commit 6339788

Browse files
authored
Merge pull request #575 from terrestris/enable-style
V5: re-enable style serialization for GeoJSON layers (MFv3)
2 parents d89faa4 + 32fac14 commit 6339788

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

.gitignore

+2
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,5 @@ dist
55
*.log
66
npm-debug.log*
77
package-lock.json
8+
coverage
9+
*.iml

.nvmrc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
18

src/serializer/MapFishPrintV3GeoJsonSerializer.js

+3-7
Original file line numberDiff line numberDiff line change
@@ -142,12 +142,12 @@ export class MapFishPrintV3GeoJsonSerializer extends BaseSerializer {
142142
if (!serializedFeature.properties) {
143143
serializedFeature.properties = {};
144144
}
145-
// serializedFeature.properties[this.constructor.FEAT_STYLE_PROPERTY] = styleName;
145+
serializedFeature.properties[this.constructor.FEAT_STYLE_PROPERTY] = styleName;
146146
});
147147
}
148148
});
149149

150-
const serialized = {
150+
return {
151151
...super.serialize(layer, opts, viewResolution),
152152
...{
153153
geoJson: {
@@ -156,15 +156,11 @@ export class MapFishPrintV3GeoJsonSerializer extends BaseSerializer {
156156
},
157157
name: layer.get('name') || 'Vector Layer',
158158
opacity: layer.getOpacity(),
159-
// TODO Currently not supported, GeoStyler MapFish JSON StyleParser should
160-
// be used here!
161-
style: {},
159+
style: serializedStyles,
162160
type: this.constructor.TYPE_GEOJSON
163161
},
164162
...opts
165163
};
166-
167-
return serialized;
168164
}
169165

170166
/**

0 commit comments

Comments
 (0)