- Issue created by @ashraf.hussain
- ๐ฎ๐ณIndia ashraf.hussain Bangalore
Uploading patch for resolving the above issue.
- Status changed to Needs review
9 months ago 10:54am 5 March 2024 - ๐ฎ๐ณIndia ashraf.hussain Bangalore
Uploading the patch for fixing the error when clicking Index Now.
1. Error: Call to a member function getFields() on null in Drupal\appsearch\Utility\Database::prepareNodeToIndex() (line 60 of /app/docroot/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Utility/Database.php)The above error happens when the Language Code for the node is 'pt-br' or 'es-mx'.
filter_var($node_id, FILTER_SANITIZE_NUMBER_INT) does not work in those cases to get the node id.2. Error: Call to a member function getFileUri() on null in Drupal\appsearch\Utility\Database::mapFieldValues() (line 110 of /app/docroot/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Utility/Database.php
The above error happens when the $field->entity is empty.
Both the issue are resolved in this patch.
- ๐ฎ๐ณIndia mohrerao
@ ashraf.hussain, Thanks for reporting this issue. The current implementation extracts Numbers. This could create problems if the content type also has numbers in it. eg: No_1_FanClub/2323:pt-br
return filter_var($node_id, FILTER_SANITIZE_NUMBER_INT);
Instead of removing +/_ symbol from the string i would prefer to extract string between '/' and ':' characters so that we would be sure of extracting the node ID.
- Status changed to Needs work
3 months ago 1:56pm 14 August 2024 - ๐ฎ๐ณIndia anandhi karnan Chennai
anandhi karnan โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia anandhi karnan Chennai
anandhi karnan โ changed the visibility of the branch 3425694-elastic_enterprise_search_index_issue to hidden.
- ๐ฎ๐ณIndia anandhi karnan Chennai
anandhi karnan โ changed the visibility of the branch 3425694-elastic_enterprise_search to hidden.
- Merge request !10issue #3425694 Elastic Enterprise Search indexing issue fix โ (Open) created by anandhi karnan
- Status changed to Needs review
2 months ago 5:46am 6 September 2024 - ๐ฎ๐ณIndia anandhi karnan Chennai
I have hidden the branches that were created from the incorrect version. The actual fix for this issue can now be found in Merge Request 10. Please review that merge request for the correct updates.
- Merge request !11issue #3425694 Elastic Enterprise Search indexing issue fix โ (Open) created by anandhi karnan