PHP Warning on admin-ajax.php

Question

PHP Warning on admin-ajax.php

Closed this issue 8 months ago · 8 comments

E_WARNING: Invalid argument supplied for foreach()
in remove_menu_page called at wp-content/plugins/10up-experience/includes/plugins.php (32)

I believe this is due to admin_init hook on https://github.com/10up/10up-experience/blob/develop/includes/plugins.php#L31

Answer 1 · 2018-08-17T00:32:55.000Z

@rickalee I haven't been able to reproduce this - could you give me some specific steps I could follow so I can make sure my imagined fix will work? I 100% believe you, looking at the code, and I believe changing to the admin_menu hook instead would fix it.

Answer 2 · 2018-08-17T15:32:29.000Z

@helen Found this in New Relic reports. Not replicated elsewhere. Will reach out to you via Slack with links due to confidentiality concerns.

Answer 3 · 2018-08-20T19:49:47.000Z

Just making a note that the admin_menu hook doesn't fix this because of the way custom menu pages are added. Will have to look into another solution.

Answer 4 · 2018-12-05T13:36:16.000Z

@rickalee @helen I'm also seeing this in New Relic. I'm happy to help debug and submit a PR.

Answer 5 · 2018-12-05T14:43:12.000Z

@ivanlopez Yes please! cc @tlovett1

Answer 6 · 2018-12-05T15:56:20.000Z

I've created a merge request that addresses this issue at #40

Answer 7 · 2021-09-25T22:49:08.000Z

Can we close this issue now the PR has been merged?

cc @tlovett1 @ivanlopez

Answer 8 · 2023-10-26T12:00:24.000Z

Closing as this was fixed