The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Noticed that in pay.admin.inc on line 160 it says
$pmid = isset($pay_method->pmid) ? $pay_method->pmid : get_class($pay_method);
since get_class will only ever return a string I guess this isn't right??
So thought I'd propose the attached as a patch - there may well be a better way of getting the pmid at that point I'm figuring this out in amongst other work so apologies if I'm confused!
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | pay.admin_.patch | 646 bytes | paul.dambra |
| pay.admin_.patch | 646 bytes | paul.dambra |
Comments
Comment #1
paul.dambra commentedJust noticed while I'm in there that in
function pay_admin_method_form(&$form_state, $pay_method = NULL)there is referred to:
$form['#pay_method']and
$form['pay_method']I'm not sure which is right (or indeed if either is wrong although it does look it) so don't want to edit my patch in the wrong direction.
Comment #2
paul.dambra commentedSeems this was an ID-ten-T situation... and my custom plug in was at fault... apologies and facepalm