Merge pull request #444 from stac-utils/pv/fix-collection-aggregation

collection-level aggregation how has the collection id properly applied

Author: Phil Varner

src/lib/api.js

@@ -639,15 +639,6 @@ const aggregate = async function (collectionId,
   const ids = extractIds(queryParameters)
   const collections = extractCollectionIds(queryParameters)
 
-  let linkEndpoint = endpoint
-  let collectionEndpoint
-
-  if (collectionId) {
-    queryParameters.collections = [collectionId]
-    linkEndpoint = `${endpoint}/collections/${collectionId}`
-    collectionEndpoint = `${endpoint}/collections/${collectionId}`
-  }
-
   const searchParams = pickBy({
     datetime,
     intersects: geometry,
@@ -656,6 +647,15 @@ const aggregate = async function (collectionId,
     collections,
   })
 
+  let linkEndpoint = endpoint
+  let collectionEndpoint
+
+  if (collectionId) {
+    searchParams.collections = [collectionId]
+    linkEndpoint = `${endpoint}/collections/${collectionId}`
+    collectionEndpoint = `${endpoint}/collections/${collectionId}`
+  }
+
   logger.debug('Aggregate parameters: %j', searchParams)
 
   let esResponse