Redundant code and potential inaccurate check in script-loader.php

[MattyRob]

Since [44651] removed the build steps for a gzipped version of TinyMCE that has remained code in script-loader.php that checks if gzip is supported on the server, these checks are redundant and superfluous.

They may also result in the uncompressed files being used when the compressed files are available and could be used.

script-loader.php, line 57

$compressed = $compress_scripts && $concatenate_scripts && isset( $_SERVER['HTTP_ACCEPT_ENCODING'] )
                        && false !== stripos( $_SERVER['HTTP_ACCEPT_ENCODING'], 'gzip' ) && ! $force_uncompressed;

can are replaced with:

$compressed = $compress_scripts && $concatenate_scripts && ! $force_uncompressed;

The documentation further down may also need updating.

Trac ticket: https://core.trac.wordpress.org/ticket/61862

[hbhalodia]

Hi @MattyRob, Have raised the PR for the same.

Thank You,

[MattyRob]

@hbhalodia - patch looks good but inline documentation further down still references wp-tinymce.js.gz so that many also need adjusting.

[hbhalodia]

Hi @MattyRob @SergeyBiryukov, I have updated the inline documentation to not reference the gz. Can you please take a look if that is needs to be updated?

Thank You,

[MattyRob]

That looks good to me but would need a core committer to approve and merge it.

[SergeyBiryukov]

In 58902:

Script Loader: Remove redundant check in wp_register_tinymce_scripts().

Since removing the build steps for a gzipped version of TinyMCE, the check whether gzip is supported on the server is superfluous. It may also result in the uncompressed files being used when the compressed files are available and could be used.

Follow-up to [44114], [44651].

Props MattyRob, hbhalodia.
Fixes #61862.

Thanks for the PR! Merged in r58902.