Heat And Climate

Heat And Climate

409k Downloads

Green Gold Pendant適用時に単体の原木を破壊できない

SkyTheory opened this issue · 4 comments

commented

ItemColorPendant2のGreenを所持中に、単体の(周囲に同種のブロックが存在しない)原木を破壊できません。
原因はMainUtil.getLumberTargetListはその状況では空のLinkedHashSetを返すからだと思われます。
差し出がましい真似かもしれませんが、これが意図した仕様ではないならばItemColorPendant2の189行目の
return true; を return !set.isEmpty();
に変更で解消すると思います。(PRのやり方が良く分からないので、issue内での提案となります……)

commented

0538ab3 でset.isEmpty()時の挙動を修正しました。
たぶんこれで大丈夫だと思います。

0538ab3#diff-791e21c0d3ace0d5576ca648416cc0a9d3cb946207b3e6b52e30ebcf742bdcb3R196

commented

ありがとうございます! 燃焼チャンバーやシンクの新モデル等々、次回更新を楽しみに待ってますー!

commented

ご報告ありがとうございます。
次回修正で更新します。

>ItemColorPendant2の189行目のreturn true; を return !set.isEmpty();に変更

この方法では別の不具合が発生します(!owner.world.isRemoteの判定を挟むため、この方法だと鯖蔵で結果が食い違い、ブロック更新に不具合が出います)ので、EMPTY時の条件分岐を用意すると思います。

commented

対応ありがとうございます。
ですが、今度はisEmptyの際の処理にworld.setBlockToAirが入っていないため、アイテムのみドロップして原木がワールドに残ったままになってしまっています……。