Fixed misleading variable name

This commit is contained in:
Maurice Parker 2020-10-31 17:31:48 -05:00
parent cbb481c3f7
commit 5fdbd4b9d0

View File

@ -631,63 +631,63 @@ private extension SidebarOutlineDataSource {
return false return false
} }
func violatesAccountSpecificBehavior(_ parentNode: Node, _ draggedFeed: PasteboardWebFeed) -> Bool { func violatesAccountSpecificBehavior(_ dropTargetNode: Node, _ draggedFeed: PasteboardWebFeed) -> Bool {
return violatesAccountSpecificBehavior(parentNode, Set([draggedFeed])) return violatesAccountSpecificBehavior(dropTargetNode, Set([draggedFeed]))
} }
func violatesAccountSpecificBehavior(_ parentNode: Node, _ draggedFeeds: Set<PasteboardWebFeed>) -> Bool { func violatesAccountSpecificBehavior(_ dropTargetNode: Node, _ draggedFeeds: Set<PasteboardWebFeed>) -> Bool {
if violatesDisallowFeedInRootFolder(parentNode) { if violatesDisallowFeedInRootFolder(dropTargetNode) {
return true return true
} }
if violatesDisallowFeedCopyInRootFolder(parentNode, draggedFeeds) { if violatesDisallowFeedCopyInRootFolder(dropTargetNode, draggedFeeds) {
return true return true
} }
if violatesDisallowFeedInMultipleFolders(parentNode, draggedFeeds) { if violatesDisallowFeedInMultipleFolders(dropTargetNode, draggedFeeds) {
return true return true
} }
return false return false
} }
func violatesDisallowFeedInRootFolder(_ parentNode: Node) -> Bool { func violatesDisallowFeedInRootFolder(_ dropTargetNode: Node) -> Bool {
guard let parentAccount = nodeAccount(parentNode), parentAccount.behaviors.contains(.disallowFeedInRootFolder) else { guard let parentAccount = nodeAccount(dropTargetNode), parentAccount.behaviors.contains(.disallowFeedInRootFolder) else {
return false return false
} }
if parentNode.representedObject is Account { if dropTargetNode.representedObject is Account {
return true return true
} }
return false return false
} }
func violatesDisallowFeedCopyInRootFolder(_ parentNode: Node, _ draggedFeeds: Set<PasteboardWebFeed>) -> Bool { func violatesDisallowFeedCopyInRootFolder(_ dropTargetNode: Node, _ draggedFeeds: Set<PasteboardWebFeed>) -> Bool {
guard let parentAccount = nodeAccount(parentNode), parentAccount.behaviors.contains(.disallowFeedCopyInRootFolder) else { guard let dropTargetAccount = nodeAccount(dropTargetNode), dropTargetAccount.behaviors.contains(.disallowFeedCopyInRootFolder) else {
return false return false
} }
for draggedFeed in draggedFeeds { for draggedFeed in draggedFeeds {
if parentAccount.accountID != draggedFeed.accountID { if dropTargetAccount.accountID != draggedFeed.accountID {
return false return false
} }
} }
if parentNode.representedObject is Account && (NSApplication.shared.currentEvent?.modifierFlags.contains(.option) ?? false) { if dropTargetNode.representedObject is Account && (NSApplication.shared.currentEvent?.modifierFlags.contains(.option) ?? false) {
return true return true
} }
return false return false
} }
func violatesDisallowFeedInMultipleFolders(_ parentNode: Node, _ draggedFeeds: Set<PasteboardWebFeed>) -> Bool { func violatesDisallowFeedInMultipleFolders(_ dropTargetNode: Node, _ draggedFeeds: Set<PasteboardWebFeed>) -> Bool {
guard let parentAccount = nodeAccount(parentNode), parentAccount.behaviors.contains(.disallowFeedInMultipleFolders) else { guard let dropTargetAccount = nodeAccount(dropTargetNode), dropTargetAccount.behaviors.contains(.disallowFeedInMultipleFolders) else {
return false return false
} }
for draggedFeed in draggedFeeds { for draggedFeed in draggedFeeds {
if parentAccount.hasWebFeed(withURL: draggedFeed.url) { if dropTargetAccount.hasWebFeed(withURL: draggedFeed.url) {
return true return true
} }
} }