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

IDとシステム名の修正 #222

Merged
merged 3 commits into from
Jun 28, 2020
Merged

IDとシステム名の修正 #222

merged 3 commits into from
Jun 28, 2020

Conversation

ysakasin
Copy link
Member

変更されたIDは以下の通り

変更前 変更後
Alter_raise AlterRaise
Chaos Flare ChaosFlare
Elric! Elric
MetallicGuadian MetallicGuardian

Ref #190

@ysakasin ysakasin requested a review from ochaochaocha3 June 26, 2020 09:45
@ysakasin ysakasin added the bug バグってる! label Jun 26, 2020
@ochaochaocha3
Copy link
Member

変更内容を見ると、「Tunnels & Trolls」→「TunnelsAndTrolls」のID変更も行われているのでしょうか?

@ochaochaocha3
Copy link
Member

configBcDice.rbの $allGameTypes については、IRCボットのGUIアプリしか使っていなかった(どどんとふでも使っていない)ため、変更しないか、すべて削除しても問題ないようです。

@ochaochaocha3
Copy link
Member

以下で変更点をカバーできているでしょうか?(diffから作りました)

IDの変更

変更前 変更後
Alter_raise AlterRaise
Chaos Flare ChaosFlare
Elric! Elric
MetallicGuadian MetallicGuardian
Tunnels & Trolls TunnelsAndTrolls

ゲームシステム名・ソートキーの変更

太字 の箇所が変更点。

ゲームシステム名変更前 ゲームシステム名変更後 ソートキー変更前 ソートキー変更後
心衝想機TRPGアルトレイズ アルトレイズ あるとれいす (変更なし)
アリアンロッド アリアンロッドRPG ありあんろつと ありあんろつとRPG
在りて遍くオルガレイン (変更なし) おるかれいん ありてあまねくおるかれいん
犯罪活劇RPGバッドライフ バッドライフ はつとらいふ (変更なし)
ブラインド・ミトス ブラインド・ミトスRPG ふらいんとみとす ふらいんとみとすRPG
Chill 3 Chill 3rd Edition ちる3 (変更なし)
クトゥルフ クトゥルフ神話TRPG くとうるふ くとうるふしんわTRPG
新クトゥルフ 新クトゥルフ神話TRPG しんくとうるふ しんくとうるふしんわTRPG
晃天のイルージオ (変更なし) いるうしお こうてんのいるうしお
壊れた世界のポストマン (変更なし) ほすとまん こわれたせかいのほすとまん
デッドラインヒーローズ デッドラインヒーローズRPG てつとらいんひいろおす (変更なし)
エムブリオマシン エムブリオマシンRPG えむふりおましん えむふりおましんRPG
エンドブレイカー エンドブレイカー! えんとふれいかあ (変更なし)
フルメタル・パニック! フルメタル・パニック!RPG ふるめたるはにつく ふるめたるはにつくRPG
グランクレスト グランクレストRPG くらんくれすと くらんくれすとRPG
ガンドッグ・ゼロ ガンドッグゼロ かんとつくせろ (変更なし)
紫縞のリヴラドール (変更なし) りうらとおる ししまのりうらとおる
ハンターズムーン ハンターズ・ムーン はんたあすむうん (変更なし)
ログ・ホライズン ログ・ホライズンTRPG ろくほらいすん ろくほらいすんTRPG
メタリックガーディアン メタリックガーディアンRPG めたりつくかあていあん めたりつくかあていあんRPG
モノトーン・ミュージアム モノトーンミュージアムRPG ものとおんみゆうしあむ ものとおんみゆうしあむRPG
ナイトウィザード2版 ナイトウィザード The 2nd Edition ないとういさあと2 (変更なし)
ナイトウィザード3版 ナイトウィザード The 3rd Edition ないとういさあと3 (変更なし)
片道勇者 片道勇者TRPG かたみちゆうしや かたみちゆうしやTRPG
央華封神RPG第三版 央華封神RPG 第三版 おうかほうしん3 おうかほうしんRPG3
パラサイトブラッド パラサイトブラッドRPG はらさいとふらつと はらさいとふらつとRPG
ファンタズムアドベンチャー ファンタズム・アドベンチャー ふあんたすむあとへんちやあ (変更なし)
六門世界2nd 六門世界RPG セカンドエディション ろくもんせかい2 ろくもんせかいRPG2
Standard RPG System スタンダードRPGシステム すたんたあとRPGしすてむ (変更なし)
シャドウラン第4版 シャドウラン 4th Edition しやとうらん4 (変更なし)
シャドウラン第5版 シャドウラン 5th Edition しやとうらん5 (変更なし)
真・女神転生TRPG 覚醒篇 真・女神転生TRPG 覚醒篇 しんめかみてんせいTRPGかくせいへん (変更なし)
少女展爛会 少女展爛会TRPG しようしよてんらんかい しようしよてんらんかいTRPG
ステラーライフ ステラーライフTRPG すてらあらいふ すてらあらいふTRPG
ソードワールド ソードワールドRPG そおとわあると (変更なし)
青春疾患セラフィザイン セラフィザイン せいしゆんしつかんせらふいさいん せらふいさいん
トーキョーN◎VA トーキョーN◎VA とおきよおのは (変更なし)
トワイライト・ガンスモーク トワイライトガンスモーク とわいらいとかんすもおく (変更なし)
ワールドオブダークネス ワールド・オブ・ダークネス わあるとおふたあくねす (変更なし)

@ysakasin
Copy link
Member Author

変更内容を見ると、「Tunnels & Trolls」→「TunnelsAndTrolls」のID変更も行われているのでしょうか?

そうです。すっかり抜けてました、すみません。

システム名のまとめは億劫でしてませんでした。ありがとうございます 🙇
以前作成した作業用スプレッドシートの内容を適用しています。

@ysakasin
Copy link
Member Author

configBcDice.rbの $allGameTypes については、IRCボットのGUIアプリしか使っていなかった(どどんとふでも使っていない)ため、変更しないか、すべて削除しても問題ないようです。

Ver 2の命も短いので、残しておこうと思います。害のあるものでもないので。

@ochaochaocha3
Copy link
Member

configBcDice.rb については、厳密に変えるとなると変更漏れの「TunnelsAndTrolls」の分もさらに変えないといけないので、それならば一気に消すのでもいいかな、という感じでした。いかがでしょうか?

@ysakasin
Copy link
Member Author

なるほど、そうですね。

さらっと確認しましたが、 src/bcdiceGui.rbsrc/createExe.rb$allGameTypes を使っているので、それらのコードが丸ごと消せるなら、$allGameTypes も消して良いかと思います。GUI部分はbcdice-ircに分離したし、上記消せますかね?

消すなら別PR出します。

@ochaochaocha3
Copy link
Member

さらっと確認しましたが、 src/bcdiceGui.rbsrc/createExe.rb$allGameTypes を使っているので、それらのコードが丸ごと消せるなら、$allGameTypes も消して良いかと思います。GUI部分はbcdice-ircに分離したし、上記消せますかね?

はい、もう消して問題ないと思います。仰るとおり別PRで行うのが良さそうですね。

@ysakasin ysakasin merged commit dbf1a17 into master Jun 28, 2020
@ysakasin ysakasin deleted the rename_id branch June 30, 2020 14:41
ysakasin added a commit that referenced this pull request Jul 11, 2020
ysakasin added a commit that referenced this pull request Aug 30, 2020
IDとシステム名の修正
ysakasin added a commit that referenced this pull request Aug 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグってる!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants