【fix】修复不同图层设置掩膜，掩膜有相交时错乱的问题 review by xiongjj · SuperMap/iClient-JavaScript@bd766e9 · GitHub

SuperMap / iClient-JavaScript Public

• Notifications You must be signed in to change notification settings
• Fork 288
• Star 906

• Code
• Issues 5
• Pull requests 14
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit bd766e9

Browse filesBrowse the repository at this point in the historyBrowse files

songyumeng

committed

【fix】修复不同图层设置掩膜，掩膜有相交时错乱的问题 review by xiongjj

1 parent 9d193da commit bd766e9
Copy full SHA for bd766e9

2 files changed

+2-2Lines changed: 2 additions & 2 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• src/openlayers/core
  • Util.js
• test/openlayers/core
  • UtilSpec.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎src/openlayers/core/Util.js‎

Copy file name to clipboardExpand all lines: src/openlayers/core/Util.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -515,7 +515,7 @@ export class Util {
515	515	Util.unsetMask(todoLayers);
516	516	todoLayers.forEach((layer) => {
517	517	layer.classNameBak_ = layer.className_;
518		- layer.className_ = 'ol_mask_layer';
	518	+ layer.className_ = `ol_mask_layer_${layer.ol_uid}`;
519	519	layer.clipRender = clipRender;
520	520	layer.extentBak_ = layer.getExtent();
521	521	layer.setExtent(clipLayer.getSource().getExtent());

Collapse file

‎test/openlayers/core/UtilSpec.js‎

Copy file name to clipboardExpand all lines: test/openlayers/core/UtilSpec.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -252,7 +252,7 @@ describe('openlayers_Util', () => {
252	252	]);
253	253	Util.setMask(layer, polygon);
254	254	expect(layer.classNameBak_).toBe('ol-layer');
255		- expect(layer.className_).toBe('ol_mask_layer');
	255	+ expect(layer.className_).toContain('ol_mask_layer_');
256	256	expect(layer.clipRender).not.toBeNull;
257	257	expect(layer.setExtent).toHaveBeenCalled;
258	258	expect(layer.changed).toHaveBeenCalled;

0 commit comments

Comments

0 (0)