Crash on cell deletion/insertion in sectioned list #76

New Issue

Closed

opened 2025-12-29 15:23:45 +01:00 by adam · 6 comments

adam commented 2025-12-29 15:23:45 +01:00

Owner

Copy Link

Originally created by @avshiyanov on GitHub (Sep 8, 2016).

I have many sections on the list, for example 5, first one have only one element in section, when I do transation.delete(obj) App crashes. The same occurs if section does not exist and I do insert
Crash
CoreData: error: Serious application error. An exception was caught from the delegate of NSFetchedResultsController during a call to -controllerDidChangeContent:. Invalid update: invalid number of sections. The number of sections contained in the table view after the update (1) must be equal to the number of sections contained in the table view before the update (2), plus or minus the number of sections inserted or deleted (0 inserted, 0 deleted). with userInfo (null)

Here is deletion logic:
CoreStore.beginAsynchronous { (transaction) -> Void in
transaction.delete(task)
transaction.commit()
}

In data source I have:
// MARK: ListObserver

func listMonitorWillChange(monitor: ListMonitor<ListEntityType>) {
    self.tableView.beginUpdates()
}

func listMonitorDidChange(monitor: ListMonitor<ListEntityType>) {
    self.tableView.endUpdates()
}

func listMonitor(monitor: ListMonitor<ListEntityType>,
                 didInsertObject object: ListEntityType, toIndexPath indexPath: NSIndexPath) {
    print (indexPath)
    print (listMonitor.numberOfSections())
    self.tableView.insertRowsAtIndexPaths([indexPath], withRowAnimation: .Automatic)
}

func listMonitor(monitor: ListMonitor<ListEntityType>,
                 didDeleteObject object: ListEntityType, fromIndexPath indexPath: NSIndexPath) {
    print (indexPath)
    print (listMonitor.numberOfSections())
    self.tableView.deleteRowsAtIndexPaths([indexPath], withRowAnimation: .Automatic)
}

func listMonitor(monitor: ListMonitor<ListEntityType>,
                 didUpdateObject object: ListEntityType, atIndexPath indexPath: NSIndexPath) {
    tableView.reloadRowsAtIndexPaths([indexPath], withRowAnimation: .None)
}

func listMonitor(monitor: ListMonitor<ListEntityType>,
                 didMoveObject object: ListEntityType,
                               fromIndexPath: NSIndexPath, toIndexPath: NSIndexPath) {
    tableView.moveRowAtIndexPath(fromIndexPath, toIndexPath: toIndexPath)
}

Originally created by @avshiyanov on GitHub (Sep 8, 2016). I have many sections on the list, for example 5, first one have only one element in section, when I do transation.delete(obj) App crashes. The same occurs if section does not exist and I do insert Crash CoreData: error: Serious application error. An exception was caught from the delegate of NSFetchedResultsController during a call to -controllerDidChangeContent:. Invalid update: invalid number of sections. The number of sections contained in the table view after the update (1) must be equal to the number of sections contained in the table view before the update (2), plus or minus the number of sections inserted or deleted (0 inserted, 0 deleted). with userInfo (null) Here is deletion logic: CoreStore.beginAsynchronous { (transaction) -> Void in transaction.delete(task) transaction.commit() } In data source I have: // MARK: ListObserver ``` func listMonitorWillChange(monitor: ListMonitor<ListEntityType>) { self.tableView.beginUpdates() } func listMonitorDidChange(monitor: ListMonitor<ListEntityType>) { self.tableView.endUpdates() } func listMonitor(monitor: ListMonitor<ListEntityType>, didInsertObject object: ListEntityType, toIndexPath indexPath: NSIndexPath) { print (indexPath) print (listMonitor.numberOfSections()) self.tableView.insertRowsAtIndexPaths([indexPath], withRowAnimation: .Automatic) } func listMonitor(monitor: ListMonitor<ListEntityType>, didDeleteObject object: ListEntityType, fromIndexPath indexPath: NSIndexPath) { print (indexPath) print (listMonitor.numberOfSections()) self.tableView.deleteRowsAtIndexPaths([indexPath], withRowAnimation: .Automatic) } func listMonitor(monitor: ListMonitor<ListEntityType>, didUpdateObject object: ListEntityType, atIndexPath indexPath: NSIndexPath) { tableView.reloadRowsAtIndexPaths([indexPath], withRowAnimation: .None) } func listMonitor(monitor: ListMonitor<ListEntityType>, didMoveObject object: ListEntityType, fromIndexPath: NSIndexPath, toIndexPath: NSIndexPath) { tableView.moveRowAtIndexPath(fromIndexPath, toIndexPath: toIndexPath) } ```

adam closed this issue 2025-12-29 15:23:45 +01:00

adam commented 2025-12-29 15:23:46 +01:00

Author

Owner

Copy Link

@avshiyanov commented on GitHub (Sep 8, 2016):

CoreData: error: Serious application error. An exception was caught from the delegate of NSFetchedResultsController during a call to -controllerDidChangeContent:. Invalid update: invalid number of sections. The number of sections contained in the table view after the update (1) must be equal to the number of sections contained in the table view before the update (2), plus or minus the number of sections inserted or deleted (0 inserted, 0 deleted). with userInfo (null)

@avshiyanov commented on GitHub (Sep 8, 2016): CoreData: error: Serious application error. An exception was caught from the delegate of NSFetchedResultsController during a call to -controllerDidChangeContent:. Invalid update: invalid number of sections. The number of sections contained in the table view after the update (1) must be equal to the number of sections contained in the table view before the update (2), plus or minus the number of sections inserted or deleted (0 inserted, 0 deleted). with userInfo (null)

adam commented 2025-12-29 15:23:46 +01:00

Author

Owner

Copy Link

@JohnEstropia commented on GitHub (Sep 8, 2016):

@avshiyanov I'm guessing this is because of moveRowAtIndexPath and reloadRowsAtIndexPaths, which do not really work well with NSFetchResultsController's event logic.

You can check that UITableViews work well with the CoreStoreDemo app. You'll see the main difference here:

    func listMonitor(_ monitor: ListMonitor<Palette>, didUpdateObject object: Palette, atIndexPath indexPath: IndexPath) {

        if let cell = self.tableView.cellForRow(at: indexPath) as? PaletteTableViewCell {

            let palette = Static.palettes[indexPath]
            cell.colorView?.backgroundColor = palette.color
            cell.label?.text = palette.colorText
        }
    }

    func listMonitor(_ monitor: ListMonitor<Palette>, didMoveObject object: Palette, fromIndexPath: IndexPath, toIndexPath: IndexPath) {

        self.tableView.deleteRows(at: [fromIndexPath], with: .automatic)
        self.tableView.insertRows(at: [toIndexPath], with: .automatic)
    }

For updates, query the cell instance from the tableView and update it directly. For moves, delete the row first then re-insert.

@JohnEstropia commented on GitHub (Sep 8, 2016): @avshiyanov I'm guessing this is because of `moveRowAtIndexPath` and `reloadRowsAtIndexPaths`, which do not really work well with `NSFetchResultsController`'s event logic. You can check that UITableViews work well with the CoreStoreDemo app. You'll see the main difference here: ``` swift func listMonitor(_ monitor: ListMonitor<Palette>, didUpdateObject object: Palette, atIndexPath indexPath: IndexPath) { if let cell = self.tableView.cellForRow(at: indexPath) as? PaletteTableViewCell { let palette = Static.palettes[indexPath] cell.colorView?.backgroundColor = palette.color cell.label?.text = palette.colorText } } func listMonitor(_ monitor: ListMonitor<Palette>, didMoveObject object: Palette, fromIndexPath: IndexPath, toIndexPath: IndexPath) { self.tableView.deleteRows(at: [fromIndexPath], with: .automatic) self.tableView.insertRows(at: [toIndexPath], with: .automatic) } ``` For updates, query the cell instance from the tableView and update it directly. For moves, delete the row first then re-insert.

adam commented 2025-12-29 15:23:46 +01:00

Author

Owner

Copy Link

@avshiyanov commented on GitHub (Sep 8, 2016):

@JohnEstropia When I comment moveRowAtIndexPath and reloadRowsAtIndexPaths I have the same issue, problem still persists on insertion/deletion for Sections only

@avshiyanov commented on GitHub (Sep 8, 2016): @JohnEstropia When I comment moveRowAtIndexPath and reloadRowsAtIndexPaths I have the same issue, problem still persists on insertion/deletion for Sections only

adam commented 2025-12-29 15:23:47 +01:00

Author

Owner

Copy Link

@avshiyanov commented on GitHub (Sep 8, 2016):

monitorSectionedList: func buildListMonitorByTime(withPredicate predicate: NSPredicate?) ->
ListMonitor {
return CoreStore.monitorSectionedList(
From(TaskRecord),
SectionBy("dateName"),
OrderBy(.Descending("createdAt")),
Tweak { (fetchRequest) -> Void in
let sortDescriptor = NSSortDescriptor(key: "createdAt", ascending: false)
fetchRequest.sortDescriptors = [sortDescriptor]
})
}

@avshiyanov commented on GitHub (Sep 8, 2016): monitorSectionedList: func buildListMonitorByTime(withPredicate predicate: NSPredicate?) -> ListMonitor<NSManagedObject> { return CoreStore.monitorSectionedList( From(TaskRecord), SectionBy("dateName"), OrderBy(.Descending("createdAt")), Tweak { (fetchRequest) -> Void in let sortDescriptor = NSSortDescriptor(key: "createdAt", ascending: false) fetchRequest.sortDescriptors = [sortDescriptor] }) }

adam commented 2025-12-29 15:23:47 +01:00

Author

Owner

Copy Link

@avshiyanov commented on GitHub (Sep 8, 2016):

@JohnEstropia Found mistake on my side, forgot implement ListSectionObserver, without it didInsertSection, didDeleteSection does not call for me. Closed

@avshiyanov commented on GitHub (Sep 8, 2016): @JohnEstropia Found mistake on my side, forgot implement ListSectionObserver, without it didInsertSection, didDeleteSection does not call for me. Closed

adam commented 2025-12-29 15:23:47 +01:00

Author

Owner

Copy Link

@JohnEstropia commented on GitHub (Sep 8, 2016):

Glad you worked it out :)

@JohnEstropia commented on GitHub (Sep 8, 2016): Glad you worked it out :)

Sign in to join this conversation.

No Branch/Tag Specified

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

9.3.0

9.2.0

9.1.0

9.0.0

8.1.0

8.0.1

8.0.0

7.3.1

7.3.0

7.2.0

7.1.0

7.0.4

7.0.3

7.0.2

7.0.1

7.0.0

6.3.2

6.3.1

6.3.0

6.2.1

6.2.0

6.1.0

6.0.0

5.3.1

5.3.0

5.2.0

5.1.1

5.1.0

5.0.0

4.2.4

4.2.3

4.2.2

4.2.1

4.2.0

4.1.4

4.1.3

4.1.1

4.1.0

4.0.5

4.0.4

4.0.3

4.0.2

4.0.1

4.0.0

4.0.0-beta3

4.0.0-beta2

4.0.0-beta

3.1.1

3.1.0

3.0.3

3.0.2

3.0.1

3.0.0

2.1.3

2.1.2

2.1.1

2.1.0

2.0.6

2.0.5

2.0.4

2.0.3

2.0.2

2.0.1

2.0.0

1.6.10

1.6.9

1.6.8

1.6.7

1.6.6

1.6.5

1.6.4

1.6.3

1.6.2

1.6.1

1.6.0

1.5.4

1.5.3

1.5.2

1.5.1

1.5.0

1.4.4

1.4.3

1.4.2

1.4.1

1.4.0

1.3.6

1.3.5

1.3.4

1.3.3

1.3.2

1.3.1

1.3.0

1.2.1

1.2.0

1.0.2

1.0.1

1.0.0

0.2.1

0.2.0

0.1.2

0.1.1

0.1.0

Labels

Clear labels

abandoned

corestore bug

discussion

duplicate

enhancement

fixed

fixed

help wanted

in progress

investigating

ios/compiler bug

irreproducible

looking for workarounds

not a bug

pending

pending docs update

pull-request

Mirrored from GitHub Pull Request

question

regression

user bug

wontfix

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

adam

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/CoreStore#76