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

filter-hook-argument-issue #3082

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

mohammaddanish-cedcoss
Copy link

wc_stripe_force_save_source, This filter has been defined with 2 arguments at 3 places but at one place it is defined with one argument,

at these places, the filter has been defined with two arguments

includes/class-wc-gateway-stripe.php at line no. 433 includes/abstracts/abstract-wc-stripe-payment-gateway.php at line no. 811 includes/abstracts/abstract-wc-stripe-payment-gateway.php at line no. 1304

So need to correct this code because it is giving the fatal error, when try to use add_filter function

Fixes #

Changes proposed in this Pull Request:

Testing instructions


  • Covered with tests (or have a good reason not to test in description ☝️)
  • Added changelog entry in both changelog.txt and readme.txt (or does not apply)
  • Tested on mobile (or does not apply)

Post merge

wc_stripe_force_save_source, This filter has been defined with 2 arguments at 3 places but at one place it is defined with one argument,

at these places, the filter has been defined with two arguments

includes/class-wc-gateway-stripe.php at line no. 433
includes/abstracts/abstract-wc-stripe-payment-gateway.php at line no. 811
includes/abstracts/abstract-wc-stripe-payment-gateway.php at line no. 1304

So need to correct this code because it is giving the fatal error, when try to use add_filter function
@github-actions github-actions bot added the Stale label Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant