Skip to content

Commit

Permalink
Default to empty string instead of null for argument parsing
Browse files Browse the repository at this point in the history
  • Loading branch information
vaperion committed Jul 7, 2023
1 parent bf361ab commit 3e112ac
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions core/src/main/java/me/vaperion/blade/context/Context.java
Original file line number Diff line number Diff line change
Expand Up @@ -60,10 +60,9 @@ public String argument(int index) {

@Nullable
public <T> T parseArgument(int index, Class<T> argumentClass) {
return parseArgument(index, argumentClass, null);
return parseArgument(index, argumentClass, "");
}


@Nullable
public <T> T parseArgument(int index, Class<T> argumentClass, String defaultValue) {
ArgumentProvider<T> provider = blade.getResolver().recursiveResolveProvider(argumentClass, Collections.emptyList());
Expand All @@ -76,7 +75,7 @@ public <T> T parseArgument(int index, Class<T> argumentClass, String defaultValu

@Nullable
public <T> T parseArgument(int index, ArgumentProvider<T> provider) {
return parseArgument(index, provider, null);
return parseArgument(index, provider, "");
}

@Nullable
Expand Down

0 comments on commit 3e112ac

Please sign in to comment.