Skip to content

feat(appeng): enhance appeng integration APIs#188

Merged
Luca-Guettinger merged 8 commits intoGTNewHorizons:masterfrom
hinyb:fix
Apr 25, 2026
Merged

feat(appeng): enhance appeng integration APIs#188
Luca-Guettinger merged 8 commits intoGTNewHorizons:masterfrom
hinyb:fix

Conversation

@hinyb
Copy link
Copy Markdown

@hinyb hinyb commented Apr 24, 2026

enhance appeng integration APIs

  • support long in request()
  • make size optional
  • enhance updateAE to allow pass a table

Copy link
Copy Markdown

@Luca-Guettinger Luca-Guettinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The critical issue is the silent default change from 64 to 1 on the database path of requestItems, which will break existing OC programs.

Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Copy link
Copy Markdown

@Luca-Guettinger Luca-Guettinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The !args.isInteger(2) fix for the default amount breaks the table path by overriding "size", and getStackFromDatabase is the wrong utility for the fluid database path because it consumes the amount argument as an irrelevant ItemStack stackSize.

Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
Comment thread src/main/scala/li/cil/oc/integration/appeng/UpgradeAE.scala Outdated
@Luca-Guettinger Luca-Guettinger merged commit bfa92e1 into GTNewHorizons:master Apr 25, 2026
1 check passed
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