]
Sergei Egorov commented on FURNACE-112:
---------------------------------------
[~gastaldi] done. PR is linked to the issue. I tried to keep binary compatibility, also
it's "0 changes in tests", so should be safe.
Separate deploy/enable/disable methods from
AddonRepository/MutableAddonRepository
----------------------------------------------------------------------------------
Key: FURNACE-112
URL:
https://issues.jboss.org/browse/FURNACE-112
Project: Forge: Furnace
Issue Type: Enhancement
Components: Addon Manager
Affects Versions: 2.23.4.Final
Reporter: Sergei Egorov
Hi!
Looks like AddonRepository is doing two different jobs - addons repository *and* storing
the state. But if one wants to store the state of deployed & enabled addons, there is
no interface to implement to keep filesystem storage of artifacts, but having state
somewhere else (DB, in memory data grid, just in memory, not the file system)
I think following methods should have their own interface:
isEnabled()
listAll()
listEnabled()
listEnabledCompatibleWithVersion()
enable()
disable()
it will make it possible to provide non-FS implementation of the state storage.
Thanks!