Yes, I still had a variable region declared in Person that was over-riding
the one in the base class I guess, so the setter was setting the super
class but the subclass value was being persisted.
On Thu, Nov 26, 2015 at 8:45 PM, Stephen Cameron <steve.cameron...@gmail.com
> wrote:
> 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
>>> > >> > > >
>>> > >> > >
>>> > >> >
>>> > >>
>>> > >
>>> > >
>>> >
>>>
>>
>>
>
