From 8ed8ff8c1365ab8e02b7ee861c5c7bc0d9bc048b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andr=C3=A9=20Maneiro?=
 <583546+oandregal@users.noreply.github.com>
Date: Fri, 17 Nov 2023 12:48:13 +0100
Subject: [PATCH] Update comment for constants

---
 .../edit-site/src/components/dataviews/add-filter.js   |  3 ++-
 .../edit-site/src/components/dataviews/in-filter.js    |  1 +
 .../edit-site/src/components/dataviews/view-list.js    | 10 ++++++++--
 3 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/packages/edit-site/src/components/dataviews/add-filter.js b/packages/edit-site/src/components/dataviews/add-filter.js
index 723700fae8857d..374a90eab9df03 100644
--- a/packages/edit-site/src/components/dataviews/add-filter.js
+++ b/packages/edit-site/src/components/dataviews/add-filter.js
@@ -22,7 +22,8 @@ const {
 	DropdownMenuItemV2,
 } = unlock( componentsPrivateApis );
 
-const ENUMERATION_TYPE = 'enumeration'; // TODO: merge with the one in filters.js
+// TODO: find a place where these constants can be shared across components.
+const ENUMERATION_TYPE = 'enumeration';
 
 export default function AddFilter( { fields, view, onChangeView } ) {
 	const filters = [];
diff --git a/packages/edit-site/src/components/dataviews/in-filter.js b/packages/edit-site/src/components/dataviews/in-filter.js
index 4154e0576101c1..cb76bab0f870a9 100644
--- a/packages/edit-site/src/components/dataviews/in-filter.js
+++ b/packages/edit-site/src/components/dataviews/in-filter.js
@@ -7,6 +7,7 @@ import {
 } from '@wordpress/components';
 import { __, sprintf } from '@wordpress/i18n';
 
+// TODO: find a place where these constants can be shared across components.
 export const OPERATOR_IN = 'in';
 
 export default ( { filter, view, onChangeView } ) => {
diff --git a/packages/edit-site/src/components/dataviews/view-list.js b/packages/edit-site/src/components/dataviews/view-list.js
index f4e0f3b55e4212..fc8b4075025792 100644
--- a/packages/edit-site/src/components/dataviews/view-list.js
+++ b/packages/edit-site/src/components/dataviews/view-list.js
@@ -52,6 +52,11 @@ const sortingItemsInfo = {
 	desc: { icon: arrowDown, label: __( 'Sort descending' ) },
 };
 const sortIcons = { asc: chevronUp, desc: chevronDown };
+
+// TODO: find a place where these constants can be shared across components.
+const ENUMERATION_TYPE = 'enumeration';
+const OPERATOR_IN = 'in';
+
 function HeaderMenu( { dataView, header } ) {
 	if ( header.isPlaceholder ) {
 		return null;
@@ -68,7 +73,7 @@ function HeaderMenu( { dataView, header } ) {
 	const sortedDirection = header.column.getIsSorted();
 
 	let filter;
-	if ( header.column.columnDef.type === 'enumeration' ) {
+	if ( header.column.columnDef.type === ENUMERATION_TYPE ) {
 		filter = {
 			field: header.column.columnDef.id,
 			elements: [
@@ -197,7 +202,8 @@ function HeaderMenu( { dataView, header } ) {
 														return (
 															field !==
 																filter.field ||
-															operator !== 'in'
+															operator !==
+																OPERATOR_IN
 														);
 													}
 												);