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

PHP Fatal error: Allowed memory size of 134217728 bytes exhausted #1404

Closed
ryanlabelle opened this issue Jun 22, 2017 · 4 comments
Closed

PHP Fatal error: Allowed memory size of 134217728 bytes exhausted #1404

ryanlabelle opened this issue Jun 22, 2017 · 4 comments

Comments

@ryanlabelle
Copy link

Issue description:

When using the Number control I get a server 500. This happens when I go up or down.

PHP Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 32 bytes) in plugins/kirki/field/class-kirki-field-number.php on line 69, referer: wp-admin/customize.php?return=%2Fembark%2Fwp-admin%2Fplugins.php&changeset_uuid=98e6aa69-3c26-4f60-a360-a32ff2a83e5d

Version used:

3.0.3

Using theme_mods or options?

Code to reproduce the issue (config + field(s))

Embark_Kirki::add_field( 'embark_theme', array(
    'type'        => 'number',
    'settings'    => 'themo_nav_top_margin',
    'label'       => esc_html__( 'Navigation Top Margin', 'embark' ),
    'description' => esc_html__( 'Set top margin value for the navigation bar', 'embark' ),
    'section'     => 'menu',
    'default'     => 19,
    'choices'     => array(
        'min'  => '0',
        'max'  => '300',
        'step' => '1',
    ),
    'output' => array(
        array(
            'element'  => '.navbar .navbar-nav',
            'property' => 'margin-top',
            'units'    => 'px',
        ),
        array(
            'element'  => '.navbar .navbar-toggle',
            'property' => 'top',
            'units'    => 'px',
        ),
        array(
            'element'  => '.themo_cart_icon',
            'property' => 'margin-top',
            'value_pattern' => 'calc($px + 12px)'
        ),
    ),
) );
@aristath
Copy link
Contributor

That's weird...

That last commit (16c1d05) should fix your issue, can you please test it?

@ryanlabelle
Copy link
Author

That worked! Thanks!!

@aristath
Copy link
Contributor

Better fix for this was added in cdccd55
Will be included in 3.0.4 later today.

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

3 participants
@aristath @ryanlabelle and others