Pachyderm: Fix pagination

Pachyderm/Model/Timeline.swift

@@ -17,22 +17,25 @@ public enum Timeline {
 }
 
 extension Timeline {
-    func request(range: RequestRange) -> Request<[Status]> {
+    var endpoint: String {
-        var request: Request<[Status]>
         switch self {
         case .home:
-            request = Request(method: .get, path: "/api/v1/timelines/home")
+            return "/api/v1/timelines/home"
-        case let .public(local):
+        case .public:
-            request = Request(method: .get, path: "/api/v1/timelines/public")
+            return "/api/v1/timelines/public"
-            if local {
-                request.queryParameters.append("local" => true)
-            }
         case let .tag(hashtag):
-            request = Request(method: .get, path: "/api/v1/timeliens/tag/\(hashtag)")
+            return "/api/v1/timelines/tag/\(hashtag)"
         case let .list(id):
-            request = Request(method: .get, path: "/api/v1/timelines/list/\(id)")
+            return "/api/v1/timelines/list/\(id)"
         case .direct:
-            request = Request(method: .get, path: "/api/v1/timelines/direct")
+            return "/api/v1/timelines/direct"
+        }
+    }
+    
+    func request(range: RequestRange) -> Request<[Status]> {
+        var request = Request<[Status]>(method: .get, path: endpoint)
+        if case .public(true) = self {
+            request.queryParameters.append("local" => true)
         }
         request.range = range
         return request

Pachyderm/Request/Parameter.swift

@@ -74,7 +74,8 @@ extension Array where Element == Parameter {
     
     var queryItems: [URLQueryItem] {
         return compactMap {
-            URLQueryItem(name: $0.name, value: $0.value)
+            guard let value = $0.value else { return nil }
+            return URLQueryItem(name: $0.name, value: value)
         }
     }
 }

Pachyderm/Request/Request.swift

@@ -40,17 +40,26 @@ extension Request {
         }
         set {
             let rangeParams = newValue.queryParameters
-            let max = rangeParams.first { $0.name == "max_id" }
+            if let max = rangeParams.first(where: { $0.name == "max_id" }) {
-            let since = rangeParams.first { $0.name == "since_id" }
+                if let i = queryParameters.firstIndex(where: { $0.name == "max_id" }) {
-            let count = rangeParams.first { $0.name == "count" }
-            if let max = max, let i = queryParameters.firstIndex(where: { $0.name == "max_id" }) {
                     queryParameters[i] = max
+                } else {
+                    queryParameters.append(max)
                 }
-            if let since = since, let i = queryParameters.firstIndex(where: { $0.name == "since_id" }) {
+            }
+            if let since = rangeParams.first(where: { $0.name == "since_id" }) {
+                if let i = queryParameters.firstIndex(where: { $0.name == "since_id" }) {
                     queryParameters[i] = since
+                } else {
+                    queryParameters.append(since)
                 }
-            if let count = count, let i = queryParameters.firstIndex(where: { $0.name == "count" }) {
+            }
+            if let count = rangeParams.first(where: { $0.name == "count" }) {
+                if let i = queryParameters.firstIndex(where: { $0.name == "count" }) {
                     queryParameters[i] = count
+                } else {
+                    queryParameters.append(count)
+                }
             }
         }
     }

Pachyderm/Response/Pagination.swift

@@ -30,9 +30,9 @@ extension Pagination {
         var range: RequestRange {
             switch kind {
             case .next:
-                return .after(id: id, count: limit)
-            case .prev:
                 return .before(id: id, count: limit)
+            case .prev:
+                return .after(id: id, count: limit)
             }
         }