Skip to content

Dynamic Commands 3#562

Closed
AlphaModder wants to merge 5 commits intoSpongePowered:feature/commandsfrom
AlphaModder:feature/dynamic-commands-3
Closed

Dynamic Commands 3#562
AlphaModder wants to merge 5 commits intoSpongePowered:feature/commandsfrom
AlphaModder:feature/dynamic-commands-3

Conversation

@AlphaModder
Copy link
Contributor

Continuation of #539 for zml's new API. I'll write a description in more detail when it isn't 2 AM...

Copy link
Contributor

Choose a reason for hiding this comment

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

Missing javadocs

@Zidane Zidane added this to the Revision 2.1 milestone Apr 11, 2015
@zml2008 zml2008 modified the milestones: Revision 2.0, Revision 2.1 Apr 12, 2015
zml2008 added a commit to SpongePowered/SpongeForge that referenced this pull request Apr 12, 2015
zml2008 added a commit to SpongePowered/SpongeForge that referenced this pull request Apr 12, 2015
zml2008 added a commit to SpongePowered/SpongeForge that referenced this pull request Apr 13, 2015
@AlphaModder
Copy link
Contributor Author

No reason for this anymore.

zml2008 added a commit to SpongePowered/SpongeForge that referenced this pull request Apr 17, 2015
zml2008 added a commit that referenced this pull request Apr 17, 2015
Fixes #551. Closes #539. Closes #532. Closes #523.

Add CommandResult object to hold results of command execution.
Closes #552

Add LocatedSource class for command sources with a location

Allow configurable conflict resoultion at command execution time
Closes #562

Tweak formatting, work on dimension, world, and child command elements

Add an abstract command element that supports matching by pattern and apply it to world, player, catalog type, and enum values argument types
zml2008 added a commit to SpongePowered/SpongeForge that referenced this pull request Apr 17, 2015
Implement dynamic command alias resolution.
This is the implementation for SpongePowered/SpongeAPI#562

Convert Sponge command to new commands API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants