Changeset 61783

Timestamp:

03/02/2026 07:50:05 PM (8 months ago)

Author:

westonruter

Message:

General: Update wp.sanitize.stripTags() to return empty string when not passed a string.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/10994

Follow-up to r61585, r61578.

Props westonruter, jonsurrell, dmsnell, hugod.
See #64574, #64274.

Location:

trunk

Files:

• src/js/_enqueues/wp/sanitize.js (modified) (1 diff)
• tests/qunit/wp-includes/js/wp-sanitize.js (modified) (1 diff)

trunk/src/js/_enqueues/wp/sanitize.js

@@ -24 +24 @@
                  */
                 stripTags: function( text ) {
-                        if ( ! text ) {
+                        if ( 'string' !== typeof text ) {
                                 return '';
                         }

trunk/tests/qunit/wp-includes/js/wp-sanitize.js

@@ -18 +18 @@
 } );
 
-QUnit.test( 'stripTags should convert numbers to strings', function( assert ) {
-        const result = wp.sanitize.stripTags( 123 );
-        assert.strictEqual( result, '123', 'stripTags( 123 ) should return "123"' );
+QUnit.test( 'stripTags should return empty string for truthy non-strings', function( assert ) {
+        assert.strictEqual( wp.sanitize.stripTags( 123 ), '', 'stripTags( 123 ) should return ""' );
+        assert.strictEqual( wp.sanitize.stripTags( true ), '', 'stripTags( true ) should return ""' );
+        assert.strictEqual( wp.sanitize.stripTags( [ 6, 7 ] ), '', 'stripTags( [ 6, 7 ] ) should return ""' );
+        assert.strictEqual( wp.sanitize.stripTags( { foo: 'bar' } ), '', 'stripTags( ( { foo: \'bar\' } ) should return ""' );
 } );