Paradaux
Issues

PAR-316

0

Service-layer consolidation + *Service naming normalisation

Pending ReleaseNormalUnassignedChestShopImprovement

Thorough service-layer pass. Consolidate one-trick ponies + normalise naming so every service is *Service/*ServiceImpl:

  • Protection: fold Security (a facade over ProtectionService) + VanillaShopProtection (its default provider) into ProtectionService/Impl.
  • Rename: AdminBypass->AdminBypassService, PreviewHandler->PreviewService, ChestShopSign->SignService, MetricsModule->MetricsService (static->@Singleton).
  • Market cluster: MarketHook + MarketRecords -> a MarketService (instance; holds the Treasury market APIs + builds the DTOs).
  • Helpers: fold ItemAliasModule into ItemService; move ItemInfoLines (a data accumulator) to model/. Kept as-is: AccountService, BusinessAccountService, EconomyService, InfoService, InventoryService, ItemCodeService, ItemService, MaterialService, ShopBlockService, ShopFinderService, ShopService, TransactionService, MarketResyncService, CustomItemResolver (SPI).

Resources

Comments

No comments yet.

Activity

  • ParadauxIO linked a commit — Commit 6761669 — Merge MarketHook + MarketRecords into a MarketService (PAR-316)
  • ParadauxIO linked a commit — Commit a1789b6 — Fold ItemAliasModule into ItemService; move ItemInfoLines to model/ (PAR-316)
  • ParadauxIO linked a commit — Commit ee00985 — Normalise service naming to *Service/*ServiceImpl (PAR-316)
  • ParadauxIO linked a commit — Commit fbf7d5e — Fold Security + VanillaShopProtection into ProtectionService (PAR-316)
  • tesks changed status to Status → Pending Release
  • tesks created the issue