Alter the node/N/signups/admin signup summary fieldset with per-status limit data

dww - September 21, 2009 - 04:18
Project:Signup Status
Version:6.x-1.0-alpha2
Component:Signup status limit
Category:task
Priority:normal
Assigned:dww
Status:closed
Description

Once #578592: Cleanup + fix the "Signup summary" at the top of node/N/signups/admin and #581652: Add a {signup_log}.count_towards_limit column and adjust limit logic accordingly land, we're going to want to alter that summary form from signup_status_limit to display the right info about the limit type on the node, and if configured for per-status limits, info about the current limits...

#1

dww - September 21, 2009 - 04:44

This patch depends on #578592-2: Cleanup + fix the "Signup summary" at the top of node/N/signups/admin, but I think this looks pretty nice for both global vs. per-status signup limits.

AttachmentSize
583154-1.signup_status_limit_summary_alter.total_.png 70.97 KB

#2

dww - September 21, 2009 - 04:45

Sorry, here's the other screenshot (showing how it looks when the node is configured for per-status limits) and the patch...

AttachmentSize
583154-1.signup_status_limit_summary_alter.patch 2.63 KB
583154-1.signup_status_limit_summary_alter.status.png 73.14 KB

#3

mlsamuelson - September 21, 2009 - 05:15
Status:postponed» reviewed & tested by the community

I've tested the patch, after applying #578592: Cleanup + fix the "Signup summary" at the top of node/N/signups/admin and it works as advertised. I don't see any problems with it. :)

#4

dww - September 21, 2009 - 05:25
Status:reviewed & tested by the community» fixed

Committed to HEAD. Thanks!

#5

roball - September 27, 2009 - 00:32
Version:6.x-1.x-dev» 6.x-1.0-alpha2
Status:fixed» active

Hm, I don't have the columns "Status" and "Operations", as shown in the attached screenshot. Am I missing something?

AttachmentSize
Signup details.png 9.82 KB

#6

dww - September 27, 2009 - 01:44
Status:active» fixed

@roball: Please read the README.txt file included with signup_status. In particular, the "Setup" section. That's what you're missing.

#7

roball - September 27, 2009 - 11:41

Problem solved as discussed in #589030: No 'Edit signup' link on the node/N/signups/admin tab. Thanks for your help.

#8

System Message - October 11, 2009 - 11:50
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.