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

Warnings due to missing arguments #30

Closed
growdev opened this issue Oct 2, 2016 · 3 comments
Closed

Warnings due to missing arguments #30

growdev opened this issue Oct 2, 2016 · 3 comments

Comments

@growdev
Copy link
Contributor

growdev commented Oct 2, 2016

WordPress: 4.6.1
EDD: 2.6.8
EDD Metrics: 0.4.0

Loading the metrics page by going to Downalods > Metrics causes the following warnings:

[02-Oct-2016 17:38:09 UTC] PHP Warning:  Missing argument 2 for EDD_Metrics_Functions::metrics_batch_1() in /Users/daniel/code/growdev/edd-metrics/includes/class-edd-metrics-functions.php on line 84
[02-Oct-2016 17:38:09 UTC] PHP Stack trace:
[02-Oct-2016 17:38:09 UTC] PHP   1. {main}() /Users/daniel/sites/shopplugins/wp-admin/admin-ajax.php:0
[02-Oct-2016 17:38:09 UTC] PHP   2. do_action() /Users/daniel/sites/shopplugins/wp-admin/admin-ajax.php:91
[02-Oct-2016 17:38:09 UTC] PHP   3. call_user_func_array:{/Users/daniel/sites/shopplugins/wp-includes/plugin.php:524}() /Users/daniel/sites/shopplugins/wp-includes/plugin.php:524
[02-Oct-2016 17:38:09 UTC] PHP   4. EDD_Metrics_Functions->metrics_batch_1() /Users/daniel/sites/shopplugins/wp-includes/plugin.php:524
[02-Oct-2016 17:38:11 UTC] PHP Warning:  Missing argument 2 for EDD_Metrics_Functions::metrics_batch_2() in /Users/daniel/code/growdev/edd-metrics/includes/class-edd-metrics-functions.php on line 125
[02-Oct-2016 17:38:11 UTC] PHP Stack trace:
[02-Oct-2016 17:38:11 UTC] PHP   1. {main}() /Users/daniel/sites/shopplugins/wp-admin/admin-ajax.php:0
[02-Oct-2016 17:38:11 UTC] PHP   2. do_action() /Users/daniel/sites/shopplugins/wp-admin/admin-ajax.php:91
[02-Oct-2016 17:38:11 UTC] PHP   3. call_user_func_array:{/Users/daniel/sites/shopplugins/wp-includes/plugin.php:524}() /Users/daniel/sites/shopplugins/wp-includes/plugin.php:524
[02-Oct-2016 17:38:11 UTC] PHP   4. EDD_Metrics_Functions->metrics_batch_2() /Users/daniel/sites/shopplugins/wp-includes/plugin.php:524
@scottopolis
Copy link
Owner

Looking into this

growdev added a commit to growdev/edd-metrics that referenced this issue Oct 2, 2016
@growdev
Copy link
Contributor Author

growdev commented Oct 2, 2016

I've got a fix in a PR

@scottopolis
Copy link
Owner

Fixed in 526f302

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

2 participants