The form where admins can enter payments uses a custom theme function. This function doesn't output the form_build_id element - this makes it impossible to add ajax to the form, because it can't be retrieved from the cache.

CommentFileSizeAuthor
#3 1540750-remove-theme-payment-form.patch3.18 KBlongwave
#1 1540750-payment-form-build-id.patch597 byteswodenx

Comments

wodenx’s picture

wodenx commented
Status: Active » Needs review
StatusFileSize
new 1540750-payment-form-build-id.patch597 bytes
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

I would like to drop that theme function entirely if possible, or at least call drupal_render_children($form) instead of explicitly rendering all fields.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
StatusFileSize
new 1540750-remove-theme-payment-form.patch3.18 KB

The attached patch uses Form API and CSS to replace the broken theme function.

Where are you using Ajax on this form?

wodenx’s picture

wodenx commented

In connection with http://drupal.org/project/uc_custom_payment, planning to allow admins to enter payment details when adding a custom payment, so need to add the form elements when that method is selected.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Fixed

Committed #3.

Status: Fixed » Closed (fixed)

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