Needs work
Project:
Signup
Version:
5.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
24 Jan 2008 at 16:05 UTC
Updated:
4 Feb 2013 at 13:06 UTC
Jump to comment: Most recent file
Comments
Comment #1
dwwa) New features only go in HEAD (5.x-2.x-dev)
b) You need to wrap those strings in t() so they're translatable.
c) Seems like a lot of code duplication -- perhaps we could have a simple helper method that handles special cases in outputting signup data?
Comment #2
stborchertAdded a new function to render the data and output the checkbox value as
t('Yes')ort('No').Data of all other elements is untouched (unless you want a
check_plain()).Comment #3
dwwAs expected, this doesn't apply after #137609: Theme function for signup_build_signup_data. I'll reroll now.
Comment #4
dwwBah, this is a can of worms in general. If the signup form is nested at all (the whole point of those recursive theme functions being recursive), this code doesn't work:
Now that #137609 is done, sites can at least fix this themselves if they want. I'm sort of tempted to won't fix, but if someone wants to work out a clean general solution, I'll take another look.
Comment #5
duaelfrThis version of Signup is not supported anymore. The issue is closed for this reason.
Please upgrade to a supported version and feel free to reopen the issue on the new version if applicable.
This issue has been automagically closed by a script.
Comment #6
simon georges commentedReverting recent closing.