My silly mistake! On Thu, Nov 26, 2015 at 8:40 PM, Stephen Cameron <steve.cameron...@gmail.com > wrote:
> That is not the problem then, > > Its a bit bizarre actually, I generated some logging and added some > messages, I see that two objects Participation and Participant do have > their region value set but the Person object doesn't. > > All these objects extend the same base class, but the SQL insert created > for Person doesn't have the value whereas it does for the others. > > The code is as follows: > > @Programmatic > public void setUpdatedBy(String updatedBy) { > if (getCreatedBy() == null) { > setCreatedBy(updatedBy); > System.out.println(">>>updatedBy: " + updatedBy); > if (userRepository != null) { > System.out.println(">>>repositoryExists"); > ApplicationUser user = > userRepository.findByUsername(updatedBy); > if (user != null && user.getTenancy() != null) { > String path = user.getTenancy().getPath(); > String name = path.substring(path.lastIndexOf("/") + > 1); > if (name.isEmpty()) > name = "GLOBAL"; > System.out.println(">>>setting Region for name: " + > name); > setRegion(regions.regionForName(name)); > }else{ > System.out.println(">>>>user tenancy not found"); > setRegion(regions.regionForName("GLOBAL")); //TODO > temp fix > } > } > else{ > System.out.println(">>>userRepository is null"); > setRegion(regions.regionForName("GLOBAL")); //TODO temp fix > } > } else > setLastModifiedBy(updatedBy); > } > > The stdout: > > > 20:20:25,720 [Native qtp209845522-18 DEBUG] SELECT > 'au.com.scds.chats.dom.module.general.Person' AS > NUCLEUS_TYPE,a0.createdby,a0.createdon,a0.lastmodifiedby,a0.lastmodifiedon,a0.birthdate,a0.email,a0.emailaddress,a0.firstname,a0.homephonenumber,a0.middlename,a0.mobilephonenumber,a0.oldid,a0.preferredname,a0.surname,a0.person_id > FROM person a0 WHERE a0.surname = <'Ncks'> > >>>updatedBy: sven > >>>repositoryExists > 20:20:25,732 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@76fe870e > " > 20:20:25,732 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@76fe870e > " > 20:20:25,732 [Native qtp209845522-18 DEBUG] SELECT > 'org.isisaddons.module.security.dom.user.ApplicationUser' AS > NUCLEUS_TYPE,a0.accounttype,a0.emailaddress,a0.encryptedpassword,a0.familyname,a0.faxnumber,a0.givenname,a0.knownas,a0.phonenumber,a0.status,a0.username, > a0.id,a0.version FROM applicationuser a0 WHERE a0.username = <'sven'> > >>>setting Region for name: GLOBAL > 20:20:25,738 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@7dfe372f > " > 20:20:25,738 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@7dfe372f > " > 20:20:25,738 [Native qtp209845522-18 DEBUG] SELECT > 'au.com.scds.chats.dom.module.general.names.Region' AS > NUCLEUS_TYPE,a0.`name` FROM region a0 WHERE a0.`name` = <'GLOBAL'> > 20:20:25,785 [Datastore qtp209845522-18 DEBUG] Using > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@74efbe8f" for > connection > "org.datanucleus.store.rdbms.datasource.dbcp.PoolingDataSource$PoolGuardConnectionWrapper@15e2c48d > " > 20:20:25,785 [Datastore qtp209845522-18 DEBUG] Using > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@74efbe8f" for > connection > "org.datanucleus.store.rdbms.datasource.dbcp.PoolingDataSource$PoolGuardConnectionWrapper@15e2c48d > " > 20:20:25,792 [Native qtp209845522-18 DEBUG] INSERT INTO > person > (birthdate,contacttype_name,email,emailaddress,englishskill_englishskill_id,firstname,homephonenumber,mailaddress_location_id,middlename,mobilephonenumber,oldid,preferredname,region_name,salutation_name,streetaddress_location_id,surname,createdby,createdon,lastmodifiedby,lastmodifiedon) > VALUES > (<1960-10-19>,<null>,<null>,<null>,<null>,<'Stevie'>,<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>,<'Nicks'>,<'sven'>,<2015-11-26 > 20:20:25.74>,<null>,<null>) > 20:20:25,795 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@74efbe8f" > 20:20:25,795 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@74efbe8f" > >>>updatedBy: sven > >>>repositoryExists > 20:20:25,808 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@6822e803 > " > 20:20:25,808 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@6822e803 > " > 20:20:25,809 [Native qtp209845522-18 DEBUG] SELECT > 'org.isisaddons.module.security.dom.user.ApplicationUser' AS > NUCLEUS_TYPE,a0.accounttype,a0.emailaddress,a0.encryptedpassword,a0.familyname,a0.faxnumber,a0.givenname,a0.knownas,a0.phonenumber,a0.status,a0.username, > a0.id,a0.version FROM applicationuser a0 WHERE a0.username = <'sven'> > >>>setting Region for name: GLOBAL > 20:20:25,812 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@6949a458 > " > 20:20:25,812 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@6949a458 > " > 20:20:25,812 [Native qtp209845522-18 DEBUG] SELECT > 'au.com.scds.chats.dom.module.general.names.Region' AS > NUCLEUS_TYPE,a0.`name` FROM region a0 WHERE a0.`name` = <'GLOBAL'> > 20:20:25,815 [Datastore qtp209845522-18 DEBUG] Using > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@7fb858f5" for > connection > "org.datanucleus.store.rdbms.datasource.dbcp.PoolingDataSource$PoolGuardConnectionWrapper@15e2c48d > " > 20:20:25,815 [Datastore qtp209845522-18 DEBUG] Using > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@7fb858f5" for > connection > "org.datanucleus.store.rdbms.datasource.dbcp.PoolingDataSource$PoolGuardConnectionWrapper@15e2c48d > " > 20:20:25,816 [Native qtp209845522-18 DEBUG] INSERT INTO > participant > (lifehistory_lifehistory_id,loneliness_loneliness_id,person_person_id,socialfactors_socialfactors_id,status,createdby,createdon,lastmodifiedby,lastmodifiedon,region_name) > VALUES (<null>,<null>,<8>,<null>,<'ACTIVE'>,<'sven'>,<2015-11-26 > 20:20:25.814>,<null>,<null>,<'GLOBAL'>) > 20:20:25,817 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@7fb858f5" > 20:20:25,817 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@7fb858f5" > 20:20:25,825 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@6eb9aa9a > " > 20:20:25,825 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@6eb9aa9a > " > 20:20:25,826 [Native qtp209845522-18 DEBUG] SELECT > 'au.com.scds.chats.dom.module.participant.Participation' AS > NUCLEUS_TYPE,a0.createdby,a0.createdon,a0.lastmodifiedby,a0.lastmodifiedon,a0.dropofftime,a0.oldid,a0.pickuptime,a0.roleid,a0.transportnotes,a0.participation_id > FROM participation a0 WHERE a0.activity_activity_id = <1> > 20:20:25,831 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@34a88772" > 20:20:25,831 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@34a88772" > 20:20:25,832 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@5578fea9 > " > 20:20:25,832 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@5578fea9 > " > 20:20:25,832 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.approximateenddatetime,b0.copiedfromactivityid,b0.costforparticipant,b0.description,b0.isrestricted,b0.`name`,b0.oldid,b0.scheduleid,b0.startdatetime,b0.activity_id,b0.classifier > FROM participation a0 INNER JOIN activity b0 ON a0.activity_activity_id = > b0.activity_id WHERE a0.participation_id = <3> > 20:20:25,835 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@456274da" > 20:20:25,835 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@456274da" > 20:20:25,836 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@57f25c32 > " > 20:20:25,836 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@57f25c32 > " > 20:20:25,837 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.status,b0.participant_id > FROM participation a0 INNER JOIN participant b0 ON > a0.participant_participant_id = b0.participant_id WHERE a0.participation_id > = <3> > 20:20:25,839 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@1554ed8e" > 20:20:25,839 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@1554ed8e" > 20:20:25,840 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@76978806 > " > 20:20:25,840 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@76978806 > " > 20:20:25,840 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.birthdate,b0.email,b0.emailaddress,b0.firstname,b0.homephonenumber,b0.middlename,b0.mobilephonenumber,b0.oldid,b0.preferredname,b0.surname,b0.person_id > FROM participant a0 INNER JOIN person b0 ON a0.person_person_id = > b0.person_id WHERE a0.participant_id = <3> > 20:20:25,843 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@66f47ad4" > 20:20:25,843 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@66f47ad4" > 20:20:25,844 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@64779748 > " > 20:20:25,844 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@64779748 > " > 20:20:25,844 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.approximateenddatetime,b0.copiedfromactivityid,b0.costforparticipant,b0.description,b0.isrestricted,b0.`name`,b0.oldid,b0.scheduleid,b0.startdatetime,b0.activity_id,b0.classifier > FROM participation a0 INNER JOIN activity b0 ON a0.activity_activity_id = > b0.activity_id WHERE a0.participation_id = <6> > 20:20:25,845 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@e5ed16d" > 20:20:25,845 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@e5ed16d" > 20:20:25,846 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@328b4a67 > " > 20:20:25,846 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@328b4a67 > " > 20:20:25,846 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.status,b0.participant_id > FROM participation a0 INNER JOIN participant b0 ON > a0.participant_participant_id = b0.participant_id WHERE a0.participation_id > = <6> > 20:20:25,848 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@507670dd" > 20:20:25,848 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@507670dd" > 20:20:25,849 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@fec3f19 > " > 20:20:25,849 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@fec3f19 > " > 20:20:25,849 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.birthdate,b0.email,b0.emailaddress,b0.firstname,b0.homephonenumber,b0.middlename,b0.mobilephonenumber,b0.oldid,b0.preferredname,b0.surname,b0.person_id > FROM participant a0 INNER JOIN person b0 ON a0.person_person_id = > b0.person_id WHERE a0.participant_id = <6> > 20:20:25,852 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@46f3839a" > 20:20:25,852 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@46f3839a" > 20:20:25,853 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@7546f02f > " > 20:20:25,853 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@7546f02f > " > 20:20:25,854 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.approximateenddatetime,b0.copiedfromactivityid,b0.costforparticipant,b0.description,b0.isrestricted,b0.`name`,b0.oldid,b0.scheduleid,b0.startdatetime,b0.activity_id,b0.classifier > FROM participation a0 INNER JOIN activity b0 ON a0.activity_activity_id = > b0.activity_id WHERE a0.participation_id = <7> > 20:20:25,855 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@a3e5286" > 20:20:25,855 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@a3e5286" > 20:20:25,856 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@2b9465d6 > " > 20:20:25,856 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@2b9465d6 > " > 20:20:25,856 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.status,b0.participant_id > FROM participation a0 INNER JOIN participant b0 ON > a0.participant_participant_id = b0.participant_id WHERE a0.participation_id > = <7> > 20:20:25,858 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@3c2b0e6c" > 20:20:25,858 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@3c2b0e6c" > 20:20:25,859 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@676002a > " > 20:20:25,859 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@676002a > " > 20:20:25,859 [Native qtp209845522-18 DEBUG] SELECT > b0.createdby,b0.createdon,b0.lastmodifiedby,b0.lastmodifiedon,b0.birthdate,b0.email,b0.emailaddress,b0.firstname,b0.homephonenumber,b0.middlename,b0.mobilephonenumber,b0.oldid,b0.preferredname,b0.surname,b0.person_id > FROM participant a0 INNER JOIN person b0 ON a0.person_person_id = > b0.person_id WHERE a0.participant_id = <7> > 20:20:25,863 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@41ab2f40" > 20:20:25,863 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@41ab2f40" > >>>updatedBy: sven > >>>repositoryExists > 20:20:25,871 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@65dc18d2 > " > 20:20:25,871 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@65dc18d2 > " > 20:20:25,871 [Native qtp209845522-18 DEBUG] SELECT > 'org.isisaddons.module.security.dom.user.ApplicationUser' AS > NUCLEUS_TYPE,a0.accounttype,a0.emailaddress,a0.encryptedpassword,a0.familyname,a0.faxnumber,a0.givenname,a0.knownas,a0.phonenumber,a0.status,a0.username, > a0.id,a0.version FROM applicationuser a0 WHERE a0.username = <'sven'> > >>>setting Region for name: GLOBAL > 20:20:25,875 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@7a2183e6 > " > 20:20:25,875 [Datastore qtp209845522-18 DEBUG] Closing > PreparedStatement > "org.datanucleus.store.rdbms.datasource.dbcp.DelegatingPreparedStatement@7a2183e6 > " > 20:20:25,881 [Native qtp209845522-18 DEBUG] SELECT > 'au.com.scds.chats.dom.module.general.names.Region' AS > NUCLEUS_TYPE,a0.`name` FROM region a0 WHERE a0.`name` = <'GLOBAL'> > 20:20:25,885 [Datastore qtp209845522-18 DEBUG] Using > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@63c44673" for > connection > "org.datanucleus.store.rdbms.datasource.dbcp.PoolingDataSource$PoolGuardConnectionWrapper@15e2c48d > " > 20:20:25,885 [Datastore qtp209845522-18 DEBUG] Using > PreparedStatement > "org.datanucleus.store.rdbms.ParamLoggingPreparedStatement@63c44673" for > connection > "org.datanucleus.store.rdbms.datasource.dbcp.PoolingDataSource$PoolGuardConnectionWrapper@15e2c48d > " > 20:20:25,886 [Native qtp209845522-18 DEBUG] INSERT INTO > participation > (activity_activity_id,arrivingtransporttype_name,departingtransporttype_name,dropofftime,oldid,participant_participant_id,pickuptime,roleid,transportnotes,createdby,createdon,lastmodifiedby,lastmodifiedon,region_name) > VALUES > (<1>,<null>,<null>,<null>,<null>,<8>,<null>,<null>,<null>,<'sven'>,<2015-11-26 > 20:20:25.883>,<null>,<null>,<'GLOBAL'>) > > > On Thu, Nov 26, 2015 at 8:11 PM, Dan Haywood <d...@haywood-associates.co.uk > > wrote: > >> If you just new them up: >> >> Customer cust = new Customer(); >> >> then the framework doesn't know about it and any services will be null. >> >> You can then use: >> >> domainObjectContainer.injectServicesInto(cust); >> >> ~~~ >> >> Alternatively, use: >> >> Customer cust = container.newTransientInstance(Customer.class); >> >> to do both steps in one. >> >> HTH >> Dan >> >> >> On 26 November 2015 at 09:09, Stephen Cameron <steve.cameron...@gmail.com >> > >> wrote: >> >> > Hi, >> > >> > I think I've read somewhere that domain services aren't always injected >> > into entities by default and you sometimes have to trigger this >> yourself? >> > >> > Is this a correct? >> > >> > Thank >> > >> > >> > On Thu, Nov 26, 2015 at 10:00 AM, Stephen Cameron < >> > steve.cameron...@gmail.com> wrote: >> > >> > > I cannot try alternatives till later this evening, will let you know >> > > of results. If I cannot get tenancies working for tomorrow its not the >> > end >> > > of the world. I was getting "record not found" errors last night due >> to >> > > making region allowsNull="false", so a bit of a panic, given its >> > > mostly been working very smoothly now for a while. >> > > >> > > On Thu, Nov 26, 2015 at 9:40 AM, Dan Haywood < >> > d...@haywood-associates.co.uk >> > > > wrote: >> > > >> > >> On 25 November 2015 at 22:26, Stephen Cameron < >> > steve.cameron...@gmail.com >> > >> > >> > >> wrote: >> > >> >> > >> > Hi Dan, >> > >> > >> > >> > I did not explain it that well sorry. >> > >> > >> > >> > I want to get rid of nulls completely, the fact that I do handle >> them >> > is >> > >> > just a work-around for the fact that they are still being created, >> but >> > >> only >> > >> > for People. That is, if a new Participant is registered it looks to >> > see >> > >> if >> > >> > there is an existing Person with the same details then adds a new >> one >> > if >> > >> > not present (a single Person can be a Participant and/or a >> Volunteer). >> > >> The >> > >> > Participant is getting its region set in setUpdatedBy() but the >> Person >> > >> is >> > >> > not, I need to debug this. >> > >> > >> > >> > >> > >> In 1.10.0, a better approach is probably to add a subscriber to the >> > >> lifecycle events: >> > >> >> > >> @DomainService(nature=NatureOfService.DOMAIN) >> > >> public class RegionSubscriber extends AbstractSubscriber { >> > >> >> > >> @Subscribe >> > >> public void on(ObjectPersistingEvent ev) { >> > >> Object domainObject = ev.getSource(); >> > >> } >> > >> >> > >> @Subscribe >> > >> public void on(ObjectUpdatingEvent ev) { >> > >> Object domainObject = ev.getSource(); >> > >> } >> > >> } >> > >> >> > >> >> > >> >> > >> >> > >> > As general interest is handling such things via such a service >> better, >> > >> that >> > >> > is to avoid injecting services into entities if possible? >> > >> > >> > >> > >> > >> There's nothing wrong with injecting services into entities; it >> allows >> > >> them >> > >> to perform their responsibilities without leaking details of how >> they do >> > >> it >> > >> (their implementation) to the caller. I don't hold by this "pass in >> the >> > >> services" and "double dispatch"... I see that as a code smell. >> > >> >> > >> On the other hand, we now have a very rich set of domain events; in >> AOP >> > >> terms these are jointpoints (or is it pointcuts? see [1]), anyway >> they >> > are >> > >> a different way to cause "stuff to happen". >> > >> >> > >> Should one have the entity call a services, or rely on subscribers to >> > >> domain events? With the latter, the entity is very clean and the >> > >> responsibility kept completely separate. On the other hand, with >> events >> > >> the order in which the subscribers fire is non-deterministic. >> > >> >> > >> >> > >> >> > >> >> > >> >> > >> >> > >> > Last minute tweeks aren't the best idea, but the tests that I do >> have >> > >> are >> > >> > proving their value. These particular issues are probably a result >> of >> > no >> > >> > integration tests yet. >> > >> > >> > >> > >> > >> It's not clear to me if my suggestion to use the TenancyEvaluator SPI >> > >> helped ? Or, indeed, whether you have a reasonable workaround in >> your >> > >> current handling of the nulls? >> > >> >> > >> >> > >> >> > >> >> > >> > Thanks >> > >> > >> > >> > >> > >> >> > >> [1] >> > >> >> > >> >> > >> http://stackoverflow.com/questions/15447397/spring-aop-whats-the-difference-between-joinpoint-and-pointcut >> > >> >> > >> >> > >> > On Thu, Nov 26, 2015 at 8:09 AM, Dan Haywood < >> > >> d...@haywood-associates.co.uk >> > >> > > >> > >> > wrote: >> > >> > >> > >> > > Hi Steve, >> > >> > > >> > >> > > Not sure I follow quite all the ins-and-outs of this, but what >> could >> > >> well >> > >> > > be of use is the optional ApplicationTenancyPathEvaluator domain >> > >> > > service[1]. If an implementation is registered (on the >> classpath), >> > >> then >> > >> > > the security module will delegate to this [2] in order to >> determine >> > >> who >> > >> > > gets to see/edit what. >> > >> > > >> > >> > > From what I did follow, it sounds like you need to treat objects >> > with >> > >> a >> > >> > > null region in a particular way? If so, then hopefully you can >> put >> > >> this >> > >> > > "special case" logic in an implementation of said service. >> > >> > > >> > >> > > HTH >> > >> > > Dan >> > >> > > >> > >> > > >> > >> > > >> > >> > > [1] >> > >> > > >> > >> > > >> > >> > >> > >> >> > >> https://github.com/isisaddons/isis-module-security/blob/master/dom/src/main/java/org/isisaddons/module/security/dom/tenancy/ApplicationTenancyPathEvaluator.java >> > >> > > [2] >> > >> > > >> > >> > > >> > >> > >> > >> >> > >> https://github.com/isisaddons/isis-module-security/blob/master/dom/src/main/java/org/isisaddons/module/security/facets/TenantedAuthorizationFacetFactory.java#L72 >> > >> > > >> > >> > > >> > >> > > On 25 November 2015 at 20:25, Stephen Cameron < >> > >> > steve.cameron...@gmail.com> >> > >> > > wrote: >> > >> > > >> > >> > > > Hi, >> > >> > > > >> > >> > > > I have a problem with Application Tenancies not working the >> way I >> > >> > want, I >> > >> > > > want to drive it off a field 'region', such that a person is >> > >> assigned a >> > >> > > > tenancy of GLOBAL, NORTH, SOUTH, NORTH-WEST and these are also >> the >> > >> > > regions >> > >> > > > that they are working in too, so when they create a record >> their >> > >> region >> > >> > > > gets assigned to the new record and that controls who else can >> see >> > >> the >> > >> > > > record. >> > >> > > > >> > >> > > > This mostly works, but there is one small glitch. The main >> > problem I >> > >> > have >> > >> > > > is setting it up so that region is non-null and any records >> > created >> > >> by >> > >> > a >> > >> > > > GLOBAL tenancy user are seen by all users, while those created >> by >> > a >> > >> > > NORTH, >> > >> > > > SOUTH or NORTH-WEST user are seen only by users with the same >> > >> tenancy. >> > >> > By >> > >> > > > changing the region of a record it can change tenancy. >> > >> > > > >> > >> > > > Presently this is broken, or at least it I think it was working >> > >> > correctly >> > >> > > > when I allowed a record with a region of null to be the GLOBAL >> > >> ones. I >> > >> > > need >> > >> > > > to have this working for a demo tomorrow, so advice is >> welcomed on >> > >> how >> > >> > > best >> > >> > > > to set this up properly. >> > >> > > > >> > >> > > > My base class that controls all this here: >> > >> > > > >> > >> > > > >> > >> > > >> > >> > >> > >> >> > >> https://github.com/Stephen-Cameron-Data-Services/isis-chats/blob/master/dom/src/main/java/au/com/scds/chats/dom/AbstractChatsDomainEntity.java >> > >> > > > >> > >> > > > The other small glitch is that a person record created in the >> > >> process >> > >> > of >> > >> > > > creating a participant doesn't get assigned a region >> > automatically. >> > >> I >> > >> > > have >> > >> > > > a work-around for that in place at present, anything that has a >> > null >> > >> > > region >> > >> > > > gets presented as having a global tenancy. >> > >> > > > >> > >> > > > >> > >> > > > Thanks >> > >> > > > >> > >> > > > Steve Cameron >> > >> > > > >> > >> > > >> > >> > >> > >> >> > > >> > > >> > >> > >