[jboss-dev-forums] [Design of JBoss Portal] - Re: LDAP Support expectations

[bdaw]

It doesn't matter, I had enough other to work do ;-) 

busy time for everybody :)

Thanks for your solution approach, which confirms my understanding of the code. I already have the hibernate extension, group object, group module and portlet - for at least display/add/change/remove groups and display users within the group.
  | For this I extended the common files:
  | in: core/src/resources/portal-core-sar:
  | - jboss-service.xml
  | in: core/src/resources/portal-core-war:
  | - Resource.properties
  | - jboss-portlet.xml
  | - portlet.xml
  | - portlet-instances.xml
  | in identity/src/resources/hibernate:
  | - domain.hbm.xml
  | This may not be the 'non intrusive way' you proposed. 

It's not bad. I see that you mostly changed configuration files - natural thing to plug in new feature. As long as it doesn't change class files it's quite non intrusive ;)
Only thing that may be not nice is the domain.hbm.xml file. The better way would be to create separate configuration file.
So let's say we'll do something like

core/src/resources/portal-core-sar/conf/hibernate/groups/hibernate.cfg.xml (your hibernate configuration)
identity/src/resources/hibernate/identity/domain.hbm.xml (current file)
identity/src/resources/hibernate/groups/domain.hbm.xml (your schema mapping)

then you need to register 5th hibernate mbean using "org.jboss.portal.jems.hibernate.SessionFactoryBinder" in 
core/src/resources/portal-core-sar/META-INF/jboss-service.xml
with name like "portal:service=Hibernate,type=Groups" 
So  you will have your own SessionFactory and it will act like an addon to the portal schema. 
With such approach it will be easy to enable/disable your services without affecting default portal services

The current content of identity package is quite fluent and may change a little. Another thing is that identity modules configuration will probably be moved from jboss-service.xml to separate a configuration file. There is early draft for that here but it's not very defined yet.

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3985070#3985070

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=3985070