BehaviorPackLoader組み込み

DaisukeDaisuke commented 3 years ago

不要と感じた場合、このpull requestを無言にて閉じていただいて問題ありません。

以下のリポジトリより手動マージ... https://github.com/DaisukeDaisuke/BehaviorPackLoader/tree/for4.0.0_handler

テスト: 現在、諸事情により実験用データベースを用意出来ない為、未実施

メリット

(なし) サーバー起動時間短縮

デメリット

pmmpの仕様変更の毎に発生するPRレビューの手間など

未解決問題

id逆順問題(調査予定...) ログイン時アイテム透明化問題(CustomItemLoaderでは再現できない為、BehaviorPackの問題と推測...) ↓対策コード... https://github.com/DaisukeDaisukeTeam/RunForMoney2/blob/2fc0201849ce184d09a6789d125fd4288f98f7b5/src/kametan0730mc/RunForMoney2/EventListener.php#L46-L48

注意点

BehaviorPack設置フォルダに関しましては以下のディレクトリに変更致します...(申し訳ないです...)

[PocketMine-MP]/plugin_data/BehaviorPackLoader/
↓
[PocketMine-MP]/behavior_packs/

kametan0730 commented 3 years ago

PullRequestありがとうございます。id逆順問題とはサーバー側とクライアント側でアイテムが別のアイテムと認識される問題を引き起こすものでしょうか。現在私のサーバーではその問題に対処するために、Itemの登録はBPLではなくresources/vanillaに直接書き込んでおります。また、BPLは別のプラグインとしてあった方がわかりやすい気もします。

DaisukeDaisuke commented 3 years ago

忙しい中、レビュー、ご意見ありがとうございます。

分かりにくく、申し訳ございません。その認識にて問題ございません。

また、私の調査不足により、ご不便/ご迷惑をお掛けして本当に申し訳ございません。

id逆順問題について、今後も長期的に調査したい思います... 今後もBehaviorPackLoader/派生プラグインを宜しくお願い致します。

kametan0730 commented 3 years ago

ありがとうございます、私もその問題については詳しく調べようと思います。今後ともよろしくお願いします。

KASOSP / RunForMoney2