Does not remove capacity variable when deselected [#1081208]

Hey,

I had an event content type with a capacity field selected at first, however, at some point I decided to remove the capacity restriction, so I deselected the field from the event's content type edit interface.. Unfortunately, the cck_signup_field_capacity_event variable was not deleted from the db, so all subsequent signups were still checked against the field..

Should be easy enough to fix.

Cheers

Comments

Comment #1

jhedstrom

English

Portland, OR

CreditAttribution: jhedstrom commented 8 March 2011 at 19:25

Status:

Active

» Fixed

The cck_signup_get_remaining_capacity() function now checks if signups are enabled for the given event type. Committed to 6.x and 7.x.

Comment #2

jhedstrom

English

Portland, OR

CreditAttribution: jhedstrom commented 8 March 2011 at 19:31

And I should have mentioned, this still doesn't remove the variables, but it doesn't break if they remain set. If you want the variables removed, you can manually un-check/un-select them on the node type form.

Comment #3

arski CreditAttribution: arski commented 9 March 2011 at 09:13

hey,

that's what I did - I deselected the capacity field for my node type, but the value was not deleted from the DB variables table :(

Comment #4

arski CreditAttribution: arski commented 9 March 2011 at 12:06

Hey again,

I'm a bit confused by this - because the function already checked if the capacity field was enabled for the content type - only when it did this it checked if a particular variable was set (which never gets unset) so I would think the issue is still there? Or are you doing it based on the variable value now?

Cheers