Updating entity with relationships does not update parent context #10

New Issue

adam · 2025-12-29T15:22:01+01:00

adam commented

2025-12-29 15:22:01 +01:00

Originally created by @andreaperizzato on GitHub (Oct 1, 2015).

Hi,
I have two entities related by a one-to-one relationships (let's call them A and B) and a ListObserver observing the list of As. Both entities conform to ImportableUniqueObject and the relationship is set on A as follows:

func updateFromImportSource(source: [String : AnyObject], inTransaction transaction: BaseDataTransaction) throws {
        self.name = source["name"] as! String
        if let otherSource = source["other"] as? [String : String] {
            self.other = try transaction.importUniqueObject(Into(OtherUniqueEntity), source: otherSource)!
        }
    }

Then I have an async transaction updating the list of As.
When I commit the transaction, I noticed that only the temporary context is saved, while the parent context is not because its hasChanges is false. This causes the changes not to be merged in the main context has the list observer not to be updated.
Removing the relationships from the model solves the problem.

Originally created by @andreaperizzato on GitHub (Oct 1, 2015). Hi, I have two entities related by a one-to-one relationships (let's call them `A` and `B`) and a `ListObserver` observing the list of `A`s. Both entities conform to `ImportableUniqueObject` and the relationship is set on `A` as follows: ``` swift func updateFromImportSource(source: [String : AnyObject], inTransaction transaction: BaseDataTransaction) throws { self.name = source["name"] as! String if let otherSource = source["other"] as? [String : String] { self.other = try transaction.importUniqueObject(Into(OtherUniqueEntity), source: otherSource)! } } ``` Then I have an async transaction updating the list of `A`s. When I commit the transaction, I noticed that only the temporary context is saved, while the parent context is not because its `hasChanges` is `false`. This causes the changes not to be merged in the main context has the list observer not to be updated. Removing the relationships from the model solves the problem.

adam added the wontfix label 2025-12-29 15:22:01 +01:00

adam closed this issue

2025-12-29 15:22:01 +01:00

adam commented

2025-12-29 15:22:02 +01:00

@JohnEstropia commented on GitHub (Oct 1, 2015):

@andreaperizzato
Is the inverse relationship properly set for other? If so, I don't see why it wouldn't work (we have similar setups in our own project), so there must be something we're missing.

Are you sure that self.other is being set to a new instance of OtherUniqueEntity? If the previous value of self.other is the same instance returned by transaction.importUniqueObject(), your ListMonitor will not receive an update notification (as you have seen where hasChanges=false).

@JohnEstropia commented on GitHub (Oct 1, 2015): @andreaperizzato Is the inverse relationship properly set for `other`? If so, I don't see why it wouldn't work (we have similar setups in our own project), so there must be something we're missing. Are you sure that `self.other` is being set to a new instance of `OtherUniqueEntity`? If the previous value of `self.other` is the same instance returned by `transaction.importUniqueObject()`, your `ListMonitor` will not receive an update notification (as you have seen where `hasChanges=false`).

adam commented

2025-12-29 15:22:02 +01:00

@JohnEstropia commented on GitHub (Oct 3, 2015):

@andreaperizzato Did you resolve this problem?

@JohnEstropia commented on GitHub (Oct 3, 2015): @andreaperizzato Did you resolve this problem?

adam referenced this issue

2025-12-29 15:32:11 +01:00

[PR #10] [MERGED] Fix building with Carthage #447

Sign in to join this conversation.

Branches Tags

develop

master

prototype/concurrency

minIOS13

prototype/containers

transactionSourceTracking

prototype/swiftui

minIOS11

swift5_3

prototype/propertyWrapperFields

prototype/propertyWrappers

prototype/propertyWrapper

Swift5

throwables

prototype/Swift_4_1

prototype/Swift_4_0

prototype/Swift_3_2

prototype/queryBuilders

temp/develop

corestore4_develop

prototype/mainContextToPSC

master_ios_7_to_9

swift_2_3

gh-pages

_corespotlight-test

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/CoreStore#10