Skip to content
This repository has been archived by the owner on Sep 25, 2023. It is now read-only.

dictionary bug fix #1180

Closed
wants to merge 1 commit into from
Closed

dictionary bug fix #1180

wants to merge 1 commit into from

Conversation

zengjiwen
Copy link

No description provided.

@whtiehack
Copy link
Contributor

为什么说这里有bug?

@zengjiwen
Copy link
Author

为什么说这里有bug?

key应该是server type,这里是server id

@whtiehack
Copy link
Contributor

感觉不是bug吧
lib/util/appUtil

pomelo/lib/util/appUtil.js

Lines 130 to 134 in e1615c1

var loadServers = function(app) {
app.loadConfigBaseApp(Constants.RESERVED.SERVERS, Constants.FILEPATH.SERVER);
var servers = app.get(Constants.RESERVED.SERVERS);
var serverMap = {}, slist, i, l, server;
for (var serverType in servers) {

SERVERS: 'servers',

@zengjiwen
Copy link
Author

确实,是我搞错了,sorry

@zengjiwen zengjiwen closed this Sep 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants