Merge pull request #310 from raisaanjani92/shouldShowMenuForItemAt

Use optional in collectionView:shouldShowMenuForItemAtIndexPath:
badoo · May 30, 2017 · 0eb7ac5 · 0eb7ac5
2 parents c2b378d + dd421e7
commit 0eb7ac5
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/Chatto/Source/ChatController/BaseChatViewController+Presenters.swift b/Chatto/Source/ChatController/BaseChatViewController+Presenters.swift
@@ -84,7 +84,10 @@ extension BaseChatViewController: ChatCollectionViewLayoutDelegate {
     }
 
     @objc(collectionView:shouldShowMenuForItemAtIndexPath:)
-    open func collectionView(_ collectionView: UICollectionView, shouldShowMenuForItemAt indexPath: IndexPath) -> Bool {
+    open func collectionView(_ collectionView: UICollectionView, shouldShowMenuForItemAt indexPath: IndexPath?) -> Bool {
+        // Note: IndexPath set optional due to https://github.com/badoo/Chatto/issues/310
+        // Might be related: https://bugs.swift.org/browse/SR-2417
+        guard let indexPath = indexPath else { return false }
         return self.presenterForIndexPath(indexPath).shouldShowMenu()
     }