Skip to content

VillageNames ​

14 merged pull requests.

New Features ​

  • #8 Adds MixinGuiMerchant to intercept villager name rendering in GUIs and applies formatting codes from language file entries like gui.Villager.name=§6%s. Falls back to normal display if not configured. - @Ranzuu
    • Added: GUI name color customization via resource packs, MixinGuiMerchant for name rendering, Language file entries for GUI name formatting
  • #7 Adds configuration flag in VillageGeneratorConfigHandler and updates all biome structure generators to conditionally place beds based on the config setting. - @Caedis
    • Added: Bed spawning configuration option

Changes ​

  • #9 Removed the colour codes from the mod name in VillageNames, generated Tags.VERSION through generateGradleTokenClass and used it for Reference.VERSION, and removed DevVersionWarning. - @tiffit
    • Removed: Gold color codes from mod name, DevVersionWarning class, CURRENT_VERSION file
    • Fixed: Version always displaying as @VERSION@
  • #3 Added allowVillageBrewingStands ("Decor: Allow Village Brewing Stands", default false) to VillageGeneratorConfigHandler, and wrapped the brewing stand placement in DesertStructures, JungleStructures, PlainsStructures, SavannaStructures, SnowyStructures, SwampStructures and TaigaStructures in that check. - @UltraProdigy
    • Added: Configuration flag for brewing stand generation

Bug Fixes ​

  • #14 Adds null checks to ReputationHandler to prevent NPE when accessing NBT data. Changes from 49 to 33 lines. - @koolkrafter5
    • Fixed: NPE log spam in ReputationHandler
  • #13 Corrects cast issue in EntityMonitorHandler (3 line change in cast logic). - @Alexdoru
    • Fixed: Class cast exception in EntityMonitorHandler
  • #10 Fixes NPE in ModObjects.setModLecternBlock (details in linked log: https://mclo.gs/aiursJJ). - @Alexdoru
    • Fixed: NPE in ModObjects.setModLecternBlock
  • #1 Removes __OBFID annotations from four files in block color, model, entity renderer, and entity classes. These strings are internal to the reobfuscator and are not allowed in the codebase. - @Dream-Master
    • Removed: __OBFID strings from EntityFallingBlockCP, ModelPlane, RenderZombieVillagerModern, EntityGuardian
    • Fixed: Build error caused by forbidden __OBFID markers

Performance ​

  • #2 Caches spawn cap calculations (refreshed every ~40 ticks), replaces HashMap<ChunkCoordIntPair, Boolean> with TLongHashSet to eliminate boxing, and caches Monument Bounding Boxes (refreshed every ~5 minutes) in SpawnEventListener. Avoids constant NBT parsing. - @mitchej123
    • Added: Spawn cap caching system, Monument bounding box cache, TLongHashSet for chunk tracking

Removals ​

  • #6 Deletes VersionChecker class (162 lines) and removes related initialization and configuration from VillageNames and GeneralConfig. Also removes reference from Reference utility. - @Alexdoru
    • Removed: VersionChecker class, Version checking initialization

Internal / Refactor ​

  • #12 Removes dead code from GeneralConfig, ConfigInGame, CommonProxy and various handlers. Simplifies SpawnNamingHandler, EntityInteractHandler, and event tracking. - @Alexdoru
    • Removed: Unused configuration options, Dead code in multiple handlers and utilities
  • #11 Simplifies and reduces EntityMonitorHandler from 483 to 398 lines. Adds WitcheryHelper utility class. Updates dependencies. Removes reference from Reference utility class. - @boubou19
    • Added: WitcheryHelper utility class
    • Removed: Unused code from EntityMonitorHandler
    • Fixed: Entity monitor display glitch
  • #5 Removes reflection from multiple handlers and replaces with accessor and injection mixins. Adds VillageNamesCoreMod entry point, mixin classes for ChunkProvider variants, EntityVillager, EntityZombie, and StructureVillagePieces. Updates Gradle and dependency versions. - @ABKQPO
    • Added: Mixin-based accessors for chunk providers, villagers, zombies, and structure pieces, VillageNamesCoreMod class, Mixin infrastructure
    • Removed: Reflection-based access patterns
    • Fixed: Village generation in flat worlds
  • #4 Deletes 46 lines of redundant event registration from ConfigInit. - @Alexdoru
    • Removed: Unused Forge event registration code from ConfigInit

Contributors ​

Everyone who authored, reviewed or co-authored a change on this page, with how many of them:

@Alexdoru (10), @Dream-Master (4), @Caedis (3), @boubou19 (2), @koolkrafter5 (2), @UltraProdigy (2), @ABKQPO (1), @Algent (1), @Ethryan (1), @mitchej123 (1), @Ranzuu (1), @serenibyss (1), @Sopel97 (1), @tiffit (1)