[brianjhanson]

Description

Companion to craftcms/cms#20592, which removes getTriggerHtml() from core element actions. Two Commerce actions overrode it to write trigger JavaScript.

CreateSale
  before: getTriggerHtml() → JS that redirects to sales/new?purchasableIds=…
  after:  performAction()  → RedirectResponse(sales/new?purchasableIds=…)   (same as CreateDiscount)

SetDefaultVariant
  before: getTriggerHtml() → JS creating a trigger with bulk: false
  after:  removed; supportsBulk() === false already limits it to one selection

Evidence

• New test in tests/Feature/Http/Controllers/ProductIndexTest.php:

  perform CreateSale on [hoodie]
    → 200, json.redirect === cp/commerce/store-management/{store}/sales/new?purchasableIds={hoodie.id}
  

• Full suite (1,003 passed), ECS and PHPStan pass locally against current 6.x CMS.

Nothing here depends on the CMS PR, so it can merge first.

🤖 Generated with Claude Code