Patch (to be ported)
Project:
Signup
Version:
7.x-1.x-dev
Component:
Views integration
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
8 Sep 2011 at 18:10 UTC
Updated:
15 Jan 2015 at 00:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jerenus commentedGood idea. I think that would be useful for the support of views and VBO in the future. Can you give us more information about your further ideas? We can discuss it here.
Comment #2
dkane commentedI too would love to see some signup functions incorporated into VBO. If I recall most of the functionality was available in 6.x - it just seems to have gotten lost in the port to 7.x as the ways VBO handles operations has changed. When you bring up the VBO admin view it says its trying to use a missing format, which seems to me like its looking for the old Views Bulk Operations Table display type. So maybe we just need to expose that functionality to the new bulk operations field? Granted I'm not a programmer, so that may be easier said than done.
Ideally the functionality I'd like to include would be:
Mark User as Attended
Mark User as Did Not Attend
Sign up user
Cancel Signup
I'm sure there are more that others would like to see as well, but these are the 4 that I am currently missing for my site. At the moment I am trying to build a work around by triggering rules with VBO that will do this - however I am having trouble figuring out the correct order of operations to do this with rules as it seems signup keeps looking for a signup data object, and I'm not quite sure what that is or how to use it effectively, although I believe that is a conversation for the signup_rules page.
Many thanks for this fantastic module and your upkeep of it. It is a crucial player in my site, and I am much appreciative of your hard work.
Comment #3
bhosmer commentedI came across this issue recently, and wanted to add some notes:
In
views/default_views/signup_user_vbo_admin_list.view.phpon line 121:it is obsolete in the way that Bulk Operations works now.
I changed it to:
And I can load the view fine now.
This makes the "missing plugin" error go away, but really solves nothing in enabling bulk operations on attendance.
I started following the Handbook Page for VBO and hacked together a mockup in the
signup.module.It looks like we'll need a new bulk action, and an update to the signup_log table with some joins to update all users signed up to a particular nid.
Maintainers:
Do you have any thoughts or ideas with implementing this and updating this issue?
Comment #4
bhosmer commentedI'm updating this to a feature request, since the current module doesn't seem to support VBO just yet.
Comment #5
bhosmer commentedComment #6
sgabe commentedFirst of all, thank you for contributing! I just took a quick look on the patch and noticed some issues with the code in general.
Please, watch out for trailing whitespaces.
This modification of the watchdog entry seems unnecessary.
Use object, NOT "stdClass".
Functions without return values must not have @return documentation.
You can read about the code formatting requirements (and much more) in the Coding standards, especially in the Documentation and comment standrads section.
Comment #7
bhosmer commentedThanks for catching those typos.
Comment #8
spelcheck commentedHad to change
$context['rows']->sidtoreset($context['rows'])->sidin the bulk view functions because $context['rows'] contained each result in a separate numbered array. reset() shows the first (and only) row array for each iteration of the functions. Also, I remember it appeared some of the original cancel, attending and not-attending functions were using FALSE and TRUE to write to the signup_log table, which had to be changed to numeric 0 and 1 respectively. Sorry I don't have time to supply a patch.