Merge pull request #15 from explicitcall/master

kylef · kylef · commit 7f55eb9f588c · 2014-09-09T15:57:50.000+01:00
fixed array out of bounds crash
diff --git a/QueryKit/QuerySet.swift b/QueryKit/QuerySet.swift
@@ -102,9 +102,11 @@ public class QuerySet<T : NSManagedObject> : SequenceType, Equatable {
             request.fetchLimit = 1
 
             var error:NSError?
-            let items = context.executeFetchRequest(request, error:&error)
-
-            return (object:items?[0] as T?, error:error)
+            if let items = context.executeFetchRequest(request, error:&error) {
+              return (object:items.first as T?, error:error)
+            } else {
+              return (object: nil, error: error)
+            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -102,9 +102,11 @@ public class QuerySet<T : NSManagedObject> : SequenceType, Equatable {`
`102`	`102`	`request.fetchLimit = 1`
`103`	`103`
`104`	`104`	`var error:NSError?`
`105`		`- let items = context.executeFetchRequest(request, error:&error)`
`106`		`-`
`107`		`- return (object:items?[0] as T?, error:error)`
	`105`	`+ if let items = context.executeFetchRequest(request, error:&error) {`
	`106`	`+ return (object:items.first as T?, error:error)`
	`107`	`+ } else {`
	`108`	`+ return (object: nil, error: error)`
	`109`	`+ }`
`108`	`110`	`}`
`109`	`111`	`}`
`110`	`112`