Hi guys,

as a follow up for my last two mails, I solved my issue this way :
- I created a new module named ldap-schema-manager-tests
- I migrated all the ldap-schema-manager tests to this new module

Now, there are not anymore cyclic dependencies.

Of course, it's not a perfect solution, and we had a lengthly discussion with Pierre-Arnaud about how shared could have been organized to avoid such a hack.

The main problem is that the SchemaManager class was supposed to be a simple container, to be used by both the server and the client API. However, it should be possible to ask it to load a specific schema as a whole (schemaManager.load( "nis" ) for instance). n order to do that, there is a close relation between the ldap-schema-manager module with the ldap-schema-loader and ldap-schema modules.

A better solution would be to refactor the whole stuff, allowing us to load schemas in another format that the ADS one, by injecting a specific loader into the SchemaManager. That's an option, but I think we can think about it a bit later.

Right now, I'm continuing my investigation of the ACI module, as we have more or less 7 issues to fix.

--
Regards,
Cordialement,
Emmanuel Lécharny
www.nextury.com


Reply via email to