Problem/motivation:

The commerce_vantiv.eprotect.js has code that delegates the checkout form's submit button, but does not detach that functionality e.g. when a vantiv payment gateway form is replaced by another, causing its logic to still apply to the checkout form when it shouldn't.

Proposed resolution:

  1. Add the click handler only when the required form element(s) exist.
  2. Remove the click handler in drupalBehaviors::detach.

This is tricky behavior but it should be testable. I'll leave that for another issue, though.

Comments

steveoliver created an issue. See original summary.

  • steveoliver committed 2e9280c on 8.x-1.x
    Issue #2915930 by steveoliver: Attach and detach eProtect functionality...
steveoliver’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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