Allow registry of Converters with inner CommandManagers

EngineHub / Piston

A generic command system, with tie-ins to many Minecraft platforms such as Bukkit, Forge, and Sponge.

GNU General Public License v3.0

17 stars 3 forks source link

Allow registry of Converters with inner CommandManagers #23

Closed DarkArc closed 4 years ago

DarkArc commented 4 years ago

CommandManager commandManager = service.newCommandManager();
commandManager.registerConverter(Key.of(Foo.class), new FooConverter());
topLevelCommandManager.registerManager(componentManager);

This currently fails (i.e. commandManager is unable to resolve Foo) as the topLevelCommandManager is the only CommandManager who's converters are used.