Closed (fixed)
Project:
Search API attachments
Version:
9.0.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Dec 2023 at 09:13 UTC
Updated:
20 Nov 2024 at 08:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
shaxa commentedComment #3
shaxa commentedComment #4
shaxa commentedComment #5
ismaelromero commentedComment #6
ismaelromero commentedCurrently going through this issue and the suggested fix. I’ll post my review soon.
Comment #7
ismaelromero commentedAfter completing the installation of the module, applying the relevant patch, and configuring both the module and the Apache Tika extractor, I conducted a series of comprehensive tests. The results confirm that it is now possible to utilize file content in view modes.
Below are images illustrating the process and the results obtained:
Article display configuration
Article edit with attached file
Attachment presented as a generic file
Attachment with content displayed
Based on my testing, I can confirm that the requirements are being adequately met.
I welcome any additional comments or suggestions.
Comment #8
ismaelromero commentedComment #9
izus commentedhi,
thanks for this work
did you test the case where we have many file fields :
example :
field1 with one.pdf
and
field2 with two.pdf
does the extra field behave well to display both of pdf content in a single view mode ?
Comment #10
ismaelromero commentedHi Izus, thank you for your observation and comment.
I conducted the tests by creating a new content type with two file fields (Multi-file Article):

I then created a test content item using this content type:

Both files are displayed correctly in a single view mode, as shown:

However, I noticed a potential issue: if the content of the first PDF file is too lengthy, the entire text is rendered on the page, which may push down or obscure the content of the second file. This behavior might require a solution, such as limiting the displayed content length.
I remain open to any further comments and/or suggestions.
Comment #12
izus commentedthis is now merged
thanks