signup link does not work in views

:) Thanks dww. I'll keep watching the issue queue.

woa that was fast, much faster than reasonable person could have ask for in light of the pro bono nature of the service.

I applied the patches and there was slight hick-up with the last hunk not working. I visually examined the original file, the patch file, and the reject file and still can't figure out why it wouldn't patch. So I went ahead and patched the last hunk manually.

Happy to report everything is working !!! :)

Yip yip hurray for dww. :D

Again, thank you very much dww.

With regards,

Tien Doan

====
below is the content of the reject file:
====

*************** class signup_handler_field_signup_node_l
*** 72,98 ****
}
}

- function check_access($tab, $node) {
- global $user;
- switch ($tab) {
case 'signup':
- return variable_get('signup_form_location', 'node') == 'tab' && user_access('sign up for content');

case 'list':
- return variable_get('signup_display_signup_user_list', 'signup') != 'none' && user_access('view all signups');

- case 'admin':
- case 'add':
- $admin_all = user_access('administer all signups');
- $admin_own = user_access('administer signups for own content') && ($node->uid == $user->uid);
- return ($admin_all || $admin_own);

- case 'broadcast':
- $email_all = user_access('email all signed up users');
- $email_own = user_access('email users signed up for own content') && $user->uid == $node->uid;
- return $email_all || $email_own;

}
}

}
--- 75,123 ----
}
}

+ function check_access($node, $tab) {
+ switch ($tab) {
case 'signup':
+ // Slightly complicated, since _signup_menu_access() is testing if we
+ // want the signup tab, but here we just want to know if the user can
+ // signup at all.
+ return user_access('sign up for content') && _signup_needs_output($node) && variable_get('signup_form_location', 'node') != 'none';

case 'list':
+ // Also can't use _signup_menu_access() since we don't have to assume
+ // a tab for this access check.
+ $view_all = user_access('view all signups');
+ $admin = _signup_menu_access($node, 'admin');
+ return (($view_all || $admin) && _signup_needs_output($node));
+
+ default:
+ // All the other tabs can just use _signup_menu_access() directly.
+ return _signup_menu_access($node, $tab);

+ }
+ }

+ function get_url($node, $tab, &$link_options) {
+ switch ($tab) {
+ case 'signup':
+ if (variable_get('signup_form_location', 'node') == 'node') {
+ $link_options['fragment'] = 'signup';
+ return "node/$node->nid";
+ }
+ break;
+
+ case 'list':
+ $user_list = variable_get('signup_display_signup_user_list', 'signup');
+ if ($user_list == 'signup' || $user_list == 'embed-view') {
+ $link_options['fragment'] = 'signup';
+ return "node/$node->nid";
+ }
+ break;

}
+ // If we didn't already special-case the URL, we know it's just a subtab
+ // on the "signups" tab for the node.
+ return "node/$node->nid/signups/$tab";
}

}