3.18 - Filter to change the output method

[piotrbak]

User Story As a user, I’d like to be able to print the Google Fonts CSS as an internal style

Acceptance Criteria

• By default, the feature store CSS in the file, it'd be possible to change it with the filter and print it as internal CSS
• The feature will keep the original structure as data-wpr-hosted-gf-parameters argument

[Khadreal]

This should be pick up after #7063 is completed.

Scope a solution:

In the rewrite_fonts created in #7063 we check if the filter is true, if true we then the css will be printed as internal in the head tag. WP_Filesystem_Direct should be added to the class constructor of the Controller class which would have been created in previous task.

Sample code

$internal_styling = wpm_apply_filters_typed( 'boolean', 'rocket_internal_fonts_styling', false );

if( $internal_styling ) {
    return $this->set_font_internal_style( $html, $css_path );
}

private function set_font_internal_style( $html, $css_path ) {
    if ( ! preg_match( '#</title\s*>#', $html, $matches ) ) {
        return $html;
    }

    $title   = $matches[0];
    $internal_style = $title;

    if ( ! $this->filesystem->exists( $css_path ) ) {
        return $html;
    }

    $css = '<style data-wpr-hosted-gf-parameters>'
    $css .= $this->filesystem->get_contents( $css_path );
    $css .= '</style>';

    return $css;
}

[remyperona]

LGTM

[Miraeld]

Looks good to me as well