dynamic_avia making WAY too many calls

Hi,

A few releases ago we added a unique timestamp to the merged filename because when updating the content of a CSS or js file or of Custom CSS option without changing theme version the generated hash for the filename remained unchanged and browsers did not recognize to reload the file but used the cached version instead until this cache expires – means the changes are not visible to visitors.

The problem you have is server related. It seems that your server is caching WP database options and returning that cache instead of the last updated values.
As we save the generated filenames in WP database options Enfold gets the wrong info that the merged files do not exist and therefore creates new ones on every pageload.

For the next version (4.7.2.1 or later) I added a filter where you can remove the timestamp.

This means that each created file has the same filename and overwrites the already existing one until your server returns the correct DB value that the file exists and Enfold can stop creating new files.

To functions.php of your child theme add the following (https://github.com/KriesiMedia/enfold-library/blob/master/actions%20and%20filters/Performance/avf_merged_files_unique_id.php):

function my_custom_merged_files_unique_id( $uniqid, $file_type, $data, $enqueued, $file_group_name, $conditions )
{
return '';
}

add_filter( 'avf_merged_files_unique_id', 'my_custom_merged_files_unique_id', 10, 6 );

Best regards,
Günter

Hi,

Did you update to 4.7.3 and clear server cache?

Best regards,
Günter

Hi,
@dorisa Günter asked me to check your installed files via FTP, so I replied to your other post because it was your and you can add info in the Private Content area there, this post is not yours so it’s not private.
Lets please continue there.

Best regards,
Mike

Hi,

@Illingco if you do not want the theme to use a timestamp with the file merging and compression then you would continue to use this filter. I have tested it on my install and the filter works correctly, for example I replaced the timestamp with “my_stamp” using the filter like this:

function my_custom_merged_files_unique_id( $uniqid, $file_type, $data, $enqueued, $file_group_name, $conditions )
{
return 'my_stamp';
}
add_filter( 'avf_merged_files_unique_id', 'my_custom_merged_files_unique_id', 10, 6 );

and this is the results:

So the steps I took were to add the above filter then go to Performance > File Compression and enable “Delete old CSS and JS files” at the bottom of the page, enable css & js file merging and compression, and “save all changes”, then go to the front page and reload.
If css & js file merging and compression was already enabled, then go to the Quick CSS and add a blank space to make the “save all changes” clickable and change the css in the theme to force a new file to be created when the front page is reloaded.
Another example of the filter being used, I saved a copy of the merged files and forced the theme to create a new file, in the screenshot notice the time of the file, these files have the same name and would have overwritten each other because there is no timestamp.

As for your update error, this looks like you tried to overwrite the theme folder instead of deleting it and creating a new theme folder. Is this correct?

Best regards,
Mike