When creating or editing a Registration Type, the "Hold expiration hours" and "Hold expiration state" are not saving. This prevents the expire feature from working. Patch incoming!

CommentFileSizeAuthor
#2 fix_registration_type_expire_settings_not_saving-2818141-2.patch823 bytesanemirovsky

Comments

anemirovsky created an issue. See original summary.

anemirovsky’s picture

anemirovsky commented
Status: Active » Needs review
StatusFileSize
new fix_registration_type_expire_settings_not_saving-2818141-2.patch823 bytes

This patch saves the held_expire and held_expire_state values, which corrects this problem.

rwilson0429’s picture

rwilson0429 commented

Patch in #2 works. In my case, when editing a Registration Type, the "Use a token to set the Open Date" and "Use a token to set the Open Date" were not saving as well. This patch fixed that.

However, unchecking the check boxes alone did not do it. I had to remove the tokens completely from the open and close date fields for this work.

Thanks for the patch.

mpoloskey’s picture

mpoloskey commented
Status: Needs review » Reviewed & tested by the community

Patch #2 works,

ph7’s picture

ph7 commented

When will this patch be committed?

fonant’s picture

fonant commented

Might be nicer to have these two items as additional fields in the registration_type table? Would need a hook_update_N() to add them.

gcb’s picture

gcb
he/him
Location Portland, OR
commented
Status: Reviewed & tested by the community » Fixed

Committed with minor modification to include any future data elements automatically. Thanks for the patch!

  • gcb committed d781b8e on 7.x-2.x authored by anemirovsky 
    Issue #2818141 by anemirovsky: Registration Type Expire Settings Not...

Status: Fixed » Closed (fixed)

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