Return empty string from wp.shortcode.replace() callback is ignored

[prionkor]

I was trying to just remove a shortcode by returning empty string from wp.shortcode.replace() callback.

For example

var text = "This is some content with shortcode. [foo]some content[/foo]

text = wp.shortcode.replace('foo', text, function(tag){
    
    // attributes and content values are broken.

    return '';

});

The callback function simply get ignored. No replacement happened for the shortcode. If I return anything other then empty string for example ' ' (string with space) it does replace the shortcode.

I am not sure if this behavior was intended or not.

[danielbachhuber]

I am not sure if this behavior was intended or not.

It's going to be hard to determine the intended behavior :( This code landed in r22004, a pretty hefty changeset.

[prionkor]

For me the replace function should be replace the shortcode no matter what is returned from the callback function. Returning empty string should work like removing the shortcode.

[swissspidy]

I'd say the type of result should be properly checked instead of doing something like return result ? left + result + right : match;. An empty string is a legit use case and AFAIK also possible using the PHP equivalent.

[talldanwp]

I've attached 35216.diff.

This change makes wp.shortcode.replace() behave more like its php counterpart do_shortcode

If the result of the callback is a string, number or truthy value, we replace the shortcode with that value (so therefore if an empty string is returned from the callback we'll see the desired behaviour).

For other falsey values that are not strings or numbers returned from the callback, the shortcode and shortcode content will be removed

Tests have been added for the various values that might be returned from the callback (empty string, null, undefined, 0, empty string)