Don't add glowing layer component if empty

Author: TrigamDev

src/main/java/dev/trigam/field/impl/FieldBannerBlockEntity.java

@@ -8,4 +8,6 @@ public interface FieldBannerBlockEntity {
 
     void field$setLayerGlowing( int layerIndex, boolean glowing );
 
+    boolean field$isEmpty ();
+
 }

src/main/java/dev/trigam/field/item/ModifyItemInit.java

@@ -1,15 +1,12 @@
 package dev.trigam.field.item;
 
-import dev.trigam.field.component.GlowingLayersComponent;
-import dev.trigam.field.component.ItemComponentInit;
 import dev.trigam.field.config.FieldConfig;
 import net.fabricmc.fabric.api.item.v1.DefaultItemComponentEvents;
 import net.minecraft.component.DataComponentTypes;
 import net.minecraft.component.type.EquippableComponent;
 import net.minecraft.entity.EquipmentSlot;
 import net.minecraft.item.Item;
 import net.minecraft.item.Items;
-
 import java.util.Arrays;
 import java.util.List;
 
@@ -47,15 +44,6 @@ public static void init() {
                     );
                 }
             }
-            // Glowing Banners
-            for ( Item banner : BANNERS ) {
-                context.modify( banner, builder ->
-                    builder.add(
-                        ItemComponentInit.GLOWING_LAYERS,
-                        GlowingLayersComponent.DEFAULT
-                    )
-                );
-            }
         });
     }
 }

src/main/java/dev/trigam/field/mixin/component/BannerEntityComponents.java

@@ -39,6 +39,11 @@ public BannerEntityComponents( BlockEntityType<?> type, BlockPos pos, BlockState
         this.glowingLayers.setLayerGlowing( layerIndex, glowing );
     }
 
+    @Override
+    public boolean field$isEmpty () {
+        return this.glowingLayers.glowingLayers.isEmpty();
+    }
+
     @Inject(
         method = "readComponents",
         at = @At( value = "TAIL" )
@@ -52,7 +57,7 @@ private void readComponents( BlockEntity.ComponentsAccess components, CallbackIn
         at = @At( value = "TAIL" )
     )
     private void addComponents( ComponentMap.Builder builder, CallbackInfo ci ) {
-        builder.add( ItemComponentInit.GLOWING_LAYERS, this.glowingLayers );
+        if ( !this.field$isEmpty() ) builder.add( ItemComponentInit.GLOWING_LAYERS, this.glowingLayers );
     }
 
     @Inject(
@@ -77,7 +82,7 @@ private void readNbt( NbtCompound nbt, RegistryWrapper.WrapperLookup registries,
         at = @At( value = "TAIL" )
     )
     private void writeNbt( NbtCompound nbt, RegistryWrapper.WrapperLookup registries, CallbackInfo ci ) {
-        if ( this.glowingLayers != null ) {
+        if ( this.glowingLayers != null && !this.field$isEmpty() ) {
             nbt.put( "glowing_layers",
                 GlowingLayersComponent.CODEC.encodeStart(
                     registries.getOps( NbtOps.INSTANCE ),