CommentFileSizeAuthor
#10 duplicate_transaction_error-2160792-10.patch721 bytescoldcode
#7 3160792-prevent-duplicate-tranx-ref.patch601 bytestolu_
#6 160792-duplicate_transaction_reference_fixer.patch1010 bytescoldcode
#5 3160792-duplicate_transaction_reference_fixed.patch2.94 KBcoldcode
#4 0001-fix_duplicate_transaction_message.patch1.16 KBcoldcode
#2 3160792-duplicate_transaction_reference.patch0 bytescoldcode

Issue fork commerce_paystack-3160792

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

coldcode created an issue. See original summary.

coldcode’s picture

coldcode commented
StatusFileSize
new 3160792-duplicate_transaction_reference.patch0 bytes
tolu_’s picture

tolu_ commented
Status: Patch (to be ported) » Needs review
coldcode’s picture

coldcode commented
StatusFileSize
new 0001-fix_duplicate_transaction_message.patch1.16 KB
coldcode’s picture

coldcode commented
StatusFileSize
new 3160792-duplicate_transaction_reference_fixed.patch2.94 KB
coldcode’s picture

coldcode commented
StatusFileSize
new 160792-duplicate_transaction_reference_fixer.patch1010 bytes
tolu_’s picture

tolu_ commented
StatusFileSize
new 3160792-prevent-duplicate-tranx-ref.patch601 bytes

Good start #6, this avoids calling a new service and also keeps a unique refrence to the order

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented

Hello @tolu_, @coldcode
How can I reproduce the issue?

Thanks!

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented

Removed empty file

coldcode’s picture

coldcode commented
StatusFileSize
new duplicate_transaction_error-2160792-10.patch721 bytes

A duplicate transaction error is thrown when a user tries to make a payment. This error is thrown because the reference number already exists hence the need for uuid as a reference number.

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Assigned: Unassigned » i-trokhanenko
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Needs review » Reviewed & tested by the community

Patch #7 looks good to me, thanks!

i-trokhanenko opened merge request !1

  • i-trokhanenko committed be71e2a on 8.x-1.x 
    Issue #3160792 by coldcode, i-trokhanenko, tolu_: Duplicate Transaction...
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks!

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Component: E-commerce » Code
Assigned: i-trokhanenko » Unassigned

Status: Fixed » Closed (fixed)

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