Changeset 12341

Timestamp:

12/16/2022 04:55:38 AM (21 months 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.
      */