Add compatibility with Linkart mod

[GeeTransit]

Linkart has an interface we can use to check if a minecart is a train (instead of reading from NBT, which you've mentioned was computationally expensive). The Mods utility enum handles optional dependencies (such as Linkart).

For all carts in a train except the leader, they use the modified speed logic (so that they can match the speed of the first cart). I've removed checks for this.hasPassengers() as the code already checks it for standalone carts (in moveOnRailOverwrite).

For the leading cart, nothing much changes - it'll use the vanilla logic if the cart is a chest/hopper minecart or if it's empty, and if it has a passenger it'll use the modified logic. The only change is for ensuring trains accelerate together. I loop through all the following carts to check that the whole train is at the same speed before accelerating (to prevent it from breaking apart).

Note that higher speeds might not work until this Linkart patch smoothing out minecart velocity is merged.

EDIT: I've built a 1.19.4 JAR containing these changes on the releases of my fork.

[audaki]

I just checked your changes, this is much more than a fix and therefore out of scope of what I want to maintain.

Feel free to fork this mod with your changes and publish it on your own.

[GeeTransit]

I've reverted and simplified the changes above. It's closer to what the previous issue suggested as a fix (which is to make follower minecarts also have increased speed).

On Audaki Cart Engine's side, I added a hook for other mods to specify if a cart should use the modified speed logic or not.

On Linkart's side, it can register into the hook and specify that follower minecarts should use the modified speed logic. The acceleration capping logic (to keep the train together) can be implemented from Linkart's side (by using a ModifyArg on the .move() call).

Not sure if a new PR is needed since it's on the same branch.

Also no worries if it's out of scope again - CC0 is amazing :)

[audaki]

@GeeTransit

Hey, I had to reopen this pull request to see your changes. I checked your changes and thought about it. The changes are still kind of invasive and I do have the problem that I can't invest much time into maintaining atm. That's why I kept the scope as small as possible, as vanilla as possible, just like a bug fix for too low speed.

So I really think it'd be best if you forked it and published the mod yourself. I've already seen you have some ideas like different speeds by different ground types and I think many players would enjoy your fork!

kind regards, Kira

PS: Have fun with the fork 😊