Re: [gatein-dev] Fwd: Setup Root password feature improvement

On 11.12.2013 07:48, Hai Nguyen wrote: ---------- Forwarded message ---------- From: Hai Nguyen <haint(a)exoplatform.com&gt; Date: Wed, Dec 11, 2013 at 8:39 AM Subject: Re: [gatein-dev] Setup Root password feature improvement To: Marek Posolda <mposolda(a)redhat.com&gt; On Tue, Dec 10, 2013 at 6:08 PM, Marek Posolda <mposolda(a)redhat.com&gt;wrote: > On 9.12.2013 11:35, Hai Nguyen wrote: > > > >> > - The SetupOrganizationDatabaseInitializer was clone of >> > OrganizationDatabaseInitializer (in core service), that is hard for >> maintain >> >> Yes, I agree that clone was not more elegant way, but there was not >> possibility to extend OrganizationDatabaseInitializer with a custom class, >> so clone was the only option to add custom behaviour. >> >> I would suggest to give the ability to extend this class and override >> logic from other localization rather that exo.core module. >> >> IMO, the new behaviors are not much to introduce a new custom class. > Beside, I don't know why we need introduce more "updateUsers" property that > is not necessary. If you looking for my PR, you will see we don't need > SetupOrganizationDatabaseInitializer for this feature > > Property "updateUsers" is another thing unrelated to Root user > password setup. It's mentioned in this JIRA > https://issues.jboss.org/browse/GTNPORTAL-3296 . It would be nice if > OrganizationDatabaseInitializer from eXo core doesn't have all methods > private, so we don't need to fork whole class into GateIn if some minor > change is required. I can create JIRA to eXo core for this and also for > "updateUsers" support. WDYT? > > +1 :) Created: https://jira.exoplatform.org/browse/COR-313 Add "updateUsers" option to OrganizationDatabaseInitializer https://jira.exoplatform.org/browse/COR-314 Refactor OrganizationDatabaseInitializer to be better extensible I would be happy to not remove SetupOrganizationDatabaseInitializer from GateIn master and 3.6.x until COR-313 will be available in exo.core and dependency updated in GateIn to version where it's available. Thanks, Marek _______________________________________________ gatein-dev mailing listgatein-dev@lists.jboss.orghttps://lists.jboss.org/mailman/listinfo/gatein-dev _______________________________________________ gatein-dev mailing list gatein-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/gatein-dev