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

do_action ('qm/... never does anything #872

Open
kupietools opened this issue Jun 16, 2024 · 5 comments
Open

do_action ('qm/... never does anything #872

kupietools opened this issue Jun 16, 2024 · 5 comments

Comments

@kupietools
Copy link

I'm trying to use QM to log from some plugins I'm developing and it never logs anything. In fact it appears to be silently causing the plugin functions to fail. I know because adding BugFu logging steps immediately before do_action('qm/start','foo') or do_action('qm/stop','foo') works, but when I move the logging immediately after the do_action it never logs. 'qm/notice' doesn't do anything either.

My logs are also filling with these:
[16-Jun-2024 06:46:09 UTC] PHP Fatal error: Allowed memory size of 268435456 bytes exhausted (tried to allocate 970752 bytes) in /path/to/plugins/query-monitor/collectors/db_queries.php on line 246

@montchr
Copy link

montchr commented Jul 5, 2024

I am also seeing nothing logged when using the actions. Though I am not running into memory allocation issues.

@montchr
Copy link

montchr commented Jul 5, 2024

QM::debug(...) also results in no output.

@crstauf
Copy link
Contributor

crstauf commented Jul 8, 2024

@kupietools @montchr What version of QM are you using?

@montchr
Copy link

montchr commented Jul 12, 2024

@crstauf v3.16.3

@johnbillion
Copy link
Owner

It looks like this, and other reports recently, essentially boil down to the possibility of infinite recursion when translation functions are called. There are a few places in QM where a translation is loaded during the data collection steps, where it should really be deferred until the output.

I'm going to address this more thoroughly in a future release.

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

4 participants