It'd be nice to have the following hooks in the documentation:

  • hook_commerce_payment_transaction_insert()
  • hook_commerce_payment_transaction_update()
  • hook_commerce_payment_transaction_delete()
CommentFileSizeAuthor
#6 2630292-6-add_commerce-payment_hook_documentation.patch1.62 KBnvahalik
#5 2630292-5-add_commerce-payment_hook_documentation.patch1.62 KBnvahalik
#2 2630292-2-add_commerce-payment_hook_documentation.patch1.13 KBnvahalik

Comments

nvahalik created an issue. See original summary.

nvahalik’s picture

nvahalik commented
Status: Active » Needs review
StatusFileSize
new 2630292-2-add_commerce-payment_hook_documentation.patch1.13 KB
mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Commerce Sprint

Documentation FTW, PhpStorm hook autocomplete helper FTW.

rszrama’s picture

rszrama commented
Status: Reviewed & tested by the community » Needs work

I feel like we have another issue open for these sorts of generic entity API hooks for all entity types. We can go ahead and add Payment Transaction entity type hook docs through this one, but let's also add doc blocks for the presave and load hooks.

nvahalik’s picture

nvahalik commented
Status: Needs work » Needs review
StatusFileSize
new 2630292-5-add_commerce-payment_hook_documentation.patch1.62 KB

Updated patch to include the load hook. Presave is already included in the API file.

nvahalik’s picture

nvahalik commented
StatusFileSize
new 2630292-6-add_commerce-payment_hook_documentation.patch1.62 KB

Changed it from "entity ID" to transaction_id. Should be good to go now.

  • rszrama committed 3ab0edb on 7.x-1.x authored by nvahalik 
    Issue #2630292 by nvahalik: Add commerce_payment entity hook...
rszrama’s picture

rszrama commented
Status: Needs review » Fixed

Nice, I missed that presave was already in there. : P

Status: Fixed » Closed (fixed)

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