Performance request/issues

  • Resolved Forge12 Interactive GmbH

    (@forge12)


    7 years, 3 months ago

    Hey there,

    we are currently working with your plugin and noticed that your plugin fires a few ajax requests even if they are not required. This results into some loading issues on the backend system.

    The function I’m talking about is placed within the wp-gotowebinar.php file:
    function wp_gotowebinar_send_tinymce_data(){}

    All ajax functions require a active tinyMCE editor, therefor i would suggest to validate if there is a valid tinymce editor before firing the ajax requests.

    Would that be possible?

    Best regards and keep on the good work.

    Best Regards
    Marc

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Northern Beaches Websites

    (@northernbeacheswebsites)

    7 years, 3 months ago

    Hi @forge12,

    We will eventually be removing support for tinymce. At the moment we need it to fire it pretty regularly in the admin area because it needs to work on all posts, pages and custom post types. The scope of this function could be tidied up a bit, but I am not an expert on how to best do this. If you want to contribute code in this area that would be great.

    Thread Starter Forge12 Interactive GmbH

    (@forge12)

    7 years, 3 months ago

    Hi @northernbeacheswebsites

    thanks for your reply. Currently the tinymce functions are firing on every page, even if there is no tinymce editor available.

    I guess the first step would be to extend the “wp_gotowebinar_send_tinymce_data()” in “wp-gotowebinar.php” with the following line:

    
// starting line: 720
jQuery( document ).ready( function( $ ) {
  // new code, simple request if there is a tinymce editor defined.
  if(typeof(tinyMCE) == 'undefined'){
    return;
  }
  // your code 
  var data = {
	'action'	: 'get_timezones_list', // wp ajax action
	'security'	: '<?php echo $nonce; ?>' // nonce value created earlier
  };
  // ... more code...

    This will ensure that the 4 ajax calls only fired if there is a tiny mce editor available.

    An aditional update would be to merge the 4 ajax calls.

    If you have any further questions do not hesitate to contact me.

    Best Regards
    Marc

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

The topic ‘Performance request/issues’ is closed to new replies.