New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement EZP-24275: Indexable Image field type #1240
Conversation
{ | ||
if ( $fileInfo->isDir() ) | ||
{ | ||
if ( !self::isIgnoredPath( dirname( $path ) ) ) |
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.
can't these two if be combined?
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.
Hi, this part was actually copied from FileSearchBaseIntegrationTest
, it is here temporarily until we implement search tests for all field types.
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.
ok.
+1 |
1 similar comment
+1 |
5cbf8c4
to
713835e
Compare
Implement EZP-24275: Indexable Image field type
This PR resolves https://jira.ez.no/browse/EZP-24275
A subtask of https://jira.ez.no/browse/EZP-24232
Additionally to the file name, which is the default value used by Field criterion and sort clause, Image field type also indexes alternative text and file size. These can be targeted by using
CustomFieldInterface:: setCustomField()
method. Test framework is here refactored to allow testing of these custom fields. However this is a bit awkward, and calls for a refactoring to make testing simpler for the new implementations.Since not all searchable field types have Indexable definition implemented, this required
FileSearchBaseIntegrationTest
. This is temporary and will be removed when all searchable field types are covered.Dedicated methods to provide Field search target value was needed, this was taken from #1230.
Solr's base
CriterionVisitor
now implementsprepareValue()
method, which is used to convert and escape the search target value to the proper Solr representation. This is taken from #1229.Additional fixes
Small bug in Elasticsearch's
FieldRange
criterion visitor, where wrong variable was checked, causingnull
to be used as a filter in case when there are no field filters defined. This did not affect the behavior as Elasticsearch seems to be tolerant to it.Followups