PAR-308
0
utils/ consistency pass — naming, private ctors, consolidation
Pending ReleaseNormalUnassignedChestShopImprovement
Pass over utils/ for consistency:
- Dissolve ShopBlockUtil (findNeighbor + face constants) into BlockUtil; dissolve the now-trivial ImplementationAdapter (Paper snapshot pass-throughs) into BlockUtil.getState + InventoryUtil.getHolder/getLeftSide/getRightSide.
- Rename Messages -> MessageUtil for suffix consistency.
- Add missing private constructors + final to the pure-static utils (BlockUtil, LocationUtil, NameUtil, NumberUtil, PriceUtil, QuantityUtil, StringUtil).
- Kept: Permissions (permission-node constants registry, not a generic helper bag), SimpleCache (generic utility type), and the cohesive domain utils kept separate (NumberUtil vs PriceUtil vs QuantityUtil; StringUtil vs NameUtil vs LocationUtil).
Follow-ups (separate): AdminInventory is a stateful 38-method Inventory impl mislocated in utils/ (needs re-engineering); FileFormatter/java.util.logging should migrate to slf4j like the sibling plugins.
Resources
- commit24af419 utils/ consistency pass: naming, private ctors, consolidation (PAR-308)ParadauxIO/hibernia-economy
Comments
No comments yet.
Activity
- tesks changed status to Status → Pending Release
- ParadauxIO linked a commit — Commit 24af419 — utils/ consistency pass: naming, private ctors, consolidation (PAR-308)
- tesks created the issue