Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose components' capabilities #1837

Closed
fnuecke opened this issue May 29, 2016 · 7 comments
Closed

Expose components' capabilities #1837

fnuecke opened this issue May 29, 2016 · 7 comments
Labels
make-a-pr-if-you-want-this Feature request expired. PRs generally welcome

Comments

@fnuecke
Copy link
Member

fnuecke commented May 29, 2016

Allow components to provide capabilities, make them available to the outside (i.e. have cases, racks, robots, drones forward them).

@Vexatos
Copy link
Contributor

Vexatos commented May 29, 2016

Tablets too, maybe?

@fnuecke
Copy link
Member Author

fnuecke commented May 29, 2016

FTR: needs ItemStack capabilities which is not possible with the way tablet machines/components are currently handled in OC. Will look into that again at a later point, when I restructure that.

@gjgfuj
Copy link

gjgfuj commented May 30, 2016

How would you handle if multiple components expose the same capability?

@fnuecke
Copy link
Member Author

fnuecke commented May 30, 2016

First one with hasCapability==true wins, pretty sure that's how it's for MCMP (which in this case is quite comparable), too.

@gjgfuj
Copy link

gjgfuj commented May 31, 2016

Mcmp has sides, because they're still technically blocks. Items wouldn't
provide sides. Also, that sounds like a flaw in mcmp, unless there's a
order that makes sense.

Sorry, something went wrong.

@fnuecke
Copy link
Member Author

fnuecke commented May 31, 2016

Items wouldn't provide sides.

Tell that to whoever decided to use the same interface for ItemStacks >_> (yes, the capabilities for items also take an EnumFacing parameter).

Sorry, something went wrong.

@gjgfuj
Copy link

gjgfuj commented May 31, 2016

I'd still suggest including a priority system for forwarding capabilities,
or not providing it at all. (Forwarding item capabilities when you have >1
item non-sided is fundamentally different to forwarding caps in a block
space.)

Sorry, something went wrong.

@payonel payonel added make-a-pr-if-you-want-this Feature request expired. PRs generally welcome and removed feature-accepted labels May 12, 2021
@payonel payonel closed this as completed May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
make-a-pr-if-you-want-this Feature request expired. PRs generally welcome
Projects
None yet
Development

No branches or pull requests

4 participants