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.
dereine is clever and his attached patch allows the node widget to work properly with references or other potential cases when $values->nid might not exist in the handler's render.
dereine++
Comment | File | Size | Author |
---|---|---|---|
#2 | vud-6.x-2.x-1253390-1.patch | 2.85 KB | seandunaway |
vud-6.x-2.x-views-fix-nid.patch | 1.22 KB | seandunaway | |
Comments
Comment #1
seandunaway CreditAttribution: seandunaway commentedComment #2
seandunaway CreditAttribution: seandunaway commentedRe-rolled to get comments working. Apparently it's not only with node references but with any view in Views 3.
Attached.
Comment #3
seandunaway CreditAttribution: seandunaway commentedApplies cleanly against 6.x-3.x-dev also.
Comment #4
marvil07 CreditAttribution: marvil07 commentedI think you are right. It seems like views behaviour have changed from 2 to 3. Now it do not add the primary key of the base table by default. So yes, we would need to change it to be explicit about the primary key we are using on our views field.
I'm now taking a look at the patch.
Thanks for reporting it and providing code :-).
Comment #5
marvil07 CreditAttribution: marvil07 commentedI have just pushed this patch to 6.x-3.x and 6.x-2.x.
What's new on the patch: same change for vud_node_handler_field_widget handler and minor detals on comments.
Thanks all,