Skip to content

EZStorage 2のDollyの挙動を修正#104

Merged
tier940 merged 5 commits intodevfrom
patch-ez-storage2
Jun 25, 2023
Merged

EZStorage 2のDollyの挙動を修正#104
tier940 merged 5 commits intodevfrom
patch-ez-storage2

Conversation

@sjcl
Copy link
Copy Markdown
Member

@sjcl sjcl commented Jun 25, 2023

  • 2つ以上スタックされているDollyにブロックを格納した際に、スタック全体にブロックを格納してしまう問題を修正
    • クリエイティブモードでは元の挙動を維持
  • ブロックが格納されているDollyはスタック出来ないように変更
  • 何らかの手段でブロックが格納されているDollyをスタックしても、設置時に1つだけ減るように修正
    • クリエイティブモードでは元の挙動を維持
  • 通常設置できない場所にChestを設置できてしまう問題を修正
  • 1ブロック未満の大きさのブロックの下をターゲットにDollyからブロックを展開しようとした場合、上書きできるべきでは無いターゲットの上のブロックを上書きして消滅してしまう問題を修正
  • Storage Coreの収納が不安定な問題を修正

@sjcl
Copy link
Copy Markdown
Member Author

sjcl commented Jun 25, 2023

oof

IDEAのフォーマッタの設定ファイルありますか...?

@tier940
Copy link
Copy Markdown
Member

tier940 commented Jun 25, 2023

oof

IDEAのフォーマッタの設定ファイルありますか...?

Gradleの6番(Apply Spotless)回せばOKですん。

@sjcl sjcl requested a review from tier940 June 25, 2023 12:06
@sjcl
Copy link
Copy Markdown
Member Author

sjcl commented Jun 25, 2023

bansoukouしている問題の修正はこのPRとは別で。

@tier940
Copy link
Copy Markdown
Member

tier940 commented Jun 25, 2023

次で良いので下記みたいにフラグを付けてほしいのと、いまだとEZ必須になっちゃうのでmixinのjson名をEZ専用と分るように変更して Loader.isModLoaded で分岐してくださると幸いです...
https://github.com/GTModpackTeam/GTExpert-Core/blob/dev/gradle.properties#L21

@tier940
Copy link
Copy Markdown
Member

tier940 commented Jun 25, 2023

LGTM

@tier940 tier940 merged commit 9b8c5a8 into dev Jun 25, 2023
@tier940 tier940 deleted the patch-ez-storage2 branch June 25, 2023 12:13
@sjcl
Copy link
Copy Markdown
Member Author

sjcl commented Jun 25, 2023

次で良いので下記みたいにフラグを付けてほしいのと、いまだとEZ必須になっちゃうのでmixinのjson名をEZ専用と分るように変更して Loader.isModLoaded で分岐してくださると幸いです... https://github.com/GTModpackTeam/GTExpert-Core/blob/dev/gradle.properties#L21

現状のBuild Scriptだとmixinのjsonは1つだけなのが前提なので、今後他のmodのmixinも作りたくなったときに問題が発生しそうです。(見た感じ何も変更しなくても動きはするが、代表のmixinを1つ選ぶことになり、気持ち悪い感じになりそう。)
そこで、

  • フラグを debug_mixins のようにして、mixinのデバッグをしたいときは全てのmixinの依存関係を読み込むようにする, mixinのjsonは1つにまとめる
  • Build Scriptに変更を加えて、複数のmixinのjsonに対応する

のどちらが良いでしょうか..?

@tier940
Copy link
Copy Markdown
Member

tier940 commented Jun 25, 2023

上でお願いします!
因みに下の場合は GregTechCEu/Buildscripts の更新が来ると多分消えます...

@sjcl
Copy link
Copy Markdown
Member Author

sjcl commented Jun 25, 2023

承知しました👌

因みに下の場合は GregTechCEu/Buildscripts の更新が来ると多分消えます...

なるほどー、いちいちマージするのもやたら面倒そうなので、触らないほうが良いですね🙄

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.

2 participants