Issue lxcid#77 - Fixing a crash when previousIndexPath is nil

joshuafeldman · joshuafeldman · commit 84af63f682f2 · 2016-05-18T15:02:33.000-07:00
diff --git a/LXReorderableCollectionViewFlowLayout/LXReorderableCollectionViewFlowLayout.m b/LXReorderableCollectionViewFlowLayout/LXReorderableCollectionViewFlowLayout.m
@@ -173,7 +173,7 @@ - (void)invalidateLayoutIfNecessary {
     NSIndexPath *newIndexPath = [self.collectionView indexPathForItemAtPoint:self.currentView.center];
     NSIndexPath *previousIndexPath = self.selectedItemIndexPath;
     
-    if ((newIndexPath == nil) || [newIndexPath isEqual:previousIndexPath]) {
+    if ((newIndexPath == nil) || (previousIndexPath == nil) || [newIndexPath isEqual:previousIndexPath]) {
         return;
     }
     

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,7 @@ - (void)invalidateLayoutIfNecessary {`
`173`	`173`	`NSIndexPath *newIndexPath = [self.collectionView indexPathForItemAtPoint:self.currentView.center];`
`174`	`174`	`NSIndexPath *previousIndexPath = self.selectedItemIndexPath;`
`175`	`175`
`176`		`- if ((newIndexPath == nil) \|\| [newIndexPath isEqual:previousIndexPath]) {`
	`176`	`+ if ((newIndexPath == nil) \|\| (previousIndexPath == nil) \|\| [newIndexPath isEqual:previousIndexPath]) {`
`177`	`177`	`return;`
`178`	`178`	`}`
`179`	`179`