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.
Follow up of #1448764: Ticketlist doesn't show edit link for Views.
We need to add the node ids, so we can check the right access for operations (edit/delete).
Comment | File | Size | Author |
---|---|---|---|
#3 | storm_views_nid-1463288-3.patch | 39.56 KB | kfritsche |
#1 | storm_views_nid-1463288-1.patch | 7.94 KB | kfritsche |
Comments
Comment #1
kfritscheHere a patch for tickets.
If this works, then i will do this for the other content types.
For me it does, but didn't test it very well yet.
Comment #2
kfritscheComment #3
kfritscheHere now the patch for the other content types.
Comment #4
juliangb CreditAttribution: juliangb commented#3: storm_views_nid-1463288-3.patch queued for re-testing.
Comment #6
juliangb CreditAttribution: juliangb commentedI don't think this is required.
@kfritsche, would you be able to elaborate on why this is needed?
Comment #7
juliangb CreditAttribution: juliangb commentedComment #8
kfritscheThis was needed so we can have the edit links at the end of the tables in views.
Somehow it seems there are two changes here.
1. Add 'additional_fields_for_access' so fields which are needed to check for correct access are added in the views handler.
2. Make nid accessible without using node relation.
Not sure if it works correctlt, when only using the first one.
Comment #9
juliangb CreditAttribution: juliangb commentedThanks for the explanation. I had thought that Views did the access checks automatically, but this isn't an area that I know much about.
I'm moving this issue into the D7 version, where Views are being used by default, and this will be easier to check.
Comment #10
juliangb CreditAttribution: juliangb commentedWe're moving to using node grants, which means the views obey access controls by default.