Remove some assumptions about the type of the attachment controller

bergerhealer / TrainCarts

Minecarts redefined

MIT License

205 stars 63 forks source link

Remove some assumptions about the type of the attachment controller #454

Closed 56738 closed 2 years ago

56738 commented 2 years ago

Fixes some issues which prevent using a custom AttachmentManager:

Add getViewers to AttachmentManager so other manager implementations can provide viewers too
Make sure the manager is a controller (hasController) before casting

bergerkiller commented 2 years ago

The cartattachment stuff is a bit confusing and some of it doesn't fully work with a proper api, but I guess thats just the way it is. I'll add it in to fix those issues when working outside tc's attachment manager.