jquery.masonry.js Not Found

[pondermatic]

When SCRIPT_DEBUG is true and the --dev option is used with grunt build , the un-minified version of jQuery Masonry is requested but not found.

[pondermatic]

This is a followup to #45869.

[isabel_brison]

Thanks for the patch @pondermatic! However, the issue has since been fixed: ​https://github.com/WordPress/wordpress-develop/blob/master/Gruntfile.js#L211, so I'm closing this ticket.

[SergeyBiryukov]

Replying to isabel_brison:

However, the issue has since been fixed: ​https://github.com/WordPress/wordpress-develop/blob/master/Gruntfile.js#L211

Hmm, the line that the patch removes is still there, so it's not clear to me why or how this is fixed :)

Reopening for a closer review.

[isabel_brison]

Hmm, the line that the patch removes is still there, so it's not clear to me why or how this is fixed :)

Whoops, that's right! Not sure how I jumped to that conclusion :/

So I tested and the patch applies cleanly and works correctly. It can be checked by enabling Twenty Fourteen, and then adding a widget to the Footer Widget Area in the Customizer. Without the patch, jquery.masonry.js returns a 301. With the patch, it returns a 200.

One thing I noticed is this unminified script is requested whatever the value of SCRIPT_DEBUG; instead it seems to depend on whether $wp_version has -src. It may be tangential to this ticket, but I'm wondering why that's the case.

[azaozz]

The proper fix seems to be to fix the URL in script-loader.php. WP doesn't include non-minified version of jquery-masonry (the Masonry v2 shim) in the build files. Looking at Gruntfile.js not including it seems to have been a decision, not an error. Fix would be to replace $dev_suffix with .min at ​https://github.com/WordPress/wordpress-develop/blob/master/src/wp-includes/script-loader.php#L842.

[desrosj]

Replying to azaozz:

The proper fix seems to be to fix the URL in script-loader.php. WP doesn't include non-minified version of jquery-masonry (the Masonry v2 shim) in the build files.

I agree with this. Because the v2 release is only included as a courtesy (developers should actually switch to using the masonry script handle instead), hard coding .min would fix this error without adding another unminified file to the release package.

There is also no unminified version of the masonry script handle either. But, this is a somewhat old library (no updates in 3 years), so I don't think we should add it now.

[desrosj]

Introduced in [22228] as part of #20346.

[desrosj]

In 51094:

Script Loader: Fix 404 errors for the jquery-masonry script when SCRIPT_DEBUG is true.

The unminified version of jquery-masonry has never been included in Core. This hard codes the .min into the file source to prevent these errors.

Props pondermatic, isabel_brison, SergeyBiryukov, azaozz, desrosj.
Fixes #47353.