FreshRSS
/
NetNewsWire
mirror of https://github.com/Ranchero-Software/NetNewsWire.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Rebuild the tree controller when deleting/undoing-deleting from the sidebar. Fix #292.

This commit is contained in:
Brent Simmons 2018-02-25 22:02:55 -08:00
parent fc38a485d6
commit 0debd033de
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Commands/DeleteFromSidebarCommand.swift
View File

@ -14,6 +14,7 @@ import Data
final class DeleteFromSidebarCommand: UndoableCommand {
let treeController: TreeController
let undoManager: UndoManager
let undoActionName: String
var redoActionName: String {
@ -22,7 +23,7 @@ final class DeleteFromSidebarCommand: UndoableCommand {
private let itemSpecifiers: [SidebarItemSpecifier]
init?(nodesToDelete: [Node], undoManager: UndoManager) {
init?(nodesToDelete: [Node], treeController: TreeController, undoManager: UndoManager) {
guard DeleteFromSidebarCommand.canDelete(nodesToDelete) else {
return nil
@ -31,6 +32,7 @@ final class DeleteFromSidebarCommand: UndoableCommand {
return nil
}
self.treeController = treeController
self.undoActionName = actionName
self.undoManager = undoManager
@ -45,6 +47,7 @@ final class DeleteFromSidebarCommand: UndoableCommand {
BatchUpdate.shared.perform {
itemSpecifiers.forEach { $0.delete() }
treeController.rebuild()
}
registerUndo()
}
@ -53,6 +56,7 @@ final class DeleteFromSidebarCommand: UndoableCommand {
BatchUpdate.shared.perform {
itemSpecifiers.forEach { $0.restore() }
treeController.rebuild()
}
registerRedo()
}

2
Evergreen/MainWindow/Sidebar/SidebarViewController.swift
View File

@ -129,7 +129,7 @@ import RSCore
let nodesToDelete = treeController.normalizedSelectedNodes(selectedNodes)
guard let undoManager = undoManager, let deleteCommand = DeleteFromSidebarCommand(nodesToDelete: nodesToDelete, undoManager: undoManager) else {
guard let undoManager = undoManager, let deleteCommand = DeleteFromSidebarCommand(nodesToDelete: nodesToDelete, treeController: treeController, undoManager: undoManager) else {
return
}