How to map table with two references

[mwpowellhtx]

I have a table Vehicle with effectively two references. One to MotorVehicleDivision which must exist.

References(x => x.MotorVehicleDivision)
    .Not.Nullable();

Another to Owner which may or may not exist. In context vehicle may be found in the wild as it were, no owner, so may be null.

References(x => x.Owner)
    .Column(nameof(Vehicle.Owner).ToLower() + ColumnNames.Id)
    .Nullable();

MotorVehicleDivision side of things, the mapping is, AllDeleteOrphans is correct here, I believe, if for any reason an MVD instance goes away, should pull all the Vehicle objects along with it:

HasMany(x => x.Vehicles)
    .Inverse()
    .OrderBy(x => x.VehicleIdentificationNumber)
    .Cascade.AllDeleteOrphan();

Owner side is a bit trickier, if Player goes away for any reason should leave Vehicle intact, so we simply Refresh, I think, Inverse as well, of course.

HasMany(x => x.Vehicles)
    .Inverse()
    .Cascade.Refresh()
    .Table(JoinNames([TableNames.efcore, TableNames.wbe, nameof(Vehicle).ToLower()]));

Having some difficulty saving, somewhere along the way the relationships are being dropped and the transaction is rolling back.

Anyone better versed in this sort of mapping, perhaps, any insights?

Initially I thought perhaps a many-to-many joining table, however, this is really overkill, Vehicle cannot have multiple owners, so the next best shot at it, is this, I think.

Internally we respond with appropriate collections, observability, and so forth, to ensure that the properties are properly affected prior to SaveOrUpdate.

I am getting an error: NHibernate.ObjectDeletedException: 'deleted object would be re-saved by cascade (remove deleted object from associations)[WhalleyBotEnhanced.Player#xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx]'.

Saving is properly contained in a try catch and transacted so I get commits and rollbacks "for free" as part of that callback. I've verified that much of the repository pattern works, just having a bit of difficulty persuading the one and a half parents (MVD the true parent, Owner being 'parent' but may be null).

Any ideas how to better approach this one?

Thanks!

[mwpowellhtx]

Postgres table structure wise, both columns motorVehicleDivisionId and ownerId are appropriately measured as foreign keys, ownerId of course can be NULL.

My guess is something about the Cascade in the mapping is not especially kosher with the use case. Is there a better method of cascading I could be using, or perhaps none at all?

It is possible for a Vehicle in the wild to not have any owner whatsoever, which I think is part partial what may be happening.

HasMany(x => x.Vehicles)
    .Inverse()
    .Cascade.Refresh()
    .Table(JoinNames([TableNames.efcore, TableNames.wbe, nameof(Vehicle).ToLower()]));

[mwpowellhtx]

The database logs do not show any corresponding INSERT behaviors, and is trying to instead SELECT all of what I think are the 100+ vehicles records. So something is obviously breaking down in one of the HasMany() or References(), though I'm not sure exactly what at the moment.

Full thrown exception:

0: OnSave: NHibernate.ObjectDeletedException: deleted object would be re-saved by cascade (remove deleted object from associations)[WhalleyBotEnhanced.Player#xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx]
0: OnSave: NHibernate.ObjectDeletedException:    at NHibernate.Impl.SessionImpl.ForceFlush(EntityEntry entityEntry)
   at NHibernate.Event.Default.DefaultSaveOrUpdateEventListener.EntityIsTransient(SaveOrUpdateEvent event)
   at NHibernate.Event.Default.DefaultSaveOrUpdateEventListener.PerformSaveOrUpdate(SaveOrUpdateEvent event)
   at NHibernate.Event.Default.DefaultSaveOrUpdateEventListener.OnSaveOrUpdate(SaveOrUpdateEvent event)
   at NHibernate.Impl.SessionImpl.FireSaveOrUpdate(SaveOrUpdateEvent event)
   at NHibernate.Impl.SessionImpl.SaveOrUpdate(String entityName, Object obj)
   at NHibernate.Engine.CascadingAction.SaveUpdateCascadingAction.Cascade(IEventSource session, Object child, String entityName, Object anything, Boolean isCascadeDeleteEnabled)
   at NHibernate.Engine.Cascade.CascadeToOne(Object parent, Object child, IType type, CascadeStyle style, Object anything, Boolean isCascadeDeleteEnabled)
   at NHibernate.Engine.Cascade.CascadeAssociation(Object parent, Object child, IType type, CascadeStyle style, Object anything, Boolean isCascadeDeleteEnabled)
   at NHibernate.Engine.Cascade.CascadeProperty(Object parent, Object child, IType type, CascadeStyle style, String propertyName, Object anything, Boolean isCascadeDeleteEnabled)
   at NHibernate.Engine.Cascade.CascadeCollectionElements(Object parent, Object child, CollectionType collectionType, CascadeStyle style, IType elemType, Object anything, Boolean isCascadeDeleteEnabled)
   at NHibernate.Engine.Cascade.CascadeCollection(Object parent, Object child, CascadeStyle style, Object anything, CollectionType type)
   at NHibernate.Engine.Cascade.CascadeAssociation(Object parent, Object child, IType type, CascadeStyle style, Object anything, Boolean isCascadeDeleteEnabled)
   at NHibernate.Engine.Cascade.CascadeProperty(Object parent, Object child, IType type, CascadeStyle style, String propertyName, Object anything, Boolean isCascadeDeleteEnabled)
   at NHibernate.Engine.Cascade.CascadeOn(IEntityPersister persister, Object parent, Object anything)
   at NHibernate.Event.Default.AbstractFlushingEventListener.CascadeOnFlush(IEventSource session, IEntityPersister persister, Object key, Object anything)
   at NHibernate.Event.Default.AbstractFlushingEventListener.PrepareEntityFlushes(IEventSource session)
   at NHibernate.Event.Default.AbstractFlushingEventListener.FlushEverythingToExecutions(FlushEvent event)
   at NHibernate.Event.Default.DefaultFlushEventListener.OnFlush(FlushEvent event)
   at NHibernate.Impl.SessionImpl.Flush()
   at NHibernate.Impl.SessionImpl.FlushBeforeTransactionCompletion()
   at NHibernate.Impl.SessionImpl.BeforeTransactionCompletion(ITransaction tx)
   at NHibernate.Transaction.AdoTransaction.Commit()
   at WhalleyBotEnhanced.Repositories.Repository.Transact(Action`2 action) in L:\Source\SCUM\WhalleyBotEnhanced\wbenhanced-proto\src\WhalleyBotEnhanced.Data.Persistence\Repositories\Repository.cs:line 48
   at WhalleyBotEnhanced.Repositories.Repository.SaveOrUpdate(IReadOnlyCollection`1 entities, Boolean transact) in L:\Source\SCUM\WhalleyBotEnhanced\wbenhanced-proto\src\WhalleyBotEnhanced.Data.Persistence\Repositories\Repository.cs:line 192
   at WhalleyBotEnhanced.Repositories.Repository.SaveOrUpdate(Object entity, Boolean transact) in L:\Source\SCUM\WhalleyBotEnhanced\wbenhanced-proto\src\WhalleyBotEnhanced.Data.Persistence\Repositories\Repository.cs:line 79
   at WhalleyBotEnhanced.Models.WhalleySessionViewModel.OnSave() in L:\Source\SCUM\WhalleyBotEnhanced\wbenhanced-proto\src\WhalleyBotEnhanced.Wpf\Models\WhalleySessionViewModel.cs:line 386

[mwpowellhtx]

Well here's a hair brained crazy idea... And I've got something similar already in Squad entities, I introduce effectively a Zero or Null Squad use case, when Player is effectively solo, for data modeling purposes. So instead of perhaps allowing Owner to be null, could default to a Null or Zero Player, perhaps, indicating non-Owner owner. Assuming of course this is the issue fouling up the works.

[mwpowellhtx]

I am starting to wonder if perhaps in the case of an optional null or instance HasOne is more appropriate. Whereas MotorVehicleDivision really does need to be the Reference. How would that work though... HasOne (nullable?) from the Vehicle side, Owner (Player) side, HasMany, still?