acquia · rajeshreeputra · Dec 27, 2024 · Dec 30, 2024 · Dec 30, 2024 · Dec 30, 2024
diff --git a/composer.lock b/composer.lock
diff --git a/modules/acquia_cms_article/config/optional/field.storage.node.field_article_type.yml b/modules/acquia_cms_article/config/optional/field.storage.node.field_article_type.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_article
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Article Type'
 id: node.field_article_type

diff --git a/modules/acquia_cms_article/config/optional/node.type.article.yml b/modules/acquia_cms_article/config/optional/node.type.article.yml
@@ -5,6 +5,8 @@ dependencies:
     module:
       - acquia_cms_article
 third_party_settings:
+  acquia_cms_search:
+    search_index: content
   acquia_cms_common:
     workflow_id: editorial
     workbench_email_templates:
@@ -18,7 +20,6 @@ third_party_settings:
         open_graph: open_graph
         schema_article: schema_article
         twitter_cards: twitter_cards
-    search_index: content
     subtype:
       field: field_article_type
       facet: articles_article_type

diff --git a/modules/acquia_cms_article/config/optional/views.view.articles.yml b/modules/acquia_cms_article/config/optional/views.view.articles.yml
@@ -6,7 +6,6 @@ dependencies:
     - search_api.index.content
     - views.view.articles_fallback
   module:
-    - acquia_cms_common
     - acquia_cms_search
     - search_api
     - user

diff --git a/modules/acquia_cms_common/acquia_cms_common.module b/modules/acquia_cms_common/acquia_cms_common.module
@@ -64,23 +64,6 @@ function acquia_cms_common_form_alter(&$form, FormStateInterface $form_state, $f
   }
 }
 
-/**
- * Implements hook_views_data().
- */
-function acquia_cms_common_views_data() {
-  return [
-    'views' => [
-      'main_listing_pages_view' => [
-        'title' => t('Main listing pages - Text area'),
-        'help' => t('Insert a text area inside a main view listing pages if the search server is available. If the view is not based on a Search API index, this behaves like a standard text area.'),
-        'area' => [
-          'id' => 'main_listing_pages_view',
-        ],
-      ],
-    ],
-  ];
-}
-
 /**
  * Implements hook_theme().
  */

diff --git a/modules/acquia_cms_common/config/optional/field.storage.node.field_categories.yml b/modules/acquia_cms_common/config/optional/field.storage.node.field_categories.yml
@@ -5,7 +5,7 @@ dependencies:
     - node
     - taxonomy
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: Categories
 id: node.field_categories

diff --git a/modules/acquia_cms_common/config/schema/acquia_cms_common.schema.yml b/modules/acquia_cms_common/config/schema/acquia_cms_common.schema.yml
@@ -38,7 +38,19 @@ node.type.*.third_party.acquia_cms_common:
       type: string
       label: 'Machine name of the sitemap variant where the content type will be added'
 
-field.storage.*.*.third_party.acquia_cms_common:
+acquia_cms_common.settings:
+  type: config_object
+  label: 'Acquia CMS settings'
+  mapping:
+    user_login_redirection:
+      type: boolean
+      label: 'Enable special redirection on user login for specific roles'
+    starter_kit_name:
+      type: string
+      label: 'The starterkit name'
+
+# This is added to make test pass.
+field.storage.*.*.third_party.acquia_cms_search:
   type: mapping
   label: 'Acquia CMS settings'
   mapping:
@@ -52,14 +64,3 @@ field.storage.*.*.third_party.acquia_cms_common:
     search_label:
       type: string
       label: 'The label of the field in the Search API index'
-
-acquia_cms_common.settings:
-  type: config_object
-  label: 'Acquia CMS settings'
-  mapping:
-    user_login_redirection:
-      type: boolean
-      label: 'Enable special redirection on user login for specific roles'
-    starter_kit_name:
-      type: string
-      label: 'The starterkit name'
diff --git a/modules/acquia_cms_event/config/optional/field.storage.node.field_door_time.yml b/modules/acquia_cms_event/config/optional/field.storage.node.field_door_time.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_event
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Door Time'
 id: node.field_door_time

diff --git a/modules/acquia_cms_event/config/optional/field.storage.node.field_event_duration.yml b/modules/acquia_cms_event/config/optional/field.storage.node.field_event_duration.yml
@@ -7,7 +7,7 @@ dependencies:
     module:
       - acquia_cms_event
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Event Duration'
 id: node.field_event_duration

diff --git a/modules/acquia_cms_event/config/optional/field.storage.node.field_event_end.yml b/modules/acquia_cms_event/config/optional/field.storage.node.field_event_end.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_event
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Event End Date'
 id: node.field_event_end

diff --git a/modules/acquia_cms_event/config/optional/field.storage.node.field_event_start.yml b/modules/acquia_cms_event/config/optional/field.storage.node.field_event_start.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_event
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Event Start Date'
 id: node.field_event_start

diff --git a/modules/acquia_cms_event/config/optional/field.storage.node.field_event_type.yml b/modules/acquia_cms_event/config/optional/field.storage.node.field_event_type.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_event
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Event Type'
 id: node.field_event_type

diff --git a/modules/acquia_cms_event/config/optional/node.type.event.yml b/modules/acquia_cms_event/config/optional/node.type.event.yml
@@ -5,6 +5,8 @@ dependencies:
     module:
       - acquia_cms_event
 third_party_settings:
+  acquia_cms_search:
+    search_index: content
   acquia_cms_common:
     workflow_id: editorial
     workbench_email_templates:
@@ -22,7 +24,6 @@ third_party_settings:
       field: field_event_type
       facet: events_event_type
     sitemap_variant: default
-    search_index: content
   menu_ui:
     available_menus:
       - main

diff --git a/modules/acquia_cms_page/config/optional/node.type.page.yml b/modules/acquia_cms_page/config/optional/node.type.page.yml
@@ -9,6 +9,8 @@ dependencies:
     module:
       - acquia_cms_page
 third_party_settings:
+  acquia_cms_search:
+    search_index: content
   acquia_cms_common:
     workflow_id: editorial
     workbench_email_templates:
@@ -23,7 +25,6 @@ third_party_settings:
         schema_article: schema_article
         twitter_cards: twitter_cards
     sitemap_variant: default
-    search_index: content
   menu_ui:
     available_menus:
       - main

diff --git a/modules/acquia_cms_person/config/optional/field.storage.node.field_email.yml b/modules/acquia_cms_person/config/optional/field.storage.node.field_email.yml
@@ -7,7 +7,7 @@ dependencies:
     module:
       - acquia_cms_person
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: Email
 id: node.field_email

diff --git a/modules/acquia_cms_person/config/optional/field.storage.node.field_job_title.yml b/modules/acquia_cms_person/config/optional/field.storage.node.field_job_title.yml
@@ -7,7 +7,7 @@ dependencies:
     module:
       - acquia_cms_person
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: Job Title
 id: node.field_job_title

diff --git a/modules/acquia_cms_person/config/optional/field.storage.node.field_person_telephone.yml b/modules/acquia_cms_person/config/optional/field.storage.node.field_person_telephone.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_person
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Person Telephone'
 id: node.field_person_telephone

diff --git a/modules/acquia_cms_person/config/optional/field.storage.node.field_person_type.yml b/modules/acquia_cms_person/config/optional/field.storage.node.field_person_type.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_person
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Person Type'
 id: node.field_person_type

diff --git a/modules/acquia_cms_person/config/optional/node.type.person.yml b/modules/acquia_cms_person/config/optional/node.type.person.yml
@@ -5,6 +5,8 @@ dependencies:
     module:
       - acquia_cms_person
 third_party_settings:
+  acquia_cms_search:
+    search_index: content
   acquia_cms_common:
     workflow_id: editorial
     workbench_email_templates:
@@ -22,7 +24,6 @@ third_party_settings:
       field: field_person_type
       facet: people_person_type
     sitemap_variant: default
-    search_index: content
   menu_ui:
     available_menus:
       - main

diff --git a/modules/acquia_cms_place/config/optional/field.storage.node.field_place_address.yml b/modules/acquia_cms_place/config/optional/field.storage.node.field_place_address.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_place
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Place Address'
 id: node.field_place_address

diff --git a/modules/acquia_cms_place/config/optional/field.storage.node.field_place_telephone.yml b/modules/acquia_cms_place/config/optional/field.storage.node.field_place_telephone.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_place
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Place Telephone'
 id: node.field_place_telephone

diff --git a/modules/acquia_cms_place/config/optional/field.storage.node.field_place_type.yml b/modules/acquia_cms_place/config/optional/field.storage.node.field_place_type.yml
@@ -8,7 +8,7 @@ dependencies:
     module:
       - acquia_cms_place
 third_party_settings:
-  acquia_cms_common:
+  acquia_cms_search:
     search_index: content
     search_label: 'Place Type'
 id: node.field_place_type

diff --git a/modules/acquia_cms_place/config/optional/node.type.place.yml b/modules/acquia_cms_place/config/optional/node.type.place.yml
@@ -5,6 +5,8 @@ dependencies:
     module:
       - acquia_cms_place
 third_party_settings:
+  acquia_cms_search:
+    search_index: content
   acquia_cms_common:
     workflow_id: editorial
     workbench_email_templates:
@@ -22,7 +24,6 @@ third_party_settings:
       field: field_place_type
       facet: places_place_type
     sitemap_variant: default
-    search_index: content
   menu_ui:
     available_menus:
       - main

diff --git a/modules/acquia_cms_search/acquia_cms_search.info.yml b/modules/acquia_cms_search/acquia_cms_search.info.yml
@@ -4,7 +4,6 @@ description: "Provides powerful search capabilities to the site"
 type: module
 core_version_requirement: ^10.1 || ^11
 dependencies:
-  - acquia_cms_common:acquia_cms_common
   - collapsiblock:collapsiblock
   - facets:facets
   - facets_pretty_paths:facets_pretty_paths

diff --git a/modules/acquia_cms_search/acquia_cms_search.install b/modules/acquia_cms_search/acquia_cms_search.install
@@ -5,6 +5,7 @@
  * Contains installation routines for Acquia CMS Search.
  */
 
+use Drupal\acquia_cms_search\Facade\FacetFacade;
 use Drupal\node\Entity\NodeType;
 
 /**
@@ -24,19 +25,23 @@ function acquia_cms_search_install($is_syncing) {
 
     // Retroactively enable indexing for any content types that existed before
     // this module was installed.
-    $node_types = NodeType::loadMultiple();
+    $entity_type_manager = \Drupal::entityTypeManager();
+    $node_types = $entity_type_manager->getStorage('node_type')->loadMultiple();
     array_walk($node_types, 'acquia_cms_search_node_type_insert');
-    $enabled_modules = \Drupal::service('module_handler')->getModuleList();
-    $enabled_modules = array_keys($enabled_modules);
-    _acquia_cms_search_add_category_facet($enabled_modules);
-  }
-}
 
-/**
- * Implements hook_module_preinstall().
- */
-function acquia_cms_search_module_preinstall($module) {
-  \Drupal::service('acquia_cms_common.utility')->setModulePreinstallTriggered($module);
+    // Add the Search Category facet if the field exists.
+    $field_storage = $entity_type_manager->getStorage('field_storage_config')
+      ->loadByProperties(['field_name' => 'field_categories']);
+    if ($field_storage) {
+      \Drupal::classResolver(FacetFacade::class)->addFacet([
+        'id' => 'search_category',
+        'name' => 'Category',
+        'url_alias' => 'category',
+        'field_identifier' => 'field_categories',
+      ]);
+    }
+
+  }
 }
 
 /**