Update or Replace Transfer API

[Hiiragi283]

現状

• Experimental扱いで使ってるプロジェクトがほぼない
• 最新版と比べると劣る部分が多い
• そもそもTransactionの設計を理解するのがなかなか手間
  という課題を抱えています

そこで私から

1. 最新版のTransfer APIをバックポート
2. ForgeのIItemHandlerに近いスロット管理式のインターフェースを設ける
   という案をそれぞれ出します

[turtton]

2案で新規にAPIを生やしたら

設計を理解するのがなかなか手間

の問題が再発するだけだと思うので、Transfer APIをバックポートするのが無難だと思いました。
ちなみにwikiのこのへんの内容って古かったりするんですかね？

[Hiiragi283]

まあStorage<T>がIterable<StorageView<T>>を継承してるぐらいすかね

[turtton]

ん？良くわからないけど内容が古いわけではなさそう？
この辺の話ですか？
https://github.com/FabricMC/fabric/blob/1.21/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/api/transfer/v1/storage/Storage.java#L64

[Hiiragi283]

基本的な設計とかは変わってないはずです

ちなみにMC1165はIterableを取得するためだけにtransaction開かないといけないのでカス（経験談）

[turtton]

確かそれが最新だと良くなってるんでしたっけ？

[Hiiragi283]

はい
単純なIterableの実装に置き換えられたので、transactionなしで取得できるようになりました

[Hiiragi283]

あとこれらに付随してFluidVariantAttributeHandlerも欠かせないですね

これがないせいで液体の名前とか温度を引っ張ってこれない

[turtton]

transfer-apiごと全部ポートすれば丸く収まりそうですね

[Hiiragi283]

ざっくり見た感じ、この2つがデカい変更すっね
あとは@ApiStatus.Experimentalを全部引っ剥がす

mixinやimplは頑張って合わせる（力技）

[Hiiragi283]

とりあえず本家のコードで全部置換して目立ったエラーは修正して上げました

[turtton]

おつかれさまです。PRにしてもらえたら見ますよー

[Hiiragi283]

では早速