PHP warnings

  • Resolved pixtweaks

    (@pixtweaks)


    4 months, 3 weeks ago

    Hi, I turn on logging of all errors and I see from your plugin this

    Error 2 - E_WARNING

    fopen(/wp-content/plugins/mailoptin/src/connections/NinjaFormsConnect/integrations/WordPressUserRegistrationConnect.php): Failed to open stream: No such file or directory

    /wp-admin/includes/class-wp-filesystem-direct.php:66

    #1 /wp-admin/includes/class-wp-filesystem-direct.php(66): fopen()
    #2 /wp-content/plugins/mailoptin/src/connections/NinjaFormsConnect/Connect.php(93): WP_Filesystem_Direct->put_contents()
    #3 /wp-content/plugins/mailoptin/src/connections/NinjaFormsConnect/Connect.php(53): MailOptin\NinjaFormsConnect\Connect->generate_classes()
    #4 /wp-includes/class-wp-hook.php(324): MailOptin\NinjaFormsConnect\Connect->register_nf_actions()
    #5 /wp-includes/plugin.php(205): WP_Hook->apply_filters()
    #6 /wp-content/plugins/ninja-forms/ninja-forms.php(790): apply_filters()
    #7 /wp-includes/class-wp-hook.php(324): Ninja_Forms->plugins_loaded()

    Another part is that this plugin is throwing this error on pages should not be doing anything, like on admin tools.php
    Can you please restrict where this plugin is loading its code only for parts is actually necessary? It’s loading 258 file and taking 4MB of ram on places it does nothing, just slowing down website. Please, for better and faster WP. Thank you!

    • This topic was modified 4 months, 3 weeks ago by pixtweaks.
Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author Collins Agbonghama

    (@collizo4sky)

    4 months, 3 weeks ago

    You should not be getting that error because the file mailoptin/src/connections/NinjaFormsConnect/Integrations/WordPressUserRegistrationConnect exists in our plugin folder. Can you confirm it does on your server?

    As for optimizing the plugin for memory usage, we are going to work on that. Thanks for bringing it to our attention.

    Thread Starter pixtweaks

    (@pixtweaks)

    4 months, 3 weeks ago

    Hi, I found the issue. One of folder in chain has capital letter and code is probing lowercase. It’s in file /wp-content/plugins/mailoptin/src/connections/NinjaFormsConnect/Connect.php

    $filename = dirname(__FILE__) . "/integrations/$key.php"; //original code
    $filename = dirname(__FILE__) . "/Integrations/$key.php"; // correct with capital "I"

    I don’t understand structure of your plugin but generating classes should probably happened only in cron or when plugin is activated. Thank you!

    Plugin Author Collins Agbonghama

    (@collizo4sky)

    4 months, 3 weeks ago

    That code only runs if paraventure, the class is missing like in your case.

    Thanks for spotting the issue. We’ve applied the fix and made changes to improve performance.

    An update will go live next week.

Viewing 3 replies - 1 through 3 (of 3 total)

You must be logged in to reply to this topic.