Changeset 12341 for sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-blocks.php

Timestamp:

12/16/2022 04:55:38 AM (4 years ago)

Author:

dd32

Message:

Support Forums: Blocks: Store emoji's as emojis, not as img.emoji.

This allows emojis to be rendered by the viewer, rather than the conversions being based on the posters browser support.

See #6608.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-blocks.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-blocks.php

@@ -54 +54 @@
 
                 // Implement Blocks-no-empty-replies.
-                add_filter( 'bbp_new_reply_pre_content', [ $this, 'bbp_new_reply_pre_content' ] );
+                add_filter( 'bbp_new_reply_pre_content', [ $this, 'no_empty_block_replies' ] );
+                add_filter( 'bbp_new_topic_pre_content', [ $this, 'no_empty_block_replies' ] );
+
+                // Reverse twemoji replacements. Before bbPress sanitization gets to it.
+                add_filter( 'bbp_new_reply_pre_content', [ $this, 'reverse_twemoji_upon_save' ], 5 );
+                add_filter( 'bbp_new_topic_pre_content', [ $this, 'reverse_twemoji_upon_save' ], 5 );
         }
 
@@ -286 +291 @@
          * Don't allow empty block replies.
          */
-        public function bbp_new_reply_pre_content( $content ) {
+        public function no_empty_block_replies( $content ) {
                 // Remove HTML tags.
                 $no_html_content = wp_strip_all_tags( $content, true );
@@ -303 +308 @@
 
         /**
+         * Reverse twemoji upon save.
+         *
+         * @see https://core.trac.wordpress.org/ticket/52219
+         */
+        public function reverse_twemoji_upon_save( $content ) {
+                // <img ... class="emoji" alt="emojihere" ... />
+                if ( false === str_contains( $content, 'emoji' ) ) {
+                        return $content;
+                }
+
+                $content = wp_unslash( $content );
+
+                // Replace all img.emoji with the alt text.
+                $content = preg_replace(
+                        '~<img[^>]+class="emoji"[^>]+alt="(.*?)"+[^>]+>~i',
+                        '$1',
+                        $content
+                );
+
+                return wp_slash( $content ); // Expect slashed.
+        }
+
+        /**
          * Register pre-defs for the forums.
          */