feat(trees): sort Chronostrat tree by period instead of name, add ranges

config/common/schema_localization_en.json

@@ -3479,7 +3479,7 @@
       "guid": {"name": "GUID", "desc": "GUID", "format": null, "ishidden": true, "isuiformatter": false, "picklistname": null, "type": "java.lang.String", "isrequired": false, "weblinkname": null},
       "highestchildnodenumber": {"name": "Highest Child Node Number", "desc": "Highest Child Node Number", "format": null, "ishidden": true, "isuiformatter": false, "picklistname": null, "type": "java.lang.Integer", "isrequired": false, "weblinkname": null},
       "isaccepted": {"name": "Is Preferred", "desc": "Is Preferred", "format": null, "ishidden": true, "isuiformatter": false, "picklistname": null, "type": "java.lang.Boolean", "isrequired": false, "weblinkname": null},
-      "isbiostrat": {"name": "Is BioStratigraphy", "desc": "Is BioStratigraphy", "format": null, "ishidden": false, "isuiformatter": false, "picklistname": null, "type": null, "isrequired": false, "weblinkname": null},
+      "isbiostrat": {"name": "Is Biostratigraphy", "desc": "Is Biostratigraphy", "format": null, "ishidden": false, "isuiformatter": false, "picklistname": null, "type": null, "isrequired": false, "weblinkname": null},
       "modifiedbyagent": {"name": "Modified By Agent", "desc": "Modified By Agent", "format": null, "ishidden": true, "isuiformatter": false, "picklistname": null, "type": null, "isrequired": false, "weblinkname": null},
       "name": {"name": "Name", "desc": "Name", "format": null, "ishidden": false, "isuiformatter": false, "picklistname": null, "type": "java.lang.String", "isrequired": false, "weblinkname": null},
       "nodenumber": {"name": "Node Number", "desc": "Node Number", "format": null, "ishidden": true, "isuiformatter": false, "picklistname": null, "type": "java.lang.Integer", "isrequired": false, "weblinkname": null},

specifyweb/backend/trees/tests/test_trees.py

@@ -349,6 +349,8 @@ def _run_for_row():
                 self.earth.id,
                 "geographyid",
                 False,
+                False,
+                'all',
                 session,
             )
             expected = [
@@ -362,8 +364,13 @@ def _run_for_row():
                     None,
                     None,
                     "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     self.na.children.count(),
                     "NA Syn 0, NA Syn 1",
+                    "NULL",
                 ),
                 (
                     na_syn_0.id,
@@ -375,8 +382,13 @@ def _run_for_row():
                     self.na.id,
                     self.na.fullname,
                     "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     na_syn_0.children.count(),
                     None,
+                    "NULL",
                 ),
                 (
                     na_syn_1.id,
@@ -388,8 +400,13 @@ def _run_for_row():
                     self.na.id,
                     self.na.fullname,
                     "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     na_syn_1.children.count(),
                     None,
+                    "NULL",
                 ),
             ]
 
@@ -402,6 +419,8 @@ def _run_for_row():
                 self.na.id,
                 "name",
                 False,
+                False,
+                'all',
                 session,
             )
             expected = [
@@ -415,8 +434,13 @@ def _run_for_row():
                     None,
                     None,
                     "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     self.usa.children.count(),
                     "USA Syn 0, USA Syn 1, USA Syn 2",
+                    "NULL",
                 ),
                 (
                     usa_syn_0.id,
@@ -428,8 +452,13 @@ def _run_for_row():
                     self.usa.id,
                     self.usa.fullname,
                     "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     0,
                     None,
+                    "NULL",
                 ),
                 (
                     usa_syn_1.id,
@@ -441,8 +470,13 @@ def _run_for_row():
                     self.usa.id,
                     self.usa.fullname,
                     "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     0,
                     None,
+                    "NULL",
                 ),
                 (
                     usa_syn_2.id,
@@ -454,8 +488,13 @@ def _run_for_row():
                     self.usa.id,
                     self.usa.fullname,
                     "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     0,
                     None,
+                    "NULL",
                 ),
             ]
             self.assertCountEqual(results, expected)
@@ -498,6 +537,8 @@ def _run_for_row():
                 root.id,
                 "name",
                 True,
+                False,
+                'all',
                 session,
             )
             expected = [
@@ -511,8 +552,13 @@ def _run_for_row():
                     None,
                     None,
                     animalia.author,
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     0,
                     metazoa.fullname,
+                    "NULL",
                 ),
                 (
                     metazoa.id,
@@ -524,8 +570,13 @@ def _run_for_row():
                     animalia.id,
                     animalia.fullname,
                     metazoa.author,
+                    "NULL",
+                    "NULL",
+                    "NULL",
+                    "NULL",
                     0,
                     None,
+                    "NULL",
                 ),
             ]
 

specifyweb/backend/trees/views.py

@@ -165,13 +165,15 @@ def tree_view(request, treedef, tree: TREE_TABLE, parentid, sortfield):
     implementation/design decisions
     """
     include_author = request.GET.get('includeauthor', False) and tree == 'taxon'
+    include_start_end_periods = request.GET.get('includestartendperiods', False) and tree == 'geologictimeperiod'
+    biostrat = request.GET.get('biostrat', 'all')
     with sqlmodels.session_context() as session:
         set_group_concat_max_len(session.connection())
-        results = get_tree_rows(treedef, tree, parentid, sortfield, include_author, session)
+        results = get_tree_rows(treedef, tree, parentid, sortfield, include_author, include_start_end_periods, biostrat, session)
     return HttpResponse(toJson(results), content_type='application/json')
 
 
-def get_tree_rows(treedef, tree, parentid, sortfield, include_author, session):
+def get_tree_rows(treedef, tree, parentid, sortfield, include_author, include_start_end_periods, biostrat, session):
     tree_table = spmodels.datamodel.get_table(tree)
     parentid = None if parentid == 'null' else int(parentid)
 
@@ -210,10 +212,55 @@ def get_tree_rows(treedef, tree, parentid, sortfield, include_author, session):
         node.AcceptedID.label("accepted_id"),
         accepted.fullName.label("accepted_fullname"),
         (node.author if include_author else literal("NULL")).label("author"),
+        (
+            node.startPeriod
+            if include_start_end_periods
+            else literal("NULL")
+        ).label("start_period"),
+        (
+            node.startUncertainty
+            if include_start_end_periods
+            else literal("NULL")
+        ).label("start_uncertainty"),
+        (
+            node.endPeriod
+            if include_start_end_periods
+            else literal("NULL")
+        ).label("end_period"),
+        (
+            node.endUncertainty
+            if include_start_end_periods
+            else literal("NULL")
+        ).label("end_uncertainty"),
         func.coalesce(child_counts.c.child_count, 0).label("child_count"),
         synonym_names.c.synonyms.label("synonyms"),
+        (
+            node.isBioStrat
+            if tree == 'geologictimeperiod'
+            else literal("NULL")
+        ).label("is_biostrat"),
     ]
 
+    if biostrat != 'all' and tree == 'geologictimeperiod':
+        # Count all matching descendants (any depth) using nested-set ranges
+        if biostrat == 'bio':
+            descendant_match = child.isBioStrat == True
+        else:
+            descendant_match = (child.isBioStrat == False) | (child.isBioStrat.is_(None))
+        matching_descendant_count = (
+            select(func.count(distinct(child._id)))
+            .where(
+                child.nodeNumber > node.nodeNumber,
+                child.nodeNumber <= node.highestChildNodeNumber,
+                descendant_match,
+                getattr(child, tree_table.name + "TreeDefID") == int(treedef),
+            )
+            .correlate(node)
+            .scalar_subquery()
+            .label("matching_descendant_count")
+        )
+        cols.append(matching_descendant_count)
+
     query = (
         select(*cols)
         .outerjoin(accepted, node.AcceptedID == accepted._id)

specifyweb/frontend/js_src/lib/components/Preferences/UserDefinitions.tsx

@@ -1677,6 +1677,13 @@ export const userPreferenceDefinitions = {
             renderer: ColorPickerPreferenceItem,
             container: 'label',
           }),
+          displayChronoPeriods: definePref<boolean>({
+            title: preferencesText.displayChronoPeriods(),
+            requiresReload: false,
+            visible: true,
+            defaultValue: true,
+            type: 'java.lang.Boolean',
+          }),
           rankThreshold: definePref<number>({
             title: preferencesText.rankThreshold(),
             description: preferencesText.rankThresholdDescription(),

specifyweb/frontend/js_src/lib/components/QueryComboBox/__tests__/biostratConditions.test.ts

@@ -0,0 +1,71 @@
+import { requireContext } from '../../../tests/helpers';
+import { serializeResource } from '../../DataModel/serializers';
+import { tables } from '../../DataModel/tables';
+import { queryFieldFilterSpecs } from '../../QueryBuilder/FieldFilterSpec';
+import { getQueryComboBoxConditions } from '../helpers';
+
+requireContext();
+
+describe('getQueryComboBoxConditions with BioStrat typeSearchName', () => {
+  const getBaseParams = () => ({
+    // Use a non-tree resource to avoid toTreeTable() requiring tree data
+    resource: new tables.CollectionObject.Resource(
+      {},
+      { noBusinessRules: true }
+    ),
+    fieldName: 'name' as const,
+    treeData: undefined,
+    collectionRelationships: undefined,
+    relatedTable: tables.GeologicTimePeriod,
+    subViewRelationship: undefined,
+    treeDefinition: undefined,
+  });
+
+  test('adds isBioStrat=true filter when typeSearchName is BioStrat', () => {
+    const conditions = getQueryComboBoxConditions({
+      ...getBaseParams(),
+      typeSearchName: 'BioStrat',
+    });
+    const serialized = conditions.map(serializeResource);
+    expect(serialized).toContainEqual(
+      expect.objectContaining({
+        fieldName: 'isBioStrat',
+        startValue: 'true',
+        operStart: queryFieldFilterSpecs.true.id,
+      })
+    );
+  });
+
+  test('does NOT add isBioStrat filter when typeSearchName is something else', () => {
+    const conditions = getQueryComboBoxConditions({
+      ...getBaseParams(),
+      typeSearchName: 'ChronosStrat',
+    });
+    const serialized = conditions.map(serializeResource);
+    expect(
+      serialized.filter(({ fieldName }) => fieldName === 'isBioStrat')
+    ).toHaveLength(0);
+  });
+
+  test('does NOT add isBioStrat filter when typeSearchName is undefined', () => {
+    const conditions = getQueryComboBoxConditions({
+      ...getBaseParams(),
+    });
+    const serialized = conditions.map(serializeResource);
+    expect(
+      serialized.filter(({ fieldName }) => fieldName === 'isBioStrat')
+    ).toHaveLength(0);
+  });
+
+  test('does NOT add isBioStrat filter for non-GeologicTimePeriod tables', () => {
+    const conditions = getQueryComboBoxConditions({
+      ...getBaseParams(),
+      relatedTable: tables.Taxon,
+      typeSearchName: 'BioStrat',
+    });
+    const serialized = conditions.map(serializeResource);
+    expect(
+      serialized.filter(({ fieldName }) => fieldName === 'isBioStrat')
+    ).toHaveLength(0);
+  });
+});

specifyweb/frontend/js_src/lib/components/QueryComboBox/helpers.ts

@@ -81,6 +81,7 @@ export function getQueryComboBoxConditions({
   subViewRelationship,
   relatedTable,
   treeDefinition,
+  typeSearchName,
 }: {
   readonly resource: SpecifyResource<AnySchema>;
   readonly fieldName: string;
@@ -89,6 +90,7 @@ export function getQueryComboBoxConditions({
   readonly relatedTable: SpecifyTable;
   readonly subViewRelationship: Relationship | undefined;
   readonly treeDefinition: string | undefined;
+  readonly typeSearchName?: string | undefined;
 }): RA<SpecifyResource<SpQueryField>> {
   const fields: WritableArray<SpecifyResource<SpQueryField>> = [];
   const treeResource = toTreeTable(resource);
@@ -164,6 +166,22 @@ export function getQueryComboBoxConditions({
     );
   }
 
+  /**
+   * When looking up BioStrat values for PaleoContext, only show
+   * GeologicTimePeriod nodes where isBioStrat is TRUE.
+   */
+  if (
+    typeSearchName === 'BioStrat' &&
+    relatedTable === tables.GeologicTimePeriod
+  )
+    fields.push(
+      QueryFieldSpec.fromPath(tables.GeologicTimePeriod.name, ['isBioStrat'])
+        .toSpQueryField()
+        .set('isDisplay', false)
+        .set('operStart', queryFieldFilterSpecs.true.id)
+        .set('startValue', 'true')
+    );
+
   if (
     typeof collectionRelationships === 'object' &&
     fieldName === 'collectionRelType'