Viewing 12 posts - 1 through 12 (of 12 total)
  • Author
    Posts
  • July 1, 2019 at 1:37 pm #1114888
    Jason

    Hi Team,

    While debugging an issue with Merge & Combine Scripts (see https://kriesi.at/support/topic/issues-with-style-and-script-merging/#post-1114878 ) I found that aviaAsset_avia-head-scripts never gets generated and always returns an error.

    My avia-head-scripts array looks like this:

    Array
(
    [hash] => avia-head-scripts-cfa788754642173dafbd830bd7969fdb
    [version] => 4.5.7
    [remove] => Array
        (
            [avia-compat-js] => Array
                (
                    [name] => avia-compat
                    [url] => https://na/wp-content/themes/enfold/js/avia-compat.js
                    [path] => wp-content/themes/enfold/js/avia-compat.js
                    [print] => 1
--snip--

    the print=>1 trips this line, which removes it from the content generation and I get no file.
    if(!$remove['print'])

    Any suggestions?

    July 4, 2019 at 12:27 pm #1115731
    Günter

    Hey Jason,

    Thank you for using Enfold.

    The file avia-head-scripts-xxxxxx only gets generated when you enqueue scripts to the header. Enfold enqueue all the scripts that are merged in footer.

    avia-compat.js is printed “hardcoded” in the head area. Therefore it is not necessary to generate the avia-head-scripts file.

    If you change the last parameter of wp_enqueue_script to false (e.g. for an alb element you use) and toggle the compression you will see that avia-head-scripts file is generated.

    Best regards,
    Günter

    July 4, 2019 at 1:04 pm #1115746
    Jason

    The constant error seems a little odd, but thanks for the response!

    I hope the other thread gets a response though as the merge and compress is not working on my staging site due to the SSL cert.

    Cheers,
    Jason

    July 4, 2019 at 5:26 pm #1115845
    Günter

    Hi,

    https://wordpress.stackexchange.com/questions/167898/is-it-safe-to-use-sslverify-true-for-with-wp-remote-get-wp-remote-post#answer-167910

    As stated in this answer – I’m not a system administrator but could you check with your hoster / or sysadmin if there could be a problem?

    We actually create the file and try to read it afterwards to make sure it is avaiable. Ignoring this as suggested in the other post could result in a broken site.

    Best regards,
    Günter

    July 4, 2019 at 5:27 pm #1115847
    Günter

    Hi,

    https://wordpress.stackexchange.com/questions/167898/is-it-safe-to-use-sslverify-true-for-with-wp-remote-get-wp-remote-post#answer-167910

    As stated in this answer – I’m not a system administrator but could you check with your hoster / or sysadmin if there could be a problem?

    We actually create the file and try to read it afterwards to make sure it is avaiable. Ignoring this as suggested in the other post could result in a broken site.

    Best regards,
    Günter

    July 5, 2019 at 12:47 am #1115942
    Jason

    I understand why you need to verify the compressed file is available, but your check should not mandate a signed cert; ie. you should allow self-signed certs when doing this check.

    Cheers,
    Jason

    July 5, 2019 at 11:31 am #1116014
    Günter

    Hi,

    In file enfold\config-templatebuilder\avia-template-builder\php\asset-manager.class.php around line 430 you find:

    
$request 	= wp_remote_get($file);

    Try to replace it with:

    
$request 	= wp_remote_get( $file, array( 'sslverify' => false ) );

    Hope this helps to fix the problem.

    Best regards,
    Günter

    July 7, 2019 at 2:40 am #1116353
    Jason

    Yes that helps. Is there a chance you can include the fix in the next version? At least make it optional.

    Cheers,
    Jason

    July 8, 2019 at 3:25 am #1116534
    Michael

    I’m having this issue as well. See https://kriesi.at/support/topic/issues-with-style-and-script-merging/#post-1116531

    I proposed that they add an option switch to the theme settings that allow the user to turn off and on the SSL verification, in the case of self-signed certs.

    July 8, 2019 at 3:23 pm #1116700
    Günter

    Hey!

    There will be a theme option “Http security level for checking readability of merged files” in Performace Tab in the next release.

    Regards,
    Günter

    July 8, 2019 at 4:39 pm #1116720
    Michael

    When is the next release scheduled?

    July 8, 2019 at 4:50 pm #1116729
    Günter

    Hi,


    @smartstartinc     – answered your topic.

    Best regards,
    Günter

  • Author
    Posts
Viewing 12 posts - 1 through 12 (of 12 total)
  • You must be logged in to reply to this topic.