Card-Forge
Keyword Offering update like Defiler as Additional Cost Increase + Reduction
the Keyword Offering works similar like the Defiler Ability
when updating the Defiler Ability for Card Text Changes #1828 we might use the same mechanic for Offering so we don't need extra variables anymore.
if possible we might look into Emerge for this too
This issue has not been updated in a while and has now been marked as stale. Stale messages will be auto closed.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
This issue has not been updated in a while and has now been marked as stale. Stale messages will be auto closed.
I think keeping AI support will be the tricky part.
For Offering/Emerge, i think once i get the Functionality going, the AI part shouldn't be that much of a problem
@tool4ever you recently touched Offering,
https://github.com/Card-Forge/forge/blob/2d20ce07e3ec70b615e9b3ea039ba888c68d3188/forge-gui/src/main/java/forge/player/HumanPlay.java#L520-L540 shouldn't HumanPlay call this Method in CostPayment? https://github.com/Card-Forge/forge/blob/2d20ce07e3ec70b615e9b3ea039ba888c68d3188/forge-game/src/main/java/forge/game/cost/CostPayment.java#L336-L364
Also, isn't this part redundant? https://github.com/Card-Forge/forge/blob/2d20ce07e3ec70b615e9b3ea039ba888c68d3188/forge-gui/src/main/java/forge/player/HumanPlay.java#L606-L615
i might need to clean this up if i want to do Convoke/Improvise later🤔
I think it's been tricky to merge the logic because of AI and human differences there