Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.I sat down today to work on adding XML export functionality to Views Export module, and found that this is going to be incredibly tedious with all the various permission checking for "export views as X" permissions everywhere.
Proposal: make it just "export views" permissions, and then limit to specific roles in the access permissions on the view itself. Patch coming up.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | views_bonus-permissions-165067-1.patch | 4.04 KB | webchick |
Comments
Comment #1
webchickI should point out I ran this idea past Earl and he approves. Here's the patch.
Comment #2
webchickOh, I should explain some of this.
I took out the permission checking in a few of these functions, because this is already done for you in hook_menu, so it is redundant to do it twice.
Comment #3
robloachWhoops... It seems you're using HEAD instead of DRUPAL-5. There was a major overhaul in the DRUPAL-5 branch that fixed the design a bit. It would be good to merge this patch with that branch though, of course.
Should there be a note in HEAD stating that DRUPAL-5 is the latest branch?
Comment #4
robloachI'll merge them today, no worries. Thanks a lot!
Comment #5
robloachThank you!
Comment #6
(not verified) commented