不要コードの削除 簡易版盗賊の鼻の整理#1
Open
elleonard wants to merge 1 commit intoSigureya:masterfrom
Open
Conversation
elleonard
commented
Oct 14, 2020
| * イベントコマンドを書くのが面倒という人向け | ||
| * @type boolean | ||
| * @default false | ||
| * |
Author
There was a problem hiding this comment.
このパラメータはワンボタンで起動することを想定した簡易版盗賊の鼻のために書かれたもののようです。
ワンボタンで起動する盗賊の鼻は長らくWIPの状態で放置されている上、以前お話した際にワンボタンコモンで実現可能であるため積極的に実装するつもりはないと仰っていたので、実際に使われているメッセージ周りの設定以外を削除しています。
elleonard
commented
Oct 14, 2020
| * @desc キーボードの割り当てです | ||
| * キーコードは各自調べてください | ||
| * @type number | ||
| * @command showChestCountMessage |
Author
There was a problem hiding this comment.
MV用プラグインコマンドの CountChest と同じ処理を行うMZ用コマンドです。
elleonard
commented
Oct 14, 2020
| } | ||
| return { | ||
| countChestEmptyText:String(params.CountChestEmptyText), | ||
| inputDefine :createCommonDefine(params.CountChestinputSetting), |
Author
There was a problem hiding this comment.
CountChestinputSetting はパラメータには存在せず、簡易版盗賊の鼻設定をONにするとプラグインごと動かなくなっていました。
ワンボタン起動用のものと判断し、削除しています。
Owner
|
2年前のプルリクエストですが、おいおい対応します。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
不要と思しきデッドコードを消したり、簡易版盗賊の鼻として定義されていた仕様を整理してはいかがでしょう、という提案です。
古いコードのようなので、リファクタの余地は大いにありそうでしたが、まずは不要なものを削除するところから、という判断のもと、既存のロジックには手を入れていません。