Ticket #27907: 27907.4.diff

src/wp-includes/js/shortcode.js

@@ -37 +37 @@
                         // If we matched a leading `[`, strip it from the match
                         // and increment the index accordingly.
                         if ( match[1] ) {
-                                result.match = result.match.slice( 1 );
+                                result.content = result.content.slice( 1 );
                                 result.index++;
                         }
 
                         // If we matched a trailing `]`, strip it from the match.
                         if ( match[7] ) {
-                                result.match = result.match.slice( 0, -1 );
+                                result.content = result.content.slice( 0, -1 );
                         }
 
                         return result;
@@ -353 +353 @@
                         return text + '</' + options.tag + '>';
                 }
         });
-}());
- No newline at end of file
+}());

tests/qunit/wp-includes/js/shortcode.js

@@ -57 +57 @@
                 equal( result, undefined, 'foo shortcode not found when escaped with params' );
         });
 
+        test( 'next() should find shortcodes that are incorrectly escaped by newlines', function() {
+                var result;
+
+                result = wp.shortcode.next( 'foo', 'this has the [\n[foo]] shortcode' );
+                equal( result.index, 15, 'shortcode found when incorrectly escaping the start of it' );
+
+                result = wp.shortcode.next( 'foo', 'this has the [[foo]\n] shortcode' );
+                equal( result.index, 14, 'shortcode found when incorrectly escaping the end of it' );
+        });
+
+        test( 'next() should still work when there are not equal ammounts of square brackets', function() {
+                var result;
+
+                result = wp.shortcode.next( 'foo', 'this has the [[foo] shortcode' );
+                equal( result.index, 14, 'shortcode found when there are offset square brackets' );
+
+                result = wp.shortcode.next( 'foo', 'this has the [foo]] shortcode' );
+                equal( result.index, 13, 'shortcode found when there are offset square brackets' );
+        });
+
         test( 'next() should find the second instances of the shortcode when the first one is escaped', function() {
                 var result;
 
@@ -123 +143 @@
                 equal( result, 'this bar has the [[foo param="bar"]] shortcode escaped', 'escaped foo with params not replaced but unescaped foo replaced' );
         });
 
+        test( 'replace() should replace improperly escaped shortcodes that include newlines', function() {
+                var result;
+
+                result = wp.shortcode.replace( 'foo', 'this [foo] has the [[foo param="bar"]\n] shortcode ', shortcodeReplaceCallback );
+                equal( result, 'this bar has the [bar\n] shortcode ', 'escaping with newlines should not actually escape the content' );
+
+                result = wp.shortcode.replace( 'foo', 'this [foo] has the [\n[foo param="bar"]] shortcode ', shortcodeReplaceCallback );
+                equal( result, 'this bar has the [\nbar] shortcode ', 'escaping with newlines should not actually escape the content' );
+        });
+
         test( 'replace() should not replace the shortcode when it is an incomplete match', function() {
                 var result;