add an option to add markers instantly on banner place

closes #5
typos

Author: RealMuffinTime

src/main/java/dev/nincodedo/bluemapbanners/BlueMapBanners.java

@@ -65,7 +65,9 @@ public void onInitialize() {
     }
 
     private ActionResult useBlock(PlayerEntity player, World world, Hand hand, BlockHitResult hitResult) {
-        if (player.isSpectator() || player.getMainHandStack().isEmpty() && player.getOffHandStack().isEmpty()) {
+        if (player.isSpectator() ||
+                (player.getMainHandStack().isEmpty() && player.getOffHandStack().isEmpty()) ||
+                ConfigManager.getInstance().getBoolConfig("markerAddInstantOnBannerPlace")) {
             return ActionResult.PASS;
         }
         if (player.getMainHandStack().isOf(Items.MAP) || player.getOffHandStack().isOf(Items.MAP) || player.getMainHandStack().isOf(Items.FILLED_MAP) || player.getOffHandStack().isOf(Items.FILLED_MAP)) {

src/main/java/dev/nincodedo/bluemapbanners/ConfigManager.java

@@ -28,11 +28,12 @@ public ConfigManager()
     {
         this.configFilePath = FabricLoader.getInstance().getConfigDir().resolve( "bluemap-banners/bluemap-banners.properties" ).toString();
 
-        // Initialize hashtable with default values
+        // Initialize the hashtable with default values
         defaultConfig.put("notifyPlayerOnBannerPlace", "true");
         defaultConfig.put("notifyPlayerOnMarkerAdd", "true");
         defaultConfig.put("notifyPlayerOnMarkerRemove", "true");
         defaultConfig.put("notifyGlobalOnMarkerRemove", "false");
+        defaultConfig.put("markerAddInstantOnBannerPlace", "false");
         defaultConfig.put("markerMaxViewDistance", "10000000");
         defaultConfig.put("bluemapUrl", "https://your-url-to-bluemap.com/");
 
@@ -70,7 +71,7 @@ private void readConfigFile() {
 
     public void updateConfigFile() {
         try {
-            // read current file and save found configs
+            // read the current file and save found configs
             String line;
             StringBuilder stringBuilder = new StringBuilder();
             BufferedReader fileIn = new BufferedReader(new FileReader(this.configFilePath));
@@ -106,7 +107,7 @@ public void updateConfigFile() {
     }
 
     public void initialConfigFile() {
-        // Create config folder if not exist
+        // Create the config folder if not exist
         try {
             Files.createDirectories(FabricLoader.getInstance().getConfigDir());
         } catch (IOException e) {

src/main/java/dev/nincodedo/bluemapbanners/mixin/BlockItemMixin.java

@@ -1,7 +1,9 @@
 package dev.nincodedo.bluemapbanners.mixin;
 
 import dev.nincodedo.bluemapbanners.ConfigManager;
+import dev.nincodedo.bluemapbanners.MarkerManager;
 import net.minecraft.block.BlockState;
+import net.minecraft.block.entity.BannerBlockEntity;
 import net.minecraft.entity.player.PlayerEntity;
 import net.minecraft.item.BlockItem;
 import net.minecraft.item.ItemStack;
@@ -23,8 +25,26 @@ public class BlockItemMixin {
     @Inject(at = @At("HEAD"), method = "postPlacement")
     public void onPlaceInject(BlockPos pos, World world, PlayerEntity player, ItemStack stack, BlockState state, CallbackInfoReturnable<Boolean> cir) {
         if (state.isIn(BlockTags.BANNERS)) {
-            if (ConfigManager.getInstance().getBoolConfig("notifyPlayerOnBannerPlace")) {
-                player.sendMessage(Text.literal("[BlueMap Banners] Use a map item on the banner to add a marker on the ").append(Text.literal("web map").setStyle(Style.EMPTY.withClickEvent(new ClickEvent.OpenUrl(URI.create(ConfigManager.getInstance().getConfig("bluemapUrl")))).withUnderline(true))).append(Text.of(".")), false);
+            MarkerManager markerManager = MarkerManager.getInstance();
+            ConfigManager configManager = ConfigManager.getInstance();
+            if (configManager.getBoolConfig("markerAddInstantOnBannerPlace")) {
+                BannerBlockEntity bannerBlockEntity = (BannerBlockEntity) world.getBlockEntity(pos);
+                if (bannerBlockEntity != null && !markerManager.doesMarkerExist(bannerBlockEntity)) {
+                    String name;
+                    if (bannerBlockEntity.getCustomName() != null) {
+                        name = bannerBlockEntity.getCustomName().getString();
+                    } else {
+                        String blockTranslationKey = state.getBlock().getTranslationKey();
+                        name = Text.translatable(blockTranslationKey).getString();
+                    }
+                    markerManager.addMarker(bannerBlockEntity, name);
+                    if (configManager.getBoolConfig("notifyPlayerOnMarkerAdd"))
+                        player.sendMessage(Text.literal("[BlueMap Banners] You added a marker to the ").append(Text.literal("web map").setStyle(Style.EMPTY.withClickEvent(new ClickEvent.OpenUrl(URI.create(ConfigManager.getInstance().getConfig("bluemapUrl")))).withUnderline(true))).append(Text.of(".")), false);
+                }
+            } else {
+                if (configManager.getBoolConfig("notifyPlayerOnBannerPlace")) {
+                    player.sendMessage(Text.literal("[BlueMap Banners] Use a map item on the banner to add a marker on the ").append(Text.literal("web map").setStyle(Style.EMPTY.withClickEvent(new ClickEvent.OpenUrl(URI.create(ConfigManager.getInstance().getConfig("bluemapUrl")))).withUnderline(true))).append(Text.of(".")), false);
+                }
             }
         }
     }