Thomas Vandahl wrote:
> Now, first step. Currently, all the loaders have their own caching. I
> propose to shift the cache into the AssemblerBrokerService. That would
> mean a common cache for all types of modules, where the key would
> consist of the module type, a colon and the name of the module like in
> "action:LoginUser". All exception handling would be concentrated at one
> place and the loaders could go in the next step.
Done.

Second step of reorganization:
- Loaders are still there
- AssemblerFactory extends Loader now, I'm planning to merge the two at
least for the existing implementations
- The AssemblerFactories got a getLoader() method which allows to
retrieve Loaders and cache sizes only by specifying the module NAME.
- The AssemblerBrokerService now has a method getLoader(type).
- In the TemplateService this allows to query the AssemblerBrokerService
for configured module types.
- The TemplateService needs a lot of adjustments to overcome the
hard-coded seven module types -> next step.

Please review.

Bye, Thomas.


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to