Changeset 48096

Timestamp:

06/19/2020 10:06:28 PM (6 years ago)

Author:

SergeyBiryukov

Message:

Build/Test Tools: Replace the banner text from wp-emoji-loader.min.js in formatting.php during build process.

This ensures that /*! This file is auto-generated */ comment is not included on front end as part of the inline Emoji detection script.

Follow-up to [41271], [46589].

Props superpoincare, SergeyBiryukov.
Fixes #49861.

File:

• trunk/Gruntfile.js (modified) (4 diffs)

trunk/Gruntfile.js

@@ -1032 +1032 @@
                                                 {
                                                         match: /\/\/ START: emoji arrays[\S\s]*\/\/ END: emoji arrays/g,
-                                                        replacement: function () {
+                                                        replacement: function() {
                                                                 var regex, files,
                                                                         partials, partialsSet,
@@ -1058 +1058 @@
 
                                                                 // Sort the entities list by length, so the longest emoji will be found first.
-                                                                emojiArray = entities.split( '\n' ).sort( function ( a, b ) {
+                                                                emojiArray = entities.split( '\n' ).sort( function( a, b ) {
                                                                         return b.length - a.length;
                                                                 } );
@@ -1098 +1098 @@
                                         }
                                 ]
+                        },
+                        emojiBannerText: {
+                                options: {
+                                        patterns: [
+                                                {
+                                                        match: new RegExp( '\\s*' + BANNER_TEXT.replace( /[\/\*\!]/g, '\\$&' ) ),
+                                                        replacement: ''
+                                                }
+                                        ]
+                                },
+                                files: [
+                                        {
+                                                expand: true,
+                                                flatten: true,
+                                                src: [
+                                                        BUILD_DIR + 'wp-includes/formatting.php'
+                                                ],
+                                                dest: BUILD_DIR + 'wp-includes/'
+                                        }
+                                ]
                         }
                 },
@@ -1420 +1440 @@
                                 'includes:emoji',
                                 'includes:embed',
+                                'replace:emojiBannerText'
                         ] );
                 }