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.
First of all, thanks for all the hard work, New Zeal!
I'm opening a series of feature requests to track progress on this module.
The current widget assumes that users know the name of the View. In certain set-ups with non-technical users or restricted permissions, this can be hard to use, since users would have to "guess" the name of the View they're trying to reference. An additional widget which presents the options as a Select list would solve this issue.
Comment | File | Size | Author |
---|---|---|---|
#4 | offer_view_choice_as-2816567-4.patch | 6.76 KB | joekers |
Comments
Comment #2
NewZeal CreditAttribution: NewZeal as a volunteer commentedSuggestion noted. There are a number of improvements that can be done to the selection process. Thanks.
Comment #3
joekersI could do with this on a project I'm working on so I'm going to give it a try.
Comment #4
joekersI haven't had any chance to take a look at this until today.
Working patch attached but there's one thing I'm not sure of:
validateElement
removing this and the line that adds it (element_validate) gives this error:InvalidArgumentException: Value is not a valid entity. in Drupal\Core\Entity\Plugin\DataType\EntityReference->setValue()
Any feedback would be much appreciated :)
Comment #5
NewZeal CreditAttribution: NewZeal as a volunteer commentedI've pushed this with the alpha 6 release. Because this is still alpha and the plugin is a separate script, then it works to use this method for testing by the community. I also invite you to be a maintainer of the module. Let me know if you want to accept.
Comment #6
joekersYeah I'd like to be a maintainer of the module, thanks!
Comment #7
NewZeal CreditAttribution: NewZeal as a volunteer commentedOk, you have full privileges. Thanks. We'll work towards a beta version once we have dealt with this issue: https://www.drupal.org/node/2816571 .
Comment #8
joekersThanks. Sounds good to me.
Comment #9
joekersClosing this issue as the code has been committed and any issues can be reported separately (as with #2832518: "None" Option Throws Error When Trying To Remove Selected View).