Move un-/register new extensions to the config files

[ghost]

It would be better, if the registering and unregistering of extensions could be achieved with the configuration, instead of using the onBootstrap() hook in every module.

Something like:

return [
    'ldc_user_profile' => [
        'registered_extensions' => [
            'MyModule\UserProfile\Extension' => true,
            'MyOtherModule\UserProfile\Extension' => false,
        ]
    ]
];

Please tell me, if you think this is good. I can try to make a PR for it, if you want to.

[adamlundrigan]

That's a very good idea. It should be as simple as adding the registered_extensions key to ModuleOptions and add some code to onBootstrap to pull the ProfileService instance from the SM and register the listed extensions.

My choice would be to use the SM name of the extension's main class (AbstractExtension), so the default set of registered extensions would look like this:

return [
    'ldc_user_profile' => [
        'registered_extensions' => [
            'ldc-user-profile_extension_zfcuser' => true,
        ]
    ]
];

And the onBootstrap code would look something like this:

$sm = $e->getApplication()->getServiceManager();
$extensions = $sm->get('ldc-user-profile_module_options')->getRegisteredExtensions();
$extensionService = $sm->get('ldc-user-profile_service');

foreach ( $extensions as $extension => $isEnabled ) {
    $objExt = $sm->get($extension);
    if ( ! $objExt instanceof AbstractExtension ) {
        continue;
    }
    $isEnabled ? $extensionService->registerExtension($objExt)
                       : $extensionService->unregisterExtension($objExt);
}

If you would like to send a PR for this you're more than welcome to :) If not I'll make some time to implement it in the next few days.

[ghost]

I will try to make a PR tomorrow, at least until tuesday. Thanks for being open to new ideas.

[ghost]

I decided to move the register and unregister action to the factory. The onBootstrap method can then be removed and the process doesn't have to be done for each request. Instead it will only be executed, if the ProfileService is really needed.

[adamlundrigan]

Fixed in #8