update CHANGELOG, lint fix

Author: thomas loubrieu

CHANGELOG.md

@@ -11,8 +11,11 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 - Added catalogs route support to enable federated hierarchical catalog browsing and navigation in the STAC API. [#547](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/547)
 
+
 ### Changed
 
+- Have opensearch datetime, geometry and collections fields defined as constant strings [#553](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/553)
+
 ### Fixed
 
 - Fix unawaited coroutine in `stac_fastapi.core.core`. [#551](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/551)

stac_fastapi/opensearch/stac_fastapi/opensearch/database_logic.py

@@ -445,7 +445,9 @@ def apply_ids_filter(search: Search, item_ids: List[str]):
     @staticmethod
     def apply_collections_filter(search: Search, collection_ids: List[str]):
         """Database logic to search a list of STAC collection ids."""
-        return search.filter("terms", **{DatabaseLogic.COLLECTION_FIELD:collection_ids})
+        return search.filter(
+            "terms", **{DatabaseLogic.COLLECTION_FIELD: collection_ids}
+        )
 
     @staticmethod
     def apply_free_text_filter(search: Search, free_text_queries: Optional[List[str]]):
@@ -506,10 +508,18 @@ def apply_datetime_filter(
                     ),
                     Q(
                         "bool",
-                        must_not=[Q("exists", field=DatabaseLogic.PROPERTIES_DATETIME_FIELD)],
+                        must_not=[
+                            Q("exists", field=DatabaseLogic.PROPERTIES_DATETIME_FIELD)
+                        ],
                         filter=[
-                            Q("exists", field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD),
-                            Q("exists", field=DatabaseLogic.PROPERTIES_END_DATETIME_FIELD),
+                            Q(
+                                "exists",
+                                field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD,
+                            ),
+                            Q(
+                                "exists",
+                                field=DatabaseLogic.PROPERTIES_END_DATETIME_FIELD,
+                            ),
                             Q(
                                 "range",
                                 properties__start_datetime={
@@ -543,17 +553,29 @@ def apply_datetime_filter(
                     ),
                     Q(
                         "bool",
-                        must_not=[Q("exists", field=DatabaseLogic.PROPERTIES_DATETIME_FIELD)],
+                        must_not=[
+                            Q("exists", field=DatabaseLogic.PROPERTIES_DATETIME_FIELD)
+                        ],
                         filter=[
-                            Q("exists", field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD),
-                            Q("exists", field=DatabaseLogic.PROPERTIES_END_DATETIME_FIELD),
+                            Q(
+                                "exists",
+                                field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD,
+                            ),
+                            Q(
+                                "exists",
+                                field=DatabaseLogic.PROPERTIES_END_DATETIME_FIELD,
+                            ),
                             Q(
                                 "range",
-                                properties__start_datetime={"lte": datetime_search["lte"]},
+                                properties__start_datetime={
+                                    "lte": datetime_search["lte"]
+                                },
                             ),
                             Q(
                                 "range",
-                                properties__end_datetime={"gte": datetime_search["gte"]},
+                                properties__end_datetime={
+                                    "gte": datetime_search["gte"]
+                                },
                             ),
                         ],
                     ),
@@ -568,7 +590,10 @@ def apply_datetime_filter(
                 filter_query = Q(
                     "bool",
                     filter=[
-                        Q("exists", field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD),
+                        Q(
+                            "exists",
+                            field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD,
+                        ),
                         Q("exists", field=DatabaseLogic.PROPERTIES_END_DATETIME_FIELD),
                         Q(
                             "range",
@@ -584,7 +609,10 @@ def apply_datetime_filter(
                 filter_query = Q(
                     "bool",
                     filter=[
-                        Q("exists", field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD),
+                        Q(
+                            "exists",
+                            field=DatabaseLogic.PROPERTIES_START_DATETIME_FIELD,
+                        ),
                         Q("exists", field=DatabaseLogic.PROPERTIES_END_DATETIME_FIELD),
                         Q(
                             "range",