Skip to content

Commit 38eac95

Browse files
committed
apply creation time filter to submissions rather than submission_defs, fixes #1458
1 parent 203ebb1 commit 38eac95

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/model/query/geo-extracts.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ const getSubmissionFeatureCollectionGeoJson = (formPK, IDs, fieldPaths, submitte
3535
const doPathInProperties = arrayHasElements(fieldPaths) ? sql`, 'properties', NULL` : sql`, 'properties', json_build_object('fieldpath', path)`;
3636
const doAggregateByPath = arrayHasElements(fieldPaths) ? sql`` : sql`, path`;
3737
const submitterFilter = arrayHasElements(submitterIds) ? sql`AND sdef."submitterId" = ANY(${sql.array(submitterIds, 'int4')})` : sql``;
38-
const TSTZRangeFilter = arrayHasElements(TSTZRange) ? sql`AND sdef."createdAt" <@ tstzrange(${TSTZRange[0]}, ${TSTZRange[1]}, ${TSTZRange[2]})` : sql``;
38+
const TSTZRangeFilter = arrayHasElements(TSTZRange) ? sql`AND sub."createdAt" <@ tstzrange(${TSTZRange[0]}, ${TSTZRange[1]}, ${TSTZRange[2]})` : sql``;
3939
const reviewStateFilter = arrayHasElements(reviewStates) ? sql`AND ${stateFilterToQueryFragments(['sub', 'reviewState'], reviewStates)}` : sql``;
4040
const deletedFilter = deleted ? sql`AND sub."deletedAt" IS NOT NULL` : sql`AND sub."deletedAt" IS NULL`;
4141
const queryLimit = limit ? sql`LIMIT ${limit}` : sql``;
@@ -63,13 +63,13 @@ const getSubmissionFeatureCollectionGeoJson = (formPK, IDs, fieldPaths, submitte
6363
${submissionDefIdFilter}
6464
${onlyCurrentFilter}
6565
${submitterFilter}
66-
${TSTZRangeFilter}
6766
)
6867
INNER JOIN submissions sub ON (
6968
sdef."submissionId" = sub.id
7069
${submissionIdFilter}
7170
${deletedFilter}
7271
${reviewStateFilter}
72+
${TSTZRangeFilter}
7373
)
7474
INNER JOIN form_field_geo ffgeo ON (
7575
ffgeo.formschema_id = fdef."schemaId"

0 commit comments

Comments
 (0)