Plugin build for 4184b7e is ready 🛎️!

• Download build

@Soare-Robert-Daniel tested and it's working well 👍

@Soare-Robert-Daniel,

I tested this with the final merge and it doesn't work to me. Charts are getting refreshed only after one hour not each ten minutes.

I see that the code change added here: https://github.com/Codeinwp/visualizer/pull/1161/files#diff-6a2bceb3434041451d4f3359e058407647985c3f74f400623aeafe9675cd0b92R213

Change the Recurrence of visualizer_schedule_refresh_db to 10 minutes, however, there is also visualizer_schedule_import hook that remains 1 hour and because of it the chart doesn't get refresh.

I changed the code in my test instance to:

wp_clear_scheduled_hook( 'visualizer_schedule_refresh_db' );
wp_clear_scheduled_hook( 'visualizer_schedule_import' );
wp_schedule_event( strtotime( 'midnight' ) - get_option( 'gmt_offset' ) * HOUR_IN_SECONDS, apply_filters( 'visualizer_chart_schedule_interval', 'visualizer_ten_minutes' ), 'visualizer_schedule_refresh_db' );
wp_schedule_event( strtotime( 'midnight' ) - get_option( 'gmt_offset' ) * HOUR_IN_SECONDS, apply_filters( 'visualizer_chart_schedule_interval', 'visualizer_ten_minutes' ), 'visualizer_schedule_import' );

So added same for visualizer_schedule_import as you did for visualizer_schedule_refresh_db and then it worked - charts got refreshed after 10 minutes.

Can you fix this?

Thanks!

As an idea, 'visualizer_schedule_refresh_db' indirectly call 'visualizer_schedule_import'.

The route is like this:

• At every 10 minutes, 'visualizer_schedule_refresh_db' is called.
• This updates the schedule times for the charts; if one of the charts needs to be updated, this triggers this function

  visualizer/classes/Visualizer/Module/Setup.php

  Line 427 in 4184b7e

  $this->refresh_db_for_chart( null, $chart_id, true );

• And in case of remote CSV, it invokes the same hooks as the schedule for 'visualizer_schedule_import'

  visualizer/classes/Visualizer/Module/Setup.php

  Line 349 in 4184b7e

  do_action( 'visualizer_schedule_import' );

Technically, the second schedule is not needed unless the first schedule is missing a chart type to update that the second might not.

What type of source (along with the URL) did you use?

@Soare-Robert-Daniel, I used both the CSV and Database and it wasn't refresh in any of them, I think you can easily replicate it yourself as well. Let me know if it works for you

Hi @Soare-Robert-Daniel,

Any update here?

@vytisbulkevicius, here is the PR https://github.com/Codeinwp/visualizer-pro/pull/481

I also created this issue https://github.com/Codeinwp/visualizer-pro/issues/482

Sorry for the issue. My attention and understanding are not in the best shape 😞, I misinterpreted the process.

🎉 This PR is included in version 3.11.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀