Changeset 48096

Timestamp:

06/19/2020 10:06:28 PM (4 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'
             ] );
         }