-
Notifications
You must be signed in to change notification settings - Fork 0
[SDPAP-6684] Verify items in search index drush command #39
[SDPAP-6684] Verify items in search index drush command #39
Conversation
composer.json
Outdated
@@ -6,12 +6,19 @@ | |||
"require": { | |||
"drupal/search_api": "^1.11", | |||
"drupal/elasticsearch_connector": "^7.0", | |||
"dpc-sdp/tide_core": "^3.0.0" | |||
"dpc-sdp/tide_core": "dev-feature/SRM-512_update_drupal_to_9.4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will be reverted before merging
dev-tools.sh
Outdated
@@ -33,6 +33,6 @@ | |||
# stable version. | |||
# | |||
# Uncomment and set the Dev-Tools's commit value and commit this change. | |||
# export GH_COMMIT=COMMIT_SHA | |||
export GH_COMMIT=960eba39ed9543b17654dbd5e06f3a2ab2b1fb4b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will be reverted before merging
src/Commands/TideSearchCommands.php
Outdated
/** | ||
* Helper to convert node ids. | ||
*/ | ||
public static function convertToNodeIds($item) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @MdNadimHossain ,
I think you don't need the function convertToNodeIds
.
the $results
is an object that implements ResultInterface
, so I think you can get the actual result by using $results->getExtraData('elasticsearch_response')
or $results->getAllExtraData()
, then loop through to get nid
from the result.
Thanks @vincent-gao for pointing out that. |
JIRA
https://digital-vic.atlassian.net/browse/SDPAP-6684
Changes
Testing PR
https://nginx-php.pr-142.content-reference-sdp-vic-gov-au.sdp4.sdp.vic.gov.au
Test steps