Skip to content

Master fixbug#140

Merged
ParnDeedlit merged 2 commits intomasterfrom
master_fixbug
Mar 24, 2022
Merged

Master fixbug#140
ParnDeedlit merged 2 commits intomasterfrom
master_fixbug

Conversation

@gyj12
Copy link
Copy Markdown
Member

@gyj12 gyj12 commented Mar 18, 2022

1.组图层支持透明度设置;
2.leaflet的Zondy.Map.MapWMTSLayer接口的增加tokenKey字段

@gyj12 gyj12 requested a review from ParnDeedlit March 18, 2022 03:33
Copy link
Copy Markdown
Member

@ParnDeedlit ParnDeedlit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

增加图层样式参数

Comment on lines +158 to +166

/**
* @private
* @member Zondy.Object.CDisplayStyle.prototype.LayerStyles
* @type {Map}
* @description 图层显示参数Map<{@link Zondy.Object.DynShowStyle}>
* @default null
*/
this.LayerStyles = options.LayerStyles !== undefined ? options.LayerStyles : null;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

增加layerStyle参数

Copy link
Copy Markdown
Member

@ParnDeedlit ParnDeedlit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

增加token规则

Comment on lines +141 to +151
if (this.options.token) {
if (this.options.tokenKey) {
obj[this.options.tokenKey] = this.options.token;
} else {
if (url.indexOf('tianditu') > -1) {
obj.tk = this.options.token;
} else {
obj.token = this.options.token;
}
}
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个地方是拼接url的时候使用,如果能使用一种规则来确定 { tokenKey:'tk' , tokenValue: 'xxxxxx'} 就好了

@ParnDeedlit ParnDeedlit merged commit 3478419 into master Mar 24, 2022
@ParnDeedlit ParnDeedlit deleted the master_fixbug branch March 24, 2022 02:19
mayuanye2008 pushed a commit that referenced this pull request May 17, 2023
【SDK】【Leaflet】【leaflet的Zondy.Map.MapWMTSLayer接口的增加tokenKey字段】
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants