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

Stream settings exclude error for big wp_users table #551

Closed
Katamo opened this issue May 23, 2014 · 2 comments · Fixed by #552
Closed

Stream settings exclude error for big wp_users table #551

Katamo opened this issue May 23, 2014 · 2 comments · Fixed by #552
Labels

Comments

@Katamo
Copy link

Katamo commented May 23, 2014

No sure if related with wp-stream but Im getting:

Fatal error: Allowed memory size of 268435456 bytes exhausted (tried to allocate 130968 bytes) in /wp-content/plugins/pods/includes/data.php on line 1033

When I try to open stream plugin settings, tab "Exclude": wp-admin/admin.php?page=wp_stream_settings&tab=exclude

My wp_users table has thousands of users. The problem disappear when I replace the current table with an empty table.

Disabling the field authors_and_roles by commenting lines 302-309 on settings.php solves the issue too.

/*array(
    'name'        => 'authors_and_roles',
    'title'       => esc_html__( 'Authors & Roles', 'stream' ),
    'type'        => 'select2_user_role',
    'desc'        => esc_html__( 'No activity will be logged for these authors and/or roles.', 'stream' ),
    'choices'     => self::get_roles(),
    'default'     => array(),
),*/
@frankiejarrett
Copy link
Contributor

@Katamo Thanks for isolating the bug for us.

It seems that the AJAX loading of users isn't being handled properly here. We have fixed this recently on the list table screen in #428 it could be that the same just needs to be done here in the Exclude tab.

The fix for this could possibly be done alongside #527 in the same PR.

@frankiejarrett
Copy link
Contributor

@Katamo This issue has been fixed and released in version 1.4.6 of Stream.

http://wordpress.org/plugins/stream/changelog/

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 a pull request may close this issue.

2 participants