Bump project to 2.0.5

This issue was being caused by the fact that the overridden
-setDelegate: method on PXSourceListDelegateDataSourceProxy doesn't
correctly remove the old delegate as observing the PXSourceList
notifications.

Setting the same delegate twice will cause this issue to
occur.

Examples/ViewBasedSourceList/AppDelegate.m

@@ -116,7 +116,10 @@ static NSString * const draggingType = @"SourceListExampleDraggingType";
     NSImage *albumImage = [NSImage imageNamed:@"album"];
     [albumImage setTemplate:YES];
 
-    PXSourceListItem *newItem = [PXSourceListItem itemWithTitle:@"New Album" identifier:nil icon:albumImage];
+    PhotoCollection *collection = [PhotoCollection collectionWithTitle:@"New Album" identifier:nil type:PhotoCollectionTypeUserCreated];
+    [self.modelObjects addObject:collection];
+
+    PXSourceListItem *newItem = [PXSourceListItem itemWithRepresentedObject:collection icon:albumImage];
     [self.albumsItem addChildItem:newItem];
 
     NSUInteger childIndex = [[self.albumsItem children] indexOfObject:newItem];
@@ -132,11 +135,12 @@ static NSString * const draggingType = @"SourceListExampleDraggingType";
     PXSourceListItem *selectedItem = [self.sourceList itemAtRow:self.sourceList.selectedRow];
     PXSourceListItem *parentItem = self.albumsItem;
 
-
     [self.sourceList removeItemsAtIndexes:[NSIndexSet indexSetWithIndex:[parentItem.children indexOfObject:selectedItem]]
                                  inParent:parentItem
                             withAnimation:NSTableViewAnimationSlideUp];
 
+    [self.modelObjects removeObject:selectedItem.representedObject];
+
     // Only 'album' items can be deleted.
     [parentItem removeChildItem:selectedItem];
 }

PXSourceList-Info.plist

@@ -17,11 +17,11 @@
 	<key>CFBundlePackageType</key>
 	<string>FMWK</string>
 	<key>CFBundleShortVersionString</key>
-	<string>2.0.0</string>
+	<string>2.0.5</string>
 	<key>CFBundleSignature</key>
 	<string>????</string>
 	<key>CFBundleVersion</key>
-	<string>2.0.0</string>
+	<string>2.0.5</string>
 	<key>NSHumanReadableCopyright</key>
 	<string>Copyright © 2009-14 Alex Rozanski and other contributors. All rights reserved.</string>
 	<key>NSPrincipalClass</key>

PXSourceList.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name         = 'PXSourceList'
-  s.version      = '2.0.0'
+  s.version      = '2.0.5'
   s.author       = { 'Alex Rozanski' => 'alex@rozanski.me' }
   s.license      = 'BSD'
   s.homepage     = 'https://github.com/Perspx/PXSourceList'
@@ -23,6 +23,6 @@ Pod::Spec.new do |s|
   s.osx.deployment_target = '10.7'
 
   s.public_header_files = 'PXSourceList/*.h'
-  s.source       = { :git => 'https://github.com/Perspx/PXSourceList.git', :tag => '2.0.0' }
+  s.source       = { :git => 'https://github.com/Perspx/PXSourceList.git', :tag => '2.0.5' }
   s.source_files = 'PXSourceList/**/*.{h,m}'
 end

PXSourceList/Internal/PXSourceListDelegateDataSourceProxy.h

@@ -13,8 +13,8 @@
 @interface PXSourceListDelegateDataSourceProxy : NSProxy <NSOutlineViewDelegate, NSOutlineViewDataSource, PXSourceListDelegate, PXSourceListDataSource>
 
 @property (weak, nonatomic) PXSourceList *sourceList;
-@property (weak, nonatomic) id <PXSourceListDelegate> delegate;
-@property (weak, nonatomic) id <PXSourceListDataSource> dataSource;
+@property (unsafe_unretained, nonatomic) id <PXSourceListDelegate> delegate;
+@property (unsafe_unretained, nonatomic) id <PXSourceListDataSource> dataSource;
 
 - (id)initWithSourceList:(PXSourceList *)sourceList;
 

PXSourceList/Internal/PXSourceListDelegateDataSourceProxy.m

@@ -36,7 +36,7 @@ static NSArray * __fastPathForwardingDataSourceMethods = nil;
 {
     __outlineViewDelegateMethods = px_methodNamesForProtocol(@protocol(NSOutlineViewDelegate));
     __outlineViewDataSourceMethods = px_methodNamesForProtocol(@protocol(NSOutlineViewDataSource));
-    __fastPathForwardingDelegateMethods = px_methodNamesForProtocol(@protocol(PXSourceListDelegate));
+    __fastPathForwardingDelegateMethods = [self fastPathForwardingDelegateMethods];
     __fastPathForwardingDataSourceMethods = px_methodNamesForProtocol(@protocol(PXSourceListDataSource));
 
     __requiredOutlineViewDataSourceMethods = @[NSStringFromSelector(@selector(outlineView:numberOfChildrenOfItem:)),
@@ -70,7 +70,7 @@ static NSArray * __fastPathForwardingDataSourceMethods = nil;
 - (void)setDelegate:(id<PXSourceListDelegate>)delegate
 {
     if (self.delegate)
-        [[NSNotificationCenter defaultCenter] removeObserver:self.delegate name:nil object:self];
+        [[NSNotificationCenter defaultCenter] removeObserver:self.delegate name:nil object:self.sourceList];
 
     _delegate = delegate;
 
@@ -100,11 +100,12 @@ static NSArray * __fastPathForwardingDataSourceMethods = nil;
 
 - (BOOL)respondsToSelector:(SEL)aSelector
 {
-    if ([self.sourceList respondsToSelector:aSelector])
-        return YES;
-
     NSString *methodName = NSStringFromSelector(aSelector);
 
+    // Only let the source list override NSOutlineView delegate and data source methods.
+    if ([self.sourceList respondsToSelector:aSelector] && ([__outlineViewDataSourceMethods containsObject:methodName] || [__outlineViewDelegateMethods containsObject:methodName]))
+        return YES;
+
     if ([__requiredOutlineViewDataSourceMethods containsObject:methodName])
         return YES;
 
@@ -127,17 +128,14 @@ static NSArray * __fastPathForwardingDataSourceMethods = nil;
     return class_conformsToProtocol(object_getClass(self), protocol);
 }
 
+// Fast-path delegate and data source methods aren't handled here; they are taken care of in -forwardingTargetForSelector:.
 - (NSMethodSignature *)methodSignatureForSelector:(SEL)aSelector
 {
     NSString *methodName = NSStringFromSelector(aSelector);
 
     struct objc_method_description description = {NULL, NULL};
 
-    if ([__fastPathForwardingDelegateMethods containsObject:methodName])
-        description = px_methodDescriptionForProtocolMethod(@protocol(PXSourceListDelegate), aSelector);
-    else if ([__fastPathForwardingDataSourceMethods containsObject:methodName])
-        description = px_methodDescriptionForProtocolMethod(@protocol(PXSourceListDataSource), aSelector);
-    else if ([__outlineViewDelegateMethods containsObject:methodName])
+    if ([__outlineViewDelegateMethods containsObject:methodName])
         description = px_methodDescriptionForProtocolMethod(@protocol(NSOutlineViewDelegate), aSelector);
     else if ([__outlineViewDataSourceMethods containsObject:methodName])
         description = px_methodDescriptionForProtocolMethod(@protocol(NSOutlineViewDataSource), aSelector);
@@ -397,6 +395,18 @@ static NSArray * __fastPathForwardingDataSourceMethods = nil;
     return YES;
 }
 
++ (NSArray *)fastPathForwardingDelegateMethods
+{
+    NSMutableArray *methods = [px_methodNamesForProtocol(@protocol(PXSourceListDelegate)) mutableCopy];
+
+    // Add the NSControl delegate methods manually (unfortunately these aren't part of a formal protocol).
+    [methods addObject:px_methodNameForSelector(@selector(controlTextDidEndEditing:))];
+    [methods addObject:px_methodNameForSelector(@selector(controlTextDidBeginEditing:))];
+    [methods addObject:px_methodNameForSelector(@selector(controlTextDidChange:))];
+
+    return methods;
+}
+
 #pragma mark - Notifications
 
 - (void)registerDelegateToReceiveNotification:(NSString*)notification withSelector:(SEL)selector

PXSourceList/Internal/PXSourceListRuntimeAdditions.h

@@ -16,5 +16,6 @@ extern NSString * const px_protocolIsRequiredMethodKey;
 
 NSArray *px_allProtocolMethods(Protocol *protocol);
 NSArray *px_methodNamesForProtocol(Protocol *protocol);
+id px_methodNameForSelector(SEL selector);
 
 struct objc_method_description px_methodDescriptionForProtocolMethod(Protocol *protocol, SEL selector);

PXSourceList/Internal/PXSourceListRuntimeAdditions.m

@@ -26,7 +26,7 @@ NSArray *px_allProtocolMethods(Protocol *protocol)
 
         for (unsigned int j = 0; j < numberOfMethodDescriptions; ++j) {
             struct objc_method_description methodDescription = methodDescriptions[j];
-            [methodList addObject:@{px_protocolMethodNameKey: NSStringFromSelector(methodDescription.name),
+            [methodList addObject:@{px_protocolMethodNameKey: px_methodNameForSelector(methodDescription.name),
                                     px_protocolMethodArgumentTypesKey: [NSString stringWithUTF8String:methodDescription.types],
                                     px_protocolIsRequiredMethodKey: @(isRequiredMethod)}];
         }
@@ -47,6 +47,11 @@ NSArray *px_methodNamesForProtocol(Protocol *protocol)
     return methodNames;
 }
 
+id px_methodNameForSelector(SEL selector)
+{
+    return NSStringFromSelector(selector);
+}
+
 struct objc_method_description px_methodDescriptionForProtocolMethod(Protocol *protocol, SEL selector)
 {
     struct objc_method_description description = {NULL, NULL};

PXSourceList/PXSourceList.m

@@ -15,7 +15,6 @@
 //Layout constants
 static const CGFloat minBadgeWidth = 22.0;              // The minimum badge width for each item (default 22.0).
 static const CGFloat badgeHeight = 14.0;                // The badge height for each item (default 14.0).
-static const CGFloat badgeMargin = 5.0;                 // The spacing between the badge and the cell for that row.
 static const CGFloat rowRightMargin = 5.0;              // The spacing between the right edge of the badge and the edge of the table column.
 static const CGFloat iconSpacing = 2.0;                 // The spacing between the icon and it's adjacent cell.
 static const CGFloat disclosureTriangleSpace = 18.0;    // The indentation reserved for disclosure triangles for non-group items.

PXSourceList/PXSourceListBadgeCell.m

@@ -9,6 +9,8 @@
 
 #import "PXSourceListBadgeCell.h"
 
+#import "PXSourceList.h"
+
 //Drawing constants
 static inline NSColor *badgeBackgroundColor() { return [NSColor colorWithCalibratedRed:(152/255.0) green:(168/255.0) blue:(202/255.0) alpha:1]; }
 static inline NSColor *badgeHiddenBackgroundColor() { return [NSColor colorWithDeviceWhite:(180/255.0) alpha:1]; }
@@ -40,11 +42,27 @@ static const CGFloat badgeLeftAndRightPadding = 5.0;
 	NSDictionary *attributes;
 	NSColor *backgroundColor;
 
-	if(self.isHighlighted) {
+	if(self.isHighlighted || self.backgroundStyle == NSBackgroundStyleDark) {
 		backgroundColor = [NSColor whiteColor];
 
         NSResponder *firstResponder = controlView.window.firstResponder;
-        BOOL isFocused = [firstResponder isKindOfClass:[NSView class]] && [(NSView *)firstResponder isDescendantOf:controlView];
+        BOOL isFocused = NO;
+
+        // Starting with the closest ancestor of the control view and the first responder (to make sure both views are in the
+        // same subtree of the view hierarchy), keep going up the view hierarchy until we hit a PXSourceList instance to tell
+        // if the source list is focused.
+        // This covers both the cell-based and view-based cases as well as if a child view of the NSTableCellView (such as
+        // a text field) is focused.
+        if ([firstResponder isKindOfClass:[NSView class]]) {
+            NSView *view = [(NSView*)firstResponder ancestorSharedWithView:controlView];
+            do {
+                if ([view isKindOfClass:[PXSourceList class]]) {
+                    isFocused = YES;
+                    break;
+                }
+            } while ((view = [view superview]));
+        }
+
         NSColor *textColor;
 
 		if (isMainWindowVisible && isFocused)

ReleaseNotes.md

@@ -1,5 +1,22 @@
 # PXSourceList Release Notes
 
+## 2.0.5
+- Fix #43: sourceListDeleteKeyPressedOnRows: called twice. This was caused by an issue where PXSourceList was incorrectly removing the old delegate as an observer of PXSourceList notifications in -setDelegate:.
+
+## 2.0.4
+- PR #41: fix a Zeroing Weak References problem. This fixes an issue where using an `NSWindow`, `NSWindowController` or `NSViewController` as a PXSourceList delegate or dataSource would cause problems on 10.7 because prior to 10.8, these classes could not be referenced by zeroing weak references.
+- Remove unused `badgeMargin` constant from PXSourceList.m.
+
+## 2.0.3
+- Fix #40: Editing titles on cell based source list causes exception.
+- Fix issue in view-based source list example where items created with the add button couldn't be dragged.
+
+## 2.0.2
+- Fix #39: Badges not drawn correctly when Source List row is selected.
+
+## 2.0.1
+- Add missing note to the 2.0.0 release notes about marking `-[PXSourceList delegate]` and `-[PXSourceList dataSource]` as unavailable using \_\_attribute\_\_.
+
 ## 2.0.0
 
 ### New Features
@@ -11,6 +28,7 @@
 - Added a generic `PXSourceListItem` data source model class which can be used for easily constructing data source models without having to implement your own class.
 
 ### API Changes
+- **Incompatible change.* Marked `-[PXSourceList delegate]` and -[PXSourceList dataSource]` as unavailable using the “unavailable” \_\_attribute\_\_. These methods shouldn’t be used because of the internal implementation of PXSourceList, and have been documented as such since version 0.8. However, adding this \_\_attribute\_\_ is more robust because a compile-time error will be generated if you use either of these methods.
 - Added view-based Source List delegate methods to `PXSourceListDelegate`, namely:
 	- `-sourceList:viewForItem:`
 	- `-sourceList:rowViewForItem:`
@@ -67,4 +85,4 @@
 - Removed `SourceListItem` from the old example project as it has been superseded by `PXSourceListItem`.
 - Removed the TODO.rtf file from the project as all issues are now being tracked through GitHub.
 - Upgraded the Xcode project to the Xcode 5 project format. `LastUpgradeCheck` was updated from `0450` to`0500`.
-- Added a Release Notes file ;)
+- Added a Release Notes file ;)