Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add proper text preferably donation amount to Paypal item title instead of custom amount text #2071

Closed
marutim opened this issue Sep 13, 2017 · 4 comments
Assignees
Milestone

Comments

@marutim
Copy link

marutim commented Sep 13, 2017

Issue Overview

It happens for both single or multi level forms. When a donation is processed with fee recovery checked and payment is done via paypal. The paypal item title is suffixed with the "custom amount text". This also happens even when the custom amount is disabled for the form.

For example if you set donation amount is $10 and fee recovery is $0.20 and Custom amount text is "More the better" the paypal item title will be Donation form name - "More
the better". When custom amount is disabled or custom amount text is not set, it uses the default custom amount text which is "Custom Amount"

Expected Behavior

Paypal item title should be suffixed with proper text, which can be the correct level for multi level form or no suffix or the donated total amount (including the recovery fee)

Current Behavior

Paypal item title is suffixed with the custom amount text

Possible Solution

In the function give_build_paypal_item_title(), the logic is set to consider the donation amount to custom when it doesn't match to any level amount. We need to make an exception for fee recovery addition.

Steps to Reproduce (for bugs)

  • Make a donation using the single or multi level form, check the fee recovery and process the donation via Paypal.
  • In the Paypal page, check for the item title.
  • Please review the screenshots for better understanding

Screenshots

paypal - multi level donation
paypal item title- multilevel
paypal- single donation
paypal itemtitle - single donation

@marutim marutim changed the title Add donation amount to Paypal item title instead of custom amount text Add proper text preferably donation amount to Paypal item title instead of custom amount text Sep 13, 2017
@mathetos
Copy link
Member

Excellently written issue @marutim Thanks!

@DevinWalker DevinWalker added this to the 1.8.14 milestone Sep 13, 2017
@mehul0810 mehul0810 mentioned this issue Sep 14, 2017
3 tasks
@marutim
Copy link
Author

marutim commented Sep 14, 2017

Thanks @mathetos :)

@ravinderk
Copy link
Collaborator

@mehul0810 Improve as deprecated function described in comment https://github.com/WordImpress/Give/pull/2075/files#diff-becfcb2300911b460438043c3c352ba3R640

@ravinderk ravinderk reopened this Sep 18, 2017
@mehul0810 mehul0810 mentioned this issue Sep 18, 2017
3 tasks
@DevinWalker
Copy link
Member

Good call with deprecating the function. Don't want to cause errors for folks who may be using it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants