Skip to content

Commit

Permalink
chore: reformat project
Browse files Browse the repository at this point in the history
  • Loading branch information
broccolai committed Oct 26, 2023
1 parent 9161c5e commit 235560f
Show file tree
Hide file tree
Showing 52 changed files with 343 additions and 315 deletions.
375 changes: 222 additions & 153 deletions .checkstyle/checkstyle.xml

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions .editorconfig
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ indent_style = space
insert_final_newline = true
max_line_length = 130
tab_width = 4
ij_continuation_indent_size = 8
ij_continuation_indent_size = 4
ij_formatter_off_tag = @formatter:off
ij_formatter_on_tag = @formatter:on
ij_formatter_tags_enabled = false
Expand Down Expand Up @@ -137,7 +137,7 @@ ij_java_for_statement_wrap = off
ij_java_generate_final_locals = true
ij_java_generate_final_parameters = true
ij_java_if_brace_force = always
ij_java_imports_layout = *, |, javax.**, java.**, |, $*
ij_java_imports_layout = *, |, $*
ij_java_indent_case_from_switch = true
ij_java_insert_inner_class_imports = false
ij_java_insert_override_annotation = true
Expand Down
2 changes: 1 addition & 1 deletion api/src/main/java/broccolai/tags/api/TagsApi.java
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ public static void register(final @NonNull Injector injector) {
INJECTOR = injector;
}

public static <T extends Service> T getService(final @NonNull Class<T> clazz) {
public static <T extends Service> T service(final @NonNull Class<T> clazz) {
return INJECTOR.getInstance(clazz);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
import broccolai.tags.api.model.Permissible;
import net.kyori.adventure.text.Component;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;

public record ConstructedTag(
int id,
Expand Down
7 changes: 3 additions & 4 deletions api/src/main/java/broccolai/tags/api/model/user/TagsUser.java
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,18 @@

import broccolai.tags.api.model.tag.ConstructedTag;
import broccolai.tags.api.model.user.impl.ConsoleTagsUser;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;

import java.util.Optional;
import java.util.UUID;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;

public interface TagsUser {

TagsUser CONSOLE = new ConsoleTagsUser();

@NonNull UUID uuid();

void setCurrent(@Nullable ConstructedTag tag);
void current(@Nullable ConstructedTag tag);

@NonNull Optional<Integer> current();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,10 @@

import broccolai.tags.api.model.tag.ConstructedTag;
import broccolai.tags.api.model.user.TagsUser;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;

import java.util.Optional;
import java.util.UUID;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;

public final class ConsoleTagsUser implements TagsUser {

Expand All @@ -20,7 +19,7 @@ public final class ConsoleTagsUser implements TagsUser {
}

@Override
public void setCurrent(final @Nullable ConstructedTag tag) {
public void current(final @Nullable ConstructedTag tag) {
this.currentTag = tag != null ? tag.id() : null;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,10 @@

import broccolai.tags.api.model.tag.ConstructedTag;
import broccolai.tags.api.model.user.TagsUser;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;

import java.util.Optional;
import java.util.UUID;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;

public final class PlayerTagsUser implements TagsUser {

Expand All @@ -27,7 +26,7 @@ public PlayerTagsUser(
}

@Override
public void setCurrent(final @Nullable ConstructedTag tag) {
public void current(final @Nullable ConstructedTag tag) {
this.current = tag != null ? tag.id() : null;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,9 @@

import broccolai.tags.api.model.Service;
import broccolai.tags.api.model.user.TagsUser;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.Optional;
import java.util.UUID;
import org.checkerframework.checker.nullness.qual.NonNull;

public interface DataService extends Service {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,11 @@
import broccolai.tags.api.model.Service;
import broccolai.tags.api.model.tag.ConstructedTag;
import broccolai.tags.api.model.user.TagsUser;
import java.util.Collection;
import net.kyori.adventure.text.Component;
import net.kyori.adventure.text.minimessage.tag.resolver.TagResolver;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.Collection;

public interface MessageService extends Service {

TagResolver prefix();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,11 @@
import broccolai.tags.api.model.tag.ConstructedTag;
import broccolai.tags.api.model.tag.TagDisplayInformation;
import broccolai.tags.api.model.user.TagsUser;
import java.util.Collection;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.checkerframework.checker.nullness.qual.Nullable;
import org.jetbrains.annotations.NotNull;

import java.util.Collection;

public interface TagsService extends Service {

@NotNull ConstructedTag defaultTag();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,11 @@

import broccolai.tags.api.model.Service;
import broccolai.tags.api.model.user.TagsUser;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.Collection;
import java.util.List;
import java.util.Map;
import java.util.UUID;
import org.checkerframework.checker.nullness.qual.NonNull;

public interface UserService extends Service {

Expand Down
2 changes: 2 additions & 0 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,8 @@ subprojects {
ci(true)
publishing(true)
}

checkstyle("10.12.4")
}

processResources {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
import cloud.commandframework.paper.PaperCommandManager;
import com.google.inject.Guice;
import com.google.inject.Injector;
import java.util.Collection;
import org.bukkit.Bukkit;
import org.bukkit.command.CommandSender;
import org.bukkit.command.ConsoleCommandSender;
Expand All @@ -33,8 +34,6 @@
import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.Collection;

public final class BukkitTagsPlatform extends JavaPlugin implements TagsPlatform {

private static final @NonNull Class<? extends Listener>[] BUKKIT_LISTENERS = ArrayUtilities.create(
Expand Down Expand Up @@ -62,7 +61,7 @@ public void onEnable() {
this.plugin = injector.getInstance(TagsPlugin.class);
this.plugin.start();

CommandManager<CommandUser> commandManager = this.getCommandManager(
CommandManager<CommandUser> commandManager = this.commandManager(
injector.getInstance(MessageService.class)
);

Expand All @@ -77,7 +76,6 @@ public void onEnable() {
);
}


this.plugin.commands(commandManager, BUKKIT_COMMANDS);
}

Expand All @@ -86,7 +84,7 @@ public void onDisable() {
this.plugin.shutdown();
}

private CommandManager<CommandUser> getCommandManager(
private CommandManager<CommandUser> commandManager(
final @NonNull MessageService messageService
) {
try {
Expand All @@ -107,7 +105,7 @@ private CommandManager<CommandUser> getCommandManager(

new MinecraftExceptionHandler<CommandUser>()
.withDefaultHandlers()
.apply(commandManager, (e) -> e);
.apply(commandManager, e -> e);

commandManager.registerExceptionHandler(UserArgument.UserArgumentException.class, (user, ex) -> {
user.sendMessage(messageService.commandErrorUserNotFound(ex.input()));
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
package broccolai.tags.bukkit.commands.context;

import broccolai.tags.api.model.user.impl.ConsoleTagsUser;
import java.util.UUID;
import org.bukkit.command.CommandSender;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.UUID;

public final class BukkitConsoleCommandUser extends BukkitCommandUser {

public BukkitConsoleCommandUser(final @NonNull CommandSender source) {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
package broccolai.tags.bukkit.commands.context;

import java.util.UUID;
import org.bukkit.entity.Player;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.UUID;

public final class BukkitPlayerCommandUser extends BukkitCommandUser {

private final Player player;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,11 @@
import broccolai.tags.bukkit.service.BukkitTaskService;
import broccolai.tags.core.platform.TagsPlatform;
import com.google.inject.AbstractModule;
import java.io.File;
import org.bukkit.plugin.Plugin;
import org.checkerframework.checker.nullness.qual.NonNull;
import org.slf4j.Logger;

import java.io.File;

public final class PlatformModule extends AbstractModule {

private final @NonNull BukkitTagsPlatform plugin;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,10 @@ public BasicIntegration(final @NonNull UserService userService, final @NonNull T

@EventHandler
public void onChat(final @NonNull AsyncChatEvent event) {
event.renderer(((source, name, message, viewer) -> {
event.renderer((source, name, message, viewer) -> {
ConstructedTag tag = this.tagsService.load(this.userService.get(source.getUniqueId()));
return message.replaceText(builder -> builder.matchLiteral("%tag%").replacement(tag.component()));
}));
});
}

}
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ public String getVersion() {
public String onRequest(final @NonNull OfflinePlayer player, final @NonNull String identifier) {
TagsUser user = this.userService.get(player.getUniqueId());

if (identifier.equalsIgnoreCase("current")) {
if ("current".equalsIgnoreCase(identifier)) {
ConstructedTag tag = this.tagsService.load(user);
return LEGACY.serialize(tag.component());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,7 @@
import broccolai.tags.api.service.TaskService;
import broccolai.tags.api.service.UserService;
import com.google.inject.Inject;

import java.util.UUID;

import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerJoinEvent;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,10 @@
import broccolai.tags.core.config.LocaleConfiguration;
import broccolai.tags.core.util.FormatingUtilites;
import com.google.inject.Inject;
import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;
import net.kyori.adventure.text.Component;
import net.kyori.adventure.text.TextComponent;
import net.kyori.adventure.text.format.NamedTextColor;
import org.bukkit.Material;
import org.bukkit.inventory.ItemStack;
Expand All @@ -24,10 +26,6 @@
import org.incendo.interfaces.paper.type.ChestInterface;
import org.slf4j.Logger;

import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;

public final class TagsMenuFactory {

private static final ItemStackElement<ChestPane> BOTTOM_BAR_ELEMENT = new ItemStackElement<>(
Expand Down Expand Up @@ -85,28 +83,26 @@ private PaginatedTransform<ItemStackElement<ChestPane>, ChestPane, PlayerViewer>
private List<ItemStackElement<ChestPane>> createTagElements(final @NonNull TagsUser user) {
return this.tagsService.allTags(user)
.stream()
.map((tag) -> createTagElement(user, tag))
.map(tag -> this.createTagElement(user, tag))
.toList();
}

private ItemStackElement<ChestPane> createTagElement(final @NonNull TagsUser user, final @NonNull ConstructedTag tag) {


Material material = matchMaterialOrDefault(tag.displayInformation().material());
Material material = this.matchMaterialOrDefault(tag.displayInformation().material());
ItemStack item = PaperItemBuilder.ofType(material)
.name(tag.component())
.lore(this.createTagLore(tag))
.build();

return ItemStackElement.of(item, (ctx) -> {
return ItemStackElement.of(item, ctx -> {
this.actionService.select(user, tag);
});
}

private List<Component> createTagLore(final @NonNull ConstructedTag tag) {
List<Component> result = new ArrayList<>();

result.addAll(formatTagReason(tag.reason()));
result.addAll(this.formatTagReason(tag.reason()));

result.add(Component.empty());
result.add(this.locale.equip.asComponent());
Expand All @@ -117,7 +113,7 @@ private List<Component> createTagLore(final @NonNull ConstructedTag tag) {
private List<Component> formatTagReason(final @NonNull String reason) {
return FormatingUtilites.splitString(reason, 30)
.stream()
.map((text) -> Component.text(text, NamedTextColor.WHITE))
.map(text -> Component.text(text, NamedTextColor.WHITE))
.collect(Collectors.toList());
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,13 @@
import broccolai.tags.core.service.user.PipelineUserService;
import com.google.inject.Inject;
import com.google.inject.Injector;
import java.util.List;
import java.util.Objects;
import org.bukkit.Bukkit;
import org.bukkit.OfflinePlayer;
import org.bukkit.entity.Player;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.List;
import java.util.Objects;

public final class BukkitPipelineUserService extends PipelineUserService {

@Inject
Expand Down
3 changes: 1 addition & 2 deletions core/src/main/java/broccolai/tags/core/TagsPlugin.java
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,8 @@
import com.google.inject.Injector;
import com.google.inject.Singleton;
import com.zaxxer.hikari.HikariDataSource;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.Collection;
import org.checkerframework.checker.nullness.qual.NonNull;

@Singleton
public final class TagsPlugin {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,8 @@
import cloud.commandframework.CommandManager;
import cloud.commandframework.context.CommandContext;
import com.google.inject.Inject;
import org.checkerframework.checker.nullness.qual.NonNull;

import java.util.Collection;
import org.checkerframework.checker.nullness.qual.NonNull;

public final class TagsAdminCommand implements PluginCommand {

Expand Down
Loading

0 comments on commit 235560f

Please sign in to comment.