Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch: prevent strange remaining count #1

Merged
merged 1 commit into from
Dec 9, 2019
Merged

Patch: prevent strange remaining count #1

merged 1 commit into from
Dec 9, 2019

Conversation

katabame
Copy link
Contributor

@katabame katabame commented Dec 9, 2019

人数制限解除や規定人数等、ロール構成が自由な状態でMatchするとロール毎の人数情報が正しく取得できずおかしな数値を返してしまいます。

本来はプラグイン側のコードを変更するべきでしょうが、ひとまずオーバーレイ側で24人以上の値を受け取った場合はロール構成自由と判断し、残り人数を不明表示するように変更しています。

@qitana qitana changed the base branch from master to develop December 9, 2019 02:03
@qitana qitana changed the base branch from develop to master December 9, 2019 02:05
@qitana qitana merged commit 9124474 into qitana:master Dec 9, 2019
@qitana
Copy link
Owner

qitana commented Dec 9, 2019

ありがとうございます。現象は認識しております。
プラグインの趣旨として、主にルーレットでマッチ先を知るために使用する想定でいたため、
制限解除等では異常表示となっても特に支障が無いと判断しておりました。

コード側での対応ですが、確実に対応するためには、ロール構成が自由な場合なのかを区別するべきですが、送られてくるネットワークデータの識別コードにはそれらの違いがないため、
中のデータ構造を比較・確認する必要があり、根本対応には多少の時間がかかるかもしれません。

qitana added a commit that referenced this pull request Jan 16, 2024
Patch: prevent strange remaining count
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.

None yet

2 participants