• Resolved Appinco

    (@appinco)


    Hi Raul,

    First and foremost I wish to thank you for upkeeping this longstanding support for this free plugin which actually is better than all other (paid) solutions available today.

    My reason for opening this topic is that it I cannot figure what’s going on with the plugin gravity forms. No matter how I configure FVM, ignore all gravity form scripts and even deregister all gravity forms scripts so they do not load at all, when the plugin is activated the scripts will always be double in the generated header.js and footer.js.

    By the way the plugin is written the scripts by default do not load deferred and throughout the dom. This is solved by applying the snippets in the following gist (https://gist.github.com/eriteric/5d6ca5969a662339c4b3), yet has no impact on FVM.

    When the gravity forms scripts are deregistered and no longer load, still the FVM generated cache is splitted to header.js + footer.js. Only when gravity forms is deactivated does the merging of all scripts go into a single footer.js.

    To confirm it wasn’t influenced by other plugins or the theme I set up a clean wordpress env with only the 2 plugins FMV + gravity forms and twentytwenty theme.

    Do you know what’s going on here and how we can get gravity forms included into the single footer.js? Or otherwise somehow completely exclude gravity forms from FVM, since apparently just ignoring the scripts isn’t enough?

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Raul P.

    (@alignak)

    Hello,

    Thanks for the message.

    Several of those questions are addressed on our faqs:
    https://www.ads-software.com/plugins/fast-velocity-minify/faq/

    Some others, I’m not sure I understand.

    Are gravity forms js or css files being included twice on the page, one on the header and another on the footer? And are they exactly the same files?

    When you deregister gravity forms, you will still have header and footer scripts, provided something else exists that needs to me merged both in the header and footer.
    Are any of those header or footer files empty? If not, then it’s correct.

    FVM will merge all files that are enqueued in the header into the header, and the ones in the footer into the footer.
    It’s up to the plugin authors to decide where do they want to enqueue the scripts.

    You cannot always move all scripts to the footer.
    Some, may need to set variables and functions before some inlined content on the page, and some specifically need to do the opposite.

    The order of scripts matter.
    Do not try to move them around unless you are absolutely sure it’s not causing any undefined or other conflicts.

    If you ignore gravity forms files on the ignore list, FVM will leave them in place.

    If FVM is not merging the scripts, they are probably not enqueueing the scripts properly, or in the right hook location, or they are using a different priority.

    Enqueuing scripts should follow the official recommendation, to be detectable by FVM.

    https://developer.www.ads-software.com/themes/basics/including-css-javascript/
    https://www.wpbeginner.com/wp-tutorials/how-to-properly-add-javascripts-and-styles-in-wordpress/

    You could use your functions.php file to deregister and dequeue all related css and js files from gravity forms, and then you could enqueue them properly following the official method.
    You may need to change the handle name in some cases, as well.

    I’ll add that plugin to my list of tests for the next release, but what I said still applies.

    Thread Starter Appinco

    (@appinco)

    Hello,

    Let me clarify a bit more on the issue at hand, all scripts are properly enqueued in the wp_enqueue_scripts hook in the theme and activated plugins (just a couple). These have no issues with merging in the footer and are properly minified. The scripts as enqueued in the theme are enqueued into the footer as following;

    
    add_action( 'wp_enqueue_scripts', 'enqueue_scripts' );
    function enqueue_scripts() {
        wp_enqueue_script( 'theme-mobile', $base.'/resource/js/main.mobile.js', array('jquery'), null, true );
        // More enqueued scripts
    }
    

    This works without issues. However the moment I activate the plugin gravity forms all/most other scripts in the theme and plugins no longer are merged/processed on pages where gravity forms does not load/enqueue any scripts. This is very strange because on pages where there do are forms and thus make gravity forms enqueue scripts suddenly all theme and other plugin scripts do load & merge fine.

    Gravity forms only enqueues scripts on pages that use it’s forms through shortcodes. I am unable to figure out why it is that all scripts are merged fine on pages where gravity forms does load scripts, yet scripts are not merged on pages where gravity forms does not load scripts. (Forcefully deregistering all scripts enqueued by the plugin does not seem to help)

    Below is a codeblock of how the plugin enqueues it’s scripts, which seems to be properly enqueued to me.

    
    /**
     * Register needed hooks.
     *
     * @since 2.4.10
     */
    public function init() {
    
    	parent::init();
    
    	add_action( 'wp_enqueue_scripts', array( $this, 'maybe_enqueue_form_scripts' ) );
    
    }
    
    
    /**
     * Parse current post's blocks for Gravity Forms block and enqueue required form scripts.
     *
     * @since  2.4.10
     */
    public function maybe_enqueue_form_scripts() {
    
    	global $wp_query;
    
            // Some truncated code here
    
    	foreach ( $wp_query->posts as $post ) {
    
                    // Some truncated code here
    
    		// Enqueue scripts for found forms.
    		foreach ( $form_ids as $form_id => $ajax ) {
    			$form = GFAPI::get_form( $form_id );
    			GFFormDisplay::enqueue_form_scripts( $form, $ajax );
    		}
    
    	}
    
    }
    
    
    public static function enqueue_form_scripts( $form, $ajax = false ) {
    
    	// adding pre enqueue scripts hook so that scripts can be added first if a need exists
    	/**
    	 * Fires before any scripts are enqueued (form specific using the ID as well)
    	 *
    	 * @param array $form The Form Object
    	 * @param bool  $ajax Whether AJAX is on or off (True or False)
    	 */
    	gf_do_action( array( 'gform_pre_enqueue_scripts', $form['id'] ), $form, $ajax );
    
    	if ( ! get_option( 'rg_gforms_disable_css' ) ) {
    
    		wp_enqueue_style( 'gforms_reset_css' );
    
    		if ( self::has_datepicker_field( $form ) ) {
    			wp_enqueue_style( 'gforms_datepicker_css' );
    		}
    
    		wp_enqueue_style( 'gforms_formsmain_css' );
    		wp_enqueue_style( 'gforms_ready_class_css' );
    		wp_enqueue_style( 'gforms_browsers_css' );
    
    		if ( is_rtl() ) {
    			wp_enqueue_style( 'gforms_rtl_css' );
    		}
    	}
    
    	if ( self::has_conditional_logic( $form ) ) {
    		wp_enqueue_script( 'gform_conditional_logic' );
    	}
    
    	if ( self::has_datepicker_field( $form ) ) {
    		wp_enqueue_script( 'gform_datepicker_init' );
    	}
    
    	if ( $ajax || self::has_price_field( $form ) || self::has_password_strength( $form ) || GFCommon::has_list_field( $form ) || GFCommon::has_credit_card_field( $form ) || self::has_conditional_logic( $form ) || self::has_currency_format_number_field( $form ) || self::has_calculation_field( $form ) || self::has_recaptcha_field( $form ) || self::has_checkbox_field( $form, true ) || self::has_js_merge_tag( $form ) || GFCommon::has_repeater_field( $form ) ) {
    		wp_enqueue_script( 'gform_gravityforms' );
    	}
    
    	if ( GFCommon::has_multifile_fileupload_field( $form ) ) {
    		wp_enqueue_script( 'plupload-all' );
    	}
    
    	if ( self::has_fileupload_field( $form ) ) {
    		wp_enqueue_script( 'gform_gravityforms' );
    		GFCommon::localize_gform_gravityforms_multifile();
    	}
    
    	if ( self::has_enhanced_dropdown( $form ) || self::has_pages( $form ) ) {
    		wp_enqueue_script( 'gform_json' );
    		wp_enqueue_script( 'gform_gravityforms' );
    	}
    
    	if ( self::has_character_counter( $form ) ) {
    		wp_enqueue_script( 'gform_textarea_counter' );
    	}
    
    	if ( self::has_input_mask( $form ) ) {
    		wp_enqueue_script( 'gform_masked_input' );
    	}
    
    	if ( self::has_enhanced_dropdown( $form ) && ! wp_script_is( 'chosen' ) ) {
    		wp_enqueue_script( 'gform_chosen' );
    	}
    
    	if ( self::has_enhanced_dropdown( $form ) ) {
    		if ( wp_script_is( 'chosen', 'registered' ) ) {
    			wp_enqueue_script( 'chosen' );
    		} else {
    			wp_enqueue_script( 'gform_chosen' );
    		}
    	}
    
    	if ( self::has_placeholder( $form ) ) {
    		wp_enqueue_script( 'gform_placeholder' );
    	}
    
    	/**
    	 * Fires after any scripts are enqueued (form specific using the ID as well)
    	 *
    	 * @param array $form The Form Object
    	 * @param bool  $ajax Whether AJAX is on or off (True or False)
    	 */
    	gf_do_action( array( 'gform_enqueue_scripts', $form['id'] ), $form, $ajax );
    
    	// enqueue jQuery every time form is displayed to allow 'gform_post_render' js hook
    	// to be available to users even when GF is not using it
    	wp_enqueue_script( 'jquery' );
    
    	if ( wp_script_is( 'gform_gravityforms' ) ) {
    		wp_localize_script( 'gform_gravityforms', 'gf_global', GFCommon::gf_global( false, true ) );
    	}
    
    }
    
    • This reply was modified 4 years, 9 months ago by Appinco. Reason: Corrected code blocks
    Plugin Author Raul P.

    (@alignak)

    Hi, at the moment I am a bit busy to look into this, but I’ll certainly test this on my end.

    Can you tell me which theme you are using, and which other plugins you are using?

    Plugin Author Raul P.

    (@alignak)

    OK, I installed a demo version using a comercial theme and added gravity forms to it.
    I then created a form and added it by shortcode to one of the pages.

    I am not able to replicate the issue you have.

    FVM is merging files both on pages with or without gravity forms.

    I can only speculate that either you have some other plugin messing up with the scripts or css order, or some cache is messed up.

    If you would like me to take a look, send me a mysqldump and a zip file of your site (you can delete the users table if you like) so I can replicate the environment.

    You can reach out via https://fastvelocity.com/ contact form and referencing this topic.

    Thanks

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Gravity forms unable to merge’ is closed to new replies.