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.
I'm not sure that this makes as much sense for a ConfigEntity, but we'll see how it pans out.
Mostly this stems from not knowing when to use execute()/executeDisplay/preview().
It only handles page displays right now, I'll expand it to blocks and ajax later.
Comment | File | Size | Author |
---|---|---|---|
#16 | vdc-1857422-16.patch | 32.91 KB | billywardrop |
#1 | vdc-1857422-1.patch | 5.04 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettPatch attached.
Comment #2
tim.plunkettPostponing on #1811828: Use #attached to find the css/js of a view and #1849356: Add a HTTP response code area handler
We want the render improvements from the first, and we have to support the usecase of the second.
Comment #4
Jelle_SSmall remark on current patch.
In the docblock of views_page():
$name
=>$view
Comment #5
tim.plunkettYeah that whole change to the function signature should be reverted anyway. Did some testing and it was not a smart idea :)
Comment #6
tim.plunkettThis doesn't a actually make any sense.
Comment #7
benjy CreditAttribution: benjy commentedThis would be a nice feature basically a view could be rendered when you have an entity reference to a view.
Comment #8
tim.plunkettComment #10
andypostComment #11
andypostComment #12
Manuel Garcia CreditAttribution: Manuel Garcia at Appnovation commentedComment #13
danylevskyiWe are on DrupalCon Dublin and we are going to fix this issue.
Comment #14
danylevskyiComment #15
billywardrop CreditAttribution: billywardrop as a volunteer commentedI am working on this at DrupalCon Dublin
Comment #16
billywardrop CreditAttribution: billywardrop as a volunteer commentedPatch attached
Comment #17
billywardrop CreditAttribution: billywardrop as a volunteer commentedThe patch is wrong, I'm working on a new patch
Comment #18
billywardrop CreditAttribution: billywardrop as a volunteer commentedI have checked the patch and the code is to old for the latest version of Drupal 8. The file structure no longer exists for this file:
"core/modules/views/lib/Drupal/views/Plugin/Core/Entity/View.php"
This patch needs a rework.
Comment #19
billywardrop CreditAttribution: billywardrop as a volunteer commentedRemoved Reroll and set this issue to Needs work
Comment #20
BerdirA view is always displayed with a specific display and possibly arguments. A view builder has no mechanism for passing this along. I think this should be won't fixed, the closest that core could offer would be a special field formatter that allows to select a display and maybe arguments, token based. but maybe that configuration should actually be a on the field, so it needs its own field type.