Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hello,
first of all thank you for this great module! While selecting a custom view into the pdf view via the "include view" field, I can't seem to understand how to select a specific display for it. The only display that gets returned is the Master one. Am I missing something or is it not possible yet to select the view display into the field?
Thank you in advance
Comment | File | Size | Author |
---|---|---|---|
#4 | views_pdf-include-first-pdf-display-1421532-4.patch | 1.18 KB | Peacog |
Comments
Comment #1
etiennechataignier CreditAttribution: etiennechataignier commentedThe display is meant to be automatically determined.
But it does not display the unformatted PDF display included.
[EDIT] It seems that arguments of the included are mandatory. This solved the issue for me.
Comment #2
lsolesen CreditAttribution: lsolesen commentedSeems to be fixed. You need to have a PDF display in the included view. You cannot include the master view or any other types of view, but the PDF view.
Comment #4
Peacog CreditAttribution: Peacog commentedI'm reopening this because I had the same problem. If a display with id pdf_1 is not found, the code falls back to using the default display. In my case I had cloned a view and subsequently deleted the original pdf_1, so my included view now has a display with id pdf_2 and no pdf_1.
I've rolled a patch that looks for the first display with an id of pdf_x and sets that one. If a pdf display is not found it reverts to the default master display as before.
This is not a perfect solution. Ideally it would be possible to choose the display in the settings for the Include View field. There is an open support request for that feature here: Choose a display on the views_field_view module.
Comment #5
killua99 CreditAttribution: killua99 commentedExist a patch to handle this issues and others. Please see #2032189: Multiple issue fixing in a single patch and review others issues patch.
This is RTBC, and ready to apply.
Comment #6
killua99 CreditAttribution: killua99 commentedThanks peacog, committed