Auto_load set to yes?

  • Rookie

    (@alriksson)


    6 years, 3 months ago

    Why adding wpvivid as auto_load yes? Not needed to load everywhere especially not for a plugin that is only in use on execute.

    name value
    wpvivid_task_list 58489
    wpvivid_remote_list 15902

Viewing 15 replies - 1 through 15 (of 18 total)
1 2
  • Plugin Support nicholas2019

    (@nicholas2019)

    6 years, 3 months ago

    Hi @alriksson,

    Thanks for your feedback.

    We would appreciate it if you could more clearly decribe the issue and how did you get these values. After that, we will try to optimize our plugin.

    Regards.

    Thread Starter Rookie

    (@alriksson)

    6 years, 3 months ago

    SELECT 'autoloaded data in KiB' as name, ROUND(SUM(LENGTH(option_value))/ 1024) as value FROM wp_options WHERE autoload='yes'
UNION
SELECT 'autoloaded data count', count(*) FROM wp_options WHERE autoload='yes'
UNION
(SELECT option_name, length(option_value) FROM wp_options WHERE autoload='yes' ORDER BY length(option_value) DESC LIMIT 10)

    You should never load globally (auto_load set to yes) on each pages if not needed on majority of the pages. Your plugin is only needing it’s function once it execute.

    Plugin Support nicholas2019

    (@nicholas2019)

    6 years, 3 months ago

    Hi,

    Thanks for the information you provided. We didn’t set the auto_load to yes. However, we will check the plugin codes and figure it out.

    Regards.

    Thread Starter Rookie

    (@alriksson)

    6 years, 3 months ago

    Thanks! Keep me posted, since I will remove the plugin if this is not sorted. Does not need auto load for scheduled backups either. And I have the plugin disabled but it is still triggering auto loads in the db.

    Plugin Support nicholas2019

    (@nicholas2019)

    6 years, 3 months ago

    The issue is forwarded to our dev team. If anything updated, I will let you know.

    Regards.

    Thread Starter Rookie

    (@alriksson)

    6 years, 3 months ago

    Thanks!

    Plugin Support nicholas2019

    (@nicholas2019)

    6 years, 3 months ago

    Hi @alriksson,

    We have checked the codes and found out auto_load is set to yes by default when using update_option. Thanks for your feedback. The auto_load of options will be set to no in upcoming release.

    Regards.

    Thread Starter Rookie

    (@alriksson)

    6 years, 3 months ago

    Sounds great!

    May I ask if during uninstall do you clean up all db stored settings related to your plugin?

    Plugin Support nicholas2019

    (@nicholas2019)

    6 years, 3 months ago

    In general, we clean up all datas generated by our plugin. But some new options aren’t removed yet because of the codes update. The upcoming version will fix this as well.

    Regards.

    Thread Starter Rookie

    (@alriksson)

    6 years, 3 months ago

    Ok please make sure cleaning such as well. At least give option in the admin/settings panel to fully clean any trace and data from the plugin if deleted. Not nice to keep on loading db options once a plugin is deleted as an example.

    Plugin Support nicholas2019

    (@nicholas2019)

    6 years, 3 months ago

    We will. Thanks for your suggestions.

    Thread Starter Rookie

    (@alriksson)

    6 years, 3 months ago

    Let me know once these things is sorted, thanks.

    Plugin Support nicholas2019

    (@nicholas2019)

    6 years, 3 months ago

    Please update to version 0.9.30. After trying, please let us know if the issues are fixed.

    Regards.

    Thread Starter Rookie

    (@alriksson)

    6 years, 3 months ago

    I saw the autoload is set to no, great!

    Is all other suggestions for improvements coming in next path?

    Plugin Author wpvivid

    (@wpvivid)

    6 years, 3 months ago

    Hi, @alriksson
    Yes, thank you:)

Viewing 15 replies - 1 through 15 (of 18 total)
1 2

The topic ‘Auto_load set to yes?’ is closed to new replies.