8000 Implementing optional arguments by DerEchtePilz · Pull Request #393 · CommandAPI/CommandAPI · GitHub
[go: up one dir, main page]

Skip to content

Implementing optional arguments #393

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

Merged
merged 28 commits into from
Jan 4, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
cdb4c08
implementing optional arguments
DerEchtePilz Jan 3, 2023
96a03ec
Add optional arguments to the Kotlin DSL. Rename commandapi-kotlin-bu…
DerEchtePilz Jan 4, 2023
5b46749
add missing JavaDocs on AbstractArgument#setOptional(boolean)
DerEchtePilz Jan 4, 2023
2ad5828
fixing build error
DerEchtePilz Jan 4, 2023
3cf8297
adding documentation for optional arguments
DerEchtePilz Jan 4, 2023
6eb09e5
fixing dependency in the Bukkit Kotlin DSL example
DerEchtePilz Jan 4, 2023
7c4f635
address code review
DerEchtePilz Jan 4, 2023
a25547d
create optional_arguments.md
DerEchtePilz Jan 4, 2023
d1c0088
change /kill command to /sayhi
DerEchtePilz Jan 4, 2023
5ad1a21
make Mardown Linter happy (hopefully)
DerEchtePilz Jan 4, 2023
c82feff
fix wrong enchor end
DerEchtePilz Jan 4, 2023
f777925
address code review
DerEchtePilz Jan 4, 2023
33228d4
convert tabs to spaces in Examples.java
DerEchtePilz Jan 4, 2023
c226296
Merge branch 'dev/dev' into dev/dev
JorelAli Jan 4, 2023
210bed6
refactor AbstractCommandAPICommand#register()
DerEchtePilz Jan 4, 2023
afbd40a
Merge branch 'DerEchtePilz:dev/dev' into 'dev/dev'
DerEchtePilz Jan 4, 2023
7e36de1
simplifying AbstractCommandAPICommand#getArgumentsToRegister()
DerEchtePilz Jan 4, 2023
458b8d8
refactor AbstractCommandAPICommand#register() again
DerEchtePilz Jan 4, 2023
3e7c6ca
Add a section 'Setting existing arguments as optional arguments' to o…
DerEchtePilz Jan 4, 2023
d913acc
remove the List parameter from AbstractCommandAPICommand#getCommandsT…
DerEchtePilz Jan 4, 2023
6266990
Merge branch 'dev/dev' into dev/dev
DerEchtePilz Jan 4, 2023
ce835c6
fix a bug where a required argument would be registered as an optiona…
DerEchtePilz Jan 4, 2023
473bc75
implement multiple getOrDefault methods for the CommandArguments class
DerEchtePilz Jan 4, 2023
a31b755
adding documentation for the getOrDefault method
DerEchtePilz Jan 4, 2023
572beb9
address code review
DerEchtePilz Jan 4, 2023
c8c4960
make CommandArguments#get(int) not throw an ArrayIndexOutOfBoundsExce…
DerEchtePilz Jan 4, 2023
3008d60
Merge branch 'dev/dev' into dev/dev
DerEchtePilz Jan 4, 2023
a76530d
fix variable names after merge
DerEchtePilz Jan 4, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
create optional_arguments.md
  • Loading branch information
DerEchtePilz committed Jan 4, 2023
commit a25547d07541f36c544c5c9826ba91302c93f09a
Original file line number Diff line number Diff line change
Expand Up @@ -1235,7 +1235,7 @@ fun argumentkillcmd() {
CommandAPICommand("kill")
.withOptionalArguments(PlayerArgument("target"))
.executesPlayer(PlayerCommandExecutor { player, args ->
val target: Player? = args.get("target") as Player?
val target: Player? = args["target"] as Player?
if (target != null) {
target.health = 0.0
} else {
Expand Down
1 change: 1 addition & 0 deletions docssrc/src/SUMMARY.md